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

[PW-3060] Draft saga orchestration #243

Open
wants to merge 32 commits into
base: master
Choose a base branch
from

Conversation

rpbaltazar
Copy link
Contributor

@rpbaltazar rpbaltazar commented Jan 20, 2020

Refer to the issue id if any. Include the link to the issue. E.g:

Draft saga orchestration

Describe the changes made. What does this PR changes that might be critical. If any critical decisions have been made, make sure you explain the rationale for these decisions.

  • Installed Karafka gem to handle communication with Kafka
  • Updated Chown flow to use kafka messages instead of api calls and shared Sidekiq queue knowledge
  • Updated IAM to trigger kafka message when there are tenant changes

How does the implementation addresses the problem

After merging this, we reduce the dependency between services in ROS and Whistler. Each service takes care of itself and triggers an event for notifying other services of these changes.

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