)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"b6e2ad2432d2badea9a21f0d6090040d29d36892","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dai Dang Van \u003cdaidv@vn.fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-06-03 16:25:47 +0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Mixing binary and source images for T*, W*, Z* projects"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In case Kolla\u0027s users want to deploy with both of"},{"line_number":10,"context_line":"binary and source image, we should have a variable"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df140735_32996fc3","line":7,"updated":"2017-06-04 00:48:40.000000000","message":"T* W* and Z* is confused.","commit_id":"c039b3c385b659b5179b77b62281e79bce083c92"},{"author":{"_account_id":19930,"name":"Dai Dang Van","email":"daidv@omzcloud.vn","username":"daidv"},"change_message_id":"6400a93dd2bf9f1ac55081bcbd12d782eb2c648d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dai Dang Van \u003cdaidv@vn.fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-06-03 16:25:47 +0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Mixing binary and source images for T*, W*, Z* projects"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In case Kolla\u0027s users want to deploy with both of"},{"line_number":10,"context_line":"binary and source image, we should have a variable"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df140735_f28f77ca","line":7,"in_reply_to":"df140735_32996fc3","updated":"2017-06-04 03:02:41.000000000","message":"Humm, I just think we mix binary and source image for T*, W* and Z* project during deployment, not for T* services.\n\nCan you give me some advices about a better commit message title?\n\nVery thank you in advance.","commit_id":"c039b3c385b659b5179b77b62281e79bce083c92"}],"ansible/roles/watcher/defaults/main.yml":[{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"6fbded629f6653d6c0f5807eee5becf44bef275a","unresolved":false,"context_lines":[{"line_number":46,"context_line":"watcher_tag: \"{{ openstack_release }}\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"watcher_engine_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-engine\""},{"line_number":49,"context_line":"watcher_engine_tag: \"{{ watcher_tag }}\""},{"line_number":50,"context_line":"watcher_engine_image_full: \"{{ watcher_engine_image }}:{{ watcher_engine_tag }}\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"watcher_api_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-api\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df140735_ed42fbaa","line":49,"range":{"start_line":49,"start_character":1,"end_line":49,"end_character":21},"updated":"2017-06-03 13:42:44.000000000","message":"And when watcher_engine_tag is different from watch_api_image, how do you realize it?","commit_id":"c039b3c385b659b5179b77b62281e79bce083c92"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"17401043f0361b6f089925de77f5d47659ed3a4e","unresolved":false,"context_lines":[{"line_number":46,"context_line":"watcher_tag: \"{{ openstack_release }}\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"watcher_engine_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-engine\""},{"line_number":49,"context_line":"watcher_engine_tag: \"{{ watcher_tag }}\""},{"line_number":50,"context_line":"watcher_engine_image_full: \"{{ watcher_engine_image }}:{{ watcher_engine_tag }}\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"watcher_api_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-api\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df140735_8d325f5d","line":49,"range":{"start_line":49,"start_character":0,"end_line":49,"end_character":39},"updated":"2017-06-03 13:40:23.000000000","message":"Usually, we change the openstack_release in /etc/kolla/global.yml, and the default vaule is defined in group_vars/all.yml,  but i think it not a good idea to change the roles/{project}/defauls/main.yml","commit_id":"c039b3c385b659b5179b77b62281e79bce083c92"},{"author":{"_account_id":19930,"name":"Dai Dang Van","email":"daidv@omzcloud.vn","username":"daidv"},"change_message_id":"a5bc541731e9823317b2a662c2b82316df90ead2","unresolved":false,"context_lines":[{"line_number":46,"context_line":"watcher_tag: \"{{ openstack_release }}\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"watcher_engine_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-engine\""},{"line_number":49,"context_line":"watcher_engine_tag: \"{{ watcher_tag }}\""},{"line_number":50,"context_line":"watcher_engine_image_full: \"{{ watcher_engine_image }}:{{ watcher_engine_tag }}\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"watcher_api_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-api\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df140735_0d908fec","line":49,"range":{"start_line":49,"start_character":1,"end_line":49,"end_character":21},"in_reply_to":"df140735_4ddc47e3","updated":"2017-06-03 14:07:34.000000000","message":"And meanwhile all of projects are using openstack_release tag, you can config for watcher deployment with special tag(for all service of watcher) by watcher_tag instead of watcher_engine_tag, watcher_api_tag and watcher_applier_tag.","commit_id":"c039b3c385b659b5179b77b62281e79bce083c92"},{"author":{"_account_id":19930,"name":"Dai Dang Van","email":"daidv@omzcloud.vn","username":"daidv"},"change_message_id":"3ba0ce23af8f91a870607b731e8e5ed81ff85f76","unresolved":false,"context_lines":[{"line_number":46,"context_line":"watcher_tag: \"{{ openstack_release }}\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"watcher_engine_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-engine\""},{"line_number":49,"context_line":"watcher_engine_tag: \"{{ watcher_tag }}\""},{"line_number":50,"context_line":"watcher_engine_image_full: \"{{ watcher_engine_image }}:{{ watcher_engine_tag }}\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"watcher_api_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-api\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df140735_6d5eeb76","line":49,"range":{"start_line":49,"start_character":0,"end_line":49,"end_character":39},"in_reply_to":"df140735_8d325f5d","updated":"2017-06-03 13:59:54.000000000","message":"I\u0027m on my way to implement my blueprint mixing-binary-and-source-image that we just push two new variables for each project to do:\n1. mixing install type\n2. get more flexible during deployment with special tag for each project.\n\nDo you want to push all of them into all.yml?","commit_id":"c039b3c385b659b5179b77b62281e79bce083c92"},{"author":{"_account_id":19930,"name":"Dai Dang Van","email":"daidv@omzcloud.vn","username":"daidv"},"change_message_id":"3ba0ce23af8f91a870607b731e8e5ed81ff85f76","unresolved":false,"context_lines":[{"line_number":46,"context_line":"watcher_tag: \"{{ openstack_release }}\""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"watcher_engine_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-engine\""},{"line_number":49,"context_line":"watcher_engine_tag: \"{{ watcher_tag }}\""},{"line_number":50,"context_line":"watcher_engine_image_full: \"{{ watcher_engine_image }}:{{ watcher_engine_tag }}\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"watcher_api_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ watcher_install_type }}-watcher-api\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df140735_4ddc47e3","line":49,"range":{"start_line":49,"start_character":1,"end_line":49,"end_character":21},"in_reply_to":"df140735_ed42fbaa","updated":"2017-06-03 13:59:54.000000000","message":"And when we want to have a different tag for engine vs api, we will add each of them to global.yml config file.\nBtw, by default, we will get the same tag (opestack_release) for  all service for each project.","commit_id":"c039b3c385b659b5179b77b62281e79bce083c92"}]}