Skip to content

ci(packaging): enable el10 and deb13 for connectors - #376

Merged
Nivoky merged 3 commits into
developfrom
MON-206776-enh-ci-reenable-el10-trixie-packaging
Aug 11, 2026
Merged

ci(packaging): enable el10 and deb13 for connectors#376
Nivoky merged 3 commits into
developfrom
MON-206776-enh-ci-reenable-el10-trixie-packaging

Conversation

@Nivoky

@Nivoky Nivoky commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Fixes: MON-206776 CTOR-2410

Part 2 of 2. Blocked by #377, which must merge first and deliver a fresh centreon-stream-connectors-lib to el10 and trixie unstable. CI here cannot go green before that.

Description

Building the Centreon 26.09 AMI on AlmaLinux 10 fails because no centreon-stream-connector-* package exists for el10. el10 and trixie were enabled by #339, then commented out by #357 because the Centreon 26.10 repositories were not published yet. They now exist for both distributions.

This PR restores them in the package, test and deliver matrices of stream-connectors.yml, keeping the max-parallel: 25 added by #357.

libcurl4 on Debian 13

libcurl4 does not exist in trixie, not even as a virtual package: the 64-bit time_t transition renamed it libcurl4t64. 15 of the 19 connectors declare it, so they would have been uninstallable on Debian 13. The dependency name is now selected from matrix.distrib. librdkafka1 is unaffected, it is still provided in trixie.

Why this is split from #377

The connectors test job installs the connector package built in its own run, but resolves centreon-stream-connectors-lib from the published repositories. The el10 and trixie libraries published today are stale, because delivery to them has been off since #357:

  • el10 - the published library predates ab81407 (enh(dependencies): package missing lua dependencies #363), which changed google/auth/oauth.lua from require("crypto") to require("openssl"). Test el10 google fails with module 'crypto' not found.
  • trixie - the published library declares Depends: ... lua5.3 and installs under /usr/share/lua/5.3/, pulling lua5.3 into the test container while lua-curl, lua-cffi and lua-openssl for trixie are built for 5.4. Every connector loading one of them fails with module not found.

#377 republishes a current library for both, after which the tests here resolve it and pass.

Verification

  • actionlint and yamllint 1.35.1 pass locally with the CI flags and ruleset.
  • The dependency selection block was simulated across distributions: libcurl4t64 is used only for trixie.
  • Every el10 runtime dependency resolves: centreon-broker-core (rpm-standard 26.10/el10), lua-curl (rpm-standard), lua-lsqlite3/lua-cffi/lua-openssl/lua-tz/lua-base64 (rpm-plugins el10), lua-socket and lua-sec (EPEL 10), librdkafka (AlmaLinux 10 AppStream).

Notes for reviewers

  • Changing a workflow file makes detect-changes set packaging: true, so this PR builds and tests all 19 connectors across 8 distributions. The matrix run is large by design.
  • At 26.10 only unstable is populated for el10 and trixie in the standard repo (stable and testing-release currently hold placeholders), so the test jobs resolve centreon-broker-core from unstable.
  • Re-run CI here only after ci(packaging): enable el10 and deb13 for the lib #377 has merged and its develop run has delivered.

@Nivoky
Nivoky requested a review from a team as a code owner August 10, 2026 15:28
@Nivoky
Nivoky requested review from kduret and mushroomempires and removed request for a team August 10, 2026 15:28
Tpo76
Tpo76 previously approved these changes Aug 10, 2026
kduret
kduret previously approved these changes Aug 10, 2026
@Nivoky
Nivoky dismissed stale reviews from kduret and Tpo76 via 288b7e9 August 10, 2026 16:16
@Nivoky Nivoky changed the title ci(packaging): enable el10 and deb13 builds ci(packaging): enable el10 and deb13 for connectors Aug 10, 2026
@Nivoky
Nivoky marked this pull request as draft August 10, 2026 16:18
@Nivoky
Nivoky marked this pull request as ready for review August 10, 2026 16:18
Nivoky added 2 commits August 11, 2026 10:21
The Centreon 26.09 AMI build on AlmaLinux 10 fails because no
centreon-stream-connector-* package exists for el10. Both el10 and
trixie were disabled when the install/uninstall test job was added,
because the Centreon 26.10 repositories were not published yet. They
now exist for both distributions, so restore them in the package,
test and deliver matrices of the connectors and library workflows.

Two additional fixes were needed for Debian 13:

- The aggregated <distrib>-<version>-testing apt suite is not
  published for trixie, which only has testing-release and
  testing-hotfix. Use those instead, consistent with the rpm branch
  of the same action, otherwise apt-get update fails on a 404.
- libcurl4 no longer exists in trixie: the 64-bit time_t transition
  renamed it libcurl4t64 and left no virtual package behind. Without
  this, 15 of the 19 connectors are uninstallable on Debian 13.

The lua-* dependency workflows and docker-packaging already cover
el10 and trixie, so they need no change.

Refs: MON-206776

Assisted-by: Claude Code (claude-opus-5)
The library and shared action changes move to their own pull request,
which must land first: the connectors test job resolves
centreon-stream-connectors-lib from the published repositories, and
the el10 and trixie libraries published today are stale. Delivering
a fresh library is a prerequisite for these tests to pass.

Refs: MON-206776

Assisted-by: Claude Code (claude-opus-5)
@Nivoky
Nivoky force-pushed the MON-206776-enh-ci-reenable-el10-trixie-packaging branch from 288b7e9 to 8f2c55c Compare August 11, 2026 08:21
The scoping commit reverted stream-connectors-lib.yml and the
test-packages action to their content at the time, which was before
the library pull request landed. Replayed by the rebase on top of the
current develop, it undid that pull request instead of being a no-op.

Restore both files to their develop content, leaving this branch with
stream-connectors.yml as its only change.

Refs: MON-206776

Assisted-by: Claude Code (claude-opus-5)
@sdepassio
sdepassio requested review from Tpo76 and kduret August 11, 2026 09:38
@Nivoky
Nivoky merged commit dd0aeb9 into develop Aug 11, 2026
315 checks passed
@Nivoky
Nivoky deleted the MON-206776-enh-ci-reenable-el10-trixie-packaging branch August 11, 2026 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants