Skip to content

konflux: enable hermetic builds#3723

Merged
joelcapitao merged 2 commits intocoreos:testing-develfrom
jbtrystram:hermetic_builds
Jan 20, 2026
Merged

konflux: enable hermetic builds#3723
joelcapitao merged 2 commits intocoreos:testing-develfrom
jbtrystram:hermetic_builds

Conversation

@jbtrystram
Copy link
Member

@jbtrystram jbtrystram changed the title konflux: generate hermeto lockfiles DNM konflux: hermetic builds Sep 4, 2025
@jbtrystram jbtrystram force-pushed the hermetic_builds branch 3 times, most recently from ef7b013 to c1a33f6 Compare September 5, 2025 09:25
@jbtrystram jbtrystram force-pushed the hermetic_builds branch 6 times, most recently from a6fd835 to 5514868 Compare September 25, 2025 13:12
@jbtrystram
Copy link
Member Author

/retest

@joelcapitao
Copy link
Member

/test fedora-coreos-testing-devel-on-pull-request

@jbtrystram jbtrystram force-pushed the hermetic_builds branch 6 times, most recently from b2f34fd to af8d40c Compare September 30, 2025 14:40
@jbtrystram jbtrystram changed the title DNM konflux: hermetic builds konflux: enable hermetic builds Sep 30, 2025
@jbtrystram jbtrystram force-pushed the hermetic_builds branch 4 times, most recently from 3105692 to ca82cec Compare October 7, 2025 14:39
@jbtrystram
Copy link
Member Author

requires coreos/fedora-coreos-pipeline#1245

@joelcapitao joelcapitao added jira For syncing to Jira. Only works for issues (i.e. not PRs) and removed jira For syncing to Jira. Only works for issues (i.e. not PRs) labels Oct 27, 2025
@joelcapitao
Copy link
Member

joelcapitao commented Dec 2, 2025

See c0abc9c#r171804321 the glib2 URL in rpms.lock.yaml returns 404.

coreos/coreos-assembler#4381 should fix the issue we are hitting

@joelcapitao
Copy link
Member

/retest

@joelcapitao joelcapitao force-pushed the hermetic_builds branch 2 times, most recently from ffe764a to 0b210f8 Compare December 4, 2025 11:58
joelcapitao added a commit to joelcapitao/fedora-coreos-config that referenced this pull request Dec 4, 2025
This commit should be removed as we'll rebase this PR
on top of coreos#3723
@joelcapitao
Copy link
Member

/retest

1 similar comment
@joelcapitao
Copy link
Member

/retest

@joelcapitao
Copy link
Member

/retest

@joelcapitao joelcapitao force-pushed the hermetic_builds branch 4 times, most recently from 0a69b36 to 694c80f Compare December 18, 2025 16:14
@joelcapitao
Copy link
Member

/retest

1 similar comment
@joelcapitao
Copy link
Member

/retest

@joelcapitao joelcapitao force-pushed the hermetic_builds branch 2 times, most recently from 5771d25 to 8a475ae Compare January 6, 2026 14:37
In hermetic builds there is no access to the network. Detect this by
looking for the `cachi2.repo` that is injected by konflux during the
build.

In this case we make sure to not use any of our own repo and rely on
the repo created by hermeto.
@joelcapitao joelcapitao merged commit bac3ef5 into coreos:testing-devel Jan 20, 2026
9 of 10 checks passed
Comment on lines +11 to +15
# cachi2 is the repo Konflux injects when hermetic build is enabled and
# is self-sufficient to pull all the required RPMs.
if [ ! -f "/etc/yum.repos.d/cachi2.repo" ]; then
cp /src/fedora-coreos-continuous.repo /etc/yum.repos.d
fi
Copy link
Member

Choose a reason for hiding this comment

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

not copying the fedora-coreos-continuous repo here means the step below with --repo fedora-coreos-continuous will fail.

I wonder if the buildroot-prep stuff is just never going to work with hermetic?

OR - better, let's just fold calling buildroot-prep into build-rootfs as a another function that gets called (which shells out to the script). This means it can run after we call inject_yumrepos() and can use the hermeto repo to pull the RPM from if we want.

Copy link
Member

@joelcapitao joelcapitao Jan 20, 2026

Choose a reason for hiding this comment

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

Good point, let's give a try.
But note that the line -repo fedora-coreos-continuous is not run at all since a while now. Actually, the whole script is noop at the moment. But we still need to support what you proposed for future need.

Copy link
Member

Choose a reason for hiding this comment

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants