Skip to content

merge greenboot-success.target inside heathcheck service - #63

Merged
say-paul merged 3 commits into
fedora-iot:mainfrom
say-paul:systemd-service-unification
Jul 22, 2025
Merged

merge greenboot-success.target inside heathcheck service#63
say-paul merged 3 commits into
fedora-iot:mainfrom
say-paul:systemd-service-unification

Conversation

@say-paul

@say-paul say-paul commented Jul 9, 2025

Copy link
Copy Markdown
Member

enabling greenboot-healthcheck.service will auto enable greenboot-success.target , so user does not need to enable it manually.

@say-paul
say-paul force-pushed the systemd-service-unification branch from f1c50c4 to a639c4c Compare July 9, 2025 09:03
No need to manually enable greenboot-success target
CI tests updated
@say-paul
say-paul force-pushed the systemd-service-unification branch from f93626b to cec0ce5 Compare July 10, 2025 06:09
@say-paul
say-paul force-pushed the systemd-service-unification branch 2 times, most recently from 0f2e3f1 to 74a4d3d Compare July 10, 2025 12:18
Comment thread src/lib/greenboot.rs
Comment thread usr/lib/systemd/system/greenboot-healthcheck.service
@say-paul

Copy link
Copy Markdown
Member Author

@say-paul

Copy link
Copy Markdown
Member Author

tested rpm-ostree locally.

@mmartinv mmartinv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
(Although not sure why the build_and_test action is warning about unused imports and functions)

@say-paul

Copy link
Copy Markdown
Member Author

(Although not sure why the build_and_test action is warning about unused imports and functions)

its because there are two branches of testing

  1. Almost entire suite where the /boot remount ro(default) -> rw(required to modify grub param) and remounting back to ro is mocked up.
  2. remaining 2 scenario to test when /boot is already in desired state ro/rw so that it dont schage it from there.

@say-paul
say-paul merged commit 45ce0c2 into fedora-iot:main Jul 22, 2025
12 of 18 checks passed
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.

4 participants