Bug 1555368 - Network prefix length value is pre-filled but not effective if the user doesn't retype it.
Summary: Network prefix length value is pre-filled but not effective if the user doesn...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: cockpit-ovirt
Classification: oVirt
Component: Hosted Engine
Version: 0.11.12
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.2.3
: ---
Assignee: Phillip Bailey
QA Contact: Yihui Zhao
URL:
Whiteboard:
: 1555366 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-03-14 14:50 UTC by Simone Tiraboschi
Modified: 2018-05-10 06:33 UTC (History)
12 users (show)

Fixed In Version: cockpit-ovirt-0.11.21-0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-05-10 06:33:41 UTC
oVirt Team: SLA
Embargoed:
rule-engine: ovirt-4.2+
yzhao: testing_ack+


Attachments (Terms of Use)
Screenshot 1 (85.41 KB, image/png)
2018-03-14 14:51 UTC, Simone Tiraboschi
no flags Details
Screenshot 2 (89.53 KB, image/png)
2018-03-14 14:52 UTC, Simone Tiraboschi
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 89467 0 master POST wizard: Make CIDR field required for static network configs 2018-03-26 21:27:01 UTC

Description Simone Tiraboschi 2018-03-14 14:50:46 UTC
Description of problem:
Network prefix length value is pre-filled but not effective if the user doesn't retype it.

Compare screenshot 1 with screenshot 2

Version-Release number of selected component (if applicable):
cockpit-ovirt-dashboard.noarch        0.11.16-1.el7.centos

How reproducible:
100%

Steps to Reproduce:
1. Try deploying hosted-engine from cockpit
2. Keep the default network prefix length value 
3. Double check it in preview tab

Actual results:
There is a default in the first tab but it got lost if the user doesn't retype it

Expected results:
The default is correctly consumed

Additional info:

Comment 1 Simone Tiraboschi 2018-03-14 14:51:47 UTC
Created attachment 1408046 [details]
Screenshot 1

Comment 2 Simone Tiraboschi 2018-03-14 14:52:19 UTC
Created attachment 1408047 [details]
Screenshot 2

Comment 3 Simone Tiraboschi 2018-03-14 15:19:07 UTC
*** Bug 1555366 has been marked as a duplicate of this bug. ***

Comment 4 Phillip Bailey 2018-03-14 15:25:33 UTC
(In reply to Simone Tiraboschi from comment #0)
> Description of problem:
> Network prefix length value is pre-filled but not effective if the user
> doesn't retype it.

> Actual results:
> There is a default in the first tab but it got lost if the user doesn't
> retype it

The text that's there is a placeholder, not a pre-filled value. It's the same as it is for the IP value. It's to show what type of value is expected to be entered there, but the field still has to be filled in.

> Expected results:
> The default is correctly consumed

We can add a default value or we can mark it as required, which will alert the user that it needs to be filled in.

Comment 5 Yihui Zhao 2018-03-19 09:28:11 UTC
Tested with 11.17, from the Comment 4, it is not a pre-filled value. 

Or consider to add a default value or mark it as required.

Comment 6 Yihui Zhao 2018-04-13 01:59:39 UTC
Tested with cockpit-ovirt-0.11.21-0.1, now the network prefix length value is marked as required, which will alert the user that it needs to be filled in.


So moving to verified.

Comment 7 Sandro Bonazzola 2018-05-10 06:33:41 UTC
This bugzilla is included in oVirt 4.2.3 release, published on May 4th 2018.

Since the problem described in this bug report should be
resolved in oVirt 4.2.3 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


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