Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NO-JIRA: cmd/cluster/core/dump: Gather PodDisruptionBudgets too #3263

Merged
merged 1 commit into from Dec 6, 2023

Conversation

wking
Copy link
Member

@wking wking commented Dec 1, 2023

What this PR does / why we need it:

Pod disruption budgets are important enough to be included in Insights tarballs, and they aren't all that big, so we should pick them up as a standard part of interesting namespaces too. This will help with things like auditing issues where we failed to drain (PDB too strict) or disrupted a core workload (PDB too weak or missing). Similar to openshift/oc@c32fac0143 (openshift/oc#750).

Checklist

  • Subject and description added to both, commit and PR.
  • Relevant issues have been referenced.
  • This change includes docs.
  • This change includes unit tests.

Pod disruption budgets are important enough to be included in Insights
tarballs [1], and they aren't all that big, so we should pick them up
as a standard part of interesting namespaces too.  This will help with
things like auditing issues where we failed to drain (PDB too strict)
or disrupted a core workload (PDB too weak or missing).  Similar to
openshift/oc@c32fac0143 (pkg/cli/admin/inspect/namespace: Gather PDBs
too, 2021-03-03, openshift/oc#750).

[1]: openshift/insights-operator#185
@openshift-ci openshift-ci bot requested review from csrwng and hasueki December 1, 2023 00:19
@openshift-ci openshift-ci bot added area/cli Indicates the PR includes changes for CLI and removed do-not-merge/needs-area labels Dec 1, 2023
@wking
Copy link
Member Author

wking commented Dec 1, 2023

CI looks good.

@wking wking changed the title cmd/cluster/core/dump: Gather PodDisruptionBudgets too NO-JIRA: cmd/cluster/core/dump: Gather PodDisruptionBudgets too Dec 1, 2023
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Dec 1, 2023
@openshift-ci-robot
Copy link

@wking: This pull request explicitly references no jira issue.

In response to this:

What this PR does / why we need it:

Pod disruption budgets are important enough to be included in Insights tarballs, and they aren't all that big, so we should pick them up as a standard part of interesting namespaces too. This will help with things like auditing issues where we failed to drain (PDB too strict) or disrupted a core workload (PDB too weak or missing). Similar to openshift/oc@c32fac0143 (openshift/oc#750).

Checklist

  • Subject and description added to both, commit and PR.
  • Relevant issues have been referenced.
  • This change includes docs.
  • This change includes unit tests.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@wking
Copy link
Member Author

wking commented Dec 1, 2023

I'm going with NO-JIRA for now, but I'm happy to create an OCPBUGS if you think this is worth backporting.

@csrwng
Copy link
Contributor

csrwng commented Dec 4, 2023

/approve

Copy link
Contributor

openshift-ci bot commented Dec 4, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: csrwng, wking

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 4, 2023
@sjenning
Copy link
Contributor

sjenning commented Dec 4, 2023

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 4, 2023
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 7f8fcfb and 2 for PR HEAD ec520e1 in total

@wking
Copy link
Member Author

wking commented Dec 6, 2023

/retest-required

Now that openshift/cluster-config-operator#388 has landed, and possibly fixed the RHEL 8 vs. RHEL 9 dynamic linking issues.

Copy link
Contributor

openshift-ci bot commented Dec 6, 2023

@wking: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-merge-bot openshift-merge-bot bot merged commit 39a0c45 into openshift:main Dec 6, 2023
12 checks passed
@wking wking deleted the dump-pdbs branch December 6, 2023 15:46
@openshift-bot
Copy link

[ART PR BUILD NOTIFIER]

This PR has been included in build ose-hypershift-container-v4.16.0-202312061931.p0.g39a0c45.assembly.stream for distgit hypershift.
All builds following this will include this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/cli Indicates the PR includes changes for CLI jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants