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(¶m);
+
+ 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(¶m);
+
+ 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
next prev parent 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