jedcunningham commented on code in PR #39510:
URL: https://github.com/apache/airflow/pull/39510#discussion_r1595576751
##########
docs/apache-airflow/installation/installing-from-pypi.rst:
##########
@@ -331,6 +331,17 @@ dependencies compatible with just airflow core at the
moment Airflow was release
pip install "apache-airflow==${AIRFLOW_VERSION}" --constraint
"${CONSTRAINT_URL}"
+.. note::
+
+ Airflow uses `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+ This data assists Airflow maintainers in better understanding how Airflow
is used.
+ Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+ security fixes. Additionally, this information supports key decisions
related to the development Roadmap.
Review Comment:
```suggestion
security fixes. Additionally, this information supports key decisions
related to the development roadmap.
```
##########
docs/apache-airflow/faq.rst:
##########
@@ -522,3 +522,15 @@ This means ``explicit_defaults_for_timestamp`` is disabled
in your mysql server
#. Set ``explicit_defaults_for_timestamp = 1`` under the ``mysqld`` section in
your ``my.cnf`` file.
#. Restart the Mysql server.
+
+Does Airflow collect any telemetry data?
+----------------------------------------
+
+Airflow integrates `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+This data assists Airflow maintainers in better understanding how Airflow is
used.
+Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+security fixes. Additionally, this information supports key decisions related
to the development Roadmap.
+
+Users can easily opt out of analytics in various ways documented
+`here <https://docs.scarf.sh/gateway/#do-not-track>`__ and by setting the
:ref:`config:scarf_analytics__enabled` option
+to ``False``. Airflow also respects ``SCARF_ANALYTICS=false`` environment
variable.
Review Comment:
```suggestion
to ``False``. Airflow also respects the ``SCARF_ANALYTICS=false``
environment variable.
```
##########
airflow/config_templates/config.yml:
##########
@@ -2591,3 +2591,22 @@ sensors:
type: float
example: ~
default: "604800"
+scarf_analytics:
+ description: |
+ Airflow integrates `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+ This data assists Airflow maintainers in better understanding how Airflow
is used.
+ Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+ security fixes. Additionally, this information supports key decisions
related to the development Roadmap.
+
+ Users can easily opt out of analytics in various ways documented
+ `here <https://docs.scarf.sh/gateway/#do-not-track>`__ and by setting the
``enabled`` option
Review Comment:
Is this link right? We aren't planning to use the Scarf gateway?
##########
docs/apache-airflow/installation/installing-from-pypi.rst:
##########
@@ -331,6 +331,17 @@ dependencies compatible with just airflow core at the
moment Airflow was release
pip install "apache-airflow==${AIRFLOW_VERSION}" --constraint
"${CONSTRAINT_URL}"
+.. note::
+
+ Airflow uses `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+ This data assists Airflow maintainers in better understanding how Airflow
is used.
+ Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+ security fixes. Additionally, this information supports key decisions
related to the development Roadmap.
+
+ Users can easily opt out of analytics in various ways documented
+ `here <https://docs.scarf.sh/gateway/#do-not-track>`__ and by setting the
:ref:`config:scarf_analytics__enabled` option
+ to ``False``. Airflow also respects ``SCARF_ANALYTICS=false`` environment
variable.
Review Comment:
```suggestion
to ``False``. Airflow also respects the ``SCARF_ANALYTICS=false``
environment variable.
```
##########
airflow/config_templates/config.yml:
##########
@@ -2591,3 +2591,22 @@ sensors:
type: float
example: ~
default: "604800"
+scarf_analytics:
+ description: |
+ Airflow integrates `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+ This data assists Airflow maintainers in better understanding how Airflow
is used.
+ Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+ security fixes. Additionally, this information supports key decisions
related to the development Roadmap.
+
+ Users can easily opt out of analytics in various ways documented
Review Comment:
```suggestion
Users can easily opt-out of analytics in various ways documented
```
##########
docs/apache-airflow/installation/installing-from-pypi.rst:
##########
@@ -331,6 +331,17 @@ dependencies compatible with just airflow core at the
moment Airflow was release
pip install "apache-airflow==${AIRFLOW_VERSION}" --constraint
"${CONSTRAINT_URL}"
+.. note::
+
+ Airflow uses `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+ This data assists Airflow maintainers in better understanding how Airflow
is used.
+ Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+ security fixes. Additionally, this information supports key decisions
related to the development Roadmap.
+
+ Users can easily opt out of analytics in various ways documented
Review Comment:
```suggestion
Users can easily opt-out of analytics in various ways documented
```
##########
airflow/config_templates/config.yml:
##########
@@ -2591,3 +2591,22 @@ sensors:
type: float
example: ~
default: "604800"
+scarf_analytics:
+ description: |
+ Airflow integrates `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+ This data assists Airflow maintainers in better understanding how Airflow
is used.
+ Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+ security fixes. Additionally, this information supports key decisions
related to the development Roadmap.
+
+ Users can easily opt out of analytics in various ways documented
+ `here <https://docs.scarf.sh/gateway/#do-not-track>`__ and by setting the
``enabled`` option
+ to ``False``. Airflow also respects ``SCARF_ANALYTICS=false`` environment
variable.
Review Comment:
```suggestion
to ``False``. Airflow also respects the ``SCARF_ANALYTICS=false``
environment variable.
```
##########
docs/apache-airflow/faq.rst:
##########
@@ -522,3 +522,15 @@ This means ``explicit_defaults_for_timestamp`` is disabled
in your mysql server
#. Set ``explicit_defaults_for_timestamp = 1`` under the ``mysqld`` section in
your ``my.cnf`` file.
#. Restart the Mysql server.
+
+Does Airflow collect any telemetry data?
+----------------------------------------
+
+Airflow integrates `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+This data assists Airflow maintainers in better understanding how Airflow is
used.
+Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+security fixes. Additionally, this information supports key decisions related
to the development Roadmap.
+
+Users can easily opt out of analytics in various ways documented
Review Comment:
```suggestion
Users can easily opt-out of analytics in various ways documented
```
##########
airflow/config_templates/config.yml:
##########
@@ -2591,3 +2591,22 @@ sensors:
type: float
example: ~
default: "604800"
+scarf_analytics:
+ description: |
+ Airflow integrates `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+ This data assists Airflow maintainers in better understanding how Airflow
is used.
+ Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+ security fixes. Additionally, this information supports key decisions
related to the development Roadmap.
Review Comment:
```suggestion
security fixes. Additionally, this information supports key decisions
related to the development roadmap.
```
nit
##########
docs/apache-airflow/faq.rst:
##########
@@ -522,3 +522,15 @@ This means ``explicit_defaults_for_timestamp`` is disabled
in your mysql server
#. Set ``explicit_defaults_for_timestamp = 1`` under the ``mysqld`` section in
your ``my.cnf`` file.
#. Restart the Mysql server.
+
+Does Airflow collect any telemetry data?
+----------------------------------------
+
+Airflow integrates `Scarf <https://about.scarf.sh/>`__ to collect basic
telemetry data during operation.
+This data assists Airflow maintainers in better understanding how Airflow is
used.
+Insights gained from this telemetry are critical for prioritizing patches,
minor releases, and
+security fixes. Additionally, this information supports key decisions related
to the development Roadmap.
Review Comment:
```suggestion
security fixes. Additionally, this information supports key decisions
related to the development roadmap.
```
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
