Skip to content

Rhel10 Support for Ops Agent#2191

Open
XuechunHou wants to merge 26 commits intomasterfrom
rhel9-distro
Open

Rhel10 Support for Ops Agent#2191
XuechunHou wants to merge 26 commits intomasterfrom
rhel9-distro

Conversation

@XuechunHou
Copy link
Contributor

@XuechunHou XuechunHou commented Jan 29, 2026

Description

Fixed 3p tests, ops agent integration tests on Rhel10.
Ops agent integration tests and 3p tests no longer run against rhel8 during presubmit, they now run against rhel10.

Related issue

b/444772452

How has this been tested?

Checklist:

  • Unit tests
    • Unit tests do not apply.
    • [] Unit tests have been added/modified and passed for this PR.
  • Integration tests
    • Integration tests do not apply.
    • Integration tests have been added/modified and passed for this PR.
  • Documentation
    • This PR introduces no user visible changes.
    • This PR introduces user visible changes and the corresponding documentation change has been made.
  • Minor version bump
    • This PR introduces no new features.
    • This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
    • This PR bumps the version.

@XuechunHou XuechunHou marked this pull request as draft January 29, 2026 21:35
@XuechunHou XuechunHou marked this pull request as ready for review February 4, 2026 21:12
@XuechunHou XuechunHou changed the title Added more Rhel9 images, Run integration tests against Rhel10 in presubmits Rhel10 Support for Ops Agent Feb 4, 2026
@XuechunHou XuechunHou added the kokoro:force-run Forces kokoro to run integration tests on a CL label Feb 4, 2026
@stackdriver-instrumentation-release stackdriver-instrumentation-release removed the kokoro:force-run Forces kokoro to run integration tests on a CL label Feb 4, 2026
-DFLB_CONFIG_YAML=OFF \
-DFLB_IN_SYSTEMD=On \
-DFLB_MSGPACK_TO_JSON_INIT_BUFFER_SIZE=1.5 -DFLB_MSGPACK_TO_JSON_REALLOC_BUFFER_SIZE=.10 \
-DFLB_CONFIG_YAML=OFF
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did we need this? Was this removed from fluent bit?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change adds the flag -DFLB_PREFER_SYSTEM_LIB_ZSTD=On to the CMake command for RHEL 10. The modifications in this file do not affect other distributions.

Additionally, I have removed -DFLB_CONFIG_YAML=OFF because it is already included in the CMake command on line 31 (see screenshot).

Why is this change needed?
This change addresses the Fluent Bit core dump issue on RHEL 10 (see: Issue #11068). To resolve this, I have upgraded Fluent Bit to version 4.2.1 and also had to make the same changes as in this PR.

sudo yum -y update

sudo yum -y install java-1.8.0-openjdk java-1.8.0-openjdk-devel cassandra
JAVA_VERSION=1.8.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure what changed here? Why was this needed, I'm curious?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

AI...

reverted, thanks

Comment on lines 28 to 30
if [[ "$MAJOR_VERSION_ID" == "10" ]]; then
# For RHEL 10 (MySQL 8.4+), we use the default (caching_sha2_password)
echo "# Using default caching_sha2_password for RHEL 10" | sudo tee -a /etc/my.cnf
Copy link
Member

Choose a reason for hiding this comment

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

If this behaviour is tied to the mysql version then why not just check MYSQL_VERSION instead of the OS major version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, updated

sudo rpm --import https://repo.mysql.com/RPM-GPG-KEY-mysql-2022
fi

MYSQL_VERSION="mysql80"
Copy link
Member

Choose a reason for hiding this comment

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

FYI -- support for 8.0 will be ending this April so we'll have an opportunity to simplify all of this soon and test 8.4 only. But also 9.0 is a thing now which we should consider adding soon too (not in this PR).

# if os is rhel 9, crb is required to access epel
case "$VERSION_ID" in
9*) sudo yum config-manager --set-enabled crb;;
10*) sudo yum config-manager --set-enabled crb;;
Copy link
Member

Choose a reason for hiding this comment

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

Can 9 and 10 be combined? They're doing the same thing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, merged.

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.

4 participants