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

Feat: Added isolation groups for commonly used internal tasks #216

Merged

Conversation

rq-dbrady
Copy link
Contributor

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

Changes in this PR
Added isolation groups to :

  • Inline Tasks
  • JQ Tasks
  • Wait Tasks

Describe the new behavior from this PR, and why it's needed
This will allow isolationGroups for commonly used internal Tasks end result is to break out the name spacing in queues so each can be distributed for workflows with high usage of a internal tasks.

  • There is a small bug fix in here for the wait tasks , that in the event it does not start , the call to the start method will insert this in the correct state to be evaluated for the next sweep/decider run.

@v1r3n v1r3n self-requested a review July 22, 2024 04:35
@v1r3n v1r3n merged commit 7230de1 into conductor-oss:main Jul 22, 2024
2 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