Refactor Quoting Service into an event driven solution to improve performance

[PaulGregoryBaker]

Goal:

As a Mojaloop Community member
I want to have a performant quoting service
so that I can scale the quoting servce

Note: The current quote service performs this action in memory, and a better solution would be to split the servce into two microservices with Kafka events. This will decouple the participant API responses from the work being perfromed and free's up the nodeJS event loop.

Private Zenhub Image

Acceptance Criteria:

• Verify the Quote service is split into two microservices namely an quotes API service and and quotes handler service
• Verify that a configurable kafka topic is used to pass messages between the servcies (kafka configuration should follow same pattern as central ledger)
• Verify that the GP test pass
• Verfy that there is a new runtime CLI option (using commander lib; same as central ledger)
• Verfy that a health endpoint is added to the new servces

Complexity: <High|Medium|Low> > A short comment to remind the reason for the rating

Uncertainty: <High|Medium|Low> > A short comment to remind the reason for the rating

---

Tasks:

• Define proper Kafka config for Producers and Consumer
• Update logic of API handlers to publish a payload into separate kafka queues (quotes/bulk_quotes get/put/post)
• Create a consumer to handle all new kafka topics
• Add/update unit-tests
• Update test-harness to use a new version of Quoting service
• Update Help charts (to deploy consumers as well)
• Check GP tests
• Add healthCheck functionality
• Add unit-tests for all "helper" functionalities: feat(3666): refactor quoting service into an event driven solution to improve performance quoting-service#321 (comment)

Done

• Acceptance Criteria pass
• Designs are up-to date
• Unit Tests pass
• Integration Tests pass
• Code Style & Coverage meets standards
• Changes made to config (default.json) are broadcast to team and follow-up tasks added to update helm charts and other deployment config.
• TBD

Pull Requests:

• feat(3666): refactor quoting service into an event driven solution to improve performance quoting-service#321
• feat(3666): added quoting-handlers to docker-compose ml-core-test-harness#45
• feat(3666): updated charts for new quoting-service helm#588

Follow-up:

• N/A

Dependencies:

• N/A

Accountability:

• Owner: TBC
• QA/Review: TBC

[PaulGregoryBaker]

Hey team! Please add your planning poker estimate with Zenhub @aaronreynoza @kleyow @oderayi @sri-miriyala @vijayg10

[elnyry-sam-k]

@PaulGregoryBaker and @geka-evk , thanks for this issue. Please bring this up at the DA at the earliest; I'm not opposed to this and it makes sense but it is a significant change.