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

add columns to TransportServer CRD #1715

Merged
merged 2 commits into from
Feb 22, 2021

Conversation

chen23
Copy link
Contributor

@chen23 chen23 commented Feb 22, 2021

Following recommendation from: F5Networks/charts#44 (comment)

This changes the format to add columns to the output for TransportServers i.e.:

$ kubectl get ts -n cluster1
NAME              VIRTUALSERVERADDRESS   VIRTUALSERVERPORT   POOL          POOLPORT   AGE
frontend1         10.1.10.82             8081                frontend1     80         3h20m
frontend2         10.1.10.82             8082                frontend2     80         3h20m
frontend3         10.1.10.82             8083                frontend3     80         3h20m
frontend4         10.1.10.82             8084                frontend4     80         3h20m
frontend5         10.1.10.82             8085                frontend5     80         3h20m
my-frontend-crd   10.1.10.82             8080                my-frontend   80         3h55m

this looks nicer than the default of

NAME              AGE
frontend1         3h21m
frontend2         3h21m
frontend3         3h21m
frontend4         3h21m
frontend5         3h21m
my-frontend-crd   3h55m

Copy link
Contributor

@vklohiya vklohiya left a comment

Choose a reason for hiding this comment

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

Looks Fine

@vklohiya
Copy link
Contributor

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vklohiya
Copy link
Contributor

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vklohiya vklohiya merged commit 1494a71 into F5Networks:master Feb 22, 2021
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