So, according to the thread we have a few action items:

- Decide if we'll drop export domain and iso domain in 4.4

- Move requirements from safelease to vdsm for numactl, dmidecode and virt-v2v if not already done

- Elect a maintainer for safelease for 4.3 scope

- Deprecate safelease in 4.3 and remove it on master if we agree on removing iso and export domain in 4.4

Il giorno mar 2 apr 2019 alle ore 18:14 Nir Soffer <nsoffer@redhat.com> ha scritto:

On Tue, Apr 2, 2019 at 6:40 PM Dan Kenigsberg <danken@redhat.com> wrote:

On Tue, Apr 2, 2019 at 6:07 PM Nir Soffer <nsoffer@redhat.com> wrote:

On Tue, Apr 2, 2019 at 5:00 PM Sandro Bonazzola <sbonazzo@redhat.com> wrote:

Hi,

I stumbled upon safelease package, introduced in oVirt 3.6.

I realigned the spec file with Fedora Rawhide: https://gerrit.ovirt.org/#/c/99123/

and then I stopped working on it and decided to open a thread here.

safelease package is required in vdsm.

I searched for the home page for this package since it moved and found: https://ovirt.org/develop/developer-guide/vdsm/safelease.html

This says that sanlock is meant to obsolete safelease.

I'm assuming that safelease was used in 3.6 and replaced later by sanlock then kept for backward compatibility.

In 4.3 we dropped support for 3.6 level clusters, is this package still needed?

safelease is our clusterlock with V1 storage domains - export and iso domains.

https://github.com/oVirt/vdsm/blob/f433ed5aaf67729b787cf82ee21b0f17af968be4/lib/vdsm/storage/clusterlock.py#L127

https://github.com/oVirt/vdsm/blob/master/lib/vdsm/storage/sd.py#L320

Once we remove these domains we can remove also safelease.

If it's still needed, why is it requiring:

# Numactl is not available on s390[x] and ARM

%ifnarch s390 s390x %{arm}

Requires: numactl

%endif

%ifarch x86_64

Requires: python2-dmidecode

Requires: dmidecode

Requires: virt-v2v

%endif

These are hacks Yaniv added so we can make vdsm noarch package. Since then we reverted

back to vdsm arch specific package but the bad requirements remained in safelease.

We can safely remove the requirements from safelease if vdsm requires these packages, but

I'm not sure who has time to work on safelease.

I think it is time to remove export and iso domain in 4.4.

Would it be possible?

If an ovirt-4.3 storage pool has an ISO domain, and we add an ovirt-4.4 host to it, we would like it to be able to become SPM.

In rhel 8.1, vdsm 4.4, I don't want to support export or iso domain regardless of the

cluster version.

We don't have the time to port all code in vdsm to python 3. If you want python 3, you need

to remove some features.

If you want to mix 4.4. host with 4.3, env, detach the iso domain and export domain?

Tal, what do you think?

--

SANDRO BONAZZOLA

MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV

Red Hat EMEA

sbonazzo@redhat.com