Change in ovirt-engine[master]: packaging: check if ovirt-provider-ovn is installed based on...

Code Review gerrit at ovirt.org
Thu Jun 8 14:29:19 UTC 2017


>From Sandro Bonazzola <sbonazzo at redhat.com>:

Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: check if ovirt-provider-ovn is installed based on provider id
......................................................................


packaging: check if ovirt-provider-ovn is installed based on provider id

ovirt-provider-ovn will only be installed if not
previously installed by engine-setup.
The provider id saved in the postinstall file will
be used to make the check.

Note that this patch does not add a check against
a provider installation done outside engine-setup.
This will be handled separately.

Change-Id: Id8077f42a4f409e14e8341d46b93491211b7e329
Signed-off-by: Marcin Mirecki<mmirecki at redhat.com>
---
M packaging/setup/ovirt_engine_setup/engine/constants.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/network/ovirtproviderovn.py
2 files changed, 18 insertions(+), 5 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, approved
  Marcin Mirecki: Verified
  Jenkins CI: Passed CI tests
  Yedidyah Bar David: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/77779
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id8077f42a4f409e14e8341d46b93491211b7e329
Gerrit-PatchSet: 14
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list