Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package cockpit-machines for openSUSE:Factory checked in at 2025-07-01 11:34:14 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/cockpit-machines (Old) and /work/SRC/openSUSE:Factory/.cockpit-machines.new.7067 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "cockpit-machines" Tue Jul 1 11:34:14 2025 rev:21 rq:1289270 version:333 Changes: -------- --- /work/SRC/openSUSE:Factory/cockpit-machines/cockpit-machines.changes 2025-05-26 18:34:50.049643501 +0200 +++ /work/SRC/openSUSE:Factory/.cockpit-machines.new.7067/cockpit-machines.changes 2025-07-01 11:35:17.276128655 +0200 @@ -1,0 +2,15 @@ +Thu Jun 26 11:58:26 UTC 2025 - Miika Alikirri <miika.aliki...@suse.com> + +- Patch cockpit-machines to ignore domain not found errors + when domain is deleted (bsc#1236383) + * added nic-domain-not-found.patch + +------------------------------------------------------------------- +Mon Jun 16 08:22:25 UTC 2025 - Alice Brooks <alice.bro...@suse.com> + +- Update to 333 + * Bug fixes + * The "shareable" attribute of disks is no longer modified by Cockpit + * Virtual network interfaces can now select source mode + +------------------------------------------------------------------- Old: ---- cockpit-machines-330.tar.gz New: ---- cockpit-machines-333.tar.gz nic-domain-not-found.patch ----------(New B)---------- New: when domain is deleted (bsc#1236383) * added nic-domain-not-found.patch ----------(New E)---------- ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ cockpit-machines.spec ++++++ --- /var/tmp/diff_new_pack.h6zumh/_old 2025-07-01 11:35:24.288419213 +0200 +++ /var/tmp/diff_new_pack.h6zumh/_new 2025-07-01 11:35:24.292419378 +0200 @@ -17,7 +17,7 @@ Name: cockpit-machines -Version: 330 +Version: 333 Release: 0 Summary: Cockpit user interface for virtual machines License: LGPL-2.1-or-later AND MIT @@ -30,6 +30,7 @@ Patch10: hide-docs.patch Patch11: load-css-overrides.patch Patch12: uefi-default-firmware.patch +Patch13: nic-domain-not-found.patch BuildArch: noarch BuildRequires: appstream-glib BuildRequires: make ++++++ _scmsync.obsinfo ++++++ --- /var/tmp/diff_new_pack.h6zumh/_old 2025-07-01 11:35:24.332421036 +0200 +++ /var/tmp/diff_new_pack.h6zumh/_new 2025-07-01 11:35:24.336421202 +0200 @@ -1,6 +1,6 @@ -mtime: 1746598548 -commit: 7fae75958b412ed94597330710febf349acf81d8978ab25c2d8728d62389ccfc +mtime: 1750939322 +commit: 2a87cce8dbcefd04968d5914fd1cea431c57d809051d5bc96cd36598ce09a240 url: https://src.opensuse.org/cockpit/cockpit-machines.git -revision: 7fae75958b412ed94597330710febf349acf81d8978ab25c2d8728d62389ccfc +revision: 2a87cce8dbcefd04968d5914fd1cea431c57d809051d5bc96cd36598ce09a240 projectscmsync: https://src.opensuse.org/cockpit/_ObsPrj.git ++++++ build.specials.obscpio ++++++ ++++++ cockpit-machines-330.tar.gz -> cockpit-machines-333.tar.gz ++++++ ++++ 54218 lines of diff (skipped) ++++++ nic-domain-not-found.patch ++++++ >From 6c982496f7b9cd9ef08308e1edd820b3910a7833 Mon Sep 17 00:00:00 2001 From: Miika Alikirri <miika.aliki...@suse.com> Date: Wed, 25 Jun 2025 14:07:00 +0300 Subject: [PATCH] nics: ignore domain not found error caused by domain being removed If user is in the VMDetails page during migration, the VM might get deleted before domainInterfaceAddresses is done processing. This race condition can end up causing a confusing view where the UI shows a redundant "Domain not found" error, and the regular "VM doesn't exist" page at the same time --- src/components/vm/nics/vmNicsCard.tsx | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/components/vm/nics/vmNicsCard.jsx b/src/components/vm/nics/vmNicsCard.jsx index 8b309ac0..79efe152 100644 --- a/src/components/vm/nics/vmNicsCard.jsx +++ b/src/components/vm/nics/vmNicsCard.jsx @@ -239,8 +239,15 @@ export class VmNetworkTab extends React.Component { domainInterfaceAddresses({ connectionName: this.props.vm.connectionName, objPath: this.props.vm.id }) .then(domifaddressAllSources => { const allRejected = !domifaddressAllSources.some(promise => promise.status == 'fulfilled'); + // If user is in the VMDetails page during migration, the VM might get deleted + // before domainInterfaceAddresses is done processing. This race condition can end up + // causing a confusing view where the UI shows a redundant "Domain not found error" + // and the regular VM doesn't exist page, at the same time. + const domainNotFound = domifaddressAllSources.some(promise => + promise.status === 'rejected' && promise.reason?.message.startsWith("Domain not found:") + ); - if (allRejected) + if (allRejected && !domainNotFound) this.props.onAddErrorNotification({ text: cockpit.format(_("Failed to fetch the IP addresses of the interfaces present in $0"), this.props.vm.name), detail: [...new Set(domifaddressAllSources.map(promise => promise.reason ? promise.reason.message : ''))].join(', '), ++++++ node_modules.obscpio ++++++ /work/SRC/openSUSE:Factory/cockpit-machines/node_modules.obscpio /work/SRC/openSUSE:Factory/.cockpit-machines.new.7067/node_modules.obscpio differ: char 62, line 1 ++++++ node_modules.spec.inc ++++++ ++++ 1033 lines (skipped) ++++ between /work/SRC/openSUSE:Factory/cockpit-machines/node_modules.spec.inc ++++ and /work/SRC/openSUSE:Factory/.cockpit-machines.new.7067/node_modules.spec.inc ++++++ package-lock.json ++++++ ++++ 2949 lines (skipped) ++++ between /work/SRC/openSUSE:Factory/cockpit-machines/package-lock.json ++++ and /work/SRC/openSUSE:Factory/.cockpit-machines.new.7067/package-lock.json