jedcunningham commented on code in PR #34716:
URL: https://github.com/apache/airflow/pull/34716#discussion_r1356131694
##
helm_tests/webserver/test_httproute_flower.py:
##
@@ -0,0 +1,97 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor licen
jedcunningham commented on code in PR #34349:
URL: https://github.com/apache/airflow/pull/34349#discussion_r1356129378
##
airflow/www/extensions/init_views.py:
##
@@ -308,5 +318,16 @@ def init_api_experimental(app):
"The authenticated user has full access.",
Re
Kyrremann commented on code in PR #34716:
URL: https://github.com/apache/airflow/pull/34716#discussion_r1356126098
##
helm_tests/webserver/test_httproute_flower.py:
##
@@ -0,0 +1,97 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license a
Kyrremann commented on code in PR #34716:
URL: https://github.com/apache/airflow/pull/34716#discussion_r1356119603
##
chart/templates/flower/flower-httproute.yaml:
##
@@ -0,0 +1,82 @@
+{{/*
+ Licensed to the Apache Software Foundation (ASF) under one
+ or more contributor licens
dirrao opened a new issue, #34877:
URL: https://github.com/apache/airflow/issues/34877
### Apache Airflow version
2.7.1
### What happened
Airflow running the clear_not_launched_queued_tasks function on a certain
frequency (default 30 seconds). When we run the airflow on
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 84941f86d7 Add back decorator `has_access` (#34786)
potiuk merged PR #34786:
URL: https://github.com/apache/airflow/pull/34786
--
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.a
potiuk commented on issue #32335:
URL: https://github.com/apache/airflow/issues/32335#issuecomment-1758912683
> https://github.com/kubernetes/git-sync/issues/827
Interesting. I see you proposed a fix (cool!)
--
This is an automated message from the Apache Git Service.
To respond to
potiuk commented on PR #34782:
URL: https://github.com/apache/airflow/pull/34782#issuecomment-1758908356
BTW. I still do not understand how "worker" could be affected but at least
this fix might fix the liveness probe issue mentioned in
https://github.com/apache/airflow/issues/34612#issueco
potiuk commented on issue #34612:
URL: https://github.com/apache/airflow/issues/34612#issuecomment-1758907339
While I understand why the liveness probe mentioned by @Aakcht (because it
runs the app directly) in Helm chart could fail in this case, I do not think
the "worker" should have thi
potiuk commented on code in PR #34782:
URL: https://github.com/apache/airflow/pull/34782#discussion_r1356050515
##
airflow/providers/celery/executors/celery_executor_utils.py:
##
@@ -60,16 +60,26 @@
TaskInstanceInCelery = Tuple[TaskInstanceKey, CommandType, Optional[str],
potiuk commented on PR #34782:
URL: https://github.com/apache/airflow/pull/34782#issuecomment-1758902407
> I wonder if we should put the entire `app` initialise logic into a
function and have `app = _get_celery_app()` instead.
Yes. This is better idea. Also `@provider_configuration_lo
Bowrna commented on issue #34838:
URL: https://github.com/apache/airflow/issues/34838#issuecomment-1758885562
sure @ghostp13409 , ping here if you need any help
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
Seokyun-Ha commented on PR #34643:
URL: https://github.com/apache/airflow/pull/34643#issuecomment-1758863918
Thanks you all guys to review this PR ❤️ I will request to schedule PR
merge on Airflow slack.
cc. @alexott @Lee-W @utkarsharma2
--
This is an automated message from the A
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 6c50ef59cc fix(TimeSensorAsync): use DAG timezone (
boring-cyborg[bot] commented on PR #33406:
URL: https://github.com/apache/airflow/pull/33406#issuecomment-1758854997
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 contributions.
potiuk merged PR #33406:
URL: https://github.com/apache/airflow/pull/33406
--
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.a
potiuk closed issue #33256: TimeSensorAsync does not use DAG timezone to
convert naive time input
URL: https://github.com/apache/airflow/issues/33256
--
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
potiuk commented on code in PR #34349:
URL: https://github.com/apache/airflow/pull/34349#discussion_r1356013126
##
airflow/www/extensions/init_views.py:
##
@@ -308,5 +318,16 @@ def init_api_experimental(app):
"The authenticated user has full access.",
RemovedIn
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1356003454
##
airflow/www/security_manager.py:
##
@@ -738,24 +633,9 @@ def create_perm_vm_for_all_dag(self) -> None:
def check_authorization(
self,
perms:
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355994191
##
airflow/www/security_manager.py:
##
@@ -269,125 +272,47 @@ def get_user_roles(user=None):
user = g.user
return user.roles
-def get_reada
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355994191
##
airflow/www/security_manager.py:
##
@@ -269,125 +272,47 @@ def get_user_roles(user=None):
user = g.user
return user.roles
-def get_reada
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355994191
##
airflow/www/security_manager.py:
##
@@ -269,125 +272,47 @@ def get_user_roles(user=None):
user = g.user
return user.roles
-def get_reada
shubham22 commented on PR #34870:
URL: https://github.com/apache/airflow/pull/34870#issuecomment-1758820613
Thanks for adding this @Taragolis. I haven't had time to do this myself due
to travel for [Community over Code](https://communityovercode.org/)
--
This is an automated message fro
ghostp13409 commented on issue #34838:
URL: https://github.com/apache/airflow/issues/34838#issuecomment-1758817607
Hi @Bowrna,
I'm a first time contributor, and i feel like I could give this a try. Can I
work on it?
--
This is an automated message from the Apache Git Service.
To respon
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355927810
##
airflow/api_connexion/security.py:
##
@@ -48,10 +61,194 @@ def requires_access_decorator(func: T):
@wraps(func)
def decorated(*args, **kwargs):
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355974367
##
airflow/api_connexion/security.py:
##
@@ -48,10 +61,194 @@ def requires_access_decorator(func: T):
@wraps(func)
def decorated(*args, **kwargs):
michalsosn commented on issue #10385:
URL: https://github.com/apache/airflow/issues/10385#issuecomment-1758793381
[this](https://github.com/apache/airflow/compare/main...michalsosn:airflow:10385-DataprocInstantiateWorkflowTemplateOperator-on_kill)
change looks reasonable. Now I need to thor
vchiapaikeo commented on PR #34862:
URL: https://github.com/apache/airflow/pull/34862#issuecomment-1758778346
Good call. What about some additional styling?
https://github.com/apache/airflow/assets/9200263/0c9bcb63-342d-4e70-b726-a52de7388283";>
https://github.com/apache/airflow
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355941755
##
airflow/www/extensions/init_jinja_globals.py:
##
@@ -69,10 +70,13 @@ def prepare_jinja_globals():
"git_version": git_version,
"k8s_or_k8sc
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355941755
##
airflow/www/extensions/init_jinja_globals.py:
##
@@ -69,10 +70,13 @@ def prepare_jinja_globals():
"git_version": git_version,
"k8s_or_k8sc
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355941755
##
airflow/www/extensions/init_jinja_globals.py:
##
@@ -69,10 +70,13 @@ def prepare_jinja_globals():
"git_version": git_version,
"k8s_or_k8sc
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355927810
##
airflow/api_connexion/security.py:
##
@@ -48,10 +61,194 @@ def requires_access_decorator(func: T):
@wraps(func)
def decorated(*args, **kwargs):
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355925044
##
airflow/api_connexion/endpoints/xcom_endpoint.py:
##
@@ -39,14 +39,7 @@
from airflow.api_connexion.types import APIResponse
-@security.requires_access(
-
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355923325
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355894015
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355917769
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355917769
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355917769
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355917769
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355907725
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -646,13 +606,8 @@ def patch_task_instance(
return task_instance_reference_schema.dump(ti)
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355907009
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -527,13 +497,8 @@ def post_clear_task_instances(*, dag_id: str, session:
Session = NEW_SESSION) ->
Taragolis closed issue #34876: Postgres conn type should normalize to
postgresql (instead of the inverse)
URL: https://github.com/apache/airflow/issues/34876
--
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 t
Taragolis commented on issue #34876:
URL: https://github.com/apache/airflow/issues/34876#issuecomment-1758725309
Airflow Connection URL it is representation (serialisation) of Airflow
Connection, not psycopg2 URL nor SQLAlchemy URL
> Call Connection.get_uri() on a connection with conn
github-actions[bot] commented on PR #22253:
URL: https://github.com/apache/airflow/pull/22253#issuecomment-1758720195
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
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355902071
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -461,13 +436,8 @@ def get_task_instances_batch(session: Session =
NEW_SESSION) -> APIResponse:
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355894015
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355894015
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
potiuk commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355894015
##
airflow/api_connexion/endpoints/task_instance_endpoint.py:
##
@@ -61,13 +61,8 @@
T = TypeVar("T")
-@security.requires_access(
-[
-(permissions.ACTI
bakome commented on issue #32335:
URL: https://github.com/apache/airflow/issues/32335#issuecomment-1758683759
I was able to implement git-sync v4 but faced interesting problem there. I
also open a PR on git-sync because I think airflow instances can be heavily
impacted by this weird behavio
dhuang opened a new issue, #34876:
URL: https://github.com/apache/airflow/issues/34876
### Apache Airflow version
2.7.1
### What happened
Currently Airflow normalizes conn types of `postgresql` to `postgres`, which
means calling`Connection.get_uri()` always gets a `postg
ferruzzi merged PR #34784:
URL: https://github.com/apache/airflow/pull/34784
--
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
This is an automated email from the ASF dual-hosted git repository.
ferruzzi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 27671fa53d Add session configuration to deployment manager's
responsibilities. (#34866)
add 84a3daed86 Implemen
ferruzzi commented on issue #34845:
URL: https://github.com/apache/airflow/issues/34845#issuecomment-1758654459
I just double-checked and this fix should be in Airflow 2.7.2 which is
currently being voted on and should be out Very Soon :tm: ((next week I think,
but don't hold me to that))
Taragolis commented on code in PR #34807:
URL: https://github.com/apache/airflow/pull/34807#discussion_r1355852578
##
airflow/providers/amazon/aws/waiters/databrew.json:
##
@@ -0,0 +1,54 @@
+{
+"version": 2,
+"waiters": {
+"job_complete": {
+"operatio
Taragolis commented on code in PR #34807:
URL: https://github.com/apache/airflow/pull/34807#discussion_r1355837567
##
airflow/providers/amazon/aws/waiters/databrew.json:
##
@@ -0,0 +1,54 @@
+{
+"version": 2,
+"waiters": {
+"job_complete": {
+"operatio
jens-scheffler-bosch commented on issue #34830:
URL: https://github.com/apache/airflow/issues/34830#issuecomment-1758623517
Checked for #34248 and can confirm that (1) code is correctly merged and (2)
fix is working! Thanks for making the release!
--
This is an automated message from the
Taragolis commented on code in PR #34874:
URL: https://github.com/apache/airflow/pull/34874#discussion_r1355831314
##
airflow/providers/google/cloud/hooks/cloud_run.py:
##
@@ -113,9 +113,11 @@ def update_job(
@GoogleBaseHook.fallback_to_default_project_id
def execute
ChloeSheasby opened a new pull request, #34874:
URL: https://github.com/apache/airflow/pull/34874
---
This PR aims to add the functionality of the "overrides" variable for the
CloudRunExecuteJobOperator as provided by the google.cloud.run.v2 package.
-
https://cloud.google.com/ru
boring-cyborg[bot] commented on PR #34874:
URL: https://github.com/apache/airflow/pull/34874#issuecomment-1758609418
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 Contribution
jens-scheffler-bosch commented on issue #34867:
URL: https://github.com/apache/airflow/issues/34867#issuecomment-1758601396
Okay, having a circular reference in DAG triggers might be generating
effects unforseen. Can you tell about the use case? Otherwise I rather believe
that we might need
jens-scheffler-bosch commented on issue #34865:
URL: https://github.com/apache/airflow/issues/34865#issuecomment-1758596926
Just to double-check, are you able to make a regression on Airflow 2.7.1?
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
michalsosn commented on issue #10385:
URL: https://github.com/apache/airflow/issues/10385#issuecomment-1758577018
### operators in dataproc module
```
not killable, don't start a process:
class DataprocCreateClusterOperator(GoogleCloudBaseOperator)
class DataprocScaleC
jens-scheffler-bosch commented on PR #34842:
URL: https://github.com/apache/airflow/pull/34842#issuecomment-1758556457
FYI Pipeline fails but unfortunately (at least for me) the logs do not show
details, error that I can see in raw logs are:
2023-10-11T15:03:03.3370615Z Remove trailin
Taragolis commented on issue #34869:
URL: https://github.com/apache/airflow/issues/34869#issuecomment-1758539398
There are only option to overflow date it try to get it greater than greater
`-12-31T23:59:59` in this method:
- So it possible if delay (provided or backoff) greater than
ephraimbuddy commented on issue #34830:
URL: https://github.com/apache/airflow/issues/34830#issuecomment-1758528533
> > > #34072 and #34136 work as expected.
> > > I also tested a few other PRs:
> > >
> > > * Works as expected: [Add missing audit logs for default flask action:
add,
vincbeck commented on code in PR #34807:
URL: https://github.com/apache/airflow/pull/34807#discussion_r1355765277
##
airflow/providers/amazon/aws/operators/glue.py:
##
@@ -230,3 +230,99 @@ def on_kill(self):
)
if not response["SuccessfulSubmissions"]:
This is an automated email from the ASF dual-hosted git repository.
jedcunningham 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 27671fa53d Add session configuration to depl
jedcunningham merged PR #34866:
URL: https://github.com/apache/airflow/pull/34866
--
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 commented on issue #34830:
URL: https://github.com/apache/airflow/issues/34830#issuecomment-1758490902
I tested my changes, they are present in the RC. I tested only the bug
fixes, and all looks good, except #34120 as discussed above.
--
This is an automated message from the
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 8a85fb4161 Doc: Add info on getting variables an
Taragolis merged PR #34834:
URL: https://github.com/apache/airflow/pull/34834
--
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
hussein-awala commented on issue #34830:
URL: https://github.com/apache/airflow/issues/34830#issuecomment-1758489492
> > #34072 and #34136 work as expected.
> > I also tested a few other PRs:
> >
> > * Works as expected: [Add missing audit logs for default flask action:
add, edit a
flolas commented on issue #34583:
URL: https://github.com/apache/airflow/issues/34583#issuecomment-1758489883
@vincbeck yes, i hope to send a PR in the weekend.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
Taragolis opened a new pull request, #34871:
URL: https://github.com/apache/airflow/pull/34871
Deprecated `token` parameter was removed in 8.0.0 of provider, the only
available option for provide token is use Slack API Connection
---
**^ Add meaningful descripti
Taragolis opened a new pull request, #34870:
URL: https://github.com/apache/airflow/pull/34870
This info might want to be useful for someone who intent to invoke AWS
Lambda synchronously 🙄 with timeout more than 5-6 minutes
cc: @shubham22
---
**^ Add mea
sahar1454 commented on issue #34869:
URL: https://github.com/apache/airflow/issues/34869#issuecomment-1758460080
@Taragolis I don't believe the fix you linked is related to the issue I
reported. That one solves the problem for this issue:
https://github.com/apache/airflow/issues/28171
Bu
ellisms commented on code in PR #34807:
URL: https://github.com/apache/airflow/pull/34807#discussion_r1355681377
##
airflow/providers/amazon/aws/operators/glue.py:
##
@@ -230,3 +230,99 @@ def on_kill(self):
)
if not response["SuccessfulSubmissions"]:
ellisms commented on code in PR #34807:
URL: https://github.com/apache/airflow/pull/34807#discussion_r1355680039
##
airflow/providers/amazon/aws/operators/glue.py:
##
@@ -230,3 +230,99 @@ def on_kill(self):
)
if not response["SuccessfulSubmissions"]:
ferruzzi commented on code in PR #34807:
URL: https://github.com/apache/airflow/pull/34807#discussion_r1355676198
##
tests/providers/amazon/aws/waiters/test_custom_waiters.py:
##
@@ -424,3 +425,42 @@ def test_steps_failed(self, mock_list_steps):
StepIds=[self.ST
uranusjr closed pull request #33307: Add helpers to better handle session in
tests
URL: https://github.com/apache/airflow/pull/33307
--
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 comm
uranusjr commented on PR #34851:
URL: https://github.com/apache/airflow/pull/34851#issuecomment-1758284518
Because conf is not called conf in this file
https://github.com/apache/airflow/blob/03a305f6600b822d4f4ab74ed5ae8c0f8905fbc2/airflow/models/dag.py#L80
--
This is an automated
vincbeck commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355579389
##
airflow/www/extensions/init_jinja_globals.py:
##
@@ -69,10 +70,13 @@ def prepare_jinja_globals():
"git_version": git_version,
"k8s_or_k8
vincbeck commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355579727
##
airflow/api_connexion/endpoints/task_endpoint.py:
##
@@ -22,21 +22,16 @@
from airflow.api_connexion import security
from airflow.api_connexion.exceptions import
Taragolis commented on issue #34869:
URL: https://github.com/apache/airflow/issues/34869#issuecomment-1758259679
Should be fixed as part of Airflow 2.6.0
- https://github.com/apache/airflow/pull/28172
--
This is an automated message from the Apache Git Service.
To respond to the message
Taragolis closed issue #34869: Scheduler crashes with OverflowError: date value
out of range when retries are added to ShortCircuitOperator
URL: https://github.com/apache/airflow/issues/34869
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
vincbeck commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1355567508
##
airflow/api_connexion/endpoints/task_endpoint.py:
##
@@ -22,21 +22,16 @@
from airflow.api_connexion import security
from airflow.api_connexion.exceptions import
ferruzzi commented on issue #34845:
URL: https://github.com/apache/airflow/issues/34845#issuecomment-1758253375
I know this isn't really an answer, but the root cause id that when you
combine a long `dag_id` and a long `task_id`, the total length of the metric
name exceeds OTel's max name l
uranusjr commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r138587
##
airflow/api_connexion/endpoints/task_endpoint.py:
##
@@ -22,21 +22,16 @@
from airflow.api_connexion import security
from airflow.api_connexion.exceptions import
ferruzzi closed issue #34845: Invalid stat name using opentelemetry
URL: https://github.com/apache/airflow/issues/34845
--
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 unsub
uranusjr commented on code in PR #34851:
URL: https://github.com/apache/airflow/pull/34851#discussion_r1355543632
##
airflow/config_templates/config.yml:
##
@@ -2237,6 +2237,16 @@ scheduler:
type: string
example: ~
default: "15"
+default_cron_timetable:
ferruzzi commented on issue #34845:
URL: https://github.com/apache/airflow/issues/34845#issuecomment-1758214925
> should we add them to the exemption list?
Must have been some awkward timing, adding new non-compliant names should
have been prevented by the unit tests Yeah, I guess
o-nikolas commented on PR #34747:
URL: https://github.com/apache/airflow/pull/34747#issuecomment-1758208475
Nice work @cBiscuitSurprise, I'm glad to see this one finally closed out.
Smart idea to make the new behaviour configurable and off by default, it limits
the blast radius of doing thi
o-nikolas commented on issue #31826:
URL: https://github.com/apache/airflow/issues/31826#issuecomment-1758161057
Fantastic! Thanks @Bowrna, I've assigned the issue to you :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
boring-cyborg[bot] commented on issue #34869:
URL: https://github.com/apache/airflow/issues/34869#issuecomment-1758153133
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
sahar1454 opened a new issue, #34869:
URL: https://github.com/apache/airflow/issues/34869
### Apache Airflow version
Other Airflow 2 version (please specify below)
### What happened
In airflow version `2.5.1`, adding retries on `ShortCircuitOperator` causes
the Scheduler
tagoad commented on issue #23542:
URL: https://github.com/apache/airflow/issues/23542#issuecomment-1758100903
It looks like this issue has been addressed, @rajrao you should update to
the latest Airflow version
#27523
--
This is an automated message from the Apache Git Service.
To res
tagoad closed issue #23542: Grid View: Task order rearranges constantly with
auto-refresh on
URL: https://github.com/apache/airflow/issues/23542
--
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 s
theaadya opened a new pull request, #34868:
URL: https://github.com/apache/airflow/pull/34868
I've written out the code that was previously only available as an image in
`CONTRIBUTORS_QUICK_START_VSCODE.rst`, making it more convenient for users as
they don't have to manually transcribe it f
boring-cyborg[bot] commented on issue #34867:
URL: https://github.com/apache/airflow/issues/34867#issuecomment-1758071871
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
1 - 100 of 151 matches
Mail list logo