Promote PodOS field to GA

[ravisantoshgudimetla]

xref: kubernetes/enhancements#2802

[ravisantoshgudimetla]

/sig windows

[ravisantoshgudimetla]

cc @marosset @jsturtevant

[netlify]

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

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

[marosset]

/cc
I'll review after code freeze

[tallclair]

Please include an update to https://github.com/kubernetes/website/blob/main/content/en/docs/concepts/security/pod-security-standards.md with the changes from kubernetes/kubernetes#105919 (make sure to update the FAQ question in addition to the restrictions table).

/assign

[reylejano]

/milestone 1.25
/assign @krol3
/cc @kcmartin

[reylejano]

/assign @cathchu

[ravisantoshgudimetla]

Sure @tallclair, will open docs PR this week.

@tallclair : docs PR for pod security standards update:

#35772

[marosset]

Should we move the information about the api field at https://kubernetes.io/docs/concepts/windows/user-guide/ to https://kubernetes.io/docs/concepts/workloads/pods/?

[ravisantoshgudimetla]

Should we move the information about the api field at https://kubernetes.io/docs/concepts/windows/user-guide/ to https://kubernetes.io/docs/concepts/workloads/pods/?

I don't have a strong opinion on this. We started working on this field so that it'll be more useful to the windows users but it hasn't reached a stage where it is more useful to other OSes except for the pod security standards restricted mode. If you feel that this can move to a more neutral place, I don't have any issues with it.

[tallclair]

/unassign

I'll review #35772 instead.

[marosset]

Should we move the information about the api field at https://kubernetes.io/docs/concepts/windows/user-guide/ to https://kubernetes.io/docs/concepts/workloads/pods/?

I don't have a strong opinion on this. We started working on this field so that it'll be more useful to the windows users but it hasn't reached a stage where it is more useful to other OSes except for the pod security standards restricted mode. If you feel that this can move to a more neutral place, I don't have any issues with it.

I don't have a strong opinion.

[sftim]

Should we move the information about the api field at https://kubernetes.io/docs/concepts/windows/user-guide/ to https://kubernetes.io/docs/concepts/workloads/pods/?

We should definitely update https://kubernetes.io/docs/concepts/workloads/pods/ to explaing the Pod OS details. Linux users aren't very likely to read Guide for scheduling Windows containers in Kubernetes.

[ravisantoshgudimetla]

Should we move the information about the api field at https://kubernetes.io/docs/concepts/windows/user-guide/ to https://kubernetes.io/docs/concepts/workloads/pods/?

We should definitely update https://kubernetes.io/docs/concepts/workloads/pods/ to explaing the Pod OS details. Linux users aren't very likely to read Guide for scheduling Windows containers in Kubernetes.

Sure, as mentioned in this comment, I'll open PR once k/k opens.

[ravisantoshgudimetla]

@sftim - Any other concerns before we merge this PR?

[sftim]

Sure, as mentioned in this comment, I'll open PR once k/k opens.

I was asking for an update to the Pod concept guide. I also want an update to the OpenAPI docs for the Pod API (which then ends up updating Pod), but those concept docs changes are needed ahead of the 1.25 release.

With only rare exceptions, we expect to have GA features graduate to stable with documentation already in place.

[ravisantoshgudimetla]

I was asking for an update to the Pod concept guide. I also want an update to the OpenAPI docs for the Pod API (which then ends up updating Pod), but those concept docs changes are needed ahead of the 1.25 release.

To be clear, you want this entire description of this field to be moved to pod concepts doc?

[sftim]

To be clear, you want this entire description of this field to be moved to pod concepts doc?

You can do either a:

• move
• copy

The outcome must be that when someone reads https://kubernetes.io/docs/concepts/workloads/pods/, they are aware that you can specify the name of the OS that this Pod is expected to run on.

[ravisantoshgudimetla]

To be clear, you want this entire description of this field to be moved to pod concepts doc?

You can do either a:

• move
• copy

The outcome must be that when someone reads https://kubernetes.io/docs/concepts/workloads/pods/, they are aware that you can specify the name of the OS that this Pod is expected to run on.

Opened another PR: #35985

[marosset]

LGTM (from sig-windows)
We can keep the Windows specific behaviors documented at https://kubernetes.io/docs/concepts/windows/user-guide/ and the more general behavior documented at https://kubernetes.io/docs/concepts/workloads/pods/ (which is added in a #35985 )

[ravisantoshgudimetla]

@sftim - Any other concerns before we merge this PR?

[reylejano]

applying lgtm from marosett in this comment
/lgtm

/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: reylejano

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:

• content/en/OWNERS [reylejano]

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]

LGTM label has been added.

Git tree hash: 8d7162f02948aabd15f92584bdeaa1ce2ed29577