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

Docker improvements #6075

Merged
merged 9 commits into from
Aug 7, 2024
Merged

Docker improvements #6075

merged 9 commits into from
Aug 7, 2024

Conversation

Amndeep7
Copy link
Contributor

@Amndeep7 Amndeep7 commented Aug 6, 2024

  • Simplify dockerfile by doing the chown/chmod on copy
  • Try out macos runner - not possible see 857ba1f
  • Multiarch docker build - not feasible, just heimdall lite is over 3.75hrs and going
  • Try out to see if we can use matrixes to combine the workflows for push-latest vs release and heimdall-lite vs heimdall
  • Try using the docker metadata action to generate the tag info instead of using some random string manipulation action

… to save a lot of space by not having duplicate layers

Signed-off-by: Amndeep Singh Mann <[email protected]>
Signed-off-by: Amndeep Singh Mann <[email protected]>
… the setup buildx command earlier since it'll hopefully install docker then

Signed-off-by: Amndeep Singh Mann <[email protected]>
…w am experimenting with QEMU to do the emulation.

Turns out that we need at least macos-15 and m3 chip runners in order to support virtualization properly.  Currently, the runner version is on 14 and m1.

Signed-off-by: Amndeep Singh Mann <[email protected]>
Signed-off-by: Amndeep Singh Mann <[email protected]>
Signed-off-by: Amndeep Singh Mann <[email protected]>
@Amndeep7 Amndeep7 marked this pull request as draft August 6, 2024 07:54
@Amndeep7 Amndeep7 marked this pull request as ready for review August 7, 2024 01:19
@Amndeep7 Amndeep7 merged commit a7bb04b into master Aug 7, 2024
10 of 11 checks passed
@Amndeep7 Amndeep7 deleted the docker_improvements branch August 7, 2024 01:22
Copy link

sonarcloud bot commented Aug 7, 2024

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.

1 participant