Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish multi-architecture images #70

Merged
merged 2 commits into from
Jan 9, 2023
Merged

Conversation

jeevb
Copy link
Contributor

@jeevb jeevb commented Jan 7, 2023

Addresses: flyteorg/flyte#2935

Signed-off-by: Jeev B <jeevb@users.noreply.github.com>
@jeevb
Copy link
Contributor Author

jeevb commented Jan 7, 2023

Tested against flyteconsole so far.

wild-endeavor
wild-endeavor previously approved these changes Jan 7, 2023
Signed-off-by: Jeev B <jeevb@users.noreply.github.com>
@@ -30,6 +30,13 @@ jobs:
- name: Prime docker cache
run: (docker load -i /tmp/tmp/docker-images/snapshot-builder.tar || true) && (docker load -i /tmp/tmp/docker-images/snapshot.tar || true)
- name: Build dockerfile
Copy link
Contributor Author

@jeevb jeevb Jan 7, 2023

Choose a reason for hiding this comment

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

This pipeline is currently being used by flyteadmin, flytepropeller, and datacatalog to build images for e2e tests. AFAICT, these images are not pushed to any registry. Can someone confirm?

Eventually, we might consider running tests on all supported architectures, but this might be currently too slow on GHA runners with emulation. We can leave this as-is for now, and only build amd64 images for testing purposes.

Copy link
Contributor

Choose a reason for hiding this comment

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

i don't think these are pushed, i can't actually tell why this step is necessary, all three of these have a separate push step after.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Some of the abovementioned repos use them for e2e tests.

@wild-endeavor
Copy link
Contributor

@jeevb jeevb marked this pull request as ready for review January 9, 2023 22:25
@wild-endeavor wild-endeavor merged commit 27211db into master Jan 9, 2023
@wild-endeavor wild-endeavor deleted the jeev/multiarch-builds branch January 9, 2023 22:25
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.

2 participants