[OE-core] [PATCHv2] mdadm:add mdmonitor.service

Richard Purdie richard.purdie at linuxfoundation.org
Mon Jul 29 09:24:10 UTC 2019


On Mon, 2019-07-29 at 09:33 +0800, Zang Ruochen wrote:
> -The original file mdmonitor.service is as follows:
> ...
> > [Service]
> > Environment=  MDADM_MONITOR_ARGS=--scan
> > EnvironmentFile=-/run/sysconfig/mdadm
> > ExecStartPre=-/usr/lib/mdadm/mdadm_env.sh
> > ExecStart=/sbin/mdadm --monitor -y $MDADM_MONITOR_ARGS
> ...
> -It has a syntax error and it doesn't work properly, 
>  so add new file mdmonitor.service to overwrite it.

This is better as it tells us what the problem is but it still isn't
quite what I asked for.

Can we patch the file that mdadm is installing so the difference is
clear to anyone reading the recipe? The patch will also have comments
as above so that people don't have to find this commit to understand
why we're changing it and what the problem is. If/as/when upstream
fixes it we'll also be able to drop the patch, rather than not knowing
if/when its fixed.

Thanks,

Richard



More information about the Openembedded-core mailing list