Skip to content

Conversation

@martijneken
Copy link
Contributor

@martijneken martijneken commented Oct 10, 2024

Test proxy-wasm/proxy-wasm-cpp-sdk#173

Works locally:

$ bazelisk test -c opt //test/extensions/...
...
Executed 690 out of 690 tests: 690 tests pass.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link

Hi @martijneken, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #36526 was opened by martijneken.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Oct 10, 2024
@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).
envoyproxy/dependency-shepherds assignee is @RyanTheOptimist

🐱

Caused by: #36526 was opened by martijneken.

see: more, trace.

@martijneken
Copy link
Contributor Author

2 failures, both look transient:

/retest

@martijneken
Copy link
Contributor Author

One more timeout in swift-tests. Try once more.

/retest

@martijneken martijneken changed the title [TESTING/WIP] Check CI for emsdk update in proxy-wasm-cpp-sdk PR#172 [TESTING/WIP] Check CI for emsdk update in proxy-wasm-cpp-sdk PR#173 Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deps Approval required for changes to Envoy's external dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants