-
Notifications
You must be signed in to change notification settings - Fork 458
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
fix: inducing overridden DockerRegistryId in case docker registry is overridden #4178
Conversation
} | ||
|
||
if ciPipeline.CiTemplate == nil { | ||
impl.logger.Warn("returning as ciPipeline.CiTemplate is found nil") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is response in this case
dockerRegistryId := ciPipeline.CiTemplate.DockerRegistryId | ||
if len(*dockerRegistryId) == 0 { | ||
impl.logger.Warn("returning as dockerRegistryId is found empty") | ||
return nil, nil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
throw custom error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This custom error was not being thrown earlier also, so don't know it may impact other flows also outside this function which will increase the scope of this maybe
Kudos, SonarCloud Quality Gate passed!
|
Description
When we override the base build config (docker registry), at the time of inducing the image pull secret in the manifest we only checked for base build config, and not the overridden docker registry config. This pr aims at fixing the same issue.
Fixes #4179
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:
Does this PR introduce a user-facing change?