public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [PATH datacenter-manager 0/4] Fix or ignore lintian warnings
@ 2026-02-20 14:41 Lukas Wagner
  2026-02-20 14:41 ` [PATH datacenter-manager 1/4] debian: silence 'field-too-long Build-Depends' error Lukas Wagner
                   ` (4 more replies)
  0 siblings, 5 replies; 7+ messages in thread
From: Lukas Wagner @ 2026-02-20 14:41 UTC (permalink / raw)
  To: pdm-devel

Fixed the missing man page, the other ones seemed to be fine to be
ignored.

Lukas Wagner (4):
  debian: silence 'field-too-long Build-Depends' error
  debian: add uses-dpkg-database-directly override for pdmAtoB
  debian: add lintian-override allowing the banner service in
    getty.target
  docs: add simple man page for pdmAtoB

 debian/proxmox-datacenter-manager.install        |  1 +
 .../proxmox-datacenter-manager.lintian-overrides |  2 ++
 debian/source/lintian-overrides                  |  1 +
 docs/Makefile                                    |  1 +
 docs/conf.py                                     |  1 +
 docs/pdmAtoB/man1.rst                            | 16 ++++++++++++++++
 6 files changed, 22 insertions(+)
 create mode 100644 debian/source/lintian-overrides
 create mode 100644 docs/pdmAtoB/man1.rst

-- 
2.47.3





