From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate001.proxmox.com (gate001.proxmox.com [45.144.208.40]) by lore.proxmox.com (Postfix) with ESMTPS id 1B62D1FF0AF for ; Thu, 10 Sep 2026 18:20:55 +0200 (CEST) Received: from gate001.proxmox.com (localhost.localdomain [127.0.0.1]) by gate001.proxmox.com (Proxmox) with ESMTP id 16655215F2; Thu, 10 Sep 2026 18:20:39 +0200 (CEST) From: =?UTF-8?q?Michael=20K=C3=B6ppl?= To: pve-devel@lists.proxmox.com Subject: [PATCH test-tools 0/4] remove PDM submodule Date: Thu, 10 Sep 2026 18:20:26 +0200 Message-ID: <20260910162030.1776719-1-m.koeppl@proxmox.com> X-Mailer: git-send-email 2.47.3 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1789057223384 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.713 Adjusted score from AWL reputation of From: address DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment (newer systems) RCVD_IN_DNSWL_MED -2.3 Sender listed at https://www.dnswl.org/, medium trust SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: CZAW4PNIKSSM4FCJJZYDLTK57HIHNZSQ X-Message-ID-Hash: CZAW4PNIKSSM4FCJJZYDLTK57HIHNZSQ X-MailFrom: m.koeppl@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox VE development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: The submodule was initially added to avoid reimplementing add_remote and some of the structs and enums used for interacting with both the PDM as well as the PVE API. In both cases, the functionality is so limited that it doesn't really warrant pulling PDM into the build over reimplementing these few pieces of code. These patches remove the dependency from proxmox-test-scheduler (where it was unused anyway) and proxmox-test-instance and all the dependencies that were only pulled in for proxmox-datacenter-manager. The functionality for adding remotes is reimplemented here, but keeps only the parts actually required for proxmox-test-instance's use case. I ran multiple different plans with and without clusters, adding the test instances as remotes for a PDM instance and did not notice any problems. I also verified that `wait_for_task` still works as expected since it was using `TaskStateType` from pdm_api_types. proxmox-test-tools: Michael Köppl (4): scheduler: drop the unused pdm-api-types dependency instance: register PDM remotes without pdm-client instance: source certificate and task types directly buildsys: drop the proxmox-datacenter-manager submodule .gitmodules | 3 - Cargo.toml | 10 --- Makefile | 4 +- README.md | 3 - debian/control | 11 --- proxmox-datacenter-manager | 1 - proxmox-test-instance/Cargo.toml | 6 +- proxmox-test-instance/src/api/pdm_client.rs | 76 +++++++++++++++++---- proxmox-test-instance/src/api/pve_client.rs | 26 ++++++- proxmox-test-scheduler/Cargo.toml | 2 - 10 files changed, 92 insertions(+), 50 deletions(-) delete mode 100644 .gitmodules delete mode 160000 proxmox-datacenter-manager Summary over all repositories: 10 files changed, 92 insertions(+), 50 deletions(-) -- Generated by murpp 0.12.1