You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/11/27 14:50:21 UTC
[GitHub] [airflow] potiuk opened a new pull request #19855: Add information about supported OS-es for Apache Airflow
potiuk opened a new pull request #19855:
URL: https://github.com/apache/airflow/pull/19855
While answering some Stack Overflow questions I found and was
generally very surprised that we do not mention POSIX compliance
and "only use Linux for production" in our prerequisites.
This PR adds a note about it - both in GitHub and in our installation
prerequisites.
<!--
Thank you for contributing! Please make sure that your code changes
are covered with tests. And in case of new features or big changes
remember to adjust the documentation.
Feel free to ping committers for the review!
In case of existing issue, reference it using one of the following:
closes: #ISSUE
related: #ISSUE
How to write a good git commit message:
http://chris.beams.io/posts/git-commit/
-->
---
**^ Add meaningful description above**
Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed.
In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
In case of backwards incompatible changes please leave a note in [UPDATING.md](https://github.com/apache/airflow/blob/main/UPDATING.md).
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on a change in pull request #19855: Add information about supported OS-es for Apache Airflow
Posted by GitBox <gi...@apache.org>.
potiuk commented on a change in pull request #19855:
URL: https://github.com/apache/airflow/pull/19855#discussion_r757803938
##########
File path: docs/apache-airflow/installation/prerequisites.rst
##########
@@ -43,3 +43,12 @@ Starting with Airflow 2.1.2, Airflow is tested with Python 3.6, 3.7, 3.8, and 3.
The minimum memory required we recommend Airflow to run with is 4GB, but the actual requirements depends
wildly on the deployment options you have
+
+**Note**: Airflow currently can be run on POSIX-compliant Operating Systems. For development it is regularly
+tested on fairly modern Linux Distros and recent versions of MacOS.
+On Windows you can run it via WSL2 (Windows Subsystem for Linux 2) or via Linux Containers.
+The work to add Windows support is tracked via `#10388 <https://github.com/apache/airflow/issues/10388>`__ but
+it is not a high priority. You should only use Linux-based distros as "Production" execution environment
+as this is the only environment that is supported. The only distro that is used in our CI tests and that
+is used in the `Community managed DockerHub image <https://hub.docker.com/p/apache/airflow>`__ is
+``Debian Buster``.
Review comment:
We tried to move to bullseye already (and we will come-back to it) but there were issues with missing mssql / odbc support (https://github.com/MicrosoftDocs/sql-docs/issues/6804 and PR here https://github.com/apache/airflow/pull/18279).
So - since this is a "general" statement I think I prefer to stick to specific version (and update it when we upgrade - which is likely in January as described in the comment from Microsoft.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #19855: Add information about supported OS-es for Apache Airflow
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #19855:
URL: https://github.com/apache/airflow/pull/19855#issuecomment-980750953
The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full test matrix is needed and add the 'full tests needed' label. Then you should rebase it to the latest main or amend the last commit of the PR, and push it with --force-with-lease.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] dstandish commented on a change in pull request #19855: Add information about supported OS-es for Apache Airflow
Posted by GitBox <gi...@apache.org>.
dstandish commented on a change in pull request #19855:
URL: https://github.com/apache/airflow/pull/19855#discussion_r757803125
##########
File path: docs/apache-airflow/installation/prerequisites.rst
##########
@@ -43,3 +43,12 @@ Starting with Airflow 2.1.2, Airflow is tested with Python 3.6, 3.7, 3.8, and 3.
The minimum memory required we recommend Airflow to run with is 4GB, but the actual requirements depends
wildly on the deployment options you have
+
+**Note**: Airflow currently can be run on POSIX-compliant Operating Systems. For development it is regularly
+tested on fairly modern Linux Distros and recent versions of MacOS.
+On Windows you can run it via WSL2 (Windows Subsystem for Linux 2) or via Linux Containers.
+The work to add Windows support is tracked via `#10388 <https://github.com/apache/airflow/issues/10388>`__ but
+it is not a high priority. You should only use Linux-based distros as "Production" execution environment
+as this is the only environment that is supported. The only distro that is used in our CI tests and that
+is used in the `Community managed DockerHub image <https://hub.docker.com/p/apache/airflow>`__ is
+``Debian Buster``.
Review comment:
I thought I saw a reference to bullseye somewhere but that might have been in astronomer. Might be best to just say debian since the version will be updated from time to time.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk merged pull request #19855: Add information about supported OS-es for Apache Airflow
Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #19855:
URL: https://github.com/apache/airflow/pull/19855
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org