Bug 1380639

Summary: [RFE] Split ui plugin into separate package
Product: [oVirt] cockpit-ovirt Reporter: Fabian Deutsch <fdeutsch>
Component: RFEAssignee: Marek Libra <mlibra>
Status: CLOSED WONTFIX QA Contact: dguo
Severity: high Docs Contact:
Priority: high    
Version: ---CC: bugs, cshao, fdeutsch, michal.skrivanek, rbarry, sbonazzo, weiwang, ycui
Target Milestone: ---Keywords: FutureFeature
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-10 13:54:44 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Virt RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Fabian Deutsch 2016-09-30 08:46:46 UTC
Description of problem:
Currently the ui plugin resides in the same package as the cockpit plugin.

It would be good for shipping and code separation reasons to split this part into a separate package.

Comment 1 Michal Skrivanek 2016-10-01 06:16:33 UTC
What actual issue does it cause?

Comment 2 Fabian Deutsch 2016-10-01 20:14:48 UTC
Mainly packaging reasons: The ui plugin does not share any code with the cockpit bits.
For easier packaging and an easier release process it would be good to separate them.

Comment 4 Michal Skrivanek 2017-02-01 08:54:55 UTC
it can be removed in 4.2, replaced with proper subtab directly in engine code

Comment 5 dguo 2017-02-28 03:57:25 UTC
(In reply to Michal Skrivanek from comment #4)
> it can be removed in 4.2, replaced with proper subtab directly in engine code

QE need to clarify that there will be no cockpit-ovirt-uiplugin.noarch package, and will find such "cockpit" tab on ovirt-engine webadmin directly, right?

Comment 6 Michal Skrivanek 2017-02-28 11:11:14 UTC
yes, there will be no such package.
However whether there will be a subtab or a different way how to open cockpit is to be decided. It was suggested to replace that with a hyperlink opening new window with cockpit, or a menu item in Hosts main tab to do that. Adding Sandro to share his thoughts since cockpit interface is mandatory for node configuration now

Comment 7 Sandro Bonazzola 2017-03-07 13:17:46 UTC
(In reply to Michal Skrivanek from comment #6)
> yes, there will be no such package.
> However whether there will be a subtab or a different way how to open
> cockpit is to be decided. It was suggested to replace that with a hyperlink
> opening new window with cockpit, or a menu item in Hosts main tab to do
> that. Adding Sandro to share his thoughts since cockpit interface is
> mandatory for node configuration now

Not sure what's the question here.
This bug is just moving the engine ui plugin for cockpit out of cockpit-ovirt to its own git repo.

The design of the UI plugin itself should be discussed in a RFE or in devel mailing list I think. On my side I've no strong preference on its implementation.

Comment 8 Michal Skrivanek 2017-03-10 13:54:44 UTC
we'll drop the plugin and replace with a better suited solution.
tracked in https://trello.com/c/UrPaq9Oo