[PATCH 1/1] fix(rootfs): close progress reporter after last step finished

[Felix Moessbauer]

We currently stop the progress reporter after each step. This is
incorrect, as finish must only be called after the last step. However,
the impact is quite limited. Currently the only observable effect is
that - when running in debug mode - a summary of all previous execution
times is printed per step (instead of once).

We further want to call finish also if a step raises, hence we move it
to the finally block.

Fixes: eae54341 ("rootfs: Add missing umounts in rootfs ...")
Signed-off-by: Felix Moessbauer <felix.mo...@siemens.com>
---
meta/classes-recipe/rootfs.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes-recipe/rootfs.bbclass b/meta/classes-recipe/rootfs.bbclass
index a58ad031..8e45c10d 100644
--- a/meta/classes-recipe/rootfs.bbclass
+++ b/meta/classes-recipe/rootfs.bbclass
@@ -458,8 +458,8 @@ python do_rootfs_install() {

if "release-after" in (d.getVarFlag(cmd, 'isar-apt-lock') or ""):
bb.utils.unlockfile(lock)
- progress_reporter.finish()
finally:
+ progress_reporter.finish()
bb.build.exec_func('rootfs_do_umounts', d)
}
addtask rootfs_install before do_rootfs_postprocess after do_unpack
--
2.53.0

[Florian Bezdeka]

On Mon, 2026-06-29 at 17:04 +0200, Felix Moessbauer wrote:
> We currently stop the progress reporter after each step. This is
> incorrect, as finish must only be called after the last step. However,
> the impact is quite limited. Currently the only observable effect is
> that - when running in debug mode - a summary of all previous execution
> times is printed per step (instead of once).
>
> We further want to call finish also if a step raises, hence we move it
> to the finally block.
>
> Fixes: eae54341 ("rootfs: Add missing umounts in rootfs ...")
> Signed-off-by: Felix Moessbauer <felix.mo...@siemens.com>

Haven't checked that in depth now, but at first glance it seems that the
problem existed even before the commit referenced as Fixes:

Anyway:

Reviewed-by: Florian Bezdeka <florian...@siemens.com>

[MOESSBAUER, Felix]

On Tue, 2026-06-30 at 11:55 +0200, Florian Bezdeka wrote:
> On Mon, 2026-06-29 at 17:04 +0200, Felix Moessbauer wrote:
> > We currently stop the progress reporter after each step. This is
> > incorrect, as finish must only be called after the last step. However,
> > the impact is quite limited. Currently the only observable effect is
> > that - when running in debug mode - a summary of all previous execution
> > times is printed per step (instead of once).
> >
> > We further want to call finish also if a step raises, hence we move it
> > to the finally block.
> >
> > Fixes: eae54341 ("rootfs: Add missing umounts in rootfs ...")
> > Signed-off-by: Felix Moessbauer <felix.mo...@siemens.com>
>
> Haven't checked that in depth now, but at first glance it seems that the
> problem existed even before the commit referenced as Fixes:

I checked that, and the issue was introduced with the mentioned commit.
Prior to that, the progress_reporter.finish was at the same
indentation, but as there was no try-finally block, it was outside of
the for loop. For details, see (commit prior to yours).

https://github.com/ilbers/isar/blob/b57c49c8d316c2fd5a217f13ec4900fea599d52e/meta/classes/rootfs.bbclass#L262

Felix

[Zhihang Wei]

Applied to next, thanks.

Zhihang