Closed flavianmissi closed 3 months ago
@flavianmissi: This pull request references Jira Issue OCPBUGS-31848, 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.
we want this to be pre-merge tested. holding until qe-approved. /hold
Thanks for the patch. Before pre-merge test, I'm planning to complete review on Monday. Feel free to ping me, if I forget.
test failure looks unrelated. /test e2e-agnostic-ovn-cmd
Thanks /lgtm
/jira refresh
@flavianmissi: This pull request references Jira Issue OCPBUGS-31848, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @zhouying7780
/label qe-approved
@flavianmissi: This pull request references Jira Issue OCPBUGS-31848, which is valid.
Requesting review from QA contact: /cc @zhouying7780
Thank you /lgtm /retest /hold until e2e-metal-ipi-ovn-ipv6 is green as it is only optional job
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ardaguclu, flavianmissi
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
e2e-metal-ipi-ovn-ipv6 is green /hold cancel
@flavianmissi: all tests passed!
Full PR test history. Your PR dashboard.
@flavianmissi: Jira Issue OCPBUGS-31848: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-31848 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
This PR has been included in build openshift-enterprise-cli-container-v4.17.0-202406111641.p0.g7b485f7.assembly.stream.el9 for distgit openshift-enterprise-cli. All builds following this will include this PR.
instead, set the cluster role binding owner reference to the temporary must-gather namespace. when the namespace is deleted, kubernetes handles the deletion of the cluster role binding for us.
in cases where must-gather runs ends in failure, this change ensures that once users manually delete the namespace, the cluster role binding is also deleted.