jedcunningham commented on a change in pull request #20199:
URL: https://github.com/apache/airflow/pull/20199#discussion_r769873047



##########
File path: chart/UPDATING.rst
##########
@@ -43,6 +43,11 @@ Default Airflow image is updated to ``2.2.2``
 
 The default Airflow image that is used with the Chart is now ``2.2.2``, 
previously it was ``2.2.1``.
 
+Changes in webserver and flower ``NetworkPolicy``
+"""""""""""""""""""""""""""""""""""""""""""""""""

Review comment:
       ```suggestion
   Changes in webserver and flower ``NetworkPolicy`` default ports
   """""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""
   ```

##########
File path: chart/UPDATING.rst
##########
@@ -43,6 +43,11 @@ Default Airflow image is updated to ``2.2.2``
 
 The default Airflow image that is used with the Chart is now ``2.2.2``, 
previously it was ``2.2.1``.
 
+Changes in webserver and flower ``NetworkPolicy``
+"""""""""""""""""""""""""""""""""""""""""""""""""
+
+``webserver.networkPolicy.ingress.ports`` and 
``flower.networkPolicy.ingress.ports`` moved away from named port to port 
number to avoid issues with OpenShift.

Review comment:
       ```suggestion
   The defaults for ``webserver.networkPolicy.ingress.ports`` and 
``flower.networkPolicy.ingress.ports`` moved away from using named ports to 
numerical ports to avoid issues with OpenShift.
   ```




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to