Skip to content

Commit 4b824b4

Browse files
authored
Remove workaround from user.pp
The issue referenced in the workaround (https://tickets.puppetlabs.com/browse/PUP-9706) has been closed and puppetlabs/puppet#7527 was merged to address the underlying issue. Therefore this workaround should no longer be needed and the `managehome` option should be customizable. Full issue description reported here #438
1 parent 3cedba7 commit 4b824b4

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

manifests/user.pp

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -234,13 +234,8 @@
234234
ensure => 'absent',
235235
forcelocal => $forcelocal,
236236
home => $_home,
237-
managehome => false, # Workaround for PUP-9706; see below.
237+
managehome => $managehome,
238238
}
239-
# The core `user` resource will fail when removing users on Solaris if
240-
# `ensure => 'absent', managhome => true` and the homedir does not exist.
241-
# We therefore force `managehome => false` when `ensure => 'absent'`, and
242-
# remove the homedir as a separate operation.
243-
# See https://tickets.puppetlabs.com/browse/PUP-9706
244239
if $purge_user_home {
245240
file { $_home:
246241
ensure => 'absent',

0 commit comments

Comments
 (0)