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

add Rabbitmq 3.8 #24

Merged
merged 7 commits into from
May 13, 2024
Merged

add Rabbitmq 3.8 #24

merged 7 commits into from
May 13, 2024

Conversation

pierreboissinot
Copy link
Member

@pierreboissinot pierreboissinot commented Jan 22, 2024

Add rabbitmq 3.8 service to be used as a message broker between n apps.

@pierreboissinot pierreboissinot self-assigned this Jan 22, 2024
@pierreboissinot pierreboissinot changed the title DRAFT: Rabbitmq add Rabbitmq 3.8 Jan 23, 2024
@pierreboissinot pierreboissinot removed the request for review from afrancois-lephare April 29, 2024 13:50
@pierreboissinot
Copy link
Member Author

@thislg pourrais-tu jeter un oeil stp ?

Copy link
Member

@thislg thislg left a comment

Choose a reason for hiding this comment

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

J'ai juste un doute sur la nécessité d'avoir ça dans la stack / est-ce qu'on est obligé de mettre une version mineure aussi précise ? La 3.8 est EOL depuis environ 2 ans.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
pierreboissinot and others added 2 commits May 2, 2024 08:52
Co-authored-by: Thibaut Selingue <[email protected]>
Co-authored-by: Thibaut Selingue <[email protected]>
@pierreboissinot
Copy link
Member Author

@thislg

J'ai juste un doute sur la nécessité d'avoir ça dans la stack

Sur les projets IMAPROTECT c'est un service qui est autant partagé qu'une database relationnelle. A l'avenir nous devrions aussi utilisé davantage rabbitmq via symfony/messenger plutôt que Doctrine.

est-ce qu'on est obligé de mettre une version mineure aussi précise ?

Oui, une mineure peut inclure des BC break pour les client librairies

La 3.8 est EOL depuis environ 2 ans.

Oui, je l'ai déjà signalé au SI du client mais pour l'instant nous devons nous contraindre à cette version obsolète.

@afrancois-lephare
Copy link

afrancois-lephare commented May 2, 2024

@thislg @pierreboissinot

Sur les projets IMAPROTECT c'est un service qui est autant partagé qu'une database relationnelle. A l'avenir nous devrions aussi utilisé davantage rabbitmq via symfony/messenger plutôt que Doctrine.

C'est en effet plus optimisé mais tout dépend du portefeuille du client au final.

Et pour s'affranchir de ça et pouvoir déployer les services qu'on veut, l'un des moyens serait de déployer des images dockers. Il faudrait peser le pour et contre mais c'est un autre sujet qui sort du ticket

CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Thibaut Selingue <[email protected]>
@thislg thislg merged commit 4867e4d into main May 13, 2024
1 check passed
@thislg thislg deleted the rabbitmq branch May 13, 2024 14:23
thislg added a commit that referenced this pull request May 13, 2024
* chore: typo

* feat: add rabbitmq

* docs: add rabbitmq 3.8

* refactor: rename rabbitmq to 3.8

* docs(readme): typo

Co-authored-by: Thibaut Selingue <[email protected]>

* docs(readme): typo

Co-authored-by: Thibaut Selingue <[email protected]>

* Update CHANGELOG.md

Co-authored-by: Thibaut Selingue <[email protected]>

---------

Co-authored-by: Thibaut Selingue <[email protected]>
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