Skip to content

[Do not merge] testing qat with CI#5424

Closed
soulxu wants to merge 3 commits intoistio:masterfrom
soulxu:test_qat
Closed

[Do not merge] testing qat with CI#5424
soulxu wants to merge 3 commits intoistio:masterfrom
soulxu:test_qat

Conversation

@soulxu
Copy link
Copy Markdown
Contributor

@soulxu soulxu commented Mar 27, 2024

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
@soulxu soulxu requested a review from a team March 27, 2024 00:02
@istio-policy-bot
Copy link
Copy Markdown

😊 Welcome @soulxu! This is either your first contribution to the Istio proxy repo, or it's been
a while since you've been here.

You can learn more about the Istio working groups, Code of Conduct, and contribution guidelines
by referring to Contributing to Istio.

Thanks for contributing!

Courtesy of your friendly welcome wagon.

@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 27, 2024
@istio-testing
Copy link
Copy Markdown
Collaborator

Hi @soulxu. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@soulxu
Copy link
Copy Markdown
Contributor Author

soulxu commented Mar 27, 2024

ok, I'm not a member yet.

@zirain would you like to help me trigger the ci for testing the qat? I can't figure out what happened for the qat through those logs #5411 (comment), it seems all about the wasmvm.

So I'm trying to trigger the ci and check the error log.

Thanks!

@lei-tang
Copy link
Copy Markdown
Contributor

/ok-to-test

@istio-testing istio-testing added ok-to-test Set this label allow normal testing to take place for a PR not submitted by an Istio org member. and removed needs-ok-to-test labels Mar 27, 2024
@istio-testing istio-testing added the needs-rebase Indicates a PR needs to be rebased before being merged label Mar 27, 2024
@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed needs-rebase Indicates a PR needs to be rebased before being merged size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 27, 2024
@istio-testing
Copy link
Copy Markdown
Collaborator

istio-testing commented Mar 27, 2024

@soulxu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
test-arm-arm64_proxy d1397c1 link true /test test-arm-arm64
Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@soulxu
Copy link
Copy Markdown
Contributor Author

soulxu commented Mar 27, 2024

ok, I can ensure what is problem here and working on fix in the Envoy side envoyproxy/envoy#33143

@soulxu soulxu closed this Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Set this label allow normal testing to take place for a PR not submitted by an Istio org member. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants