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

Add 'samba::additional_config' option #52

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

parkr
Copy link
Contributor

@parkr parkr commented Jun 21, 2024

Hey @rehanone, another one for you to consider. There are a bunch of unsupported server configs, so I wonder if we could add a very flexible system for allowing the user to specify them.

This allows the user of the module to specify arbitrary server configuration options. For example, in hiera:

samba::additional_config:
  server_multi_channel_support: true
  aio_read_size: 1
  aio_write_size: 1
  min_receivefile_size: 16384

This would write the following to smb.conf:

[global]
    server multi channel support = yes
    aio read size = 1
    aio write size = 1
    min receivefile size = 16384

Fixes #31.

@parkr
Copy link
Contributor Author

parkr commented Jun 21, 2024

I'm working on the spec failures and will force-push a new commit momentarily.

This allows the user of the module to specify arbitrary server
configuration options. For example, in hiera:

    samba::additional_config:
      server_multi_channel_support: yes
      aio_read_size: 1
      aio_write_size: 1
      min_receivefile_size: 16384

This would write the following to /etc/samba/smb.conf:

    server multi channel support = yes
    aio read size = 1
    aio write size = 1
    min receivefile size = 16384
@parkr parkr force-pushed the arbitrary-options branch from 497798e to 3169add Compare June 21, 2024 18:35
@rehanone rehanone merged commit ddee063 into rehanone:master Jun 28, 2024
4 checks passed
@parkr
Copy link
Contributor Author

parkr commented Jun 28, 2024

Thanks so much @rehanone!

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.

Feature Request : Extra Samba Option
2 participants