1215511 - Socorro crontabber app not finding & associating signatures with no args

Status: RESOLVED FIXED

(Socorro :: Backend, task)

Description

[K Lars Lohn [:lars] [:klohn]]

the crontabber job BugzillaCronApp is supposed to make associations in the Bugs table between signatures and bug numbers. It is not doing so for the signatures that that were inserted into Bugzilla with no arguments.

It appears to be trouble with the query that BugzillaCronApp gives to Bugzilla.  It is supposed to find all bugs that have had a state change N days into the past. (N's actual config name is "days_into_past"). Unfortunately, there is a problem where code overrides the selected N and substitutes the date of the last successful run.  This causes it to miss the changes that inserted the new signatures.  we need to run the query with an N such that now() - N includes 2015-10-13, the day the signatures were updated.

as the code stands right now, we cannot do this.  N is overridden at https://github.com/mozilla/socorro/blob/master/socorro/cron/jobs/bugzilla.py#L66   We cannot even force it to use a literal query because this line will fail without a date to insert: https://github.com/mozilla/socorro/blob/master/socorro/cron/jobs/bugzilla.py#L74 

resolve this issue, 1st with a run that includes the date 2015-10-13 and then 2nd with a way that N overrides "last_success" not the other way around.

Comment 1

[Peter Bengtsson [:peterbe]]

Attachment: Link to Github pull-request: https://github.com/mozilla/socorro/pull/3043

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/494d3b9b889163f5020e93db52b9d087ef635a2f
fixes bug 1215511 - bugzilla crontabber app and days_into_past, r=twobraids

https://github.com/mozilla/socorro/commit/ec517854665d8501aef7be3f62d54a4156d52aff
Merge pull request #3043 from peterbe/bug-1215511-bugzilla-crontabber-app-and-days_into_past

fixes bug 1215511 - bugzilla crontabber app and days_into_past