]> git.proxmox.com Git - pve-manager.git/commit
vzdump: apt: notification: do not include domain in 'hostname' field
authorLukas Wagner <l.wagner@proxmox.com>
Mon, 10 Jun 2024 08:40:25 +0000 (10:40 +0200)
committerFabian Grünbichler <f.gruenbichler@proxmox.com>
Thu, 4 Jul 2024 12:57:08 +0000 (14:57 +0200)
commit2952bfe4137b18f5a14756905b090528d48cdd67
treef6c7655c2a1d8662dcfd5fce12e2eb6dc7383a8e
parent431ae86b08831565b104261fdedf2b03db206c3f
vzdump: apt: notification: do not include domain in 'hostname' field

 - The man page warns about the usage of `hostname -f`, since a host
   may have multiple domains (or none at all)
 - The fallback PVE::INotify::nodename() already only returned the
   hostname without the domain part
 - Fencing notifications didn't include the domain part anyway

This may result in soft-breakage for any users who have already relied
on the domain being present. If there is need for it, it could include
a fqdn metadata field.

The hostname property used for rendering the notification template
is unaffected for now.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
PVE/API2/APT.pm
PVE/VZDump.pm