Skip to content

Conversation

@SalmaElsoly
Copy link
Contributor

@SalmaElsoly SalmaElsoly commented Nov 27, 2025

Description

This pull request introduces a distributed locking mechanism for nodes using Redis, to keep node resources synchronized

Changes

  • Added a new DistributedLocks interface and a Redis-based implementation (RedisLocker) for managing node and workflow locks, including methods to acquire, release, and query locks.
  • Updated NodeHandler and DeploymentHandler to use the distributed locking mechanism. Now, before performing operations like reserving a node, deploying a cluster, or adding a node, the handlers attempt to acquire locks on the relevant nodes. If a node is already locked, the request will be rejected with a conflict error.
  • Added worker that scan keys and release locks of completed workflows

Related Issues

#872

Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstring

@SalmaElsoly SalmaElsoly force-pushed the master_distributed_locks branch 3 times, most recently from d0623bd to ec85e80 Compare November 30, 2025 06:07
@SalmaElsoly SalmaElsoly marked this pull request as ready for review November 30, 2025 06:14
@SalmaElsoly SalmaElsoly requested a review from xmonader November 30, 2025 06:44
@SalmaElsoly SalmaElsoly marked this pull request as draft December 8, 2025 12:24
@SalmaElsoly SalmaElsoly force-pushed the master_distributed_locks branch from ec85e80 to 2ec13a8 Compare December 14, 2025 15:02
@SalmaElsoly SalmaElsoly marked this pull request as ready for review December 15, 2025 07:29
@SalmaElsoly SalmaElsoly force-pushed the master_distributed_locks branch from 2884a17 to 3da4962 Compare December 22, 2025 07:13
@SalmaElsoly SalmaElsoly force-pushed the master_distributed_locks branch from 3da4962 to 6c8bbc8 Compare December 22, 2025 07:18
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.

4 participants