-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update to v1.30 #333
Merged
Merged
Update to v1.30 #333
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
11ef0b0
Update envoy dependency to v1.30.1
ashishb-solo a3b8fd2
Change repository back to upstream envoy
ashishb-solo 45338db
Remove wavm
ashishb-solo d246825
Add some compile flags to get CEL to compile
ashishb-solo e62caf8
Get matchers working
ashishb-solo 330bbf7
Fix a few flags that were moved
ashishb-solo f3bc648
Get HTTP transformation to compile
ashishb-solo 771fe14
Fix some compile errors
ashishb-solo 47a056c
addWatch callback must now return absl::Status
ashishb-solo 093698a
Fix type signature of createProtocolOptionsConfig
ashishb-solo aceeeb6
Add changelog
ashishb-solo 31c242d
Update extensions_build_config.bzl
ashishb-solo b9c1149
Move changelog
ashishb-solo 4f0e904
Get inja_transformer_test.cc to compile
ashishb-solo 0058721
Fix a broken test
ashishb-solo 1dbb2c4
Add router_check tool
ashishb-solo d7ba911
Merge branch 'master' into update-to-v1.30
ashishb-solo 1de6578
Update bazel version
ashishb-solo 07b4004
do_ci.sh debug flags
ashishb-solo 00a16a6
Update envoy-build-ubuntu image
ashishb-solo e0a5027
Remove debug lines from do_ci.sh
ashishb-solo 7dbd7cd
Remove envoy.string_matcher.lua
ashishb-solo 80ab7da
Remove envoy.tracers.opentelemetry.samplers.dynatrace from bazel/exte…
ashishb-solo fd1628f
Update changelog/v1.30.1-patch1/update-to-upstream-envoy-v1.30.yaml
ashishb-solo f91e837
Update source/extensions/filters/http/aws_lambda/config.cc
ashishb-solo 5b43f5a
Revert "Update source/extensions/filters/http/aws_lambda/config.cc"
ashishb-solo 02afc7e
Fix one more review comment
ashishb-solo a706ea8
Revert `fmt::vformat` change in inja_transformer_test.cc
ashishb-solo 831067b
Add comments explaining why `//test/tools` exist
ashishb-solo 08d1401
CHange auto variables to `std::string`
ashishb-solo dea2369
Change `fmt::vformat` back to `fmt::format`
ashishb-solo 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
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 |
---|---|---|
@@ -1 +1 @@ | ||
6.3.2 | ||
6.5.0 |
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
File renamed without changes.
File renamed without changes.
10 changes: 10 additions & 0 deletions
10
changelog/v1.30.1-patch1/update-to-upstream-envoy-v1.30.yaml
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,10 @@ | ||
changelog: | ||
- type: DEPENDENCY_BUMP | ||
dependencyOwner: envoyproxy | ||
dependencyRepo: envoy | ||
dependencyTag: v1.30.1 | ||
resolvesIssue: false | ||
description: >- | ||
Update Envoy to v1.30.1 and switch back off our fork | ||
issueLink: https://github.com/solo-io/envoy-gloo-ee/issues/768 | ||
|
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 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 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
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.
Do we still need to do this if engine is now included by default in 1.30?
It sounds like we are cruising for a bruising in upcoming releases if we rely on this
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.
i don't believe
std::regex
is present in upstream envoy. i see a few references to that class, but no functions that are publicly exported that we can use for usingstd:regex
. i could be wrong here though, let me know of a way that we can use it