Skip to content

OBSDOCS-1524 COO must-gather #88594

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

OBSDOCS-1524 COO must-gather #88594

wants to merge 1 commit into from

Conversation

gabriel-rh
Copy link
Contributor

@gabriel-rh gabriel-rh commented Feb 13, 2025 •
edited
Loading

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 13, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Feb 13, 2025 •
edited
Loading

🤖 Mon Feb 17 09:43:32 - Prow CI generated the docs preview:

https://88594--ocpdocs-pr.netlify.app/openshift-dedicated/latest/support/gathering-cluster-data.html
https://88594--ocpdocs-pr.netlify.app/openshift-enterprise/latest/observability/cluster_observability_operator/cluster-observability-operator-overview.html
https://88594--ocpdocs-pr.netlify.app/openshift-enterprise/latest/support/gathering-cluster-data.html
https://88594--ocpdocs-pr.netlify.app/openshift-rosa-hcp/latest/support/gathering-cluster-data.html
https://88594--ocpdocs-pr.netlify.app/openshift-rosa/latest/support/gathering-cluster-data.html

@gabriel-rh gabriel-rh added coo Cluster Observability Operator peer-review-needed Signifies that the peer review team needs to review this PR labels Feb 13, 2025
@lpettyjo lpettyjo 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 Feb 14, 2025
@lpettyjo lpettyjo self-requested a review February 14, 2025 00:13
@lpettyjo lpettyjo added this to the Continuous Release milestone Feb 14, 2025
lpettyjo
lpettyjo reviewed Feb 14, 2025
View reviewed changes
Copy link
Contributor

@lpettyjo lpettyjo left a comment

Choose a reason for hiding this comment

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

LGTM!

@lpettyjo lpettyjo 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 Feb 14, 2025
lihongyan1
lihongyan1 reviewed Feb 16, 2025
View reviewed changes
modules/coo-must-gather.adoc Outdated
----
$ oc adm must-gather
--image-stream=openshift/must-gather \
--image=quay.io/redhat-user-workloads/cluster-observabilit-tenant/cluster-observability-operator-container
Copy link

Choose a reason for hiding this comment

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

Image name should be
registry.redhat.io/cluster-observability-operator/cluster-observability-rhel8-operator

@lihongyan1
Copy link

The feature verification failed, will slip into next version for bug COO-584

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Feb 17, 2025

@gabriel-rh: 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.

@gabriel-rh
Copy link
Contributor Author

The feature verification failed, will slip into next version for bug COO-584

I have removed this from COO 1.0 docs based on the above - will restore for COO 1.1

@bergerhoffer
Copy link
Contributor

The branch/enterprise-4.19 label has been added to this PR.

This is because your PR targets the main branch and is labeled for enterprise-4.18. And any PR going into main must also target the latest version branch (enterprise-4.19).

If the update in your PR does NOT apply to version 4.19 onward, please re-target this PR to go directly into the appropriate version branch or branches (enterprise-4.x) instead of main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lihongyan1 lihongyan1 lihongyan1 left review comments

@lpettyjo lpettyjo lpettyjo left review comments

Assignees
No one assigned
Labels
branch/enterprise-4.12 branch/enterprise-4.13 branch/enterprise-4.14 branch/enterprise-4.15 branch/enterprise-4.16 branch/enterprise-4.17 branch/enterprise-4.18 branch/enterprise-4.19 coo Cluster Observability Operator peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
Continuous Release
Development

Successfully merging this pull request may close these issues.
5 participants
@gabriel-rh @ocpdocs-previewbot @lihongyan1 @bergerhoffer @lpettyjo