[mongodb] devservice executed when host provided

[dcdh]

Description

When the quarkus.mongodb.connection-string is provided the mongo devservice do not launch a container because the developer wants to use the mongo instance provided.

This behavior is defined here: https://github.com/quarkusio/quarkus/blob/main/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/DevServicesMongoProcessor.java#L150

When the quarkus.mongodb.hosts is used to define a connection instead of quarkus.mongodb.connection-string a mongo container is created and started by the devservice at startup. Is it possible to not start a container to use the hosts provided ?

Implementation ideas

Replace this:

boolean needToStart = !ConfigUtils.isPropertyPresent(configPrefix + "connection-string");

by

boolean needToStart = !ConfigUtils.isPropertyPresent(configPrefix + "connection-string") && !ConfigUtils.isPropertyPresent(configPrefix + "hosts");

[quarkus-bot]

/cc @geoand (devservices), @loicmathieu (mongodb), @stuartwdouglas (devservices)

[geoand]

Is it possible to not start a container to use the hosts provided ?

I think what you propose makes sense makes sense. As you have already analyzed the code, would you like to provide a PR?

[dcdh]

Just a question regarding the fix. How can I do - what is the strategy to do integration testing to check that the mongo database has been started or not depending on the properties ?

[geoand]

I would look at what the existing tests with flapdoodle

[gsmet]

TBH, I would be fine with not having an IT for this one.