Skip to content

Adding rmq#177

Merged
adnazujolakisic merged 10 commits into
mainfrom
rabbit-mq
Apr 21, 2026
Merged

Adding rmq#177
adnazujolakisic merged 10 commits into
mainfrom
rabbit-mq

Conversation

@adnazujolakisic

@adnazujolakisic adnazujolakisic commented Apr 15, 2026

Copy link
Copy Markdown
Contributor
  • RabbitMQ messaging — Wired the shop to a dedicated order-notifications queue: order-service publishes after successful checkout; payload includes order metadata and optional baggage on message headers for mirrord filtering.
  • New notifications-service — Small Node service that subscribes to that queue, logs/handling as implemented
  • Order service — Direct checkout path publishes to Rabbit;
  • Kubernetes — Deployments/env for Rabbit URL + queue name on order-service and notifications-service; shop kustomization includes the new app; optional mirrord operator CRs for RMQ cluster + workload queue registry tied to that deployment.
  • mirrord config — Added/updated mirrord.json (and repo .mirrord/*.json) for notifications-service: operator target, split_queues for order-notifications (RMQ + message_filter on baggage)
  • CI / image — Workflow to build and push the notifications-service image to GHCR so clusters can pull it.

@aris-cub

Copy link
Copy Markdown
Contributor

LGTM. Can we merge this over the weekend as this is a big change?

@aris-cub aris-cub self-requested a review April 16, 2026 15:11
@adnazujolakisic

Copy link
Copy Markdown
Contributor Author

@aris-cub no worries i am still working on it

@aris-cub

Copy link
Copy Markdown
Contributor

You can merge after your last demo tomorrow. As it will be mine and @danielbaker weekend already.

@adnazujolakisic

Copy link
Copy Markdown
Contributor Author

@aris-cub i didn't finish visualization but this should be working. Let me know if i could merge this tomorrow, i dont have demos half of the day which will give me some time to fix if something fails though there are no a lot of changes to the xisting files.

@aris-cub

Copy link
Copy Markdown
Contributor

@adnazujolakisic Sure we are off today and tomorrow so no demos for me

@adnazujolakisic adnazujolakisic merged commit baa8b96 into main Apr 21, 2026
7 checks passed
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.

2 participants