Open cybertron opened 3 weeks ago
@cybertron: This pull request references Jira Issue OCPBUGS-36175, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
/jira refresh
@cybertron: This pull request references Jira Issue OCPBUGS-36175, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @zhaozhanqi
/test e2e-metal-ipi-ovn-ipv6 /test e2e-metal-ipi-ovn-dualstack
/retest-required
None of these templates are used in hypershift so this can't have broken it.
/lgtm
/assign @yuqi-zhang /assign @sinnykumari
Yes, you're right, we should drop it from those as well. New revision coming shortly.
New changes are detected. LGTM label has been removed.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: cybertron, mkowalski Once this PR has been reviewed and has the lgtm label, please ask for approval from sinnykumari. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
/retest-required
@cybertron: 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-vsphere-ovn-zones | 9542711f993cd01403544d1b082cebb6b453de70 | link | false | /test e2e-vsphere-ovn-zones |
ci/prow/e2e-vsphere-ovn-upi | 9542711f993cd01403544d1b082cebb6b453de70 | link | false | /test e2e-vsphere-ovn-upi |
ci/prow/e2e-vsphere-ovn-upi-zones | 9542711f993cd01403544d1b082cebb6b453de70 | link | false | /test e2e-vsphere-ovn-upi-zones |
ci/prow/e2e-aws-ovn-upgrade | 9542711f993cd01403544d1b082cebb6b453de70 | link | true | /test e2e-aws-ovn-upgrade |
Full PR test history. Your PR dashboard.
It turns out we don't need this in some cases and in others we can get by with specific capabilities instead of a fully privileged container. This is more secure because it means even a compromised container won't have as much ability to cause trouble.
- What I did
- How to verify it
- Description for the changelog