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
[feature] support OpenBSD #41
Conversation
in most cases, you set `enbled: true` in a service task in tasks. it might look same when you set `enabled: true` in BOTH a service task and a service handler, but it would bites you when you have `arguments` in a service task. `arguments` is not implemented in all platforms. it is in OpenBSD. when both the service task and the handler has `enabled: true`, but only one of them has `arguments`, then, `idempotency_test` will mysteriously fail because the one with `arguments` modifies `rc.conf.local` and the other as well. this happens when: * the platform is OpenBSD * a service in task and a service in handler has `enabled: true` * only one of them has `arguments` * `arguments` is not identical to the default
fixes #14
tasks/install-OpenBSD.yml
Outdated
- name: Create _fluentd user | ||
user: | ||
name: _fluentd | ||
comment: Fluend daemon |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Fluend/Fluentd/
vars/OpenBSD.yml
Outdated
__fluentd_package_name: rubygem-fluentd | ||
__fluentd_service_name: fluentd | ||
__fluentd_config_dir: "/etc/fluentd" | ||
__fluentd_config_file: "{{ __fluentd_config_dir }}/fluent.conf" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/__fluentd_config_dir/fluentd_config_dir/ or users must modify fluentd_config_file when fluentd_config_dir is set.
README.md
Outdated
| `__fluentd_package_name` | `rubygem-fluentd` | | ||
| `__fluentd_service_name` | `fluentd` | | ||
| `__fluentd_config_dir` | `/etc/fluentd` | | ||
| `__fluentd_config_file` | `{{ __fluentd_config_dir }}/fluent.conf` | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/__fluentd_config_dir/fluentd_config_dir/
fixes #14
ruby version is hard-coded and should be fixed by writing
ansible-role-ruby