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

Bugfix task execution from runner in Windows #42426

Merged

Conversation

jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Sep 23, 2024

While "playing around" with making the Edge Worker (AIP-69) running on Windows I noticed that the local task runner fails as the signal implementation in Windows does not carry SIGKILL - but as this is just a cosmetic log message extension (and whitespace was missing) this is just a small bugfix and then... EdgeWorker is running in Windows!

related: #10388

FYI @AutomationDev85

@jscheffl jscheffl added the AIP-69 Edge Executor label Sep 23, 2024
@jscheffl jscheffl added this to the Airflow 2.10.3 milestone Sep 23, 2024
@boring-cyborg boring-cyborg bot added the area:Scheduler including HA (high availability) scheduler label Sep 23, 2024
@jscheffl jscheffl merged commit 737900a into apache:main Sep 25, 2024
53 checks passed
@jscheffl jscheffl added type:bug-fix Changelog: Bug Fixes area:core and removed area:Scheduler including HA (high availability) scheduler labels Sep 25, 2024
jscheffl added a commit to jscheffl/airflow that referenced this pull request Sep 25, 2024
jscheffl added a commit that referenced this pull request Sep 25, 2024
joaopamaral pushed a commit to joaopamaral/airflow that referenced this pull request Oct 21, 2024
utkarsharma2 pushed a commit that referenced this pull request Oct 23, 2024
utkarsharma2 pushed a commit that referenced this pull request Oct 24, 2024
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AIP-69 Edge Executor area:core type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants