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

Adding example to use gid in file module #83049

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from
Open

Conversation

Remz42
Copy link

@Remz42 Remz42 commented Apr 16, 2024

SUMMARY

I was searching a way to use gid to create a file.
When i inspected the code, i found it was possible to use gid as string (example: group: "1000").
I updated the doc and added an example.

ISSUE TYPE
  • Docs Pull Request
ADDITIONAL INFORMATION

@ansibot ansibot added needs_triage Needs a first human triage before being processed. module This issue/PR relates to a module. labels Apr 16, 2024
@Akasurde Akasurde added needs_verified This issue needs to be verified/reproduced by maintainer and removed needs_triage Needs a first human triage before being processed. labels Apr 16, 2024
@Akasurde Akasurde self-requested a review April 16, 2024 15:15
@@ -33,7 +33,7 @@ class ModuleDocFragment(object):
type: raw
owner:
description:
- Name of the user that should own the filesystem object, as would be fed to I(chown).
- Name of the user or gid that should own the filesystem object, as would be fed to I(chown).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part of the change is incorrect; you cannot pass a GID to owner.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry. I changed the wrong line. It should be ok now.

@QJoly
Copy link

QJoly commented Apr 17, 2024

It's good to know that the GID can be used for this module, Thanks @Remz42

@webknjaz
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MrDevOS
Copy link

MrDevOS commented Apr 25, 2024

I think it's a great idea, I was wondering if it was possible to do that, thanks @Remz42

@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module This issue/PR relates to a module. needs_verified This issue needs to be verified/reproduced by maintainer stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants