Skip to content

test: add performance test workflow DHIS2-20175#22026

Merged
enricocolasante merged 1 commit into
masterfrom
DHIS2-20175
Sep 26, 2025
Merged

test: add performance test workflow DHIS2-20175#22026
enricocolasante merged 1 commit into
masterfrom
DHIS2-20175

Conversation

@teleivo
Copy link
Copy Markdown
Contributor

@teleivo teleivo commented Sep 24, 2025

  • add independent Maven module dhis-test-performance were we are going to put our Gatling performance tests into. @enricocolasante will start adding one in org.hisp.dhis.test.tracker
  • add workflow .github/workflows/run-performance-tests.yml to compare 2 versions
  • the workflow can be manually triggered using the GitHub CLI. See .github/workflows/run-performance-tests.yml for how
  • the workflow runs the tests in Docker on our self-hosted performance test runner. Only one workflow run will run at a time so runs don't influence each other
  • Gatling HTML reports and a simulation.csv extracted from the binary simulation.log is uploaded. These artifacts are kept for 90 days.
  • run summary describes the test environment, the artifacts and how to get them. Example
    https://github.com/dhis2/dhis2-core/actions/runs/18009761816
2025-09-25-155913

Next

  • @enricocolasante will add a test relating to tracker events
  • I'll look into if we can speed up the DB setup. I already create a DB image using the restored dump so the comparison run can use the cached image. Subsequent runs also benefit from this. That first creation is still rather slow though.

@teleivo teleivo force-pushed the DHIS2-20175 branch 2 times, most recently from 3cea802 to 8019510 Compare September 24, 2025 04:33
@teleivo teleivo force-pushed the DHIS2-20175 branch 17 times, most recently from b6a1ae8 to d030ef6 Compare September 25, 2025 13:54
@teleivo teleivo marked this pull request as ready for review September 25, 2025 14:04
@teleivo teleivo requested a review from a team as a code owner September 25, 2025 14:04
Comment thread dhis-2/dhis-test-performance/pom.xml Outdated
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@ameenhere ameenhere left a comment

Choose a reason for hiding this comment

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

Very nice start @teleivo !

@enricocolasante enricocolasante merged commit cea9ae0 into master Sep 26, 2025
16 of 17 checks passed
@enricocolasante enricocolasante deleted the DHIS2-20175 branch September 26, 2025 12:31
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