Skip to content
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

Add a new logic related to version and 1 more namespace to gather dat… #3

Merged
merged 1 commit into from
Sep 19, 2022

Conversation

Jooho
Copy link
Contributor

@Jooho Jooho commented Aug 17, 2022

Issue: #2

Signed-off-by: jooho jlee@redhat.com

@VaishnaviHire
Copy link
Contributor

/lgtm

Worked well for me.

@VaishnaviHire VaishnaviHire self-requested a review August 24, 2022 14:00
@VaishnaviHire
Copy link
Contributor

Tested with custom built image : quay.io/vhire/must-gather:pr3

 oc adm must-gather --image=quay.io/vhire/must-gather:pr3

cc: @bdattoma

collection-scripts/version Outdated Show resolved Hide resolved
@bdattoma
Copy link

bdattoma commented Aug 25, 2022

is there a Jira related to this PR? @Jooho @VaishnaviHire

@VaishnaviHire
Copy link
Contributor

is there a Jira related to this PR? @Jooho @VaishnaviHire
https://issues.redhat.com/browse/RHODS-4922

@bdattoma
Copy link

could you please generate a new build for the must gather image? @pablofelix is gonna follow this topic

@VaishnaviHire
Copy link
Contributor

could you please generate a new build for the must gather image? @pablofelix is gonna follow this topic

Updated the build.

 oc adm must-gather --image=quay.io/vhire/must-gather:pr3

…a 'rhods-notebooks'

Signed-off-by: jooho <jlee@redhat.com>
Copy link

@pablofelix pablofelix left a comment

Choose a reason for hiding this comment

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

/lgtm

@Jooho
Copy link
Contributor Author

Jooho commented Sep 12, 2022

@VaishnaviHire could you please merge it and build an official must-gather image?

@VaishnaviHire
Copy link
Contributor

@LaVLaS I am unable to merge this. Do you have access to merge this ?

@samuelvl samuelvl merged commit d93ca7e into red-hat-data-services:main Sep 19, 2022
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.

None yet

5 participants