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.

Sign up for GitHub

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

Jump to bottom

blog: Add blogpost for user namespaces in 1.25 #35483

Closed
wants to merge 2 commits into from
Closed

blog: Add blogpost for user namespaces in 1.25 #35483

wants to merge 2 commits into from

Conversation

rata
Copy link
Member

@rata rata commented Jul 28, 2022 •
edited

@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 Jul 28, 2022
@netlify
Copy link

netlify bot commented Jul 28, 2022

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

Name Link
🔨 Latest commit 97ae337
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/62e263fec37f3e000a2be364

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. area/blog Issues or PRs related to the Kubernetes Blog subproject labels Jul 28, 2022
@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Jul 28, 2022
@rata rata mentioned this pull request Jul 28, 2022
Open
26 tasks
@reylejano reylejano added this to Planned in Blog articles Aug 2, 2022
@katcosgrove
Copy link
Contributor

Hi from the Comms team! Just a reminder that the Ready to Review deadline for feature blogs is Tuesday, August 16. You will also be assigned a publication date. Is there anything we can do to help you right now?

@sftim sftim removed this from Planned in Blog articles Aug 22, 2022
@sftim
Copy link
Contributor

sftim commented Aug 22, 2022

Can we assign a publication date anyway, to make the remaining work easier?

@rata
Copy link
Member Author

rata commented Aug 23, 2022

@sftim who decides the publication date?

As I mentioned to @katcosgrove on slack, I happen to be unavailable now and unsure when I'll be back to work on this (might be close to a month).

@sftim
Copy link
Contributor

sftim commented Aug 24, 2022

Who decides the publication date?

For the post-release blog article series, that's the release comms team.

@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. area/release-eng Issues or PRs related to the Release Engineering subproject language/fr Issues or PRs related to French language language/ja Issues or PRs related to Japanese language language/pt Issues or PRs related to Portuguese language language/zh Issues or PRs related to Chinese language sig/release Categorizes an issue or PR as relevant to SIG Release. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 29, 2022
@rata rata marked this pull request as ready for review August 29, 2022 15:36
@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 Aug 29, 2022
@k8s-ci-robot k8s-ci-robot added the language/vi Issues or PRs related to Vietnamese language label Aug 29, 2022
@giuseppe
content: fix typo
b3affab 
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe
blog: Add blogpost for user namespaces in 1.25
e8ea05b 
Signed-off-by: Rodrigo Campos <rodrigoca@microsoft.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Aug 29, 2022
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign kcmartin for approval by writing /assign @kcmartin in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@sftim
Copy link
Contributor

sftim commented Aug 30, 2022

@katcosgrove would you like to assign (or suggest) a date for this article?

@Sea-n
Copy link
Member

Sea-n commented Aug 30, 2022

Removing some wrong labels.

/remove-area release-eng
/remove-language fr ja pt vi zh
/remove-sig release

@k8s-ci-robot k8s-ci-robot removed area/release-eng Issues or PRs related to the Release Engineering subproject language/fr Issues or PRs related to French language language/ja Issues or PRs related to Japanese language language/pt Issues or PRs related to Portuguese language language/vi Issues or PRs related to Vietnamese language language/zh Issues or PRs related to Chinese language sig/release Categorizes an issue or PR as relevant to SIG Release. labels Aug 30, 2022
Sea-n
Sea-n reviewed Aug 30, 2022
View reviewed changes
content/en/docs/concepts/workloads/pods/user-namespaces.md
A pod can opt-in to use user nameapces by setting the `pod.spec.hostUsers` field
A pod can opt-in to use user namespaces by setting the `pod.spec.hostUsers` field
Copy link
Member

Choose a reason for hiding this comment

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

I think it's more clear to fix the typo in another PR.

Copy link
Member

Choose a reason for hiding this comment

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

typo fix here: #36417

@sftim
Copy link
Contributor

sftim commented Aug 30, 2022

@rata, this PR must target the main branch? Could you change the merge base, rebase locally, and force-push?

(if you're not sure about those steps, another option is to open a new PR and add a comment here that lets us find the new PR)

@giuseppe
Copy link
Member

opened a new PR for main here: #36416

@rata can you please close this PR?

@nate-double-u nate-double-u mentioned this pull request Aug 30, 2022
Merged
@nate-double-u
Copy link
Contributor

closing in favour of #36416

/close

@k8s-ci-robot
Copy link
Contributor

@nate-double-u: Closed this PR.

In response to this:

closing in favour of #36416

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@giuseppe giuseppe giuseppe left review comments

@Sea-n Sea-n Sea-n left review comments

@nate-double-u nate-double-u Awaiting requested review from nate-double-u

@onlydole onlydole Awaiting requested review from onlydole

@bgrant0607 bgrant0607 Awaiting requested review from bgrant0607

@bradtopol bradtopol Awaiting requested review from bradtopol

Assignees
No one assigned
Labels
area/blog Issues or PRs related to the Kubernetes Blog subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@rata @katcosgrove @sftim @k8s-ci-robot @Sea-n @giuseppe @nate-double-u