-
Notifications
You must be signed in to change notification settings - Fork 7
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
DeploymentSettings doesn't handle EnvironmentVariables if it's a secret and fails/ignore silently #447
Comments
Once it's fixed, if some info or an example could be added to the documentation here, that would be very helpful. Thanks! |
@bryantallison We have some examples on creation of DeploymentSettings via PSP - typescript example, + in python and yaml But definitely agree it would be better to also have examples in the registry, I'll see how easy that is |
This issue has been addressed in PR #475 and shipped in release v0.27.4. |
Hey @IaroslavTitov this is better, thank you! I'm seeing a slightly different but related issue with v0.27.4: Given two secrets and one plaintext: var ghToken = config.RequireSecret("ghtoken");
// ...
EnvironmentVariables =
{
{ "GH_TOKEN", ghToken },
{ "GH_TOKEN_OUTPUT", Output.CreateSecret("my_secret_token") },
{ "NOT_SECRET", "I love otters" },
}, I see three secrets in the console which isn't what I expected (2 secret, 1 plaintext): ![]() |
Yes, I called it out in the PR, unfortunately this is a bug beyond PSP, with pulumi itself. For C# resources, it will turn all Env Vars secret if just one is secret. |
Aha, thank you for pointing that out. Upvoting pulumi/pulumi-dotnet#22 now😄 |
What happened?
Probably related to #419 - I've tried multiple ways to set an env var with a secret and cannot get anything to show up in Pulumi Cloud. The
up
succeeds but silently does not create the env var if one contains a secret.I've tried creating a secret inline, pulling an existing secret from config, doing applies, nothing seems to work.
Note that if there is a single secret value in the env var at all, none of the env vars are set, even plaintext ones.
Example
Output of
pulumi about
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).
The text was updated successfully, but these errors were encountered: