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] Documentation is missing information on lack of support for symbolic file modes #64624

Open
TimidRobot opened this issue Jul 10, 2023 · 1 comment
Labels
Documentation Relates to Salt documentation good first issue good for someone new to salt help-wanted Community help is needed to resolve this severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around time-estimate-single-day

Comments

@TimidRobot
Copy link

Description
The documentation is completely silent on the lack of symbolic file modes.

Steps to Reproduce the behavior

Expected behavior
For example:

  • salt/salt/states/file.py

    Lines 102 to 104 in b45c319

    When using a mode that includes a leading zero you must wrap the
    value in single quotes. If the value is not wrapped in quotes it
    will be read by YAML as an integer and evaluated as an octal.
  • Something like the following might be clearer:

    Symbolic file modes are not supported. When using an octal file mode that includes a leading zero, you must wrap the value in single quotes. If the value is not wrapped in quotes, it will be read by YAML as an integer and evaluated as an octal number instead of an octal file mode.

Additional context
Related:

@TimidRobot TimidRobot added Bug broken, incorrect, or confusing behavior needs-triage labels Jul 10, 2023
@OrangeDog
Copy link
Contributor

I think it's sufficiently clear that it means numerical mode only.

The permissions to set on this file, e.g. 644, 0775, or 4664.

@OrangeDog OrangeDog added Documentation Relates to Salt documentation and removed Bug broken, incorrect, or confusing behavior labels Jul 11, 2023
@barbaricyawps barbaricyawps added time-estimate-single-day severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around good first issue good for someone new to salt help-wanted Community help is needed to resolve this and removed severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around needs-triage labels Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Relates to Salt documentation good first issue good for someone new to salt help-wanted Community help is needed to resolve this severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around time-estimate-single-day
Projects
None yet
Development

No branches or pull requests

3 participants