Skip to content

Comments

feat: fallback config for testkit actor system#2377

Merged
aludwiko merged 2 commits intomainfrom
4462-kalixtestkit-provide-configuration-to-testsystem
Nov 11, 2025
Merged

feat: fallback config for testkit actor system#2377
aludwiko merged 2 commits intomainfrom
4462-kalixtestkit-provide-configuration-to-testsystem

Conversation

@aludwiko
Copy link
Contributor

@aludwiko aludwiko commented Nov 11, 2025

ActorSystem.create(
"KalixTestkit",
ConfigFactory.parseString("akka.http.server.preview.enable-http2 = true"));
ConfigFactory.parseString("akka.http.server.preview.enable-http2 = true")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TBH I'm not sure about this, because I'm reusing the same fallback config for kalix actor system and for test actor system, maybe this should be sth separate, e.g. KalixTestKit(Main.createKalix(), fallbackTestConfig).start(fallbackConfig) ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Was also thinking about that but then ended up with "it is probably fine".

Did you try it out to set testkit config like reported by user?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, checked that

@aludwiko aludwiko merged commit 8ce0eec into main Nov 11, 2025
55 checks passed
@aludwiko aludwiko deleted the 4462-kalixtestkit-provide-configuration-to-testsystem branch November 11, 2025 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants