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

Remove tools image references and opt for auroraboot #192

Merged
merged 6 commits into from
Nov 15, 2024

Conversation

jimmykarily
Copy link
Collaborator

@jimmykarily jimmykarily commented Nov 13, 2024

Because the tools image has been merged to the auroraboot one here: kairos-io/AuroraBoot#110

Part of kairos-io/kairos#1633

Fixes kairos-io/kairos#1634 and kairos-io/kairos#1635

because it has been merged with auroraboot here:

kairos-io/AuroraBoot#110

as part of kairos-io/kairos#1633

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Copy link
Member

@Itxaka Itxaka left a comment

Choose a reason for hiding this comment

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

looks good but Im worried about bundling the osbuilder tools with the controller changes...

jimmykarily added a commit to kairos-io/AuroraBoot that referenced this pull request Nov 15, 2024
When the user says "--output mydir" they really expect the artifacts to
be in `mydir`, not in `mydir/build`.

See also:

- kairos-io/osbuilder#192 (comment)
- kairos-io/kairos-docs#311 (comment)

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
jimmykarily added a commit to kairos-io/AuroraBoot that referenced this pull request Nov 15, 2024
When the user says "--output mydir" they really expect the artifacts to
be in `mydir`, not in `mydir/build`.

See also:

- kairos-io/osbuilder#192 (comment)
- kairos-io/kairos-docs#311 (comment)

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
jimmykarily added a commit to kairos-io/AuroraBoot that referenced this pull request Nov 15, 2024
When the user says "--output mydir" they really expect the artifacts to
be in `mydir`, not in `mydir/build`.

See also:

- kairos-io/osbuilder#192 (comment)
- kairos-io/kairos-docs#311 (comment)

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
jimmykarily added a commit to kairos-io/AuroraBoot that referenced this pull request Nov 15, 2024
When the user says "--output mydir" they really expect the artifacts to
be in `mydir`, not in `mydir/build`.

See also:

- kairos-io/osbuilder#192 (comment)
- kairos-io/kairos-docs#311 (comment)

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
jimmykarily added a commit to kairos-io/AuroraBoot that referenced this pull request Nov 15, 2024
When the user says "--output mydir" they really expect the artifacts to
be in `mydir`, not in `mydir/build`.

See also:

- kairos-io/osbuilder#192 (comment)
- kairos-io/kairos-docs#311 (comment)

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
because it has been fixed here:

kairos-io/AuroraBoot#115

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
@jimmykarily jimmykarily merged commit 4aa7211 into master Nov 15, 2024
10 checks passed
@jimmykarily jimmykarily deleted the 1633-remove-tools-image branch November 15, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Osbuilder should use the Auroraboot image
2 participants