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

[bug fix] Update octal permission syntax #80

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

chadek
Copy link
Contributor

@chadek chadek commented Feb 12, 2024

I was working on an ansible role for nocodb but I could not manage to pass idempotent test on this workflow because of a really weird behavior of this role that I use as a dependency: first run create configuration dir folder, at second run, it appears the folder as 02755 permission instead of 0755, so it set permission again. It appears that it might be because of octal permission syntax which can lead to inconsistent result according to file module documentation quoted below.

For those used to /usr/bin/chmod remember that modes are actually octal numbers. You must give Ansible enough information to parse them correctly. For consistent results, quote octal numbers (for example, '644' or '1777') so Ansible receives a string and can do its own conversion from string into number. Adding a leading zero (for example, 0755) works sometimes, but can fail in loops and some other circumstances.

Copy link

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label Jun 14, 2024
@chadek
Copy link
Contributor Author

chadek commented Jun 15, 2024

up

@github-actions github-actions bot removed the stale label Jun 21, 2024
@geerlingguy geerlingguy merged commit 7e08c0c into geerlingguy:master Jul 26, 2024
3 checks passed
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.

2 participants