Re: [PR] feat: full support for google credentials in gcloud-aio clients [airflow]

2024-01-18 Thread via GitHub
m1racoli commented on code in PR #36849: URL: https://github.com/apache/airflow/pull/36849#discussion_r1458522587 ## airflow/providers/google/cloud/hooks/bigquery.py: ## @@ -3513,11 +3518,11 @@ async def get_table_client( access to the specified project.

Re: [PR] check sagemaker training job status before deferring SageMakerTrainingOperator [airflow]

2024-01-18 Thread via GitHub
pankajkoti commented on code in PR #36685: URL: https://github.com/apache/airflow/pull/36685#discussion_r1458515289 ## airflow/providers/amazon/aws/hooks/sagemaker.py: ## @@ -1300,3 +1300,13 @@ def create_auto_ml_job( if "BestCandidate" in res:

(airflow) branch main updated: Add `get_permitted_menu_items` in auth manager (#36871)

2024-01-18 Thread potiuk
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 a68c63fc87 Add `get_permitted_menu_items` in auth

Re: [PR] Add `get_permitted_menu_items` in auth manager [airflow]

2024-01-18 Thread via GitHub
potiuk merged PR #36871: URL: https://github.com/apache/airflow/pull/36871 -- 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:

Re: [PR] Secret masker ignores passwords with special chars [airflow]

2024-01-18 Thread via GitHub
amoghrajesh commented on PR #36692: URL: https://github.com/apache/airflow/pull/36692#issuecomment-1899915510 @aritra24 I personally think this should be the case by design, aligning with the postgres documentation too. Check my comment:

Re: [I] Airflow does not mask secrets in Database URI's when secret has a special character. [airflow]

2024-01-18 Thread via GitHub
amoghrajesh commented on issue #36688: URL: https://github.com/apache/airflow/issues/36688#issuecomment-1899914323 I think it is by design that we have to percent encode the passwords containing special characters. The postgres documentation also has a mention of it:

Re: [PR] [POC] Standardize deprecations in Airflow [airflow]

2024-01-18 Thread via GitHub
kacpermuda commented on code in PR #36876: URL: https://github.com/apache/airflow/pull/36876#discussion_r1458507297 ## airflow/deprecation.py: ## Review Comment: The idea i wanted to discuss is if we should move all deprecation related Warnings and decorators into a

Re: [PR] [POC] Standardize deprecations in Airflow [airflow]

2024-01-18 Thread via GitHub
kacpermuda commented on code in PR #36876: URL: https://github.com/apache/airflow/pull/36876#discussion_r1458504663 ## airflow/providers/google/cloud/links/dataproc.py: ## @@ -71,13 +70,16 @@ def __getattr__(name: str) -> Any: ) -@attr.s(auto_attribs=True) +@deprecated( +

[I] Support for configmaps (as env var & volume) in the template fields in the KubernetesPodOperator [airflow]

2024-01-18 Thread via GitHub
vizeit opened a new issue, #36888: URL: https://github.com/apache/airflow/issues/36888 ### Description Currently configmaps are not supported in the template fields in the KubernetesPodOperator. It will be helpful to add this support to dynamically attach configmaps to a pod

Re: [I] Inconsistent run_ids on UI vs API creating dagRuns [airflow]

2024-01-18 Thread via GitHub
abhishekbhakat closed issue #36867: Inconsistent run_ids on UI vs API creating dagRuns URL: https://github.com/apache/airflow/issues/36867 -- 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

[I] Support for SLA in Mapped Tasks [airflow]

2024-01-18 Thread via GitHub
amolsr opened a new issue, #36887: URL: https://github.com/apache/airflow/issues/36887 ### Description ![image](https://github.com/apache/airflow/assets/38129418/fce3048d-6d1d-497e-a114-9f5d6da20189) ### Use case/motivation We need to use the SLA with the mapped

Re: [PR] draft for map_index_template to overwrite map_index in the UI [airflow]

2024-01-18 Thread via GitHub
uranusjr commented on code in PR #36797: URL: https://github.com/apache/airflow/pull/36797#discussion_r1458454855 ## airflow/models/taskinstance.py: ## @@ -2331,6 +2336,10 @@ def _run_raw_task( self.task = self.task.prepare_for_execution() context =

Re: [PR] Fix WasbPrefixSensor arg inconsistency between sync and async mode [airflow]

2024-01-18 Thread via GitHub
pankajkoti commented on code in PR #36806: URL: https://github.com/apache/airflow/pull/36806#discussion_r1458439047 ## airflow/providers/microsoft/azure/triggers/wasb.py: ## @@ -102,26 +102,28 @@ class WasbPrefixSensorTrigger(BaseTrigger): ``copy``, ``deleted``

[I] Airflow DagProcessor raise AttributeError: module 'multiprocessing' has no attribute 'connection' if dag folder is empty [airflow]

2024-01-18 Thread via GitHub
nicnguyen3103 opened a new issue, #36885: URL: https://github.com/apache/airflow/issues/36885 ### Apache Airflow version 2.8.0 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? Using PostgresSQL backend, and an empty

Re: [I] New forced "Graph" view redirection breaks links from logs dag_run list table. [airflow]

2024-01-18 Thread via GitHub
wpromatt commented on issue #36884: URL: https://github.com/apache/airflow/issues/36884#issuecomment-1899797878 Mentioned the same on the PR introducing the forced redirect. https://github.com/apache/airflow/pull/32958 -- This is an automated message from the Apache Git Service. To

[I] New forced "Graph" view redirection breaks links from logs dag_run list table. [airflow]

2024-01-18 Thread via GitHub
wpromatt opened a new issue, #36884: URL: https://github.com/apache/airflow/issues/36884 ### Apache Airflow version Other Airflow 2 version (please specify below) ### If "Other Airflow 2 version" selected, which one? 2.7.3 ### What happened? It seems this

Re: [PR] Remove old graph [airflow]

2024-01-18 Thread via GitHub
wpromatt commented on PR #32958: URL: https://github.com/apache/airflow/pull/32958#issuecomment-1899760156 This is also true when clicking on dag_run from the list dag_run tables. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[I] TypeError: SqlAlchemySessionInterface.__init__() missing 6 required positional arguments [airflow]

2024-01-18 Thread via GitHub
habibutsu opened a new issue, #36883: URL: https://github.com/apache/airflow/issues/36883 ### Apache Airflow version 2.8.0 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? airflow is not started ### What you

Re: [PR] Remove old graph [airflow]

2024-01-18 Thread via GitHub
wpromatt commented on PR #32958: URL: https://github.com/apache/airflow/pull/32958#issuecomment-1899727667 @potiuk The only thing I can't figure out with this new view is how to go to the "graph" view from a link to the task logs. The redirection doesn't work when the dag run you're

Re: [PR] fix for role and permission count in airflow export cli command [airflow]

2024-01-18 Thread via GitHub
Bowrna commented on code in PR #36589: URL: https://github.com/apache/airflow/pull/36589#discussion_r1458255173 ## airflow/providers/fab/auth_manager/cli_commands/role_command.py: ## @@ -176,7 +176,8 @@ def roles_export(args): kwargs = {} if not args.pretty else

[PR] The task is stuck in a queued state forever in case of pod launch errors [airflow]

2024-01-18 Thread via GitHub
dirrao opened a new pull request, #36882: URL: https://github.com/apache/airflow/pull/36882 What happened When the K8 executor is unable to launch the worker pod due to permissions issues or an invalid namespace. The K8 executor keep trying to launch the worker pod and the errors

Re: [PR] avoid retrying after KubernetesPodOperator has been marked as failed [airflow]

2024-01-18 Thread via GitHub
Lee-W commented on code in PR #36749: URL: https://github.com/apache/airflow/pull/36749#discussion_r1458229220 ## airflow/providers/cncf/kubernetes/operators/pod.py: ## @@ -573,10 +575,17 @@ def execute_sync(self, context: Context): self.pod, istio_enabled,

Re: [PR] avoid retrying after KubernetesPodOperator has been marked as failed [airflow]

2024-01-18 Thread via GitHub
jedcunningham commented on code in PR #36749: URL: https://github.com/apache/airflow/pull/36749#discussion_r1458206012 ## airflow/providers/cncf/kubernetes/operators/pod.py: ## @@ -670,10 +678,15 @@ def write_logs(self, pod: k8s.V1Pod): def post_complete_action(self, *,

Re: [PR] avoid retrying after KubernetesPodOperator has been marked as failed [airflow]

2024-01-18 Thread via GitHub
jedcunningham commented on code in PR #36749: URL: https://github.com/apache/airflow/pull/36749#discussion_r1458204102 ## airflow/providers/cncf/kubernetes/operators/pod.py: ## @@ -573,10 +575,17 @@ def execute_sync(self, context: Context): self.pod,

Re: [PR] add impersonation_chain support when calling Bigquery Operators in deferrable mode [airflow]

2024-01-18 Thread via GitHub
Lee-W commented on PR #36341: URL: https://github.com/apache/airflow/pull/36341#issuecomment-1899527739 Hi @m1racoli, thanks for your feedback! I just took a look at your new PRs. Both of them look good. I think we might no longer need this PR, but I'll mark it as draft for now before we

Re: [PR] feat: full support for google credentials in gcloud-aio clients [airflow]

2024-01-18 Thread via GitHub
Lee-W commented on code in PR #36849: URL: https://github.com/apache/airflow/pull/36849#discussion_r1458183618 ## airflow/providers/google/cloud/hooks/bigquery.py: ## @@ -3513,11 +3518,11 @@ async def get_table_client( access to the specified project.

Re: [I] ECS Executor - Overriding Additional ECS Task Properties [airflow]

2024-01-18 Thread via GitHub
shubham22 commented on issue #35490: URL: https://github.com/apache/airflow/issues/35490#issuecomment-1899437696 @syedahsn - confused about which approach you're calling "best" here. The main difference between your config and my suggestion is the merge and replace of _list_ configs that

Re: [I] Kubernetes doubling logs [airflow]

2024-01-18 Thread via GitHub
github-actions[bot] commented on issue #35019: URL: https://github.com/apache/airflow/issues/35019#issuecomment-1899420021 This issue has been automatically marked as stale because it has been open for 14 days with no response from the author. It will be closed in next 7 days if no further

Re: [PR] Fix WasbPrefixSensor arg inconsistency between sync and async mode [airflow]

2024-01-18 Thread via GitHub
Lee-W commented on code in PR #36806: URL: https://github.com/apache/airflow/pull/36806#discussion_r1458115709 ## airflow/providers/microsoft/azure/triggers/wasb.py: ## @@ -102,26 +102,28 @@ class WasbPrefixSensorTrigger(BaseTrigger): ``copy``, ``deleted``

Re: [PR] add deferrable mode to RedshiftDataOperator [airflow]

2024-01-18 Thread via GitHub
Lee-W commented on code in PR #36586: URL: https://github.com/apache/airflow/pull/36586#discussion_r1458113287 ## airflow/providers/amazon/aws/hooks/redshift_data.py: ## @@ -201,3 +216,54 @@ def get_table_primary_key( break return pk_columns or None

Re: [I] SECURITY_MANAGER_CLASS not being picked up in webserver_config.py [airflow]

2024-01-18 Thread via GitHub
o-nikolas commented on issue #36875: URL: https://github.com/apache/airflow/issues/36875#issuecomment-1899405607 CC @vincbeck -- 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

Re: [I] apache-airflow-providers-mongo does not disable SSL/TLS correctly [airflow]

2024-01-18 Thread via GitHub
boring-cyborg[bot] commented on issue #36881: URL: https://github.com/apache/airflow/issues/36881#issuecomment-1899402572 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

[I] apache-airflow-providers-mongo does not disable SSL/TLS correctly [airflow]

2024-01-18 Thread via GitHub
NoahStapp opened a new issue, #36881: URL: https://github.com/apache/airflow/issues/36881 ### Apache Airflow Provider(s) mongo ### Versions of Apache Airflow Providers apache-airflow-providers-mongo version 3.4.0 ### Apache Airflow version 2 ###

Re: [I] ECS Executor - Overriding Additional ECS Task Properties [airflow]

2024-01-18 Thread via GitHub
syedahsn commented on issue #35490: URL: https://github.com/apache/airflow/issues/35490#issuecomment-1899379318 Coming a bit late to the conversation, but I just want to make sure I'm following correctly. Is it true that from @shubham22 's example, the "best" final config would look

Re: [I] depends_on_past not working [airflow]

2024-01-18 Thread via GitHub
RNHTTR closed issue #36857: depends_on_past not working URL: https://github.com/apache/airflow/issues/36857 -- 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,

Re: [I] depends_on_past not working [airflow]

2024-01-18 Thread via GitHub
RNHTTR commented on issue #36857: URL: https://github.com/apache/airflow/issues/36857#issuecomment-1899365806 The `random_fail` task doesn't have `depends_on_past=True`, so new DAG runs are able to continue. If you update the `random_fail` task as follows, it will work as expected:

Re: [PR] Add `get_permitted_menu_items` in auth manager [airflow]

2024-01-18 Thread via GitHub
o-nikolas commented on PR #36871: URL: https://github.com/apache/airflow/pull/36871#issuecomment-1899360697 Are you planning to add the override of that method for the amazon auth provider in a followup PR? -- This is an automated message from the Apache Git Service. To respond to the

(airflow) branch main updated: Fix bug introduced by replacing spaces by + in run_id (#36877)

2024-01-18 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new ec8fab5d2a Fix bug introduced by

Re: [PR] Fix bug introduced by replacing spaces by + in run_id [airflow]

2024-01-18 Thread via GitHub
ephraimbuddy merged PR #36877: URL: https://github.com/apache/airflow/pull/36877 -- 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:

Re: [I] Marking success or failled of a single or group of tasks effects other tasks under different execution dates [airflow]

2024-01-18 Thread via GitHub
boring-cyborg[bot] commented on issue #36879: URL: https://github.com/apache/airflow/issues/36879#issuecomment-1899209034 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

[I] Marking success or failled of a single or group of tasks effects other tasks under different execution dates [airflow]

2024-01-18 Thread via GitHub
rorymcstay opened a new issue, #36879: URL: https://github.com/apache/airflow/issues/36879 ### Apache Airflow version Other Airflow 2 version (please specify below) ### If "Other Airflow 2 version" selected, which one? 2.3.3 ### What happened? When marking

[PR] Run Trigger Page: Configurable number of recent configs [airflow]

2024-01-18 Thread via GitHub
AchimGaedkeLynker opened a new pull request, #36878: URL: https://github.com/apache/airflow/pull/36878 Some users are heavily relying on running DAGs manually by altering or repeating previous configurations. The number (more precisely limit) of previous runs sent to the

(airflow) branch constraints-main updated: Updating constraints. Github run id:7574996133

2024-01-18 Thread github-bot
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 04527b1382 Updating

Re: [PR] Implement task duration page in react. [airflow]

2024-01-18 Thread via GitHub
bbovenzi commented on PR #35863: URL: https://github.com/apache/airflow/pull/35863#issuecomment-1899128549 I fixed the issue if there are no TIs and some breadcrumb header navigation. We can leave the average values for a follow up PR -- This is an automated message from the Apache Git

Re: [PR] [POC] Standardize deprecations in Airflow [airflow]

2024-01-18 Thread via GitHub
Taragolis commented on code in PR #36876: URL: https://github.com/apache/airflow/pull/36876#discussion_r1457894304 ## airflow/providers/google/cloud/links/dataproc.py: ## @@ -71,13 +70,16 @@ def __getattr__(name: str) -> Any: ) -@attr.s(auto_attribs=True) +@deprecated( +

(airflow) branch main updated: EC2 `CreateInstance`: terminate instances in on_kill (#36828)

2024-01-18 Thread vincbeck
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 d18c01a7a2 EC2 `CreateInstance`: terminate

Re: [PR] EC2 CreateInstance: terminate instances in on_kill [airflow]

2024-01-18 Thread via GitHub
vincbeck merged PR #36828: URL: https://github.com/apache/airflow/pull/36828 -- 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:

[PR] [POC] Standardize deprecations in Airflow [airflow]

2024-01-18 Thread via GitHub
kacpermuda opened a new pull request, #36876: URL: https://github.com/apache/airflow/pull/36876 ## TLDR; I propose standardizing our approach to deprecating elements in the Airflow codebase, aiming for improved readability and easier automated parsing. This PR serves as a

Re: [PR] EC2 CreateInstance: terminate instances in on_kill [airflow]

2024-01-18 Thread via GitHub
syedahsn commented on code in PR #36828: URL: https://github.com/apache/airflow/pull/36828#discussion_r1457843632 ## airflow/providers/amazon/aws/operators/ec2.py: ## @@ -183,22 +183,36 @@ def execute(self, context: Context): MaxCount=self.max_count,

(airflow) branch main updated: add deferrable mode to RedshiftDataOperator (#36586)

2024-01-18 Thread eladkal
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 9563dc573b add deferrable mode to

Re: [PR] add deferrable mode to RedshiftDataOperator [airflow]

2024-01-18 Thread via GitHub
eladkal merged PR #36586: URL: https://github.com/apache/airflow/pull/36586 -- 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:

Re: [PR] add deferrable mode to RedshiftDataOperator [airflow]

2024-01-18 Thread via GitHub
syedahsn commented on code in PR #36586: URL: https://github.com/apache/airflow/pull/36586#discussion_r1457834082 ## airflow/providers/amazon/aws/hooks/redshift_data.py: ## @@ -201,3 +216,54 @@ def get_table_primary_key( break return pk_columns or

Re: [I] SECURITY_MANAGER_CLASS not being picked up in webserver_config.py [airflow]

2024-01-18 Thread via GitHub
boring-cyborg[bot] commented on issue #36875: URL: https://github.com/apache/airflow/issues/36875#issuecomment-1898994078 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

[I] SECURITY_MANAGER_CLASS not being picked up in webserver_config.py [airflow]

2024-01-18 Thread via GitHub
CharlieJ15420 opened a new issue, #36875: URL: https://github.com/apache/airflow/issues/36875 ### Apache Airflow version 2.8.0 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? I've just upgraded from 2.7.3 to 2.8.0 and

Re: [I] Status of testing of Apache Airflow 2.8.1rc1 [airflow]

2024-01-18 Thread via GitHub
vchiapaikeo commented on issue #36808: URL: https://github.com/apache/airflow/issues/36808#issuecomment-1898965585 https://github.com/apache/airflow/pull/35536 and https://github.com/apache/airflow/pull/36753 look good -- This is an automated message from the Apache Git Service. To

Re: [I] airflow db init -> Failed to open the referenced table 'dataset' using MySQL [airflow]

2024-01-18 Thread via GitHub
Taragolis closed issue #36866: airflow db init -> Failed to open the referenced table 'dataset' using MySQL URL: https://github.com/apache/airflow/issues/36866 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[PR] Test [airflow]

2024-01-18 Thread via GitHub
ibarrionuevo opened a new pull request, #36873: URL: https://github.com/apache/airflow/pull/36873 This is a test pull request greated for CI/CD vulnerability testing purposes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Test [airflow]

2024-01-18 Thread via GitHub
ibarrionuevo closed pull request #36873: Test URL: https://github.com/apache/airflow/pull/36873 -- 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:

Re: [PR] Test [airflow]

2024-01-18 Thread via GitHub
boring-cyborg[bot] commented on PR #36873: URL: https://github.com/apache/airflow/pull/36873#issuecomment-1898925439 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

Re: [I] Cannot pass data scan rules yaml file in DataplexCreateOrUpdateDataQualityScanOperator [airflow]

2024-01-18 Thread via GitHub
Taragolis closed issue #36860: Cannot pass data scan rules yaml file in DataplexCreateOrUpdateDataQualityScanOperator URL: https://github.com/apache/airflow/issues/36860 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
Taragolis merged PR #36868: URL: https://github.com/apache/airflow/pull/36868 -- 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:

(airflow) branch main updated: Adding information about AliFone company's use of Airflow (#36868)

2024-01-18 Thread taragolis
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 938f0c1f3c Adding information about AliFone

[PR] Add `get_permitted_menu_items` in auth manager [airflow]

2024-01-18 Thread via GitHub
vincbeck opened a new pull request, #36871: URL: https://github.com/apache/airflow/pull/36871 Currently the menu in Airflow UI is using the filter [is_menu_visible](https://github.com/dpgaspar/Flask-AppBuilder/blob/master/flask_appbuilder/filters.py#L133) to display or not each menu item.

(airflow-site) branch gh-pages updated (02127b6501 -> 36af463c92)

2024-01-18 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch gh-pages in repository https://gitbox.apache.org/repos/asf/airflow-site.git discard 02127b6501 Rewritten history to remove past gh-pages deployments new 36af463c92 Rewritten history to

(airflow) branch main updated: Update the release guide (#36863)

2024-01-18 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 6ad479aa0f Update the release guide

Re: [PR] Update the release guide [airflow]

2024-01-18 Thread via GitHub
ephraimbuddy merged PR #36863: URL: https://github.com/apache/airflow/pull/36863 -- 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:

(airflow) branch main updated: More complete FAQ and context about releasing next patchlevel release (#36858)

2024-01-18 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new b994a65c0c More complete FAQ and context

Re: [PR] More complete FAQ and context about releasing next patchlevel release [airflow]

2024-01-18 Thread via GitHub
ephraimbuddy merged PR #36858: URL: https://github.com/apache/airflow/pull/36858 -- 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:

Re: [PR] Fix documentation on IAM permissions for EcsRunTaskOperator [airflow]

2024-01-18 Thread via GitHub
vincbeck merged PR #36865: URL: https://github.com/apache/airflow/pull/36865 -- 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:

Re: [PR] Fix documentation on IAM permissions for EcsRunTaskOperator [airflow]

2024-01-18 Thread via GitHub
boring-cyborg[bot] commented on PR #36865: URL: https://github.com/apache/airflow/pull/36865#issuecomment-1898791309 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

(airflow) branch main updated: Fix documentation on IAM permissions for `EcsRunTaskOperator` (#36865)

2024-01-18 Thread vincbeck
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 2d330cb487 Fix documentation on IAM permissions

Re: [I] Incorrect IAM permissions in EcsRunTaskOperator docs [airflow]

2024-01-18 Thread via GitHub
vincbeck closed issue #36864: Incorrect IAM permissions in EcsRunTaskOperator docs URL: https://github.com/apache/airflow/issues/36864 -- 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

Re: [PR] More complete FAQ and context about releasing next patchlevel release [airflow]

2024-01-18 Thread via GitHub
ephraimbuddy commented on code in PR #36858: URL: https://github.com/apache/airflow/pull/36858#discussion_r1457679181 ## dev/WHAT_GOES_INTO_THE_NEXT_RELEASE.md: ## @@ -0,0 +1,131 @@ + + + + +**Table of Contents** *generated with [DocToc](https://github.com/thlorenz/doctoc)* +

Re: [PR] Attempt to update Connexion library to version 3 [airflow]

2024-01-18 Thread via GitHub
vincbeck commented on PR #36052: URL: https://github.com/apache/airflow/pull/36052#issuecomment-1898788598 You will also have to update the method in `AwsAuthManager`, but that should be straightforward -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Attempt to update Connexion library to version 3 [airflow]

2024-01-18 Thread via GitHub
vincbeck commented on PR #36052: URL: https://github.com/apache/airflow/pull/36052#issuecomment-1898786641 > > > Update init_api_auth_provider function. > > > In get_api_endpoints function we use connexion_app.add_api and expect FlaskApi as a return object, but add_api function always

Re: [PR] heartbeat recovery message [airflow]

2024-01-18 Thread via GitHub
Bowrna commented on code in PR #34457: URL: https://github.com/apache/airflow/pull/34457#discussion_r1457670440 ## airflow/jobs/job.py: ## @@ -201,12 +215,15 @@ def heartbeat( job = Job._update_heartbeat(job=self, session=session)

(airflow-site) branch main updated: Fixing the link for privacy notice (#937)

2024-01-18 Thread potiuk
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 da1a333878 Fixing the link for privacy notice

Re: [PR] Fixing the link for privacy notice [airflow-site]

2024-01-18 Thread via GitHub
potiuk merged PR #937: URL: https://github.com/apache/airflow-site/pull/937 -- 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:

Re: [PR] More complete FAQ and context about releasing next patchlevel release [airflow]

2024-01-18 Thread via GitHub
potiuk commented on code in PR #36858: URL: https://github.com/apache/airflow/pull/36858#discussion_r1457644415 ## dev/WHAT_GOES_INTO_THE_NEXT_RELEASE.md: ## @@ -0,0 +1,131 @@ + + + + +**Table of Contents** *generated with [DocToc](https://github.com/thlorenz/doctoc)* + +-

Re: [I] CSRF token to expire with session when config file exists [airflow]

2024-01-18 Thread via GitHub
boring-cyborg[bot] commented on issue #36869: URL: https://github.com/apache/airflow/issues/36869#issuecomment-1898708011 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

[I] CSRF token to expire with session when config file exists [airflow]

2024-01-18 Thread via GitHub
NickiForte opened a new issue, #36869: URL: https://github.com/apache/airflow/issues/36869 ### Apache Airflow version 2.8.0 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? Despite

Re: [PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
pankajastro commented on code in PR #36868: URL: https://github.com/apache/airflow/pull/36868#discussion_r1457605300 ## INTHEWILD.md: ## @@ -522,4 +523,4 @@ Currently, **officially** using Airflow: 1. [Zymergen](https://www.zymergen.com/) 1. [Zynga](https://www.zynga.com) 1.

Re: [PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
pankajastro commented on code in PR #36868: URL: https://github.com/apache/airflow/pull/36868#discussion_r1457601230 ## INTHEWILD.md: ## @@ -522,4 +523,4 @@ Currently, **officially** using Airflow: 1. [Zymergen](https://www.zymergen.com/) 1. [Zynga](https://www.zynga.com) 1.

Re: [PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
pankajastro commented on code in PR #36868: URL: https://github.com/apache/airflow/pull/36868#discussion_r1457601230 ## INTHEWILD.md: ## @@ -522,4 +523,4 @@ Currently, **officially** using Airflow: 1. [Zymergen](https://www.zymergen.com/) 1. [Zynga](https://www.zynga.com) 1.

Re: [PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
oregisam commented on code in PR #36868: URL: https://github.com/apache/airflow/pull/36868#discussion_r1457577921 ## INTHEWILD.md: ## @@ -523,3 +523,4 @@ Currently, **officially** using Airflow: 1. [Zynga](https://www.zynga.com) 1. [Ørsted](https://orsted.com/en)

Re: [I] Airflow tasks test is broken for xcom_pull based tasks [airflow]

2024-01-18 Thread via GitHub
jccalvojackson commented on issue #24232: URL: https://github.com/apache/airflow/issues/24232#issuecomment-1898645631 how one would go about running a single task that depends on an xcom, even if not using `airflow tasks test`? to use `airflow tasks run` it needs to be associated to a

Re: [PR] Attempt to update Connexion library to version 3 [airflow]

2024-01-18 Thread via GitHub
VladaZakharova commented on PR #36052: URL: https://github.com/apache/airflow/pull/36052#issuecomment-1898634172 @RobbeSneyders , hope you can help :) ``` TypeError: get() got an unexpected keyword argument 'environ_overrides' this error is the most frequent now. For fixing it we need

Re: [PR] Index optimized fast path to avoid slower groupby queryplan [airflow]

2024-01-18 Thread via GitHub
joshowen commented on PR #33242: URL: https://github.com/apache/airflow/pull/33242#issuecomment-1898635315 > hi @joshowen i moved the query builder to a function and added some testing for the query structure. please take a look. i changed the structure of the query slightly so that we

Re: [PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
pankajastro commented on code in PR #36868: URL: https://github.com/apache/airflow/pull/36868#discussion_r1457549241 ## INTHEWILD.md: ## @@ -523,3 +523,4 @@ Currently, **officially** using Airflow: 1. [Zynga](https://www.zynga.com) 1. [Ørsted](https://orsted.com/en)

Re: [PR] Attempt to update Connexion library to version 3 [airflow]

2024-01-18 Thread via GitHub
VladaZakharova commented on PR #36052: URL: https://github.com/apache/airflow/pull/36052#issuecomment-1898612794 > > Update init_api_auth_provider function. > > In get_api_endpoints function we use connexion_app.add_api and expect FlaskApi as a return object, but add_api function always

Re: [PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
boring-cyborg[bot] commented on PR #36868: URL: https://github.com/apache/airflow/pull/36868#issuecomment-1898608184 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

[PR] Adding information about AliFone company's use of Airflow [airflow]

2024-01-18 Thread via GitHub
oregisam opened a new pull request, #36868: URL: https://github.com/apache/airflow/pull/36868 --- **^ Add meaningful description above** Read the **[Pull Request

Re: [I] Inconsistent run_ids on UI vs API creating dagRuns [airflow]

2024-01-18 Thread via GitHub
abhishekbhakat commented on issue #36867: URL: https://github.com/apache/airflow/issues/36867#issuecomment-1898597510 UI calls go to this function: https://github.com/apache/airflow/blob/681859c7bffabce0c294060d811db2fb16851816/airflow/www/views.py#L1941-L1951 Whereas API goes to:

Re: [PR] Fix WasbPrefixSensor arg inconsistency between sync and async mode [airflow]

2024-01-18 Thread via GitHub
pankajkoti commented on code in PR #36806: URL: https://github.com/apache/airflow/pull/36806#discussion_r1457527454 ## airflow/providers/microsoft/azure/triggers/wasb.py: ## @@ -102,26 +102,28 @@ class WasbPrefixSensorTrigger(BaseTrigger): ``copy``, ``deleted``

(airflow) branch main updated: Change default `parquet_row_group_size` in `BaseSQLToGCSOperator` (#36817)

2024-01-18 Thread eladkal
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 681859c7bf Change default

Re: [PR] Change default `parquet_row_group_size` in `BaseSQLToGCSOperator` [airflow]

2024-01-18 Thread via GitHub
eladkal merged PR #36817: URL: https://github.com/apache/airflow/pull/36817 -- 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:

Re: [I] Change SQL to GCS operators default row group size when output is Parquet [airflow]

2024-01-18 Thread via GitHub
eladkal closed issue #36793: Change SQL to GCS operators default row group size when output is Parquet URL: https://github.com/apache/airflow/issues/36793 -- 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

[I] Inconsistent run_ids on UI vs API creating dagRuns [airflow]

2024-01-18 Thread via GitHub
abhishekbhakat opened a new issue, #36867: URL: https://github.com/apache/airflow/issues/36867 ### Apache Airflow version main (development) ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? As part of this

(airflow) branch main updated: Optimize `SnowflakeSqlApiOperator` execution in deferrable mode (#36850)

2024-01-18 Thread phanikumv
This is an automated email from the ASF dual-hosted git repository. phanikumv 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 27dc7e80df Optimize `SnowflakeSqlApiOperator`

Re: [PR] check query status before SnowflakeSqlApiOperator execute in deferrable mode [airflow]

2024-01-18 Thread via GitHub
phanikumv merged PR #36850: URL: https://github.com/apache/airflow/pull/36850 -- 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   2   >