Closed Bug 668992 Opened 13 years ago Closed 13 years ago

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mstange, Assigned: mstange)

References

Details

Attachments

(1 file)

Attached patch v1Splinter Review
There are three possible choices for the default value of Config.wooBugURL:
 1. "php/starcomment.php"
 2. "http://brasstacks.mozilla.com/starcomment.php"
 3. "http://tbpl.mozilla.org/php/starcomment.php"

Choice 1 has the disadvantage of only working on exactly one server: tbpl.mozilla.org. That's because the server needs access to elasticsearch1.metrics.sjc1.mozilla.com, and normal servers don't have that. (tbpl.mozilla.org got it in bug 638515.)

Choice 2 has the advantage of working from every server, but might not be as reliable, as Jonathan says in bug 601743 comment 24:
(In reply to comment #24)
> Only that brasstacks is probably not quite as reliable as the tbpl machine;
> it's sort of a sandbox for a lot of automation tools, and the webserver
> sometimes gets restarted, etc.

It would also make the local starcomment.php file unnecessary, unless we changed Config.wooBugURL to "php/starcomment.php" only on tbpl.mozilla.org and merged with repo changes on every deployment.

Choice 3 looks like a good way out.
Attachment #543608 - Flags: review?(arpad.borsos)
Comment on attachment 543608 [details] [diff] [review]
v1

Review of attachment 543608 [details] [diff] [review]:
-----------------------------------------------------------------

As discussed in icq: Is the script on tbpl.mozilla.org really allowed to write to the woo database? Need to verify that first.
Attachment #543608 - Flags: review?(arpad.borsos) → review+
We can do that when we deploy it, and back it out if it doesn't work.
Seems to have worked out in the end.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: