Bug 1191419 - Move config classes into subpackage [NEEDINFO]
Summary: Move config classes into subpackage
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-node
Classification: oVirt
Component: General
Version: ---
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ovirt-3.6.0-rc
: 3.2
Assignee: Douglas Schilling Landgraf
QA Contact: Ying Cui
URL:
Whiteboard: node
Depends On:
Blocks: node-cockpit
TreeView+ depends on / blocked
 
Reported: 2015-02-11 10:06 UTC by Fabian Deutsch
Modified: 2016-02-02 14:40 UTC (History)
12 users (show)

Fixed In Version: ovirt-node-3.6.0-0.24.20151209gitc0fa931.el7ev
Doc Type: Bug Fix
Doc Text:
http://www.ovirt.org/Features/Node/subpackages
Clone Of:
Environment:
Last Closed: 2016-02-02 14:40:00 UTC
oVirt Team: Node
Embargoed:
ykaul: needinfo? (bazulay)
rule-engine: ovirt-3.6.0+
ylavi: planning_ack+
rule-engine: devel_ack+
ycui: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 38295 0 master MERGED spec: split sources in subpackages Never

Description Fabian Deutsch 2015-02-11 10:06:42 UTC
Description of problem:
Currently the ovirt-node package includes everything from the TUI and installer.
Because in future we want to use cockpit, we should split the "backend" config.defaults package into a subpackage, to be able to use it standalone.

That package has a couple of dependencies (like utils) which also need to get into that core package.

Version-Release number of selected component (if applicable):
3.5

Comment 1 Fabian Deutsch 2015-02-11 10:07:41 UTC
The idea is to separate the TUI/installer from the actual classes doing the configuration.

Comment 2 Fabian Deutsch 2015-02-12 11:52:06 UTC
After moving the configuration modules into a separate package, some important things to remember are:
1. Add necessary Requires on affected packages
2. Ensure that necessary dependencies  are also included (ovirt-node.utils)
3. Ensure that NO old code (ovirtnode.*) is required as a dependency.

Comment 3 Fabian Deutsch 2015-02-26 16:09:20 UTC
The premise is to only make packaging changes, no code changes for now.


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