Re: [PATCH datacenter-manager 6/6] api: update logging variable

["Lukas Wagner" <l.wagner@proxmox.com>]

On Wed Oct 7, 2026 at 9:45 AM CEST, Thomas Ellmenreich wrote:
> Update the variable used to set the log level for the datacenter-api. The
> old 'PROXMOX_DEBUG' variable is replaced by 'PDM_LOG' to match the
> pattern used by all other products.
>
> If 'PROXMOX_DEBUG' is set in the environment, a warning is emitted so
> that administrators may switch to the new variable.
>
> Suggested-by: Lukas Wagner <l.wagner@proxmox.com>
> Signed-off-by: Thomas Ellmenreich <t.ellmenreich@proxmox.com>

Btw, there is also another daemon where this should be fixed in

server/src/bin/proxmox-datacenter-privileged-api.rs

(the privileged API daemon running as root)

All other binaries in the project already seem to be using "PDM_LOG".

As mentioned earlier, this might be something that could/should wait
until the next major release. Might be also fine to just change it right
away, the environment variable is not documented anywhere.

@Gabriel, do you have any opinions on the proxmox.git patches?

> ---
>  server/src/bin/proxmox-datacenter-api/main.rs | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/server/src/bin/proxmox-datacenter-api/main.rs b/server/src/bin/proxmox-datacenter-api/main.rs
> index 2fc84c1d..fa8c2eb1 100644
> --- a/server/src/bin/proxmox-datacenter-api/main.rs
> +++ b/server/src/bin/proxmox-datacenter-api/main.rs
> @@ -47,11 +47,18 @@ fn main() -> Result<(), Error> {
>  
>      server::env::sanitize_environment_vars();
>  
> -    proxmox_log::Logger::from_env("PROXMOX_DEBUG", LevelFilter::INFO)
> +    proxmox_log::Logger::from_env("PDM_LOG", LevelFilter::INFO)
>          .journald_on_no_workertask()
>          .tasklog_pbs()
>          .init()?;
>  
> +    // TODO: remove after a few versions
> +    if std::env::var("PROXMOX_DEBUG").is_ok() {
> +        proxmox_log::warn!(
> +            "to set the log level, please use the PDM_LOG variable, as it has replaced PROXMOX_DEBUG"
> +        );
> +    }
> +
>      let debug = proxmox_log::enabled!(Level::DEBUG);
>  
>      if std::env::args().nth(1).is_some() {