Skip to content

Commit

Permalink
fix: debian_snapshot architecture
Browse files Browse the repository at this point in the history
Regardless of the `platform_transition_filegroup`, the architecture
needs a `select` to properly set the architecture in the manifest.

I verified this by running the test and inspecting the generated config
JSON. Ideally, this should be encoded in a test but I'm not sure how to
go about it.
  • Loading branch information
jjmaestro committed Dec 2, 2024
1 parent 830764f commit ccbc65d
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion examples/debian_snapshot/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,10 @@ cacerts(

oci_image(
name = "image",
architecture = "arm64",
architecture = select({
"@platforms//cpu:arm64": "arm64",
"@platforms//cpu:x86_64": "amd64",
}),
env = {
# Required to use the SSL certs from `cacerts()`
"SSL_CERT_FILE": "/etc/ssl/certs/ca-certificates.crt",
Expand Down

0 comments on commit ccbc65d

Please sign in to comment.