[GitHub] [airflow] potiuk commented on a change in pull request #19010: Workaround libstdcpp TLS error

2021-10-18 Thread GitBox
potiuk commented on a change in pull request #19010: URL: https://github.com/apache/airflow/pull/19010#discussion_r731386158 ## File path: Dockerfile.ci ## @@ -400,6 +400,16 @@ LABEL org.apache.airflow.distro="debian" \ org.opencontainers.image.title="Continuous Integration

[GitHub] [airflow] potiuk commented on pull request #19042: Update CONTRIBUTORS_QUICK_START.rst

2021-10-18 Thread GitBox
potiuk commented on pull request #19042: URL: https://github.com/apache/airflow/pull/19042#issuecomment-946250086 > Sorry, I think I got you confused. I mean that they should be in this pull request. Close the others and add the changes to this one Yeah. Agre e those are better toget

[GitHub] [airflow] potiuk edited a comment on pull request #19042: Update CONTRIBUTORS_QUICK_START.rst

2021-10-18 Thread GitBox
potiuk edited a comment on pull request #19042: URL: https://github.com/apache/airflow/pull/19042#issuecomment-946250086 > Sorry, I think I got you confused. I mean that they should be in this pull request. Close the others and add the changes to this one Yeah. Agree those are better

[GitHub] [airflow] github-actions[bot] commented on pull request #17485: Pass explicit state, not a success boolean

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #17485: URL: https://github.com/apache/airflow/pull/17485#issuecomment-946259391 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 y

[GitHub] [airflow] github-actions[bot] commented on pull request #16166: Fixing SES backend - source (mail_from) must be string.

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #16166: URL: https://github.com/apache/airflow/pull/16166#issuecomment-946259427 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 y

[GitHub] [airflow] uranusjr closed pull request #16166: Fixing SES backend - source (mail_from) must be string.

2021-10-18 Thread GitBox
uranusjr closed pull request #16166: URL: https://github.com/apache/airflow/pull/16166 -- 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-uns

[GitHub] [airflow] uranusjr commented on pull request #16166: Fixing SES backend - source (mail_from) must be string.

2021-10-18 Thread GitBox
uranusjr commented on pull request #16166: URL: https://github.com/apache/airflow/pull/16166#issuecomment-946272873 Continuing in #18042. -- 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 spec

[GitHub] [airflow] uranusjr commented on a change in pull request #18042: Fixing ses email backend

2021-10-18 Thread GitBox
uranusjr commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r731410907 ## File path: airflow/config_templates/config.yml ## @@ -1358,6 +1358,14 @@ example: "/path/to/my_html_content_template_file" default: ~

[GitHub] [airflow] uranusjr commented on a change in pull request #18042: Fixing ses email backend

2021-10-18 Thread GitBox
uranusjr commented on a change in pull request #18042: URL: https://github.com/apache/airflow/pull/18042#discussion_r731411083 ## File path: airflow/config_templates/config.yml ## @@ -1358,6 +1358,14 @@ example: "/path/to/my_html_content_template_file" default: ~

[GitHub] [airflow] github-actions[bot] commented on pull request #19010: Workaround libstdcpp TLS error

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #19010: URL: https://github.com/apache/airflow/pull/19010#issuecomment-946275408 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and tak

[GitHub] [airflow] github-actions[bot] commented on pull request #19057: Update docstring to let users use `node_selector`

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #19057: URL: https://github.com/apache/airflow/pull/19057#issuecomment-946277243 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not m

[GitHub] [airflow] uranusjr commented on a change in pull request #19034: Allow Param to support a default value of ``None``

2021-10-18 Thread GitBox
uranusjr commented on a change in pull request #19034: URL: https://github.com/apache/airflow/pull/19034#discussion_r731414520 ## File path: airflow/models/param.py ## @@ -38,37 +48,44 @@ class Param: :type schema: dict """ -def __init__(self, default: Any = Non

[GitHub] [airflow] potiuk merged pull request #19010: Workaround libstdcpp TLS error

2021-10-18 Thread GitBox
potiuk merged pull request #19010: URL: https://github.com/apache/airflow/pull/19010 -- 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-unsub

