public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Shannon Sterz <s.sterz@proxmox.com>
To: pdm-devel@lists.proxmox.com
Subject: [pdm-devel] [PATCH datacenter-manager 1/2] cli: admin: add `support-eligibility` subcommand
Date: Mon, 15 Dec 2025 17:21:29 +0100	[thread overview]
Message-ID: <20251215162129.403124-3-s.sterz@proxmox.com> (raw)
In-Reply-To: <20251215162129.403124-2-s.sterz@proxmox.com>

add cli methods to allow updating and checking the subscription
information of pdm itself. before, the cli only had a cli method that
prints the subscription status for each remote without providing
information on whether the pdm host itself is eligible for support.

the term `support-eligiblity` for the subcommand was chosen over
`subscription`, because:

1. there are no subscriptions for pdm itself, only for the remotes it
is connected to. users may be confused that they can't set a
subscription here like they can with other proxmox products

2. avoid confusion with the `remote subscriptions` subcommand

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---
 cli/admin/src/main.rs                |  2 +
 cli/admin/src/support_eligibility.rs | 66 ++++++++++++++++++++++++++++
 2 files changed, 68 insertions(+)
 create mode 100644 cli/admin/src/support_eligibility.rs

diff --git a/cli/admin/src/main.rs b/cli/admin/src/main.rs
index e6a74df..f4cbc80 100644
--- a/cli/admin/src/main.rs
+++ b/cli/admin/src/main.rs
@@ -9,6 +9,7 @@ use proxmox_router::RpcEnvironment;
 use proxmox_schema::api;
 
 mod remotes;
+mod support_eligibility;
 
 fn main() {
     //pbs_tools::setup_libc_malloc_opts(); // TODO: move from PBS to proxmox-sys and uncomment
@@ -36,6 +37,7 @@ fn main() {
             "report",
             CliCommand::new(&API_METHOD_GENERATE_SYSTEM_REPORT),
         )
+        .insert("support-eligibility", support_eligibility::cli())
         .insert("versions", CliCommand::new(&API_METHOD_GET_VERSIONS));
 
     let mut rpcenv = CliEnvironment::new();
diff --git a/cli/admin/src/support_eligibility.rs b/cli/admin/src/support_eligibility.rs
new file mode 100644
index 0000000..0f2cfca
--- /dev/null
+++ b/cli/admin/src/support_eligibility.rs
@@ -0,0 +1,66 @@
+use serde_json::{json, Value};
+
+use proxmox_router::cli::{
+    format_and_print_result_full, get_output_format, CliCommand, CliCommandMap,
+    CommandLineInterface,
+};
+use proxmox_router::{ApiHandler, RpcEnvironment};
+use proxmox_schema::api;
+
+pub fn cli() -> CommandLineInterface {
+    CliCommandMap::new()
+        .insert("get", CliCommand::new(&API_METHOD_SUPPORT_ELIGIBILITY))
+        .insert(
+            "update",
+            CliCommand::new(&API_METHOD_UPDATE_SUPPORT_ELIGIBILITY),
+        )
+        .into()
+}
+
+#[api]
+/// Get the support eligibility information.
+async fn support_eligibility(
+    param: Value,
+    rpcenv: &mut dyn RpcEnvironment,
+) -> Result<(), anyhow::Error> {
+    let info = &server::api::nodes::subscription::API_METHOD_GET_SUBSCRIPTION;
+
+    let mut data = match info.handler {
+        ApiHandler::Async(handler) => (handler)(json!({}), info, rpcenv).await?,
+        _ => unreachable!(),
+    };
+
+    let output_format = get_output_format(&param);
+
+    format_and_print_result_full(
+        &mut data,
+        &info.returns,
+        &output_format,
+        &Default::default(),
+    );
+    Ok(())
+}
+
+#[api]
+/// Update the support eligibility information.
+async fn update_support_eligibility(
+    param: Value,
+    rpcenv: &mut dyn RpcEnvironment,
+) -> Result<(), anyhow::Error> {
+    let info = &server::api::nodes::subscription::API_METHOD_CHECK_SUBSCRIPTION;
+
+    let mut data = match info.handler {
+        ApiHandler::Async(handler) => (handler)(json!({}), info, rpcenv).await?,
+        _ => unreachable!(),
+    };
+
+    let output_format = get_output_format(&param);
+
+    format_and_print_result_full(
+        &mut data,
+        &info.returns,
+        &output_format,
+        &Default::default(),
+    );
+    Ok(())
+}
-- 
2.47.3



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


  reply	other threads:[~2025-12-15 16:21 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-15 16:21 [pdm-devel] [PATCH datacenter-manager 0/2] support eligibility checking in the cli Shannon Sterz
2025-12-15 16:21 ` Shannon Sterz [this message]
2025-12-15 16:21 ` [pdm-devel] [PATCH datacenter-manager 2/2] server: add the new `support-eligibility` subcommand to the report Shannon Sterz
2025-12-16  8:14 ` [pdm-devel] [PATCH datacenter-manager 0/2] support eligibility checking in the cli Lukas Wagner
2025-12-16  8:57   ` Maximiliano Sandoval
2025-12-16 17:24 ` [pdm-devel] applied: " Thomas Lamprecht

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=20251215162129.403124-3-s.sterz@proxmox.com \
    --to=s.sterz@proxmox.com \
    --cc=pdm-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