@mguentner Sorry for the delay on this. Do you mind doing a git rebase? Please keep in mind that PermissionsStartOnly has been deprecated.

@aanderse No worries. Thanks for taking a look.

mxisd development has made some progress in the meantime:
1.4.x seems stable enough. I will test it very soon.
I will might simply push to that branch and we can omit the 1.3.1.

OK sounds good. I'll wait to hear back on a version update.

@aanderse Ready :)

@GrahamcOfBorg build mxisd
@GrahamcOfBorg test mxisd

So it appears someone has removed mxisd from nixos/tests/all-tests.nix 😞
@mguentner bonus points if you push another commit on top of this to add mxisd (back?) to all-tests.nix

@aanderse Pushed

@mguentner thanks!
@GrahamcOfBorg test mxisd

@mguentner can you briefly explain why the code in preStart is no longer required? It looks like you're moving from a mutable config file to immutable so are there any backward compatibility/upgrade issues to consider?

@aanderse
At some point mxisd required the config to be named application.yaml. Pointing it to the /nix/store path produced by writeText did not work because of the filename like hash-something-application.yaml.

That's why I opted for simply copying the config to the work directory. I used chmod 444 to prevent mutation.

So it was done because of the configuration requirements of mxisd < 1.3, not because it mutates the config file during runtime.
I don't expect / didn't run into any upgrade issues because of that.

If an existing config has something in extraConfig however, this might need to be adjusted as config format changed slightly. It's no longer some yaml-inspired format but standard-yaml.

More on that: https://github.com/kamax-matrix/mxisd/wiki/Upgrade-Notes#v130

@mguentner you have been super helpful in my review of this change. Thank you!