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

Change fake pull secret to a parsable value #578

Merged
merged 1 commit into from
Mar 30, 2021

Conversation

Reamer
Copy link
Contributor

@Reamer Reamer commented Mar 30, 2021

Due to various problems, we should change our fake pull secret example to a fake secret that is parsable.
Related issues:

Note: At the moment I have not tested an okd installation with this pull-secret, but I will do so this week.
EDIT: Done without problems.

@fortinj66
Copy link
Contributor

Thank You!

@vrutkovs vrutkovs merged commit 35281e5 into okd-project:master Mar 30, 2021
@derekm
Copy link

derekm commented Apr 1, 2021

I'm testing this now on a clean install... I will let you know how it goes! Installation has been a pain in recent weeks, fighting issues in latest oVirt and latest OKD... Last week, I stopped using my redhat pull secret and was relying on the dummy pull secret, and the installer would just sit dead in the water after provisioning the master VMs. Here's hoping this gets me further along like previously when I was using my redhat pull secret (but those installs would fail due to crashy ovirt engine)!

Glad I found this!

@derekm
Copy link

derekm commented Apr 2, 2021

This new pull secret is working, and installation is proceeding as previously... I'm still seeing a very crashy engine VM. Engine was running idle for 5 hrs before I started installation, but is now crashing every 5 or 10 minutes as installation proceeds.

@Reamer Reamer deleted the pull_secret branch April 2, 2021 08:56
@Reamer
Copy link
Contributor Author

Reamer commented Apr 2, 2021

Thank you very much for your feedback. I installed two new clusters yesterday with the new fake pullsecrect without any problems.

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

Successfully merging this pull request may close these issues.

4 participants