Update required contexts for BMO #630
Conversation
The BMO e2e tests have now been required for PRs in BMO for a couple of weeks already. They test more extensively and efficiently than the old integration test so I suggest we make it optional. Signed-off-by: Lennart Jern <lennart.jern@est.tech>
metal3-io-bot
added
the
size/XS
lentzi90
force-pushed
the
lentzi90/bmo-required-contexts
branch
from
dbe3923 to
b8d3164
Compare
|
LGTM |
metal3-io-bot
added
the
approved
|
/cc @Rozzii @smoshiur1237 |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: smoshiur1237, tuminoid 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 |
|
/override test-ubuntu-integration-main |
|
@smoshiur1237: smoshiur1237 unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file. 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-ubuntu-integration-main |
|
@tuminoid: Overrode contexts on behalf of tuminoid: 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. |
|
@lentzi90: Updated the
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. |
Removed on GH side too. |
The BMO e2e tests have now been required for PRs in BMO for a couple of weeks already. They test more extensively and efficiently than the old integration test so I suggest we make it optional.