Skip to content

Conversation

@delafthi
Copy link
Contributor

@delafthi delafthi commented Nov 1, 2025

  • restructure module from podman-linux to platform-agnostic podman
  • move linux-specific implementation to modules/services/podman/linux/
  • add darwin module with declarative machine management
  • implement launchd-based watchdog for auto-starting machines
  • maintains backward compatibility with existing linux functionality

Checklist

  • Change is backwards compatible.

  • Code formatted with nix fmt or
    nix-shell -p treefmt nixfmt deadnix keep-sorted --run treefmt.

  • Code tested through nix run .#tests -- test-all or
    nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.
    • Generate a news entry. See News
    • Basic tests added. See Tests
  • If this PR adds an exciting new feature or contains a breaking change.

    • Generate a news entry. See News

@delafthi delafthi force-pushed the push-rplzqunssuyz branch 2 times, most recently from 5612f58 to ad8d7ec Compare November 1, 2025 21:28
@delafthi delafthi force-pushed the push-rplzqunssuyz branch 9 times, most recently from 811381d to cb6e129 Compare November 4, 2025 15:50
@delafthi delafthi marked this pull request as ready for review November 4, 2025 15:50
@delafthi delafthi requested a review from khaneliman November 4, 2025 15:50
@khaneliman khaneliman requested review from bamhm182 and n-hass November 4, 2025 16:37
@n-hass
Copy link
Contributor

n-hass commented Nov 4, 2025

I've just had a skim and will take a more thorough look later. Thanks for the contribution.

I don't believe it's conventional to namespace service settings under the platform the feature is available for, eg the services.podman.darwin.machines. Instead give the user a useful error if they configure a machine on a non-Darwin system.

Any thoughts anyone else?

@khaneliman
Copy link
Collaborator

khaneliman commented Nov 5, 2025

I agree I don't really like darwin namespacing really... Should be able to hide options / disable options on darwin if they can't be used.

          visible = pkgs.stdenv.hostPlatform.isLinux;
          readOnly = pkgs.stdenv.hostPlatform.isDarwin;

or just don't make them available through not importing ?

@delafthi
Copy link
Contributor Author

delafthi commented Nov 5, 2025

Any thoughts anyone else?

I agree. An implementation without a darwin namespace would be nicer.

@delafthi delafthi force-pushed the push-rplzqunssuyz branch 3 times, most recently from 5da68ee to 1f40bb5 Compare November 5, 2025 09:39
- restructure module from `podman-linux` to platform-agnostic `podman`
- move linux-specific implementation to `modules/services/podman/linux/`
- add darwin module with declarative machine management
- implement launchd-based watchdog for auto-starting machines
- maintains backward compatibility with existing linux functionality
delafthi added a commit to delafthi/dotfiles that referenced this pull request Nov 5, 2025
This change removes our custom podman-darwin module that manually managed
Podman machines on macOS. Instead, we now rely on home-manager's native
podman module which includes proper launchd integration for machine
management.

Waits for: nix-community/home-manager#8102
delafthi added a commit to delafthi/dotfiles that referenced this pull request Nov 5, 2025
This change removes our custom podman-darwin module that manually managed
Podman machines on macOS. Instead, we now rely on home-manager's native
podman module which includes proper launchd integration for machine
management.

Waits for: nix-community/home-manager#8102
delafthi added a commit to delafthi/dotfiles that referenced this pull request Nov 7, 2025
This change removes our custom podman-darwin module that manually managed
Podman machines on macOS. Instead, we now rely on home-manager's native
podman module which includes proper launchd integration for machine
management.

Waits for: nix-community/home-manager#8102
delafthi added a commit to delafthi/dotfiles that referenced this pull request Nov 7, 2025
This change removes our custom podman-darwin module that manually managed
Podman machines on macOS. Instead, we now rely on home-manager's native
podman module which includes proper launchd integration for machine
management.

Waits for: nix-community/home-manager#8102
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