gluster / gluster-ansible-infra

Ansible role to enable and deploy the backend of a Gluster cluster.
GNU General Public License v3.0
28 stars 36 forks source link

vdsm-python package on Debian based systems #119

Open rohankrishnadev opened 3 years ago

rohankrishnadev commented 3 years ago

In role backend_setup there is a task to check whether the package vdsm-python is installed. This seems to be specific to operating systems in the RedHat family. This isn't an issue as ignore_errors is set to yes https://github.com/gluster/gluster-ansible-infra/blob/bd5dda37fd37bb8279898a796add47bab0d95463/roles/backend_setup/tasks/main.yml#L11

However, later on there is a task to 'Re-generate new LVM Filrer rules' which uses vdsm-python. This task fails on Debian like systems. This should either be modified to include operating systems from the Debian family or ignore_errors could be set to yes on this task as well. https://github.com/gluster/gluster-ansible-infra/blob/bd5dda37fd37bb8279898a796add47bab0d95463/roles/backend_setup/tasks/regenerate_new_lvm_filter_rules.yml#L4

hunter86bg commented 3 years ago

We should use ansible-native way to detect if the package (oVirt) is deployed or not. https://github.com/gluster/gluster-ansible-infra/pull/122 should fix it . Tested on Ubuntu so far.