Bug 1780269

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 WONTFIX QA Contact: Yadan Pei <yapei>
Severity: low Docs Contact:
Priority: unspecified    
Version: 4.3.0CC: aos-bugs, hasha, jokerman, yanpzhan, yapei
Target Milestone: ---   
Target Release: 4.3.z   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1779858 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-01 13:02:05 UTC Type: ---
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: 1779858    
Bug Blocks:    

Description Samuel Padgett 2019-12-05 15:37:43 UTC
+++ This bug was initially created as a clone of Bug #1779858 +++

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 shahan 2019-12-06 09:46:23 UTC
After remove "authSecret" from "specDescriptor" in CSV.
the field shows "red asterisk" as required field and it used the textbox not the dropdown
However the field did not show "red asterisk" when  I dont remove "Auth Secret" in CSV.
Does it the expected behavior?

Comment 3 Yanping Zhang 2019-12-10 03:33:02 UTC
I tested on OCP 4.3 with payload 4.3.0-0.nightly-2019-12-09-181855, and had the same results as Comment 2. 
And the result is also the same with the reproduce steps in bug description. I'm confused if the fix in pr is correct or enough.
Tested steps:
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

I'd like to assign back first, pls feel free to change again.

Comment 5 Samuel Padgett 2020-05-01 13:02:05 UTC
This will be fixed in 4.4 under Bug #1779858. As it's low severity, we don't plan to backport.