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

GCP: Make the sentence "Enter 2 empty lines to finish" less confusing && add option to specify the Service Account file. #451

Closed
vlours opened this issue Feb 4, 2022 · 5 comments
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.

Comments

@vlours
Copy link
Contributor

vlours commented Feb 4, 2022

As a no-English first language speaker, I’ve understood the sentence "[Enter 2 empty lines to finish]" as something:

Enter 2 empty lines to “cancel”

So it will be better to use a strong word (or much better avoid to hit enter twice ?) to Confirm

Press Enter two additional times to confirm your choice

# grep -n -A2 -B2 "Enter 2 empty lines to finish" cloud-credential-operator/vendor/github.com/AlecAivazis/survey/v2/multiline.go
33-{{- else }}
34-  {{- if .Default}}{{color "white"}}({{.Default}}) {{color "reset"}}{{end}}
35:  {{- color "cyan"}}[Enter 2 empty lines to finish]{{color "reset"}}
36-{{- end}}`
37-

This will have less confusion here:

33-{{- else }}
34-  {{- if .Default}}{{color "white"}}({{.Default}}) {{color "reset"}}{{end}}
35:  {{- color "cyan"}}[Press Enter two additional times to confirm your choice]{{color "reset"}}
36-{{- end}}`
37-

Another way to avoid this will be to allow to pass the GCP service account JSON file with an option like --gcp-sa-json

@vlours vlours changed the title Make the sentence "Enter 2 empty lines to finish" less confusing. GCP: Make the sentence "Enter 2 empty lines to finish" less confusing. Feb 4, 2022
@openshift-bot
Copy link
Contributor

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci openshift-ci bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 5, 2022
@vlours
Copy link
Contributor Author

vlours commented May 11, 2022

I've created another Issue with the Owner of the multiline.go file: AlecAivazis/survey#419 to address the first part of this issue (the confusing double empty line).

But that will be awesome if the ccoctl command will have an option to pass the Service Account directly, such as: --sa-json <Service_Account_full_PATH.json>

@vlours vlours changed the title GCP: Make the sentence "Enter 2 empty lines to finish" less confusing. GCP: Make the sentence "Enter 2 empty lines to finish" less confusing && add option to specify the Service Account file. May 11, 2022
@2uasimojo
Copy link
Member

Hi @vlours. Sorry for the delay here -- we tend to pay attention to Jira rather than issues. It would be great if you could open this as a new Story in the CCO-* project. Thanks!

/cc @akhil-rane

@vlours
Copy link
Contributor Author

vlours commented Jun 14, 2022

Thanks @2uasimojo.

I've created a Story in Jira as suggested:
CCO-224 - [RFE] addoitional option for ccoctl to allow Service Account in GCP

Feel free to close this Issue if no longer relevant.

Cheers,

@2uasimojo
Copy link
Member

Thanks @vlours. Closing in favor of CCO-224.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Development

No branches or pull requests

3 participants