GoogleCloudPlatform / GoogleCloudPlatform/prometheus-engine

Unify Dockerfiles with Prom fork pattern (vendor + built split)

Open
#1,865 0 comments 0 reactions 1 assignee Claimed by @gmp-bot View on GitHub
bug
Dominant language
Go
Stars
232
Forks
109
Avg merge
1d 3h
Merged PRs (30d)
27

Description

To ensure we test docker building step as close possible to internal CI, let's port here (and AM) what we did in Prometheus fork so:
* Dockerfile split into separate vendor and build steps https://github.com/GoogleCloudPlatform/prometheus/blob/10ff5b96bd822da7c1477fc796f4b4b19f88b8cc/Dockerfile.google#L36
* We use exactly same steps to build image as internally using buildx https://github.com/GoogleCloudPlatform/prometheus/blob/10ff5b96bd822da7c1477fc796f4b4b19f88b8cc/.github/workflows/presubmit.yml#L55

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.