Skip to content

Improve must-gather directory naming with version and test information#14729

Draft
dahorak wants to merge 1 commit intored-hat-storage:masterfrom
dahorak:add-test-name-to-must-gather-files
Draft

Improve must-gather directory naming with version and test information#14729
dahorak wants to merge 1 commit intored-hat-storage:masterfrom
dahorak:add-test-name-to-must-gather-files

Conversation

@dahorak
Copy link
Contributor

@dahorak dahorak commented Mar 20, 2026

Update must-gather directory names to include ODF and OCP versions for better clarity and easier log identification. For failed test cases, also include the test name in the directory name.

Directory naming examples:

  • Success: ocs_must_gather_ODF-4.21, ocp_must_gather_OCP-4.15
  • Failure: ocs_must_gather_ODF-4.21_test_name, ocp_must_gather_OCP-4.15_test_name

This makes it immediately clear which versions the logs are from and which test failed, improving log organization and debugging workflow.

Update must-gather directory names to include ODF and OCP versions for
better clarity and easier log identification. For failed test cases,
also include the test name in the directory name.

Directory naming examples:
- Success: ocs_must_gather_ODF-4.21, ocp_must_gather_OCP-4.15
- Failure: ocs_must_gather_ODF-4.21_test_name, ocp_must_gather_OCP-4.15_test_name

This makes it immediately clear which versions the logs are from and
which test failed, improving log organization and debugging workflow.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Daniel Horak <dahorak@redhat.com>
@dahorak dahorak self-assigned this Mar 20, 2026
@dahorak dahorak added the team/ecosystem Ecosystem team related issues/PRs label Mar 20, 2026
@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines label Mar 20, 2026
@openshift-ci
Copy link

openshift-ci bot commented Mar 20, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dahorak

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

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: acceptance
PR Test Path: tests/
Additional Test Params:
OCP VERSION: 4.21
OCS VERSION: 4.21
tested against branch: master

Job PASSED.

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

Labels

do-not-merge/work-in-progress size/S PR that changes 10-29 lines team/ecosystem Ecosystem team related issues/PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants