Skip to content

Commit 91b3fe7

Browse files
committed
pynfs: install packaged xdrlib3 module on Debian-derived distros
Daniel Gomez reported that Debian distros have this module packaged and that it should be preferred over a pip install. Move the install to install-deps and change Debian distros to use the packaged module instead. Fedora and SuSE distros still use pip. Reviewed-by: Chuck Lever <[email protected]> Reviewed-by: Daniel Gomez <[email protected]> Signed-off-by: Jeff Layton <[email protected]>
1 parent 07f7f97 commit 91b3fe7

File tree

4 files changed

+9
-5
lines changed

4 files changed

+9
-5
lines changed

playbooks/roles/pynfs/tasks/install-deps/debian/main.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
- swig
1313
- python3-gssapi
1414
- python3-ply
15+
- python3-standard-xdr-lib
1516
state: present
1617
update_cache: yes
1718
tags: [ 'pynfs', 'deps' ]

playbooks/roles/pynfs/tasks/install-deps/redhat/main.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,3 +23,7 @@
2323
- swig
2424
- python3-gssapi
2525
- python3-ply
26+
27+
- name: Install xdrlib3
28+
ansible.builtin.pip:
29+
name: xdrlib3

playbooks/roles/pynfs/tasks/install-deps/suse/main.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,7 @@
1313
- python3-gssapi
1414
- python3-ply
1515
state: present
16+
17+
- name: Install xdrlib3
18+
ansible.builtin.pip:
19+
name: xdrlib3

playbooks/roles/pynfs/tasks/main.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -70,11 +70,6 @@
7070
when:
7171
- not kdevops_workflows_dedicated_workflow|bool
7272

73-
- name: Install xdrlib3
74-
tags: [ 'pynfs' ]
75-
ansible.builtin.pip:
76-
name: xdrlib3
77-
7873
- name: Build pynfs
7974
tags: [ 'pynfs' ]
8075
command: ./setup.py build

0 commit comments

Comments
 (0)