Skip to content

Get version from image - must-gather #11

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.

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

Merged
merged 3 commits into from
Dec 28, 2022

Conversation

razo7
Copy link
Member

@razo7 razo7 commented Aug 22, 2022

According to the must-gather enhancement, there is a version file that indicates the product (first line) and the version (second line).

Currently the printed version is only in y.z format, due to regular expression, when y and z are one digit each. Resulting in "0.4" version for https://registry.redhat.io/workload-availability/self-node-remediation-must-gather-rhel8:v0.4.0, rather than "0.0.4".

Fix bug https://issues.redhat.com/browse/ECOPROJECT-982

@razo7
Copy link
Member Author

razo7 commented Aug 23, 2022

HOLD. Wait for a response after we have opened a discussion in ECOPROJECT-983 about possible bug in default OCP must-gather https://coreos.slack.com/archives/CFDM5CQMN/p1661184777988329

Maybe openshift/must-gather#327 would resolve https://issues.redhat.com/browse/ECOPROJECT-982 without the need for this PR.

@razo7 razo7 force-pushed the version-file-must-gather branch from 92c0b1a to c27f164 Compare December 28, 2022 11:42
@razo7
Copy link
Member Author

razo7 commented Dec 28, 2022

Similar fix to a similar bug ECOPROJECT-983

@mshitrit
Copy link
Member

/lgtm

@razo7 razo7 merged commit 9e9b952 into medik8s:main Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants