Germanium's process-upload.py script doesn't record in scriptactivity table

Bug #182932 reported by Tom Haddon
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Julian Edwards

Bug Description

/srv/launchpad.net/codelines/current/scripts/process-upload.py on germanium doesn't produce an entry in the scriptactivity table, so there's no way of automatically checking for it.

Related branches

Curtis Hovey (sinzui)
affects: launchpad-foundations → soyuz
Revision history for this message
Julian Edwards (julian-edwards) wrote :

I think this is as trivial as s/LaunchpadScript/LaunchpadCronScript/ in lib/lp/soyuz/scripts/soyuz_process_upload.py

Changed in soyuz:
status: New → Triaged
importance: Undecided → High
tags: added: soyuz-upload trivial
Revision history for this message
Julian Edwards (julian-edwards) wrote :

This change would affect cocoplum's invocations as well, I don't know how we'd tell them apart if they both start recording in scriptactivity.

visibility: private → public
Revision history for this message
Julian Edwards (julian-edwards) wrote :

Ok we can probably stuff the hostname into the __init__ "name" arg for LaunchpadScript.

Changed in soyuz:
milestone: none → 10.04
assignee: nobody → Julian Edwards (julian-edwards)
Revision history for this message
William Grant (wgrant) wrote :

Julian, scriptactivity already has an explicit hostname field.

Revision history for this message
Julian Edwards (julian-edwards) wrote :

Missed that, even better.

Changed in soyuz:
status: Triaged → In Progress
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit
Changed in soyuz:
status: In Progress → Fix Committed
tags: added: qa-needstesting
tags: added: qa-ok
removed: qa-needstesting
Changed in soyuz:
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.