8000 Fix 64624 document symbolic mode limitation by mu-lippy · Pull Request #67928 · saltstack/salt · GitHub
[go: up one dir, main page]
Skip to content

Fix 64624 document symbolic mode limitation #67928

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 25, 2025
Merged

Fix 64624 document symbolic mode limitation #67928

merged 2 commits into from
Jun 25, 2025
+15 −0

Conversation

mu-lippy
Copy link

What does this PR do?

Adds documentation notes to clarify that Salt's file module only supports numeric mode specifications (like 644) and does not support symbolic modes (like u+rw) that are commonly used with chmod.

What issues does this PR fix or reference?

Fixes #64624

Merge requirements satisfied?

Commits signed with GPG?

No

@mu-lippy mu-lippy requested a review from a team as a code owner March 28, 2025 15:12
@welcome Welcome
Copy link
welcome bot commented Mar 28, 2025

Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here’s some information that may help as you continue your Salt journey.
Please be sure to review our Code of Conduct. Also, check out some of our community resources including:

There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar.
If you have additional questions, email us at saltproject.pdl@broadcom.com. We’re glad you’ve joined our community and look forward to doing awesome things with you!

@dwoz dwoz force-pushed the fix-64624-document-symbolic-mode-limitation branch from 21a91f5 to a7be150 Compare May 5, 2025 23:18
@twangboy twangboy force-pushed the fix-64624-document-symbolic-mode-limitation branch from a7be150 to 8c6fd31 Compare May 19, 2025 22:32
@twangboy twangboy added the test:full Run the full test suite label May 19, 2025
@twangboy twangboy added this to the Argon v3008.0 milestone May 19, 2025
@dwoz dwoz merged commit 52acc3a into saltstack:master Jun 25, 2025
1450 of 1458 checks passed
@welcome Welcome
Copy link
welcome bot commented Jun 25, 2025
80B0

Congratulations on your first PR being merged! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@twangboy twangboy twangboy approved these changes

Assignees
No one assigned
Labels
test:full Run the full test suite
Projects
None yet
Milestone
Argon v3008.0
Development

Successfully merging this pull request may close these issues.

[BUG] Documentation is missing information on lack of support for symbolic file modes
3 participants
@mu-lippy @twangboy @dwoz
0