Skip to content

Comments

[os_must_gather] Continue getting pod resources when metrics available#3694

Open
danpawlik wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:add-info-must-gather
Open

[os_must_gather] Continue getting pod resources when metrics available#3694
danpawlik wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:add-info-must-gather

Conversation

@danpawlik
Copy link
Contributor

On some deployments especially on CRC, where we don't enable openshift-monitoring, the log files are empty.
In that case, those tasks can be just skipped.

@danpawlik danpawlik requested review from a team and elfiesmelfie February 18, 2026 13:19
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 18, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign akrog for approval. For more information see the Code Review Process.

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

Details 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

On some deployments especially on CRC, where we don't enable
openshift-monitoring, the log files are empty.
In that case, those tasks can be just skipped.

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@danpawlik danpawlik force-pushed the add-info-must-gather branch from 24ead21 to 3b426f2 Compare February 19, 2026 10:21
elfiesmelfie added a commit to elfiesmelfie/telemetry-operator that referenced this pull request Feb 19, 2026
Copy link
Contributor

@elfiesmelfie elfiesmelfie left a comment

Choose a reason for hiding this comment

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

I'm running this with one of the telemetry-operator jobs, which needs openshift-monitoring enabled.

openstack-k8s-operators/telemetry-operator#849

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants