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

Use public scada address - scada.hashicorp.cloud #120

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

manish-hashicorp
Copy link
Contributor

🛠️ Description

Update the default scada address to "scada.hashicorp.cloud" to remove the "internal" part of it. Note that existing URL will continue to work as its used by HCP dataplanes but we would like to avoid publishing an "internal" address to the public if possible. This is part of implementing https://hashicorp.atlassian.net/browse/HCPCO2-159.

⚠️ This should not be merged until https://github.com/hashicorp/cloud-scada-broker/pull/249 is merged and new domain being available until prod run of https://github.com/hashicorp/cloud-terraform/pull/2713 goes through.

🔗 External Links

https://hashicorp.atlassian.net/browse/HCPCO2-159

🚢 Release Note

Release note for CHANGELOG:

NONE

👍 Definition of Done

  • SDK added
  • SDK updated
  • Tests added?
  • Docs updated?

Changing it to "scada.hashicorp.cloud" to remove the "internal" part of
it. Note that existing URL will continue to work as its used by HCP
dataplanes but we would like to avoid publishing an "internal" address
to the public if possible. This is part of implementing
https://hashicorp.atlassian.net/browse/HCPCO2-159.
@manish-hashicorp manish-hashicorp requested a review from a team September 6, 2022 23:25
@manish-hashicorp manish-hashicorp changed the title Update default scada address Change to public scada address Sep 6, 2022
@manish-hashicorp manish-hashicorp changed the title Change to public scada address Use public scada address - scada.hashicorp.cloud Sep 6, 2022
@manish-hashicorp
Copy link
Contributor Author

Thanks for the review @bcmdarroch. Would it be possible to make a release after merging this?

@bcmdarroch bcmdarroch merged commit 3bf7fd7 into main Sep 9, 2022
@bcmdarroch bcmdarroch deleted the update-scada-url branch September 9, 2022 19:43
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.

2 participants