Closed neisw closed 1 week ago
@neisw: This pull request explicitly references no jira issue.
/hold
code lgtm, but two possible improvements
optional suggestion (could also do a followup)
/lgtm
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: deads2k, neisw
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test unit
@neisw: 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/e2e-aws-ovn-single-node-upgrade | b1acc0a2495e6cbd75a33e73972b77cdb27ca915 | link | false | /test e2e-aws-ovn-single-node-upgrade |
ci/prow/e2e-aws-ovn-single-node | b1acc0a2495e6cbd75a33e73972b77cdb27ca915 | link | false | /test e2e-aws-ovn-single-node |
ci/prow/e2e-aws-ovn-single-node-serial | b1acc0a2495e6cbd75a33e73972b77cdb27ca915 | link | false | /test e2e-aws-ovn-single-node-serial |
ci/prow/e2e-gcp-csi | b1acc0a2495e6cbd75a33e73972b77cdb27ca915 | link | false | /test e2e-gcp-csi |
ci/prow/e2e-aws-ovn-upgrade | b1acc0a2495e6cbd75a33e73972b77cdb27ca915 | link | false | /test e2e-aws-ovn-upgrade |
Full PR test history. Your PR dashboard.
/hold cancel
/retest-required
Remaining retests: 0 against base HEAD e06689a1ea032ed0dc0cf0b82ca209a3487e8271 and 2 for PR HEAD b1acc0a2495e6cbd75a33e73972b77cdb27ca915 in total
Builds on https://github.com/openshift/origin/pull/28705 to create json output for bq loading