Skip to content

fix(perf): make Gatling simulations runnable again#1838

Merged
netomi merged 3 commits into
eclipse-openvsx:mainfrom
gnugomez:gnugomez/main/gatling
May 15, 2026
Merged

fix(perf): make Gatling simulations runnable again#1838
netomi merged 3 commits into
eclipse-openvsx:mainfrom
gnugomez:gnugomez/main/gatling

Conversation

@gnugomez
Copy link
Copy Markdown
Member

@gnugomez gnugomez commented May 14, 2026

The simulations were broken by a Netty version conflict, spring boot was pining a lower version of netty so gatling was not working.

Ideally we may want to consider extracting these tests to their own package.

Within this patch I'm also consolidating the scripts for running perf tests as gradle tasks.

Assisted-By: github:claude-opus-4.6, anthropic:claude-opus-4-7[1m]

This fixes #1827

The simulations were broken by a Netty version conflict, spring boot was pining a lower version of netty so gatling was not working.

Ideally we may want to consider extracting these tests to their own package.

Within this patch I'm also consolidating the scripts for running perf tests as gradle tasks.

Assisted-By: github:claude-opus-4.6, anthropic:claude-opus-4-7[1m]
Comment thread server/.gitignore
@gnugomez gnugomez requested a review from netomi May 14, 2026 09:23
netomi added 2 commits May 15, 2026 13:05
…vocation as gatlingRun task has caching disabled by default which, running without --rerun-tasks does not trigger an app reload
@netomi
Copy link
Copy Markdown
Contributor

netomi commented May 15, 2026

tested the PR locally and it works after a few small modifications:

  • remove the unique namespace generation again as it breaks the publish extensions scenario and all others that build on it
  • update README to not run gatlingRun with --rerun-tasks as this triggers a full rebuild and reload of the application, the gatlingRun task is not cached at all so gradle does the right thing
  • remove unrelated files and added them to the ignore

Copy link
Copy Markdown
Contributor

@netomi netomi left a comment

Choose a reason for hiding this comment

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

LGTM, tyvm

@netomi netomi merged commit b06016e into eclipse-openvsx:main May 15, 2026
8 checks passed
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.

Fix Gatling configuration

2 participants