salt-extensions / saltext-prometheus

Salt Extension Modules for Prometheus
https://saltext-prometheus.readthedocs.io/en/latest/
Apache License 2.0
10 stars 6 forks source link

fix mode set to int octal instead of octal notation #21

Closed nicholasmhughes closed 1 year ago

nicholasmhughes commented 1 year ago

What does this PR do?

Fixes permissions set incorrectly in transition from full text file management (int octal) to salt.modules.file.set_mode management (octal notation).

Previous Behavior

0644 passed as mode would be converted to 420

New Behavior

0644 is retained through passing to salt.modules.file.set_mode

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.