Closed
Bug 1088577
Opened 10 years ago
Closed 10 years ago
Run |vagrant provision| on prod/stage/dev to pick up the changes to apache/treeherder-service.conf
Categories
(Tree Management :: Treeherder, defect, P1)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: mdoglio)
References
Details
Attachments
(1 file)
In order for the changes in bug 1063411 to take effect, we need to |vagrant provision| so that the updated puppet/files/apache/treeherder-service.conf gets copied to "${apache_vhost_path}/treeherder-service.conf":
https://github.com/mozilla/treeherder-service/blob/master/puppet/manifests/classes/apache.pp#L30
Changes this will bring across:
https://github.com/mozilla/treeherder-service/commit/5fd276b12db0993cb9f9e5a9f510ff5b3d2ad8c2
https://github.com/mozilla/treeherder-service/commit/0f85ac1768ad0c7e788dcff6a2da37ab46e947a2
https://github.com/mozilla/treeherder-service/commit/defa7a2bd815e183cda296843149ef87f09ddef1
Assignee | ||
Comment 1•10 years ago
|
||
I manually updated the vhost file on both dev and stage. Running the provisioning on dev raised errors fwiw.
I had to make a change to the vhost file to make it work, I'll attach the PR here
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8511921 -
Flags: review?(emorley)
Reporter | ||
Comment 3•10 years ago
|
||
Comment on attachment 8511921 [details]
Github PR #264 on treeherder-service
Thank you for sorting this :-)
Attachment #8511921 -
Flags: review?(emorley) → review+
Comment 4•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/treeherder-service
https://github.com/mozilla/treeherder-service/commit/fbc6b853e1e34d2295c13033a6fb2303daa70490
Bug 1088577 - update apache vhost to remove ui prefix
https://github.com/mozilla/treeherder-service/commit/78fe1169d9e8fff3418b85650608dbfacaadec09
Merge pull request #264 from mozilla/bug-1088577-update-apache-vhost
Bug 1088577 - update apache vhost to remove ui prefix
Assignee | ||
Comment 5•10 years ago
|
||
There is only production left, we need :fubar's superpowers
Reporter | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → mdoglio
You need to log in
before you can comment on or make changes to this bug.
Description
•