Bug 1191953

Summary: user shouldn't be allowed to create alias and bond interfaces without specifying identifer
Product: Red Hat Satellite Reporter: Sachin Ghai <sghai>
Component: WebUIAssignee: Marek Hulan <mhulan>
WebUI sub component: Foreman QA Contact: Sachin Ghai <sghai>
Status: CLOSED ERRATA Docs Contact:
Severity: medium    
Priority: medium CC: bkearney, jmontleo
Version: 6.1.0Keywords: Triaged
Target Milestone: Unspecified   
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/9429
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-08-12 05:24:46 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:    
Bug Blocks: 1193987    
Attachments:
Description Flags
see interface can be created without specifying identifier
none
creating an alias interface without identifier throws error on UI
none
creating a bond interface without identifier throws error on UI none

Description Sachin Ghai 2015-02-12 09:54:02 UTC
Created attachment 990824 [details]
see interface can be created without specifying identifier

Description of problem:
I can create alias and bond interfaces without specifying identifier. I think its required to link the virtual interface with the physical one.

If we don't pass identifier then I think it won't be configured on provisioned host via config template.

Version-Release number of selected component (if applicable):
Satellite-6.1.0-RHEL-6-20150210.0
Sat6.1.0 beta snap2

How reproducible:
always

Steps to Reproduce:
1. Add_interface of type Bond and alias
2.
3.

Actual results:
Alias and bond Interfaces can be created without interface identifier

Expected results:
User should get validation error as identifier are required for linking purpose.

Additional info:

Comment 2 Marek Hulan 2015-02-18 14:14:35 UTC
Created redmine issue http://projects.theforeman.org/issues/9429 from this bug

Comment 3 Bryan Kearney 2015-02-19 15:07:57 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/9429 has been closed
-------------
Marek Hulán
Applied in changeset commit:4feb1576f820b08712afd58a41b01cdfc2f9d600.

Comment 6 Bryan Kearney 2015-02-25 11:05:00 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/9429 has been closed
-------------
Marek Hulán
Applied in changeset commit:4feb1576f820b08712afd58a41b01cdfc2f9d600.

Comment 9 Sachin Ghai 2015-03-09 08:58:36 UTC
Verification of this bz is blocked due to bz 1198925.

Comment 10 Sachin Ghai 2015-03-17 07:06:34 UTC
verified with:

sat6.1 beta snap6 compose2 (Satellite-6.1.0-RHEL-6-20150311.1).

Now if user creates managed alias and bond interfaces without specifying identifier, UI throws error that identifier can't be blank. Thanks for fixing this. Please see the screenshot for verification.

Comment 11 Sachin Ghai 2015-03-17 07:07:32 UTC
Created attachment 1002667 [details]
creating an alias interface without identifier throws error on UI

Comment 12 Sachin Ghai 2015-03-17 07:08:27 UTC
Created attachment 1002668 [details]
creating a bond interface without identifier throws error on UI

Comment 13 Bryan Kearney 2015-08-11 13:31:24 UTC
This bug is slated to be released with Satellite 6.1.

Comment 14 errata-xmlrpc 2015-08-12 05:24:46 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/RHSA-2015:1592