Skip to content

Conversation

@rhdedgar
Copy link

Adding multi-architecture ARM64 build support to the release process.

Closes RHAIENG-2781.

Addresses
llamastack/llama-stack#4290 (comment)
as a follow-up to
llamastack/llama-stack#4290.

Signed-off-by: Doug Edgar <dedgar@redhat.com>
docker images | cat

echo "Pushing docker image"
echo "Building and pushing multi-arch OpenShift-compatible image"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
echo "Building and pushing multi-arch OpenShift-compatible image"
echo "Building and pushing multi-arch image"

Copy link
Author

Choose a reason for hiding this comment

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

+1, I've updated the comment.

docker build "$LLAMA_STACK_DIR" \
BASE_TAG="llamastack/distribution-$distro:$TAG_SUFFIX-base"
docker buildx build "$LLAMA_STACK_DIR" \
--platform linux/amd64,linux/arm64 \
Copy link
Collaborator

Choose a reason for hiding this comment

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

should linux/amd64,linux/arm64 be a top var?

Copy link
Author

Choose a reason for hiding this comment

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

Sure, I've now replaced the 4 occurrences of that string with a new variable.

Signed-off-by: Doug Edgar <dedgar@redhat.com>
Signed-off-by: Doug Edgar <dedgar@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants