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

the role does not support OpenBSD #14

Closed
trombik opened this issue Nov 30, 2016 · 1 comment · Fixed by #41
Closed

the role does not support OpenBSD #14

trombik opened this issue Nov 30, 2016 · 1 comment · Fixed by #41
Assignees
Milestone

Comments

@trombik
Copy link
Contributor

trombik commented Nov 30, 2016

The official ports tree does not have a port.

Considerations:

  • Install the gem? bit messy and I do not like it
@trombik trombik self-assigned this Dec 26, 2016
trombik pushed a commit that referenced this issue May 22, 2017
@trombik
Copy link
Contributor Author

trombik commented May 22, 2017

installing via gem is not ideal but we need a reliable log shipping mechanism on routers as well (maybe especially).

we should have had a build system for OpenBSD, but creating one takes time. until then, use gem.

@trombik trombik removed the pending label May 22, 2017
@trombik trombik added this to the 1.3.0 milestone May 22, 2017
trombik pushed a commit that referenced this issue May 23, 2017
* do NOT use `enabled: true` in handlers

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

* support OpenBSD

fixes #14

* QA

* merge master

* follow changes in master

* update README

* typo

#41 (review)

* use user-defindable variable

* apply same fix: 5892477

* sync
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant