Closed
Conversation
…val is high (apache#21413) The finished dagrun was still being seen as running when we call dag.get_num_active_runs because the session was not flushed. This PR fixes it
Building image for now shoudl be forced to linux/amd64 as this is the only supported platform. When using BUILDKIT, the default platform depends on the OS/processor, but until we implement multi-platform images we force them to linux/amd64. Setting it as parameter of docker build instead of env variables is more explicit and allows to copy&paste the whole command to reproduce it outside of breeze when verbose is used independently if you are on Linux, MacOS Intel/ARM.
Co-authored-by: Tzu-ping Chung <tp@astronomer.io>
There is a new rule in markdownlint which has been violated in main when new version of pre-commits is installed introduced in the apache#21734
Make task instance rendered template fields available in the REST API. Co-authored-by: Mocheng Guo <mocheng.guo@airbnb.com>
…#21793) Some of the subcommands here don't actually need a full dag, so there is no point paying the (possibly long) time to parse a dagfile if we could go directly to the DB instead. Closes apache#21450
rustikk
requested review from
dstandish,
ephraimbuddy,
mik-laj,
potiuk and
turbaszek
as code owners
Contributor
Author
|
I did the rebase wrong so I'm closing this and making a new PR. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
12 participants
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.
closes: #21768
When the default_args is anything but a dictionary or None a not so helpful error shows. This commit adds a TypeError if anything other than a dictionary or None is passed to default_args at either the dag level, the taskgroup level or the task level and returns a TypeError saying "default_args must be a dictionary".