Skip to content

add bazel build args into test framework and do not fail silently#3025

Closed
bianpengyuan wants to merge 2 commits intoistio:release-1.7from
bianpengyuan:bazel-bin-build-args-1.7
Closed

add bazel build args into test framework and do not fail silently#3025
bianpengyuan wants to merge 2 commits intoistio:release-1.7from
bianpengyuan:bazel-bin-build-args-1.7

Conversation

@bianpengyuan
Copy link
Copy Markdown
Contributor

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:

Release note:

@istio-testing istio-testing added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Sep 25, 2020
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Sep 25, 2020
@istio-testing
Copy link
Copy Markdown
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@istio-testing istio-testing added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 25, 2020
@bianpengyuan
Copy link
Copy Markdown
Contributor Author

/test all

@bianpengyuan
Copy link
Copy Markdown
Contributor Author

/test all

@bianpengyuan bianpengyuan marked this pull request as ready for review September 29, 2020 18:06
@bianpengyuan bianpengyuan requested a review from a team September 29, 2020 18:06
@istio-testing istio-testing removed the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Sep 29, 2020
@Mythra
Copy link
Copy Markdown

Mythra commented Sep 29, 2020

This will be fixed by: #3029 (included this patch as a result). Think we can close this, can you confirm @bianpengyuan ?

@bianpengyuan
Copy link
Copy Markdown
Contributor Author

Ah yes, nvm. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants