<div dir="ltr">Hi,<div><br></div><div>To my knowledge, systemd has not been integrated in all distributions.</div><div>It would be nice not to break the build of multipath-tools on those environments.</div><div><br></div><div>
The current patchset breaks the build on non-systemd systems as soon as patch 06/13.</div><div>I can apply the patchset from 01/13 to 05/13 included, and let you refactor the systemd integration proper in a new series.</div>
<div><br></div><div>Is that ok with you ?</div><div><br></div><div>Best regards,<br></div><div>Christophe Varoqui</div><div><a href="http://www.opensvc.com">www.opensvc.com</a></div><div><br></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Nov 22, 2013 at 10:12 AM, Hannes Reinecke <span dir="ltr"><<a href="mailto:hare@suse.de" target="_blank">hare@suse.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 11/22/2013 12:17 AM, Benjamin Marzinski wrote:<br>
> I don't understand this option?  When I played around with it, it seems<br>
> like it only causes problems.  For instance.  If I remove all the<br>
> multipath devices with "multipath -F", then multipathd stops.<br>
> Incidentally, this pisses off the watchdog timer.<br>
><br>
> This pops up in /var/log/messages<br>
><br>
> Nov 21 11:04:05 ask-08 systemd[1]: multipathd.service watchdog timeout!<br>
> Nov 21 11:04:05 ask-08 systemd[1]: Unit multipathd.service entered<br>
> failed state<br>
><br>
> and checking the status shows the service as failed.<br>
><br>
> # service multipathd status<br>
> multipathd.service - Device-Mapper Multipath Device Controller<br>
>    Loaded: loaded (/usr/lib/systemd/system/multipathd.service; enabled)<br>
>    Active: failed (Result: watchdog) since Thu 2013-11-21 11:04:00 CST; 3min 25s ago<br>
>   Process: 22687 ExecStart=/sbin/multipathd -d -s -n (code=exited, status=0/SUCCESS)<br>
>    Status: "shutdown"<br>
><br>
</div>Yeah, because I forgot to set the exit status via sd_notify.<br>
I'll be updating a patchset to include that.<br>
<div class="im"><br>
> But the bigger issue is that multipathd is now stopped, and running<br>
> "multipath" doesn't bring it back again.<br>
<br>
</div>Ah. Yes, you are absolutely right, of course.<br>
I've forgot about this.<br>
<div class="im"><br>
> You will create the multipath<br>
> devices, but nothing will be monitoring them.  The same thing happens if<br>
> the multipathd service is started before the any multipathable devices<br>
> are discovered.  It starts up and then shuts back down (again tripping<br>
> the watchdog timer). When those devices finally get discovered, there is<br>
> nothing to listening for the uevents, so no multipath devices ever get<br>
> created.<br>
><br>
> I must be missing something here.<br>
><br>
</div>No, it's actually true what you've said.<br>
Multipathd can only listen to events if it's started.<br>
But that's precisely what systemd is for, right?<br>
Starting a service when something is written onto a socket?<br>
Guess it need some more work here.<br>
<br>
But I could finally do my long-term project of merging<br>
multipath and multipathd to share the same codebase, with<br>
multipath just using the multipathd CLI and having no logic<br>
on it's own.<br>
<br>
But in the light of this, this patch is indeed not working<br>
as designed, and should not be applied as of now.<br>
<br>
Which doesn't affect the other parts in the series, which<br>
_definitely_ should be applied.<br>
Christophe, how to proceed?<br>
<br>
Do you need a new patchset with the last patch removed,<br>
or are you fine with the current one and just not applying<br>
the last patch?<br>
<br>
Cheers,<br>
<br>
Hannes<br>
<span class="HOEnZb"><font color="#888888">--<br>
Dr. Hannes Reinecke                   zSeries & Storage<br>
<a href="mailto:hare@suse.de">hare@suse.de</a>                          <a href="tel:%2B49%20911%2074053%20688" value="+4991174053688">+49 911 74053 688</a><br>
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg<br>
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)<br>
</font></span></blockquote></div><br></div>