Skip to content

Conversation

@lplinux
Copy link
Contributor

@lplinux lplinux commented Sep 17, 2025

Description

We've deployed a new version of consul server (upgraded from 1.21.2 to 1.21.3) and we've lost the ability to see on the UI the list of URLs for certain services that instead of IP Addresses they are FQDN from AWS services.
This might be related to the IP Validation introduced under #22468

Testing & Reproduction steps

  1. On an already created cluster, try to register a service with address equals to an FQDN
consul services register \
-http-addr=https://consul-server.kube-system:8501/ \
-address=random-instance-sadgadsfgad.eu-north-2.es.amazonaws.com \
-port=443 \
-id=random-instance \
-name=random-instance \
-ca-file=/consul/tls/ca/tls.crt \
-ca-path=/consul/tls/ca/
  1. Check the service URL on the UI

Links

PR Checklist

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

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@lplinux lplinux requested a review from a team as a code owner September 17, 2025 15:03
@hashicorp-cla-app
Copy link

hashicorp-cla-app bot commented Sep 17, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the theme/ui Anything related to the UI label Sep 17, 2025
@suresh-hashicorp suresh-hashicorp self-assigned this Sep 18, 2025
@suresh-hashicorp suresh-hashicorp added the backport/all Apply backports for all active releases per .release/versions.hcl label Sep 18, 2025
@suresh-hashicorp
Copy link
Contributor

Thank you @lplinux for the PR.
Can you also add a changelog for this? Guidelines

@lplinux
Copy link
Contributor Author

lplinux commented Sep 18, 2025

Thank you @lplinux for the PR. Can you also add a changelog for this? Guidelines

Thanks @suresh-hashicorp . I've added the changelog.

@suresh-hashicorp suresh-hashicorp force-pushed the ui/bugfix/ui-format-allow-fqdn branch from 5c75378 to 5067816 Compare September 26, 2025 03:13
@suresh-hashicorp
Copy link
Contributor

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@lplinux CLA sign is pending. I can approve and merge after this.

@lplinux
Copy link
Contributor Author

lplinux commented Sep 26, 2025

CLA assistant check
Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

@lplinux CLA sign is pending. I can approve and merge after this.

I've signed it 3 times already. I hope this last time is the good one.
Thanks @suresh-hashicorp

Copy link
Contributor

@suresh-hashicorp suresh-hashicorp left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @lplinux

@suresh-hashicorp suresh-hashicorp merged commit 6df6707 into hashicorp:main Sep 26, 2025
58 checks passed
@hc-github-team-consul-core hc-github-team-consul-core added backport/1.21 This release series is longer active on CE, use backport/ent/1.21 backport/ent/1.18 Changes are backported to 1.18 ent backport/ent/1.19 Changes are backported to 1.19 ent backport/ent/1.20 backport to ent 1.20 labels Sep 26, 2025
@suresh-hashicorp suresh-hashicorp added backport/ent/1.21 changes are backported to 1.21 ent and removed backport/ent/1.18 Changes are backported to 1.18 ent backport/ent/1.19 Changes are backported to 1.19 ent backport/ent/1.20 backport to ent 1.20 labels Sep 26, 2025
@hc-github-team-consul-core
Copy link
Collaborator

📣 Hi @suresh-hashicorp! a backport is missing for this PR [22779] for versions [1.21] please perform the backport manually and add the following snippet to your backport PR description:

<details>
	<summary> Overview of commits </summary>
		- <<backport commit 1>>
		- <<backport commit 2>>
		...
</details>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/all Apply backports for all active releases per .release/versions.hcl backport/ent/1.21 changes are backported to 1.21 ent backport/1.21 This release series is longer active on CE, use backport/ent/1.21 theme/ui Anything related to the UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants