Skip to content

sys-apps/ignition: bump to 2.22.0 #3182

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Aug 12, 2025

In this PR we update Ignition to 2.22.0.

This pulls:

CI w/ all providers: http://localhost:8080/job/container/job/packages_all_arches/6465/cldsv/ 🟢 except for some Akamai tests but it's not related to this PR.

I took the opportunity to simplify the ebuild:

  • remove coreos-go eclass dependency
  • generate a Manifest
  • drop live ebuild

@tormath1 tormath1 self-assigned this Aug 12, 2025
@tormath1 tormath1 added the main label Aug 12, 2025
@tormath1 tormath1 force-pushed the tormath1/ignition-update branch from 4ffda61 to f127107 Compare August 12, 2025 15:14
@tormath1 tormath1 moved this to ✅ Testing / in Review in Flatcar tactical, release planning, and roadmap Aug 13, 2025
@tormath1 tormath1 marked this pull request as ready for review August 13, 2025 07:24
@tormath1 tormath1 requested a review from a team as a code owner August 13, 2025 07:24
Comment on lines 4903 to 4912
diff --git a/go.mod b/go.mod
index 63d73c98..f94bbafa 100644
--- a/go.mod
+++ b/go.mod
@@ -1,4 +1,4 @@
-module github.com/coreos/ignition/v2
+module github.com/flatcar/ignition/v2

go 1.22.0
toolchain go1.23.6
Copy link
Member

Choose a reason for hiding this comment

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

This change should probably stay in the sed s/coreos/flatcar/ patch, no?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct, thanks!

@@ -14,6 +14,7 @@ require (
@@ -1,4 +1,4 @@
-module github.com/coreos/ignition/v2
+module github.com/flatcar/ignition/v2
Copy link
Member

Choose a reason for hiding this comment

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

This change probably should remain being a part of the sed s/coreos/flatcar/ patch, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct, thanks!

* remove coreos-go eclass dependency
* generate a Manifest
* drop live ebuild

Signed-off-by: Mathieu Tortuyaux <[email protected]>
@tormath1 tormath1 force-pushed the tormath1/ignition-update branch from f127107 to 8025362 Compare August 13, 2025 13:46
Signed-off-by: Mathieu Tortuyaux <[email protected]>
@tormath1 tormath1 force-pushed the tormath1/ignition-update branch from 8025362 to a2d43da Compare August 13, 2025 13:55
Copy link

github-actions bot commented Aug 13, 2025

Build action triggered: https://github.com/flatcar/scripts/actions/runs/16958767245

@tormath1
Copy link
Contributor Author

Let's hold this until the current main branch has been branched-out to avoid bringing more stuffs into the current release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Testing / in Review
Development

Successfully merging this pull request may close these issues.

2 participants