Re: [I] log_filename_template writes different try_number, then when it reads the log [airflow]

2023-10-07 Thread via GitHub
github-actions[bot] commented on issue #24958: URL: https://github.com/apache/airflow/issues/24958#issuecomment-1751628330 This issue has been automatically marked as stale because it has been open for 365 days without any activity. There has been several Airflow releases since last activit

Re: [I] Align TriggerDagRunOperator with logical date terminology [airflow]

2023-10-07 Thread via GitHub
github-actions[bot] commented on issue #26916: URL: https://github.com/apache/airflow/issues/26916#issuecomment-1751628316 This issue has been automatically marked as stale because it has been open for 365 days without any activity. There has been several Airflow releases since last activit

Re: [I] Airflow postgres connection field schema points to database name [airflow]

2023-10-07 Thread via GitHub
GarimaUttam commented on issue #34799: URL: https://github.com/apache/airflow/issues/34799#issuecomment-1751631019 Sir @Taragolis I want to start contributing as a newcomer. Plz suggest me how to start contributing. Should I ask you to assign them or should we start cloning and try to make

Re: [PR] Migrate provider-specific executor docs to providers [airflow]

2023-10-07 Thread via GitHub
eladkal commented on PR #34809: URL: https://github.com/apache/airflow/pull/34809#issuecomment-1751656980 > It's currently unclear why I'm able to reference `` :doc:`apache-airflow:` ``, but I can't seem to link, for example, `` :doc:`apache-airflow-providers-celery:celery_executor` `` >

Re: [PR] Fix broken breeze by fixing packaging version [airflow]

2023-10-07 Thread via GitHub
hussein-awala commented on PR #34701: URL: https://github.com/apache/airflow/pull/34701#issuecomment-1751668317 > > Some CI jobs load the scripts files from main ( > > @hussein-awala (and others) for the future - the way for committers to test such a PR is to make PR from "airflow" re

