rabbah commented on a change in pull request #4216: Proposing Lean OpenWhisk
URL: https://github.com/apache/incubator-openwhisk/pull/4216#discussion_r253545570
##########
File path: ansible/roles/controller/tasks/deploy.yml
##########
@@ -6,14 +6,15 @@
- import_tasks: docker_login.yml
-- name: get controller name and index
+- name: get controller name, index and image name
set_fact:
controller_name: "{{ name_prefix ~ host_group.index(inventory_hostname) }}"
controller_index:
"{{ (controller_index_base|int) + host_group.index(inventory_hostname) }}"
+ image_name: "{{ image_name | default('controller') }}"
-- name: "pull the {{ docker.image.tag }} image of controller"
- shell: "docker pull {{docker_registry}}{{ docker.image.prefix }}/controller:{{docker.image.tag}}"
+- name: pull the {{ image_name }}:{{ docker.image.tag }} image of controller
+ shell: "docker pull {{docker_registry}}{{ docker.image.prefix }}/{{image_name}}:{{docker.image.tag}}"
Review comment:
this is just controller now no? and you can nuke prev task.
----------------------------------------------------------------
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
|