Skip to content

wasm: update Proxy-Wasm C++ host to the latest (2021-08-02).#17547

Merged
yanavlasov merged 5 commits intoenvoyproxy:mainfrom
mathetake:update-cpp-host
Aug 6, 2021
Merged

wasm: update Proxy-Wasm C++ host to the latest (2021-08-02).#17547
yanavlasov merged 5 commits intoenvoyproxy:mainfrom
mathetake:update-cpp-host

Conversation

@mathetake
Copy link
Copy Markdown
Member

Signed-off-by: Takeshi Yoneda [email protected]

Depends on proxy-wasm/proxy-wasm-cpp-host#183

@repokitteh-read-only
Copy link
Copy Markdown

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #17547 was opened by mathetake.

see: more, trace.

@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: #17547 was opened by mathetake.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Jul 30, 2021
@mathetake
Copy link
Copy Markdown
Member Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17547 (comment) was created by @mathetake.

see: more, trace.

@mathetake
Copy link
Copy Markdown
Member Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17547 (comment) was created by @mathetake.

see: more, trace.

@mathetake mathetake changed the title wasm: update Proxy-Wasm C++ host. wasm: update Proxy-Wasm C++ host to the latest (2021-08-02). Aug 2, 2021
Signed-off-by: Takeshi Yoneda <[email protected]>
@mathetake
Copy link
Copy Markdown
Member Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17547 (comment) was created by @mathetake.

see: more, trace.

@mathetake mathetake marked this pull request as ready for review August 3, 2021 02:01
@mathetake
Copy link
Copy Markdown
Member Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17547 (comment) was created by @mathetake.

see: more, trace.

@mathetake
Copy link
Copy Markdown
Member Author

ping @moderation for dep:)

Copy link
Copy Markdown
Contributor

@PiotrSikora PiotrSikora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

It looks that coverage is broken because of #17564.

@mathetake
Copy link
Copy Markdown
Member Author

made a PR for coverage failure #17573

@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 Aug 3, 2021
alyssawilk pushed a commit that referenced this pull request Aug 3, 2021
seems like common/quic is flaky and started failing on some unrelated branches (e.g. https://dev.azure.com/cncf/envoy/_build/results?buildId=84219&view=logs&j=bbe4b42d-86e6-5e9c-8a0b-fea01d818a24&t=e00c5a13-c6dc-5e9a-6104-69976170e881 on #17547) due to the recent change #17564, so this PR lowers the limit for it to 91.2 according to the number reported ^ and on the main branch.
Signed-off-by: Takeshi Yoneda [email protected]

Signed-off-by: Takeshi Yoneda <[email protected]>
@yanavlasov yanavlasov merged commit 70aa8c8 into envoyproxy:main Aug 6, 2021
@mathetake mathetake deleted the update-cpp-host branch August 8, 2021 00:41
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
seems like common/quic is flaky and started failing on some unrelated branches (e.g. https://dev.azure.com/cncf/envoy/_build/results?buildId=84219&view=logs&j=bbe4b42d-86e6-5e9c-8a0b-fea01d818a24&t=e00c5a13-c6dc-5e9a-6104-69976170e881 on envoyproxy#17547) due to the recent change envoyproxy#17564, so this PR lowers the limit for it to 91.2 according to the number reported ^ and on the main branch.
Signed-off-by: Takeshi Yoneda [email protected]

Signed-off-by: Takeshi Yoneda <[email protected]>
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 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