[dm-devel] [PATCH 0/2] multipath-tools: update docs

[dm-devel] [PATCH 0/2] multipath-tools: update docs

[Xose Vazquez Perez]

Xose Vazquez Perez (2):
  multipath-tools: update devel repo info in README.md
  multipath-tools: delete README.alua

 README.alua | 24 ------------------------
 README.md   | 10 +++-------
 2 files changed, 3 insertions(+), 31 deletions(-)
 delete mode 100644 README.alua

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>
-- 
2.37.1

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

[dm-devel] [PATCH 1/2] multipath-tools: update devel repo info in README.md

[Xose Vazquez Perez]

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 | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/README.md b/README.md
index f06f8cea..dcf51f20 100644
--- a/README.md
+++ b/README.md
@@ -3,7 +3,6 @@
 multipath-tools for Linux
 =========================
 
-
 https://github.com/opensvc/multipath-tools
 
 This package provides the following binaries to drive the Device Mapper multipathing driver:
@@ -42,14 +41,12 @@ Go to: https://github.com/opensvc/multipath-tools/tags
 Select a release-tag and then click on "zip" or "tar.gz".
 
 
-Source code
-===========
+Devel code
+==========
 
 To get latest devel code:
 
-    git clone https://github.com/opensvc/multipath-tools.git
-
-Github page: https://github.com/opensvc/multipath-tools
+    git clone -b queue https://github.com/openSUSE/multipath-tools
 
 
 Building multipath-tools
@@ -149,4 +146,3 @@ The multipath-tools source code is covered by several different licences.
 Refer to the individual source files for details.
 Source files which do not specify a licence are shipped under LGPL-2.0
 (see `LICENSES/LGPL-2.0`).
-
-- 
2.37.1

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

[dm-devel] [PATCH 2/2] multipath-tools: delete README.alua

[Xose Vazquez Perez]

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.alua | 24 ------------------------
 1 file changed, 24 deletions(-)
 delete mode 100644 README.alua

diff --git a/README.alua b/README.alua
deleted file mode 100644
index 5d2b1c64..00000000
--- a/README.alua
+++ /dev/null
@@ -1,24 +0,0 @@
-This is a rough guide, consult your storage device manufacturer documentation.
-
-ALUA is supported in some devices, but usually it's disabled by default.
-To enable ALUA, the following options should be changed:
-
-- EMC CLARiiON/VNX:
-   "Failover Mode" should be changed to "4" or "Active-Active mode(ALUA)-failover mode 4"
-
-- HPE 3PAR, Primera, and Alletra 9000:
-   "Host:" should be changed to "Generic-ALUA Persona 2 (UARepLun, SESLun, ALUA)".
-
-- Promise VTrak/Vess:
-   "LUN Affinity" and "ALUA" should be changed to "Enable", "Redundancy Type"
-   must be "Active-Active".
-
-- LSI/Engenio/NetApp RDAC class, as NetApp SANtricity E/EF Series and OEM arrays:
-   "Select operating system:" should be changed to "Linux DM-MP (Kernel 3.10 or later)".
-
-- NetApp ONTAP:
-   To check ALUA state: "igroup show -v <igroup_name>", and to enable ALUA:
-   "igroup set <igroup_name> alua yes".
-
-- Huawei OceanStor:
-   "Host Access Mode" should be changed to "Asymmetric".
-- 
2.37.1

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

Re: [dm-devel] [PATCH 0/2] multipath-tools: update docs

[Martin Wilck]

On Thu, 2022-07-14 at 21:05 +0200, Xose Vazquez Perez wrote:
> Xose Vazquez Perez (2):
>   multipath-tools: update devel repo info in README.md
>   multipath-tools: delete README.alua

Why complete delete it?

Martin

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

Re: [dm-devel] [PATCH 0/2] multipath-tools: update docs

[Xose Vazquez Perez]

On 7/14/22 21:38, Martin Wilck wrote:

> On Thu, 2022-07-14 at 21:05 +0200, Xose Vazquez Perez wrote:
>> Xose Vazquez Perez (2):
>>    multipath-tools: update devel repo info in README.md
>>    multipath-tools: delete README.alua
> 
> Why complete delete it?

It was written five years ago, and now maybe it is outdated/irrelevant.
Three options were provided; let it be, delete it, or move the info to README.md
Treat it like rfc.

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

Re: [dm-devel] [PATCH 0/2] multipath-tools: update docs

[Martin Wilck]

On Fri, 2022-07-15 at 01:09 +0200, Xose Vazquez Perez wrote:
> On 7/14/22 21:38, Martin Wilck wrote:
> 
> > On Thu, 2022-07-14 at 21:05 +0200, Xose Vazquez Perez wrote:
> > > Xose Vazquez Perez (2):
> > >    multipath-tools: update devel repo info in README.md
> > >    multipath-tools: delete README.alua
> > 
> > Why complete delete it?
> 
> It was written five years ago, and now maybe it is
> outdated/irrelevant.
> Three options were provided; let it be, delete it, or move the info
> to README.md
> Treat it like rfc.

For now, I'd move the information to README.md.

It clearly identifies the devices it's talking about. Sure, there
have been several generations of firmware updates since then, and
ALUA support is now mainstream on both the storage and host side.
But it also recommends to consult the storage documentation.
So, I believe keeping it won't hurt.

Perhaps we should just delete the sentence "usually it's disabled by
default"?

Martin

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

Re: [dm-devel] [PATCH 0/2] multipath-tools: update docs

[Xose Vazquez Perez]

On 7/15/22 09:33, Martin Wilck wrote:

 >> ALUA is supported in some devices, but usually it's disabled by default.
 >> To enable ALUA, the following options should be changed:

> Perhaps we should just delete the sentence "usually it's disabled by
> default"?

I do find the sentence to be accurate. "usually" does the trick.

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

Re: [dm-devel] [PATCH 0/2] multipath-tools: update docs

[Martin Wilck]

On Thu, 2022-07-14 at 21:05 +0200, Xose Vazquez Perez wrote:
> Xose Vazquez Perez (2):
>   multipath-tools: update devel repo info in README.md
>   multipath-tools: delete README.alua
> 
>  README.alua | 24 ------------------------
>  README.md   | 10 +++-------
>  2 files changed, 3 insertions(+), 31 deletions(-)
>  delete mode 100644 README.alua
> 
> 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>

Reviewed-by: Martin Wilck <mwilck@suse.com>

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