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
Support for a daemon's restart-condition #361
Support for a daemon's restart-condition #361
Conversation
retest this please |
restart-condition: | ||
type: string | ||
enum: | ||
- never |
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.
Should this be "no"?
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.
The policy, as for daemon
is to bring in the exact systemd
names
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.
Which at least according to this is "no," not "never." Right? Or am I reading something wrong?
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.
I grabbed them from here https://github.com/ubuntu-core/snappy/blob/master/docs/meta.md
So more research is in order
Couple minor comments, but looks good 👍 |
f0e8211
to
7ddcf28
Compare
Just makes it valid syntax in snapcraft.yaml Signed-off-by: Sergio Schvezov <sergio.schvezov@ubuntu.com>
7ddcf28
to
85384d3
Compare
- on-failure | ||
- on-abnormal | ||
- on-abort | ||
- always |
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.
Completely removing the "no?"
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.
Yes for now snapcore/snapd#567
…rt-condition Support for a daemon's restart-condition
…ort-restart-condition Support for a daemon's restart-condition
Just makes it valid syntax in snapcraft.yaml
Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com