* modules/user/manifests/virtual/localuser.pp: This reverts part of
0d82c62 which was providing a temporary transition correcting
permissions and ownership as well as removing the results of a
previous failed implementation of keyfile management. Unfortunately
that also caused nodepool images to no longer set up accounts
correctly, but reverting the transitional bits fixes that issue.
Change-Id: Ic3e7f0a3b82bbc8a23707b88f9fe381e9d70e356
We need specific key ids for keys, and if the key id changes, we need
a way to delete old ones. We also need the file to be writable by at
least the user so that puppet doesn't complain.
Change-Id: I5718b80d844d5f95149d0e23d98960879955c43c
There is a builtin type for adding authorized_keys. We started
using it for the restricted access for the puppet triggering key,
and it seems to make sense to be consistent across the board.
Change-Id: I76ffb0136c58c05fa7c8abb793cb8e3b03cb5f6c
Changes:
groups now a variable with a preset, since there may be instances
where sudo/admin will NOT be desired.
home now entirely a variable, defaults to previous action, but
handles cases where created user may not belong in /home
managed home flag, defaulted to enabled.
No impact to current functionality. Allows for more granular changes
in the future. Typo fix for managehome.
Change-Id: Id0921f5b28ea0ffd2230d94e87673e6b39ac060e
This patch adds the user::virtual::disable function to the user module.
This will allow puppet to remove a user, ssh keys, and screen sessions
while preserving the user home directory.
This patch adds future functionality without impacting the currently
configured infrastructure.
Change-Id: I2933e6857094398f86c2a7e6eaabe9898a1d3078
This is a sytle guide update for puppet. Including definations within a
class is not recommented.
Change-Id: Id108d61dade1f99350691627236f1a13c12c10bc
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
Reviewed-on: https://review.openstack.org/14986
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Killed trailing whitespace in the puppet repo files using
`sed -r 's/\s+$//'`. Skip binary files and html templates for lodgeit
and mailman.
Change-Id: Ib43493161d8f0e8fae1426b22fb1737832ca14cd
Reviewed-on: https://review.openstack.org/12969
Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Includes:
Fixing jenkins sudoers from jeblair
Removing tarmac depend from jeblair
Remove wheel group from jeblair
Change-Id: I86ec23f7aeafc31cc70b0f3a648739bb7316fc61
