-
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
KEP-4222: Add roundtrip tests to Unstructured via CBOR and JSON. #122832
Merged
k8s-ci-robot
merged 6 commits into
kubernetes:master
from
benluddy:cbor-fuzz-native-to-unstructured-via
May 30, 2024
Merged
KEP-4222: Add roundtrip tests to Unstructured via CBOR and JSON. #122832
k8s-ci-robot
merged 6 commits into
kubernetes:master
from
benluddy:cbor-fuzz-native-to-unstructured-via
May 30, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Skipping CI for Draft Pull Request. |
k8s-ci-robot
added
do-not-merge/work-in-progress
Indicates that a PR should not merge because it is a work in progress.
kind/feature
Categorizes issue or PR as related to a new feature.
size/XXL
Denotes a PR that changes 1000+ lines, ignoring generated files.
do-not-merge/release-note-label-needed
Indicates that a PR should not merge because it's missing one of the release note labels.
sig/api-machinery
Categorizes an issue or PR as relevant to SIG API Machinery.
cncf-cla: yes
Indicates the PR's author has signed the CNCF CLA.
needs-triage
Indicates an issue or PR lacks a `triage/foo` label and requires one.
needs-priority
Indicates a PR lacks a `priority/foo` label and requires one.
area/apiserver
area/cloudprovider
area/dependency
Issues or PRs related to dependency changes
area/kube-proxy
area/kubectl
area/test
kind/api-change
Categorizes issue or PR as related to adding, removing, or otherwise changing an API
sig/architecture
Categorizes an issue or PR as relevant to SIG Architecture.
sig/auth
Categorizes an issue or PR as relevant to SIG Auth.
sig/cli
Categorizes an issue or PR as relevant to SIG CLI.
sig/cloud-provider
Categorizes an issue or PR as relevant to SIG Cloud Provider.
sig/cluster-lifecycle
Categorizes an issue or PR as relevant to SIG Cluster Lifecycle.
sig/instrumentation
Categorizes an issue or PR as relevant to SIG Instrumentation.
sig/network
Categorizes an issue or PR as relevant to SIG Network.
sig/node
Categorizes an issue or PR as relevant to SIG Node.
sig/storage
Categorizes an issue or PR as relevant to SIG Storage.
sig/testing
Categorizes an issue or PR as relevant to SIG Testing.
labels
Jan 17, 2024
/test pull-kubernetes-unit |
deads2k
reviewed
May 23, 2024
deads2k
reviewed
May 23, 2024
deads2k
reviewed
May 23, 2024
benluddy
force-pushed
the
cbor-fuzz-native-to-unstructured-via
branch
from
May 23, 2024 19:44
2487b15
to
5db2347
Compare
k8s-ci-robot
removed
the
lgtm
"Looks good to me", indicates that a PR is ready to be merged.
label
May 23, 2024
deads2k
reviewed
May 23, 2024
/cc @liggitt |
k8s-ci-robot
added
cncf-cla: no
Indicates the PR's author has not signed the CNCF CLA.
needs-rebase
Indicates a PR cannot be merged because it has merge conflicts with HEAD.
and removed
cncf-cla: yes
Indicates the PR's author has signed the CNCF CLA.
labels
May 23, 2024
Implementing custom marshaling on several API types for CBOR makes the upstream CBOR library an indirect dependency of several staging modules.
benluddy
force-pushed
the
cbor-fuzz-native-to-unstructured-via
branch
from
May 29, 2024 21:50
5db2347
to
d7cccf3
Compare
k8s-ci-robot
added
cncf-cla: yes
Indicates the PR's author has signed the CNCF CLA.
and removed
cncf-cla: no
Indicates the PR's author has not signed the CNCF CLA.
needs-rebase
Indicates a PR cannot be merged because it has merge conflicts with HEAD.
labels
May 29, 2024
testing looks good and thorough. this is the next step for CBOR serialization and leaves other serializers alone. /lgtm |
k8s-ci-robot
added
the
lgtm
"Looks good to me", indicates that a PR is ready to be merged.
label
May 30, 2024
LGTM label has been added. Git tree hash: 9850aa6115cd5f8c57da2cff51b9f3fd7505eed7
|
/approve |
k8s-ci-robot
added
the
approved
Indicates a PR has been approved by an approver from all required OWNERS files.
label
May 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
approved
Indicates a PR has been approved by an approver from all required OWNERS files.
area/apiserver
area/cloudprovider
area/code-generation
area/dependency
Issues or PRs related to dependency changes
area/kube-proxy
area/kubectl
area/test
cncf-cla: yes
Indicates the PR's author has signed the CNCF CLA.
kind/api-change
Categorizes issue or PR as related to adding, removing, or otherwise changing an API
kind/feature
Categorizes issue or PR as related to a new feature.
lgtm
"Looks good to me", indicates that a PR is ready to be merged.
needs-priority
Indicates a PR lacks a `priority/foo` label and requires one.
release-note-none
Denotes a PR that doesn't merit a release note.
sig/api-machinery
Categorizes an issue or PR as relevant to SIG API Machinery.
sig/architecture
Categorizes an issue or PR as relevant to SIG Architecture.
sig/auth
Categorizes an issue or PR as relevant to SIG Auth.
sig/cli
Categorizes an issue or PR as relevant to SIG CLI.
sig/cloud-provider
Categorizes an issue or PR as relevant to SIG Cloud Provider.
sig/cluster-lifecycle
Categorizes an issue or PR as relevant to SIG Cluster Lifecycle.
sig/instrumentation
Categorizes an issue or PR as relevant to SIG Instrumentation.
sig/network
Categorizes an issue or PR as relevant to SIG Network.
sig/node
Categorizes an issue or PR as relevant to SIG Node.
sig/storage
Categorizes an issue or PR as relevant to SIG Storage.
sig/testing
Categorizes an issue or PR as relevant to SIG Testing.
size/XL
Denotes a PR that changes 500-999 lines, ignoring generated files.
triage/accepted
Indicates an issue or PR is ready to be actively worked on.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What type of PR is this?
/kind feature
/sig api-machinery
What this PR does / why we need it:
This is an alpha requirement for KEP-4222 (CBOR). It tests the paths native-to-CBOR-to-unstructured-to-CBOR-to-native and native-to-JSON-to-unstructured-to-JSON-to-native, verifying that the original object is roundtripped faithfully and that the intermediate unstructured objects are identical across the CBOR and JSON paths. This is especially relevant for the case of a dynamic client interacting with built-in API resources using the CBOR encoding.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: