cronscripts/request_daily_builds.py is not verbose enough on default logging

Bug #619555 reported by Steve McInerney on 2010-08-18
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Medium
Tim Penhey

Bug Description

A complete run:
2010-08-17 23:52:17 INFO Creating lockfile: /var/lock/launchpad-request_daily_builds.lock
2010-08-17 23:52:42 INFO Requested 118 daily builds.

That's it. just the 2 lines.
As a minimum, it'd be good to get a 'Deleting locking' message to easily note the script ending; otherwise it's a bit on the guess work side.
I assume, that -v++ options would give more details on the builds kicked off.
Given this is the default setting, it *may* be useful for basic debug to display what builds are kicked off?
Or we can add a -v or more to the cron entry for same?

Related branches

Steve McInerney (spm) on 2010-08-18
tags: added: canonical-losa-lp
Tim Penhey (thumper) on 2010-08-19
Changed in launchpad-code:
status: New → Triaged
importance: Undecided → Medium
tags: added: recipe
Tim Penhey (thumper) wrote :

Steve, the lock file removal is logged at the debug level (don't ask why, it is in contrib/glock.py). If you want this logged at INFO like the lockfile acquisition, please file a separate bug as that will require changing heaps of tests :-(

I've updated the code in the related branch to log at debug level the recipes that are having the build requested. If you want this recorded in our logfiles, you should at a -v to the cron job.

Changed in launchpad-code:
status: Triaged → In Progress
assignee: nobody → Tim Penhey (thumper)
Tim Penhey (thumper) on 2010-12-15
tags: added: bugjam2010
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
William Grant (wgrant) on 2010-12-22
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui) on 2010-12-23
Changed in launchpad:
status: Fix Committed → Fix Released
Curtis Hovey (sinzui) on 2011-01-11
Changed in launchpad:
milestone: none → 11.01
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers