public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed
@ 2026-05-19  8:50 Fiona Ebner
  2026-05-19 11:51 ` Shannon Sterz
  2026-05-19 12:19 ` applied: " Fabian Grünbichler
  0 siblings, 2 replies; 5+ messages in thread
From: Fiona Ebner @ 2026-05-19  8:50 UTC (permalink / raw)
  To: pve-devel

Since the $noerr parameter was not passed along, the get_pkg
subroutine would log a failure when it couldn't determine the
installed package version for old kernel metapackages, e.g.:
> FAIL: unable to determine installed pve-kernel-6.5 version.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 PVE/CLI/pve8to9.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PVE/CLI/pve8to9.pm b/PVE/CLI/pve8to9.pm
index 6512ff44..02d13f0e 100644
--- a/PVE/CLI/pve8to9.pm
+++ b/PVE/CLI/pve8to9.pm
@@ -259,7 +259,7 @@ sub check_pve_packages {
             my $outdated_kernel_meta_pkgs = [];
             for my $kernel_meta_version ('6.2', '6.5', '6.8', '6.11') {
                 my $pkg = "pve-kernel-${kernel_meta_version}";
-                if ($get_pkg->($pkg)) {
+                if ($get_pkg->($pkg, 1)) {
                     push @$outdated_kernel_meta_pkgs, $pkg;
                 }
             }
-- 
2.47.3





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

* Re: [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed
  2026-05-19  8:50 [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed Fiona Ebner
@ 2026-05-19 11:51 ` Shannon Sterz
  2026-05-19 12:19 ` applied: " Fabian Grünbichler
  1 sibling, 0 replies; 5+ messages in thread
From: Shannon Sterz @ 2026-05-19 11:51 UTC (permalink / raw)
  To: Fiona Ebner, pve-devel

On Tue May 19, 2026 at 10:50 AM CEST, Fiona Ebner wrote:
> Since the $noerr parameter was not passed along, the get_pkg
> subroutine would log a failure when it couldn't determine the
> installed package version for old kernel metapackages, e.g.:
>> FAIL: unable to determine installed pve-kernel-6.5 version.
>
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
>  PVE/CLI/pve8to9.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/PVE/CLI/pve8to9.pm b/PVE/CLI/pve8to9.pm
> index 6512ff44..02d13f0e 100644
> --- a/PVE/CLI/pve8to9.pm
> +++ b/PVE/CLI/pve8to9.pm
> @@ -259,7 +259,7 @@ sub check_pve_packages {
>              my $outdated_kernel_meta_pkgs = [];
>              for my $kernel_meta_version ('6.2', '6.5', '6.8', '6.11') {
>                  my $pkg = "pve-kernel-${kernel_meta_version}";
> -                if ($get_pkg->($pkg)) {
> +                if ($get_pkg->($pkg, 1)) {
>                      push @$outdated_kernel_meta_pkgs, $pkg;
>                  }
>              }

lgtm and also work for me, so consider this:

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




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

* applied: [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed
  2026-05-19  8:50 [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed Fiona Ebner
  2026-05-19 11:51 ` Shannon Sterz
@ 2026-05-19 12:19 ` Fabian Grünbichler
  2026-05-19 12:27   ` Fiona Ebner
  1 sibling, 1 reply; 5+ messages in thread
From: Fabian Grünbichler @ 2026-05-19 12:19 UTC (permalink / raw)
  To: pve-devel, Fiona Ebner


On Tue, 19 May 2026 10:50:06 +0200, Fiona Ebner wrote:
> Since the $noerr parameter was not passed along, the get_pkg
> subroutine would log a failure when it couldn't determine the
> installed package version for old kernel metapackages, e.g.:
> > FAIL: unable to determine installed pve-kernel-6.5 version.
> 
> 

Applied to master and stable-8, thanks!

[1/1] pve8to9: avoid logging failure when old kernel metapackages are not installed
      commit: 2385f49c54c452f7676efe8f9296335917db060e

Best regards,
-- 
Fabian Grünbichler <f.gruenbichler@proxmox.com>




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

* Re: applied: [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed
  2026-05-19 12:19 ` applied: " Fabian Grünbichler
@ 2026-05-19 12:27   ` Fiona Ebner
  2026-05-19 12:40     ` Fabian Grünbichler
  0 siblings, 1 reply; 5+ messages in thread
From: Fiona Ebner @ 2026-05-19 12:27 UTC (permalink / raw)
  To: Fabian Grünbichler, pve-devel

Am 19.05.26 um 2:19 PM schrieb Fabian Grünbichler:
> 
> On Tue, 19 May 2026 10:50:06 +0200, Fiona Ebner wrote:
>> Since the $noerr parameter was not passed along, the get_pkg
>> subroutine would log a failure when it couldn't determine the
>> installed package version for old kernel metapackages, e.g.:
>>> FAIL: unable to determine installed pve-kernel-6.5 version.
>>
>>
> 
> Applied to master and stable-8, thanks!

In stable-8, $get_pkg() does not have the $noerr argument yet.




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

* Re: applied: [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed
  2026-05-19 12:27   ` Fiona Ebner
@ 2026-05-19 12:40     ` Fabian Grünbichler
  0 siblings, 0 replies; 5+ messages in thread
From: Fabian Grünbichler @ 2026-05-19 12:40 UTC (permalink / raw)
  To: Fiona Ebner, pve-devel

On May 19, 2026 2:27 pm, Fiona Ebner wrote:
> Am 19.05.26 um 2:19 PM schrieb Fabian Grünbichler:
>> 
>> On Tue, 19 May 2026 10:50:06 +0200, Fiona Ebner wrote:
>>> Since the $noerr parameter was not passed along, the get_pkg
>>> subroutine would log a failure when it couldn't determine the
>>> installed package version for old kernel metapackages, e.g.:
>>>> FAIL: unable to determine installed pve-kernel-6.5 version.
>>>
>>>
>> 
>> Applied to master and stable-8, thanks!
> 
> In stable-8, $get_pkg() does not have the $noerr argument yet.

thanks, added that as a follow-up!




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

end of thread, other threads:[~2026-05-19 12:40 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-19  8:50 [PATCH manager] pve8to9: avoid logging failure when old kernel metapackages are not installed Fiona Ebner
2026-05-19 11:51 ` Shannon Sterz
2026-05-19 12:19 ` applied: " Fabian Grünbichler
2026-05-19 12:27   ` Fiona Ebner
2026-05-19 12:40     ` Fabian Grünbichler

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