pankajkoti commented on code in PR #36416:
URL: https://github.com/apache/airflow/pull/36416#discussion_r1436303356
##
airflow/providers/amazon/aws/operators/redshift_cluster.py:
##
@@ -535,19 +535,27 @@ def execute(self, context: Context):
time.sleep(self._
maiconkkl opened a new issue, #36432:
URL: https://github.com/apache/airflow/issues/36432
### Apache Airflow version
2.8.0
### If "Other Airflow 2 version" selected, which one?
current
### What happened?
When trying to implement custom authentication I am re
boring-cyborg[bot] commented on issue #36432:
URL: https://github.com/apache/airflow/issues/36432#issuecomment-1869304416
Thanks for opening your first issue here! Be sure to follow the issue
template! If you are willing to raise PR to address this issue please do so, no
need to wait for ap
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 af9328e6ff Remove duplicated methods in K8S pod op
eladkal merged PR #36427:
URL: https://github.com/apache/airflow/pull/36427
--
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: commits-unsubscr...@airflow.
dirrao opened a new pull request, #36431:
URL: https://github.com/apache/airflow/pull/36431
Remove getattr in es task handler when airflow min version bumped to 2.6
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
dirrao closed pull request #36430: Remove get_default_delete_local_copy method
from wasb after bumping 2.6
URL: https://github.com/apache/airflow/pull/36430
--
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.
eladkal pushed a change to branch remove-experimental-listener
in repository https://gitbox.apache.org/repos/asf/airflow.git
was be957e18ff note that task instance, dag and lifecycle listeners are
non-experimental
The rev
This is an automated email from the ASF dual-hosted git repository.
joshfell 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 7bd998e2ec fix(providers/amazon): remove event['m
josh-fell merged PR #36417:
URL: https://github.com/apache/airflow/pull/36417
--
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: commits-unsubscr...@airflo
This is an automated email from the ASF dual-hosted git repository.
joshfell 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 d986bed292 Consolidate loading delete_local_logs
josh-fell merged PR #36422:
URL: https://github.com/apache/airflow/pull/36422
--
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: commits-unsubscr...@airflo
This is an automated email from the ASF dual-hosted git repository.
joshfell 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 b99e4ecb1c Clean OSS task handler code after bump
josh-fell merged PR #36420:
URL: https://github.com/apache/airflow/pull/36420
--
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: commits-unsubscr...@airflo
This is an automated email from the ASF dual-hosted git repository.
joshfell 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 72a6264e20 Clean WASB task handler code after bum
josh-fell merged PR #36421:
URL: https://github.com/apache/airflow/pull/36421
--
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: commits-unsubscr...@airflo
This is an automated email from the ASF dual-hosted git repository.
joshfell 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 c45516096f Remove invalid todo comment from flink
josh-fell merged PR #36424:
URL: https://github.com/apache/airflow/pull/36424
--
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: commits-unsubscr...@airflo
github-actions[bot] commented on PR #32221:
URL: https://github.com/apache/airflow/pull/32221#issuecomment-1869161633
This pull request has been automatically marked as stale because it has not
had recent activity. It will be closed in 5 days if no further activity occurs.
Thank you for you
hussein-awala opened a new pull request, #36427:
URL: https://github.com/apache/airflow/pull/36427
We had to remove this method once we bumped the min airflow version to 2.5
or when we moved the helpers functions to the Kubernetes provider.
--
This is an automated message from the Apache
hussein-awala opened a new pull request, #36425:
URL: https://github.com/apache/airflow/pull/36425
(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 unsubscri
hussein-awala opened a new pull request, #36424:
URL: https://github.com/apache/airflow/pull/36424
The file already exists, and the comment doesn't explain what should be done
later.
related: #28512
--
This is an automated message from the Apache Git Service.
To respond to the mess
shohamy7 opened a new pull request, #36423:
URL: https://github.com/apache/airflow/pull/36423
---
Related: #28790
This PR adds documentation about the Zendesk connection in Airflow.
In addition, there has been some changes in the Zendesk connection form
hussein-awala opened a new pull request, #36422:
URL: https://github.com/apache/airflow/pull/36422
(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 unsubscri
hussein-awala opened a new pull request, #36421:
URL: https://github.com/apache/airflow/pull/36421
(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 unsubscri
hussein-awala opened a new pull request, #36420:
URL: https://github.com/apache/airflow/pull/36420
(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 unsubscri
hussein-awala closed pull request #36418: Refactor _manage_executor_state by
refreshing TIs in batch
URL: https://github.com/apache/airflow/pull/36418
--
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
MemphisMeng commented on issue #35377:
URL: https://github.com/apache/airflow/issues/35377#issuecomment-1869138710
> Do I see it right that you machine does not have a `~/.ssh/id_rsa` file
and SSH client fail on this instead of directly making a fallback to provided
user/password credential
MemphisMeng commented on issue #35377:
URL: https://github.com/apache/airflow/issues/35377#issuecomment-1869138378
@Bisk1 Probably I did miss something, can you tell me what key type is yours?
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
shohamy7 commented on PR #36419:
URL: https://github.com/apache/airflow/pull/36419#issuecomment-1869107822
@Taragolis
Please review, would like to here what you are think on this PR :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
shohamy7 opened a new pull request, #36419:
URL: https://github.com/apache/airflow/pull/36419
---
Related: #28790
This PR aims to add docs about each Spark connection in a separate page.
In addition, adding a more user-friendly connection form for Spark an
Lee-W opened a new pull request, #36417:
URL: https://github.com/apache/airflow/pull/36417
…
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guid
This is an automated email from the ASF dual-hosted git repository.
husseinawala 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 a1e1dc60fb handle tzinfo in S3Hook.is_keys_un
hussein-awala merged PR #36363:
URL: https://github.com/apache/airflow/pull/36363
--
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: commits-unsubscr...@ai
hussein-awala merged PR #36379:
URL: https://github.com/apache/airflow/pull/36379
--
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: commits-unsubscr...@ai
This is an automated email from the ASF dual-hosted git repository.
husseinawala 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 69f556dd13 Stop serializing timezone-naive da
bolkedebruin commented on PR #34446:
URL: https://github.com/apache/airflow/pull/34446#issuecomment-1868990695
Thinking about this more. Our own internal types often use underscores,
like 'google_cloud_storage'. It does look silly that users then get a warning
that they cannot take action
eladkal commented on PR #36149:
URL: https://github.com/apache/airflow/pull/36149#issuecomment-1868971768
So once we have unit test for the problematic case we can redo
https://github.com/apache/airflow/pull/34337 and find a solution for the edge
case which is now covered by tests
--
Thi
bolkedebruin commented on PR #34446:
URL: https://github.com/apache/airflow/pull/34446#issuecomment-1868970081
Sorry that's not the relevant section. The relevant section is 3.1 on scheme
which excludes the underscore.
--
This is an automated message from the Apache Git Service.
To respon
kuikeelc commented on PR #34446:
URL: https://github.com/apache/airflow/pull/34446#issuecomment-1868953775
As stated before (please read back): the Conn ID CAN contain underscores.
The underscore is labeled an unreserved character as defined in (RFC3968)[
https://datatracker.ietf.org/do
asafsneh commented on issue #29366:
URL: https://github.com/apache/airflow/issues/29366#issuecomment-1868951831
Hi @potiuk , thank you for the helpful recommendation and explanation. I
apologize if my previous question may have come across as rude or entitled. It
clearly slipped my mind tha
Bowrna commented on issue #36193:
URL: https://github.com/apache/airflow/issues/36193#issuecomment-1868944089
@vincbeck The following command is used to get the current auth manager
configured:
```airflow config get-value core auth_manager```. But the scope of this CLI
command is to list
potiuk commented on issue #29366:
URL: https://github.com/apache/airflow/issues/29366#issuecomment-1868941017
As everything in Open Source - if you care about a problem the best way to
speed up a solution is to create and lead to completion PR fixing it. This is
what I heartily recommend. T
bolkedebruin commented on code in PR #36410:
URL: https://github.com/apache/airflow/pull/36410#discussion_r1436064768
##
airflow/io/path.py:
##
@@ -173,10 +176,14 @@ def bucket(self) -> str:
@property
def key(self) -> str:
if self._url:
-return sel
bolkedebruin commented on code in PR #36410:
URL: https://github.com/apache/airflow/pull/36410#discussion_r1436064768
##
airflow/io/path.py:
##
@@ -173,10 +176,14 @@ def bucket(self) -> str:
@property
def key(self) -> str:
if self._url:
-return sel
bolkedebruin commented on code in PR #36410:
URL: https://github.com/apache/airflow/pull/36410#discussion_r1436064768
##
airflow/io/path.py:
##
@@ -173,10 +176,14 @@ def bucket(self) -> str:
@property
def key(self) -> str:
if self._url:
-return sel
bolkedebruin commented on PR #36379:
URL: https://github.com/apache/airflow/pull/36379#issuecomment-1868934258
I really am not a fan of naive datetimes. They just don't make sense as they
only exist subjectively. But that being said I think the change is fine and we
should aim for integrity
bolkedebruin commented on PR #34446:
URL: https://github.com/apache/airflow/pull/34446#issuecomment-1868926624
Scheme names, which the conn_id translates to when doing `get_uri` are not
allowed to have a underscore `_` according to RFC3986
https://datatracker.ietf.org/doc/html/rfc3986#secti
Lee-W opened a new pull request, #36416:
URL: https://github.com/apache/airflow/pull/36416
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)**
kapuza opened a new issue, #36415:
URL: https://github.com/apache/airflow/issues/36415
### Apache Airflow version
2.8.0
### If "Other Airflow 2 version" selected, which one?
_No response_
### What happened?
Install new airflow 2.8.0:
```bash
pip instal
Taragolis commented on PR #36393:
URL: https://github.com/apache/airflow/pull/36393#issuecomment-1868892752
Need to fix [Static
Checks](https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.rst):
- you could run locally `breeze static-checks {id-of-failed-check}
--all-files` e.g
dirrao commented on issue #36403:
URL: https://github.com/apache/airflow/issues/36403#issuecomment-1868887063
> scheduler_task_queued_timeout
Existing options doesn't help here. I am going to raise an MR to fix this
issue very soon.
--
This is an automated message from the Apache G
asafsneh commented on issue #29366:
URL: https://github.com/apache/airflow/issues/29366#issuecomment-1868885480
Hi
Is there any ETA to fix this issue?
It also happens to us
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
Lee-W commented on PR #36341:
URL: https://github.com/apache/airflow/pull/36341#issuecomment-1868885843
> Lets wait for a fix in upstream first
Sure.
--
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
uranusjr commented on PR #34446:
URL: https://github.com/apache/airflow/pull/34446#issuecomment-1868883640
I don’t want this in. Why can’t the conn types just not use underscores? Who
are the offenders here?
--
This is an automated message from the Apache Git Service.
To respond to the me
uranusjr commented on code in PR #36410:
URL: https://github.com/apache/airflow/pull/36410#discussion_r1436030085
##
airflow/io/path.py:
##
@@ -173,10 +176,14 @@ def bucket(self) -> str:
@property
def key(self) -> str:
if self._url:
-return self._u
This is an automated email from the ASF dual-hosted git repository.
taragolis 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 1ab7b08f7d Add Apache Livy connection documentat
This is an automated email from the ASF dual-hosted git repository.
taragolis 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 89aa437ef9 Add apache kylin conn type and kylin
Taragolis merged PR #36413:
URL: https://github.com/apache/airflow/pull/36413
--
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: commits-unsubscr...@airflo
Taragolis merged PR #36412:
URL: https://github.com/apache/airflow/pull/36412
--
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: commits-unsubscr...@airflo
adihakimi commented on issue #36403:
URL: https://github.com/apache/airflow/issues/36403#issuecomment-1868869567
We’re facing the same issue on v2.6.3.
We thought that ‘scheduler_task_queued_timeout’ might help but it doesn’t.
--
This is an automated message from the Apache Git Service
Taragolis commented on code in PR #36281:
URL: https://github.com/apache/airflow/pull/36281#discussion_r1436021229
##
airflow/providers/cncf/kubernetes/pod_launcher_deprecated.py:
##
@@ -148,13 +148,13 @@ def monitor_pod(self, pod: V1Pod, get_logs: bool) ->
tuple[State, str | N
uranusjr commented on code in PR #36281:
URL: https://github.com/apache/airflow/pull/36281#discussion_r1436019316
##
airflow/providers/cncf/kubernetes/pod_launcher_deprecated.py:
##
@@ -148,13 +148,13 @@ def monitor_pod(self, pod: V1Pod, get_logs: bool) ->
tuple[State, str | No
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch pendulum-3
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 3e7f50ece3 Add backcompat to pendulum 2
add 2be8e299eb Update airflow/serialization/serialized_objects.py
uranusjr commented on code in PR #36410:
URL: https://github.com/apache/airflow/pull/36410#discussion_r1436018519
##
airflow/io/path.py:
##
@@ -173,10 +176,14 @@ def bucket(self) -> str:
@property
def key(self) -> str:
if self._url:
-return self._u
Taragolis commented on code in PR #36281:
URL: https://github.com/apache/airflow/pull/36281#discussion_r1436018174
##
airflow/providers/cncf/kubernetes/pod_launcher_deprecated.py:
##
@@ -148,13 +148,13 @@ def monitor_pod(self, pod: V1Pod, get_logs: bool) ->
tuple[State, str | N
dirrao opened a new issue, #36414:
URL: https://github.com/apache/airflow/issues/36414
### What do you see as an issue?
I have seen a couple of metrics added with tags like (dag_id, task_id,
etc.). However, I do not see them in the airflow documentation.
### Solving the proble
67 matches
Mail list logo
