Skip to content

Conversation

@kobliha
Copy link
Contributor

@kobliha kobliha commented May 26, 2025

  • Implements a rescue system by booting to the Installer Live Media image with special systemd parameters that disable booting to the graphical mode and starting Agama service
  • Some services are kept, such as DUD, Self-Update, root password

Problem

The current media do not have a Rescue System but users are asking for it

Solution

This PR does not implement any special Rescue System. It only reuses what we already have and prevents Agama from starting.

Testing

  • Parameters were tested manually on a Kernel command line

- Implements a rescue system by booting to the Installer Live Media
  image with special systemd parameters that disable booting to the
  graphical mode and starting Agama service
- Some services are keps, such as DUD and Self-Update
Copy link
Contributor

@imobachgs imobachgs left a comment

Choose a reason for hiding this comment

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

It looks fine. Did you try it?

Please, update the changes file.

@kobliha
Copy link
Contributor Author

kobliha commented May 26, 2025

I tried that just manually on a Kernel command-line

imobachgs
imobachgs previously approved these changes May 26, 2025
@imobachgs imobachgs dismissed their stale review May 26, 2025 14:32

Discussing on IRC about whether it will work.

Copy link
Contributor

@lslezak lslezak 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 as the initial implementation for some rescue system.

However there are still some questions:

  • Which services should be actually started in the rescue mode? Should we e.g. configure the network?
  • Systemd already provides it's own rescue mode, boot with systemd.unit=rescue.target live.password=password (The rescue target requires a configured root password, that's why I updated the live-password.service file in this PR to allow setting the password in the rescue mode as well.)
  • Implementation detail: Maybe we should define a separate systemd target for this?

@kobliha kobliha merged commit 6a85f2c into agama-project:master May 26, 2025
1 check passed
@kobliha kobliha deleted the simple_rescue_system branch May 26, 2025 18:10
@kobliha
Copy link
Contributor Author

kobliha commented May 26, 2025

Thanks @lslezak, yes I thought about a special target too, but it's a bigger thing and requires a better plan, more impl. time and better testing. I simply went for the simplest solution as we don't have the requirements yet. Anyway, you should open an issue or a discussion here at GH for the topic.

@imobachgs imobachgs mentioned this pull request May 26, 2025
imobachgs added a commit that referenced this pull request May 26, 2025
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.

3 participants