Closed
Bug 1363495
Opened 8 years ago
Closed 5 years ago
don't build docker images unless all tests pass
Categories
(Release Engineering Graveyard :: Applications: Balrog (backend), enhancement, P3)
Release Engineering Graveyard
Applications: Balrog (backend)
Tracking
(Not tracked)
RESOLVED
MOVED
People
(Reporter: bhearsum, Unassigned)
References
Details
There are some directives and process changes that have been passed down to us that need to be implemented on the projects we work with.
Specifically this need to be done:
* automated builds need to fail on test failures
* automated builds need to fail on linting check failures
* automated builds need to fail on regressions in code coverage
This is going to be difficult for us to do at the moment. We use taskcluster-github for CI as well as building and pushing our Docker images, but there's no support for dependencies between Tasks at the moment (this is https://bugzilla.mozilla.org/show_bug.cgi?id=1252144). There may be some hacky way we can do this, but I'm not sure how at the moment.
We also don't have code coverage going at the moment, but that should get fixed up soon in https://github.com/mozilla/balrog/pull/310.
Reporter | ||
Comment 1•8 years ago
|
||
Rok has been toying around with a workaround for bug 1252144 in https://github.com/mozilla-releng/services/pull/309. Here's an example task graph: https://tools.taskcluster.net/task-group-inspector/#/QN_8XtInRQKwCjd8bbmK0A/QN_8XtInRQKwCjd8bbmK0A?_k=j752vg
Reporter | ||
Updated•7 years ago
|
Priority: P1 → P3
Reporter | ||
Comment 2•6 years ago
|
||
This is possible in v1 of .taskcluster.yml.
Reporter | ||
Comment 3•5 years ago
|
||
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → MOVED
Updated•5 years ago
|
Product: Release Engineering → Release Engineering Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•