1045090 - Update the documentation to reflect the treeherder-ui dependency

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P2)

Description

[Dave Hunt [:davehunt] [he/him] ⌚BST]

There are a couple of places in the documentation that assume treeherder-ui is optional, however it now appears to be a requirement. For example, the installation should mention that both treeherder-server and treeherder-ui repositories should be cloned.

Comment 1

[Ed Morley [:emorley]]

Presume this is the cause of:

[/c/src/treeherder/service]$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
There are errors in the configuration of this machine. Please fix
the following errors and try again:

vm:
* The host path of the shared folder is missing: ../treeherder-ui

Comment 2

[Ed Morley [:emorley]]

Attachment: Update docs

Ideally we'd:
(a) Have a Vagrantfile.local that's in the .gitignore
(b) Have the default treeherder-service Vagrantfile not rely on the ui repo

But for now, lets at least update the docs to reflect the current state.

Comment 3

[Ed Morley [:emorley]]

(In reply to Ed Morley [:edmorley] from comment #2)
> Ideally we'd:
> (a) Have a Vagrantfile.local that's in the .gitignore

Filed bug 1056865.

> (b) Have the default treeherder-service Vagrantfile not rely on the ui repo

Filed bug 1056872 & also bug 1056868 (since the latter was the cause for my comment 1 error).

Comment 4

[Ed Morley [:emorley]]

https://github.com/mozilla/treeherder-service/commit/ecfe1f6d2594e408cda82888e233d31184e08c5c

However bug 1057703 is required before the live RTD page will reflect the change.