[dm-devel] [PATCH resend] multipath-tools: add DellEMC/ME4 (PowerVault ME4) to hardware table

[dm-devel] [PATCH resend] multipath-tools: add DellEMC/ME4 (PowerVault ME4) to hardware table

[Xose Vazquez Perez]

Info from: https://www.delltechnologies.com/asset/fr-fr/products/storage/industry-market/PowerVault_ME4_Series_and_Linux.pdf

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM-DEVEL ML <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 libmultipath/hwtable.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index 763982cd..fc85283b 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -398,6 +398,15 @@ static struct hwentry default_hw[] = {
 		.no_path_retry = 3,
 		.fast_io_fail  = 15,
 	},
+	{
+		/* PowerVault ME4 */
+		.vendor        = "DellEMC",
+		.product       = "ME4",
+		.pgpolicy      = GROUP_BY_PRIO,
+		.prio_name     = PRIO_ALUA,
+		.hwhandler     = "1 alua",
+		.pgfailback    = -FAILBACK_IMMEDIATE,
+	},
 	/*
 	 * Fujitsu
 	 */
-- 
2.32.0

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

[dm-devel] [PATCH resend] multipath-tools: delete duplicate RDAC maintainer entries

[Xose Vazquez Perez]

Already at NetApp.

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM-DEVEL ML <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 libmultipath/hwtable.c | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index 763982cd..1cae0ca5 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -439,10 +439,7 @@ static struct hwentry default_hw[] = {
 		.no_path_retry = 10,
 	},
 	{
-		/*
-		 * ETERNUS AB/HB
-		 * Maintainer: NetApp RDAC team <ng-eseries-upstream-maintainers@netapp.com>
-		 */
+		/* ETERNUS AB/HB */
 		.vendor        = "FUJITSU",
 		.product       = "ETERNUS_AHB",
 		.bl_product    = "Universal Xport",
@@ -762,11 +759,7 @@ static struct hwentry default_hw[] = {
 	 * Lenovo
 	 */
 	{
-		/*
-		 * DE Series
-		 *
-		 * Maintainer: NetApp RDAC team <ng-eseries-upstream-maintainers@netapp.com>
-		 */
+		/* DE Series */
 		.vendor        = "LENOVO",
 		.product       = "DE_Series",
 		.bl_product    = "Universal Xport",
-- 
2.32.0

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

[dm-devel] [PATCH resend] multipath-tools: dm-devel is a closed ml

[Xose Vazquez Perez]

Just for subscribers

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM-DEVEL ML <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/README.md b/README.md
index b15c265f..2d3d1023 100644
--- a/README.md
+++ b/README.md
@@ -77,7 +77,7 @@ Maintainer
 ==========
 
 Christophe Varoqui <christophe.varoqui@opensvc.com>
-Device-mapper development mailing list <dm-devel@redhat.com>
+Device-mapper development mailing list (subscribers-only) <dm-devel@redhat.com>
 
 
 Licence
-- 
2.33.0

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

[dm-devel] [PATCH resend] multipath-tools: remove Hannes as IBM arrays maintainer

[Xose Vazquez Perez]

Reviewed-by: Hannes Reinecke <hare@suse.de>
Cc: Hannes Reinecke <hare@suse.de>
Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM-DEVEL ML <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 libmultipath/hwtable.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index 763982cd..11282699 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -482,8 +482,6 @@ static struct hwentry default_hw[] = {
 	},
 	/*
 	 * IBM
-	 *
-	 * Maintainer: Hannes Reinecke <hare@suse.de>
 	 */
 	{
 		/* ProFibre 4000R */
-- 
2.32.0

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH resend] multipath-tools: dm-devel is a closed ml

[Martin Wilck]

On Wed, 2021-12-01 at 23:35 +0100, Xose Vazquez Perez wrote:
> Just for subscribers
> 
> Cc: Martin Wilck <mwilck@suse.com>
> Cc: Benjamin Marzinski <bmarzins@redhat.com>
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: DM-DEVEL ML <dm-devel@redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> ---
>  README.md | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

According to Alasdair's statement,
(https://listman.redhat.com/archives/dm-devel/2021-October/msg00037.html),
the list is "moderated for non-subscribers", not strictly subscribers-
only.

Moreover, the same README has "(subscribers-only)" just a few lines
above, in the "Mailing List" section.

Regards
Martin


--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH resend] multipath-tools: delete duplicate RDAC maintainer entries

[Martin Wilck]

@Netapp, do you approve?

On Wed, 2021-12-01 at 23:35 +0100, Xose Vazquez Perez wrote:
> Already at NetApp.
> 
> Cc: Martin Wilck <mwilck@suse.com>
> Cc: Benjamin Marzinski <bmarzins@redhat.com>
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: DM-DEVEL ML <dm-devel@redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> ---
>  libmultipath/hwtable.c | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index 763982cd..1cae0ca5 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -439,10 +439,7 @@ static struct hwentry default_hw[] = {
>                 .no_path_retry = 10,
>         },
>         {
> -               /*
> -                * ETERNUS AB/HB
> -                * Maintainer: NetApp RDAC team
> <ng-eseries-upstream-maintainers@netapp.com>
> -                */
> +               /* ETERNUS AB/HB */
>                 .vendor        = "FUJITSU",
>                 .product       = "ETERNUS_AHB",
>                 .bl_product    = "Universal Xport",
> @@ -762,11 +759,7 @@ static struct hwentry default_hw[] = {
>          * Lenovo
>          */
>         {
> -               /*
> -                * DE Series
> -                *
> -                * Maintainer: NetApp RDAC team
> <ng-eseries-upstream-maintainers@netapp.com>
> -                */
> +               /* DE Series */
>                 .vendor        = "LENOVO",
>                 .product       = "DE_Series",
>                 .bl_product    = "Universal Xport",

I'd like to have this acked by Netapp before merging it. Actually, I'd
prefer to have it acked by Fujitsu and Lenovo, too, but we don't have
contact addresses for them, so be it.

My €0.02: I'd rather not have this, because it's not obvious to
everybody that these arrays are actually Netapp systems and are thus
maintained not by their vendors but Netapp.

Martin


--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH resend] multipath-tools: delete duplicate RDAC maintainer entries

[Xose Vazquez Perez]

On 12/2/21 08:56, Martin Wilck wrote:

> 
> I'd like to have this acked by Netapp before merging it. Actually, I'd
> prefer to have it acked by Fujitsu and Lenovo, too, but we don't have
> contact addresses for them, so be it.
> 
> My €0.02: I'd rather not have this, because it's not obvious to
> everybody that these arrays are actually Netapp systems and are thus
> maintained not by their vendors but Netapp.

There are 25 RDAC arrays in hwtable, only two of them(those included in
the patch) and the NetApp one have a "Maintainer" entry.
And they can easily identified, .checker_name and .prio_name are RDAC.

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH resend] multipath-tools: delete duplicate RDAC maintainer entries

[Schremmer, Steven]

> From: Xose Vazquez Perez <xose.vazquez@gmail.com>
> Sent: Thursday, December 2, 2021 6:42 AM
> 
> On 12/2/21 08:56, Martin Wilck wrote:
> 
> >
> > I'd like to have this acked by Netapp before merging it. Actually, I'd
> > prefer to have it acked by Fujitsu and Lenovo, too, but we don't have
> > contact addresses for them, so be it.
> >
> > My €0.02: I'd rather not have this, because it's not obvious to
> > everybody that these arrays are actually Netapp systems and are thus
> > maintained not by their vendors but Netapp.
> 
> There are 25 RDAC arrays in hwtable, only two of them(those included in
> the patch) and the NetApp one have a "Maintainer" entry.
> And they can easily identified, .checker_name and .prio_name are RDAC.

NetApp would like those Maintainer entries to remain in the file as we have
agreements with Fujitsu and Lenovo to handle it this way.

Thanks,
Steve

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH resend] multipath-tools: dm-devel is a closed ml

[Xose Vazquez Perez]

On 12/2/21 08:45, Martin Wilck wrote:

> On Wed, 2021-12-01 at 23:35 +0100, Xose Vazquez Perez wrote:
>> Just for subscribers
>>
>> Cc: Martin Wilck <mwilck@suse.com>
>> Cc: Benjamin Marzinski <bmarzins@redhat.com>
>> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
>> Cc: DM-DEVEL ML <dm-devel@redhat.com>
>> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
>> ---
>>   README.md | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
> 
> According to Alasdair's statement,
> (https://listman.redhat.com/archives/dm-devel/2021-October/msg00037.html),
> the list is "moderated for non-subscribers", not strictly subscribers-
> only.
> 
> Moreover, the same README has "(subscribers-only)" just a few lines
> above, in the "Mailing List" section.

"moderated for non-subscribers" should only be for very sporadic posters.
Contributors must be in the list.
Sometimes some messages get lost because it's human-operated. And it
also overloads ml admins with unnecessary extra work.

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH resend] multipath-tools: dm-devel is a closed ml

[Martin Wilck]

On Fri, 2021-12-03 at 19:17 +0100, Xose Vazquez Perez wrote:
> On 12/2/21 08:45, Martin Wilck wrote:
> 
> > On Wed, 2021-12-01 at 23:35 +0100, Xose Vazquez Perez wrote:
> > > Just for subscribers
> > > 
> > > Cc: Martin Wilck <mwilck@suse.com>
> > > Cc: Benjamin Marzinski <bmarzins@redhat.com>
> > > Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> > > Cc: DM-DEVEL ML <dm-devel@redhat.com>
> > > Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> > > ---
> > >   README.md | 2 +-
> > >   1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > 
> > According to Alasdair's statement,
> > (
> > https://listman.redhat.com/archives/dm-devel/2021-October/msg00037.html
> > ),
> > the list is "moderated for non-subscribers", not strictly
> > subscribers-
> > only.
> > 
> > Moreover, the same README has "(subscribers-only)" just a few lines
> > above, in the "Mailing List" section.
> 
> "moderated for non-subscribers" should only be for very sporadic
> posters.
> Contributors must be in the list.
> Sometimes some messages get lost because it's human-operated. And it
> also overloads ml admins with unnecessary extra work.
> 

Fair enough. Still, "subscribers-only" is factually wrong.

Martin


--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel