Merged
Conversation
Signed-off-by: wbpcode <comems@msn.com>
* Fix URL links pointing to sample terminating connect configs * typo Signed-off-by: Dean Liu <dliu@lyft.com>
Signed-off-by: Long Dai <long0dai@foxmail.com>
Signed-off-by: Mandar U Jog <mjog@google.com>
Fix typo in the envoy document Signed-off-by: Le Yao <le.yao@intel.com>
…r doing itself (#17227) Signed-off-by: He Jie Xu <hejie.xu@intel.com>
Signed-off-by: Ryan Sutton <ryan.sutton@volunteers.acasi.info> Co-authored-by: Lizan Zhou <lizan@tetrate.io>
When getting a rebalancer by address and a wild card match is being used, the first match in the list is returned. However, if there are listeners with addresses "0.0.0.0" and "::" then the first active listener found will be used, irrespective of the IP family type. Change the behavior to always return the listener of the same IP family type as the rebalancer. Fixes #16804 Co-authored-by: yingchun.cai@volunteers.acasi.info Signed-off-by: Jacob Delgado <jacob.delgado@volunteers.acasi.info>
* build(deps): bump sphinx-copybutton from 0.3.3 to 0.4.0 in /tools/docs Bumps [sphinx-copybutton](https://github.com/executablebooks/sphinx-copybutton) from 0.3.3 to 0.4.0. - [Release notes](https://github.com/executablebooks/sphinx-copybutton/releases) - [Changelog](https://github.com/executablebooks/sphinx-copybutton/blob/master/CHANGELOG.md) - [Commits](executablebooks/sphinx-copybutton@v0.3.3...v0.4.0) --- updated-dependencies: - dependency-name: sphinx-copybutton dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump packaging from 20.9 to 21.0 in /tools/docs Bumps [packaging](https://github.com/pypa/packaging) from 20.9 to 21.0. - [Release notes](https://github.com/pypa/packaging/releases) - [Changelog](https://github.com/pypa/packaging/blob/main/CHANGELOG.rst) - [Commits](pypa/packaging@20.9...21.0) --- updated-dependencies: - dependency-name: packaging dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump packaging from 20.9 to 21.0 in /tools/testing Bumps [packaging](https://github.com/pypa/packaging) from 20.9 to 21.0. - [Release notes](https://github.com/pypa/packaging/releases) - [Changelog](https://github.com/pypa/packaging/blob/main/CHANGELOG.rst) - [Commits](pypa/packaging@20.9...21.0) --- updated-dependencies: - dependency-name: packaging dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump packaging from 20.9 to 21.0 in /tools/dependency Bumps [packaging](https://github.com/pypa/packaging) from 20.9 to 21.0. - [Release notes](https://github.com/pypa/packaging/releases) - [Changelog](https://github.com/pypa/packaging/blob/main/CHANGELOG.rst) - [Commits](pypa/packaging@20.9...21.0) --- updated-dependencies: - dependency-name: packaging dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump urllib3 from 1.26.5 to 1.26.6 in /tools/docs Bumps [urllib3](https://github.com/urllib3/urllib3) from 1.26.5 to 1.26.6. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/1.26.6/CHANGES.rst) - [Commits](urllib3/urllib3@1.26.5...1.26.6) --- updated-dependencies: - dependency-name: urllib3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump urllib3 in /.github/actions/pr_notifier Bumps [urllib3](https://github.com/urllib3/urllib3) from 1.26.5 to 1.26.6. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/1.26.6/CHANGES.rst) - [Commits](urllib3/urllib3@1.26.5...1.26.6) --- updated-dependencies: - dependency-name: urllib3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump urllib3 from 1.26.5 to 1.26.6 in /tools/dependency Bumps [urllib3](https://github.com/urllib3/urllib3) from 1.26.5 to 1.26.6. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/1.26.6/CHANGES.rst) - [Commits](urllib3/urllib3@1.26.5...1.26.6) --- updated-dependencies: - dependency-name: urllib3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump urllib3 in /tools/deprecate_version Bumps [urllib3](https://github.com/urllib3/urllib3) from 1.26.5 to 1.26.6. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/1.26.6/CHANGES.rst) - [Commits](urllib3/urllib3@1.26.5...1.26.6) --- updated-dependencies: - dependency-name: urllib3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> * build(deps): bump docutils from 0.16 to 0.17.1 in /tools/docs Bumps [docutils](http://docutils.sourceforge.net/) from 0.16 to 0.17.1. --- updated-dependencies: - dependency-name: docutils dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Ryan Northey <ryan@synca.io> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Matt Klein <mklein@lyft.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
JIT Mode can be open now. Finish the optimization of WAMR for Envoy Risk Level: Low Co-authored-by: Liang He <liang.he@intel.com> Signed-off-by: Le Yao <le.yao@intel.com>
Sub-PR of #16049. Check #16049 get more background information. This PR is last sub-PR of #16049. It simply replaces `Http::RequestHeaderMap` with general `Tracing::TraceContext` in all tracer drivers implementations. Check #16793 get more info about `Tracing::TraceContext`. After this PR, the main body of the entire general tracing system is completed. Next, we can try to use the new general tracing system in dubbo and thrift. Commit Message: replace RequestHeaderMap in tracers with general TraceContext Risk Level: Low. Testing: N/A. Docs Changes: N/A. Release Notes: N/A. Signed-off-by: wbpcode <comems@msn.com>
Additional Description: None. Risk Level: Low. Testing: Manual verification. Docs CI check. Docs Changes: Fixing ref link. Release Notes: None. Platform Specific Features: None. Signed-off-by: James Peach <jpeach@apache.org>
…#16770) Signed-off-by: Shikugawa <rei@tetrate.io>
Signed-off-by: He Jie Xu <hejie.xu@intel.com>
Signed-off-by: Paul Gallagher <pgal@google.com>
I faced a crash while opening config_dump.
[2021-06-24 14:48:48.771][934895][critical][assert] [source/common/protobuf/utility.cc:778] assert failure: type_url_field_descriptor != nullptr && value_field_descriptor != nullptr && reflection->HasField(*message, type_url_field_descriptor).
It is because we passed opaque message without any properties as follows.
- name: envoy.filters.http.router
typed_config: {}
We can avoid this crash with providing as follows
- name: envoy.filters.http.router
typed_config:
"@type": type.googleapis.com/envoy.extensions.filters.http.router.v3.Router
But, initial way should be accepted even if config_dump because Envoy accepts it while bootstrap.
This PR fixes to accept relaxed opaque type and avoid crash with it.
Additional Description:
Risk Level: Low
Testing: Unit
Signed-off-by: Shikugawa <rei@tetrate.io>
…t socket (#17163) Remove cluster's and listener's tls_context since we are using transport socket's tls_context/ Refactor the sslSocketTest : centralize the server configuration in one place --configureServerAndExpiredClientCertificate function. Note: The test could be further refactored by leveraging this function I modified, but I feel that is not worth the large amount of effort and current code also provides a bit flexibility of configuration (e.g. specifies various config like cert_hash, cert_spki in place) Move createProtocolTestOptions to unnamed namespace. Even though superiority of unnamed namespace over static is more applied to user-defined types rather than variables and functions (i.e. static no longer deprecated in standard and should do the same thing for latter two), it is still good to keep it in the unnamed namespace like other internal helper functions . Also, unnamed namespace should be encouraged for such usage in general Risk Level: Low Testing: Local tests and CI run (All tests passed) Signed-off-by: Tianyu Xia <tyxia@google.com>
This reverts commit 27b4f08, fixing the merge conflicts with the original change Signed-off-by: Snow Pettersen <snowp@lyft.com>
Split out subscription state out of xDS mux unification PR (#15473). Made base subscription state class a template. Updated existing delta state tests to work with both legacy and new implementations. Risk Level: low, the code is not being used atm Testing: updated existing tests, added new ones Signed-off-by: Dmitri Dolguikh <ddolguik@redhat.com>
…o avoid having to do toString() mapping and caching (#17018) Commit Message: Optimizes a useful special-case where we exclude stats by prefix. Ordinarily we must string-match against stringified StatName, and the stringification is both slow and requires a global symbol table lock. To avoid that becoming a performance/contention bottleneck, we keep excluded StatNames in the ThreadLocalStore caches, which can consume a lot of memory at scale. It consumes less than the stats, but it can still be significant. To avoid this problem when matching with a full token prefix, we can do the exclusion comparison without converting to a string. This is fast and doesn't require taking any locks, so we don't need to save excluded StatNames in a map. This is both faster and provides a significant reduction in memory. Additional Description: Risk Level: medium -- this adds some complexity to the StatsMatcher implementation Testing: //test/... plus new performance tests, and targeted unit tests to hit corner cases Docs Changes: n/a Release Notes: n/a Platform Specific Features: n/a Fixes: #17027 Signed-off-by: Joshua Marantz <jmarantz@google.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
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.
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]