Skip to content

[Distro] Support Ops Agent on SLES 16#2192

Open
XuechunHou wants to merge 39 commits intomasterfrom
sles16
Open

[Distro] Support Ops Agent on SLES 16#2192
XuechunHou wants to merge 39 commits intomasterfrom
sles16

Conversation

@XuechunHou
Copy link
Copy Markdown
Contributor

@XuechunHou XuechunHou commented Jan 30, 2026

Description

Add the build configuration for SLES 16.

Related issue

b/460262238

How has this been tested?

Follow the new distro guide - borrowed SLES 15's config to run integration tests. All Ops Agent integration tests passed and then reverted the temporary changes.

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 changed the title Sles16 Support Ops Agent on SLES 16 Jan 30, 2026
@LujieDuan
Copy link
Copy Markdown
Contributor

SELS 16 arm and x86 integration tests are passing:

Going to revert the temporary changes and get ready for reviews.

@LujieDuan LujieDuan changed the title Support Ops Agent on SLES 16 [Distro] Support Ops Agent on SLES 16 Mar 24, 2026
@LujieDuan LujieDuan marked this pull request as ready for review March 24, 2026 15:19

%pre
%if 0%{?suse_version} >= 1600
%systemd_pre google-cloud-ops-agent.service
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.

pkg/rpm/build.sh Outdated
# Build .rpms
RPMBUILD_ARGS=()
if [[ -n "${SLE_VERSION}" ]]; then
RPMBUILD_ARGS+=(--define "sle_version ${SLE_VERSION}")
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.

We need {sle_version} in pkg/rpm/google-cloud-ops-agent.spec, but for our building image OpenSUSE 16, this does not exist, so we need to pass this explicitly.

On OpenSUSE 16:

$ rpm --eval '%{sle_version}'
%{sle_version}
$ rpm --eval '%{suse_version}'
1600

@LujieDuan LujieDuan requested review from a team and jefferbrecht and removed request for a team March 24, 2026 15:48
@LujieDuan
Copy link
Copy Markdown
Contributor

RPM distros integration tests are passing (SLES 15 is actually SLES 16 here):
image

@LujieDuan
Copy link
Copy Markdown
Contributor

SLES 15 tests passed - reverting the temporary changes.

@LujieDuan LujieDuan requested a review from jefferbrecht March 26, 2026 14:45
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