Bug 1879382

Summary: Documentation gives example of deprecated priorityClass configuration apiVersion - 46hack
Product: OpenShift Container Platform Reporter: Luis Martinho <lmartinh>
Component: DocumentationAssignee: Michael Burke <mburke>
Status: CLOSED CURRENTRELEASE QA Contact: Sunil Choudhary <schoudha>
Severity: low Docs Contact: Vikram Goyal <vigoyal>
Priority: low    
Version: 4.5CC: aos-bugs, jokerman, schoudha
Target Milestone: ---   
Target Release: 4.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-10-28 19:40:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Luis Martinho 2020-09-16 07:08:10 UTC 
Should the documentation example to create the priorityClass be changed to reflect in the apiVersion the most recent API? scheduling.k8s.io/v1 instead of scheduling.k8s.io/v1beta1

In https://docs.openshift.com/container-platform/4.6/nodes/pods/nodes-pods-priority.html#nodes-pods-priority

Example to create the priorityClass:

cat <<EOF > pc.yaml
apiVersion: scheduling.k8s.io/v1beta1
kind: PriorityClass
metadata:
name: high-priority
value: 1000000
globalDefault: false
description: "This priority class should be used for XYZ service pods only."
EOF

-Creating the object gives the warning:
[root@bastionhost4 a-ipi-4.6.f5]# oc create -f pc.yaml
W0915 17:55:28.810304    8338 warnings.go:67] scheduling.k8s.io/v1beta1 PriorityClass is deprecated in v1.14+, unavailable in v1.22+; use scheduling.k8s.io/v1 PriorityClass
priorityclass.scheduling.k8s.io/high-priority created
Comment 1 Michael Burke 2020-09-30 20:16:42 UTC 
Luis --

Can you take a look at my PR?

Thanks,
Michael

https://github.com/openshift/openshift-docs/pull/26005
Comment 2 Luis Martinho 2020-10-01 08:17:58 UTC 
Hi Michael,

The PR contains the change I suggested, so it looks good.

Thank you
Luis
Comment 4 Michael Burke 2020-10-28 19:40:03 UTC 
Changes are live: https://docs.openshift.com/container-platform/4.6/nodes/pods/nodes-pods-priority.html#nodes-pods-priority-configuring_nodes-pods-priority