Bug 1779858

Summary: Create Operand: Required fields are missing "red asterisk" after specDescriptor overwrite
Product: OpenShift Container Platform Reporter: Samuel Padgett <spadgett>
Component: Management ConsoleAssignee: Samuel Padgett <spadgett>
Status: CLOSED ERRATA QA Contact: Yadan Pei <yapei>
Severity: low Docs Contact:
Priority: unspecified    
Version: 4.3.0CC: aos-bugs, hasha, jokerman, yanpzhan
Target Milestone: ---   
Target Release: 4.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Previously, some fields that have OLM descriptors were missing the required red asterisk in the OpenShift console "Create Operand" form. The bug has been fixed, and the fields are correctly treated as a required.
Story Points: ---
Clone Of:
: 1780269 (view as bug list) Environment:
Version: 4.4.0-0.ci-2019-12-04-051457 Cluster ID: 73e38a63-0543-4277-9bd4-018c287764e8 Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:71.0) Gecko/20100101 Firefox/71.0
Last Closed: 2020-05-13 21:54:07 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:
Bug Depends On: 1785102, 1790612    
Bug Blocks: 1780269    

Description Samuel Padgett 2019-12-04 21:08:38 UTC
https://jira.coreos.com/browse/CONSOLE-1927

All "required fields" generated based on OpenAPI Schema would show "red asterisk" for indicating as required fields.

It's noticed after those required fields got overwritten by specDescriptors, the "red asterisk" indicator for those required fields are missing.

See attached screenshots for reference.

Reproducible Steps:

    Installed Couchbase Operators to "Namespace A"
    Remove the specDescriptor for "authSecret" field from the CSV file
    Check out Creation form for CouchbaseCluster   
    --> See "authSecret" field shows "red asterisk" as required field

    Installed Couchbase Operators to "Namespace B"
    Check out Creation form for CouchbaseCluster   
    --> See "authSecret" field with specDescriptor's widget: "red asterisk" missing

Expected Behavior:

All required fields should show "red asterisk" as the indicator.

Comment 2 Yanping Zhang 2019-12-19 07:16:06 UTC
The verification is blocked, since can not create couchbase operator successfully, tracked in bug: https://bugzilla.redhat.com/show_bug.cgi?id=1785102

Comment 3 Yanping Zhang 2020-01-08 04:03:29 UTC
Checked on ocp4.3 with payload 4.3.0-0.nightly-2020-01-07-212456.
Following steps in description to test:
1.Installed Couchbase Operators to "Namespace A"
    Remove the specDescriptor for "authSecret" field from the CSV file
    Check out Creation form for CouchbaseCluster   
    --> See "authSecret" field shows "red asterisk" as required field
2.Installed Couchbase Operators to "Namespace B"
    Check out Creation form for CouchbaseCluster   
    --> See "authSecret" field with specDescriptor's widget: "red asterisk" missing
The results are the same with that in the description, seems the bug fix doesn't work.

Comment 4 Yanping Zhang 2020-01-08 04:10:47 UTC
Sorry, I should check the bug on OCP 4.4 not 4.3, ignore Comment 3.
Since the bug is still blocked by bug 1785102, will check when it can be tested.

Comment 6 Samuel Padgett 2020-02-07 22:13:52 UTC
There's a typo in the required field path in the original fix. Moving back to assigned.

Comment 8 shahan 2020-02-17 05:16:37 UTC
since couchbase operator cannot install sucessfully, here using cluster-logging instead.
1. install cluster-logging operator
2. goto ClusterLoggings creation form, noted the "Kibana Size"  has the red asterisk as required field.
3. change the displayName " Kibana Size" to "Kibana Sizetest" under specDescriptors in csv yaml.
         specDescriptors:
          - description: The desired number of Kibana Pods for the Visualization component
            displayName: Kibana Sizetest
            path: visualization.kibana.replicas
            x-descriptors:
              - 'urn:alm:descriptor:com.tectonic.ui:podCount'
4. goto ClusterLoggings creation form, noted the "Kibana Sizetest"  still has the red asterisk as required field.
verified this bug.
4.4.0-0.nightly-2020-02-16-205302

Comment 10 errata-xmlrpc 2020-05-13 21:54:07 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:0581