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
kmsv2: improve test coverage #116202
kmsv2: improve test coverage #116202
Conversation
/sig auth |
/test pull-kubernetes-e2e-kind-kms |
d35346c
to
389e59f
Compare
@@ -3,7 +3,7 @@ module k8s.io/kms/plugins/mock | |||
go 1.19 | |||
|
|||
require ( | |||
k8s.io/klog/v2 v2.90.0 | |||
k8s.io/klog/v2 v2.90.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran go mod tidy
to fix e2e job failure:
CGO_ENABLED=0 GOOS=linux GOARCH=${TARGETARCH} GO111MODULE=on go build -a -o mock-kms-plugin plugin.go
go: downloading k8s.io/klog/v2 v2.90.0
go: updates to go.mod needed; to update it:
go mod tidy
staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig/config.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig/config_test.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig/config_test.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig/config_test.go
Show resolved
Hide resolved
staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig/config_test.go
Outdated
Show resolved
Hide resolved
389e59f
to
c522d47
Compare
staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig/config_test.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig/config_test.go
Outdated
Show resolved
Hide resolved
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: enj, ritazh The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
c522d47
to
ea5a3e5
Compare
Signed-off-by: Rita Zhang <rita.z.zhang@gmail.com>
ea5a3e5
to
51db940
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
LGTM label has been added. Git tree hash: 98d7151a0fb16a11c003fe3a5f585443fa9ae545
|
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
1 similar comment
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
/retest |
/test pull-kubernetes-e2e-gce-100-performance |
/retest |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Increase test coverage for kmsv2 changes to above 75%
Which issue(s) this PR fixes:
Partially addressing #114188
Special notes for your reviewer:
k8s.io/kubernetes/staging/src/k8s.io/apiserver/pkg/storage/value/encrypt/envelope/metrics coverage: 79.1% of statements
k8s.io/kubernetes/staging/src/k8s.io/apiserver/pkg/server/options/encryptionconfig coverage: 75.2% of statements
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: