Re: [PR] Initial commit [airflow]

2023-10-04 Thread via GitHub
arjunanan6 commented on PR #34766: URL: https://github.com/apache/airflow/pull/34766#issuecomment-1748068205 @hussein-awala I didn't even know you could do a draft, so thanks for pointing that out. My final version will have a better title, details, etc. @Taragolis It is kind-of

Re: [PR] Add config to control Kubernetes Client retry behaviour [airflow]

2023-10-04 Thread via GitHub
hterik commented on PR #26710: URL: https://github.com/apache/airflow/pull/26710#issuecomment-1748064603 > Hi, this is a problem for us in production still, can we make forward progress here? It was fixed in https://github.com/apache/airflow/pull/29809 instead -- This is an

Re: [I] Links to specific dagruns don't work when older than 25 runs [airflow]

2023-10-04 Thread via GitHub
hterik commented on issue #34723: URL: https://github.com/apache/airflow/issues/34723#issuecomment-1748051842 I've done some debugging: **Problem 1.** Dag list is linking to the legacy `/graph` endpoint. This endpoint is only for backwards compatibility purposes and redirects to

Re: [PR] Validate DAG task_instance.priority_weight column overflow value during import [airflow]

2023-10-04 Thread via GitHub
tiranux commented on code in PR #34168: URL: https://github.com/apache/airflow/pull/34168#discussion_r1346793137 ## airflow/utils/dag_parameters_overflow.py: ## @@ -0,0 +1,56 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

Re: [PR] Added command for installing specific pipx packaging version [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on PR #34772: URL: https://github.com/apache/airflow/pull/34772#issuecomment-1748031161 pipx is going to make a release with a fix to this issue so this won’t be a problem soon. I don’t think it’s a good idea to fix users on an old packaging version. -- This is an

Re: [PR] Added command for installing specific pipx packaging version [airflow]

2023-10-04 Thread via GitHub
uranusjr closed pull request #34772: Added command for installing specific pipx packaging version URL: https://github.com/apache/airflow/pull/34772 -- 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

Re: [PR] Refactor Sqlalchemy queries to 2.0 style (Part 9) [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on PR #34763: URL: https://github.com/apache/airflow/pull/34763#issuecomment-1748028756 I added a commit to simplify the Pool query since it always only returns at most one item. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Refactor Sqlalchemy queries to 2.0 style (Part 9) [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34763: URL: https://github.com/apache/airflow/pull/34763#discussion_r1346773871 ## airflow/ti_deps/deps/pool_slots_available_dep.py: ## @@ -41,8 +43,7 @@ def _get_dep_statuses(self, ti, session, dep_context=None): from

[PR] Fix AWS RDS hook's DB instance state check [airflow]

2023-10-04 Thread via GitHub
AetherUnbound opened a new pull request, #34773: URL: https://github.com/apache/airflow/pull/34773 ## Problem We observed that when the `RdsDbSensor` is run against a database identifier which doesn't yet exist, the sensor fails and enters a retry sequence rather

Re: [PR] Added command for installing specific pipx packaging version [airflow]

2023-10-04 Thread via GitHub
boring-cyborg[bot] commented on PR #34772: URL: https://github.com/apache/airflow/pull/34772#issuecomment-1747981644 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] Added command for installing specific pipx packaging version [airflow]

2023-10-04 Thread via GitHub
theaadya opened a new pull request, #34772: URL: https://github.com/apache/airflow/pull/34772 I was setting up Airflow with Breeze when I encountered this error: `ModuleNotFoundError: No module named 'airflow_breeze.params'` Came across this PR

Re: [I] Missing log file when there's more than one attempt [airflow]

2023-10-04 Thread via GitHub
github-actions[bot] commented on issue #34393: URL: https://github.com/apache/airflow/issues/34393#issuecomment-1747821248 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] Open Search Provider [airflow]

2023-10-04 Thread via GitHub
cjames23 commented on code in PR #34705: URL: https://github.com/apache/airflow/pull/34705#discussion_r1346603646 ## airflow/providers/opensearch/hooks/opensearch.py: ## @@ -0,0 +1,119 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Open Search Provider [airflow]

2023-10-04 Thread via GitHub
cjames23 commented on code in PR #34705: URL: https://github.com/apache/airflow/pull/34705#discussion_r1346600199 ## docs/apache-airflow-providers-opensearch/connections/opensearch.rst: ## @@ -0,0 +1,37 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or

Re: [PR] Open Search Provider [airflow]

2023-10-04 Thread via GitHub
cjames23 commented on code in PR #34705: URL: https://github.com/apache/airflow/pull/34705#discussion_r134665 ## docs/apache-airflow-providers-opensearch/operators/opensearch.rst: ## @@ -0,0 +1,72 @@ + .. Licensed to the Apache Software Foundation (ASF) under one +or

Re: [PR] Open Search Provider [airflow]

2023-10-04 Thread via GitHub
cjames23 commented on code in PR #34705: URL: https://github.com/apache/airflow/pull/34705#discussion_r1346599556 ## airflow/providers/opensearch/operators/opensearch.py: ## @@ -0,0 +1,192 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] Open Search Provider [airflow]

2023-10-04 Thread via GitHub
cjames23 commented on code in PR #34705: URL: https://github.com/apache/airflow/pull/34705#discussion_r1346588765 ## airflow/providers/opensearch/hooks/opensearch.py: ## @@ -0,0 +1,119 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Let auth managers provide their own API endpoints [airflow]

2023-10-04 Thread via GitHub
jedcunningham commented on code in PR #34349: URL: https://github.com/apache/airflow/pull/34349#discussion_r1346584086 ## airflow/auth/managers/base_auth_manager.py: ## @@ -50,6 +50,10 @@ def get_cli_commands() -> list[CLICommand]: """ return [] +def

Re: [PR] Fix metadata port in Keda connection when usePgbouncer is false [airflow]

2023-10-04 Thread via GitHub
brokenjacobs commented on code in PR #34741: URL: https://github.com/apache/airflow/pull/34741#discussion_r1346571001 ## chart/templates/secrets/metadata-connection-secret.yaml: ## @@ -27,7 +27,8 @@ {{- $host := ternary $pgbouncerHost $metadataHost .Values.pgbouncer.enabled }}

Re: [PR] Fix metadata port in Keda connection when usePgbouncer is false [airflow]

2023-10-04 Thread via GitHub
brokenjacobs commented on code in PR #34741: URL: https://github.com/apache/airflow/pull/34741#discussion_r1346570507 ## chart/templates/secrets/metadata-connection-secret.yaml: ## @@ -48,7 +49,7 @@ data: {{- end }} {{- if and .Values.workers.keda.enabled

Re: [PR] Suppress false warning when TI state is QUEUED and TI doesn't have a start_date [airflow]

2023-10-04 Thread via GitHub
kzosabe commented on PR #34771: URL: https://github.com/apache/airflow/pull/34771#issuecomment-1747760505 About this advice in the slack conversation: > we used it also in the backfill job, you need to do the same thing. As the description of the previous PR, there are two

Re: [PR] Suppress false warning when TI state is QUEUED and TI doesn't have a start_date [airflow]

2023-10-04 Thread via GitHub
boring-cyborg[bot] commented on PR #34771: URL: https://github.com/apache/airflow/pull/34771#issuecomment-1747745695 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] Suppress false warning when TI state is QUEUED and TI doesn't have a start_date [airflow]

2023-10-04 Thread via GitHub
kzosabe opened a new pull request, #34771: URL: https://github.com/apache/airflow/pull/34771 closes: https://github.com/apache/airflow/issues/34493 conversations in slack: https://apache-airflow.slack.com/archives/CCPRP7943/p1696235301762429 draft PR for the

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346535068 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346534457 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346533744 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34507: URL: https://github.com/apache/airflow/pull/34507#discussion_r1346533744 ## dev/breeze/src/airflow_breeze/utils/publish_docs_helpers.py: ## @@ -112,7 +112,7 @@ def process_package_filters( return available_packages

[airflow] branch dependabot/npm_and_yarn/airflow/www/postcss-8.4.31 created (now 42178dd2b0)

2023-10-04 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/airflow/www/postcss-8.4.31 in repository https://gitbox.apache.org/repos/asf/airflow.git at 42178dd2b0 Bump postcss from 8.4.25 to 8.4.31 in /airflow/www No

[PR] Bump postcss from 8.4.25 to 8.4.31 in /airflow/www [airflow]

2023-10-04 Thread via GitHub
dependabot[bot] opened a new pull request, #34770: URL: https://github.com/apache/airflow/pull/34770 Bumps [postcss](https://github.com/postcss/postcss) from 8.4.25 to 8.4.31. Release notes Sourced from https://github.com/postcss/postcss/releases;>postcss's releases. 8.4.31

[airflow] branch main updated: Bump min airflow version of providers (#34728)

2023-10-04 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr 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 0c8e30e43b Bump min airflow version of providers

Re: [PR] Bump min airflow version of providers [airflow]

2023-10-04 Thread via GitHub
uranusjr merged PR #34728: URL: https://github.com/apache/airflow/pull/34728 -- 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] Update define-extra-link.rst [airflow]

2023-10-04 Thread via GitHub
boring-cyborg[bot] commented on PR #34547: URL: https://github.com/apache/airflow/pull/34547#issuecomment-1747719981 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 imports in extra link documentation (#34547)

2023-10-04 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr 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 8d6acce843 Fix imports in extra link

Re: [PR] Update define-extra-link.rst [airflow]

2023-10-04 Thread via GitHub
uranusjr merged PR #34547: URL: https://github.com/apache/airflow/pull/34547 -- 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] Expose description columns of Pools in the UI [airflow]

2023-10-04 Thread via GitHub
FarisKarim commented on issue #34751: URL: https://github.com/apache/airflow/issues/34751#issuecomment-1747702837 hey i can work on this -- 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

Re: [PR] Deferrable Operators Docs Edits [airflow]

2023-10-04 Thread via GitHub
lzdanski commented on PR #33620: URL: https://github.com/apache/airflow/pull/33620#issuecomment-174766 @eladkal - yes! Thanks for checking in - I've got time set aside to get back to it later this week/weekend, shouldn't have much more to go. -- This is an automated message from the

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
carterjfulcher commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1747635703 > @carterjfulcher can you look into the failing static check? Yes, will do -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Fixes typing in `get_provider_name_from_short_hand` and its uses [airflow]

2023-10-04 Thread via GitHub
eladkal commented on PR #34507: URL: https://github.com/apache/airflow/pull/34507#issuecomment-1747630776 @carterjfulcher can you look into the failing static check? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] Update define-extra-link.rst [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34547: URL: https://github.com/apache/airflow/pull/34547#discussion_r1346451574 ## docs/apache-airflow/howto/define-extra-link.rst: ## @@ -118,7 +120,9 @@ Console, but if we wanted to change that link we could: .. code-block:: python

Re: [PR] Update define-extra-link.rst [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34547: URL: https://github.com/apache/airflow/pull/34547#discussion_r1346451574 ## docs/apache-airflow/howto/define-extra-link.rst: ## @@ -118,7 +120,9 @@ Console, but if we wanted to change that link we could: .. code-block:: python

Re: [PR] Deferrable Operators Docs Edits [airflow]

2023-10-04 Thread via GitHub
eladkal commented on PR #33620: URL: https://github.com/apache/airflow/pull/33620#issuecomment-1747625410 @lzdanski are you still working on this PR? -- 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

Re: [PR] Update define-extra-link.rst [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34547: URL: https://github.com/apache/airflow/pull/34547#discussion_r1346451378 ## docs/apache-airflow/howto/define-extra-link.rst: ## @@ -29,6 +29,7 @@ The following code shows how to add extra links to an operator via Plugins: from

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
cBiscuitSurprise commented on PR #34747: URL: https://github.com/apache/airflow/pull/34747#issuecomment-1747621329 @vincbeck @eladkal We're preserving the "legacy" behavior by default, so no new information will be logged. Customers can "opt-in" to the new behavior by setting the

Re: [PR] Add support for ZoneInfo and generic UTC [airflow]

2023-10-04 Thread via GitHub
uranusjr commented on code in PR #34683: URL: https://github.com/apache/airflow/pull/34683#discussion_r1346441237 ## setup.cfg: ## @@ -72,6 +72,7 @@ install_requires = argcomplete>=1.10 asgiref attrs>=22.1.0 +backports.zoneinfo>=0.2.1;python_version<"3.9"

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
vincbeck commented on PR #34747: URL: https://github.com/apache/airflow/pull/34747#issuecomment-1747611933 > cBiscuitSurprise Good point! @cBiscuitSurprise Could you check that and maybe some sample of logs before and after the change? From the code it is impossible to say -- This

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
eladkal commented on PR #34747: URL: https://github.com/apache/airflow/pull/34747#issuecomment-1747606404 qouting @o-nikolas comment from the issue >But note: This change will need a careful check of Airflow logs, since v3 of Watcher can now include much more information in the logs

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346432311 ## airflow/providers/amazon/aws/log/cloudwatch_task_handler.py: ## @@ -33,22 +33,33 @@ from airflow.models import TaskInstance Review Comment:

Re: [PR] Let auth managers provide their own API endpoints [airflow]

2023-10-04 Thread via GitHub
vincbeck commented on code in PR #34349: URL: https://github.com/apache/airflow/pull/34349#discussion_r1346427303 ## airflow/auth/managers/base_auth_manager.py: ## @@ -50,6 +50,10 @@ def get_cli_commands() -> list[CLICommand]: """ return [] +def

Re: [PR] Initial commit [airflow]

2023-10-04 Thread via GitHub
Taragolis commented on PR #34766: URL: https://github.com/apache/airflow/pull/34766#issuecomment-1747584331 If this changes are correct, than I guess it more related to - https://github.com/apache/airflow/issues/28790 -- This is an automated message from the Apache Git Service. To

Re: [PR] Initial commit [airflow]

2023-10-04 Thread via GitHub
Taragolis commented on PR #34766: URL: https://github.com/apache/airflow/pull/34766#issuecomment-1747581457 > related: https://github.com/apache/airflow/issues/19739 Are you sure that this correct issue? Issue related to

Re: [PR] Update params.rst [airflow]

2023-10-04 Thread via GitHub
hussein-awala commented on code in PR #34759: URL: https://github.com/apache/airflow/pull/34759#discussion_r1346408847 ## docs/apache-airflow/core-concepts/params.rst: ## @@ -124,8 +124,10 @@ Another way to access your param is via a task's ``context`` kwarg.

Re: [PR] Add `deferred_as_active` param to consider deferred state as an active state [airflow]

2023-10-04 Thread via GitHub
hussein-awala commented on PR #34700: URL: https://github.com/apache/airflow/pull/34700#issuecomment-1747548648 > I wonder if we should have separate parameters for active + deferred instead of having a flag to switch between behaviours for the same flags. Not sure. IMHO it will be

Re: [PR] Fix metadata port in Keda connection when usePgbouncer is false [airflow]

2023-10-04 Thread via GitHub
hussein-awala commented on PR #34741: URL: https://github.com/apache/airflow/pull/34741#issuecomment-174759 > It uses the db name defined in the pgbouncer.ini, not the upstream database name. Indeed, I fixed it and I checked all the other values, all looks good! -- This is an

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
cBiscuitSurprise commented on PR #34747: URL: https://github.com/apache/airflow/pull/34747#issuecomment-1747522652 Thank you everyone for your input. I've addressed your comments. --- P.S. Would someone be able to add the `hacktoberfest-accepted` label to this PR? -- This is

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
cBiscuitSurprise commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346370906 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +

Re: [PR] Check if the lower of provided values are sensitives in config endpoint [airflow]

2023-10-04 Thread via GitHub
hussein-awala commented on code in PR #34712: URL: https://github.com/apache/airflow/pull/34712#discussion_r1346370939 ## airflow/api_connexion/endpoints/config_endpoint.py: ## @@ -123,7 +123,7 @@ def get_value(section: str, option: str) -> Response: "Config

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
cBiscuitSurprise commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346370700 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
cBiscuitSurprise commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346370418 ## airflow/providers/amazon/aws/log/cloudwatch_task_handler.py: ## @@ -33,6 +33,24 @@ from airflow.models import TaskInstance +def

Re: [PR] Open Search Provider [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34705: URL: https://github.com/apache/airflow/pull/34705#discussion_r1346323410 ## airflow/providers/opensearch/hooks/opensearch.py: ## @@ -0,0 +1,119 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] feat: enable HTTPRoute for Gateway API [airflow]

2023-10-04 Thread via GitHub
jedcunningham commented on PR #34716: URL: https://github.com/apache/airflow/pull/34716#issuecomment-1747477431 > Are we manually adding values to `values.schema.json`? I've only worked with a `schema.yaml` that then generated the `json`. Yeah, done manually. Should be pretty easy to

[PR] Reset stats logger instance in daemon context to reinitialize in chil… [airflow]

2023-10-04 Thread via GitHub
pavansharma36 opened a new pull request, #34769: URL: https://github.com/apache/airflow/pull/34769 …d process --- related: #33897 --- Clear stats logger instance to reinitialize in child process. -- This is an automated message from the

[PR] Reset stats logger instance in daemon context to reinitialize in chil… [airflow]

2023-10-04 Thread via GitHub
pavansharma36 opened a new pull request, #34768: URL: https://github.com/apache/airflow/pull/34768 …d process --- Reset stats logger instance to reinitialize in child process **^ Add meaningful description above** Read the **[Pull Request

Re: [PR] Reset stats logger instance in daemon context to reinitialize in chil… [airflow]

2023-10-04 Thread via GitHub
pavansharma36 closed pull request #34768: Reset stats logger instance in daemon context to reinitialize in chil… URL: https://github.com/apache/airflow/pull/34768 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Implement `is_authorized()` in auth manager [airflow]

2023-10-04 Thread via GitHub
ashb commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1346287224 ## airflow/www/auth.py: ## @@ -35,16 +44,100 @@ def get_access_denied_message(): return conf.get("webserver", "access_denied_message") -def

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
Taragolis commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346277470 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +json_serializer: +

Re: [I] DataflowJob is failed when wait_until_finished=True although the state is JOB_STATE_DONE [airflow]

2023-10-04 Thread via GitHub
boring-cyborg[bot] commented on issue #34767: URL: https://github.com/apache/airflow/issues/34767#issuecomment-1747400431 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] DataflowJob is failed when wait_until_finished=True although the state is JOB_STATE_DONE [airflow]

2023-10-04 Thread via GitHub
champon1020 opened a new issue, #34767: URL: https://github.com/apache/airflow/issues/34767 ### Apache Airflow version 2.7.1 ### What happened We currently use the DataflowHook in tasks of Airflow DAG. If we upgrade the version of apache-airflow-google-providers to

Re: [I] Unable to create a WinRM Connection with apache-airflow-providers-microsoft-winrm installed [airflow]

2023-10-04 Thread via GitHub
arjunanan6 commented on issue #19739: URL: https://github.com/apache/airflow/issues/19739#issuecomment-1747366427 I think I've read the documentation properly to add a new connection type, but I can't seem to get it to show up. I've pushed my changes in #34766. @eladkal @phanikumv could

[PR] Initial commit [airflow]

2023-10-04 Thread via GitHub
arjunanan6 opened a new pull request, #34766: URL: https://github.com/apache/airflow/pull/34766 related: #19739 --- **^ Add meaningful description above** Read the **[Pull Request

Re: [PR] Fix GCSToGoogleDriveOperator and gdrive system tests [airflow]

2023-10-04 Thread via GitHub
eladkal commented on code in PR #34545: URL: https://github.com/apache/airflow/pull/34545#discussion_r1346241650 ## airflow/providers/google/suite/transfers/gcs_to_gdrive.py: ## @@ -95,7 +95,7 @@ def __init__( source_bucket: str, source_object: str,

[I] need to run airflow cli command "airflow dags details" for multiple DAG IDs [airflow]

2023-10-04 Thread via GitHub
vidyapanchalZS opened a new issue, #34765: URL: https://github.com/apache/airflow/issues/34765 ### Description Hello Team, Hope you are doing well! We have one use case where we want to check last parsed time for the multiple DAGs. Currently we have one airflow cli

Re: [PR] Initial commit [airflow]

2023-10-04 Thread via GitHub
arjunanan6 closed pull request #34764: Initial commit URL: https://github.com/apache/airflow/pull/34764 -- 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] Initial commit [airflow]

2023-10-04 Thread via GitHub
arjunanan6 opened a new pull request, #34764: URL: https://github.com/apache/airflow/pull/34764 related: #19739 --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] branch main updated: Remove warning about max_tis per query > parallelism (#34742)

2023-10-04 Thread dstandish
This is an automated email from the ASF dual-hosted git repository. dstandish 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 49bb45ca86 Remove warning about max_tis per

Re: [PR] Remove warning about max_tis per query > parallelism [airflow]

2023-10-04 Thread via GitHub
dstandish merged PR #34742: URL: https://github.com/apache/airflow/pull/34742 -- 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 constraints-main updated: Updating constraints. Github run id:6408700738

2023-10-04 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 d291ca5f2c Updating

Re: [PR] Fix metadata port in Keda connection when usePgbouncer is false [airflow]

2023-10-04 Thread via GitHub
brokenjacobs commented on PR #34741: URL: https://github.com/apache/airflow/pull/34741#issuecomment-1747308494 > @brokenjacobs this should fix the issue. > > Usually we don't create patch releases for the chart, but maybe we can wait a week and release a light patch release if other

Re: [PR] Fix metadata port in Keda connection when usePgbouncer is false [airflow]

2023-10-04 Thread via GitHub
brokenjacobs commented on PR #34741: URL: https://github.com/apache/airflow/pull/34741#issuecomment-1747307914 Please see https://github.com/apache/airflow/issues/34740 the database name is also incorrect. It uses the db name defined in the pgbouncer.ini, not the upstream database name.

Re: [I] Status of testing of Apache Airflow Helm Chart 1.11.0rc2 [airflow]

2023-10-04 Thread via GitHub
brokenjacobs commented on issue #34660: URL: https://github.com/apache/airflow/issues/34660#issuecomment-1747306545 @hussein-awala please see https://github.com/apache/airflow/issues/34740 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346190563 ## airflow/providers/amazon/aws/log/cloudwatch_task_handler.py: ## @@ -33,6 +33,24 @@ from airflow.models import TaskInstance +def

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346189969 ## airflow/providers/amazon/aws/log/cloudwatch_task_handler.py: ## @@ -33,6 +33,24 @@ from airflow.models import TaskInstance +def

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346188202 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +json_serializer: +

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346187609 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +json_serializer: +

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346186245 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +json_serializer: +

[PR] Refactor Sqlalchemy queries to 2.0 style (Part 9) [airflow]

2023-10-04 Thread via GitHub
phanikumv opened a new pull request, #34763: URL: https://github.com/apache/airflow/pull/34763 Part of https://github.com/apache/airflow/issues/28723 --- **^ Add meaningful description above** Read the **[Pull Request

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
ferruzzi commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1346185137 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +json_serializer: +

Re: [PR] Update params.rst [airflow]

2023-10-04 Thread via GitHub
Taragolis commented on code in PR #34759: URL: https://github.com/apache/airflow/pull/34759#discussion_r1346171913 ## docs/apache-airflow/core-concepts/params.rst: ## @@ -124,8 +124,10 @@ Another way to access your param is via a task's ``context`` kwarg.

[airflow] branch main updated (99f320354b -> 75261cf9b5)

2023-10-04 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 99f320354b Refactor: consolidate import time in providers (#34402) add 75261cf9b5 Use

Re: [PR] Use `pendulum.tz.timezone.Timezone` in k8s tests [airflow]

2023-10-04 Thread via GitHub
Taragolis merged PR #34745: URL: https://github.com/apache/airflow/pull/34745 -- 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] RBAC OP role access menu RESOURCE_PROVIDER [airflow]

2023-10-04 Thread via GitHub
ephraimbuddy commented on PR #34626: URL: https://github.com/apache/airflow/pull/34626#issuecomment-1747227016 The changed file is not in 2.7.1 so moving to 2.8.0 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] Unable to search dags/tasks using filters results in " [airflow]

2023-10-04 Thread via GitHub
boring-cyborg[bot] commented on issue #34761: URL: https://github.com/apache/airflow/issues/34761#issuecomment-1747130064 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] Unable to search dags/tasks using filters results in " [airflow]

2023-10-04 Thread via GitHub
ravi77o opened a new issue, #34761: URL: https://github.com/apache/airflow/issues/34761 ### Apache Airflow version 2.7.1 ### What happened When I use search macros other than "Starts With" when browsing Tasks and DAGs, Airflow disregards them search. I noticed

Re: [PR] Add support for ZoneInfo and generic UTC [airflow]

2023-10-04 Thread via GitHub
bolkedebruin commented on code in PR #34683: URL: https://github.com/apache/airflow/pull/34683#discussion_r1346058893 ## setup.cfg: ## @@ -72,6 +72,7 @@ install_requires = argcomplete>=1.10 asgiref attrs>=22.1.0 +backports.zoneinfo>=0.2.1;python_version<"3.9"

Re: [PR] Add support for ZoneInfo and generic UTC [airflow]

2023-10-04 Thread via GitHub
bolkedebruin commented on code in PR #34683: URL: https://github.com/apache/airflow/pull/34683#discussion_r1346057255 ## airflow/serialization/serializers/timezone.py: ## @@ -17,17 +17,32 @@ # under the License. from __future__ import annotations -from typing import

Re: [PR] Update params.rst [airflow]

2023-10-04 Thread via GitHub
boring-cyborg[bot] commented on PR #34759: URL: https://github.com/apache/airflow/pull/34759#issuecomment-1747065130 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] Update params.rst [airflow]

2023-10-04 Thread via GitHub
Falimonda opened a new pull request, #34759: URL: https://github.com/apache/airflow/pull/34759 Fixes python_callable function assignment and sets provide_context in context kwargs example. --- **^ Add meaningful description above** Read the **[Pull

Re: [PR] Upgrade watchtower to 3.0.1 (#25019) [airflow]

2023-10-04 Thread via GitHub
vincbeck commented on code in PR #34747: URL: https://github.com/apache/airflow/pull/34747#discussion_r1345872374 ## airflow/config_templates/config.yml: ## @@ -928,6 +928,21 @@ logging: type: boolean example: ~ default: "False" +json_serializer: +

[I] CLI: Variables set should allow to set description [airflow]

2023-10-04 Thread via GitHub
eladkal opened a new issue, #34756: URL: https://github.com/apache/airflow/issues/34756 ### Body The CLI: `airflow variables set [-h] [-j] [-v] key VALUE` https://airflow.apache.org/docs/apache-airflow/stable/cli-and-env-variables-ref.html#set_repeat1 Doesn't

[airflow] branch main updated: Refactor: consolidate import time in providers (#34402)

2023-10-04 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 99f320354b Refactor: consolidate import time in

Re: [PR] Refactor: consolidate import time in providers [airflow]

2023-10-04 Thread via GitHub
vincbeck merged PR #34402: URL: https://github.com/apache/airflow/pull/34402 -- 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   >