Skip to content

wasm: fix root_context_id that was incorrectly cached across threads. (#15016)#15078

Merged
lizan merged 1 commit intoenvoyproxy:release/v1.17from
PiotrSikora:wasm-fix_cached_root_context_id-envoy_v1.17
Feb 19, 2021
Merged

wasm: fix root_context_id that was incorrectly cached across threads. (#15016)#15078
lizan merged 1 commit intoenvoyproxy:release/v1.17from
PiotrSikora:wasm-fix_cached_root_context_id-envoy_v1.17

Conversation

@PiotrSikora
Copy link
Copy Markdown
Contributor

Fixes proxy-wasm/proxy-wasm-cpp-host#125.

Signed-off-by: Piotr Sikora [email protected]

@PiotrSikora PiotrSikora requested a review from lizan as a code owner February 17, 2021 19:05
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Feb 17, 2021
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #15078 was opened by PiotrSikora.

see: more, trace.

@moderation
Copy link
Copy Markdown
Contributor

Was this PR replaced by #15016 (comment) and therefore doesn't require deps approval?

@PiotrSikora
Copy link
Copy Markdown
Contributor Author

Was this PR replaced by #15016 (comment) and therefore doesn't require deps approval?

No, this is a backport of #15016 to release/v1.17 branch, and it looks that it needs an approval from @envoyproxy/dependency-shepherds.

@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Feb 18, 2021
@lizan lizan merged commit a7d9915 into envoyproxy:release/v1.17 Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants