Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1911787

Summary:	Default StorageClass/AccessMode/VolumeMode should be changed on UI
Product:	OpenShift Container Platform	Reporter:	Guohua Ouyang <gouyang>
Component:	Console Kubevirt Plugin	Assignee:	Tomas Jelinek <tjelinek>
Status:	CLOSED NOTABUG	QA Contact:	Guohua Ouyang <gouyang>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	4.7	CC:	aos-bugs, glekner, mschatzm, oyahud, tjelinek, yzamir
Target Milestone:	---
Target Release:	---
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-12-31 10:05:50 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-12-31 08:55:01 UTC

Description of problem:
Current, the default StorageClass/AccessMode/VolumeMode is
- "Standard/RWO/Filesystem" in "Add disk" page in customize mode
- "Standard/RWO/Block" in "Boot source" step in new wizard
- "Standard/RWO/Block" in "Add source" in template list

There are several issues to use them as default, we should not deliver something not work by default.

Version-Release number of selected component (if applicable):
OCP 4.7

How reproducible:


Steps to Reproduce:
1. 
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Yaacov Zamir 2020-12-31 09:23:54 UTC

AFAIU, the default StorageClass/AccessMode/VolumeMode should use the data from:

1.
a. StorageClass - default storage class.
b. AccessMode/VolumeMode - from config.

2.
If template contain AccessMode/VolumeMode this data should be respected and used instead of data in config map.

3. Alert user and stop wizard, on this cases:
a. if no default storage class found.  
b. if no suggested AccessMode/VolumeMode for selected storage class found.
c. if selected AccessMode/VolumeMode does not match PVC.source AccessMode/VolumeMode.

@Omer hi,
Can you comment on the above suggestion to auto set StorageClass/AccessMode/VolumeMode in the UI wizard, do you think it's currect ?

cc:// @Tomas @Gilad @Matan

p.s.
We have other bugs for that need to have a clear way to define how to set StorageClass/AccessMode/VolumeMode, for example:
https://bugzilla.redhat.com/show_bug.cgi?id=1911387 - no storage class defined when no default storage class found.
https://bugzilla.redhat.com/show_bug.cgi?id=1911382 - source AccessMode/VolumeMode not matching created pvc AccessMode/VolumeMode
https://bugzilla.redhat.com/show_bug.cgi?id=1911418 - default storage class is not used

Comment 2 Omer Yahud 2020-12-31 09:50:15 UTC

We had discussed this before and the decision was that the UI is responsible to set the storage parameters according to the storage config map (forgot it's name)

Comment 3 Yaacov Zamir 2020-12-31 10:05:50 UTC

The StorageClass/AccessMode/VolumeMode is pre fill using `kubevirt-storage-class-defaults` - config map
This is the agreed apon way.

closing as not a bug.

Comment 4 Gilad Lekner 2021-01-19 09:39:05 UTC

clearing redundant need info

Comment 5 Red Hat Bugzilla 2023-09-15 00:56:53 UTC

The needinfo request[s] on this closed bug have been removed as they have been unresolved for 500 days