Skip to content

[BUILD] Upgrade testcontainers to 2.0.2 and fix Jenkinsfile#82

Merged
Arsnael merged 3 commits intolinagora:masterfrom
vttranlina:upgradeTestcontainer
Jan 19, 2026
Merged

[BUILD] Upgrade testcontainers to 2.0.2 and fix Jenkinsfile#82
Arsnael merged 3 commits intolinagora:masterfrom
vttranlina:upgradeTestcontainer

Conversation

@vttranlina
Copy link
Member

@vttranlina vttranlina commented Jan 16, 2026

Summary by CodeRabbit

  • Chores
    • Updated testcontainers dependency to version 2.0.2.
    • Test execution now includes an additional JVM property to set the API version (api.version=1.43).
    • Adjusted build/test configuration to reflect these updates.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 16, 2026

Walkthrough

The pull request updates build and test configuration. The Jenkinsfile Test stage now passes an additional JVM property -Dapi.version=1.43 to the GatlingIt tests. The build.sbt file is updated to change the Testcontainers dependency from version 1.17.1 to 2.0.2.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately captures both main changes: upgrading testcontainers to 2.0.2 and fixing the Jenkinsfile with the api.version property.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vttranlina
Copy link
Member Author

Jenkins nodes always use the Jenkinsfile from the master branch, which causes the CI to fail.
Could you please update it manually?

@Arsnael
Copy link
Member

Arsnael commented Jan 19, 2026

Jenkins nodes always use the Jenkinsfile from the master branch, which causes the CI to fail. Could you please update it manually?

Pretty sure it should not... I'm ok to merge this but can you please monitor it manually then? Thanks! :)

@Arsnael Arsnael merged commit 0d8cf8b into linagora:master Jan 19, 2026
2 of 4 checks passed
@vttranlina
Copy link
Member Author

vttranlina commented Jan 19, 2026

please monitor it manually then?

There is another issue related to the Maven repository. I tried to dig it, but it does not look simple.
It is not related to the Testcontainers version.
Anyway, the CI is green here:

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.

4 participants