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

[Backport][ipa-4-6] Web UI (topology graph): Show FQDN for nodes if they have no common D… #2921

Closed

Conversation

serg-cymbaluk
Copy link
Contributor

Manual backport of #2890

It allows to avoid confusion with identical short hostnames.

There are two cases implemented:

  • no common DNS zone: graph shows FQDN for all nodes
  • all nodes have one common DNS zone: graph shows DN relatively to the common zone

https://pagure.io/freeipa/issue/7206

…NS zone

It allows to avoid confusion with identical short hostnames.

There are two cases implemented:
- no common DNS zone: graph shows FQDN for all nodes
- all nodes have one common DNS zone: graph shows DN relatively to the common zone

https://pagure.io/freeipa/issue/7206
@flo-renaud flo-renaud changed the title Web UI (topology graph): Show FQDN for nodes if they have no common D… [Backport][ipa-4-6] Web UI (topology graph): Show FQDN for nodes if they have no common D… Mar 22, 2019
@flo-renaud flo-renaud self-assigned this Mar 22, 2019
@flo-renaud
Copy link
Contributor

Hi @serg-cymbaluk
thank you for the backport, works for me. Tested with:

  • multiple replicas in the same domain
  • multiple replicas with a common portion of their domain name
  • multiple replicas with no common portion of their domain name

@flo-renaud flo-renaud added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Mar 22, 2019
@flo-renaud
Copy link
Contributor

ipa-4-6:

  • 7b61dbf Web UI (topology graph): Show FQDN for nodes if they have no common DNS zone

@flo-renaud flo-renaud closed this Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants