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

imagerepo: Continue scan for unconfigured provider #290

Merged
merged 1 commit into from
Jul 26, 2022

Conversation

darkowlzz
Copy link
Contributor

When a given image reference is identified as a known provider but is
unconfigured, continue the reconciliation and scan to be able to scan
public repositories. Abort the reconciliation for any other error during
login.

Also add a test for scanning public repos without rate limits.

Fixes #289

When a given image reference is identified as a known provider but is
unconfigured, continue the reconciliation and scan to be able to scan
public repositories. Abort the reconciliation for any other error during
login.

Also add a test for scanning public repos without rate limits.

Signed-off-by: Sunny <[email protected]>
@darkowlzz darkowlzz added the bug Something isn't working label Jul 26, 2022
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @darkowlzz

@darkowlzz darkowlzz merged commit a984aa1 into main Jul 26, 2022
@darkowlzz darkowlzz deleted the unconfigured-provider-scan branch July 26, 2022 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

image-reflector-controller cannot pull image list
2 participants