Skip to content

Conversation

istio-testing
Copy link
Collaborator

Generated by Automator - 2022-07-18T19:56:46+00:00

@istio-testing istio-testing added auto-merge Set this label on a PR to auto-merge it on success of presubmit tests release-notes-none Indicates a PR that does not require release notes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 18, 2022
@ericvn
Copy link
Contributor

ericvn commented Jul 19, 2022

/test unit-tests_istio

@istio-testing
Copy link
Collaborator Author

@istio-testing: 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
unit-tests_istio 21089a1 link true /test unit-tests_istio

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.

@ericvn
Copy link
Contributor

ericvn commented Jul 19, 2022

The changes in istio/api#1997 will at least require a change in the TestAnalyzers/misannoted test. I'm not sure of the race failure.

@ericvn
Copy link
Contributor

ericvn commented Jul 19, 2022

#39998 includes the test update.

@howardjohn
Copy link
Member

race is a flake, already being look into. Ill fix the test

@ericvn
Copy link
Contributor

ericvn commented Jul 20, 2022

Updated as part of #40000

@ericvn ericvn closed this Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Set this label on a PR to auto-merge it on success of presubmit tests release-notes-none Indicates a PR that does not require release notes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants