Skip to content

smoktest: add newer supported os to smoke tests - #22385

Closed
aelnahas wants to merge 1 commit into
elastic:mainfrom
aelnahas:smoke-test/add-newest-os
Closed

aelnahas wants to merge 1 commit into
elastic:mainfrom
aelnahas: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:36
Copilot AI balanced review requested due to automatic review settings November 5, 2026 17:36
@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 were identified for the expanded OS smoke-test matrix and supporting infrastructure configuration.

No open issues 🎉

What changed in this PR

Expands APM Server’s OS smoke-test coverage to address missing newer versions identified in #21245.

Changes:

  • Adds Ubuntu 26.04, RHEL 10, Rocky Linux 10, and AlmaLinux 10 to the smoke-test matrix.
  • Adds corresponding Terraform image-owner, instance-type, architecture, provisioning, and SSH-user mappings.
  • 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 versions to smoke tests.
testing/​infra/​terraform/​modules/​standalone_apm_server/​main.tf Configures instances and package installation for the added versions.
.github/​workflows/​docs/​smoke-tests-os.yml.md Aligns the documented OS list with the matrix.

Review effort: Balanced

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

@aelnahas

aelnahas commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor Author

closing this, I will push this to a branch in elastic/apm-server so that I can run the smoke os workflow there for extra verification

@aelnahas aelnahas closed this Nov 5, 2026
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

2 participants