Closed Bug 1288181 Opened 8 years ago Closed 8 years ago

Handle empty changeset list from json-pushes

Categories

(Tree Management :: Treeherder: Data Ingestion, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: camd, Assigned: camd)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

gps has pushed a change where json-pushes could return empty list of changesets for some repos.  If a push has an empty chagngeset, Treeherder's current ETL code will except out on the whole list of pushes.  We should fix this so it just skips that one push.
Blocks: 1286022
Depends on: 1286426
Assignee: nobody → cdawson
Comment on attachment 8773841 [details] [review]
[treeherder] mozilla:empty-changesets > mozilla:master

this is a pretty easy one.  :)
Attachment #8773841 - Flags: review?(emorley)
Comment on attachment 8773841 [details] [review]
[treeherder] mozilla:empty-changesets > mozilla:master

r+ with the tweak :-)
Attachment #8773841 - Flags: review?(emorley) → review+
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/ff6a66a27c2c234e5820b8ffe48f17d85f1eb2db
Bug 1288181 - Handle empty changeset list from json-pushes (#1717)

If a pushlog contains hidden changesets (changesets that are
obsolete) then the call to json-pushes will return a push
with no changesets.  This was changed in bug 1286426.
For us, if `changesets` is empty, we will not be able to get
a revision, which is required for a resultset.  So we
need to just skip the push.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Blocks: 1288845
Blocks: 1365318
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: