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.

Sign up for GitHub

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

Jump to bottom

Add features to bind=True object: control tasks & start tasks #71

Open
3 tasks done
AlanCoding opened this issue Feb 11, 2025 · 1 comment · May be fixed by #122
Open
3 tasks done

Add features to bind=True object: control tasks & start tasks #71

AlanCoding opened this issue Feb 11, 2025 · 1 comment · May be fixed by #122
Labels
awx needed for AWX blocked enhancement New feature or request MVP minimum set of issues to fix

Comments

@AlanCoding
Copy link
Member

Please confirm the following

  • I agree to follow this project's code of conduct.
  • I have checked the current issues for duplicates.
  • I understand that dispatcher is open source software provided for free and that I might not receive a timely response.

Feature type

New Feature

Feature Summary

In my current patch to add bind as a task option, it is only providing static data for the moment.

This requests that we add 2 new methods to the object, these are dynamic, and interact with queues available to worker code:

  1. Run a control task via the IPC mechanism and get a response over the IPC mechanism
  2. Start another task, do not get a reply

Steps to reproduce

N/A feature request

Current results

No response

Sugested feature result

aforementioned callable features.

Additional information

Not in scope right now: async versions

A keep-alive method.

May file issue for these later.
@AlanCoding AlanCoding added enhancement New feature or request blocked labels Feb 11, 2025
@AlanCoding AlanCoding added the awx needed for AWX label Feb 24, 2025
@AlanCoding
Copy link
Member Author

Didn't really capture the "why" of this.

The AWX heartbeat task will "reap" jobs based on whether or not they are running locally. This requires getting the process list from the parent process. This is done with bind_kwargs in AWX which is an ugly custom solution there. The assumption with this issue is that we would run a control task from the heartbeat task which will return local running jobs. So this is needed for AWX functionality.

@AlanCoding AlanCoding mentioned this issue Jan 16, 2025
Closed
7 tasks
@AlanCoding AlanCoding added the MVP minimum set of issues to fix label Feb 27, 2025
@AlanCoding AlanCoding linked a pull request Mar 12, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
awx needed for AWX blocked enhancement New feature or request MVP minimum set of issues to fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow running control tasks from worker AlanCoding/dispatcher
1 participant
@AlanCoding