Skip to content

feat: review docker configuration for ActiveMQ stack#342

Open
EmmanuelDemey wants to merge 25 commits into
developfrom
feat/review-activemq-architecture-docker
Open

feat: review docker configuration for ActiveMQ stack#342
EmmanuelDemey wants to merge 25 commits into
developfrom
feat/review-activemq-architecture-docker

Conversation

@EmmanuelDemey
Copy link
Copy Markdown
Contributor

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 3, 2025

Comment thread queen-application/container/compose.keycloak.yml Outdated
Comment thread queen-application/container/compose.keycloak.yml Outdated
Comment thread queen-application/container/compose.queen-db-ciphered.yml Outdated
Comment thread queen-application/container/compose.queen-db.yml Outdated
Comment thread queen-listener-jms/pom.xml
@@ -1,52 +1,44 @@
services:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This duplicates the JMS module component

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

what do you mean @aitkarra ?

Copy link
Copy Markdown
Collaborator

@aitkarra aitkarra Dec 8, 2025

Choose a reason for hiding this comment

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

Perhaps the JMS component/service could be kept in a single module ?

Copy link
Copy Markdown
Contributor

@davdarras davdarras left a comment

Choose a reason for hiding this comment

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

  • Cannot launch the app. There seems to be some issues with compose and DockerFile when building.
  • APIs tests are not working (SQL data issues)
  • liquibase contexts are not loaded automatically (as stated in the documentation)

Comment thread documentation/multi-communication.md Outdated
Comment thread documentation/multi-communication.md Outdated
Comment thread queen-listener-jms/src/main/java/fr/insee/queen/jms/service/OutboxScheduler.java Outdated
Comment thread queen-infra-db/src/main/resources/db/scripts/truncate-data.sql
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.

3 participants