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

Remove the duplicate cockroachdb versions generated by the API response #997

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

prafull01
Copy link
Collaborator

@prafull01 prafull01 commented Jul 11, 2023

Right now, we pull the images from the https://catalog.redhat.com/ by calling an API. The response from the API is sending tag v20.1.4 and v20.1.5 twice and hence it is added to our deployment twice.

I have added a check that any image is used only once even if we receive duplicate response from API.

@prafull01 prafull01 force-pushed the version-file-type branch from 7ffa544 to 6ce369e Compare July 21, 2023 11:04
@prafull01 prafull01 changed the title Revert the version file type with no line terminators Remove the duplicate cockroachdb versions generated by the API response Jul 21, 2023
@prafull01 prafull01 requested review from udnay and rail July 21, 2023 11:15
Copy link
Member

@rail rail left a comment

Choose a reason for hiding this comment

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

LGTM. I wonder why the api return these images twice. I will take a look at that separately.

@rail rail merged commit da60e7b into master Jul 21, 2023
@rail rail deleted the version-file-type branch July 21, 2023 13:13
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