Skip to content
This repository was archived by the owner on Oct 7, 2020. It is now read-only.

WIP: test of the operator check for e2e#404

Closed
sdake wants to merge 4 commits intoistio:masterfrom
sdake:gatetest4
Closed

WIP: test of the operator check for e2e#404
sdake wants to merge 4 commits intoistio:masterfrom
sdake:gatetest4

Conversation

@sdake
Copy link
Copy Markdown
Member

@sdake sdake commented Oct 17, 2019

No description provided.

@sdake sdake requested a review from a team as a code owner October 17, 2019 16:11
@istio-testing istio-testing added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Oct 17, 2019
@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 Oct 17, 2019
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 17, 2019
@sdake
Copy link
Copy Markdown
Member Author

sdake commented Oct 17, 2019

/test e2e_operator

@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 17, 2019
@sdake
Copy link
Copy Markdown
Member Author

sdake commented Oct 17, 2019

/test e2e_operator

@istio-testing istio-testing added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 17, 2019
@istio-testing
Copy link
Copy Markdown

istio-testing commented Oct 17, 2019

@sdake: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
gencheck_operator ad0d54f link /test gencheck_operator
e2e_operator ad0d54f link /test e2e_operator
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.

@ostromart
Copy link
Copy Markdown
Contributor

ostromart commented Oct 21, 2019

Hi @sdake, will you be able to make progress on this? Otherwise we should find another owner since it's quite critical.

@irisdingbj irisdingbj mentioned this pull request Oct 24, 2019
Merged
@sdake
Copy link
Copy Markdown
Member Author

sdake commented Oct 24, 2019

@ostromart looks like @irisdingbj will take this over. Note my original work was passing in my local environment and also prow. There is some minor issue - possibly related to the usage of metallb - but possibly something else.

I'll take a look today after my morning meetings finish up.

Cheers
-steve

@sdake
Copy link
Copy Markdown
Member Author

sdake commented Oct 29, 2019

subsumed by work from @irisdingbj .

@sdake sdake closed this Oct 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sdake @istio-testing @ostromart @googlebot