Run |vagrant provision| on prod/stage/dev to pick up the changes to apache/treeherder-service.conf

RESOLVED FIXED

Status

Tree Management
Treeherder
P1
normal
RESOLVED FIXED
3 years ago
9 months ago

People

(Reporter: emorley, Assigned: mdoglio)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
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

3 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

3 years ago
Created attachment 8511921 [details] [review]
Github PR #264 on treeherder-service
Attachment #8511921 - Flags: review?(emorley)
(Reporter)

Comment 3

3 years ago
Comment on attachment 8511921 [details] [review]
Github PR #264 on treeherder-service

Thank you for sorting this :-)
Attachment #8511921 - Flags: review?(emorley) → review+

Comment 4

3 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

3 years ago
There is only production left, we need :fubar's superpowers
(Reporter)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Reporter)

Updated

9 months ago
Assignee: nobody → mdoglio
You need to log in before you can comment on or make changes to this bug.