[GitHub] markusthoemmes commented on a change in pull request #2248: Make invoker be scaled-out via ansible. (#2247)

[gi...@git.apache.org]

markusthoemmes commented on a change in pull request #2248: Make invoker be scaled-out via ansible. (#2247)
URL: https://github.com/apache/incubator-openwhisk/pull/2248#discussion_r120803093
 
 

 ##########
 File path: ansible/roles/invoker/tasks/deploy.yml
 ##########
 @@ -31,6 +31,19 @@
     linuxOptions: "-v /usr/lib/x86_64-linux-gnu/libapparmor.so.1:/usr/lib/x86_64-linux-gnu/libapparmor.so.1"
   when: whisk_version_name != "mac"
 
+- name: get running invoker information
+  uri: url="http://{{ inventory_hostname }}:{{ docker.port }}/containers/json?filters={{ '{"name":[ "invoker" ]}' | urlencode }}" return_content=yes
+  register: invokerInfo
+
+- name: determine if more than one invokers are running
+  fail: msg="more than one invokers are running"
+  when: invokerInfo.json|length > 1
+
+- name: determine if index of invoker is same with index of inventory host
+  fail: msg="invoker index is invalid {{item.Names[0]}} /invoker{{play_hosts.index(inventory_hostname)}}"
 
 Review comment:
   Could add: "invoker index is invalid. expected: /invoker{{play_hosts.index(inventory_hostname)}} found: {{item.Names[0]}}"
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services