^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATH datacenter-manager 1/4] debian: silence 'field-too-long Build-Depends' error
  2026-02-20 14:41 [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Lukas Wagner
@ 2026-02-20 14:41 ` Lukas Wagner
  2026-02-20 14:41 ` [PATH datacenter-manager 2/4] debian: add uses-dpkg-database-directly override for pdmAtoB Lukas Wagner
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 7+ messages in thread
From: Lukas Wagner @ 2026-02-20 14:41 UTC (permalink / raw)
  To: pdm-devel

We just have a lot of build dependencies.

The override was copied over from proxmox-backup.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
 debian/source/lintian-overrides | 1 +
 1 file changed, 1 insertion(+)
 create mode 100644 debian/source/lintian-overrides

diff --git a/debian/source/lintian-overrides b/debian/source/lintian-overrides
new file mode 100644
index 00000000..e067c88d
--- /dev/null
+++ b/debian/source/lintian-overrides
@@ -0,0 +1 @@
+proxmox-datacenter-manager source: field-too-long Build-Depends (* chars > 5000)
-- 
2.47.3





^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATH datacenter-manager 2/4] debian: add uses-dpkg-database-directly override for pdmAtoB
  2026-02-20 14:41 [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Lukas Wagner
  2026-02-20 14:41 ` [PATH datacenter-manager 1/4] debian: silence 'field-too-long Build-Depends' error Lukas Wagner
@ 2026-02-20 14:41 ` Lukas Wagner
  2026-02-20 14:41 ` [PATH datacenter-manager 3/4] debian: add lintian-override allowing the banner service in getty.target Lukas Wagner
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 7+ messages in thread
From: Lukas Wagner @ 2026-02-20 14:41 UTC (permalink / raw)
  To: pdm-devel

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
 debian/proxmox-datacenter-manager.lintian-overrides | 1 +
 1 file changed, 1 insertion(+)

diff --git a/debian/proxmox-datacenter-manager.lintian-overrides b/debian/proxmox-datacenter-manager.lintian-overrides
index 75869104..1f2ea905 100644
--- a/debian/proxmox-datacenter-manager.lintian-overrides
+++ b/debian/proxmox-datacenter-manager.lintian-overrides
@@ -1,3 +1,4 @@
 package-installs-apt-sources [etc/apt/sources.list.d/pdm-enterprise.sources]
+uses-dpkg-database-directly [usr/sbin/pdmAtoB]
 uses-dpkg-database-directly [usr/libexec/proxmox/proxmox-datacenter-api]
 uses-dpkg-database-directly [usr/libexec/proxmox/proxmox-datacenter-privileged-api]
-- 
2.47.3





^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATH datacenter-manager 3/4] debian: add lintian-override allowing the banner service in getty.target
  2026-02-20 14:41 [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Lukas Wagner
  2026-02-20 14:41 ` [PATH datacenter-manager 1/4] debian: silence 'field-too-long Build-Depends' error Lukas Wagner
  2026-02-20 14:41 ` [PATH datacenter-manager 2/4] debian: add uses-dpkg-database-directly override for pdmAtoB Lukas Wagner
@ 2026-02-20 14:41 ` Lukas Wagner
  2026-02-20 14:41 ` [PATH datacenter-manager 4/4] docs: add simple man page for pdmAtoB Lukas Wagner
  2026-02-24 16:22 ` [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Shannon Sterz
  4 siblings, 0 replies; 7+ messages in thread
From: Lukas Wagner @ 2026-02-20 14:41 UTC (permalink / raw)
  To: pdm-devel

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
 debian/proxmox-datacenter-manager.lintian-overrides | 1 +
 1 file changed, 1 insertion(+)

diff --git a/debian/proxmox-datacenter-manager.lintian-overrides b/debian/proxmox-datacenter-manager.lintian-overrides
index 1f2ea905..34a4e114 100644
--- a/debian/proxmox-datacenter-manager.lintian-overrides
+++ b/debian/proxmox-datacenter-manager.lintian-overrides
@@ -1,4 +1,5 @@
 package-installs-apt-sources [etc/apt/sources.list.d/pdm-enterprise.sources]
+systemd-service-file-refers-to-unusual-wantedby-target getty.target [usr/lib/systemd/system/proxmox-datacenter-manager-banner.service]
 uses-dpkg-database-directly [usr/sbin/pdmAtoB]
 uses-dpkg-database-directly [usr/libexec/proxmox/proxmox-datacenter-api]
 uses-dpkg-database-directly [usr/libexec/proxmox/proxmox-datacenter-privileged-api]
-- 
2.47.3





^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATH datacenter-manager 4/4] docs: add simple man page for pdmAtoB
  2026-02-20 14:41 [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Lukas Wagner
                   ` (2 preceding siblings ...)
  2026-02-20 14:41 ` [PATH datacenter-manager 3/4] debian: add lintian-override allowing the banner service in getty.target Lukas Wagner
@ 2026-02-20 14:41 ` Lukas Wagner
  2026-02-24 16:22   ` Shannon Sterz
  2026-02-24 16:22 ` [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Shannon Sterz
  4 siblings, 1 reply; 7+ messages in thread
From: Lukas Wagner @ 2026-02-20 14:41 UTC (permalink / raw)
  To: pdm-devel

Mostly to silence the lintian warning complaining about the missing
page. The content was copied over from PBS and adapted.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
 debian/proxmox-datacenter-manager.install |  1 +
 docs/Makefile                             |  1 +
 docs/conf.py                              |  1 +
 docs/pdmAtoB/man1.rst                     | 16 ++++++++++++++++
 4 files changed, 19 insertions(+)
 create mode 100644 docs/pdmAtoB/man1.rst

diff --git a/debian/proxmox-datacenter-manager.install b/debian/proxmox-datacenter-manager.install
index 499e07ff..3538440c 100644
--- a/debian/proxmox-datacenter-manager.install
+++ b/debian/proxmox-datacenter-manager.install
@@ -14,6 +14,7 @@ usr/share/bash-completion/completions/pdmAtoB.bc
 usr/share/bash-completion/completions/proxmox-datacenter-api.bc
 usr/share/bash-completion/completions/proxmox-datacenter-manager-admin.bc
 usr/share/bash-completion/completions/proxmox-datacenter-privileged-api.bc
+usr/share/man/man1/pdmAtoB.1
 usr/share/man/man1/proxmox-datacenter-api.1
 usr/share/man/man1/proxmox-datacenter-manager-admin.1
 usr/share/man/man1/proxmox-datacenter-privileged-api.1
diff --git a/docs/Makefile b/docs/Makefile
index 0487c97e..faa633bc 100644
--- a/docs/Makefile
+++ b/docs/Makefile
@@ -7,6 +7,7 @@ GENERATED_SYNOPSIS := \
 	config/views/config.rst \
 
 MAN1_PAGES := \
+	pdmAtoB.1 \
 	proxmox-datacenter-manager-admin.1 \
 	proxmox-datacenter-manager-client.1 \
 	proxmox-datacenter-api.1 \
diff --git a/docs/conf.py b/docs/conf.py
index 9de1d1da..39724ef3 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -95,6 +95,7 @@ man_pages = [
     ('proxmox-datacenter-api/man1', 'proxmox-datacenter-api', 'Proxmox Datacenter Manager Public API Server', [author], 1),
     ('proxmox-datacenter-privileged-api/man1', 'proxmox-datacenter-privileged-api', 'Proxmox Datacenter Manager Privileged API Server', [author], 1),
     # CLI
+    ('pdmAtoB/man1', 'pdmAtoB', 'Upgrade checker for Proxmox Datacenter Manager', [author], 1),
     ('proxmox-datacenter-manager-admin/man1', 'proxmox-datacenter-manager-admin', 'Command line tool for managing Proxmox Datacenter Manager hosts.', [author], 1),
     ('proxmox-datacenter-manager-client/man1', 'proxmox-datacenter-manager-client', 'Command line tool for connecting and controlling the remotes and resources of a Proxmox Datacenter Manager hosts.', [author], 1),
     # configs
diff --git a/docs/pdmAtoB/man1.rst b/docs/pdmAtoB/man1.rst
new file mode 100644
index 00000000..10f2e883
--- /dev/null
+++ b/docs/pdmAtoB/man1.rst
@@ -0,0 +1,16 @@
+:orphan:
+
+
+=======
+pdmAtoB
+=======
+
+Description
+===========
+
+This tool will help you to detect common pitfalls and misconfiguration before,
+and during the upgrade of a Proxmox Datacenter Manager system. Any failures or
+warnings must be addressed prior to the upgrade. If you suspect that a message
+is a false positive, you have to make carefully sure that it really is.
+
+.. include:: ../pdm-copyright.rst
-- 
2.47.3





^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATH datacenter-manager 0/4] Fix or ignore lintian warnings
  2026-02-20 14:41 [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Lukas Wagner
                   ` (3 preceding siblings ...)
  2026-02-20 14:41 ` [PATH datacenter-manager 4/4] docs: add simple man page for pdmAtoB Lukas Wagner
@ 2026-02-24 16:22 ` Shannon Sterz
  4 siblings, 0 replies; 7+ messages in thread
From: Shannon Sterz @ 2026-02-24 16:22 UTC (permalink / raw)
  To: Lukas Wagner; +Cc: pdm-devel

not sure how this happened, but: proxmox-b4 wasn't too happy about the
"PATH" typo in the subject here :P

other then the comments for patch 4/4 everything worked as intended,
consider the first three patches:

Reviewed-by: Shannon Sterz <s.sterz@proxmox.com>
Tested-by: Shannon Sterz <s.sterz@proxmox.com>

On Fri Feb 20, 2026 at 3:41 PM CET, Lukas Wagner wrote:
> Fixed the missing man page, the other ones seemed to be fine to be
> ignored.
>
> Lukas Wagner (4):
>   debian: silence 'field-too-long Build-Depends' error
>   debian: add uses-dpkg-database-directly override for pdmAtoB
>   debian: add lintian-override allowing the banner service in
>     getty.target
>   docs: add simple man page for pdmAtoB
>
>  debian/proxmox-datacenter-manager.install        |  1 +
>  .../proxmox-datacenter-manager.lintian-overrides |  2 ++
>  debian/source/lintian-overrides                  |  1 +
>  docs/Makefile                                    |  1 +
>  docs/conf.py                                     |  1 +
>  docs/pdmAtoB/man1.rst                            | 16 ++++++++++++++++
>  6 files changed, 22 insertions(+)
>  create mode 100644 debian/source/lintian-overrides
>  create mode 100644 docs/pdmAtoB/man1.rst





^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATH datacenter-manager 4/4] docs: add simple man page for pdmAtoB
  2026-02-20 14:41 ` [PATH datacenter-manager 4/4] docs: add simple man page for pdmAtoB Lukas Wagner
@ 2026-02-24 16:22   ` Shannon Sterz
  0 siblings, 0 replies; 7+ messages in thread
From: Shannon Sterz @ 2026-02-24 16:22 UTC (permalink / raw)
  To: Lukas Wagner; +Cc: pdm-devel

On Fri Feb 20, 2026 at 3:41 PM CET, Lukas Wagner wrote:
> Mostly to silence the lintian warning complaining about the missing
> page. The content was copied over from PBS and adapted.
>

this doesn't apply anymore due to [1] being applied for what i can tell.

[1]: https://git.proxmox.com/?p=proxmox-datacenter-manager.git;a=commit;h=5f5143f822ea2ee1fc72489c49c38c2e10952c90

> Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
> ---
>  debian/proxmox-datacenter-manager.install |  1 +
>  docs/Makefile                             |  1 +
>  docs/conf.py                              |  1 +
>  docs/pdmAtoB/man1.rst                     | 16 ++++++++++++++++
>  4 files changed, 19 insertions(+)
>  create mode 100644 docs/pdmAtoB/man1.rst
>
> diff --git a/debian/proxmox-datacenter-manager.install b/debian/proxmox-datacenter-manager.install
> index 499e07ff..3538440c 100644
> --- a/debian/proxmox-datacenter-manager.install
> +++ b/debian/proxmox-datacenter-manager.install
> @@ -14,6 +14,7 @@ usr/share/bash-completion/completions/pdmAtoB.bc
>  usr/share/bash-completion/completions/proxmox-datacenter-api.bc
>  usr/share/bash-completion/completions/proxmox-datacenter-manager-admin.bc
>  usr/share/bash-completion/completions/proxmox-datacenter-privileged-api.bc
> +usr/share/man/man1/pdmAtoB.1
>  usr/share/man/man1/proxmox-datacenter-api.1
>  usr/share/man/man1/proxmox-datacenter-manager-admin.1
>  usr/share/man/man1/proxmox-datacenter-privileged-api.1
> diff --git a/docs/Makefile b/docs/Makefile
> index 0487c97e..faa633bc 100644
> --- a/docs/Makefile
> +++ b/docs/Makefile
> @@ -7,6 +7,7 @@ GENERATED_SYNOPSIS := \
>  	config/views/config.rst \
>
>  MAN1_PAGES := \
> +	pdmAtoB.1 \
>  	proxmox-datacenter-manager-admin.1 \
>  	proxmox-datacenter-manager-client.1 \
>  	proxmox-datacenter-api.1 \
> diff --git a/docs/conf.py b/docs/conf.py
> index 9de1d1da..39724ef3 100644
> --- a/docs/conf.py
> +++ b/docs/conf.py
> @@ -95,6 +95,7 @@ man_pages = [
>      ('proxmox-datacenter-api/man1', 'proxmox-datacenter-api', 'Proxmox Datacenter Manager Public API Server', [author], 1),
>      ('proxmox-datacenter-privileged-api/man1', 'proxmox-datacenter-privileged-api', 'Proxmox Datacenter Manager Privileged API Server', [author], 1),
>      # CLI
> +    ('pdmAtoB/man1', 'pdmAtoB', 'Upgrade checker for Proxmox Datacenter Manager', [author], 1),
>      ('proxmox-datacenter-manager-admin/man1', 'proxmox-datacenter-manager-admin', 'Command line tool for managing Proxmox Datacenter Manager hosts.', [author], 1),
>      ('proxmox-datacenter-manager-client/man1', 'proxmox-datacenter-manager-client', 'Command line tool for connecting and controlling the remotes and resources of a Proxmox Datacenter Manager hosts.', [author], 1),
>      # configs
> diff --git a/docs/pdmAtoB/man1.rst b/docs/pdmAtoB/man1.rst
> new file mode 100644
> index 00000000..10f2e883
> --- /dev/null
> +++ b/docs/pdmAtoB/man1.rst
> @@ -0,0 +1,16 @@
> +:orphan:
> +
> +
> +=======
> +pdmAtoB
> +=======
> +
> +Description
> +===========
> +
> +This tool will help you to detect common pitfalls and misconfiguration before,

nit: i know this is pre-existing but "This tool will help you detect" would
be a bit cleaner imo.

nit: and misconfiguration*s*

> +and during the upgrade of a Proxmox Datacenter Manager system. Any failures or
> +warnings must be addressed prior to the upgrade. If you suspect that a message
> +is a false positive, you have to make carefully sure that it really is.

nit: maybe "you have to be careful when making sure that..." might parse
a bit more easily

but none of these are real blockers and should probably then also apply
to pbs (and maybe pve, did not check there).

> +
> +.. include:: ../pdm-copyright.rst





^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2026-02-24 16:21 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-02-20 14:41 [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Lukas Wagner
2026-02-20 14:41 ` [PATH datacenter-manager 1/4] debian: silence 'field-too-long Build-Depends' error Lukas Wagner
2026-02-20 14:41 ` [PATH datacenter-manager 2/4] debian: add uses-dpkg-database-directly override for pdmAtoB Lukas Wagner
2026-02-20 14:41 ` [PATH datacenter-manager 3/4] debian: add lintian-override allowing the banner service in getty.target Lukas Wagner
2026-02-20 14:41 ` [PATH datacenter-manager 4/4] docs: add simple man page for pdmAtoB Lukas Wagner
2026-02-24 16:22   ` Shannon Sterz
2026-02-24 16:22 ` [PATH datacenter-manager 0/4] Fix or ignore lintian warnings Shannon Sterz

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