Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unmanaged services registered even when disabled #1583

Open
ignasi35 opened this issue Sep 14, 2018 · 2 comments
Open

Unmanaged services registered even when disabled #1583

ignasi35 opened this issue Sep 14, 2018 · 2 comments

Comments

@ignasi35
Copy link
Contributor

Lagom Version (1.2.x / 1.3.x / etc)

1.5.x

API (Scala / Java / Neither / Both)

Seen it on scala+sbt (this is a build tool issue, actually)

Expected Behavior

If DevMode's Kafka/Cassandra services are disabled, then Service Registry doesn't include information about them.

Actual Behavior

With a simple lagom app that explicitly disables both Kafka and Cassandra in dev mode, when checking http://localhost:9008/services there's info for both kafka_native and cas_native.

Reproducible Test Case

Create a new project using sbt new lagom/lagom-scala.g8, disabled kafka and/or cassandra in build.sbt and check http://localhost:9008/services.


I think the problem may be that this code (and probably the same happens in maven Mojos) doesn't check if services are enabled or disabled.

@TimMoore
Copy link
Contributor

I think this was an intentional decision to make it easier to integrate Lagom dev mode with Cassandra and Kafka services that you run yourself. Is there a drawback?

@ignasi35
Copy link
Contributor Author

In the case of Kafka I think your comment is correct.

In the case of Cassandra, I think the registered service stated cas_native -> tcp://localhost:4000 so it's not immediately usable because 4000 is not Cassandra's default port. Users will have to either tune their external Cassandra settings (which may not be running in localhost or port 4000) or override Lagom's registry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants