Closed
Bug 1429988
Opened 6 years ago
Closed 6 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•6 years ago
|
||
|
Assignee | |
Comment 2•6 years ago
|
||
Comment on attachment 8942052 [details] [review] Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127 For review at your leisure :)
Attachment #8942052 -
Flags: review?(cdawson)
|
||
Updated•6 years ago
|
Blocks: 1416266
Component: Treeherder: Data Ingestion → Treeherder: Docs & Development
|
||
Comment 3•6 years ago
|
||
Comment on attachment 8942052 [details] [review] 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•6 years ago
|
||
Comment on attachment 8942052 [details] [review] 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•6 years ago
|
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
|
|
||
Comment 5•6 years ago
|
||
Comment on attachment 8942052 [details] [review] 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•6 years ago
|
||
Comment on attachment 8942052 [details] [review] 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•6 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•6 years ago
|
||
Comment on attachment 8942052 [details] [review] Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/3127 Great work! Thanks!
Attachment #8942052 -
Flags: review?(cdawson) → review+
|
|
||
Updated•6 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
|
||
Updated•2 years ago
|
Component: Treeherder: Docs & Development → TreeHerder
You need to log in
before you can comment on or make changes to this bug.
Description
•