Bug 1464199 - drop m2crypto requirements on ovirt-host-deploy execution
Summary: drop m2crypto requirements on ovirt-host-deploy execution
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-host-deploy
Classification: oVirt
Component: Plugins.VDSM
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:
Blocks: 1471196 1478904
TreeView+ depends on / blocked
 
Reported: 2017-06-22 15:52 UTC by Sandro Bonazzola
Modified: 2019-10-09 09:41 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
undefined
Clone Of:
Environment:
Last Closed: 2018-01-12 12:56:57 UTC
oVirt Team: Integration
Embargoed:
rule-engine: ovirt-4.2+
lsvaty: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 78539 0 master MERGED ovirt-release-master: add m2crypto back on node 2017-06-22 16:00:35 UTC
oVirt gerrit 81390 0 master POST plugins: vdsm: Remove all references to m2crypto 2017-09-04 07:05:17 UTC
oVirt gerrit 103991 0 master MERGED remove m2crypto dependency 2019-10-09 09:56:19 UTC

Description Sandro Bonazzola 2017-06-22 15:52:18 UTC
vdsm dropped dependency on m2crypto in https://gerrit.ovirt.org/76027 without removing its runtime requirement for certificate enrollment in ovirt-host-deploy.
Result is that ovirt-node can't be added anymore as hypervisor since m2crypto is not installed anymore on ovirt-node not being required by vdsm and fails running certificate enrollment.

As temporary solution we're forcing m2crypto installation on ovirt-node but since we are abandoning m2crypto in other projects we should go ahead and remove here as well.

Comment 1 Sandro Bonazzola 2017-09-04 05:59:46 UTC
Didi, can you please check if solution for bug #1471196 solved this one as well?

Comment 2 Yedidyah Bar David 2017-09-04 06:54:47 UTC
(In reply to Sandro Bonazzola from comment #1)
> Didi, can you please check if solution for bug #1471196 solved this one as
> well?

Almost.

We still have there code to install m2crypto, and code to use it, but the code to use is disabled by the patch to bug #1471196.

As discussed in private, it doesn't seem like we'll return to m2crypto for host-deploy, so I'd rather remove all code. We can always copy from git history if needed.

Comment 3 Pavol Brilla 2018-01-08 14:51:43 UTC
# yum deplist ovirt-host-deploy | grep m2crypto
# 


Host added to engine succesfully


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