Skip to content

enh(ci): switch develop building workflows to el10/mysql 8:4#3472

Open
mushroomempires wants to merge 13 commits into
developfrom
MON-201012-switch-develop-building-workflows-to-el10-mysql8.4
Open

enh(ci): switch develop building workflows to el10/mysql 8:4#3472
mushroomempires wants to merge 13 commits into
developfrom
MON-201012-switch-develop-building-workflows-to-el10-mysql8.4

Conversation

@mushroomempires

Copy link
Copy Markdown
Contributor

Description

Fixes # MON-201012

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 23.10.x
  • 24.04.x
  • 24.10.x
  • 25.10.x
  • master

How this pull request can be tested ?

Please describe the procedure to verify that the goal of the PR is matched. Provide clear instructions so that it can be correctly tested.

Any relevant details of the configuration to perform the test should be added.

Checklist

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have made corresponding changes to the documentation.
  • I have rebased my development branch on the base branch (master, maintenance).

@mushroomempires mushroomempires requested a review from a team as a code owner June 19, 2026 15:42
@mushroomempires mushroomempires requested review from kduret and tuntoja and removed request for a team June 19, 2026 15:42
@coderabbitai

coderabbitai Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • coderabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 7965b972-e8f8-4eef-b207-f0bf8f18a78e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch MON-201012-switch-develop-building-workflows-to-el10-mysql8.4

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

kduret
kduret previously approved these changes Jun 24, 2026
@jean-christophe81 jean-christophe81 requested a review from a team as a code owner June 24, 2026 10:12
@jean-christophe81 jean-christophe81 requested review from Evan-Adam and scresto31 and removed request for a team June 24, 2026 10:12
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
57 15 0 72 79.17 43m9.479059s

Failed Tests

Name Message ⏱️ Duration Suite
BECT_GRPC2 Engine and Broker not connected 30.487 s Tls
BECT_GRPC3 Engine and Broker not connected 30.581 s Tls
BECT_GRPC4 Engine and Broker not connected 30.578 s Tls
BSCSSTRR2 No information about TLS activation. 102.374 s Bbdo-Server-Client-Reversed
BSCSSTGRR2 No information about TLS activation. 30.066 s Bbdo-Server-Client-Reversed
BSCSSTRR2 No information about TLS activation. 64.241 s Bbdo-Server-Client-Reversed
BSCSSTGRR2 No information about TLS activation. 30.069 s Bbdo-Server-Client-Reversed
BSCSST2 No information about TLS activation. 91.068 s Bbdo-Server-Client
BSCSSTG1 No information about TLS activation. 30.048 s Bbdo-Server-Client
BSCSSTG2 No information about TLS activation. 30.149 s Bbdo-Server-Client
BSCSSGA3 No message about the authorization token found in the vault. 40.313 s Bbdo-Server-Client
BSCSST2 No information about TLS activation. 56.070 s Bbdo-Server-Client
BSCSSTG1 No information about TLS activation. 30.051 s Bbdo-Server-Client
BSCSSTG2 No information about TLS activation. 30.151 s Bbdo-Server-Client
BSCSSGA3 No message about the authorization token found in the vault. 35.826 s Bbdo-Server-Client

Comment thread .github/workflows/get-environment.yml Outdated

// --- Config sets ---
const configs = [alma9_mysql_8_0, alma10_mariadb_10_11, trixie_mariadb_11_8];
const collectConfigs = [alma10_mysql_8_4, alma9_mysql_8_0, alma10_mariadb_10_11, trixie_mariadb_11_8];

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.

Please consider alma10_mariadb_11_8, there will be no alma10_mariadb_10_11 supported


// --- Config sets ---
const configs = [alma9_mysql_8_0, alma10_mariadb_10_11, trixie_mariadb_11_8];
const collectConfigs = [alma10_mysql_8_0, alma9_mysql_8_0, alma10_mariadb_10_11, trixie_mariadb_11_8];

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.

Please drop alma10_mariadb_10_11

const collectConfigs = [alma10_mysql_8_0, alma9_mysql_8_0, alma10_mariadb_10_11, trixie_mariadb_11_8];
const cmaConfigs = [
alma10_mariadb_10_11, alma9_mysql_8_0, alma8_mariadb_10_11,
alma10_mysql_8_0, alma10_mariadb_10_11, alma9_mysql_8_0, alma8_mariadb_10_11,

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.

Please drop alma10_mariadb_10_11


// --- Config sets ---
const configs = [alma9_mysql_8_0, alma10_mariadb_10_11, trixie_mariadb_11_8];
const collectConfigs = [alma10_mysql_8_0, alma9_mysql_8_0, alma10_mariadb_10_11, trixie_mariadb_11_8];

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.

Please keep initial naming "configs"

: cmaConfigs;
}

console.log('matrix', matrix);

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.

Please do not remove output of matrix to help troublehsoot

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.

4 participants