[airflow] branch main updated: docs: remove extraneous '>' in provider section name (#34813)

2023-10-07 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 f34e6f889f docs: remove extraneous '>' in provid

Re: [PR] docs: remove extraneous '>' in provider section name [airflow]

2023-10-07 Thread via GitHub
Taragolis merged PR #34813: URL: https://github.com/apache/airflow/pull/34813 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflo

Re: [PR] docs: remove extraneous '>' in provider section name [airflow]

2023-10-07 Thread via GitHub
boring-cyborg[bot] commented on PR #34813: URL: https://github.com/apache/airflow/pull/34813#issuecomment-1751675901 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.

Re: [PR] Add `get_ui_field_behaviour()` method to PostgresHook [airflow]

2023-10-07 Thread via GitHub
Taragolis commented on code in PR #34811: URL: https://github.com/apache/airflow/pull/34811#discussion_r1349501681 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -362,3 +362,12 @@ def get_openlineage_database_dialect(self, connection) -> str: def get_openlineage_de

Re: [I] Airflow postgres connection field schema points to database name [airflow]

2023-10-07 Thread via GitHub
Taragolis commented on issue #34799: URL: https://github.com/apache/airflow/issues/34799#issuecomment-1751678095 > I want to start contributing as a newcomer. Plz suggest me how to start contributing. Should I ask you to assign them or should we start cloning We have a documentation h

Re: [PR] Initial commit [airflow]

2023-10-07 Thread via GitHub
arjunanan6 commented on PR #34766: URL: https://github.com/apache/airflow/pull/34766#issuecomment-1751686587 @hussein-awala or @Taragolis do you see what I am doing wrong here? I added the connection-type in provider.yaml which appears to be where the provider info and conns are looked up f

Re: [PR] Initial commit [airflow]

2023-10-07 Thread via GitHub
Taragolis commented on code in PR #34766: URL: https://github.com/apache/airflow/pull/34766#discussion_r1349516480 ## airflow/providers/microsoft/psrp/provider.yaml: ## @@ -57,3 +57,7 @@ hooks: - integration-name: Windows PowerShell Remoting Protocol python-modules:

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

2023-10-07 Thread via GitHub
hussein-awala commented on code in PR #34741: URL: https://github.com/apache/airflow/pull/34741#discussion_r1349516585 ## chart/templates/secrets/metadata-connection-secret.yaml: ## @@ -25,8 +25,10 @@ {{- $metadataHost := .Values.data.metadataConnection.host | default $default

[airflow] branch update-audit-log-security-model created (now e33634cd56)

2023-10-07 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch update-audit-log-security-model in repository https://gitbox.apache.org/repos/asf/airflow.git at e33634cd56 Add explanation for Audit log users This branch includes the following new commi

[airflow] 01/01: Add explanation for Audit log users

2023-10-07 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch update-audit-log-security-model in repository https://gitbox.apache.org/repos/asf/airflow.git commit e33634cd56968e147c8b9c36e1ee84dd7a3c75da Author: Jarek Potiuk AuthorDate: Sat Oct 7 09:45:40 2

[PR] Add explanation about audit log users to security model [airflow]

2023-10-07 Thread via GitHub
potiuk opened a new pull request, #34814: URL: https://github.com/apache/airflow/pull/34814 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)*

Re: [PR] Initial commit [airflow]

2023-10-07 Thread via GitHub
Taragolis commented on PR #34766: URL: https://github.com/apache/airflow/pull/34766#issuecomment-1751704434 Problem with incorrect path to the PsrpHook, this could be found in case of running [Static Code Checks](https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.rst) ```

Re: [PR] Initial commit [airflow]

2023-10-07 Thread via GitHub
arjunanan6 commented on PR #34766: URL: https://github.com/apache/airflow/pull/34766#issuecomment-1751723857 @Taragolis Yikes, what a terrible oversight. I saw the static checks but did not check it properly enough. Thank you! -- This is an automated message from the Apache Git Service.

Re: [PR] Add `get_ui_field_behaviour()` method to PostgresHook [airflow]

2023-10-07 Thread via GitHub
pubudu91 commented on code in PR #34811: URL: https://github.com/apache/airflow/pull/34811#discussion_r1349527685 ## airflow/providers/postgres/hooks/postgres.py: ## @@ -362,3 +362,12 @@ def get_openlineage_database_dialect(self, connection) -> str: def get_openlineage_def

[PR] Clarify audit log permissions [airflow]

2023-10-07 Thread via GitHub
pierrejeambrun opened a new pull request, #34815: URL: https://github.com/apache/airflow/pull/34815 Clarify how permissions apply to audit logs. We had multiple occurrences where people think that you can only see audit logs related to DAGs that you have read permissions on. -- This is

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

2023-10-07 Thread via GitHub
ephraimbuddy merged PR #34712: URL: https://github.com/apache/airflow/pull/34712 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@air

[airflow] branch main updated (f34e6f889f -> f044589b68)

2023-10-07 Thread ephraimanierobi
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 f34e6f889f docs: remove extraneous '>' in provider section name (#34813) add f044589b68 Check if the low

Re: [PR] Migrate provider-specific executor docs to providers [airflow]

2023-10-07 Thread via GitHub
RNHTTR commented on PR #34809: URL: https://github.com/apache/airflow/pull/34809#issuecomment-1751738799 Yeah, I see lots of references from `apache-airflow` to `apache-airflow-providers`, but when I try to do ``:doc:`apache-airflow-providers-celery:celery_executor` ``, I get the following

Re: [PR] Add explanation about audit log users to security model [airflow]

2023-10-07 Thread via GitHub
potiuk merged PR #34814: URL: https://github.com/apache/airflow/pull/34814 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.a

[airflow] branch main updated (f044589b68 -> 4b867800c5)

2023-10-07 Thread potiuk
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 f044589b68 Check if the lower of provided values are sensitives in config endpoint (#34712) add 4b867800c5 Add ex

Re: [PR] Add explanation about audit log users to security model [airflow]

2023-10-07 Thread via GitHub
potiuk commented on PR #34814: URL: https://github.com/apache/airflow/pull/34814#issuecomment-1751738900 @ephraimbuddy - maybe we can make it in 2.7.2 ? -- 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

Re: [PR] Clarify audit log permissions [airflow]

2023-10-07 Thread via GitHub
potiuk commented on PR #34815: URL: https://github.com/apache/airflow/pull/34815#issuecomment-1751739141 Same here @ephraimbuddy -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comm

[airflow] branch v2-7-test updated (7d09fcb34c -> 3207ade571)

2023-10-07 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-7-test in repository https://gitbox.apache.org/repos/asf/airflow.git omit 7d09fcb34c Update RELEASE_NOTES.rst add 9fdbe7b96a Add support for ZoneInfo and generic UTC (#34683)

[airflow] branch update-audit-log-security-model deleted (was e33634cd56)

2023-10-07 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch update-audit-log-security-model in repository https://gitbox.apache.org/repos/asf/airflow.git was e33634cd56 Add explanation for Audit log users The revisions that were on this bra

[airflow] branch v2-7-test updated (3207ade571 -> 24ab6566ff)

2023-10-07 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-7-test in repository https://gitbox.apache.org/repos/asf/airflow.git omit 3207ade571 Update RELEASE_NOTES.rst add 24ab6566ff Update RELEASE_NOTES.rst This update added new r

Re: [PR] Clarify audit log permissions [airflow]

2023-10-07 Thread via GitHub
pierrejeambrun merged PR #34815: URL: https://github.com/apache/airflow/pull/34815 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@a

[airflow] branch main updated: Clarify audit log permissions (#34815)

2023-10-07 Thread pierrejeambrun
This is an automated email from the ASF dual-hosted git repository. pierrejeambrun 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 1f3525fd93 Clarify audit log permissions (#

[airflow] branch constraints-2-7 updated: Updating constraints. Github run id:6442070045

2023-10-07 Thread github-bot
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 aa13d3f399 Updating const

[I] Airflow 2.7.1 can not start Scheduler & trigger [airflow]

2023-10-07 Thread via GitHub
ThuanDoHung opened a new issue, #34816: URL: https://github.com/apache/airflow/issues/34816 ### Apache Airflow version 2.7.1 ### What happened After upgrade from 2.6.0 to 2.7.1 (try pip uninstall apache-airflow, and clear dir airflow - remove airflow.cfg), I can start sc

Re: [I] Airflow 2.7.1 can not start Scheduler & trigger [airflow]

2023-10-07 Thread via GitHub
boring-cyborg[bot] commented on issue #34816: URL: https://github.com/apache/airflow/issues/34816#issuecomment-1751774659 Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for ap

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

2023-10-07 Thread via GitHub
hussein-awala merged PR #34741: URL: https://github.com/apache/airflow/pull/34741 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@ai

Re: [I] not using pgbouncer connection values still use pgbouncer ports/names for keda in helm chart [airflow]

2023-10-07 Thread via GitHub
hussein-awala closed issue #34740: not using pgbouncer connection values still use pgbouncer ports/names for keda in helm chart URL: https://github.com/apache/airflow/issues/34740 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[airflow] branch main updated: Fix metadata DB & port in Keda connection when usePgbouncer is false (#34741)

2023-10-07 Thread husseinawala
This is an automated email from the ASF dual-hosted git repository. husseinawala pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 38e6607cc8 Fix metadata DB & port in Keda con

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

2023-10-07 Thread via GitHub
hussein-awala commented on PR #34589: URL: https://github.com/apache/airflow/pull/34589#issuecomment-1751781980 I close it in favor of #34771. -- 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] Remove false warning when TI state is QUEUED and TI doesn't have a start_date [airflow]

2023-10-07 Thread via GitHub
hussein-awala closed pull request #34589: Remove false warning when TI state is QUEUED and TI doesn't have a start_date URL: https://github.com/apache/airflow/pull/34589 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[I] Upgrade Helm chart dependency to bitnami/postgresql 13.x.x and PostgreSQL 16 [airflow]

2023-10-07 Thread via GitHub
dnskr opened a new issue, #34817: URL: https://github.com/apache/airflow/issues/34817 ### Official Helm Chart version 1.11.0 (latest released) ### Apache Airflow version 2.7.1 ### Kubernetes Version 1.27 ### Helm Chart configuration _No respons

Re: [I] Upgrade Helm chart dependency to bitnami/postgresql 13.x.x and PostgreSQL 16 [airflow]

2023-10-07 Thread via GitHub
boring-cyborg[bot] commented on issue #34817: URL: https://github.com/apache/airflow/issues/34817#issuecomment-1751798239 Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for ap

[PR] Rename company Parimatch Tech -> GR8 Tech [airflow]

2023-10-07 Thread via GitHub
KulykDmytro opened a new pull request, #34818: URL: https://github.com/apache/airflow/pull/34818 Rename company `Parimatch Tech` to `GR8 Tech` -- 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

[PR] Warn users when reading archived docs [airflow-site]

2023-10-07 Thread via GitHub
RNHTTR opened a new pull request, #870: URL: https://github.com/apache/airflow-site/pull/870 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

[PR] allow changing of config file from default for the KubernetesResourceBaseOperator [airflow]

2023-10-07 Thread via GitHub
chapm250 opened a new pull request, #34819: URL: https://github.com/apache/airflow/pull/34819 Currently the KubernetesResourceBaseOperator doesn't allow changing from the default K8s config_file location (unlike the KubernetesPodOperator), this change allows changing the config_file locatio

Re: [PR] allow changing of config file from default for the KubernetesResourceBaseOperator [airflow]

2023-10-07 Thread via GitHub
boring-cyborg[bot] commented on PR #34819: URL: https://github.com/apache/airflow/pull/34819#issuecomment-1751842914 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] Upgrade Helm chart dependency to bitnami/postgresql 13.x.x and PostgreSQL 16 [airflow]

2023-10-07 Thread via GitHub
Taragolis commented on issue #34817: URL: https://github.com/apache/airflow/issues/34817#issuecomment-1751852652 >and PostgreSQL 16 by default. This would unlikely happen, Airflow CI do not have regular runs on Postgres 16 yet (https://github.com/apache/airflow/pull/34543) Afte

Re: [I] Support stdlib zoneinfo.ZoneInfo (new in Python 3.10) [airflow]

2023-10-07 Thread via GitHub
bolkedebruin commented on issue #19450: URL: https://github.com/apache/airflow/issues/19450#issuecomment-1751865144 yes it is in stdlib, but it does not have a standard interface in the way of 'what you see is what you get'. tzname() is a prime example. The documentation reads that it will

Re: [I] UI/UX Graph Display [airflow]

2023-10-07 Thread via GitHub
github-actions[bot] commented on issue #34429: URL: https://github.com/apache/airflow/issues/34429#issuecomment-1751872063 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] fsspec base Hook for filesystem operations [airflow]

2023-10-07 Thread via GitHub
github-actions[bot] commented on PR #33578: URL: https://github.com/apache/airflow/pull/33578#issuecomment-1751872075 This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for you

Re: [PR] Allow sc templating in logs PVC [airflow]

2023-10-07 Thread via GitHub
github-actions[bot] closed pull request #32582: Allow sc templating in logs PVC URL: https://github.com/apache/airflow/pull/32582 -- 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.

Re: [PR] Add `get_ui_field_behaviour()` method to PostgresHook [airflow]

2023-10-07 Thread via GitHub
pubudu91 commented on PR #34811: URL: https://github.com/apache/airflow/pull/34811#issuecomment-1751877628 @Taragolis Any idea about the test failures in the pipeline? The failures don't look like they're related to the change I did right? :thinking: -- This is an automated message from

Re: [PR] Use timeout when inntializing _timeout_sec of TaskStateTrigger [airflow]

2023-10-07 Thread via GitHub
Xiroo commented on code in PR #34715: URL: https://github.com/apache/airflow/pull/34715#discussion_r1349613367 ## airflow/sensors/external_task.py: ## @@ -341,6 +341,7 @@ def execute(self, context: Context) -> None: states=self.allowed_states,

Re: [PR] Use timeout when inntializing _timeout_sec of TaskStateTrigger [airflow]

2023-10-07 Thread via GitHub
Xiroo commented on code in PR #34715: URL: https://github.com/apache/airflow/pull/34715#discussion_r1349613367 ## airflow/sensors/external_task.py: ## @@ -341,6 +341,7 @@ def execute(self, context: Context) -> None: states=self.allowed_states,

Re: [PR] Feature: Allow description to be passed in when uring variables CLI [airflow]

2023-10-07 Thread via GitHub
Zhenye-Na commented on PR #34791: URL: https://github.com/apache/airflow/pull/34791#issuecomment-1751910992 Fixed unit test errors ``` breeze testing tests --test-type CLI tests/cli/test_cli_parser.py [ 9%] tests/cli/commands/t

[I] Azure Container Instance Operator uses deprecated "network profile" arg [airflow]

2023-10-07 Thread via GitHub
jackglick opened a new issue, #34820: URL: https://github.com/apache/airflow/issues/34820 ### Apache Airflow version 2.7.1 ### What happened The Azure Container Instance Operator uses the "network profile" argument which is since deprecated by Microsft: https://learn.mi

Re: [I] Azure Container Instance Operator uses deprecated "network profile" arg [airflow]

2023-10-07 Thread via GitHub
boring-cyborg[bot] commented on issue #34820: URL: https://github.com/apache/airflow/issues/34820#issuecomment-1751914804 Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for ap

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

2023-10-07 Thread via GitHub
Zhenye-Na commented on issue #34756: URL: https://github.com/apache/airflow/issues/34756#issuecomment-1751920114 https://github.com/apache/airflow/pull/34791 is ready for review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [I] Expose description columns of Pools in the UI [airflow]

2023-10-07 Thread via GitHub
Zhenye-Na commented on issue #34751: URL: https://github.com/apache/airflow/issues/34751#issuecomment-1751927395 I am having some issues starting the airflow web server via github codespaces ``` breeze start-airflow --use-airflow-version 2.7.0 --executor CeleryExecutor --airflow-ex

Re: [I] Azure Container Instance Operator uses deprecated "network profile" arg [airflow]

2023-10-07 Thread via GitHub
Lee-W commented on issue #34820: URL: https://github.com/apache/airflow/issues/34820#issuecomment-1751932073 https://github.com/apache/airflow/pull/34738 is now working on it. I think it will be included in the next azure provider release -- This is an automated message from the Apache Gi

Re: [PR] Add `get_ui_field_behaviour()` method to PostgresHook [airflow]

2023-10-07 Thread via GitHub
Lee-W commented on PR #34811: URL: https://github.com/apache/airflow/pull/34811#issuecomment-1751933819 > @Taragolis Any idea about the test failures in the pipeline? The failures don't look like they're related to the change I did right? 🤔 I would suggest trying to rebase from the la