Bug 1875794

Summary: PV encryption selection and min requirement should be guarded for OCP 4.6 + OCS 4.5 installs & should be disabled by default for OCS 4.6
Product: OpenShift Container Platform Reporter: Neha Berry <nberry>
Component: Console Storage PluginAssignee: Bipul Adhikari <badhikar>
Status: CLOSED ERRATA QA Contact: Neha Berry <nberry>
Severity: high Docs Contact:
Priority: unspecified    
Version: 4.6CC: aos-bugs, badhikar, jokerman, nthomas, ocs-bugs
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-27 16:37:52 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:
Attachments:
Description Flags
Install-page-UI none

Description Neha Berry 2020-09-04 11:48:02 UTC
Created attachment 1713769 [details]
Install-page-UI

Description of problem:
-----------------------------
OCS 4.6 will support Encryption and installation on 8CPU + 32GB RAM clusters.

Hence, these options should not show up in Install StorageCluster Service Page for OCP 4.6 + OCS 4.5 . They should be guarded completely.

Version-Release number of selected component (if applicable):
--------------------------------
OCP = 4.6.0-0.nightly-2020-08-31-220837
OCS = ocs-operator.v4.5.0-546.ci

How reproducible:
----------------
Always

Steps to Reproduce:
1. Install OCP 4.6
2. Install OCS 4.5 Storage Cluster Service and check the Install UI page.
3.

Actual results:
---------------

encyption is enabled by default in UI minimum requirement is shown - not valid for OCS 4.5

Comment 1 Neha Berry 2020-09-04 12:49:22 UTC
Also, @Bipul, as disucssed in chatroom, for OCP 4.6 + OCS 4.6, we should have default state of encryption as false/disabled. 

Adding it here for reference, as this PR would include that fix as well.

Comment 6 Neha Berry 2020-10-12 11:50:52 UTC
Thanks Bipul.

Attachment are similar to https://bugzilla.redhat.com/attachment.cgi?id=1720940


Verified the fix on 2 versions and the issue is resolved. Moving to verified state


1. OCP 4.6 + OCS 4.6


OCP = 4.6.0-0.nightly-2020-10-10-041109
OCS = ocs-operator.v4.6.0-590.ci (quay.io/rhceph-dev/ocs-registry:4.6.0-119.ci) as OCP 4.6 doesnt list the OCS operator for any version by default

Observation:
----------------
1. Encryption toggle is NOW available and is "Disabled" by default
2. minimal mode warning shows up if the total CPU <30 and total Mem <72 GiB. Doesn't show up if the minimum req. is met
3. The ? Note for Minimal deployment is removed permanently

______________________________________________________________________________

2. OCP 4.6 + OCS 4.5
OCP = 4.6.0-0.nightly-2020-10-10-041109
OCS = ocs-operator.v4.5.0-554.ci (4.5.0-rc6) as OCP 4.6 doesnt list the OCS operator for any version by default

Observation:
----------------
1. Encryption toggle is not  available
2. minimal mode warning does NOT show up even if the total CPU <30 and total Mem <72 GiB for the selected nodes
3. The "?" Note for Minimal deployment is removed permanently"?"

Comment 8 errata-xmlrpc 2020-10-27 16:37:52 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 (OpenShift Container Platform 4.6 GA Images), 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:4196