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

managedfields: Move most of fieldmanager package to managefields #115065

Merged

Conversation

apelisse
Copy link
Member

@apelisse apelisse commented Jan 13, 2023

Move most of the fieldmanager package from k8s.io/apiserver/pkg/endpoints/handler/fieldmanager to k8s.io/apimachinery/pkg/util/managedfields where it'll be much easier to import in various parts of the system, including client-go which will be able to use it for test.

This only moves code, and all references to the code. There is no behavior change. Except for the move, the gvkParser has been copied temporarily into the internal package to avoid the dependency cycle, but will have to be removed soon when the gvkParser is deprecated anyway (see #114439 which will contribute to that).

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. 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. labels Jan 13, 2023
@apelisse
Copy link
Member Author

/triage accepted
/priority important-soon

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. area/apiserver and removed 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. labels Jan 13, 2023
@k8s-ci-robot k8s-ci-robot added sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. and removed do-not-merge/needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Jan 13, 2023
@k8s-ci-robot k8s-ci-robot added sig/auth Categorizes an issue or PR as relevant to SIG Auth. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jan 13, 2023
@k8s-ci-robot k8s-ci-robot added area/test sig/testing Categorizes an issue or PR as relevant to SIG Testing. labels Jan 18, 2023
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 18, 2023
@apelisse apelisse marked this pull request as draft January 19, 2023 17:04
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 19, 2023
@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 19, 2023
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Feb 8, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Feb 9, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
soltysh pushed a commit to soltysh/kubernetes that referenced this pull request Feb 9, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
soltysh pushed a commit to soltysh/kubernetes that referenced this pull request Feb 12, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Feb 21, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Feb 21, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Feb 22, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Feb 27, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Feb 28, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Mar 1, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Mar 4, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Mar 4, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Mar 5, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
bertinatto added a commit to bertinatto/kubernetes that referenced this pull request Mar 6, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
soltysh pushed a commit to soltysh/kubernetes that referenced this pull request Mar 12, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Mar 27, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Mar 28, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
soltysh pushed a commit to soltysh/kubernetes that referenced this pull request Apr 9, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 15, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 15, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 15, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 19, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 22, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 22, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 22, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 24, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
soltysh pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 24, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 25, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 26, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
dinhxuanvu pushed a commit to dinhxuanvu/kubernetes that referenced this pull request Apr 26, 2024
…util/managedfields

Some of the code we use in openshift-tests was recently made internal
in kubernetes#115065. This patch
exposes the code we need there.
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/test cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. 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/auth Categorizes an issue or PR as relevant to SIG Auth. sig/testing Categorizes an issue or PR as relevant to SIG Testing. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

5 participants