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

Drop luet, image extractor, drop build code and multiarch images #20

Merged
merged 3 commits into from
May 16, 2023
Merged

Conversation

Itxaka
Copy link
Member

@Itxaka Itxaka commented May 12, 2023

  • drop luet
  • use image extractor for pulling images instead of luet
  • drop channel sources
  • introduce archfor images
  • drop build commands and useless code no needed
  • add pull-image command
  • bump yip to 1.x

Signed-off-by: Itxaka [email protected]

@Itxaka Itxaka linked an issue May 16, 2023 that may be closed by this pull request
@Itxaka Itxaka force-pushed the pull branch 15 times, most recently from 52e275e to a774a4e Compare May 16, 2023 11:30
 - drop luet
 - use image extractor for pulling images instead of luet
 - drop channel sources
 - introduce archfor images
 - drop build commands and useless code no needed
 - add pull-image command

Signed-off-by: Itxaka <[email protected]>
@Itxaka Itxaka marked this pull request as ready for review May 16, 2023 11:38
@Itxaka Itxaka requested a review from a team May 16, 2023 11:39
Signed-off-by: Itxaka <[email protected]>
@Itxaka
Copy link
Member Author

Itxaka commented May 16, 2023

This needs a docs update as we no longer support the auth flags for upgrade.

We now support any type of docker auth that docker provides and it extends to anything using the pull, i.e. install,upgrade,pull-image

So it will look into:

~/.docker/config.json, or %USERPROFILE%.docker\config.json by default
Docker config file location if DOCKER_CONFIG is set
${XDG_RUNTIME_DIR}/containers/auth.json for podman

In that given order. So we need to document this after this is merged

@Itxaka
Copy link
Member Author

Itxaka commented May 16, 2023

docs PR here kairos-io/kairos-docs#9

@Itxaka Itxaka linked an issue May 16, 2023 that may be closed by this pull request
@Itxaka Itxaka requested review from mauromorales and a team May 16, 2023 13:48
Copy link
Member

@mauromorales mauromorales left a comment

Choose a reason for hiding this comment

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

w00t w00t 👏

@Itxaka Itxaka merged commit ddfa30a into main May 16, 2023
@Itxaka Itxaka deleted the pull branch May 16, 2023 14:06
@@ -343,12 +343,11 @@ var _ = Describe("Install action tests", func() {

It("Fails if luet fails to unpack image", Label("image", "luet", "unpack"), func() {
Copy link
Contributor

Choose a reason for hiding this comment

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

the description should no longer mention "luet" I guess

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.

🌱 Multi-arch image support consume luet as a binary in the elemental-cli
3 participants