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

Issues setting OPENSHIFT_INSTALL_PULL_SECRET variable #888

Closed
jobbler opened this issue Dec 12, 2018 · 4 comments
Closed

Issues setting OPENSHIFT_INSTALL_PULL_SECRET variable #888

jobbler opened this issue Dec 12, 2018 · 4 comments

Comments

@jobbler
Copy link

jobbler commented Dec 12, 2018

Version

$ openshift-install version
bin/openshift-install v0.5.0-master-89-g814fbf7c1433aaf4c304c17a6f57a9bed1b08444
Terraform v0.11.10

Platform (aws|libvirt|openstack):

aws

What happened?

install fails when setting the OPENSHIFT_INSTALL_PULL_SECRET environment variable for install. Works fine when using OPENSHIFT_INSTALL_PULL_SECRET_PATH.

How to reproduce it (as minimally and precisely as possible)?

Set the variable and execute a create. The installation fails.
export OPENSHIFT_INSTALL_PULL_SECRET='{ "auths": { "quay.io": { "auth": "fdfgggsfgfgfgsfgggsggfggsgdfgg", "email": "" } } }'

Anything else we need to know?

If you set the variable as a here-doc, installation works.

export OPENSHIFT_INSTALL_PULL_SECRET=$( cat <<EOF
{ "auths": { "quay.io": { "auth": "fdfgggsfgfgfgsfgggsggfggsgdfgg", "email": "" } } }'
EOF
)

This should be included in the documentation to prevent others from having the same issue.

@crawford
Copy link
Contributor

We are removing environment variables altogether (#861). Once that merges, I'm going to close this out since it will no longer be applicable.

@wking
Copy link
Member

wking commented Dec 12, 2018

You should transition to supplying an install-config instead (as documented in #861), if you don't want to go through the wizard.

@crawford
Copy link
Contributor

#861 has merged.

/close

@openshift-ci-robot
Copy link
Contributor

@crawford: Closing this issue.

In response to this:

#861 has merged.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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

No branches or pull requests

4 participants