Skip to content

smoke test: add newer supported os to smoke tests - #22386

Merged
aelnahas merged 1 commit into
mainfrom
smoke-test/add-newest-os
Nov 5, 2026
Merged

aelnahas merged 1 commit into
mainfrom
smoke-test/add-newest-os

Conversation

@aelnahas

@aelnahas aelnahas commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Motivation/summary

Add new supported os to the supported list for our os smoke tests.

Checklist

  • Does not have user-facing changes

For functional changes, consider:

  • Is it observable through the addition of either logging or metrics?
  • Is its use being published in telemetry to enable product improvement?
  • Have system tests been added to avoid regression?

How to test these changes

Checked our Public AMI list and verified that these images exist

Related issues

Fixes: #21245

@aelnahas
aelnahas requested review from a team as code owners November 5, 2026 17:58
Copilot AI balanced review requested due to automatic review settings November 5, 2026 17:58
@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergify

mergify Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label. Could you fix it @aelnahas? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8.19 is the label to automatically backport to the 8.19 branch.
  • backport-9./d is the label to automatically backport to the 9./d branch. /d is the digit.
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

Copilot AI left a comment

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.

🟢 Approval recommended

No active review issues are reported for the expanded OS smoke-test coverage.

No open issues 🎉

What changed in this PR

Expands APM Server’s OS smoke-test coverage to newer supported releases, addressing the coverage gap described in #21245.

Changes:

  • Adds Ubuntu 26.04, RHEL 10, Rocky Linux 10, and AlmaLinux 10 to the smoke-test matrix.
  • Adds matching Terraform image owners, instance settings, installation commands, and SSH users.
  • Updates workflow documentation to reflect the expanded matrix and existing Rocky Linux 8 coverage.
File Description
testing/​smoke/​os_matrix.sh Adds four newer OS releases to the test matrix.
testing/​infra/​terraform/​modules/​standalone_apm_server/​main.tf Configures provisioning for the added OS releases.
.github/​workflows/​docs/​smoke-tests-os.yml.md Synchronizes the documented OS list with the matrix.

Review effort: Balanced

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

@carsonip carsonip left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks. Please test with a test run and also coordinate with support to reflect that on https://www.elastic.co/support/matrix

@mergify

mergify Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@aelnahas

aelnahas commented Nov 5, 2026

Copy link
Copy Markdown
Contributor Author

a smoke test is running https://github.com/elastic/apm-server/actions/runs/42079826155 will update once it is done

@aelnahas aelnahas added backport-active-all Automated backport with mergify to all the active branches and removed backport-active-all Automated backport with mergify to all the active branches labels Nov 5, 2026
@mergify

mergify Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label. Could you fix it @aelnahas? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8.19 is the label to automatically backport to the 8.19 branch.
  • backport-9./d is the label to automatically backport to the 9./d branch. /d is the digit.
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@aelnahas
aelnahas added this pull request to the merge queue Nov 5, 2026
Merged via the queue into main with commit 04ccfe3 Nov 5, 2026
83 checks passed
@aelnahas
aelnahas deleted the smoke-test/add-newest-os branch November 5, 2026 19:58
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.

smoketest: OS smoke test missing newer supported OS versions

3 participants