Skip to content

must gather attribute fix #95591

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

must gather attribute fix #95591

wants to merge 1 commit into from

Conversation

shdeshpa07
Copy link
Contributor

@shdeshpa07 shdeshpa07 commented Jul 3, 2025 •
edited
Loading

Jira

this PR fixes an attribute error for must-gather for OADP 1.5.0 on OCP 4.19

Version

  • OCP 4.19, 4.20

Preview

QE Review

  • QE has approved this change.

@shdeshpa07
must gather attribute fix
19facfb 
Signed-off-by: Shruti Deshpande <shdeshpa@redhat.com>
@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 3, 2025
@shdeshpa07
Copy link
Contributor Author

/label OADP

@openshift-ci openshift-ci bot added the OADP Label for all OADP PRs label Jul 3, 2025
@ocpdocs-previewbot
Copy link

🤖 Thu Jul 03 11:59:45 - Prow CI generated the docs preview:

https://95591--ocpdocs-pr.netlify.app/openshift-enterprise/latest/backup_and_restore/application_backup_and_restore/troubleshooting/using-the-must-gather-tool.html

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Jul 3, 2025

@shdeshpa07: 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-sigs/prow repository. I understand the commands that are listed here.

stillalearner
stillalearner approved these changes Jul 9, 2025
View reviewed changes
Copy link

@stillalearner stillalearner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@shdeshpa07
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Jul 10, 2025
@xenolinux xenolinux added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Jul 10, 2025
@xenolinux xenolinux added this to the Continuous Release milestone Jul 10, 2025
xenolinux
xenolinux reviewed Jul 10, 2025
View reviewed changes
Copy link
Contributor

@xenolinux xenolinux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xenolinux xenolinux added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Jul 10, 2025
@shdeshpa07
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xenolinux xenolinux xenolinux left review comments

@stillalearner stillalearner stillalearner approved these changes

Assignees
No one assigned
Labels
merge-review-needed Signifies that the merge review team needs to review this PR OADP Label for all OADP PRs peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
Continuous Release
Development

Successfully merging this pull request may close these issues.
4 participants
@shdeshpa07 @ocpdocs-previewbot @stillalearner @xenolinux