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 Transport Server CRD #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chen23
Copy link
Contributor

@chen23 chen23 commented Feb 19, 2021

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

@vklohiya
Copy link
Contributor

Hi Eric,

You can create a PR in f5networks/k8s-bigip-ctlr repo as well in CRD Schema File. This is the single place where we are updating and maintaining the schemas. These schemas are then downstream to chart and operator release.

@vklohiya
Copy link
Contributor

@chen23 , We have updated the schema as part of this PR-46., which is taken from k8s-bigip-ctlr repository.

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