[pve-devel] [PATCH docs 1/2] fix #5140: qdevice SSH access documentation

[pve-devel] [PATCH docs 1/2] fix #5140: qdevice SSH access documentation

[Fabian Grünbichler]

rephrase in order to not make it seem like password-based access is required.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
---
 pvecm.adoc | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/pvecm.adoc b/pvecm.adoc
index 1f78585..03fba9f 100644
--- a/pvecm.adoc
+++ b/pvecm.adoc
@@ -1052,13 +1052,13 @@ pve# pvecm qdevice setup <QDEVICE-IP>
 
 The SSH key from the cluster will be automatically copied to the QDevice.
 
-NOTE: Make sure that the SSH configuration on your external server allows root
-login via password, if you are asked for a password during this step.
+NOTE: Make sure to setup key-based access for the root user on your external
+server, or temporarily allow root login with password during the setup phase.
 If you receive an error such as 'Host key verification failed.' at this
 stage, running `pvecm updatecerts` could fix the issue.
 
-After you enter the password and all the steps have successfully completed, you
-will see "Done". You can verify that the QDevice has been set up with:
+After all the steps have successfully completed, you will see "Done". You can
+verify that the QDevice has been set up with:
 
 ----
 pve# pvecm status
-- 
2.39.2

[pve-devel] [PATCH docs 2/2] partially fix #5139: document qnetd port requirement

[Fabian Grünbichler]

we could also add a check somewhere in pve-cluster for this.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
---
 pvecm.adoc | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/pvecm.adoc b/pvecm.adoc
index 03fba9f..adfc33f 100644
--- a/pvecm.adoc
+++ b/pvecm.adoc
@@ -1100,6 +1100,9 @@ columns:
    https://manpages.debian.org/bookworm/libvotequorum-dev/votequorum_qdevice_master_wins.3.en.html].
 * `NR`: QDevice is not registered.
 
+NOTE: If your Qdevice is listed as `Not Alive` (`NA` in the output above),
+ensure that port "5403" of your external server is reachable via TCP/IP!
+
 
 Frequently Asked Questions
 ~~~~~~~~~~~~~~~~~~~~~~~~~~
-- 
2.39.2

Re: [pve-devel] [PATCH docs 2/2] partially fix #5139: document qnetd port requirement

[Fiona Ebner]

Am 21.12.23 um 10:29 schrieb Fabian Grünbichler:
> we could also add a check somewhere in pve-cluster for this.
> 
> Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
> ---
>  pvecm.adoc | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/pvecm.adoc b/pvecm.adoc
> index 03fba9f..adfc33f 100644
> --- a/pvecm.adoc
> +++ b/pvecm.adoc
> @@ -1100,6 +1100,9 @@ columns:
>     https://manpages.debian.org/bookworm/libvotequorum-dev/votequorum_qdevice_master_wins.3.en.html].
>  * `NR`: QDevice is not registered.
>  
> +NOTE: If your Qdevice is listed as `Not Alive` (`NA` in the output above),

I'd change this to "QDevice", which is what all the other text uses
(just not the output for the pvecm command).

> +ensure that port "5403" of your external server is reachable via TCP/IP!

I'd prefer to drop the quotes or even use `5403`. We could also mention
"(default port of the qnetd server)".

Let me know what you think. I can squash in the changes we agree upon
when applying.

[pve-devel] applied: [PATCH docs 2/2] partially fix #5139: document qnetd port requirement

[Thomas Lamprecht]

Am 21/12/2023 um 10:29 schrieb Fabian Grünbichler:
> we could also add a check somewhere in pve-cluster for this.
> 
> Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
> ---
>  pvecm.adoc | 3 +++
>  1 file changed, 3 insertions(+)
> 
>

applied, with Fiona's proposed changes squashed in as they made
sense to me, thanks!

[pve-devel] applied: [PATCH docs 1/2] fix #5140: qdevice SSH access documentation

[Thomas Lamprecht]

Am 21/12/2023 um 10:29 schrieb Fabian Grünbichler:
> rephrase in order to not make it seem like password-based access is required.
> 
> Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
> ---
>  pvecm.adoc | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
>

applied, thanks!