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

Give tests more specific names for easier debugging #490

Merged
merged 1 commit into from
Jan 18, 2024
Merged

Conversation

ianstanton
Copy link
Member

@ianstanton ianstanton commented Jan 18, 2024

Today, tembo-operator test instances and namespaces have names like test-coredb-35742. This makes it difficult to determine which test is running in a given namespace.

This PR updates these with more descriptive names related to the tests being performed. This will improve the test debugging experience.

Example:

❯ k get ns
NAME                                  STATUS   AGE
calico-apiserver                      Active   19h
calico-system                         Active   19h
cert-manager                          Active   19h
cnpg-test                             Active   19h
default                               Active   19h
kube-node-lease                       Active   19h
kube-public                           Active   19h
kube-system                           Active   19h
local-path-storage                    Active   19h
minio                                 Active   19h
monitoring                            Active   19h
reflector                             Active   19h
tembo-system                          Active   19h
test-app-service-89647                Active   32s
test-backup-restore-13140             Active   32s
test-basic-cnpg-89746                 Active   32s
test-cnpg-metrics-create-86071        Active   32s
test-cnpg-pgparams-9725               Active   32s
test-delete-namespace-1254            Active   32s
test-ha-basic-cnpg-81469              Active   32s
test-ha-requires-load-81952           Active   32s
test-ha-two-replicas-22084            Active   32s
test-ha-upgrade-cnpg-14185            Active   32s
test-ha-verify-extension-39815        Active   32s
test-ingress-route-tcp-52121          Active   32s
test-ingress-route-tcp-adopt-3657     Active   32s
test-ingress-route-tcp-domain-74883   Active   32s
test-pooler-43699                     Active   32s
test-requires-load-6417               Active   32s
test-restart-postgres-27327           Active   32s
test-status-configs-75282             Active   32s
tigera-operator                       Active   19h
traefik                               Active   19h

Signed-off-by: Ian Stanton <ian@tembo.io>
@ianstanton ianstanton merged commit dfa7f27 into main Jan 18, 2024
9 checks passed
@ianstanton ianstanton deleted the rename-tests branch January 18, 2024 13:59
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.

None yet

2 participants