Status

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jlorenzo, Assigned: jlorenzo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Discussed with :bhearsum over IRC. With the current docker image implementation, we can easily manually test Balrog from an end to another. Also, having the UI in its own repo makes changes that affect both front-end and back-end harder.

We can now put the UI in the main repo, without using a submodule. One way to achieve that could be to transfer the commits from one repo to another. See [1] for more details about the implementation.

Side note: Let's wait until the number of active PRs reaches 0. Otherwise, they'll have to be heavily edited before creating a new PR in the main repo.

[1] https://github.com/fxbox/foxbox/pull/406#issuecomment-212449454
One request from :bhearsum, on IRC:
> i'd prefer that everything prior to
> https://github.com/mozilla/balrog-ui/commit/5e70edaa020cf86d8d2962052857b3db773eeabe
> is squashed together - most of the stuff before that is early development, and
> i don't think there's any value to preserving its commit-by-commit history
Blocks: 1299213
I found the reason of the tests hanging. Lineman delegates the test execution to testem. Per [1], it must be present in node_modules/.bin. That wasn't the case in the Docker image. I added it to packages.json.

I also added "phantom-prebuilt". This packages makes sure PhantomJS is in $PATH by downloading it.

Tests are now running and passing [2].

Review suggestions:
* Look at the commit tree
* See diff for HEAD and HEAD~2 (HEAD~1 being the merge commit).

[1] https://github.com/linemanjs/lineman/blob/3f828e5b96b72c6a9d562a0dba23e8f2cd4d2cfc/lib/testem-utils.coffee#L5
[2] https://tools.taskcluster.net/task-inspector/#IC47ho9VRvKti6Crp1p59w/0
Attachment #8790216 - Flags: review?(bhearsum)
Posted file balrog-ui PR
Review suggestions:
* every file but .gitignore .gitkeep and README.md are deleted
* .gitignore filters out everything
* .gitkeep still let README.md be in the repo
* README.md points people to the new location of the UI (which doesn't exist until the balrog PR is merged)
Attachment #8790223 - Flags: review?(bhearsum)
Attachment #8790223 - Flags: review?(bhearsum) → review+
Attachment #8790216 - Flags: review?(bhearsum) → review+
You need to log in before you can comment on or make changes to this bug.