[pve-devel] [PATCH proxmox-backup] file-restore: fix package name for kernel/initramfs image

[pve-devel] [PATCH proxmox-backup] file-restore: fix package name for kernel/initramfs image

[Stefan Reiter]

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>
---

Note that 'proxmox-backup-restore-image' is listed as 'recommended', not
'depends', which somewhat makes sense, as CT restore works without it as well,
but maybe we should include it in the PVE installation ISO somehow? Or just make
it a depends? atm after installing a fresh PVE, the 'File Restore' button for
VMs only shows this message until a user manually installs this package.

 src/bin/proxmox_file_restore/qemu_helper.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bin/proxmox_file_restore/qemu_helper.rs b/src/bin/proxmox_file_restore/qemu_helper.rs
index 0f3a7feb..bb16bf14 100644
--- a/src/bin/proxmox_file_restore/qemu_helper.rs
+++ b/src/bin/proxmox_file_restore/qemu_helper.rs
@@ -51,7 +51,7 @@ fn validate_img_existance() -> Result<(), Error> {
     let kernel = PathBuf::from(buildcfg::PROXMOX_BACKUP_KERNEL_FN);
     let initramfs = PathBuf::from(buildcfg::PROXMOX_BACKUP_INITRAMFS_FN);
     if !kernel.exists() || !initramfs.exists() {
-        bail!("cannot run file-restore VM: package 'proxmox-file-restore' is not (correctly) installed");
+        bail!("cannot run file-restore VM: package 'proxmox-backup-restore-image' is not (correctly) installed");
     }
     Ok(())
 }
-- 
2.20.1

[pve-devel] applied: [pbs-devel] [PATCH proxmox-backup] file-restore: fix package name for kernel/initramfs image

[Thomas Lamprecht]

On 27.04.21 10:48, Stefan Reiter wrote:
> Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>
> ---
> 
> Note that 'proxmox-backup-restore-image' is listed as 'recommended', not
> 'depends', which somewhat makes sense, as CT restore works without it as well,
> but maybe we should include it in the PVE installation ISO somehow? Or just make
> it a depends? atm after installing a fresh PVE, the 'File Restore' button for
> VMs only shows this message until a user manually installs this package.
> 

sure, mistake on my part.

>  src/bin/proxmox_file_restore/qemu_helper.rs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>

applied, thanks!