Closed
Bug 1120416
Opened 10 years ago
Closed 10 years ago
Remove Bugzilla/ElasticSearch comment submission & display warning
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(2 files)
1.12 KB,
patch
|
Details | Diff | Splinter Review | |
14.58 KB,
patch
|
Details | Diff | Splinter Review |
Treeherder no longer relies upon submitBugzillaComment.php and starcomment.php for submitting failure classifications to Bugzilla and ElasticSearch. Since TBPL is nearing end of life, using TBPL to submit failure classifications to other services (Bugzilla/ElasticSearch) is something I think we should prevent, to:
* ensure there are no other tools submitting to TBPL
* to prevent inconsistency/duplication between TBPL and Treeherder submissions
* to allow us to more easily make credential changes to TBPLbot's BMO account (otherwise we have to sync across many prod/stage/dev config files at once, and I'd like to change the password at some point soon)
* to give TBPL users an additional nudge to make the transition to Treeherder
We should display a warning, so users realise their comments are not being submitted to ES/Bugzilla.
We should also rename the one remaining comment submission script ("submitBuildStar.php"), since (a) the name isn't ideal, (b) just to rule out any third party tools submitting to it.
Assignee | ||
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
Assignee | ||
Comment 3•10 years ago
|
||
I tweaked the message slightly before landing, it's now:
"Failure annotations are no longer submitted to Bugzilla/OrangeFactor. Please use Treeherder if you need stats to be tracked."
It's not marked as an error, so it doesn't persist, and isn't in red.
https://hg.mozilla.org/webtools/tbpl/rev/ca691ff528c7
https://hg.mozilla.org/webtools/tbpl/rev/b17c1d831251
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 4•10 years ago
|
||
Updated•10 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•