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 2020/09/24 16:23:54 UTC

[GitHub] [airflow] potiuk opened a new pull request #11127: Fix gitSync user in the helm Chart

potiuk opened a new pull request #11127:
URL: https://github.com/apache/airflow/pull/11127


   There was a problem with user in Git Sync mode of the Helm Chart
   in connection with the git sync image and official Airflow
   image. Since we are using the official image, most of the
   containers are run with the "50000" user, but the git-sync image
   used by the git sync user is 65533 so we have to set it as
   default. We also exposed that value as parameter, so that
   another image could be used here as well.
   
   
   ---
   **^ Add meaningful description above**
   
   Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/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/master/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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] kaxil merged pull request #11127: Fix gitSync user in the helm Chart

Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #11127:
URL: https://github.com/apache/airflow/pull/11127


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk commented on pull request #11127: Fix gitSync user in the helm Chart

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #11127:
URL: https://github.com/apache/airflow/pull/11127#issuecomment-698449764


   cc: @jaketf 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] kaxil merged pull request #11127: Fix gitSync user in the helm Chart

Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #11127:
URL: https://github.com/apache/airflow/pull/11127


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk commented on pull request #11127: Fix gitSync user in the helm Chart

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #11127:
URL: https://github.com/apache/airflow/pull/11127#issuecomment-698449764


   cc: @jaketf 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org