Skip to content

ci: install chrony to test the new chrony module#2418

Merged
Conan-Kudo merged 1 commit into
dracut-ng:mainfrom
aafeijoo-suse:chrony-ci
Jun 14, 2026
Merged

ci: install chrony to test the new chrony module#2418
Conan-Kudo merged 1 commit into
dracut-ng:mainfrom
aafeijoo-suse:chrony-ci

Conversation

@aafeijoo-suse

Copy link
Copy Markdown
Contributor

Required to run tests for #2397

Checklist

  • I have tested it locally
  • I have reviewed and updated any documentation if relevant
  • I am providing new code and test(s) for it

@aafeijoo-suse aafeijoo-suse requested a review from a team as a code owner April 30, 2026 13:41
@github-actions github-actions Bot added the test Issues related to testing label Apr 30, 2026

@devkontrol devkontrol left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This change would break Debian/Ubuntu CI

#5 16.00 E: Unable to satisfy dependencies. Reached two conflicting assignments:
#5 16.00    1. systemd-timesyncd:arm64=260.1-1 is selected for install
#5 16.00    2. systemd-timesyncd:arm64 is not selected for install because:
#5 16.00       1. chrony:arm64=4.8-3 is selected for install
#5 16.00       2. systemd-timesyncd:arm64 Conflicts time-daemon
#5 16.00          [selected chrony:arm64=4.8-3]

CC @bdrung

@bdrung

bdrung commented Apr 30, 2026

Copy link
Copy Markdown
Member

Both chrony and systemd-timesyncd provide the virtual package time-daemon and there is only one of those allowed. So to install chrony, systemd-timesyncd has to be dropped from the dockerfile.

@aafeijoo-suse

Copy link
Copy Markdown
Contributor Author

Both chrony and systemd-timesyncd provide the virtual package time-daemon and there is only one of those allowed. So to install chrony, systemd-timesyncd has to be dropped from the dockerfile.

Ok, I didn't know that. Changed.

@bdrung

bdrung commented Apr 30, 2026

Copy link
Copy Markdown
Member

No idea, why ubuntu:rolling-amd fails. I created #2419 to make the build log more verbose.

@devkontrol

Copy link
Copy Markdown
Collaborator

TEST-41 requires systemd-timesyncd

@bdrung

bdrung commented Apr 30, 2026

Copy link
Copy Markdown
Member

So separate containers? One with chrony and one with systemd-timesyncd?

@aafeijoo-suse

Copy link
Copy Markdown
Contributor Author

So separate containers? One with chrony and one with systemd-timesyncd?

Or do not test chrony in Debian/Ubuntu.

@Conan-Kudo Conan-Kudo enabled auto-merge (rebase) May 1, 2026 18:04

@devkontrol devkontrol left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This PR does not cover all distributions we have on the CI, but at the same time it makes the project moves forward.

@Conan-Kudo Conan-Kudo merged commit 7b2f752 into dracut-ng:main Jun 14, 2026
166 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Issues related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants