Closed praveenkumar closed 2 months ago
Adding a rationale explaining why we want to do this switch now would be useful.
Added.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cfergeau
The full list of commands accepted by this bot can be found here.
The pull request process is described here
New changes are detected. LGTM label has been removed.
@praveenkumar: The following tests failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/security | 538362e09717e2327eea84236367e41a4f00f905 | link | false | /test security |
ci/prow/integration-crc | 538362e09717e2327eea84236367e41a4f00f905 | link | true | /test integration-crc |
ci/prow/e2e-crc | 538362e09717e2327eea84236367e41a4f00f905 | link | true | /test e2e-crc |
Full PR test history. Your PR dashboard.
:thinking: there is no go toolset for ubi images yet no?NVMD just saw you changed the FROM image