Skip to content

[os_must_gather] Add new variable in README #3600

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
averdagu wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[os_must_gather] Add new variable in README #3600

averdagu wants to merge 1 commit into from
+1 −0

Conversation

@averdagu
Copy link
Contributor

@averdagu averdagu commented Jan 13, 2026

Commit 6cd893b added new variable called cifmw_os_must_gather_volume_percentage, adding it on the README.

@averdagu
[os_must_gather] Add new variable in README
4d1734b 
Commit 6cd893b added new variable
called cifmw_os_must_gather_volume_percentage, adding it on the
README.

Signed-off-by: Arnau Verdaguer <[email protected]>
@averdagu
Copy link
Contributor Author

averdagu commented Jan 13, 2026

Follow up patch for PR #3595 where I forgot to add the new variable into the readme

fmount
fmount approved these changes Jan 13, 2026
View reviewed changes
Copy link
Contributor

@fmount fmount left a comment

Choose a reason for hiding this comment

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

/approve
/lgtm

Thank you!

@openshift-ci openshift-ci bot added the lgtm label Jan 13, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fmount

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

The pull request process is described 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

@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Jan 13, 2026

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/e1ff25dd1f604b289ff0c75ee431fd2f

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 55m 06s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 28m 51s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 40m 20s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 46s
✔️ noop SUCCESS in 0s
❌ cifmw-pod-ansible-test FAILURE in 6m 06s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 01s
✔️ cifmw-molecule-os_must_gather SUCCESS in 24m 35s

@averdagu
Copy link
Contributor Author

averdagu commented Jan 13, 2026

recheck

@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Jan 13, 2026

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a8781419bf9847d6b711c38e60eeccd2

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 59m 33s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 21m 31s
❌ cifmw-crc-podified-edpm-baremetal FAILURE in 1h 45m 38s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 55s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 09s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 50s
✔️ cifmw-molecule-os_must_gather SUCCESS in 28m 53s

@michburk
Copy link
Contributor

michburk commented Jan 13, 2026

recheck

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

Reviewers

@fmount fmount fmount approved these changes

@michburk michburk michburk approved these changes

@evallesp evallesp Awaiting requested review from evallesp

@amartyasinha amartyasinha Awaiting requested review from amartyasinha

Assignees

@fmount fmount

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@averdagu @michburk @fmount