eladkal opened a new pull request, #33399:
URL: https://github.com/apache/airflow/pull/33399
Revert https://github.com/apache/airflow/pull/33208
--
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 PR #33309:
URL: https://github.com/apache/airflow/pull/33309#issuecomment-1678487753
The main reason was the signature of the render_templelate method that could
not be changed for the regular non-mapped operator.
At least that"s what the comment says in the
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-2-7
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-2-7 by this push:
new 3295247e67 Updating
mai-nakagawa opened a new issue, #33400:
URL: https://github.com/apache/airflow/issues/33400
### Apache Airflow version
main (development)
### What happened
I always face the following error when I try to run a BigQuery query that
accesses [connected
potiuk commented on PR #33309:
URL: https://github.com/apache/airflow/pull/33309#issuecomment-1678489645
Or maybe you think we can change the signature ? I believe (on the phone
now) the original comment mentioned that the signature cannot be changed
because the methods that would have to
eladkal merged PR #33399:
URL: https://github.com/apache/airflow/pull/33399
--
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.
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 9927bdaa88 revert change order of issues for
Lee-W opened a new pull request, #33401:
URL: https://github.com/apache/airflow/pull/33401
This is the following PR to https://github.com/apache/airflow/pull/33196 and
intends to solve the issue that the `soft_fail` argument might not work on all
sensors.
This PR handles the case
arjunanan6 opened a new issue, #33402:
URL: https://github.com/apache/airflow/issues/33402
### Apache Airflow version
2.6.3
### What happened
This is a new problem that started happening overnight for us on only one of
our environments. The only change introduced was to
potiuk commented on PR #33309:
URL: https://github.com/apache/airflow/pull/33309#issuecomment-1678486007
The main reason was the signature of the render_templelate method that could
not be changed for the regular non-mapped operator.
At least that"s what the comment says in the
RNHTTR commented on issue #33402:
URL: https://github.com/apache/airflow/issues/33402#issuecomment-1678954965
What's the value for
[delete_worker_pods](https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html#delete-worker-pods)?
--
This is an automated message from
RNHTTR commented on issue #33402:
URL: https://github.com/apache/airflow/issues/33402#issuecomment-1678960320
Can you provide reproduction steps? I haven't seen this elsewhere in 2.6.3,
and given that your environments are nearly-but-not-identical makes this sound
like a configuration
jedcunningham commented on code in PR #33390:
URL: https://github.com/apache/airflow/pull/33390#discussion_r1294648953
##
RELEASE_NOTES.rst:
##
@@ -37,6 +37,13 @@ Old Graph View is removed (#32958)
""
The old Graph View is removed. The new
This is an automated email from the ASF dual-hosted git repository.
vincbeck 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 8775f841dd Move methods from security managers
vincbeck merged PR #33044:
URL: https://github.com/apache/airflow/pull/33044
--
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.
kaxilnaik 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 e57d0c913c Doc: Add concrete examples for
This is an automated email from the ASF dual-hosted git repository.
mobuchowski pushed a change to branch openlineage_dont_run_tis_executor
in repository https://gitbox.apache.org/repos/asf/airflow.git
omit 9cf049c471 Merge branch 'main' into openlineage_dont_run_tis_executor
omit
This is an automated email from the ASF dual-hosted git repository.
mobuchowski pushed a commit to branch openlineage_dont_run_tis_executor
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 975e1a61afa86a3c5edf56b2e5b7024dba160cfd
Author: Maciej Obuchowski
AuthorDate: Sun Aug
vincbeck commented on code in PR #33219:
URL: https://github.com/apache/airflow/pull/33219#discussion_r1294686733
##
airflow/providers/amazon/aws/operators/sagemaker.py:
##
@@ -1523,3 +1523,259 @@ def execute(self, context: Context) -> str:
arn = ans["ExperimentArn"]
This is an automated email from the ASF dual-hosted git repository.
vincbeck 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 a1576d39d2 Fix outdated test name and
Lee-W opened a new pull request, #33407:
URL: https://github.com/apache/airflow/pull/33407
---
**^ Add meaningful description above**
Read the **[Pull Request
arjunanan6 commented on issue #33402:
URL: https://github.com/apache/airflow/issues/33402#issuecomment-1678787624
This new set of logs shows where the Deletion process suddenly stops:
```
[2023-08-15T11:01:58.842+] {kubernetes_executor.py:764} INFO - Deleted
pod:
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294510954
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -160,6 +162,142 @@ def get_link(
return XCom.get_value(key=XCOM_RUN_PAGE_URL_KEY,
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294511481
##
docs/apache-airflow-providers-databricks/operators/jobs_create.rst:
##
@@ -0,0 +1,91 @@
+ .. Licensed to the Apache Software Foundation (ASF) under one
+
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294514250
##
airflow/providers/databricks/hooks/databricks.py:
##
@@ -135,9 +137,27 @@ def __init__(
) -> None:
super().__init__(databricks_conn_id,
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294521512
##
airflow/providers/databricks/hooks/databricks.py:
##
@@ -135,9 +137,27 @@ def __init__(
) -> None:
super().__init__(databricks_conn_id,
ellisms commented on code in PR #33219:
URL: https://github.com/apache/airflow/pull/33219#discussion_r1294549222
##
airflow/providers/amazon/aws/hooks/sagemaker_notebook.py:
##
Review Comment:
Refactored sagemaker_notebook.py into sagemaker.py.
--
This is an automated
erdos2n commented on PR #33408:
URL: https://github.com/apache/airflow/pull/33408#issuecomment-1678909837
@eladkal , I added test cases for the `role_arn`, can you let me know if the
tests were added correctly?
Thank you
--
This is an automated message from the Apache Git Service.
To
arjunanan6 commented on issue #33402:
URL: https://github.com/apache/airflow/issues/33402#issuecomment-1678965803
Unfortunately, I do not know how to reproduce it yet. I've only seen this
problem happen since the change to job_heartbeat_sec yesterday - but that was
applied to all
vincbeck merged PR #33407:
URL: https://github.com/apache/airflow/pull/33407
--
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:
kaxil merged PR #33296:
URL: https://github.com/apache/airflow/pull/33296
--
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:
vincbeck commented on code in PR #33408:
URL: https://github.com/apache/airflow/pull/33408#discussion_r1294716332
##
airflow/providers/amazon/aws/operators/glue.py:
##
@@ -53,7 +53,8 @@ class GlueJobOperator(BaseOperator):
:param num_of_dpus: Number of AWS Glue DPUs to
ephraimbuddy opened a new pull request, #33411:
URL: https://github.com/apache/airflow/pull/33411
When a PR is referenced by other PRs, our dev tool for getting the correct
commit lists the latest commit when looking for the commit sha but we should
get the oldest.
If you run:
fgalind1 commented on PR #31066:
URL: https://github.com/apache/airflow/pull/31066#issuecomment-1679194150
does it looks good @jedcunningham /@potiuk?
--
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
ephraimbuddy opened a new pull request, #33412:
URL: https://github.com/apache/airflow/pull/33412
Test pypi doesn't work for org and our uploads are failing each time. Time
to remove it!
--
This is an automated message from the Apache Git Service.
To respond to the message,
ahidalgob opened a new pull request, #33413:
URL: https://github.com/apache/airflow/pull/33413
Add read only validation to DagRunEditForm and TaskInstanceEditForm read
only fields.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
tseruga opened a new issue, #33410:
URL: https://github.com/apache/airflow/issues/33410
### Apache Airflow version
2.6.3
### What happened
First and foremost, I understand that the current posture is that the
currently implemented SLA mechanism is buggy and is being
o-nikolas closed pull request #33397: Remove `supports_ad_hoc_ti_run` from
executor API
URL: https://github.com/apache/airflow/pull/33397
--
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
alexbegg commented on PR #33359:
URL: https://github.com/apache/airflow/pull/33359#issuecomment-1679248385
@potiuk this is now ready for review again
--
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
ellisms commented on code in PR #33219:
URL: https://github.com/apache/airflow/pull/33219#discussion_r1294857841
##
airflow/providers/amazon/aws/operators/sagemaker.py:
##
@@ -1523,3 +1523,259 @@ def execute(self, context: Context) -> str:
arn = ans["ExperimentArn"]
ellisms commented on code in PR #33219:
URL: https://github.com/apache/airflow/pull/33219#discussion_r1294874707
##
scripts/ci/docker-compose/empty/git_version:
##
Review Comment:
This should be gone now.
--
This is an automated message from the Apache Git Service.
To
insomnes opened a new issue, #33415:
URL: https://github.com/apache/airflow/issues/33415
### Apache Airflow version
2.6.3
### What happened
Got error while using `set_state()` to set task state to
`TaskInstanceState.RESTARTING`
vincbeck commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1294751595
##
airflow/auth/managers/base_auth_manager.py:
##
@@ -54,6 +58,67 @@ def get_user_id(self) -> str:
def is_logged_in(self) -> bool:
"""Return whether
erdos2n commented on code in PR #33408:
URL: https://github.com/apache/airflow/pull/33408#discussion_r1294751897
##
airflow/providers/amazon/aws/hooks/glue.py:
##
@@ -44,7 +44,8 @@ class GlueJobHook(AwsBaseHook):
:param retry_limit: Maximum number of times to retry this
erdos2n commented on code in PR #33408:
URL: https://github.com/apache/airflow/pull/33408#discussion_r1294756676
##
airflow/providers/amazon/aws/hooks/glue.py:
##
@@ -114,12 +119,14 @@ def create_glue_job_config(self) -> dict:
"ScriptLocation":
erdos2n commented on code in PR #33408:
URL: https://github.com/apache/airflow/pull/33408#discussion_r1294781445
##
tests/providers/amazon/aws/hooks/test_glue.py:
##
@@ -90,6 +90,56 @@ class JobNotFoundException(Exception):
assert result is False
erdos2n commented on code in PR #33408:
URL: https://github.com/apache/airflow/pull/33408#discussion_r1294791349
##
tests/providers/amazon/aws/hooks/test_glue.py:
##
@@ -66,6 +66,22 @@ def test_get_iam_execution_role(self, role_path):
assert "Arn" in iam_role["Role"]
o-nikolas commented on code in PR #33397:
URL: https://github.com/apache/airflow/pull/33397#discussion_r1294821164
##
airflow/executors/base_executor.py:
##
@@ -104,7 +104,6 @@ class BaseExecutor(LoggingMixin):
:param parallelism: how many jobs should run at one time. Set
ahidalgob commented on PR #33413:
URL: https://github.com/apache/airflow/pull/33413#issuecomment-1679296127
It would be nice to have a base airflow form Class so we could modify
`Form.populate_obj` there and not for every form that has a `ReadOnly` field,
but I'm not sure the change it's
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from a1576d39d2 Fix outdated test name and description in BatchSensor
(#33407)
add 5b104a9020 Fix getting
ephraimbuddy merged PR #33411:
URL: https://github.com/apache/airflow/pull/33411
--
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:
amoghrajesh commented on PR #33379:
URL: https://github.com/apache/airflow/pull/33379#issuecomment-1679270039
@hussein-awala seems like the commit didn't do the trick here. I pushed a
new fix now, might just do the trick now
--
This is an automated message from the Apache Git Service.
To
aamster commented on issue #33014:
URL: https://github.com/apache/airflow/issues/33014#issuecomment-1679297020
I would consider this a bug, not a new feature. I should never be uncertain
what airflow is going to do. Here the obvious thing I would expect is that
downstream tasks would be
insomnes closed issue #33415: `set_state` error on task state not shared with
DagRunState
URL: https://github.com/apache/airflow/issues/33415
--
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
Lee-W opened a new pull request, #33403:
URL: https://github.com/apache/airflow/pull/33403
---
**^ Add meaningful description above**
Read the **[Pull Request
sonmezonur commented on code in PR #33174:
URL: https://github.com/apache/airflow/pull/33174#discussion_r1294357934
##
tests/providers/cncf/kubernetes/operators/test_pod.py:
##
@@ -920,6 +921,64 @@ def test_pod_template_file_kwargs_override(self,
randomize_name,
uranusjr opened a new pull request, #33404:
URL: https://github.com/apache/airflow/pull/33404
Previously, with catchup=False, CronTriggerTimetable would aggressively cut
off a run if the scheduler doesn't ask to schedule the next run immediately.
This causes DAGs to seemingly mysteriously
Lee-W opened a new pull request, #33405:
URL: https://github.com/apache/airflow/pull/33405
This is the following PR to https://github.com/apache/airflow/pull/33196 and
intends to solve the issue that the soft_fail argument might not work on all
sensors.
This pull request addresses
mis98zb commented on PR #32876:
URL: https://github.com/apache/airflow/pull/32876#issuecomment-1678742961
Thank you for the answer!
But seems this does not work for me.
The `@provide_session` will call submit() when the function exit, and it's
inside the `@provide_session` of
erdos2n opened a new pull request, #33408:
URL: https://github.com/apache/airflow/pull/33408
GlueOperator currently requires `iam_role_name` and `endpoint_url` to fetch
the ARN from AWS. This ARN eventually gets passed into the config for Glue Job.
---
**^ Add
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294514377
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -24,6 +24,8 @@
from logging import Logger
from typing import TYPE_CHECKING, Any, Sequence
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294513909
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -24,6 +24,8 @@
from logging import Logger
from typing import TYPE_CHECKING, Any, Sequence
dstandish commented on issue #21867:
URL: https://github.com/apache/airflow/issues/21867#issuecomment-1678858953
Warning... this is a long one... just tried to catch up on the current
state and share some thoughts and some reflections based on learnings in AIP-52
> We need also
joshowen commented on PR #33242:
URL: https://github.com/apache/airflow/pull/33242#issuecomment-1678691128
> I think we need a test case to cover the one DAG case.
The one dag case is pretty well covered in the integration tests. I
previously had a bug in this code and used those to
dstandish commented on issue #21867:
URL: https://github.com/apache/airflow/issues/21867#issuecomment-1678789067
Yeah it will be nice to have "living doc" of sorts, a draft AIP, where the
proposal can be clearly outlined, edge cases and caveats noted, and we can
refine and iterate. Will
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294515977
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -160,6 +162,142 @@ def get_link(
return XCom.get_value(key=XCOM_RUN_PAGE_URL_KEY,
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294517299
##
airflow/providers/databricks/hooks/databricks.py:
##
@@ -135,9 +137,27 @@ def __init__(
) -> None:
super().__init__(databricks_conn_id,
alexbegg commented on PR #33359:
URL: https://github.com/apache/airflow/pull/33359#issuecomment-1678600424
Moving to a draft until I rebase the branch and make it ready
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
This is an automated email from the ASF dual-hosted git repository.
mobuchowski 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 8e738cd0ad openlineage: do not try to redact
mobuchowski merged PR #33393:
URL: https://github.com/apache/airflow/pull/33393
--
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:
boring-cyborg[bot] commented on PR #33406:
URL: https://github.com/apache/airflow/pull/33406#issuecomment-1678741630
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
SapphicCode opened a new pull request, #33406:
URL: https://github.com/apache/airflow/pull/33406
Fixes: #33256
TimeSensorAsync used to convert naive timezones to UTC regardless of DAG
timezone. This fixes that.
Please let me know if this is a significant change and I'll write
stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1294519821
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -160,6 +162,142 @@ def get_link(
return XCom.get_value(key=XCOM_RUN_PAGE_URL_KEY,
vincbeck commented on PR #33408:
URL: https://github.com/apache/airflow/pull/33408#issuecomment-1679429681
You have some static checks failure, please [run
them](https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.rst) to
fix these failures.
--
This is an automated message
vincbeck commented on code in PR #33219:
URL: https://github.com/apache/airflow/pull/33219#discussion_r1294992827
##
airflow/providers/amazon/aws/operators/sagemaker.py:
##
@@ -1523,3 +1524,246 @@ def execute(self, context: Context) -> str:
arn = ans["ExperimentArn"]
vincbeck commented on code in PR #33219:
URL: https://github.com/apache/airflow/pull/33219#discussion_r1294992979
##
airflow/providers/amazon/aws/operators/sagemaker.py:
##
@@ -1523,3 +1524,246 @@ def execute(self, context: Context) -> str:
arn = ans["ExperimentArn"]
shahar1 commented on PR #33419:
URL: https://github.com/apache/airflow/pull/33419#issuecomment-1679545805
It might be worth adding documentation of where/how it's being used to avoid
(non-)accidental deletion for the next time :)
--
This is an automated message from the Apache Git
Dan-Git-Rep opened a new pull request, #33420:
URL: https://github.com/apache/airflow/pull/33420
three options to connect to GCP not the initial specified two
---
**^ Add meaningful description above**
Read the **[Pull Request
bbovenzi opened a new pull request, #33421:
URL: https://github.com/apache/airflow/pull/33421
Fix how we handle nested groups to make sure edge labels inside of groups
are correctly positions and that nested nodes with the same label don't mess
with rendering.
| Before |
vincbeck commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1295115248
##
airflow/auth/managers/fab/fab_auth_manager.py:
##
@@ -17,13 +17,37 @@
# under the License.
from __future__ import annotations
+import itertools
+
from flask
vincbeck commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1295081179
##
airflow/auth/managers/fab/fab_auth_manager.py:
##
@@ -81,3 +148,35 @@ def get_url_user_profile(self) -> str | None:
if not
vandonr-amz commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1295115914
##
airflow/auth/managers/fab/fab_auth_manager.py:
##
@@ -17,13 +17,37 @@
# under the License.
from __future__ import annotations
+import itertools
+
from
hussein-awala commented on code in PR #33406:
URL: https://github.com/apache/airflow/pull/33406#discussion_r1295148713
##
airflow/sensors/time_sensor.py:
##
@@ -64,7 +64,7 @@ def __init__(self, *, target_time, **kwargs):
self.target_time = target_time
vincbeck commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1294951943
##
airflow/auth/managers/base_auth_manager.py:
##
@@ -54,6 +58,67 @@ def get_user_id(self) -> str:
def is_logged_in(self) -> bool:
"""Return whether
alexbegg opened a new pull request, #33416:
URL: https://github.com/apache/airflow/pull/33416
This was filling pydocstyle rule D205
---
**^ Add meaningful description above**
Read the **[Pull Request
vincbeck commented on code in PR #33219:
URL: https://github.com/apache/airflow/pull/33219#discussion_r1294989565
##
airflow/providers/amazon/aws/operators/sagemaker.py:
##
@@ -1523,3 +1524,246 @@ def execute(self, context: Context) -> str:
arn = ans["ExperimentArn"]
jens-scheffler-bosch opened a new pull request, #33417:
URL: https://github.com/apache/airflow/pull/33417
I just saw the PR #33296 in Slack and really liked it - I was struggeling to
find concrete examples myself multiple times. Thanks to @RNHTTR for making
this!
When taking a look
alexbegg commented on PR #33416:
URL: https://github.com/apache/airflow/pull/33416#issuecomment-1679426859
@ferruzzi it looks like the PR for these files being added passed the status
check before your PR changing adding rule D205, but then got merged after.
This hopefully should fix
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 5b104a9020 Fix getting correct commit from multiple referenced PR
(#33411)
add 3766ab07d7 More complete fix for
ferruzzi commented on PR #33416:
URL: https://github.com/apache/airflow/pull/33416#issuecomment-1679501589
LGTM, I'll approve and merge it as soon as the CI goes green. Thanks for
that. I made sure everything that was already committed passed, but there are
bound to be a couple that
potiuk opened a new pull request, #33419:
URL: https://github.com/apache/airflow/pull/33419
This reverts commit a29887451fa4807e56fb60e9ef72af4bcdf4924f.
---
**^ Add meaningful description above**
Read the **[Pull Request
potiuk merged PR #33418:
URL: https://github.com/apache/airflow/pull/33418
--
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:
boring-cyborg[bot] commented on PR #33420:
URL: https://github.com/apache/airflow/pull/33420#issuecomment-1679559348
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
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 3766ab07d7 More complete fix for dev release scripts to filter commit
for PR (#33418)
add cc360b73c9 Fixing
vandonr-amz commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1295115914
##
airflow/auth/managers/fab/fab_auth_manager.py:
##
@@ -17,13 +17,37 @@
# under the License.
from __future__ import annotations
+import itertools
+
from
potiuk commented on code in PR #33355:
URL: https://github.com/apache/airflow/pull/33355#discussion_r1295118289
##
airflow/operators/python.py:
##
@@ -606,7 +613,39 @@ def _prepare_venv(self, venv_path: Path) -> None:
index_urls=self.index_urls,
)
+
hussein-awala commented on issue #33402:
URL: https://github.com/apache/airflow/issues/33402#issuecomment-1679598748
Actually we set the resource version to 0 on failure:
```python
except Exception:
self.log.exception("Unknown error in
hussein-awala commented on code in PR #33420:
URL: https://github.com/apache/airflow/pull/33420#discussion_r1295123482
##
docs/apache-airflow-providers-google/connections/gcp.rst:
##
@@ -27,7 +27,7 @@ The Google Cloud connection type enables the Google Cloud
Integrations.
vincbeck commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1295119967
##
airflow/auth/managers/fab/fab_auth_manager.py:
##
@@ -17,13 +17,37 @@
# under the License.
from __future__ import annotations
+import itertools
+
from flask
hussein-awala merged PR #33420:
URL: https://github.com/apache/airflow/pull/33420
--
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:
1 - 100 of 178 matches
Mail list logo