On Tue, Sep 04, 2018 at 04:10:38PM +0200, Roberto Resoli wrote: > Il 01/09/2018 09:59, Roland Kammerer ha scritto: > > It is exactly the job of "systemctl edit" to*not* mess with system > > files shiped by packages. What it would do, is to create this file: > > > > /etc/systemd/system/linstor-satellite.service.d/override.conf > > > > And there you can, as the name says, override settings. It looks like > > some versions of systemd then required the section + a reset, as in: > > [Service] > > ExecStart= > > ExecStart=mynewsetting > > This is exactly the case for linstor-satellite.service (I tried all > variations)
Yeah, I also checked, and already updated the UG. So far I did not send it to the web-dude, but "soon", then it is also on docs.linbit.com. Till then (GH slightly messes up the mark up): https://github.com/LINBIT/linbit-documentation/blob/master/UG9/en/proxmox-linstor.adoc > > Probably I have to add that detail to the documentation, if it still > > applies to the version proxmox ships. > > Probably, but I think we are anyway condemned to live with this sort of > undocumented systemd behaviour. Hm, don't know if it is undocumented, guess not, usually the systemd documentation is pretty good. Anyways, this will change a bit in the future. I don't like that people have to copy the whole ExecStart line. What if we then change default locations? (Unlikely, but still). So in some future version linstor-statellite will also understand a "LS_KEEP_RES" (or similar) environment variable, then the whole thing becomes: [Service] Environment=LS_KEEP_RES=vm-100 I like that a lot more. Regards, rck _______________________________________________ drbd-user mailing list drbd-user@lists.linbit.com http://lists.linbit.com/mailman/listinfo/drbd-user