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/09/16 20:44:17 UTC

[GitHub] [airflow] jedcunningham opened a new pull request #18306: Chart: warn when webserver secret key isn't set

jedcunningham opened a new pull request #18306:
URL: https://github.com/apache/airflow/pull/18306


   We will start warning chart admins when they deploy with the default
   random webserver secret key, as it can lead to unnecessary restarts of
   the Airflow components.
   
   Message on the dashboard for Admin users:
   ![Screen Shot 2021-09-16 at 2 35 24 PM](https://user-images.githubusercontent.com/66968678/133683010-51e43a0d-3e4d-4b17-ab03-c082791dd391.png)
   
   New section in production guide:
   ![Screen Shot 2021-09-16 at 2 34 59 PM](https://user-images.githubusercontent.com/66968678/133683067-8f2f495b-950f-4e42-a745-3dd7d1e29ec4.png)
   
   
   Related: #18284, #18098


-- 
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] kaxil closed pull request #18306: Chart: warn when webserver secret key isn't set

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


   


-- 
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 #18306: Chart: warn when webserver secret key isn't set

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #18306:
URL: https://github.com/apache/airflow/pull/18306#issuecomment-922562536


   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 modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase 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] jedcunningham closed pull request #18306: Chart: warn when webserver secret key isn't set

Posted by GitBox <gi...@apache.org>.
jedcunningham closed pull request #18306:
URL: https://github.com/apache/airflow/pull/18306


   


-- 
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] jedcunningham merged pull request #18306: Chart: warn when webserver secret key isn't set

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


   


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