Bug 1380639 - [RFE] Split ui plugin into separate package
Summary: [RFE] Split ui plugin into separate package
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: cockpit-ovirt
Classification: oVirt
Component: RFE
Version: ---
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: ---
Assignee: Marek Libra
QA Contact: dguo
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-30 08:46 UTC by Fabian Deutsch
Modified: 2022-02-25 07:50 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-03-10 13:54:44 UTC
oVirt Team: Virt


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-44860 0 None None None 2022-02-25 07:50:28 UTC

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


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