Skip to content

[release-1.3] Minor egress fixes#5037

Closed
istio-testing wants to merge 2 commits intoistio:release-1.3from
istio-testing:cherry-pick-5025-to-release-1.3
Closed

[release-1.3] Minor egress fixes#5037
istio-testing wants to merge 2 commits intoistio:release-1.3from
istio-testing:cherry-pick-5025-to-release-1.3

Conversation

@istio-testing
Copy link
Copy Markdown
Contributor

This is an automated cherry-pick of #5025

@istio-testing istio-testing requested a review from a team as a code owner September 22, 2019 06:41
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 22, 2019
@googlebot
Copy link
Copy Markdown

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added the cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. label Sep 22, 2019
@istio-testing
Copy link
Copy Markdown
Contributor Author

@istio-testing: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
lint_istio.io_release-1.3 6131ccb link /test lint
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.

@geeknoid
Copy link
Copy Markdown
Contributor

This needs to be hand-merged...

@geeknoid geeknoid closed this Sep 24, 2019
@vadimeisenbergibm
Copy link
Copy Markdown
Contributor

@geeknoid let me do it.

@vadimeisenbergibm
Copy link
Copy Markdown
Contributor

@geeknoid I have submitted a PR #5048.

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

Labels

cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. 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