cloud-builders icon indicating copy to clipboard operation
cloud-builders copied to clipboard

Promote `gcr.io/cloud-marketplace-containers/google/bazel` for `bazel` users

Open bendory opened this issue 5 years ago • 5 comments

bendory avatar May 14 '20 19:05 bendory

You might be able to just merge https://github.com/GoogleCloudPlatform/cloud-builders/pull/504, if nobody is still using the special-case logic.

imjasonh avatar May 14 '20 19:05 imjasonh

You might be able to just merge #504, if nobody is still using the special-case logic.

Not an unreasonable approach, but rather than try to chase that down I'm going to follow the same approach I'm using in other deprecations:

  • Add a deprecation notice pointing users to an official replacement
  • Stop building the image / stop making updates
  • Leave the existing built image in place so that current users are not broken if they choose not to move

bendory avatar May 15 '20 15:05 bendory

Can we install tzdata into bazel image please? /usr/share/zoneinfo is missing in the xenial image.

kyessenov avatar Jun 25 '20 14:06 kyessenov

In the cert-manager release tooling we'd like to use gcr.io/cloud-marketplace-containers/google/bazel but we wanted a Bazel 4.2.1 image and couldn't find one, so we switched to gcr.io/cloud-builders/bazel. Unfortunately, that Docker image doesn't have any tags, only digests:

  • https://github.com/cert-manager/release/pull/47

wallrj avatar Sep 17 '21 08:09 wallrj

It looks like the marketplace container has not been updated for over a year. Maybe the documentation in this repository should remove links to it? It seems like this bazel builder is actually MORE up to date?

evanj avatar Oct 22 '21 18:10 evanj

It seems gcr.io/cloud-marketplace-containers/google/bazel is not updated. Therefore it does not seem appropriate to deprecate this builder image in favour of marketplace image.

haroonc avatar Sep 07 '23 12:09 haroonc