Re: [PATCH 2/2] mdadm: Update config man regarding default files and multi-keyword behavior.

[Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>]

On Tue, 15 Mar 2022 17:57:09 +0800
Coly Li <colyli@suse.de> wrote:

> On 3/15/22 4:55 PM, Lukasz Florczak wrote:
> > Simplify default and alternative config file and directory location
> > references from mdadm(8) as references to mdadm.conf(5). Add FILE
> > section in config man and explain order and conditions in which
> > default and alternative config files and directories are used.
> >
> > Update config man behavior regarding parsing order when multiple
> > keywords/config files are involved.
> >
> > Additionally add missing HOMECLUSTER keyword description.
> >
> > Signed-off-by: Lukasz Florczak <lukasz.florczak@linux.intel.com>  
> 
> 
> Hi Lukasz,
> 
> 
> This patch doesn't apply on branch 20220315-testing of the mdadm-CI 
> tree, could you please rebase this series on
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/colyli/mdadm.git 
> 20220315-testing
> 
> Then I will continue to test them.
> 

Hi Coly,
This is great to see that something is happening in upstream :)

I can see that you created branch where some patches were merged and
now you are reporting conflicts now. Our patches are based on last
mdadm commit (which is mdadm-4.2 ).
IMO you should try to apply them first on latest master and later
cherry-pick/ rebase them on top of your testing branch. This should
automatically resolve most of conflicts. Could you try that?

This is hard to follow all patches on list (especially that we cannot
determine in which order they will be applied). Preparing patches for
you testing branch (which could be changed in any moment), IMO is not a
good solution.

I really appreciate the work you put to enable upstream testing. If you
need some help, let me know.

Thanks,
Mariusz