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

fix: pull secret getting created even when no authentication is selected #927

Merged
merged 1 commit into from
Dec 8, 2022

Conversation

HarikrishnanBalagopal
Copy link
Contributor

Signed-off-by: Harikrishnan Balagopal harikrishmenon@gmail.com

Signed-off-by: Harikrishnan Balagopal <harikrishmenon@gmail.com>
@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Thanks for making a pull request! 😃
One of the maintainers will review and advise on the next steps.

@github-actions github-actions bot added the fix label Dec 8, 2022
@@ -143,14 +144,16 @@ func (p registryPreProcessor) preprocess(ir irtypes.IR) (irtypes.IR, error) {
ps := qaengine.FetchStringAnswer(qaKey, fmt.Sprintf("[%s] Enter the name of the pull secret : ", registry), []string{"The pull secret should exist in the namespace where you will be deploying the application."}, "", nil)
imagePullSecrets[registry] = ps
case usernamePasswordLogin:
createPullSecret = true
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
createPullSecret = true
createPullSecretFromDockerConfig = true

@ashokponkumar ashokponkumar enabled auto-merge (squash) December 8, 2022 07:26
@HarikrishnanBalagopal
Copy link
Contributor Author

Partially fixes #902

@HarikrishnanBalagopal
Copy link
Contributor Author

$ ls myproject/deploy/yamls
golang-deployment.yaml  golang-service.yaml  myproject-ingress.yaml  quay-io-imagepullsecret-secret.yaml

This fixes the pull secret getting created even when we answer

? [quay.io] What type of container registry login do you want to use?
ID: move2kube.target.imageregistry."quay.io".logintype
Hints:
- Docker login from config mode, will use the default config from your local machine.

 no authentication

@ashokponkumar ashokponkumar merged commit 6bef972 into konveyor:main Dec 8, 2022
@HarikrishnanBalagopal
Copy link
Contributor Author

$ ls myproject/deploy/yamls
all.icr.io-secret.yaml  golang-deployment.yaml  golang-service.yaml  myproject-ingress.yaml

This also fixes the pull secret getting created when we select use an existing pull secret option

? [quay.io] Enter the name of the pull secret : 
ID: move2kube.target.imageregistry."quay.io".pullsecret
Hints:
- The pull secret should exist in the namespace where you will be deploying the application.

 all.icr.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants