-
Notifications
You must be signed in to change notification settings - Fork 694
Dont assume docker path in image util #1262
Dont assume docker path in image util #1262
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
@googlebot I signed it! |
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
/gcbrun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. Didn't notice there are CI errors. Could you please fix the following?
(12:07:37) ERROR: /workdir/docker/util/run.bzl:176:18: name 'image_utils' is not defined
--
| (12:07:37) DEBUG: /workdir/cc/image.bzl:79:9: cc_image does not benefit from layers=[], got: [":cc_image_library"]
| (12:07:37) ERROR: error loading package 'tests/docker/util': Extension 'docker/util/run.bzl' has errors
| (12:07:37) ERROR: error loading package 'tests/contrib': in /workdir/docker/toolchain_container/toolchain_container.bzl: in /workdir/docker/toolchain_container/debian_pkg_tar.bzl: in /workdir/docker/package_managers/apt_key.bzl: Extension 'docker/util/run.bzl' has errors
| (12:07:37) ERROR: error loading package 'tests/docker/package_managers': in /workdir/docker/package_managers/apt_key.bzl: Extension 'docker/util/run.bzl' has errors
| (12:07:37) ERROR: error loading package 'tests/docker/toolchain_container': in /workdir/docker/toolchain_container/toolchain_container.bzl: in /workdir/docker/toolchain_container/debian_pkg_tar.bzl: in /workdir/docker/package_managers/apt_key.bzl: Extension 'docker/util/run.bzl' has errors
| (12:07:37) ERROR: Skipping '//tests/contrib:derivative_with_volume_repro_test': error loading package 'tests/contrib': in /workdir/docker/toolchain_container/toolchain_container.bzl: in /workdir/docker/toolchain_container/debian_pkg_tar.bzl: in /workdir/docker/package_managers/apt_key.bzl: Extension 'docker/util/run.bzl' has errors
d4f3b25
to
425ff84
Compare
tests seemingly passes |
buildkite is still failing with error message:
line 200 in run.bzl should probably be This is a syntax error and you should be able to repro these locally and debug them yourself. Please make sure you can build and test this locally (and that the buildkite tests pass) before pinging again (unless you get stuck and need a hand debugging) as otherwise we will be doing too much back and forth. |
/gcbrun |
tests seemingly passes |
argh. apologies for messing up your workflow here |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: massemanet, nlopezgi, smukherj1 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
"image_util.sh" does not respect the docker toolchain.
this PR generates "image_util.sh" from a template instead.