Skip to content

Fix Install Consul on Kubernetes button having localhost:3000 hard coded #22380

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

Merged
merged 1 commit into from
Jun 9, 2025

Conversation

Moose1301
Copy link
Contributor

Description

When moving around on the docs page the Install Consul on Kubernetes with the Consul K8s CLI links you to localhost:3000 no matter what

Links

https://developer.hashicorp.com/consul/docs/deploy/server/k8s/helm#install-consul-on-kubernetes-with-helm

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@Moose1301 Moose1301 requested review from a team as code owners June 4, 2025 19:44
@github-actions github-actions bot added the type/docs Documentation needs to be created/updated/clarified label Jun 4, 2025
@aimeeu aimeeu added pr/no-changelog PR does not need a corresponding .changelog entry backport/1.21 Changes are backported to 1.21 labels Jun 5, 2025
@aimeeu aimeeu added the backport/ent/1.21 changes are backported to 1.21 ent label Jun 5, 2025
Copy link
Contributor

@aimeeu aimeeu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for catching and fixing this docs bug!

Copy link

hashicorp-cla-app bot commented Jun 9, 2025

CLA assistant check
All committers have signed the CLA.

@aimeeu
Copy link
Contributor

aimeeu commented Jun 9, 2025

@Moose1301 Would you please sign the CLA at your earliest convenience? Thank you!

@Moose1301
Copy link
Contributor Author

@Moose1301 Would you please sign the CLA at your earliest convenience? Thank you!

Done

@aimeeu aimeeu merged commit 99a2b63 into hashicorp:main Jun 9, 2025
95 of 98 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/ent/1.21 changes are backported to 1.21 ent backport/1.21 Changes are backported to 1.21 pr/no-changelog PR does not need a corresponding .changelog entry pr/no-metrics-test type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants