-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Converting envoy configs to V2 #2957
Merged
Merged
Changes from 43 commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
ad4448d
This is initial work on migrating migrating envoy configs to v2.
d9f90f3
Updated yaml for envoy_service_to_service.template.json
de34d96
Fixed formatting issues for envoy_front_proxy
e4d8b82
fixed formatting for envoy_double_proxy
b123e1c
Fixed formatting for envoy_serive_to_service
86c86de
Added yaml file for access_log_helper
fc26c76
Fixed function name.(sorry for bad name)
70b6292
Updated double_proxy v2 template
5580ebd
Removed all old json files
d8b984b
Added converted google_com_proxy
a07ff79
Fixed extra spaces
05715ae
Added routing helper and router v2 template files.
fab0bee
Updated configgen with converted yaml templates
93186e1
Added intial work on converting envoy_service_to_service
4ebaf40
Minor changes to configgen.py until adding yaml as external dependency
2e22cc7
Added pyyaml external dependency
b35603e
Cleaned up envoy_front_config_v2.template
4f055dc
added envoy_service_service v2 template
e1bc087
fix for ci build and updated cds_config for service_to_service template
eff16f9
fix for ci build failure
aae77a5
Added tls context for front_proxy and double_proxy templates
e047376
Fixed build and updated envoy_front_proxy.v2.template
2fbdc9d
Adding router_helper config to envoy_front_proxy
e9093fc
Updated double proxy v2 template
2c9bb81
Updated initial work on service_to_service template
0d37bda
Added listener to service_to_service template
b346977
Updated ingress_listener function in service_to_service template.
c848ad4
Updated listeners in service_to_service template v2
d5e257c
Updates to envoy_front_proxy
4aceb54
updated merge conflict in new yaml and deleted json file for envoy_fr…
9b9f70a
Updates to envoy_double_proxy
e08f7d6
Changes to service_to_service template
f0b61d3
Removing old configs from config_validation server_test
d6284d8
Updated cluster configs in routing helper and internal_cluster_defini…
a3e5c5c
Updates to service_to_service
aaffc29
merge
44914c6
Old config files for google_com_proxy are not being
5ad8bbd
Merge remote-tracking branch 'upstream/master'
95f27b1
updated changes related to e6bf9f896c3ecfb1875a5bff53244ebfcd101a5a
affa303
Added comments from old template to double_proxy
96c1c64
Updates to front_proxy template
8f6ea92
updates after review.
840b101
Merge branch 'master' into master
srikailash b8e03c7
Updates from comparing yaml files with tool outputs
b9453ad
Updates
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
py_library( | ||
name = "yaml", | ||
srcs = glob(["lib/yaml/**/*.py"]), | ||
imports = ["lib"], | ||
visibility = ["//visibility:public"], | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
{% macro ingress_sampled_log() -%} | ||
format: "[%START_TIME%] \"%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH):256% %PROTOCOL%\" %RESPONSE_CODE% %RESPONSE_FLAGS% %BYTES_RECEIVED% %BYTES_SENT% %DURATION% %RESP(X-ENVOY-UPSTREAM-SERVICE-TIME)% \"%REQ(X-FORWARDED-FOR)%\" \"%REQ(USER-AGENT)%\" \"%REQ(X-REQUEST-ID)%\" \"%REQ(:AUTHORITY)%\"\n" | ||
{% endmacro %} | ||
|
||
{% macro ingress_full() -%} | ||
format: "[%START_TIME%] \"%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH)% %PROTOCOL%\" %RESPONSE_CODE% %RESPONSE_FLAGS% %BYTES_RECEIVED% %BYTES_SENT% %DURATION% %RESP(X-ENVOY-UPSTREAM-SERVICE-TIME)% \"%REQ(X-FORWARDED-FOR)%\" \"%REQ(USER-AGENT)%\" \"%REQ(X-REQUEST-ID)%\" \"%REQ(:AUTHORITY)%\"\n" | ||
{% endmacro %} | ||
|
||
{% macro egress_error_log() -%} | ||
format: "[%START_TIME%] \"%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH):256% %PROTOCOL%\" %RESPONSE_CODE% %RESPONSE_FLAGS% %BYTES_RECEIVED% %BYTES_SENT% %DURATION% %RESP(X-ENVOY-UPSTREAM-SERVICE-TIME)% \"%REQ(X-FORWARDED-FOR)%\" \"%REQ(USER-AGENT)%\" \"%REQ(X-REQUEST-ID)%\" \"%REQ(:AUTHORITY)%\" \"%UPSTREAM_HOST%\"\n" | ||
{% endmacro %} | ||
|
||
{% macro egress_error_amazon_service() -%} | ||
format: "[%START_TIME%] \"%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH):256% %PROTOCOL%\" %RESPONSE_CODE% %RESPONSE_FLAGS% %BYTES_RECEIVED% %BYTES_SENT% %DURATION% %RESP(X-ENVOY-UPSTREAM-SERVICE-TIME)% \"%REQ(X-FORWARDED-FOR)%\" \"%REQ(USER-AGENT)%\" \"%REQ(X-REQUEST-ID)%\" \"%REQ(:AUTHORITY)%\" \"%UPSTREAM_HOST%\" \"%RESP(X-AMZN-RequestId)%\"\n" | ||
{% endmacro %} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: inconsistent spacing. let's run
autopep8
over the python files to get a consistent format