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

Dev 1.29 user namespaces pss update #44156

Conversation

saschagrunert
Copy link
Member

Follow-up on #43803

I think it would be best to rebase the PR on top of master once dev-1.29 got integrated into it.

@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 Nov 30, 2023
@k8s-ci-robot k8s-ci-robot added this to the 1.29 milestone Nov 30, 2023
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Nov 30, 2023
Copy link

netlify bot commented Nov 30, 2023

👷 Deploy Preview for kubernetes-io-vnext-staging processing.

Name Link
🔨 Latest commit 8dc0806
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/65686f36594223000836c244

@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 30, 2023
@k8s-ci-robot k8s-ci-robot added the language/en Issues or PRs related to English language label Nov 30, 2023
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Nov 30, 2023
@saschagrunert saschagrunert force-pushed the dev-1.29-user-namespaces-pss-update branch from ef83e45 to 5034076 Compare November 30, 2023 09:33
Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(nits)

/lgtm

This PR improves the docs by fixing a bug.

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 30, 2023
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: ef7f856c0f605332165f99da616c1306840d5cd6

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 30, 2023
@k8s-ci-robot k8s-ci-robot requested a review from sftim November 30, 2023 11:16
Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
Co-authored-by: Tim Bannister <tim@scalefactory.com>
@saschagrunert saschagrunert changed the title WIP: Dev 1.29 user namespaces pss update Dev 1.29 user namespaces pss update Nov 30, 2023
@saschagrunert saschagrunert force-pushed the dev-1.29-user-namespaces-pss-update branch from 1eb44e0 to 8dc0806 Compare November 30, 2023 11:17
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 30, 2023
@saschagrunert
Copy link
Member Author

/hold

For docs to include dev-1.29 into main.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 30, 2023
@sftim
Copy link
Contributor

sftim commented Nov 30, 2023

The hold doesn't make sense to me @saschagrunert - when [under what conditions] would we unhold this PR?

@saschagrunert
Copy link
Member Author

@sftim we can unhold if everyone is fine with fixups on the dev-1.29 branch.

@katcosgrove
Copy link
Contributor

I'm fine with it!

/unhold

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 30, 2023
Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 30, 2023
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: b151e07593c2d8fd6befd6525f27644caed48333

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sftim

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 30, 2023
@k8s-ci-robot k8s-ci-robot merged commit 41135ad into kubernetes:dev-1.29 Nov 30, 2023
6 checks passed
@saschagrunert saschagrunert deleted the dev-1.29-user-namespaces-pss-update branch November 30, 2023 13:18
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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants