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

fix: issues with digital ocean and required environment variable versus pulumi value #208

Open
qdzlug opened this issue Nov 18, 2022 · 1 comment
Assignees

Comments

@qdzlug
Copy link
Contributor

qdzlug commented Nov 18, 2022

Describe the bug
This is related to #207 but slightly different. It appears that in order for the Pulumi provider to pull the token from the Pulumi config it needs to be both:

  • In the digitalocean namespace, as digitalocean:token
  • Encrypted.

I have made the first change, but in order to make the second change we are going to need to do a bit more work because of the way we inherit the provider from the superclass for the do module.

As a stopgap, I've noted in the logic when prompting for a key that it should be in the environment.

This - and #207 - both really need to be made a bit more sane given the fact we try and auth three different ways in order to make this work.

To Reproduce
Steps to reproduce the behavior:

  1. Deploy on DO w/o setting the DIGITALOCEAN_TOKEN environment variable.

Expected behavior
It should, uh, work.

Your environment
n/a

Additional context
None

@qdzlug
Copy link
Contributor Author

qdzlug commented Jan 10, 2023

Note - this is also causing the failure of DO Jenkins build jobs.

@qdzlug qdzlug self-assigned this Jan 10, 2023
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

1 participant