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

feat(RHTAPREL-618): create two repos when pushing to registry.redhat.io #80

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

mmalina
Copy link
Contributor

@mmalina mmalina commented Oct 31, 2023

Based on discussion with rbean and jvulgan in Slack, we should actually create two items in ContainerImage.repositories for images pushed to registry.redhat.io. The first one will be the same as for other pushes. The second one will have the registry and repository values converted (see the Python script documentation for more details).

The quay.io entry will be used for pulling the image for grading of the image by Clair.

@mmalina mmalina requested review from a team, theflockers, davidmogar, scoheb, johnbieren, jinqi7, cuipinghuo and Troy876 and removed request for a team October 31, 2023 15:08
@mmalina mmalina force-pushed the pyxis-image-repos branch 2 times, most recently from 33426fd to 6a00be6 Compare October 31, 2023 15:24
Copy link
Contributor

@theflockers theflockers left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@happybhati happybhati left a comment

Choose a reason for hiding this comment

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

LGTM 👍

"push_date": "1970-10-10T10:10:10.000000+00:00",
Why this date just curious ?:p

@mmalina
Copy link
Contributor Author

mmalina commented Nov 1, 2023

LGTM 👍

"push_date": "1970-10-10T10:10:10.000000+00:00", Why this date just curious ?:p

No idea, ask Johnny :D I just cloned the other entry and changed what needed changing.

@johnbieren
Copy link
Collaborator

LGTM 👍
"push_date": "1970-10-10T10:10:10.000000+00:00", Why this date just curious ?:p

No idea, ask Johnny :D I just cloned the other entry and changed what needed changing.

The good old days 😂

Based on discussion with rbean and jvulgan in Slack, we should
actually create two items in ContainerImage.repositories
for images pushed to registry.redhat.io. The first one will be
the same as for other pushes. The second one will have
the registry and repository values converted (see the Python
script documentation for more details).

The quay.io entry will be used for pulling the image
for grading of the image by Clair.

Signed-off-by: Martin Malina <[email protected]>
@mmalina mmalina merged commit 0171e10 into konflux-ci:main Nov 1, 2023
7 checks passed
@mmalina mmalina deleted the pyxis-image-repos branch November 1, 2023 16:07
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.

4 participants