[airflow] branch main updated: Workaround libstdcpp TLS error (#19010)

2021-10-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 8858309 Workaround libstdcpp TLS error (#19010) 88

[GitHub] [airflow] josh-fell commented on pull request #19052: Replacing non-attribute template_fields for BigQueryToMsSqlOperator

2021-10-18 Thread GitBox
josh-fell commented on pull request #19052: URL: https://github.com/apache/airflow/pull/19052#issuecomment-946294702 CC @dudutch -- 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

[GitHub] [airflow] mik-laj closed issue #12508: Run pip check in CI pipelines

2021-10-18 Thread GitBox
mik-laj closed issue #12508: URL: https://github.com/apache/airflow/issues/12508 -- 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.

[GitHub] [airflow] mik-laj commented on issue #12508: Run pip check in CI pipelines

2021-10-18 Thread GitBox
mik-laj commented on issue #12508: URL: https://github.com/apache/airflow/issues/12508#issuecomment-946298413 Done: https://github.com/apache/airflow/blob/e25eea052fd54c94b490a377de05c6bae4c24dbb/scripts/docker/install_airflow.sh#L63 -- This is an automated message from the Apache Git Se

[GitHub] [airflow] github-actions[bot] commented on pull request #19052: Replacing non-attribute template_fields for BigQueryToMsSqlOperator

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #19052: URL: https://github.com/apache/airflow/pull/19052#issuecomment-946300363 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not m

[GitHub] [airflow] mik-laj opened a new issue #19059: Add Python 3.10 support

2021-10-18 Thread GitBox
mik-laj opened a new issue #19059: URL: https://github.com/apache/airflow/issues/19059 ### Description According to README.md - "Support for Python and Kubernetes versions": > We support a new version of Python/Kubernetes in main after they are officially released, as soon as

[GitHub] [airflow] uranusjr commented on issue #19059: Add Python 3.10 support

2021-10-18 Thread GitBox
uranusjr commented on issue #19059: URL: https://github.com/apache/airflow/issues/19059#issuecomment-946304173 The `ModuleNotFoundError: No module named 'pwd'` error the user gets is weird though. Airflow should have no problem running on 3.10 (albeit not officially supported), and the `pw

[GitHub] [airflow] mik-laj opened a new pull request #19060: Warn about unsupported Python 3.10

2021-10-18 Thread GitBox
mik-laj opened a new pull request #19060: URL: https://github.com/apache/airflow/pull/19060 Related: https://github.com/apache/airflow/issues/19059 Some users are unaware of the supported Python versions and then have trouble using Airflow. It's best to alert them to be aware of potentia

[GitHub] [airflow] uranusjr opened a new pull request #19061: Rename trigger page label to Logical Date

2021-10-18 Thread GitBox
uranusjr opened a new pull request #19061: URL: https://github.com/apache/airflow/pull/19061 AIP-39 changed the canonical name from Execution Date, and this should match it. I'm wondering whether we should change the trigger page to have _two_ calendar pickers to represent the data

[GitHub] [airflow] github-actions[bot] commented on pull request #19061: Rename trigger page label to Logical Date

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #19061: URL: https://github.com/apache/airflow/pull/19061#issuecomment-946313479 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not m

[GitHub] [airflow] uranusjr commented on a change in pull request #19060: Warn about unsupported Python 3.10

2021-10-18 Thread GitBox
uranusjr commented on a change in pull request #19060: URL: https://github.com/apache/airflow/pull/19060#discussion_r731444613 ## File path: airflow/__main__.py ## @@ -19,20 +19,28 @@ # under the License. """Main executable module""" - +import logging import os import a

[airflow] 01/01: Rename execution date in forms and tables

2021-10-18 Thread bbovenzi
This is an automated email from the ASF dual-hosted git repository. bbovenzi pushed a commit to branch replace-execution-date-with-logical-date in repository https://gitbox.apache.org/repos/asf/airflow.git commit babe8fde5a3b17b9b50e1ddbfd21a35031b27ada Author: Brent Bovenzi AuthorDate: Mon Oct

[airflow] branch replace-execution-date-with-logical-date created (now babe8fd)

2021-10-18 Thread bbovenzi
This is an automated email from the ASF dual-hosted git repository. bbovenzi pushed a change to branch replace-execution-date-with-logical-date in repository https://gitbox.apache.org/repos/asf/airflow.git. at babe8fd Rename execution date in forms and tables This branch includes the fol

[GitHub] [airflow] josh-fell opened a new pull request #19062: Static start_date and default arg cleanup for Microsoft providers example DAGs

2021-10-18 Thread GitBox
josh-fell opened a new pull request #19062: URL: https://github.com/apache/airflow/pull/19062 There is an ongoing effort to enhance example DAGs by setting static values for `start_date` (already complete for core example DAGs), cleaning up and/or implementing useful `default_args`, and re

[GitHub] [airflow] josh-fell commented on a change in pull request #19062: Static start_date and default arg cleanup for Microsoft providers example DAGs

2021-10-18 Thread GitBox
josh-fell commented on a change in pull request #19062: URL: https://github.com/apache/airflow/pull/19062#discussion_r731458987 ## File path: airflow/providers/microsoft/azure/example_dags/example_azure_blob_to_gcs.py ## @@ -16,44 +16,33 @@ # specific language governing permi

[GitHub] [airflow] josh-fell commented on a change in pull request #19062: Static start_date and default arg cleanup for Microsoft providers example DAGs

2021-10-18 Thread GitBox
josh-fell commented on a change in pull request #19062: URL: https://github.com/apache/airflow/pull/19062#discussion_r731459244 ## File path: airflow/providers/microsoft/azure/example_dags/example_azure_container_instances.py ## @@ -27,17 +27,10 @@ with DAG( dag_id='ac

[GitHub] [airflow] bbovenzi opened a new pull request #19063: Rename execution date in forms and tables

2021-10-18 Thread GitBox
bbovenzi opened a new pull request #19063: URL: https://github.com/apache/airflow/pull/19063 Update references of "Execution Date" to "Logical Date" in forms and tables of the UI. Examples: https://user-images.githubusercontent.com/4600967/137837436-d77879cc-fcba-4d46-9024-c1160ef

[GitHub] [airflow] josh-fell commented on a change in pull request #19062: Static start_date and default arg cleanup for Microsoft providers example DAGs

2021-10-18 Thread GitBox
josh-fell commented on a change in pull request #19062: URL: https://github.com/apache/airflow/pull/19062#discussion_r731459744 ## File path: airflow/providers/microsoft/winrm/example_dags/example_winrm.py ## @@ -28,23 +28,22 @@ """ This is an example dag for using the WinRMO

[GitHub] [airflow] josh-fell commented on a change in pull request #19062: Static start_date and default arg cleanup for Microsoft providers example DAGs

2021-10-18 Thread GitBox
josh-fell commented on a change in pull request #19062: URL: https://github.com/apache/airflow/pull/19062#discussion_r731459874 ## File path: docs/apache-airflow-providers-microsoft-azure/operators/azure_blob_to_gcs.rst ## @@ -51,10 +51,10 @@ Operator transfers data from Azure

[GitHub] [airflow] prabhjyotsingh commented on pull request #19025: [Minor] Fix padding on home page

2021-10-18 Thread GitBox
prabhjyotsingh commented on pull request #19025: URL: https://github.com/apache/airflow/pull/19025#issuecomment-946336910 Nothing new looks broken 🤞. "Lint Helm Chart" failure looks unrelated. ![option2-final](https://user-images.githubusercontent.com/674497/137839218-3d80d341-7089-4

[GitHub] [airflow] github-actions[bot] commented on pull request #19063: Rename execution date in forms and tables

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #19063: URL: https://github.com/apache/airflow/pull/19063#issuecomment-946345806 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not m

[GitHub] [airflow] andrenerd closed issue #19037: Set replace_microseconds to False

2021-10-18 Thread GitBox
andrenerd closed issue #19037: URL: https://github.com/apache/airflow/issues/19037 -- 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-unsubsc

[GitHub] [airflow] kd2718 commented on a change in pull request #19036: replaced '.' with '-' and adjusted trimmed_pod_id per ticket comments

2021-10-18 Thread GitBox
kd2718 commented on a change in pull request #19036: URL: https://github.com/apache/airflow/pull/19036#discussion_r731487808 ## File path: airflow/kubernetes/pod_generator.py ## @@ -460,9 +460,9 @@ def make_unique_pod_id(pod_id: str) -> str: safe_uuid = uuid.uuid4().

[GitHub] [airflow] ashmeet13 commented on issue #17778: Add Arango hook

2021-10-18 Thread GitBox
ashmeet13 commented on issue #17778: URL: https://github.com/apache/airflow/issues/17778#issuecomment-946369213 Hi @mrbazzan, sorry should have updated here - I am not you can go ahead and pick this up. My bad. I did however start on this and you can find a bit of code that I had wr

[GitHub] [airflow] uranusjr opened a new pull request #19064: Remove distutils usages for Python 3.10

2021-10-18 Thread GitBox
uranusjr opened a new pull request #19064: URL: https://github.com/apache/airflow/pull/19064 See #19059. Version parsing capabilities when building provider packages is now provided by `packaging.version.Version` instead. Non-Python software version parsing (sqlite) is re-implemente

[GitHub] [airflow] github-actions[bot] commented on pull request #19036: replaced '.' with '-' and adjusted trimmed_pod_id per ticket comments

2021-10-18 Thread GitBox
github-actions[bot] commented on pull request #19036: URL: https://github.com/apache/airflow/pull/19036#issuecomment-946378601 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and tak

[GitHub] [airflow] Bowrna commented on issue #19045: Setting up development environment for Airflow: Pyenv commands failed in macOS

2021-10-18 Thread GitBox
Bowrna commented on issue #19045: URL: https://github.com/apache/airflow/issues/19045#issuecomment-946382127 thank you @potiuk for your description. I will fix this and raise the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [airflow] lwyszomi closed pull request #18879: Upgrade the dataproc package to 3.0.0 and migrate from v1beta2 to v1 api

2021-10-18 Thread GitBox
lwyszomi closed pull request #18879: URL: https://github.com/apache/airflow/pull/18879 -- 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-uns

[GitHub] [airflow] uranusjr commented on a change in pull request #18924: Adding dag_id_pattern parameter to the /dags endpoint

2021-10-18 Thread GitBox
uranusjr commented on a change in pull request #18924: URL: https://github.com/apache/airflow/pull/18924#discussion_r731519692 ## File path: airflow/api_connexion/endpoints/dag_endpoint.py ## @@ -60,13 +60,16 @@ def get_dag_details(dag_id): @security.requires_access([(permissi

[GitHub] [airflow] uranusjr commented on pull request #18934: Implement API generator and use it on Connection

2021-10-18 Thread GitBox
uranusjr commented on pull request #18934: URL: https://github.com/apache/airflow/pull/18934#issuecomment-946395002 Any thoughts on the general approach taken here? Again, the abstraction implemented here is not really affected by what OpenAPI implementation we use. -- This is an automat

[GitHub] [airflow] uranusjr merged pull request #18790: Google provider catch invalid secret name

2021-10-18 Thread GitBox
uranusjr merged pull request #18790: URL: https://github.com/apache/airflow/pull/18790 -- 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-uns

[GitHub] [airflow] uranusjr closed issue #18789: Google Cloud Secret Manager Backend fail on variable with a non allow character

2021-10-18 Thread GitBox
uranusjr closed issue #18789: URL: https://github.com/apache/airflow/issues/18789 -- 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] branch main updated: Google provider catch invalid secret name (#18790)

2021-10-18 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 0e95b57 Google provider catch invalid secret nam

[GitHub] [airflow] Jaxing commented on issue #19040: Upgrading from 2.1.3 to 2.20 some dags fail due to pickeling

2021-10-18 Thread GitBox
Jaxing commented on issue #19040: URL: https://github.com/apache/airflow/issues/19040#issuecomment-946406584 We have never been have never been explicitly on 3.6 that is if the default changed from 3.6 to 3.8? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] uranusjr commented on issue #19040: Upgrading from 2.1.3 to 2.20 some dags fail due to pickeling

2021-10-18 Thread GitBox
uranusjr commented on issue #19040: URL: https://github.com/apache/airflow/issues/19040#issuecomment-946407561 > that is if the default changed from 3.6 to 3.8? It did not, but even if it did there shouldn't be a problem. What seems to have happened is the other way around: The _2.1_

[GitHub] [airflow] brain-buster opened a new pull request #19065: Remove incorrect type comment in Swagger2Specification._set_defaults classmethod

2021-10-18 Thread GitBox
brain-buster opened a new pull request #19065: URL: https://github.com/apache/airflow/pull/19065 related: #19006 -- 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 unsubsc

[GitHub] [airflow] mrbazzan commented on issue #17778: Add Arango hook

2021-10-18 Thread GitBox
mrbazzan commented on issue #17778: URL: https://github.com/apache/airflow/issues/17778#issuecomment-946420073 @ashmeet13 Oh... Thanks, that'll help. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

<    1   2