Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Home Attribute on consul user resource breaks our puppet runs. #559

Closed
bogdankatishev opened this issue Jan 13, 2021 · 3 comments · Fixed by #575
Closed

New Home Attribute on consul user resource breaks our puppet runs. #559

bogdankatishev opened this issue Jan 13, 2021 · 3 comments · Fixed by #575

Comments

@bogdankatishev
Copy link
Contributor

This merge #550 breaks our Puppet runs on our current Consul Agents.

Error: Could not set home on user[consul]: Execution of '/usr/sbin/usermod -d /opt/consul/data consul' returned 8: usermod: user consul is currently used by process 894
Error: /Stage[main]/Consul::Install/User[consul]/home: change from '/home/consul' to '/opt/consul/data' failed: Could not set home on user[consul]: Execution of '/usr/sbin/usermod -d /opt/consul/data consul' returned 8: usermod: user consul is currently used by process 894

As far as I can see, many popular Puppet Modules avoid the User resource, or when they use them, they do not set the home attribute.

How could this be resolved cleanly?

@solarkennedy
Copy link
Contributor

Oh man, I didn't expect that change not work with a running process. Hmm :(

@genebean
Copy link
Contributor

I was just coming to report the same issue...

genebean added a commit to genebean/puppet-consul that referenced this issue May 17, 2021
@bplunkert
Copy link

I am seeing the same issue preventing consul deployment on 7.0.0.

genebean added a commit to genebean/puppet-consul that referenced this issue May 18, 2021
genebean added a commit to genebean/puppet-consul that referenced this issue May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants