search

linagora / tmail-backend

GNU Affero General Public License v3.0
39 stars 22 forks source link

Missing bindings for tmail pg-app #989

Closed Arsnael closed 6 months ago

Arsnael commented 6 months ago

@vttranlina it's a bit dirty but feel free to have a look.

vttranlina commented 6 months ago

read it, I'm cherry-pick some commit for my local test

vttranlina commented 6 months ago

Do we need PostgresEventStoreModule when the Memory event bus? No?

Arsnael commented 6 months ago

Do we need PostgresEventStoreModule when the Memory event bus?

Maybe not yes, should maybe load the memory one instead.

I seem to have the DistributedPostgresTmailServerTest hanging eternally btw... can't find really why though.

Arsnael commented 6 months ago

I seem to have the DistributedPostgresTmailServerTest hanging eternally btw... can't find really why though.

I tend to think something more test conf related though, as in staging with this I have no problem in distributed pg app

vttranlina commented 6 months ago

I got the same case, the integration test hanging https://github.com/linagora/tmail-backend/pull/992 after change 

import org.apache.james.DockerOpenSearchExtension;
import org.apache.james.modules.RabbitMQExtension;

to 

import com.linagora.tmail.james.app.DockerOpenSearchExtension;
import com.linagora.tmail.james.app.RabbitMQExtension;

it works

Arsnael commented 6 months ago

@vttranlina yeah I realized that as well. It works now for distributed tests on postgres app. Here it is in postgres healthcheck IT, need to change the same way :)