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

VFS options should also be accepted on the [Global] level #133

Closed
RlndVt opened this issue Jan 12, 2024 · 2 comments
Closed

VFS options should also be accepted on the [Global] level #133

RlndVt opened this issue Jan 12, 2024 · 2 comments

Comments

@RlndVt
Copy link

RlndVt commented Jan 12, 2024

e.g. the follwoing should also be valid syntax:

    - name: Include Samba Server role
      ansible.builtin.include_role:
        name: vladgh.samba.server
      vars:
        # [GLOBAL]
        # [...]
        vfs_objects:
          - name: fruit
            options:
              - name: [...]
                value: 'Yes'
          - name: streams_xattr
        samba_shares:
          - name: 'Name'
            vfs_objects:
              - name: fruit
                options:
                  - name: [...]
                    value: 'Yes'
              - name: streams_xattr

Current workaround is using samba_global_include.

Copy link

This issue 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!

@github-actions github-actions bot added the stale This issue or pull request has been marked 'stale' due to lack of recent activity label Apr 12, 2024
@vladgh vladgh removed the stale This issue or pull request has been marked 'stale' due to lack of recent activity label Apr 12, 2024
@vladgh
Copy link
Owner

vladgh commented May 9, 2024

Thank you for your suggestion, but for the sake of simplicity, I think it's best to use a global include file for this.

@vladgh vladgh closed this as completed May 9, 2024
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

No branches or pull requests

2 participants