Re: [pve-devel] [PATCH docs 1/1] add docs for maintenance mode buttons in UI

["Daniel Kral" <d.kral@proxmox.com>]

On Tue Aug 26, 2025 at 6:05 PM CEST, Thomas Skinner wrote:
> On Tue, Aug 26, 2025 at 4:44 AM Daniel Kral <d.kral@proxmox.com> wrote:
>> On Mon Aug 25, 2025 at 6:11 AM CEST, Thomas Skinner wrote:
>> > Signed-off-by: Thomas Skinner <thomas@atskinner.net>
>> > ---
>> >  ha-manager.adoc | 21 +++++++++++++++++++++
>> >  1 file changed, 21 insertions(+)
>> >
>> > diff --git a/ha-manager.adoc b/ha-manager.adoc
>> > index f16cfbb..b41a73b 100644
>> > --- a/ha-manager.adoc
>> > +++ b/ha-manager.adoc
>> > @@ -1115,7 +1115,19 @@ maintenance mode is disabled and the node is back online.
>> >  Currently you can enabled or disable the maintenance mode using the ha-manager
>> >  CLI tool.
>> >
>> > +Maintenance mode can be enabled or disabled using the buttons on the HA pane in
>> > +the {pve} web UI or by using the ha-manager CLI tool.
>> Also, I think it would be nicer to integrate that into the former
>> sentence, for example:
>>
>> Currently you can enabled or disable the maintenance mode in the HA
>> status panel in the web UI or using the ha-manager CLI tool.
>
> I actually meant for that one to be gone and replaced with the added
> sentence instead. I must've missed the removal on my local branch
> patch application. Is the replacement sentence okay or should it be
> removed and the existing sentence updated?

Ah I see, then both are fine even though I'd replace the 'using the
buttons on the HA panel' with just 'in the HA status panel' as you
describe how it's done later already, but that's just nit-picking on my
side ;-).


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel