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

Use repository index to retrieve old manifests #725

Merged
merged 4 commits into from
Sep 27, 2021
Merged

Conversation

mudler
Copy link
Contributor

@mudler mudler commented Sep 27, 2021

Fixes #710

@mudler mudler requested a review from Itxaka September 27, 2021 10:30
As it is subject to more rate-limiting than expected, list images
only before downloading them

See #710

Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Instead of polling for all images, retrieve metadata with package index.
It also relaxes errors in case of repos cleanups

Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
Signed-off-by: Ettore Di Giacinto <edigiacinto@suse.com>
@mudler mudler enabled auto-merge (squash) September 27, 2021 12:17
@mudler
Copy link
Contributor Author

mudler commented Sep 27, 2021

Builds passed, but the publish step is the one we can test only on master.. so merging, and opening follow-ups if required

@mudler mudler disabled auto-merge September 27, 2021 12:23
@mudler mudler merged commit c9f260f into master Sep 27, 2021
@mudler mudler deleted the no_list_images branch September 27, 2021 12:23
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.

ci: Fail to list remote container registry images
2 participants