fix(ci): Inconsistent use of IMAGE_REGISTRY and IMAGE_NAMESPACE Makefile variables (#25846) #25860
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #25846
This PR implements the same logic for creating full image tag in both the
Makefile(for building docker images) andhack/update-manifests.shscript:IMAGE_REGISTRYis used for FQDN of the container registry. If it is not set, the full tag won't have the registry part. It means that the default registry for the container engine will be used (it's docker.io unless configured otherwise). IfIMAGE_REGISTRYis configured while IMAGE_NAMESPACE is not (which does not have much sense) an error is generated.IMAGE_NAMESPACEcontains name of user account in the container registry.For backward compatibility it can provide both container registry and user account (like
quay.io/johnsmith). If it not set, the full tag won't have the account part.IMAGE_REPOSITORYcontains name of the image (argocdby default). Now this parameter is supported for building docker images as well.The default image parameters for the standard image tag are specified in the Makefile
and the hacks/update-manifests.sh. ArgoCD forks may override those values using the
GithHub Actions variables.
Checklist: