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

(PDB-3832) Update permissions for ini files to match puppetlabs-puppetdb module #3240

Open
wants to merge 1 commit into
base: 6.x
Choose a base branch
from

Conversation

underscorgan
Copy link
Contributor

@puppetcla
Copy link

CLA signed by all contributors.

@underscorgan
Copy link
Contributor Author

wow, this fell off my radar for way too long, but just ran a test and it seems to work

[root@dilute-datum ~]# ll /etc/puppetlabs/puppetdb/conf.d/config.ini
-rw------- 1 puppetdb puppetdb 472 Jan 29 18:48 /etc/puppetlabs/puppetdb/conf.d/config.ini

@austb
Copy link
Contributor

austb commented Jan 29, 2021

We just added a file named auth.conf, so we might want to update the command to change that as well

Base automatically changed from master to 6.x February 2, 2021 22:08
@CLAassistant
Copy link

CLAassistant commented Sep 1, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Morgan Rhodes seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@jplindquist
Copy link
Contributor

jplindquist commented Apr 13, 2022

Is there an update on this? I noticed the same thing still occurring during every update cycle, permissions are getting reset to match what the module is expecting

notice | /Stage[main]/Puppetdb::Server::Global/File[/etc/puppetlabs/puppetdb/conf.d/config.ini]/owner | current_value 'root', should be 'puppetdb' (noop) (corrective)
notice | /Stage[main]/Puppetdb::Server::Global/File[/etc/puppetlabs/puppetdb/conf.d/config.ini]/group | current_value 'root', should be 'puppetdb' (noop) (corrective)
notice | /Stage[main]/Puppetdb::Server::Global/File[/etc/puppetlabs/puppetdb/conf.d/config.ini]/mode | current_value '0644', should be '0600' (noop) (corrective)

@puppetlabs-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

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 this pull request may close these issues.

6 participants