glance-simplestreams-sync cronjob never runs

Bug #1469584 reported by Brad Marshall
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
glance-simplestreams-sync (Juju Charms Collection)
Fix Released
High
Unassigned

Bug Description

The glance-simplestreams-sync charm places an entry into /etc/cron.daily which is a symlink to /usr/share/glance-simplestreams-sync/glance-simplestreams-sync.py, but the filename is glance-simplestreams-sync.py.

As per the cron manpage this will never run:

       Files in this directory have to be owned by root, do not need to be executable (they are configuration files,
       just like /etc/crontab) and must conform to the same naming convention as used by run-parts(8): they must con‐
       sist solely of upper- and lower-case letters, digits, underscores, and hyphens. **This means that they cannot
       contain any dots.**

Emphasis added by me to point out the relevant part.

Simply renaming the cronjob to /etc/cron.daily/glance-simplestreams-sync should fix this issue.

This bug combined with LP#1356472 can make juju unbootable on an Openstack cloud using this charm.

Please let us know if you have any questions about this.

Revision history for this message
Haw Loeung (hloeung) wrote :

"""
       Additionally, the file names must conform to the
       filename requirements of run-parts: they must be entirely made up of
       letters, digits and can only contain the special signs underscores
       ('_') and hyphens ('-'). Any file that does not conform to these
       requirements will not be executed by run-parts. For example, any file
       containing dots will be ignored. This is done to prevent cron from
       running any of the files that are left by the Debian package management
       system when handling files in /etc/cron.d/ as configuration files (i.e.
       files ending in .dpkg-dist, .dpkg-orig, and .dpkg-new).
"""

Haw Loeung (hloeung)
Changed in glance-simplestreams-sync (Juju Charms Collection):
status: New → Confirmed
Revision history for this message
JuanJo Ciarlante (jjo) wrote :

FYI this Q&D diff WFM http://paste.ubuntu.com/11796446/

James Page (james-page)
Changed in glance-simplestreams-sync (Juju Charms Collection):
status: Confirmed → Triaged
importance: Undecided → High
milestone: none → 15.10
Felipe Reyes (freyes)
Changed in glance-simplestreams-sync (Juju Charms Collection):
assignee: nobody → Felipe Reyes (freyes)
Revision history for this message
Felipe Reyes (freyes) wrote :

Revision 53[0] pulled a bunch of changes, one of them is use 'glance_simplestreams_sync' for the cronjob filename, so this seems to be 'Fix Committed'.

[0] http://bazaar.launchpad.net/~openstack-charmers/charms/trusty/glance-simplestreams-sync/next/revision/53

Changed in glance-simplestreams-sync (Juju Charms Collection):
assignee: Felipe Reyes (freyes) → nobody
Haw Loeung (hloeung)
Changed in glance-simplestreams-sync (Juju Charms Collection):
status: Triaged → Fix Committed
James Page (james-page)
Changed in glance-simplestreams-sync (Juju Charms Collection):
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.