tirkarthi commented on PR #38434:
URL: https://github.com/apache/airflow/pull/38434#issuecomment-2016693194
https://github.com/apache/echarts/issues/6202
Using `chartInstance.clear()` before setOption helps with legend issue in 1
but the issue in 2 remains.
--
This is an automated
tirkarthi commented on PR #38434:
URL: https://github.com/apache/airflow/pull/38434#issuecomment-2016690966
Please let me know if a different color needs to be selected with respect to
accessibility. I noticed sometimes when all the bars are in success state then
echarts seem to select
tirkarthi opened a new pull request, #38434:
URL: https://github.com/apache/airflow/pull/38434
closes: #38420
related: #38420
I am facing two issues and not able to debug why.
1. When show landing times is not selected and on page refresh even upon
checking it the legend
collinmcnulty commented on code in PR #38373:
URL: https://github.com/apache/airflow/pull/38373#discussion_r1536720963
##
airflow/providers/samba/hooks/samba.py:
##
@@ -245,10 +245,22 @@ def setxattr(self, path, attribute, value, flags=0,
follow_symlinks=True):
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new 37876960f9 Updating
hussein-awala opened a new pull request, #38433:
URL: https://github.com/apache/airflow/pull/38433
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
github-actions[bot] closed issue #34937: Airflow Helm Chart 1.11.0 doesn't
fresh install Airflow on an On Premises Kubernetes 1.28.2 Cluster
URL: https://github.com/apache/airflow/issues/34937
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
github-actions[bot] closed pull request #34311: Fix: added re-raising of
exceptions to dag.test() execution
URL: https://github.com/apache/airflow/pull/34311
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
github-actions[bot] commented on issue #34937:
URL: https://github.com/apache/airflow/issues/34937#issuecomment-2016639935
This issue has been closed because it has not received response from the
issue author.
--
This is an automated message from the Apache Git Service.
To respond to the
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 8520778930 Temporarily remove protection on v2-9-stable to create beta
2 (#38431)
add 3ac0aaf748 Fix
eladkal closed issue #37400: Vertex AI - model versioning doesn't work with
CreateAutoMLTextTrainingJobOperator
URL: https://github.com/apache/airflow/issues/37400
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
eladkal merged PR #38417:
URL: https://github.com/apache/airflow/pull/38417
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
ephraimbuddy commented on code in PR #38094:
URL: https://github.com/apache/airflow/pull/38094#discussion_r1536707455
##
airflow/serialization/helpers.py:
##
@@ -38,7 +39,19 @@ def is_jsonable(x):
else:
return True
+max_size = conf.getint("core",
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 8520778930 Temporarily remove protection
ephraimbuddy merged PR #38431:
URL: https://github.com/apache/airflow/pull/38431
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
jscheffl commented on PR #37888:
URL: https://github.com/apache/airflow/pull/37888#issuecomment-2016621779
Other PR super-seeds this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific
jscheffl closed pull request #37888: WIP/Idea: Pass task output as outlet to
dataset trigger params
URL: https://github.com/apache/airflow/pull/37888
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
jscheffl opened a new pull request, #38432:
URL: https://github.com/apache/airflow/pull/38432
This PR is a proposal to fix/resolve the request for feature as described in
#37810 as a thin approach.
Idea for this solution:
- Allow using task result as dynamic `extra`
- If
ephraimbuddy opened a new pull request, #38431:
URL: https://github.com/apache/airflow/pull/38431
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
ephraimbuddy commented on PR #37866:
URL: https://github.com/apache/airflow/pull/37866#issuecomment-2016608212
If we call the field `tasks`, users will expect more information, not just
`task_ids`. You may change the field to task_ids. In the future, we can
deprecate that and have `tasks`
shahar1 commented on code in PR #38430:
URL: https://github.com/apache/airflow/pull/38430#discussion_r1536679389
##
airflow/providers/google/cloud/hooks/bigquery.py:
##
@@ -82,32 +83,113 @@ class BigQueryHook(GoogleBaseHook, DbApiHook):
This hook uses the Google Cloud
shahar1 opened a new pull request, #38430:
URL: https://github.com/apache/airflow/pull/38430
related: #28790
---
**^ Add meaningful description above**
Read the **[Pull Request
jscheffl commented on issue #37810:
URL: https://github.com/apache/airflow/issues/37810#issuecomment-2016560016
> This also opens the door for sending multiple things from one single
function if we allow `yield Output(...)`. I can think of future extensions that
the return value does not
Taragolis opened a new pull request, #38429:
URL: https://github.com/apache/airflow/pull/38429
related: https://github.com/apache/airflow/issues/38270
---
**^ Add meaningful description above**
Read the **[Pull Request
Taragolis commented on PR #38377:
URL: https://github.com/apache/airflow/pull/38377#issuecomment-2016544820
> NICE!
Really nice approach how it works into the `virtualenv`:
https://github.com/pypa/virtualenv/blob/main/src/virtualenv/discovery/py_info.py
but for our usage it not
Taragolis opened a new pull request, #38428:
URL: https://github.com/apache/airflow/pull/38428
Replace one on our pre-commit hook by included into the ruff rule
https://docs.astral.sh/ruff/rules/relative-imports/
This one is unsafe fixable, however I made all changes
eladkal commented on issue #38420:
URL: https://github.com/apache/airflow/issues/38420#issuecomment-2016534722
Yes I mentioned it also in
https://github.com/apache/airflow/pull/38214#issuecomment-2002074774
cc @tirkarthi
--
This is an automated message from the Apache Git Service.
To
msoni1369 commented on PR #37866:
URL: https://github.com/apache/airflow/pull/37866#issuecomment-2016517181
> > We can always add more details to the tasks as you mentioned
>
> The problem with big projects like Airflow is maintaining backward
compatibility; returning more data in
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch test-cache-refreshing
in repository https://gitbox.apache.org/repos/asf/airflow.git
omit 8abdb79671 Test cache refreshing in CI
add 106bdf172b Test cache refreshing in CI
This update
howardyoo commented on PR #37989:
URL: https://github.com/apache/airflow/pull/37989#issuecomment-2016507042
> The PR was broken with around 100 unrelated commits, I just squashed all
related commits into a single commit and rebased `main` as it was impossible to
review the PR. Could you
howardyoo commented on PR #37989:
URL: https://github.com/apache/airflow/pull/37989#issuecomment-2016506404
> The PR was broken with around 100 unrelated commits, I just squashed all
related commits into a single commit and rebased `main` as it was impossible to
review the PR. Could you
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 6d7563de33 Optimize number of versions run when CI
potiuk merged PR #38426:
URL: https://github.com/apache/airflow/pull/38426
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
pankaj pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 078e7ed135 Mysql change xcom value col type for
pankajastro merged PR #38401:
URL: https://github.com/apache/airflow/pull/38401
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch test-cache-refreshing
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard b0c97c57a7 Test cache refreshing in CI
add 8abdb79671 Test cache refreshing in CI
This update
LipuFei commented on PR #37143:
URL: https://github.com/apache/airflow/pull/37143#issuecomment-2016490984
@eladkal @potiuk @hussein-awala
Here is a summary of the changes:
- Commit 2d1800aa4afda2f802c44f38a4311576d267fb51 is to check the attribute
`TaskInstance.task` before use.
LipuFei commented on code in PR #37143:
URL: https://github.com/apache/airflow/pull/37143#discussion_r1536630170
##
airflow/sentry/__init__.py:
##
@@ -0,0 +1,30 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch test-cache-refreshing
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard 8febbde0b6 Test cache refreshing in CI
add b0c97c57a7 Test cache refreshing in CI
This update
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch test-cache-refreshing
in repository https://gitbox.apache.org/repos/asf/airflow.git
omit 19c18514f3 Test cache refreshing in CI
add 8febbde0b6 Test cache refreshing in CI
This update
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch test-cache-refreshing
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard 53f8aa6ccd Test cache refreshing in CI
add 19c18514f3 Test cache refreshing in CI
This update
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch test-cache-refreshing
in repository https://gitbox.apache.org/repos/asf/airflow.git
at 53f8aa6ccd Test cache refreshing in CI
This branch includes the following new commits:
new
potiuk opened a new pull request, #38427:
URL: https://github.com/apache/airflow/pull/38427
---
**^ Add meaningful description above**
Read the **[Pull Request
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch test-cache-refreshing
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 53f8aa6ccd2ce3ad38d79eccc11bffba822ea68e
Author: Jarek Potiuk
AuthorDate: Sat Mar 23 13:34:41 2024
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
was 37f7922fd7 Create temporary virtualenv as context manager
The revisions that were on this branch
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new 0f07c90d78 Updating
potiuk commented on PR #38426:
URL: https://github.com/apache/airflow/pull/38426#issuecomment-2016462670
Since we have more people modifying the CI scripts/breeze, optimizing the
versions that are used for testing of those scripts should help with less
resource use on CI and speed of
potiuk opened a new pull request, #38426:
URL: https://github.com/apache/airflow/pull/38426
When CI scripts change we want to run full set of tests to make sure that
the new scripts are working fine. But we do not necessarily want to run all
matrix combos of Python/Kubernetes and DB tests
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch optimize-tests-when-ci-scripts-change
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 380a8a7eceecaa37941e66d78913d2a0ae98fdf9
Author: Jarek Potiuk
AuthorDate: Sat Mar 23
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch optimize-tests-when-ci-scripts-change
in repository https://gitbox.apache.org/repos/asf/airflow.git
at 380a8a7ece Optimize number of versions run when CI scripts change
This branch
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from fac6aa4870 Use `python-on-whales` in docker tests (#38421)
add 98933ed34a Exit with error in case dependencies
potiuk merged PR #38424:
URL: https://github.com/apache/airflow/pull/38424
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
Bowrna commented on PR #38400:
URL: https://github.com/apache/airflow/pull/38400#issuecomment-2016455324
> Add deprecation details around the old metric. Can you update the metrics
documentation?
yes @dirrao Let me do it
--
This is an automated message from the Apache Git Service.
potiuk commented on PR #38424:
URL: https://github.com/apache/airflow/pull/38424#issuecomment-2016437650
Prevents cases like #38384 resulting in #38422
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new fac6aa4870 Use `python-on-whales` in docker tests
potiuk merged PR #38421:
URL: https://github.com/apache/airflow/pull/38421
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
potiuk commented on PR #38421:
URL: https://github.com/apache/airflow/pull/38421#issuecomment-2016436182
Nice refactor and python_on_whales - I remember we discussed it in
https://github.com/apache/airflow/pull/34254#issuecomment-1712790563 -> I think
it's a good one to use it now for
potiuk opened a new pull request, #38424:
URL: https://github.com/apache/airflow/pull/38424
In case dependencies are not synchronized between provider.yaml and
pyproject.toml, they got regenerated in CI, just to prevent the case we are
using old dependencies - using pre-commit script.
potiuk commented on PR #38422:
URL: https://github.com/apache/airflow/pull/38422#issuecomment-2016431004
That should prevent it in the future
https://github.com/apache/airflow/pull/38424
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
pankajastro opened a new pull request, #38423:
URL: https://github.com/apache/airflow/pull/38423
closes: https://github.com/apache/airflow/issues/37999
---
**^ Add meaningful description above**
Read the **[Pull Request
potiuk commented on PR #38422:
URL: https://github.com/apache/airflow/pull/38422#issuecomment-2016426634
Yes I will check why it has not failed in the original PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new bf28b3ed40 Update `azure-serivcebus` in
potiuk merged PR #38422:
URL: https://github.com/apache/airflow/pull/38422
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 48c8f35acf `ExternalPythonOperator` use version
potiuk merged PR #38377:
URL: https://github.com/apache/airflow/pull/38377
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from ac50669c82 update to latest service bus (#38384)
add 665d46ce17 Resolve PT012 in `telegram` provider (#38386)
No
potiuk merged PR #38386:
URL: https://github.com/apache/airflow/pull/38386
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
TiDeane commented on issue #35622:
URL: https://github.com/apache/airflow/issues/35622#issuecomment-2016423352
@brki I am, and thank you for the suggestion! I'll update the docs along
with the bugfix.
A little progress update for anyone interested: from the testing I've done,
it
shahar1 commented on code in PR #38418:
URL: https://github.com/apache/airflow/pull/38418#discussion_r1536599837
##
pyproject.toml:
##
@@ -821,7 +820,7 @@ microsoft-azure = [ # source:
airflow/providers/microsoft/azure/provider.yaml
"azure-mgmt-datalake-store>=0.5.0",
shahar1 opened a new pull request, #38422:
URL: https://github.com/apache/airflow/pull/38422
related: #38384
After merging the PR above, the pre-commit insists on updating
`pyproject.toml`
---
**^ Add meaningful description above**
Read the
github-actions[bot] closed issue #29637: EcsRegisterTaskDefinitionOperator
params are incorrectly invalidated
URL: https://github.com/apache/airflow/issues/29637
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
github-actions[bot] commented on issue #29637:
URL: https://github.com/apache/airflow/issues/29637#issuecomment-2016383226
This issue has been closed because it has not received response from the
issue author.
--
This is an automated message from the Apache Git Service.
To respond to the
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard a36fb6eb82 Create temporary virtualenv as context manager
discard ff0fc7568a Respect docker-tests
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new ac50669c82 update to latest service bus (#38384)
boring-cyborg[bot] commented on PR #38384:
URL: https://github.com/apache/airflow/pull/38384#issuecomment-2016376503
Awesome work, congrats on your first merged pull request! You are invited to
check our [Issue Tracker](https://github.com/apache/airflow/issues) for
additional
eladkal merged PR #38384:
URL: https://github.com/apache/airflow/pull/38384
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
shahar1 commented on code in PR #38418:
URL: https://github.com/apache/airflow/pull/38418#discussion_r1536239715
##
airflow/providers/google/cloud/hooks/automl.py:
##
@@ -1,642 +0,0 @@
-#
Review Comment:
Technically speaking - we didn't actually deprecate this hook, but as
77 matches
Mail list logo
