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 secrets are not created unless oAuth credentials are provided #2284

Closed
daniel-sanche opened this issue Jan 15, 2019 · 4 comments
Closed

Comments

@daniel-sanche
Copy link
Contributor

I noticed the code to create GCP service account secrets comes after a line that checks for oAuth environment variables. This seems unnecessary, because everything works fine if I provide fake values for the oAuth id and secret. Was this intentional?

This is likely causing people to run into issues like #1949 and #1950 which are hard to diagnose without looking at the code

@texasmichelle
Copy link
Member

+1 to creating service accounts even when IAP is skipped

@daniel-sanche
Copy link
Contributor Author

Opened PR here: #2313

@jlewi
Copy link
Contributor

jlewi commented Jan 28, 2019

@dansanche Can we close this issue? Does this warrant a cherry-pick onto the 0.4 branch?

@daniel-sanche
Copy link
Contributor Author

Yes, sorry this is good to close. I opened and merged a 0.4 cherry-pick PR here: #2324

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

No branches or pull requests

3 participants