Closed oshoval closed 1 year ago
I don't see any reason not to update the tag. Only the justification in the PR description is off - there is no reason to align with CNAO, as CNAO is unrelated to this project at this point. CNAO doesn't deploy knmstate anymore.
Right thanks updated
@oshoval: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-kubernetes-nmstate-e2e-handler-k8s-future | ee6cbe8a5e4643942ab30df6ed3a3ae063d63640 | link | false | /test pull-kubernetes-nmstate-e2e-handler-k8s-future |
not due to this PR right ? (i saw this lane really flaky in the past but not sure what is the current status and which tests are flaky)
https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/nmstate_kubernetes-nmstate/1130/pull-kubernetes-nmstate-e2e-handler-k8s-future/1592077109033963520 not due to this PR right ? (i saw this lane really flaky in the past but not sure what is the current status and which tests are flaky)
Yes, this is not related /approve /lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rhrazdil
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Thanks Radim
Is this a BUG FIX or a FEATURE ?: /kind enhancement
What this PR does / why we need it: Bump kubevirtci to consume a newer version
Special notes for your reviewer:
Release note: