You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openwhisk.apache.org by "style95 (via GitHub)" <gi...@apache.org> on 2023/05/31 13:27:18 UTC

[GitHub] [openwhisk] style95 commented on a diff in pull request #5420: Enable multiple controllers

style95 commented on code in PR #5420:
URL: https://github.com/apache/openwhisk/pull/5420#discussion_r1211718536


##########
ansible/environments/local/hosts.j2.ini:
##########
@@ -8,9 +8,9 @@ ansible ansible_connection=local
 
 [controllers]
 controller0         ansible_host=172.17.0.1 ansible_connection=local
-;{% if mode is defined and 'HA' in mode %}
-;controller1         ansible_host=172.17.0.1 ansible_connection=local
-;{% endif %}
+{% if mode is defined and 'HA' in mode %}
+controller1         ansible_host=172.17.0.1 ansible_connection=local

Review Comment:
   This is to enable the second controller in the CI environment to run [the HA test for controllers](https://github.com/apache/openwhisk/blob/master/tests/src/test/scala/ha/ShootComponentsTests.scala#L145).
   



-- 
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: issues-unsubscribe@openwhisk.apache.org

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