Code-Review Requests

bloomberg · Nov 2, 2018 · 36dc0f7 · 36dc0f7
1 parent 26734f6
commit 36dc0f7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/cookbooks/bcpc/recipes/pam.rb b/cookbooks/bcpc/recipes/pam.rb
@@ -46,7 +46,7 @@
   source 'pam_namespace.conf.erb'
   mode 500
   variables(lazy {{ real_home_dir_users:
-                     node['bcpc']['pam_namespace']['real_home_dir_users'].join(','),
+                     node['bcpc']['pam_namespace']['real_home_dir_users'].uniq.sort.join(','),
                    shm_polyinstantion_dir: shm_polyinstantion_dir,
                    polyinstantion_dir: polyinstantion_dir
                  }})

diff --git a/cookbooks/bcpc/templates/default/pam_namespace.init.erb b/cookbooks/bcpc/templates/default/pam_namespace.init.erb
@@ -1,4 +1,8 @@
 #!/bin/sh
+
+# This script provided from Ubuntu Bug 1081323.
+# Please remove for the distribution upstream once using pam - 1.1.8-3.6ubuntu1 (Bionic) or better
+
 # It receives polydir path as $1, the instance path as $2,
 # a flag whether the instance dir was newly created (0 - no, 1 - yes) in $3,
 # and user name in $4.