Skip to content

update task manager docs after refactoring#13315

Merged
fosterseth merged 1 commit into
ansible:develfrom
fosterseth:update_task_manager_md
Dec 12, 2022
Merged

update task manager docs after refactoring#13315
fosterseth merged 1 commit into
ansible:develfrom
fosterseth:update_task_manager_md

Conversation

@fosterseth

@fosterseth fosterseth commented Dec 8, 2022

Copy link
Copy Markdown
Member
SUMMARY

issue #13245

view rendered doc here https://github.com/fosterseth/awx/blob/update_task_manager_md/docs/task_manager_system.md

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API
  • Docs

Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated

@kdelee kdelee left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

left a few suggestions

@fosterseth fosterseth force-pushed the update_task_manager_md branch 2 times, most recently from 2ea339e to 782dec0 Compare December 8, 2022 23:23
Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated
### Task Manager Algorithm

1. Get pending, waiting, and running tasks that have `dependencies_processed = True`
2. Before processing pending tasks, the task manager first processes running tasks. This allows it to build a dependency graph and account for the currently consumed capacity in the system.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do all running tasks have to be processed before new pending tasks are processed?

Do we need to define "processing"?

Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated
Comment thread docs/task_manager_system.md Outdated
- DependencyManager and WorkflowManager
- bulk reschedule
- global task manager timeout
- blocking logic

Co-authored-by: Elijah DeLee <kdelee@redhat.com>
Co-authored-by: John R Barker <john@johnrbarker.com>
@fosterseth fosterseth force-pushed the update_task_manager_md branch from 2978cce to 916b564 Compare December 12, 2022 16:59
@fosterseth fosterseth merged commit da9b813 into ansible:devel Dec 12, 2022
@fosterseth fosterseth deleted the update_task_manager_md branch March 11, 2025 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants