Closed
Bug 678796
Opened 13 years ago
Closed 13 years ago
TCBS cron test is time zone dependent
Categories
(Socorro :: General, task)
Tracking
(Not tracked)
RESOLVED
FIXED
2.3.2
People
(Reporter: lonnen, Assigned: rhelmer)
Details
Attachments
(1 file)
919 bytes,
patch
|
rhelmer
:
review+
|
Details | Diff | Splinter Review |
unittest/cron/testTopCrashesBySignature.py fails when you run it ~ 1 am PDT. Rhelmer was able to build without problems by changing his VM's clock. It may be a difference between UTC and local time.
Reporter | ||
Comment 1•13 years ago
|
||
Making this a blocker since we Jenkins won't let us push a build unless it is passing all tests.
Severity: normal → blocker
Reporter | ||
Comment 2•13 years ago
|
||
This won't fix the actual problem, but it will remove the failing test so we can push the build to stage. It's a low risk failure, so I recommend we push this to branch only and then unblock this (but leave the bug unresolved.)
Attachment #552943 -
Flags: review?(rhelmer)
Assignee | ||
Comment 3•13 years ago
|
||
Comment on attachment 552943 [details] [diff] [review] *whistles* +1, branch-only
Attachment #552943 -
Flags: review?(rhelmer) → review+
Reporter | ||
Comment 4•13 years ago
|
||
landed in branch-only as r3435 (http://code.google.com/p/socorro/source/detail?r=3435)
Reporter | ||
Updated•13 years ago
|
Severity: blocker → normal
Comment 5•13 years ago
|
||
We'll actually fix this the right way in 2.3, I think.
Target Milestone: 2.2 → 2.3
Assignee | ||
Comment 6•13 years ago
|
||
If we keep bumping this, we can probably just get rid of old-tcbs and rm this test :)
Target Milestone: 2.3 → 2.3.1
Updated•13 years ago
|
Target Milestone: 2.3.1 → 2.3.2
Comment 7•13 years ago
|
||
Old TCBS is only relevant to Camino after this: new action is "skip the test from 2.3.2 on and hope for Camino one day or something"
Assignee: nobody → rhelmer
Comment 9•13 years ago
|
||
Commit pushed to https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/8597592ccf21d1c5ce015de4b5a9dbc0f6f63f65 Merge pull request #93 from rhelmer/bug678796-oldtcbs-unit-test-busted bug 678796 - disable failing unit test
Assignee | ||
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Component: Socorro → General
Product: Webtools → Socorro
You need to log in
before you can comment on or make changes to this bug.
Description
•