potiuk closed pull request #22183:
URL: https://github.com/apache/airflow/pull/22183
--
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 issue #22188:
URL: https://github.com/apache/airflow/issues/22188#issuecomment-1065363364
Silly me
--
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
potiuk closed issue #22188:
URL: https://github.com/apache/airflow/issues/22188
--
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 edited a comment on issue #22188:
URL: https://github.com/apache/airflow/issues/22188#issuecomment-1065362352
Hmmm. Is this not possible already with `params` ? (I could have missed that
entirely).
--
This is an automated message from the Apache Git Service.
To respond to the
potiuk commented on issue #22188:
URL: https://github.com/apache/airflow/issues/22188#issuecomment-1065362352
Hmmm. Is this not possible already with `params` ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
potiuk opened a new issue #22188:
URL: https://github.com/apache/airflow/issues/22188
### Description
I've seen recently at least two cases that the users wanted to use
constructor kwargs values in jinja templates as part of the Jinja `Context`.
### Use case/motivation
github-actions[bot] commented on pull request #22187:
URL: https://github.com/apache/airflow/pull/22187#issuecomment-1065343505
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and
potiuk opened a new pull request #22187:
URL: https://github.com/apache/airflow/pull/22187
We used some Airflow 1.10 compatibility shims in system
tests and Breeze's entrypoints, in order to be able to run
1.10 still if needed, but the need for that is completely gone
for months
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 03cdfe7 Avoid trying to kill container when it
potiuk merged pull request #22145:
URL: https://github.com/apache/airflow/pull/22145
--
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:
github-actions[bot] commented on pull request #22183:
URL: https://github.com/apache/airflow/pull/22183#issuecomment-1065323007
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch gh-pages
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
The following commit(s) were added to refs/heads/gh-pages by this push:
new e84ca47 Deploying to gh-pages
dstandish commented on a change in pull request #22159:
URL: https://github.com/apache/airflow/pull/22159#discussion_r824921045
##
File path: tests/api_connexion/endpoints/test_task_instance_endpoint.py
##
@@ -293,6 +293,57 @@ def
This is an automated email from the ASF dual-hosted git repository.
ash pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 5d9b088 Remove Snowflake limits (#22181)
add d5c039d Use relation in TI join to RTIF (#22159)
No new revisions
ashb merged pull request #22159:
URL: https://github.com/apache/airflow/pull/22159
--
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 merged pull request #560:
URL: https://github.com/apache/airflow-site/pull/560
--
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-site.git
The following commit(s) were added to refs/heads/main by this push:
new bc77a86 Improcve formatting for summit
potiuk commented on a change in pull request #22179:
URL: https://github.com/apache/airflow/pull/22179#discussion_r824913958
##
File path: airflow/providers/google/suite/hooks/drive.py
##
@@ -197,7 +197,7 @@ def upload_file(self, local_location: str, remote_location:
str) ->
potiuk commented on a change in pull request #22179:
URL: https://github.com/apache/airflow/pull/22179#discussion_r824913958
##
File path: airflow/providers/google/suite/hooks/drive.py
##
@@ -197,7 +197,7 @@ def upload_file(self, local_location: str, remote_location:
str) ->
potiuk commented on a change in pull request #22179:
URL: https://github.com/apache/airflow/pull/22179#discussion_r824913110
##
File path: airflow/providers/google/suite/hooks/drive.py
##
@@ -197,7 +197,7 @@ def upload_file(self, local_location: str, remote_location:
str) ->
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch improve-formatting-for-summit-announcement
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
omit 5a36c4b Improcve formatting for summit announcement
add 623f3a3
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch improve-formatting-for-summit-announcement
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
at 5a36c4b Improcve formatting for summit announcement
This branch includes
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch improve-formatting-for-summit-announcement
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
commit 5a36c4b509e551c917adc88da9e1e4a771c8d094
Author: Jarek Potiuk
AuthorDate: Fri
ashb commented on a change in pull request #22159:
URL: https://github.com/apache/airflow/pull/22159#discussion_r824888112
##
File path: tests/api_connexion/endpoints/test_task_instance_endpoint.py
##
@@ -293,6 +293,57 @@ def
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch gh-pages
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
The following commit(s) were added to refs/heads/gh-pages by this push:
new cfd1176 Deploying to gh-pages
This is an automated email from the ASF dual-hosted git repository.
potiuk 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 3f0f575 Update
ashb commented on a change in pull request #22159:
URL: https://github.com/apache/airflow/pull/22159#discussion_r824888112
##
File path: tests/api_connexion/endpoints/test_task_instance_endpoint.py
##
@@ -293,6 +293,57 @@ def
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 0ab1308 Show DagModel details. (#21868)
add 5d9b088 Remove Snowflake limits (#22181)
No new revisions were
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-site.git
The following commit(s) were added to refs/heads/main by this push:
new b9eccae Add Summit 2022 announcement (#559)
potiuk merged pull request #22181:
URL: https://github.com/apache/airflow/pull/22181
--
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 merged pull request #559:
URL: https://github.com/apache/airflow-site/pull/559
--
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.
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 9080a17 Updating
ulsc commented on a change in pull request #22179:
URL: https://github.com/apache/airflow/pull/22179#discussion_r824855561
##
File path: airflow/providers/google/suite/hooks/drive.py
##
@@ -197,7 +197,7 @@ def upload_file(self, local_location: str, remote_location:
str) ->
potiuk commented on pull request #22181:
URL: https://github.com/apache/airflow/pull/22181#issuecomment-1065245378
> Should we bump the lowers up too?
No need. There was no incompatible changes, so old versions of libraries
will continue to work, we should only bump lower limits if
potiuk opened a new pull request #22183:
URL: https://github.com/apache/airflow/pull/22183
The #22127 change introduced a change how execution of docker
commnds was done (due to LD_PRELOAD change) and they started to
use entrypoint_ci, however this caused undesired effect of running
Caerbannog commented on issue #21395:
URL: https://github.com/apache/airflow/issues/21395#issuecomment-1065225367
Disregard my last message.
5 days later I looked at my Sentry console again, and it was full of the
exceptions I was expecting.
So either I messed up last time, or Sentry
This is an automated email from the ASF dual-hosted git repository.
bbovenzi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 961225f Update links to new Grid/Graph view instead of relying on
redirects (#22167)
add 0ab1308 Show
bbovenzi merged pull request #21868:
URL: https://github.com/apache/airflow/pull/21868
--
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:
github-actions[bot] commented on pull request #22181:
URL: https://github.com/apache/airflow/pull/22181#issuecomment-1065197866
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and
This is an automated email from the ASF dual-hosted git repository.
ash 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 961225f Update links to new Grid/Graph view instead
ashb merged pull request #22167:
URL: https://github.com/apache/airflow/pull/22167
--
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:
ashb commented on pull request #22167:
URL: https://github.com/apache/airflow/pull/22167#issuecomment-1065185582
Random MSSQL failure (db wasn't ready)
--
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
boring-cyborg[bot] commented on pull request #22182:
URL: https://github.com/apache/airflow/pull/22182#issuecomment-1065176587
Congratulations on your first Pull Request and welcome to the Apache Airflow
community! If you have any issues or are unsure about any anything please check
our
flipstone42 opened a new pull request #22182:
URL: https://github.com/apache/airflow/pull/22182
- Add securityContext templating to statefulSet manifest
- Add securityContext commented-out to values.yaml
- Add securityContext section to values.schema.json
--
This is an automated
ashb commented on pull request #22167:
URL: https://github.com/apache/airflow/pull/22167#issuecomment-1065168729
Fixed :)
--
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.
This is an automated email from the ASF dual-hosted git repository.
ash pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 9eb1a1c Improve StatusBox rendering in Grid view (#22170)
add d08284e Add map_index to XCom model and interface
ashb merged pull request #22112:
URL: https://github.com/apache/airflow/pull/22112
--
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:
github-actions[bot] commented on pull request #22112:
URL: https://github.com/apache/airflow/pull/22112#issuecomment-1065142292
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and
potiuk commented on pull request #22162:
URL: https://github.com/apache/airflow/pull/22162#issuecomment-1065139900
Another option (probably easiest):
4. Check which library uses pandas in the CI Airflow image:
* Run the image:
```
docker run -it
potiuk commented on pull request #22181:
URL: https://github.com/apache/airflow/pull/22181#issuecomment-1065130769
This is the last "blocker" to add Python 3.10 BTW :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
potiuk opened a new pull request #22181:
URL: https://github.com/apache/airflow/pull/22181
The limits of snowflake can be removed as the conditions for
all limits are fulfilled now.
This is the last thing to remove before we can enable Python 3.10
---
**^ Add
github-actions[bot] commented on pull request #22167:
URL: https://github.com/apache/airflow/pull/22167#issuecomment-1065101732
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and
bbovenzi commented on pull request #22167:
URL: https://github.com/apache/airflow/pull/22167#issuecomment-1065101632
LGTM. But some www tests are failing
--
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
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch add-summit-announcement
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
at 4f704dc Add Summit 2022 announcement
No new revisions were added by this update.
This is an automated email from the ASF dual-hosted git repository.
bbovenzi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 7bd8b2d Add oss_task_handler into alibaba-provider and enable remote
logging to OSS (#21785)
add 9eb1a1c
bbovenzi merged pull request #22170:
URL: https://github.com/apache/airflow/pull/22170
--
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:
github-actions[bot] commented on pull request #21868:
URL: https://github.com/apache/airflow/pull/21868#issuecomment-1065090466
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not
potiuk commented on a change in pull request #22179:
URL: https://github.com/apache/airflow/pull/22179#discussion_r824685409
##
File path: airflow/providers/google/suite/hooks/drive.py
##
@@ -197,7 +197,7 @@ def upload_file(self, local_location: str, remote_location:
str) ->
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 7bd8b2d Add oss_task_handler into
potiuk closed issue #21748:
URL: https://github.com/apache/airflow/issues/21748
--
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 merged pull request #21785:
URL: https://github.com/apache/airflow/pull/21785
--
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 03d0c70 Switch unit tests for oss operator and
potiuk merged pull request #22178:
URL: https://github.com/apache/airflow/pull/22178
--
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 closed issue #17617:
URL: https://github.com/apache/airflow/issues/17617
--
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 46a120d Fix RedshiftDataOperator and update doc (#22157)
add 9e06187 Add pip_install_options to
potiuk merged pull request #22158:
URL: https://github.com/apache/airflow/pull/22158
--
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:
ashb commented on a change in pull request #22167:
URL: https://github.com/apache/airflow/pull/22167#discussion_r824678992
##
File path: airflow/www/templates/airflow/dags.html
##
@@ -39,7 +39,7 @@
-
+
Review comment:
Fixed to `__DAG_ID__~ as `$` and
eladkal commented on issue #22115:
URL: https://github.com/apache/airflow/issues/22115#issuecomment-1065075205
@prashantpaidi i think it would be good if you can open a draft PR with what
you had in mind then we can review the code. You can make descion of how you
think it should be done
boring-cyborg[bot] commented on pull request #22179:
URL: https://github.com/apache/airflow/pull/22179#issuecomment-1065069381
Congratulations on your first Pull Request and welcome to the Apache Airflow
community! If you have any issues or are unsure about any anything please check
our
ulsc opened a new pull request #22179:
URL: https://github.com/apache/airflow/pull/22179
Adding `resumable=True` parameter to `MediaFileUpload` gives the ability to
upload files bigger than 5MB to Google Drive.
--
This is an automated message from the Apache Git Service.
To respond to
prashantpaidi commented on issue #22115:
URL: https://github.com/apache/airflow/issues/22115#issuecomment-1065066706
So after importing, I should create the new variable and assign an increased
value or while passing the parameter add the value?
what should be the timeout seconds
EricGao888 commented on issue #17617:
URL: https://github.com/apache/airflow/issues/17617#issuecomment-1065062989
> Merged it . This looks cool. Thanks! Please proceed with other tests :)
All unit tests in alibaba provider have been fixed now : ) @potiuk
--
This is an automated
EricGao888 commented on pull request #21785:
URL: https://github.com/apache/airflow/pull/21785#issuecomment-1065060866
@potiuk @mik-laj Seems ready to be merged. Would you like to help me merge
the commits? Thx!
--
This is an automated message from the Apache Git Service.
To respond 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 46a120d Fix RedshiftDataOperator and update doc
potiuk merged pull request #22157:
URL: https://github.com/apache/airflow/pull/22157
--
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:
EricGao888 opened a new pull request #22178:
URL: https://github.com/apache/airflow/pull/22178
* This PR switches unit tests for oss operator and oss sensor in
alibaba-provider to use mocks instead of real OSS, as discussed in #17617
* Once the commits are merged, all unit tests in
ephraimbuddy commented on a change in pull request #22177:
URL: https://github.com/apache/airflow/pull/22177#discussion_r824605652
##
File path: scripts/ci/testing/run_offline_sql_test.sh
##
@@ -0,0 +1,29 @@
+#!/usr/bin/env bash
+# Licensed to the Apache Software Foundation
ephraimbuddy opened a new pull request #22177:
URL: https://github.com/apache/airflow/pull/22177
This ensures our offline migrations command continue to work
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
ianbuss commented on pull request #22051:
URL: https://github.com/apache/airflow/pull/22051#issuecomment-1064998198
Thanks for your review @mik-laj! I will make the changes to the PR based on
your comments. I've also found some logic bugs in the dependency being used for
the gitignore
Bowrna commented on issue #15974:
URL: https://github.com/apache/airflow/issues/15974#issuecomment-1064991960
https://github.com/apache/airflow/blob/e63f6e36d14a8cd2462e80f26fb4809ab8698380/airflow/providers/docker/operators/docker_swarm.py#L93-L113
In docker swarm, we have the
boring-cyborg[bot] commented on issue #22176:
URL: https://github.com/apache/airflow/issues/22176#issuecomment-1064921322
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the
DaliDalmas opened a new issue #22176:
URL: https://github.com/apache/airflow/issues/22176
### Describe the issue with documentation
https://airflow.apache.org/
### How to solve the problem
_No response_
### Anything else
_No response_
### Are you
riccardo-salamanna commented on issue #22096:
URL: https://github.com/apache/airflow/issues/22096#issuecomment-1064895855
Hi !
I tested the envFrom statement in the flower deployment
envFrom:
│
- secretRef:
│
101 - 183 of 183 matches
Mail list logo
