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
Member

Signed-off-by: Takeshi Yoneda takeshi@tetrate.io

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

@repokitteh-read-only
Copy link

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

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
Signed-off-by: Takeshi Yoneda <takeshi@tetrate.io>
@mathetake
Copy link
Member Author

/retest

@repokitteh-read-only
Copy link

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

🐱

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

see: more, trace.

@mathetake
Copy link
Member Author

/retest

@repokitteh-read-only
Copy link

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 <takeshi@tetrate.io>
@mathetake
Copy link
Member Author

/retest

@repokitteh-read-only
Copy link

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
Member Author

/retest

@repokitteh-read-only
Copy link

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

🐱

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

see: more, trace.

@mathetake
Copy link
Member Author

ping @moderation for dep:)

Copy link
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
Member Author

made a PR for coverage failure #17573

@moderation
Copy link
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 takeshi@tetrate.io

Signed-off-by: Takeshi Yoneda <takeshi@tetrate.io>
@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 takeshi@tetrate.io

Signed-off-by: Takeshi Yoneda <takeshi@tetrate.io>
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