-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[cli] Disable permalinks to the update details page when using self-managed backends (S3, Azure, GCS) #6251
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved from a "does what I expected it to do" perspective. Full code review left to others more qualified :)
5dc9bc1
to
b1d28fc
Compare
34afac1
to
5631ac1
Compare
5631ac1
to
67bf30f
Compare
67bf30f
to
a8aa4ba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but curious what happens if you just pass in pulumi up --suppress-permalink
CHANGELOG.md
Outdated
@@ -65,7 +72,7 @@ CHANGELOG | |||
|
|||
- [automation/python] Fixed a bug in nested configuration parsing. | |||
[#6349](https://github.com/pulumi/pulumi/pull/6349) | |||
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…naged backends (S3, Azure, GCS) Fixes: #4029 Fixes: #3537 Should the user want to get permalinks when using a self-managed backend, they can pass a flag: ``` $ pulumi up --suppress-permalink false ``` Permalinks for these self-managed backends will be suppressed on `update`, `preview`, `destroy`, `import` and `refresh` operations.
a8aa4ba
to
c77a1a9
Compare
cmdutil.Diag().Warningf(diag.Message("", "Could not get signed url for stack location: %v"), err) | ||
cmdutil.Diag().Warningf(diag.Message("", "Unable to create signed url for current backend to "+ | ||
"create a Permalink. Please visit https://www.pulumi.com/docs/troubleshooting/ "+ | ||
"for more information\n")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there going to be text at this link to help with this? What are the cases where this will happen, and what actually should the user do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are 2 places the docs will be updated
The troubleshooting section will have details as to why they get an open bucket url error - it's usually regarding using an Azure and GCp service principal to log into the self managed backends
I will also be updating the state and backends page to talk about the new defaults here of not showing a permalink
Both will be ready early Monday morning PST to make sure they are ready to go
when trying this in WLS2 , the syntax is slightly different from above if you actually want to add the argument: |
Fixes: #4029
Fixes: #3537
Should the user want to get permalinks when using a self-managed backend, they can pass a flag:
Permalinks for these self-managed backends will be suppressed on
update
,preview
,destroy
,import
andrefresh
operations.A view of the functionality:
self-managed backend
Using the SaaS