snap stop --disable svc with timer doesn't disable the timer

Bug #1842258 reported by Ian Johnson
44
This bug affects 8 people
Affects Status Importance Assigned to Milestone
snapd
Fix Released
Medium
Philip Meulengracht

Bug Description

With a service/daemon in a snap declared with a timer, `snap stop --disable` does not prevent future starts from the timer. To reproduce:
1. Install a snap with a timer service run every minute, with the service just sleeping for 30 seconds
2. Run `snap stop --disable $SNAP.$SVC`
3. Observe the service is still active with `systemctl status snap.$SNAP.$SVC.timer` (the state will be "active (waiting)"

Revision history for this message
Ian Johnson (anonymouse67) wrote :

As a consequence of this, `snap services $SNAP.$SVC` reports that the service is "disabled" after running `snap stop --disable` even though the timer is not disabled

Changed in snapd:
status: New → Triaged
Changed in snapd:
assignee: nobody → Ian Johnson (anonymouse67)
Changed in snapd:
importance: Undecided → Medium
Revision history for this message
Robie Basak (racb) wrote :

This bug is stopping certbot snap users from disabling the auto-renew timer for the purposes of arranging their own automation around updating services with renewed certificates. Details at https://github.com/certbot/certbot/issues/8555

Any chance this could be looked at please?

Revision history for this message
Paweł Stołowski (stolowski) wrote :

Yes, we will look into this (and see if this is still the case) now that the service handling has undergone major refactoring (in master only, will be released with 2.49).

Revision history for this message
Philipp Schneider (p-schneider) wrote :

It is still the case in Version 2.49, so it looks like the major refactor did not solve this issue.

Changed in snapd:
assignee: Ian Johnson (anonymouse67) → nobody
Revision history for this message
Tom Boutell (tboutell) wrote :

Just figured out this is my problem as well. Having this happen automatically with no way of disabling it is kind of a lot, but I absolutely appreciate the volunteer / nonprofit effort that goes into this.

Revision history for this message
Philip Meulengracht (the-meulengracht) wrote :

Support for this has been merged to master with this PR (https://github.com/snapcore/snapd/pull/13128). It's scheduled for 2.61

Changed in snapd:
status: Triaged → Fix Committed
Changed in snapd:
assignee: nobody → Philip Meulengracht (the-meulengracht)
Revision history for this message
Philip Meulengracht (the-meulengracht) wrote :

Until 2.61 reaches master this can be tested with the snapd in edge.

Changed in snapd:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.