Closed
Bug 1240831
Opened 10 years ago
Closed 10 years ago
ingest_push broken
Categories
(Tree Management :: Treeherder: Data Ingestion, defect)
Tree Management
Treeherder: Data Ingestion
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wlach, Assigned: wlach)
Details
Attachments
(1 file)
When trying to debug a problem with talos ingestion w/ MikeLing, I noticed that ingest_push doesn't seem to be working. E.g.:
./manage.py ingest_push --filter-job-group T mozilla-inbound 7e06ec79a132
produces no results.
It appears that buildbot is now using 16 character revisions, but the revision filter expects an exact match so the 12 character ones I was specifying was insufficient.
I'm pretty sure the fix for this is just to check if the filter string is IN the revision, rather than it be an exact match.
Comment 1•10 years ago
|
||
| Assignee | ||
Updated•10 years ago
|
Attachment #8709542 -
Flags: review?(emorley)
Updated•10 years ago
|
Attachment #8709542 -
Flags: review?(emorley) → review+
| Assignee | ||
Updated•10 years ago
|
Summary: Make revision filtering a little looser → ingest_push broken
Comment 2•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla/treeherder/commit/03a682c63788a91e67237be41357af5911752daf
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 3•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/4d8af30353c492b1ef25c862f7d935a99fcc5253
Bug 1240831 - Fix ingest_push
We weren't passing the full revision to the ingestion logic, so everything
was being skipped.
You need to log in
before you can comment on or make changes to this bug.
Description
•