Skip to content

iserver-test: Add 9.2 test and some improvements#17709

Merged
ericywl merged 12 commits intomainfrom
iservertest-add-9.2-test
Jul 18, 2025
Merged

iserver-test: Add 9.2 test and some improvements#17709
ericywl merged 12 commits intomainfrom
iservertest-add-9.2-test

Conversation

@ericywl
Copy link
Copy Markdown
Contributor

@ericywl ericywl commented Jul 17, 2025

Motivation/summary

  • Add 9.2 upgrade paths manually, since we don't have integrationservertest: Automate upgrade paths #17602 yet
  • Add input option in workflow dispatch to control which tests to run
  • Add retry for telemetrygen in case of connection issues during ingestion
  • Remove flush trigger since it seems to interfere with metrics aggregation sometimes

How to test these changes

Run workflow: https://github.com/elastic/apm-server/actions/runs/16346566007

@ericywl ericywl self-assigned this Jul 17, 2025
@ericywl ericywl added the backport-skip Skip notification from the automated backport with mergify label Jul 17, 2025
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 17, 2025

This pull request is now in conflicts. Could you fix it @ericywl? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b iservertest-add-9.2-test upstream/iservertest-add-9.2-test
git merge upstream/main
git push upstream iservertest-add-9.2-test

@ericywl ericywl force-pushed the iservertest-add-9.2-test branch from 0d0c932 to 7b3a00b Compare July 17, 2025 13:33
@ericywl ericywl marked this pull request as ready for review July 17, 2025 14:27
@ericywl ericywl requested review from a team as code owners July 17, 2025 14:27

run-upgrade:
name: Upgrade tests (Snapshot)
if: ${{ inputs.run-upgrade-tests == true }}
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.

I recall some edge cases when using workflow_dispatch default inputs and the schedule. Can you double-check if the default values are honoured when the schedule event runs?

I did run a quick search and found this StackOverflow thread or this

Copy link
Copy Markdown
Contributor Author

@ericywl ericywl Jul 18, 2025

Choose a reason for hiding this comment

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

I see, will try another way to do it instead. Thanks for the heads up!

@ericywl ericywl merged commit ada8550 into main Jul 18, 2025
19 checks passed
@ericywl ericywl deleted the iservertest-add-9.2-test branch July 18, 2025 16:45
v1v added a commit to v1v/apm-server that referenced this pull request Jul 21, 2025
…ture/use-plain-command

* 'main' of https://github.com/elastic/apm-server: (76 commits)
  changelog: Add 9.0.4 release notes (elastic#17586)
  Remove 9.1 from 9.2 lazy rollover exception (elastic#17754)
  Fix wrong usage of inputs (elastic#17751)
  iserver-test: Add 9.2 test and some improvements (elastic#17709)
  refactor: replace pkg/errors with stdlib (elastic#17697)
  docs: update release manager docs (elastic#17450)
  build(deps): bump github.com/spf13/pflag from 1.0.6 to 1.0.7 in the spf13 group (elastic#17729)
  build(deps): bump github.com/elastic/apm-data from 1.19.2 to 1.19.3 (elastic#17731)
  chore: Remove stale comment about reloader multierror (elastic#17711)
  [updatecli] Update to elastic/beats@58f44bb27b80 (elastic#17660)
  chore: deps(updatecli/policy): bump "ghcr.io/updatecli/policies/autod... (elastic#17704)
  Add temporary fix for Fleet policy issue (elastic#17682)
  chore: update test plan template (elastic#17497)
  refactor: remove unused apm tracer (elastic#17658)
  Add previously missing ignored logs (elastic#17655)
  ci: update mergify windows check (elastic#17657)
  build(deps): bump github.com/elastic/go-docappender/v2 from 2.10.0 to 2.11.0 (elastic#17570)
  build(deps): bump the dependencies group in /systemtest with 4 updates (elastic#17654)
  build(deps): bump go.opentelemetry.io/collector/pdata from 1.35.0 to 1.36.0 in the otel group (elastic#17653)
  build(deps): bump ubi9/ubi-micro in /packaging/docker (elastic#17651)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-skip Skip notification from the automated backport with mergify

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants