Bug 1326269

Summary: Host provisioning on rhev CR failed with error: OVIRT::OvirtException: NIC name required for add
Product: Red Hat Satellite Reporter: Sachin Ghai <sghai>
Component: ProvisioningAssignee: satellite6-bugs <satellite6-bugs>
Status: CLOSED WONTFIX QA Contact: Sachin Ghai <sghai>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.2.0CC: dmoessne
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-08-02 20:59:14 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:
Attachments:
Description Flags
Error is raised on UI, but it doesn't point which interface has the issue. Ideally interface tab should be highlighted with red border
none
Name field is not set as mandatory with * none

Description Sachin Ghai 2016-04-12 09:52:30 UTC
Created attachment 1146314 [details]
Error is raised on UI, but it doesn't point which interface has the issue. Ideally interface tab should be highlighted with red border

Description of problem:
I was trying to provision a host on rhev instance and I configured couple of virtual interfaces while provisioning the host.

Host provisioning failed with error:

2016-04-12 05:39:55 [app] [W] Failed to create a compute rhevm1.satellite.lab.eng.rdu2.redhat.com (RHEV) instance sghaihost.katello-latest.satellite.lab.eng.rdu2.redhat.com: NIC name required for add
 |  
 | OVIRT::OvirtException: NIC name required for add
 | /opt/theforeman/tfm/root/usr/share/gems/gems/rbovirt-0.0.36/lib/rbovirt.rb:210:in `handle_fault'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/rbovirt-0.0.36/lib/rbovirt.rb:130:in `rescue in http_post'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/rbovirt-0.0.36/lib/rbovirt.rb:127:in `http_post'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/rbovirt-0.0.36/lib/client/vm_api.rb:57:in `add_interface'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/fog-1.37.0/lib/fog/ovirt/requests/compute/add_interface.rb:8:in `add_interface'


I think there are couple of issues:

1) if nic name is required then that field should marked as mandatory with '*'
2) UI should point the 'interfaces' tab with red border and if possible, also point specific interface where nic name was missing.

Version-Release number of selected component (if applicable):
sat6.2 beta snap7.1

How reproducible:


Steps to Reproduce:
1. create a host on rhev instance
2.
3.

Actual results:
 | OVIRT::OvirtException: NIC name required for add
 | /opt/theforeman/tfm/root/usr/share/gems/gems/rbovirt-0.0.36/lib/rbovirt.rb:210:in `handle_fault'


Expected results:
1) if nic name is required then that field should marked as mandatory with '*'
2) UI should point the 'interfaces' tab with red border and if possible, also point specific interface where nic name was missing.


Additional info:

Comment 1 Sachin Ghai 2016-04-12 09:53:10 UTC
Created attachment 1146315 [details]
Name field is not set as mandatory with *

Comment 3 Bryan Kearney 2016-07-26 15:25:24 UTC
Moving 6.2 bugs out to sat-backlog.

Comment 4 Bryan Kearney 2016-07-26 15:30:38 UTC
Moving 6.2 bugs out to sat-backlog.

Comment 7 Bryan Kearney 2018-08-02 20:59:14 UTC
Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in the product in the forseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.