Skip to content

Improve Kubernetes access test coverage#26657

Merged
tigrato merged 2 commits intomasterfrom
tigrato/improve-kube-test-coverage
May 24, 2023
Merged

Improve Kubernetes access test coverage#26657
tigrato merged 2 commits intomasterfrom
tigrato/improve-kube-test-coverage

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented May 20, 2023

This PR improves the Kubernetes Access test coverage from 61% to >73%.

There are some other code paths that require special attention such as watching with filtering but those require more work.

Comment thread lib/kube/proxy/kube_creds_test.go Outdated
Comment thread lib/kube/proxy/kube_creds_test.go Outdated
Comment thread lib/kube/proxy/kube_creds_test.go Outdated
Comment thread lib/kube/proxy/kube_creds_test.go Outdated
This PR improves the Kubernetes Access test coverage from 61% to >73%.
There are some other code paths that require special attention such as
watching with filtering but those require more work.
@tigrato tigrato force-pushed the tigrato/improve-kube-test-coverage branch from cefb620 to 1482f5a Compare May 23, 2023 09:22
@tigrato tigrato force-pushed the tigrato/improve-kube-test-coverage branch from 1482f5a to ebe00c6 Compare May 23, 2023 16:41
@tigrato tigrato requested a review from r0mant May 24, 2023 18:26
@tigrato tigrato added this pull request to the merge queue May 24, 2023
Merged via the queue into master with commit 267f7c1 May 24, 2023
@tigrato tigrato deleted the tigrato/improve-kube-test-coverage branch May 24, 2023 19:12
@public-teleport-github-review-bot
Copy link
Copy Markdown

@tigrato See the table below for backport results.

Branch Result
branch/v11 Failed
branch/v12 Failed
branch/v13 Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants