Closed
Bug 1429988
Opened 7 years ago
Closed 7 years ago
Minor updates for local pulse ingestion
Categories
(Tree Management :: Treeherder, defect)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jfrench, Assigned: jfrench)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
I had some minor issues setting up for local pulse ingestion, so am making a couple of proposals (RTD, settings.py) which help the user set it up successfully and have it work right out of the box, without needing IRC help.
o specify RTD that PULSE_DATA_INGESTION_SOURCES is a 'Working example'
o set settings.py PULSE_DATA_INGESTION_SOURCES from "name:" to "exchange:" so it doesn't fail
o maybe set projects: as mozilla-inbound.# with the hash (?)
o update the RTD PULSE_DATA_INGESTION_CONFIG so it doesn't fail eg.
PULSE_DATA_INGESTION_CONFIG = env.url("PULSE_DATA_INGESTION_CONFIG","amqp://mypulseuserid:mypassword@pulse.mozilla.org:5671/?ssl=1")
Comment 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
Comment on attachment 8942052 [details]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127
For review at your leisure :)
Attachment #8942052 -
Flags: review?(cdawson)
Updated•7 years ago
|
Blocks: 1416266
Component: Treeherder: Data Ingestion → Treeherder: Docs & Development
Comment 3•7 years ago
|
||
Comment on attachment 8942052 [details]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127
I had some ideas to handle this a slightly different way. :) Thanks!
Attachment #8942052 -
Flags: review?(cdawson) → review-
Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 8942052 [details]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127
For re-review. I think I still have at least one tweak but it should be close. I also need to rebase and re-push before any merge.
Attachment #8942052 -
Flags: review- → review?(cdawson)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Comment 5•7 years ago
|
||
Comment on attachment 8942052 [details]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127
Just one tweak requested. And then we'll see where the tests go once you've rebased. Thanks for doing this! :)
Attachment #8942052 -
Flags: review?(cdawson) → review-
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8942052 [details]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127
Ok, tweak is done and I've rebased/squashed also.
Attachment #8942052 -
Flags: review- → review?(cdawson)
Comment 7•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/7247ea20b7dfbda823b3535c850ec5ca0a27086f
Bug 1429988 - Minor updates for local pulse ingestion
Comment 8•7 years ago
|
||
Comment on attachment 8942052 [details]
Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127
Great work! Thanks!
Attachment #8942052 -
Flags: review?(cdawson) → review+
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Component: Treeherder: Docs & Development → TreeHerder
You need to log in
before you can comment on or make changes to this bug.
Description
•