all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH docs] cephx-migration: slightly rephrase and extend for clarity
@ 2026-09-15 10:49 Friedrich Weber
  2026-09-15 13:17 ` applied: " Fabian Grünbichler
  0 siblings, 1 reply; 2+ messages in thread
From: Friedrich Weber @ 2026-09-15 10:49 UTC (permalink / raw)
  To: pve-devel

Namely:

- In the "Before You Begin" section, move the rolling restart to its
  own list item.

- When refreshing CephFS mounts, add a note that VMs with ISOs from
  a CephFS mount may need to be migrated (thanks Manuel!)

- In the finishing step, do not directly state the expected command
  to reduce the risk of users just copy-pasting the command (thanks
  Jonas!)

Signed-off-by: Friedrich Weber <f.weber@proxmox.com>
---
 pveceph.adoc | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/pveceph.adoc b/pveceph.adoc
index d0549ab..c46226e 100644
--- a/pveceph.adoc
+++ b/pveceph.adoc
@@ -1365,8 +1365,8 @@ without options to check the current state and follow the next steps it prints.
 .Before You Begin
 * Upgrade `pve-manager` to 9.2.17 or newer and install the latest Ceph
   packages on every node. Staged client-key rotation requires Ceph
-  19.2.6-pve3, 20.2.4-pve3, or newer on every monitor. Then complete the
-  rolling restart of all Ceph services.
+  19.2.6-pve3, 20.2.4-pve3, or newer on every monitor.
+* Then complete the rolling restart of all Ceph services.
 * Resolve health warnings and errors not listed in
   xref:pveceph_cephx_health_check_reference[Health Check Reference]. Ensure
   every node is reachable over SSH.
@@ -1464,6 +1464,9 @@ until confirmation makes the new key current and invalidates the old one.
 * The helper refreshes idle CephFS mounts and leaves busy or unresponsive
   mounts alone. Once nothing uses them, rerun the helper with `--apply` to
   retry the outstanding mounts.
+  If your virtual machines have ISOs from a CephFS mount attached, you may have
+  to detach the ISOs or migrate affected virtual machines to a different node
+  to be able to refresh the CephFS mount on that node.
 * For clients outside {pve}, distribute the staged credential from the
   xref:pveceph_cephx_external_copies[managed key files], then restart or remount
   those clients.
@@ -1491,13 +1494,8 @@ fail. Do not use `--force` to bypass a blocker.
 
 After checking disconnected clients and external copies, run the exact
 command the dry run prints. When every rotation is ready and no key needs the
-old cipher, it is:
-
-[source,bash]
-----
-/usr/share/pve-manager/migrations/pve-cephx-rotate-service-keys --apply \
-    --confirm-all-clients-refreshed --restrict-ciphers
-----
+old cipher, it will print the options `--apply --confirm-all-clients-refreshed
+--restrict-ciphers`.
 
 If any client remains incompatible, keep its key unchanged and the old cipher
 enabled rather than restricting it. You can
-- 
2.47.3





^ permalink raw reply related	[flat|nested] 2+ messages in thread

* applied: [PATCH docs] cephx-migration: slightly rephrase and extend for clarity
  2026-09-15 10:49 [PATCH docs] cephx-migration: slightly rephrase and extend for clarity Friedrich Weber
@ 2026-09-15 13:17 ` Fabian Grünbichler
  0 siblings, 0 replies; 2+ messages in thread
From: Fabian Grünbichler @ 2026-09-15 13:17 UTC (permalink / raw)
  To: pve-devel, Friedrich Weber


On Tue, 15 Sep 2026 12:49:19 +0200, Friedrich Weber wrote:
> Namely:
> 
> - In the "Before You Begin" section, move the rolling restart to its
>   own list item.
> 
> - When refreshing CephFS mounts, add a note that VMs with ISOs from
>   a CephFS mount may need to be migrated (thanks Manuel!)
> 
> [...]

Applied, thanks!

[1/1] cephx-migration: slightly rephrase and extend for clarity
      commit: 14247d9da540e08d40268d80e0d6cd2749edd588

Best regards,
-- 
Fabian Grünbichler <f.gruenbichler@proxmox.com>




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-09-15 13:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-15 10:49 [PATCH docs] cephx-migration: slightly rephrase and extend for clarity Friedrich Weber
2026-09-15 13:17 ` applied: " Fabian Grünbichler

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal