Skip to content

Fix LD reporting for noarch packages - #3196

Open
Jany26 wants to merge 1 commit into
packit:mainfrom
Jany26:ld-noarch-bug
Open

Jany26 wants to merge 1 commit into
packit:mainfrom
Jany26:ld-noarch-bug

Conversation

@Jany26

@Jany26 Jany26 commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Child buildArch subtask triggered from Koji scratch build for a noarch package can be run on any architecture - noarch, x86_64, aarch64, etc. This means that retriggers can pick a different arch than the job before, which breaks the CI reporting - two different Log Detective analysis jobs are created, when the more recent one should have replaced the old one.

Noarch package LD analyses will be named target-noarch (i.e. f45-noarch), regardless of whether the build happened on x86_64, aarch64, or whatever. This should fix the CI reporting issue. We rely on the fact that for noarch packages, buildArch tasks have label=="noarch".

I checked around 600 different buildArch koji tasks from last 2 months, none had arch == 'noarch' and at the same time label != 'noarch', so this fix should be safe.

RELEASE NOTES BEGIN

Fix LD analysis overwriting for noarch packages built on different arches.

RELEASE NOTES END

Child buildArch subtask triggered from Koji scratch build for a noarch
package can be run on any architecture - noarch, x86_64, aarch64, etc.
This means that retriggers can pick a different arch than the job
before, which breaks the CI reporting - two different Log Detective
analysis jobs are created, when the more recent one should have replaced
the old one.

Assisted-by: Codex gpt-5.6-terra
Signed-off-by: Jan Matufka <jmatufka@redhat.com>

@betulependule betulependule left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants