Bug 1461081 - Deploy ovirt-host package only instead of single packages
Summary: Deploy ovirt-host package only instead of single packages
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-host-deploy
Classification: oVirt
Component: General
Version: master
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ovirt-4.2.0
: 1.7.0
Assignee: Yedidyah Bar David
QA Contact: Pavol Brilla
URL:
Whiteboard:
Depends On: 1460609
Blocks: 1478904
TreeView+ depends on / blocked
 
Reported: 2017-06-13 13:20 UTC by Sandro Bonazzola
Modified: 2017-12-20 11:40 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-12-20 11:40:14 UTC
oVirt Team: Integration
Embargoed:
rule-engine: ovirt-4.2+
pbrilla: testing_plan_complete-
ylavi: planning_ack+
sbonazzo: devel_ack+
lsvaty: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 78306 0 master MERGED packaging: consume ovirt-host package 2017-08-28 14:41:43 UTC

Description Sandro Bonazzola 2017-06-13 13:20:27 UTC
A new package ovirt-host has been introduced in 4.2 consolidating all the requirements for running an host.
Remove all package installation / upgrades and just install / update this new package for pulling in required packages.

Comment 1 Sandro Bonazzola 2017-06-19 14:45:11 UTC
Yaniv, Martin, is it desired to have all packages currently optional already installed as dependency of ovirt-host like on oVirt Node? Or are we talking only about the absolutely required ones?

Comment 2 Martin Perina 2017-06-19 14:57:53 UTC
(In reply to Sandro Bonazzola from comment #1)
> Yaniv, Martin, is it desired to have all packages currently optional already
> installed as dependency of ovirt-host like on oVirt Node? Or are we talking
> only about the absolutely required ones?

I'm not sure I understand the question. ovirt-host should depend on all packages which we have in ovirt-host-deploy and/or host-upgrade-manager packages list, so just installing/upgrading ovirt-host will install all required dependencies. Or do we have some other dependencies not included in those list?

Comment 3 Sandro Bonazzola 2017-06-19 15:02:15 UTC
example: vdsm-gluster is an optional package which is installed only if explicitly requested by the engine during the deployment.
Same for openstack-neutron, kdump, kernel-rt, tuned, ovirt-hosted-engine-setup, and so on.

Should all these optional package move to ovirt-host as mandatory dependency or be kept in ovirt-host-deploy as optional dependency?

As a side note to myself, in order to not introduce regressions, we'll need to backport ovirt-host package down to 3.6 in order to allow to replace existing hosts in supported clusters levels.

Comment 4 Martin Perina 2017-06-19 15:16:01 UTC
(In reply to Sandro Bonazzola from comment #3)
> example: vdsm-gluster is an optional package which is installed only if
> explicitly requested by the engine during the deployment.
> Same for openstack-neutron, kdump, kernel-rt, tuned,
> ovirt-hosted-engine-setup, and so on.

Well, kdump is installed/enabled by default on EL7, but other mentioned packages are most probably not mandatory. So I'd probably not add them to ovirt-host now (but if needed/decided we can add them later). Do you agree?

> 
> Should all these optional package move to ovirt-host as mandatory dependency
> or be kept in ovirt-host-deploy as optional dependency?
> 
> As a side note to myself, in order to not introduce regressions, we'll need
> to backport ovirt-host package down to 3.6 in order to allow to replace
> existing hosts in supported clusters levels.

+1 on that, it will make 3.6 hosts handling in 4.2 a lot easier on the host upgrade manager side and it's also required to fix BZ1460954 nicely

Comment 5 Sandro Bonazzola 2017-06-19 15:24:10 UTC
(In reply to Martin Perina from comment #4)

> Well, kdump is installed/enabled by default on EL7, but other mentioned
> packages are most probably not mandatory. So I'd probably not add them to
> ovirt-host now (but if needed/decided we can add them later). Do you agree?

Ok, let's start with the mandatory ones and do the others case by case.

Comment 6 Yaniv Lavi 2017-06-21 08:51:39 UTC
(In reply to Sandro Bonazzola from comment #5)
> (In reply to Martin Perina from comment #4)
> 
> > Well, kdump is installed/enabled by default on EL7, but other mentioned
> > packages are most probably not mandatory. So I'd probably not add them to
> > ovirt-host now (but if needed/decided we can add them later). Do you agree?
> 
> Ok, let's start with the mandatory ones and do the others case by case.

I don't agree here. I would want the node package set and the RHEL package set to be the same. If the size of the node is not an issue, it will really not be an issue for a el7 host.

Comment 7 Martin Perina 2017-06-21 09:12:30 UTC
(In reply to Yaniv Lavi from comment #6)
> (In reply to Sandro Bonazzola from comment #5)
> > (In reply to Martin Perina from comment #4)
> > 
> > > Well, kdump is installed/enabled by default on EL7, but other mentioned
> > > packages are most probably not mandatory. So I'd probably not add them to
> > > ovirt-host now (but if needed/decided we can add them later). Do you agree?
> > 
> > Ok, let's start with the mandatory ones and do the others case by case.
> 
> I don't agree here. I would want the node package set and the RHEL package
> set to be the same. If the size of the node is not an issue, it will really
> not be an issue for a el7 host.

That's difference between Node and RHEL philosophy, on node you have everything installed and only configure when needed, on RHEL you have minimum installed and when needed install and configure it.

If installation of those additional dependencies doesn't have any other implications, we can add them to ovirt-host, but that's a much bigger change in host installation behaviour than just add mandatory deps to ovirt-host. So I'd probably check each optional dependency one-by-one before switching it to mandatory ...

Comment 8 Sandro Bonazzola 2017-06-21 11:03:58 UTC
Also note that this means we need at least another meta package for including ovirt-hosted-engine-setup. Otherwise we'll have a circular dependency:
ovirt-host -> ovirt-hosted-engine-setup -> ovirt-host.

Something like ovirt-host -> ovirt-hosted-engine-setup -> ovirt-host-core

Or reverting the changes done for bug #1461083 and have ovirt-host -> ovirt-hosted-engine-setup -> whatever is needed not requiring ovirt-host.

Comment 9 Yaniv Lavi 2017-06-21 13:51:51 UTC
(In reply to Sandro Bonazzola from comment #8)
> Also note that this means we need at least another meta package for
> including ovirt-hosted-engine-setup. Otherwise we'll have a circular
> dependency:
> ovirt-host -> ovirt-hosted-engine-setup -> ovirt-host.
> 
> Something like ovirt-host -> ovirt-hosted-engine-setup -> ovirt-host-core
> 
> Or reverting the changes done for bug #1461083 and have ovirt-host ->
> ovirt-hosted-engine-setup -> whatever is needed not requiring ovirt-host.

Not sure why this would happen. Fix the deps to only require this once.

Comment 10 Pavol Brilla 2017-12-15 10:24:21 UTC
ovirt-host installation on host during host-deploy, succesfull

Comment 11 Sandro Bonazzola 2017-12-20 11:40:14 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.