public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Gilles Pietri <contact+dev@gilouweb.com>
To: pve-devel@lists.proxmox.com
Subject: Re: [pve-devel] [PATCH SERIES docs/ha-manager/manager v3] close #6144: add ui button + api for node maintenance mode
Date: Wed, 4 Feb 2026 18:57:52 +0100	[thread overview]
Message-ID: <da4d4bcd-fcff-4ec6-86f6-d387002963d9@gilouweb.com> (raw)
In-Reply-To: <CALn9RMe-jsDq30KfMSQxfYWe9Lghfk0WVtn_fdfFkB7TBAZt3Q@mail.gmail.com>

Le 26/01/2026 à 02:09, Thomas Skinner a écrit :
> On Mon, Dec 22, 2025 at 8:09 AM Thomas Skinner <thomas@atskinner.net> wrote:
>>
>> On Sat, Oct 25, 2025 at 1:38 PM Thomas Skinner <thomas@atskinner.net> wrote:
>>>
>>> Continued work on node maintenance mode button in UI.
>>>
>>> Changes since V2:
>>> - Adjust permissions from Sys.Console to Sys.Modify
>>> - Fix typo in description
>>>
>>> docs:
>>>
>>> Thomas Skinner (1):
>>>    add docs for maintenance mode buttons in UI
>>>
>>>   ha-manager.adoc | 22 ++++++++++++++++++++--
>>>   1 file changed, 20 insertions(+), 2 deletions(-)
>>>
>>>
>>> ha-manager:
>>>
>>> Thomas Skinner (1):
>>>    add additional api field for lrm_mode in status check
>>>
>>>   src/PVE/API2/HA/Status.pm | 7 +++++++
>>>   1 file changed, 7 insertions(+)
>>>
>>>
>>> manager:
>>>
>>> Thomas Skinner (2):
>>>    add api setter for node maintenance mode
>>>    add UI for node maintenance enable/disable
>>>
>>>   PVE/API2/Nodes.pm             | 38 +++++++++++++++
>>>   www/manager6/ha/StatusView.js | 92 +++++++++++++++++++++++++++++++++++
>>>   2 files changed, 130 insertions(+)
>>>
>>>
>>> --
>>> 2.47.3
>>>
>>
>> Hello! Any updates on getting this patch series accepted?
> 
> Hello! Checking in again to see if this patch series can be applied.

Hi,

I'll try those in my test environment, but assuming it works, that would 
be nice to have. It may have been feedback from former versions, but 
maybe have that as an option when right-clicking the hosts would be nice 
as well?

Regards,

Gilles



      reply	other threads:[~2026-02-04 17:57 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-25 18:38 Thomas Skinner
2025-10-25 18:38 ` [pve-devel] [PATCH ha-manager v3 1/1] add additional api field for lrm_mode in status check Thomas Skinner
2025-10-25 18:38 ` [pve-devel] [PATCH manager v3 1/2] add api setter for node maintenance mode Thomas Skinner
2025-10-25 18:38 ` [pve-devel] [PATCH docs v3 1/1] add docs for maintenance mode buttons in UI Thomas Skinner
2025-10-25 18:38 ` [pve-devel] [PATCH manager v3 2/2] add UI for node maintenance enable/disable Thomas Skinner
2025-12-22 14:09 ` [pve-devel] [PATCH SERIES docs/ha-manager/manager v3] close #6144: add ui button + api for node maintenance mode Thomas Skinner
2026-01-26  1:09   ` Thomas Skinner
2026-02-04 17:57     ` Gilles Pietri [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=da4d4bcd-fcff-4ec6-86f6-d387002963d9@gilouweb.com \
    --to=contact+dev@gilouweb.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal