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

[AKS]az aks list -o table should show privateFqdn as fqdn for private clusters #12784

Merged
merged 1 commit into from
Mar 31, 2020

Conversation

gossion
Copy link
Member

@gossion gossion commented Mar 30, 2020

Description of PR (Mandatory)
(Why this PR? What is changed? What is the effect? etc. A high-quality description can accelerate the review process)
az aks list -o table does not have fqdn info for private clusters. The fqdn of private cluster is in .privateFqdn for private clusters, while it is .fqdn for public clusters. This is to make cli show correct fqdn for private clusters.

Testing Guide
(Example commands with explanations)
az aks list -o table will show correct fqdn for private clusters

History Notes:
(Fill in the following template if multiple notes are needed, otherwise PR title will be used for history note.)

[Component Name 1] (BREAKING CHANGE:) (az command:) make some customer-facing change.
[Component Name 2] (BREAKING CHANGE:) (az command:) make some customer-facing change.


This checklist is used to make sure that common guidelines for a pull request are followed.

@gossion gossion changed the title [AKS]az aks list -o table should show privateFqdn for private clusters [AKS]az aks list -o table should show privateFqdn as fqdn for private clusters Mar 30, 2020
Copy link
Member

@feiskyer feiskyer left a comment

Choose a reason for hiding this comment

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

lgtm

@yonzhan yonzhan requested a review from qwordy March 30, 2020 07:06
@yonzhan yonzhan added this to the S168 milestone Mar 30, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 30, 2020

add to S168

@yungezz
Copy link
Member

yungezz commented Mar 30, 2020

the change itself looks good. Just confirm, privatefqdn will show when output format isn't table also?

@gossion
Copy link
Member Author

gossion commented Mar 31, 2020

the change itself looks good. Just confirm, privatefqdn will show when output format isn't table also?

yes, it will

@yungezz yungezz merged commit 8e78629 into Azure:dev Mar 31, 2020
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.

5 participants