Closed Bug 1063411 Opened 5 years ago Closed 5 years ago

Adjust treeherder .htaccess to serve the UI from the root directory not /ui/

Categories

(Tree Management :: Treeherder, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: jeads)

References

Details

Attachments

(1 file)

54 bytes, text/x-github-pull-request
mdoglio
: review+
Details | Review
Currently the treeherder UI is served out of:
https://treeherder.mozilla.org/ui/

eg:
https://treeherder.mozilla.org/ui/index.html
https://treeherder.mozilla.org/ui/css/index.min.css

...maps to this on production/staging (the grunt build output):
https://github.com/mozilla/treeherder-ui/blob/master/dist/index.html
https://github.com/mozilla/treeherder-ui/blob/master/dist/css/index.min.css

...and this for dev (the unminified source):
https://github.com/mozilla/treeherder-ui/tree/master/webapp/app/index.html
https://github.com/mozilla/treeherder-ui/tree/master/webapp/app/css/index.css

In order to help with bug 1042737, it would be great to serve the above assets out of:
treeherder.mozilla.org/

rather than:
treeherder.mozilla.org/ui/

By adjusting the rewrites in the .htaccess (or wherever) for dev, staging and production.
Raising priority since I think we should try and tackle this before we start linking to treeherder from too many places.
Priority: P2 → P1
Assignee: nobody → jeads
Status: NEW → ASSIGNED
Attached file Remove ui route
Attachment #8497889 - Flags: review?(mdoglio)
Attachment #8497889 - Flags: review?(mdoglio) → review+
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
jeads: So I can run vagrant provision on dev/stage to pick this up? What about prod - guessing IT need to run it? Could you coordinate with them?
Flags: needinfo?(jeads)
Depends on: 1081059
Mauro, aiui this needs a vagrant provision to pick the changes up - what's the best way forwards with this?
Flags: needinfo?(mdoglio)
You need to run vagrant provision to apply the changes to your environment.
I'll re-run puppet on dev and stage and see if everything is cool. We need fubar to apply the changes on production.
Flags: needinfo?(mdoglio)
Flags: needinfo?(jeads)
Thank you - reopening bug so this appears on the wiki page until it's deployed.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee: jeads → nobody
Depends on: 1088577
Filed bug 1088577 to run the vagrant provision, so the request is clearer & so we can close this out in the meantime.
Assignee: nobody → jeads
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Depends on: 1124670
You need to log in before you can comment on or make changes to this bug.