Bug 1326269 - Host provisioning on rhev CR failed with error: OVIRT::OvirtException: NIC name required for add
Summary: Host provisioning on rhev CR failed with error: OVIRT::OvirtException: NIC na...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Provisioning
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium vote
Target Milestone: Unspecified
Assignee: satellite6-bugs
QA Contact: Sachin Ghai
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-04-12 09:52 UTC by Sachin Ghai
Modified: 2019-04-01 20:27 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-08-02 20:59:14 UTC
Target Upstream Version:


Attachments (Terms of Use)
Error is raised on UI, but it doesn't point which interface has the issue. Ideally interface tab should be highlighted with red border (64.05 KB, image/png)
2016-04-12 09:52 UTC, Sachin Ghai
no flags Details
Name field is not set as mandatory with * (57.94 KB, image/png)
2016-04-12 09:53 UTC, Sachin Ghai
no flags Details

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.


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