PR#295: Update standard-test-avocado role to support pip installation - standard-test-roles

standard-test-roles

#295 Update standard-test-avocado role to support pip installation

Merged 5 years ago by astepano. Opened 5 years ago by ymankad.

ymankad/standard-test-roles master into master

Update standard-test-avocado role to support pip installation

Yash Mankad • 5 years ago

f65a0ab

roles/standard-test-avocado/tasks/main.yml

file modified

+15

		`@@ -4,6 +4,21 @@`
		`with_items:`
		`- python2-avocado`
		`- rsync # need rsync for Ansible synchronize module`
		`+ when: ansible_distribution_major_version != "8"`
		`+`
		`+ # Avocado installation using pip for RHEL-8.0 Beta`
		`+ # which does not package python2 rpms`
		`+ - name: Install requirements for RHEL-8.0 Beta`
		`+ block:`
		`+ - name: Install Avocado using pip`
		`+ pip:`
		`+ name: avocado-framework`
		`+ extra_args: --user`
		`+ - name: Install rsync`
		`+ package: name=rysnc state=latest`
		`+ when:`
		`+ - ansible_distribution == "RedHat"`
		`+ - ansible_distribution_major_version == "8"`

		`- name: Make artifacts directory`
		`file: path={{ remote_artifacts }} state=directory owner=root mode=755 recurse=yes`

ymankad commented 5 years ago

Avocado is not packaged for RHEL-8.0 Beta as it does not support python2 only rpms. This fix addresses the issue by installing avocado-framework using pip when using the standard-test-avocado role on RHEL-8.0

Signed-off-by: Yash Mankad ymankad@redhat.com

astepano commented 5 years ago

Hi!
The PR looks reasonable.
Is there an issue for Avocado that we can track Avocado and RHEL8 integration?
Maybe a link to Trello card?
So we can know when to switch to RPM installation.
If there is such resource please add URL to it in your PR.
Thank you!