Bug 1788810

Summary:	"Custom" in label "flavor.template.kubevirt.io/Custom" should be in lowercase
Product:	OpenShift Container Platform	Reporter:	Guohua Ouyang <gouyang>
Component:	Console Kubevirt Plugin	Assignee:	Filip Krepinsky <fkrepins>
Status:	CLOSED ERRATA	QA Contact:	Nelly Credi <ncredi>
Severity:	low	Docs Contact:
Priority:	low
Version:	4.3.0	CC:	aos-bugs, fkrepins, gouyang, rhrazdil
Target Milestone:	---	Keywords:	Reopened
Target Release:	4.5.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Deprecated Functionality
Doc Text:	flavor.template.kubevirt.io/Custom label is no longer used to identify custom flavors	Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-07-13 17:13:00 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 Guohua Ouyang 2020-01-08 07:26:26 UTC

Description of problem:
$ oc get vm test --show-labels
NAME   AGE   RUNNING   VOLUME   LABELS
test   10m   true               app=test,flavor.template.kubevirt.io/Custom=true,os.template.kubevirt.io/centos6.0=true,vm.kubevirt.io/template-namespace=default,vm.kubevirt.io/template.revision=1,vm.kubevirt.io/template.version=v0.8.1,vm.kubevirt.io/template=uploaddisk,workload.template.kubevirt.io/server=true


Version-Release number of selected component (if applicable):
4.3.0-0.nightly-2020-01-06-005750

How reproducible:


Steps to Reproduce:
1. Create a VM with custom Flavor and then check the labels.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Tomas Jelinek 2020-01-15 11:53:01 UTC

@Guohua: Im not sure - why should it be lowercase?

Comment 2 Guohua Ouyang 2020-01-17 02:04:08 UTC

The whole page are lowercase except once letter is uppercase, I thought it should be lowercase for cosmetic.
If it's not necessary, free to close this bug.

Comment 3 Tomas Jelinek 2020-01-20 09:22:36 UTC

If it is just cosmetics I will close it for now. Changing it would mean that we should have some logic to handle VMs created before this change and also the ones after this change. Not too hard but I would like to avoid it if there is no practical reason for that.

Comment 4 Filip Krepinsky 2020-04-30 15:41:16 UTC

We have to align with vm-import-operator project which will be using this label as well in lowercase variant: https://github.com/kubevirt/vm-import-operator/pull/168

We can add logic to our UI to accept both legacy 'Custom and new 'custom' labels. We should also fallback to Custom when this label is missing

Comment 5 Filip Krepinsky 2020-05-04 12:14:48 UTC

We concluded that there is no value in having this label as it is not supported by common templates. We should fallback to the same behaviour as when we the label is missing. We should remove the use of this label in the UI and in vm-import-operator.

Comment 9 Radim Hrazdil 2020-05-20 14:21:33 UTC

Verified that the label is not used anymore.
Verified in console release-4.5 branch commit: 5994c64ee529b650bae348ef78ebc23dca8db5c5

Comment 11 errata-xmlrpc 2020-07-13 17:13:00 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:2409