Bug 1780269 - Create Operand: Required fields are missing "red asterisk" after specDescriptor overwrite
Summary: Create Operand: Required fields are missing "red asterisk" after specDescript...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Management Console
Version: 4.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: 4.3.z
Assignee: Samuel Padgett
QA Contact: Yadan Pei
URL:
Whiteboard:
Depends On: 1779858
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-12-05 15:37 UTC by Samuel Padgett
Modified: 2020-05-01 13:02 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
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
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift console pull 3689 0 'None' closed [release-4.3] Bug 1780269: Fixed the required field display issue for create operand form 2020-05-01 12:58:51 UTC

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.


Note You need to log in before you can comment on or make changes to this bug.