-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
fix unit test failure in ./pkg/api/testing/serialization_test.go #71370
fix unit test failure in ./pkg/api/testing/serialization_test.go #71370
Conversation
41c6728
to
52ca868
Compare
/retest |
/kind failing-test |
/retest |
this issue is due to the unit tests that use rather than shuffling order around, can we remove use of testapi from this test and pin the tested version to apps/v1? |
/cc @jpbetz |
52ca868
to
33f6518
Compare
/test pull-kubernetes-integration |
@liggitt comments addressed, PTAL |
extGroup := testapi.Apps | ||
defaultGroup := testapi.Default | ||
|
||
fuzzInternalObject(t, schema.GroupVersion{Group: "apps", Version: runtime.APIVersionInternal}, rs, rand.Int63()) |
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.
why was this dropped?
looks much better, but the fuzzing should be preserved, otherwise the conversion test is only checking converting an empty object, which isn't that useful |
… error like: --- FAIL: TestSetControllerConversion (0.00s) serialization_test.go:104: rs._internal.apps -> rs.v1.apps serialization_test.go:107: unexpected encoding error: no kind "ReplicaSet" is registered for version "apps/v1beta1" in scheme "k8s.io/kubernetes/pkg/api/legacyscheme/scheme.go:29"
33f6518
to
82f8f3e
Compare
@liggitt comment addressed, PTAL |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt, WanLinghao 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 |
What this PR does / why we need it:
fix unit test failure in ./pkg/api/testing/serialization_test.go with error like:
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #71369
Special notes for your reviewer:
Does this PR introduce a user-facing change?: