-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🌱 Update e2e integration triggers #1582
🌱 Update e2e integration triggers #1582
Conversation
/test metal3-ubuntu-e2e-integration-main |
@mboukhalfa: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
In response to this:
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. |
243d3d8
to
1f12d2a
Compare
/test metal3-ubuntu-e2e-integration-test-main |
Should the GH side required tests be updated for CAPM3 now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kashifest The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Yes I think |
OK, let's then run them here, so they appear as valid choices. /test ? |
@tuminoid: The following commands are available to trigger required jobs:
The following commands are available to trigger optional jobs:
Use
In response to this:
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. |
/test metal3-centos-e2e-integration-test-main I don't see anything matching |
Also, we need a Prow job to update required tests in project-infra. /hold |
there is : metal3-ubuntu-e2e-integration-test-main |
/test metal3-ubuntu-e2e-integration-test-main |
15dc218
to
3b5a538
Compare
/test metal3-ubuntu-e2e-integration-test-main |
3 similar comments
/test metal3-ubuntu-e2e-integration-test-main |
/test metal3-ubuntu-e2e-integration-test-main |
/test metal3-ubuntu-e2e-integration-test-main |
3b5a538
to
eabfa68
Compare
/test metal3-ubuntu-e2e-integration-test-main |
/test metal3-ubuntu-e2e-integration-test-main 😯 |
Whats happening here, I can see I approved it but now you started testing with something new ? is this not ready yet |
/test metal3-ubuntu-e2e-integration-test-main |
1 similar comment
/test metal3-ubuntu-e2e-integration-test-main |
@mboukhalfa: The following test failed, say
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. |
eabfa68
to
1f12d2a
Compare
Signed-off-by: Mohammed Boukhalfa <[email protected]>
1f12d2a
to
bd7bef3
Compare
PR to make prow required metal3-io/project-infra#698 |
/cherry-pick release-1.6 |
@mboukhalfa: once the present PR merges, I will cherry-pick it on top of release-1.6 in a new PR and assign it to you. In response to this:
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. |
/override test-centos-e2e-integration-main |
@mboukhalfa: Overrode contexts on behalf of mboukhalfa: test-centos-e2e-integration-main, test-ubuntu-integration-main In response to this:
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. |
/test ? |
@mboukhalfa: The following commands are available to trigger required jobs:
The following commands are available to trigger optional jobs:
Use
In response to this:
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. |
/test metal3-ubuntu-e2e-integration-test-main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold cancel |
@mboukhalfa: #1582 failed to apply on top of branch "release-1.6":
In response to this:
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. |
Updating readme after migrating e2e integration to prow jenkins operator