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

[backend/filestate] Allow pulumi stack ls to see all stacks regardless of passphrase #7660

Merged
merged 3 commits into from
Jul 28, 2021

Conversation

lukehoban
Copy link
Member

The information exposed via pulumi stack ls does not require being able to decrypt state files, but the existing logic for pulumi stack ls with the filestate backend was to fully decrypt the state file anyway, silently skipping any stacks that could not be decrypted. This led to surprising results from pulumi stack ls.

After these changes, pulumi stack ls with the filestate backend will list all stacks that are available. Notably, because there is no notion of "project" scoping in the fielstate backend (yet), pulumi stack ls will list all stacks independent of the project name.

Fixes #4798.

…ess of passphrase

The information exposed via `pulumi stack ls` does not require being able to decrypt state files, but the existing logic for `pulumi stack ls` with the filestate backend was to fully decrypt the state file anyway, silently skipping any stacks that could not be decrypted.  This led to surprising results from `pulumi stack ls`.

After these changes, `pulumi stack ls` with the filestate backend will list *all* stacks that are available.  Notably, because there is no notion of "project" scoping in the fielstate backend (yet), `pulumi stack ls` will list all stacks independent of the project name.

Fixes #4798.
@lukehoban lukehoban merged commit fa63759 into master Jul 28, 2021
@pulumi-bot pulumi-bot deleted the lukehoban/4798 branch July 28, 2021 01:37
jancespivo pushed a commit to jancespivo/pulumi that referenced this pull request Aug 4, 2021
…ess of passphrase (pulumi#7660)

* [backend/filestate] Allow `pulumi stack ls` to see all stacks regardless of passphrase

The information exposed via `pulumi stack ls` does not require being able to decrypt state files, but the existing logic for `pulumi stack ls` with the filestate backend was to fully decrypt the state file anyway, silently skipping any stacks that could not be decrypted.  This led to surprising results from `pulumi stack ls`.

After these changes, `pulumi stack ls` with the filestate backend will list *all* stacks that are available.  Notably, because there is no notion of "project" scoping in the fielstate backend (yet), `pulumi stack ls` will list all stacks independent of the project name.

Fixes pulumi#4798.
jancespivo pushed a commit to jancespivo/pulumi that referenced this pull request Aug 4, 2021
…ess of passphrase (pulumi#7660)

* [backend/filestate] Allow `pulumi stack ls` to see all stacks regardless of passphrase

The information exposed via `pulumi stack ls` does not require being able to decrypt state files, but the existing logic for `pulumi stack ls` with the filestate backend was to fully decrypt the state file anyway, silently skipping any stacks that could not be decrypted.  This led to surprising results from `pulumi stack ls`.

After these changes, `pulumi stack ls` with the filestate backend will list *all* stacks that are available.  Notably, because there is no notion of "project" scoping in the fielstate backend (yet), `pulumi stack ls` will list all stacks independent of the project name.

Fixes pulumi#4798.
@lukehoban lukehoban mentioned this pull request Aug 17, 2021
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

Successfully merging this pull request may close these issues.

pulumi stack ls with filestate backend silently skips stacks which use different passphrases
3 participants