Closed Bug 1019805 Opened 10 years ago Closed 10 years ago

Copy dashboard fails if device ID is specified and no results exist for that device

Categories

(Testing Graveyard :: Eideticker, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: davehunt, Assigned: wlach)

References

Details

Attachments

(1 file)

The first build of a new device fails in Eideticker CI because the copy-dashboard script tries to download dashboard history for a device with no prior results. We should probably just silently ignore if there are no matching results.

+ ./bin/copy-dashboard.py --device-id flame --skip-metadata http://eideticker.mozilla.org/b2g/ /var/jenkins/workspace/b2g.flame.mozilla-central.tests/dashboard/
Traceback (most recent call last):
  File "./bin/copy-dashboard.py", line 110, in <module>
    testnames = tests.json()['tests'].keys()
  File "/usr/lib/python2.7/dist-packages/requests/models.py", line 741, in json
    return json.loads(self.text, **kwargs)
  File "/usr/lib/python2.7/json/__init__.py", line 338, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
Attached patch FixSplinter Review
I think this should do the trick.
Assignee: nobody → wlachance
Attachment #8433599 - Flags: review?(dave.hunt)
Attachment #8433599 - Flags: review?(dave.hunt) → review+
Landed: https://github.com/mozilla/eideticker/commit/abac6657f4d0b64b64bee3560263352293fb055e
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Blocks: 1020215
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: