Skip to content

build: publish multi-arch images with rules_oci#40

Merged
muff1nman merged 2 commits into
cloudflare:masterfrom
muff1nman:ademaria/ai/rules-oci-multiarch
Jul 2, 2026
Merged

build: publish multi-arch images with rules_oci#40
muff1nman merged 2 commits into
cloudflare:masterfrom
muff1nman:ademaria/ai/rules-oci-multiarch

Conversation

@muff1nman

Copy link
Copy Markdown
Contributor

rules_docker and rules_k8s are unmaintained, and the image they produced was amd64-only, so sciuro could not run on arm64 nodes. Move the build to rules_oci on bzlmod and publish a linux/amd64 + linux/arm64 image index so the controller is deployable across both architectures.

Reviewed-by: Andrew DeMaria

@khos2ow khos2ow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor comments otherwise lgtm

Comment thread .github/workflows/check.yaml Outdated
Comment thread .github/workflows/release.yaml Outdated
Andrew DeMaria's AI Assistant added 2 commits June 30, 2026 16:22
rules_docker and rules_k8s are unmaintained, and the image they produced
was amd64-only, so sciuro could not run on arm64 nodes. Move the build to
rules_oci on bzlmod and publish a linux/amd64 + linux/arm64 image index so
the controller is deployable across both architectures.

Reviewed-by: Andrew DeMaria
Update golangci-lint to unblock CI, since golangci-lint-action v6 installs
the v1-era v1.64.8 binary that rejects a v2 config; v9 installs a v2 binary
that understands it. Align .golangci.yml with the Service Automation shared
baseline (v2 defaults plus the team exclusion presets) so this repo lints
consistently with the rest of our in-house services.

Reviewed-by: Andrew DeMaria
@muff1nman muff1nman force-pushed the ademaria/ai/rules-oci-multiarch branch from 451928a to 3c0e3c6 Compare June 30, 2026 22:22
@muff1nman muff1nman merged commit 3c0e3c6 into cloudflare:master Jul 2, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants