Closed munnerz closed 4 years ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: To fully approve this pull request, please assign additional approvers. We suggest the following additional approver: kragniz
Assign the PR to them by writing /assign @kragniz
in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
Bazel verify job is passing :tada:
Now just to move more things into this job!
/test bazel-e2e v1.9
/test bazel-e2e v1.9
/test bazel-e2e v1.9
/test pull-navigator-bazel-verify
/test bazel-e2e v1.9
/test bazel-e2e v1.9
/test bazel-e2e v1.9
/test bazel-e2e v1.9
/test bazel-e2e v1.9
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
/test pull-navigator-bazel-verify
@munnerz: The following tests failed, say /retest
to rerun them all:
Test name | Commit | Details | Rerun command |
---|---|---|---|
navigator-quick-verify | 4fff340c75bb11e33d659492a6630d1e4e706d54 | link | /test verify |
navigator-e2e-v1-8 | 4fff340c75bb11e33d659492a6630d1e4e706d54 | link | /test e2e v1.8 |
pull-navigator-bazel-verify | 4fff340c75bb11e33d659492a6630d1e4e706d54 | link | /test pull-navigator-bazel-verify |
navigator-e2e-v1-9 | 4fff340c75bb11e33d659492a6630d1e4e706d54 | link | /test e2e v1.9 |
navigator-e2e-v1-7 | 4fff340c75bb11e33d659492a6630d1e4e706d54 | link | /test e2e v1.7 |
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.
What this PR does / why we need it:
This is an experiment to add support for building Navigator with Bazel. It should eventually replace
make
, and will help reduce the number of dependencies we need to bake into our test-infra images.This PR also allows building docker images without Docker itself. It can currently also be used to build and unit test (i.e. gotest) all components of Navigator.
Special notes for your reviewer:
This is WIP, and work needs to be done to make this work in our CI environment as well as triggering e2e tests (and other scripts in the
hack/
directory)Release note: