Bug 1879382 - Documentation gives example of deprecated priorityClass configuration apiVersion - 46hack
Summary: Documentation gives example of deprecated priorityClass configuration apiVers...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Documentation
Version: 4.5
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: 4.6.0
Assignee: Michael Burke
QA Contact: Sunil Choudhary
Vikram Goyal
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-09-16 07:08 UTC by Luis Martinho
Modified: 2020-10-28 19:40 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-10-28 19:40:03 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.