JetSki icon indicating copy to clipboard operation
JetSki copied to clipboard

pull-secret is double-quoted

Open rvansa opened this issue 3 years ago • 5 comments

The Convert pullsecret to JSON in ansible-ipi-install/roles/set-deployment-fact/tasks.main cause the pull secret defined as

pullsecret: '{"auths": ... }'

to end up with one extra layer of quoting, failing the playbook when the pull-secret file is used on the provisioner.

rvansa avatar Sep 08 '21 13:09 rvansa

Thank you for raising this issue

github-actions[bot] avatar Sep 08 '21 13:09 github-actions[bot]

Please use pullsecret as is - without any begnning/ending quotes @rvansa and let me know how it goes

smalleni avatar Sep 08 '21 13:09 smalleni

@smalleni I have installed OCP now (disabling the conversion) but I perceive the pull-secret as an opaque string, therefore pasting it as such. Putting the JSON straight into YAML seems awkward. Maybe a check if the pullsecret is string or object would satisfy any use case.

rvansa avatar Sep 08 '21 14:09 rvansa

Ack. I think we had to make that change with something related to exporting these group_vars/all.yml as environment variables for tools like jenkins. We used to require a '' before that?

I like your idea, do you mind sending a PR?

smalleni avatar Sep 08 '21 17:09 smalleni

Created https://github.com/redhat-performance/JetSki/pull/174

rvansa avatar Sep 09 '21 07:09 rvansa

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

github-actions[bot] avatar Mar 25 '23 00:03 github-actions[bot]