-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Kubernetes executor can adopt tasks from other schedulers #10996
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pingzh
reviewed
Sep 18, 2020
bf9e030 to
1c29d60
Compare
ashb
reviewed
Sep 21, 2020
ashb
reviewed
Sep 21, 2020
ashb
reviewed
Sep 21, 2020
8b7e691 to
542d03c
Compare
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
42977a5 to
6bb369e
Compare
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
kaxil
reviewed
Sep 23, 2020
b915807 to
75e4795
Compare
ashb
previously requested changes
Sep 28, 2020
airflow/migrations/versions/bef4f3d11e8b_drop_kuberesourceversion_and_.py
Outdated
Show resolved
Hide resolved
75e4795 to
254d073
Compare
ec0a268 to
7900576
Compare
ashb
approved these changes
Oct 1, 2020
kaxil
approved these changes
Oct 1, 2020
Member
|
@dimberman is there some documentation that should be updated w/ this change? |
Contributor
Author
|
@ryw I don't think further docs needed as this isn't intended to be used externally. |
potiuk
added a commit
that referenced
this pull request
Nov 15, 2020
(cherry picked from commit 3ca11eb) Co-authored-by: Daniel Imberman <[email protected]>
potiuk
added a commit
that referenced
this pull request
Nov 16, 2020
(cherry picked from commit 3ca11eb) Co-authored-by: Daniel Imberman <[email protected]>
potiuk
added a commit
that referenced
this pull request
Nov 16, 2020
(cherry picked from commit 3ca11eb) Co-authored-by: Daniel Imberman <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area:CLI
area:Scheduler
including HA (high availability) scheduler
provider:cncf-kubernetes
Kubernetes (k8s) provider related issues
type:improvement
Changelog: Improvements
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since Kubernetes allows dynamic changing of labels for k8s pods, we can modify labels s.t.
a KubeJobWatcher can monitor tasks launched by another scheduler.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.