CHANGELOG.md

@@ -1,19 +0,0 @@
-
-## v1.1.1
-
-### Enhancements
-* Improved documentation and fixed typos.
-
-## v1.1
-
-### Enhancements
-* Manage client configuration file.
-* List the ports used by rpcbind.
-* Set statd and quotad ports.
-
-### Fix
-
-## v1.0
-
-### Features
-* Install nfs client for Debian.

README.md

@@ -1,5 +1,4 @@
-# NFS - Projet **ARRÊTÉ** au profit de [debops.nfs][debops nfs] et [debops.nfs_server][debops nfs server].
-
+# NFS
 
 1. [Overview](#overview)
 2. [Role Variables](#role-variables)
@@ -12,7 +11,7 @@
 
 ## Overview
 
-This role manages NFS client installation and configuration using fixed ports.
+Manage NFS (client) installation and configuration.
 
 ## Role Variables
 
@@ -77,5 +76,3 @@ Jérémy Gardais
 [nfs github]: https://github.com/ipr-cnrs/nfs
 [wtfpl website]: http://www.wtfpl.net/about/
 [ipr website]: https://ipr.univ-rennes1.fr/
-[debops nfs]: https://github.com/debops/debops/tree/master/ansible/roles/debops.nfs
-[debops nfs server]: https://github.com/debops/debops/tree/master/ansible/roles/debops.nfs_server

defaults/main.yml

@@ -5,10 +5,10 @@
 nfs_cli_manage: true
 nfs_cli_pkg_state: 'installed'
 nfs_cli_conf_path: '/etc/default/nfs-common'
-nfs_cli_conf_tpl: 'etc/default/nfs-common.j2' # TODO : hide this variable from user, this is for this role's internal use
+nfs_cli_conf_tpl: 'etc/default/nfs-common.j2'
 
 nfs_quota_conf_path: '/etc/default/quota'
-nfs_quota_conf_tpl: 'etc/default/quota.j2' # TODO : hide this variable from user, this is for this role's internal use
+nfs_quota_conf_tpl: 'etc/default/quota.j2'
 
 # rpcbind
 nfs_rpc_nfsd: '2049'

meta/main.yml

@@ -1,6 +1,6 @@
 galaxy_info:
   author: "Jérémy Gardais"
-  description: "This role manages NFS client installation and configuration using fixed ports"
+  description: "Manage NFS (client/server) installation and configuration"
   license: WTFPL
   company: IPR
   issue_tracker_url: https://git.ipr.univ-rennes1.fr/cellinfo/ansible.nfs/issues

tasks/main.yml

@@ -1,23 +1,23 @@
 ---
-# tasks file for role nfs
+# tasks file for nfs
 
-- name: load operating system specific variables for nfs 
-  include_vars: "{{ item }}" # includes yml file containing os specific variable, eg "debian.yml" found in role_root/vars
+- name: Load specific OS vars for nfs
+  include_vars: "{{ item }}"
   with_first_found:
     - "{{ ansible_distribution|lower }}-{{ ansible_distribution_version }}.yml"
     - "{{ ansible_distribution|lower }}.yml"
     - "{{ ansible_os_family|lower }}.yml"
 
 # client {{{
-- name: ensure nfs client package is in the required install state
+- name: client package
   package:
-    name: '{{ item }}'   # TODO: why not simply {{ nfs_cli_pkg_list }} ?
+    name: '{{ item }}'
     state: '{{ nfs_cli_pkg_state }}'
   with_items:
     - '{{ nfs_cli_pkg_list }}'
   when: nfs_cli_manage
 
-- name: configure nfs-client
+- name: CONFIG nfs-client
   template:
     src: '{{ nfs_cli_conf_tpl }}'
     dest: '{{ nfs_cli_conf_path }}'
@@ -27,7 +27,7 @@
     backup: true
   when: nfs_cli_manage
 
-- name: configure nfs-quota
+- name: CONFIG nfs-quota
   template:
     src: '{{ nfs_quota_conf_tpl }}'
     dest: '{{ nfs_quota_conf_path }}'
@@ -37,7 +37,7 @@
     backup: true
   when: nfs_cli_manage
 
-- name: declare rpcbind ports in /etc/services
+- name: CONFIG add rpcbind services
   blockinfile:
     state: present
     insertafter: EOF