Bug 1241410 - [RFE]- Add 'passthrough' column to the vNIC Profiles sub tab under Networks main tab
Summary: [RFE]- Add 'passthrough' column to the vNIC Profiles sub tab under Networks m...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: RFEs
Version: ---
Hardware: x86_64
OS: Linux
medium
low
Target Milestone: ovirt-4.2.0
: 4.2.0
Assignee: Ales Musil
QA Contact: Michael Burman
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-07-09 07:46 UTC by Michael Burman
Modified: 2019-04-28 14:27 UTC (History)
10 users (show)

Fixed In Version: ovirt-engine-4.2.0-0.0.master.20170901193740.git7900511.el7.centos.noarch.rpm
Clone Of:
Environment:
Last Closed: 2017-12-20 11:26:34 UTC
oVirt Team: Network
Embargoed:
rule-engine: ovirt-4.2+
gklein: testing_plan_complete-
ylavi: planning_ack+
rule-engine: devel_ack+
rule-engine: testing_ack+

Attachments (Terms of Use)
screenshot (55.72 KB, image/png)
2015-07-09 07:46 UTC, Michael Burman 		no flags Details
new screehosts for vNIC profile table in the new UI design (237.69 KB, application/x-gzip)
2017-06-20 07:26 UTC, Michael Burman 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 79252 0 master MERGED frontend: Add passthrough column to vNIC profiles 2017-07-12 15:39:27 UTC
oVirt gerrit 79260 0 master MERGED frontend: Fixed typo in passthrough 2017-07-12 14:55:09 UTC
oVirt gerrit 80679 0 master MERGED frontend: Add passthrough column under vNIC profiles sub tab 2017-08-22 13:13:45 UTC

Description Michael Burman 2015-07-09 07:46:44 UTC 
Created attachment 1050150 [details]
screenshot

Description of problem:
[RFE]- Add 'passthrough' column to the vNIC Profiles sub tab under Networks main tab.

Today we no indication that vNIC Profile is a 'passthrough' one, adding column to the vNIC Profiles sub tab will be a good improvement, just like we have columns for Qos Names and Port mirroring.
 

Version-Release number of selected component (if applicable):
3.6.0-0.0.master.20150627185750.git6f063c1.el6
Comment 1 Michael Burman 2017-06-20 07:25:12 UTC 
Hi Greg,

Now in the new UI design we exposed the whole vNIC profiles list, but we still missing some indication for 'passthrough' vNICs. This info only available when going to the edit vNIC profile dialog and i already requested to expose this info by new column in this BZ.

Is it possible to add such column or any other indication that this is a 'passthrough' vNIC in the new UI? 

Thanks,
Comment 2 Michael Burman 2017-06-20 07:26:15 UTC 
Created attachment 1289453 [details]
new screehosts for vNIC profile table in the new UI design
Comment 3 Greg Sheremeta 2017-06-20 20:23:01 UTC 
Sure, it's the same work as it was in old UI. Still on network team :)
Comment 4 Michael Burman 2017-06-21 04:52:14 UTC 
Great, than you Greg)
Dan, can we please do it for 4.2?
Comment 5 Yaniv Kaul 2017-06-21 04:58:24 UTC 
(In reply to Michael Burman from comment #4)
> Great, than you Greg)
> Dan, can we please do it for 4.2?

Why?
Comment 6 Michael Burman 2017-06-21 05:37:27 UTC 
What do you mean why?

Because it's important information and we don't have any indication about it outside the edit dialog. It displayed as regular vNIC, but it's not and we should show it by column or by any other way.  

Why you indicate any other property of the vNIC, such as VM QoS, Network Filter or Port Mirroring...?  because it's important.
Comment 7 Michael Burman 2017-08-16 06:38:14 UTC 
Hello Ales, 

The 'passthrough' column was added only for the main tab vNIC profile view, but not for the vNIC profiles sub tab under a spesific network.
Are we ok with it? i actually wanted this column to be added on both places.
Now we have inconsistent between the two places.
Can we do it?
Comment 8 Ales Musil 2017-08-16 06:42:16 UTC 
Hi Michael,

Sure I can add that to the other place too.
Comment 9 Michael Burman 2017-08-16 07:01:56 UTC 
(In reply to Ales Musil from comment #8)
> Hi Michael,
> 
> Sure I can add that to the other place too.

Thank you!
Comment 10 Michael Burman 2017-09-03 05:54:23 UTC 
Verified on - 4.2.0-0.0.master.20170901193740.git7900511.el7.centos
Comment 11 Sandro Bonazzola 2017-12-20 11:26:34 UTC 
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, 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.