Skip to content

Conversation

@YuryHrytsuk
Copy link
Collaborator

@YuryHrytsuk YuryHrytsuk commented Sep 17, 2025

What do these changes do?

Use new standalone cluster rabbit instead of simcore rabbit.

Extra:

  • fix pre-commit hook (custom bash scripts must not be executed in parallel until specifically designed so)

Related issue/s

Related PR/s

Devops action

  • create rabbit network (use changes from this PR)

Tasks

  • clean up (e.g. remove old rabbit volume)
  • document / automate switching procedure --> see actions required below
  • Make rabbit network smaller --> done in this commit

Actions required ⚠️

  1. 🚨 during switch simcore docker swarm stack must be removed to avoid unexpected behaviour and broken state🚨
  2. new docker node labels (done automatically via CI)
  3. new docker network (run playbook shown in this PR)
  4. after switch remove old rabbit docker volume

Checklist

  • I tested and it works

New Service Introduction

  • Service is monitored (via prometheus and grafana) --> to be done in separate PR (see tasks in main issue)

If exposed via traefik

  • Service's Public URL is included in maintenance mode --> no need as dashboard is ops service exposed via ops traefik
  • Service's Public URL is included in testing mode --> no need as dashboard is ops service exposed via ops traefik
  • Service's has Traefik (Service Loadbalancer) Healthcheck enabled --> haproxy is already doing this
  • Credentials page is updated --> see linked configuration PR
  • Url added to e2e test services (e2e test checking that URL can be accessed) --> see linked configuration PR

@YuryHrytsuk YuryHrytsuk added this to the Cheops milestone Sep 17, 2025
@YuryHrytsuk YuryHrytsuk self-assigned this Sep 17, 2025
@YuryHrytsuk YuryHrytsuk mentioned this pull request Sep 17, 2025
9 tasks
@YuryHrytsuk YuryHrytsuk marked this pull request as ready for review September 17, 2025 08:35
@YuryHrytsuk YuryHrytsuk changed the title master: switch to standalone (cluster) rabbit master: switch to standalone (cluster) rabbit ⚠️ Sep 17, 2025
Copy link
Member

@mrnicegyu11 mrnicegyu11 left a comment

Choose a reason for hiding this comment

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

lgtm thx

@YuryHrytsuk YuryHrytsuk changed the title master: switch to standalone (cluster) rabbit ⚠️ master: switch to standalone (cluster) rabbit ⚠️ 🚨 Sep 22, 2025
@YuryHrytsuk YuryHrytsuk merged commit 8e9514a into ITISFoundation:main Sep 22, 2025
3 checks passed
@YuryHrytsuk YuryHrytsuk deleted the master-use-cluster-rabbit branch September 22, 2025 12:38
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