all lists on 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal