Re: [PATCH] add a way to disable xen's udev script.

[Vincent Hanquez <vincent.hanquez@eu.citrix.com>]

On 06/14/2011 02:36 PM, Ian Campbell wrote:
> On Mon, 2011-06-13 at 09:58 +0100, Vincent Hanquez wrote:
> Oh, right, that makes sense, more so than the cfg file based scheme I
> suggested (although eventually I expect that the will be needed too to
> express the admin's wishes, that's not relevant here). Thanks for
> explaining the actual usage.

Right, I guess i should have started with that. it would have same us both some 
time.

> I think we could make it opt-in for toolstacks which want it rather than
> opt-out for those that don't, since only xend and xl want them (I
> think???) that seems like a manageable change. The xencommons initscript
> might work for touching the file, but perhaps the xend initscript and
> something internal to libxl/xl (which doesn't have it's own initscript)
> would be better. Only question is who removes it in the xl case.
>
> Since the file is automatically managed I don't think /etc is the right
> place -- probably somewhere like /var/run (which handily gets it cleaned
> up on reboot too).

Yes. I don't mind too much about the details, /run (with a fallback to /var/run 
for old distrib) is probably better indeed.

I'm not sure about xl, i had the idea that you have to switch away from xend 
when using xl by using an initscript. If that's not the case, then probably 
something internal would work.

>> This feature has been discussed with toolstack people, patch has been
>> send to maintainers already and it's about to be committed in one case
>> (on two) provided it pass my tests i've left running over the weekend.
>
> Sounds interesting, do you have a link? I didn't see anything in the
> xen-api archives, but perhaps it was elsewhere? (or is this a reference
> some other toolstack?)

No, unfortunately there's no record of the discussion since we had it directly.

The udev module has been merged in the XCP tree [1]; The "only" thing left is 
switching the hotplug module to be using this module. In XCI the full patch is 
in (but I don't have any public link to it).

[1] 
https://github.com/xen-org/xen-api-libs/commit/d6db355a4df93fafcf42cfb05df92dbae6a3b8f2

-- 
Vincent