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

Set ForceNew to false for public_endpoint #118

Closed
wants to merge 1 commit into from

Conversation

bplotnick
Copy link

I don't believe changing public_endpoint requires a recreation of the cluster. At least it doesn't in the UI

I don't believe changing public_endpoint requires a recreation of the cluster. At least it doesn't in the UI
@bplotnick bplotnick requested a review from a team May 7, 2021 20:15
@hashicorp-cla
Copy link

hashicorp-cla commented May 7, 2021

CLA assistant check
All committers have signed the CLA.

@bcmdarroch
Copy link
Contributor

Hi @bplotnick, thanks for your contribution! Unfortunately there are some further changes required before this can merge - mainly, implementing Update for this resource, which must call this endpoint. You're welcome to make those changes, but otherwise I've asked our cloud Vault team for help with this.

@bcmdarroch
Copy link
Contributor

Completed in #152

@bcmdarroch bcmdarroch closed this Jun 22, 2021
aidan-mundy added a commit that referenced this pull request Oct 3, 2023
…3-09-11

SEC-090: Automated trusted workflow pinning (2023-09-11)
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.

None yet

3 participants