eladkal merged PR #37047:
URL: https://github.com/apache/airflow/pull/37047
--
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.
amoghrajesh commented on PR #37047:
URL: https://github.com/apache/airflow/pull/37047#issuecomment-1930100941
Okay we finally have a green CI. Shall we send this in @potiuk ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
ketozhang commented on code in PR #37047:
URL: https://github.com/apache/airflow/pull/37047#discussion_r1477605769
##
docs/apache-airflow-providers-cncf-kubernetes/operators.rst:
##
@@ -77,8 +77,8 @@ You can print out the Kubernetes manifest for the pod that
would be created at
potiuk commented on PR #37047:
URL: https://github.com/apache/airflow/pull/37047#issuecomment-1923766672
fix to bactics is needed. RST does not like single backticks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
dirrao commented on code in PR #37047:
URL: https://github.com/apache/airflow/pull/37047#discussion_r1475580760
##
docs/apache-airflow-providers-cncf-kubernetes/operators.rst:
##
@@ -77,8 +77,8 @@ You can print out the Kubernetes manifest for the pod that
would be created at r
ketozhang commented on PR #37047:
URL: https://github.com/apache/airflow/pull/37047#issuecomment-1917580816
Perhaps out of scope. `pod_template_file` and `pod_template_dict` is
mutually exclusive. If both are specified then `pod_template_dict` is ignored.
This should be in the API docs.
-
ketozhang commented on code in PR #37047:
URL: https://github.com/apache/airflow/pull/37047#discussion_r1471693809
##
docs/apache-airflow-providers-cncf-kubernetes/operators.rst:
##
@@ -77,8 +77,8 @@ You can print out the Kubernetes manifest for the pod that
would be created at
dirrao commented on code in PR #37047:
URL: https://github.com/apache/airflow/pull/37047#discussion_r1471103997
##
docs/apache-airflow-providers-cncf-kubernetes/operators.rst:
##
@@ -77,8 +77,8 @@ You can print out the Kubernetes manifest for the pod that
would be created at r
ketozhang commented on code in PR #37047:
URL: https://github.com/apache/airflow/pull/37047#discussion_r1470039181
##
docs/apache-airflow-providers-cncf-kubernetes/operators.rst:
##
@@ -77,8 +77,8 @@ You can print out the Kubernetes manifest for the pod that
would be created at
amoghrajesh commented on code in PR #37047:
URL: https://github.com/apache/airflow/pull/37047#discussion_r1468895610
##
docs/apache-airflow-providers-cncf-kubernetes/operators.rst:
##
@@ -77,8 +77,8 @@ You can print out the Kubernetes manifest for the pod that
would be created
boring-cyborg[bot] commented on PR #37047:
URL: https://github.com/apache/airflow/pull/37047#issuecomment-1913480028
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 Contributors'
ketozhang opened a new pull request, #37047:
URL: https://github.com/apache/airflow/pull/37047
closes: #35969
---
Fixes the Argument precedence section of KPO docs. Removed incorrect mention
of KPO creates pods based off the Airflow connection. It should
12 matches
Mail list logo