)]}'
{"playbooks/inventory/group_vars/all.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"13bed95775f915e31ebf974f5ec01f9121889075","unresolved":false,"context_lines":[{"line_number":348,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":349,"context_line":"nova_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"## Sahara"},{"line_number":352,"context_line":"sahara_rabbitmq_host_group: \"rabbitmq_all\""},{"line_number":353,"context_line":"sahara_rabbitmq_userid: sahara"},{"line_number":354,"context_line":"sahara_rabbitmq_vhost: /sahara"},{"line_number":355,"context_line":"sahara_rabbitmq_port: \"{{ rabbitmq_port }}\""},{"line_number":356,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":357,"context_line":"sahara_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"## Swift"},{"line_number":360,"context_line":"swift_system_user_name: swift"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1ac06dbe_cebfb2fa","line":357,"range":{"start_line":351,"start_character":0,"end_line":357,"end_character":116},"updated":"2016-08-25 17:52:13.000000000","message":"Is there a reason this needs to be in all.yml? Why not put them into sahara_all.yml? The only reason to have them in all.yml is if other services need to know this information.","commit_id":"c90e7fff3bb7be7be2a807bf99002d9a2b10d5ec"},{"author":{"_account_id":16310,"name":"Flávio","email":"framalho@suse.com","username":"flaviosr"},"change_message_id":"9b12087ed754cd61c9fe8303697768df7c08a679","unresolved":false,"context_lines":[{"line_number":348,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":349,"context_line":"nova_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"## Sahara"},{"line_number":352,"context_line":"sahara_rabbitmq_host_group: \"rabbitmq_all\""},{"line_number":353,"context_line":"sahara_rabbitmq_userid: sahara"},{"line_number":354,"context_line":"sahara_rabbitmq_vhost: /sahara"},{"line_number":355,"context_line":"sahara_rabbitmq_port: \"{{ rabbitmq_port }}\""},{"line_number":356,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":357,"context_line":"sahara_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"## Swift"},{"line_number":360,"context_line":"swift_system_user_name: swift"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1ac06dbe_b120d79c","line":357,"range":{"start_line":351,"start_character":0,"end_line":357,"end_character":116},"in_reply_to":"1ac06dbe_cebfb2fa","updated":"2016-08-25 18:11:33.000000000","message":"These variables are needed by both sahara and ceilometer roles, so I guess it is the place to put them.","commit_id":"c90e7fff3bb7be7be2a807bf99002d9a2b10d5ec"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"060b39f5048b4f4b77ba91db5606100003bd3b32","unresolved":false,"context_lines":[{"line_number":352,"context_line":"sahara_rabbitmq_host_group: \"rabbitmq_all\""},{"line_number":353,"context_line":"sahara_rabbitmq_userid: sahara"},{"line_number":354,"context_line":"sahara_rabbitmq_vhost: /sahara"},{"line_number":355,"context_line":"sahara_rabbitmq_port: \"{{ rabbitmq_port }}\""},{"line_number":356,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":357,"context_line":"sahara_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":358,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_6cefbde8","line":355,"updated":"2016-08-26 13:49:54.000000000","message":"What about the sahara_rabbitmq_telemetry_* vars?","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"},{"author":{"_account_id":16310,"name":"Flávio","email":"framalho@suse.com","username":"flaviosr"},"change_message_id":"3a3d346f88b465ae8d0723e6fe735d4d976b2980","unresolved":false,"context_lines":[{"line_number":352,"context_line":"sahara_rabbitmq_host_group: \"rabbitmq_all\""},{"line_number":353,"context_line":"sahara_rabbitmq_userid: sahara"},{"line_number":354,"context_line":"sahara_rabbitmq_vhost: /sahara"},{"line_number":355,"context_line":"sahara_rabbitmq_port: \"{{ rabbitmq_port }}\""},{"line_number":356,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":357,"context_line":"sahara_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":358,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_e7dbdbeb","line":355,"in_reply_to":"fa7ab95a_6cefbde8","updated":"2016-08-26 16:23:39.000000000","message":"The sahara role do not have support for service-specific rabbitmq, just submitted a patch adding it. In that way, the sahara_rabbitmq_telemetry_* vars have been added. Thanks for pointing that out !","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"060b39f5048b4f4b77ba91db5606100003bd3b32","unresolved":false,"context_lines":[{"line_number":354,"context_line":"sahara_rabbitmq_vhost: /sahara"},{"line_number":355,"context_line":"sahara_rabbitmq_port: \"{{ rabbitmq_port }}\""},{"line_number":356,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":357,"context_line":"sahara_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"## Swift"},{"line_number":360,"context_line":"swift_system_user_name: swift"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_6c189d1e","line":357,"updated":"2016-08-26 13:49:54.000000000","message":"This should also test whether or not sahara_all is defined and not empty since the os_ceilometer role will also rely upon this var.","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"},{"author":{"_account_id":16310,"name":"Flávio","email":"framalho@suse.com","username":"flaviosr"},"change_message_id":"3a3d346f88b465ae8d0723e6fe735d4d976b2980","unresolved":false,"context_lines":[{"line_number":354,"context_line":"sahara_rabbitmq_vhost: /sahara"},{"line_number":355,"context_line":"sahara_rabbitmq_port: \"{{ rabbitmq_port }}\""},{"line_number":356,"context_line":"# If there are any Ceilometer hosts in the environment, then enable its usage"},{"line_number":357,"context_line":"sahara_ceilometer_enabled: \"{{ (groups[\u0027ceilometer_all\u0027] is defined) and (groups[\u0027ceilometer_all\u0027] | length \u003e 0) }}\""},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"## Swift"},{"line_number":360,"context_line":"swift_system_user_name: swift"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_27fdc33a","line":357,"in_reply_to":"fa7ab95a_6c189d1e","updated":"2016-08-26 16:23:39.000000000","message":"Done","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"}],"playbooks/os-sahara-install.yml":[{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"060b39f5048b4f4b77ba91db5606100003bd3b32","unresolved":false,"context_lines":[{"line_number":14,"context_line":"- name: Install sahara server"},{"line_number":15,"context_line":"  hosts: sahara_all"},{"line_number":16,"context_line":"  gather_facts: \"{{ gather_facts | default(True) }}\""},{"line_number":17,"context_line":"  max_fail_percentage: 20"},{"line_number":18,"context_line":"  user: root"},{"line_number":19,"context_line":"  pre_tasks:"},{"line_number":20,"context_line":"    - include: common-tasks/rabbitmq-servers-sort.yml"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_2ca13595","line":17,"updated":"2016-08-26 13:49:54.000000000","message":"Are you really okay with partial success/partial failure. I\u0027d prefer to see the play abort if any task fails.","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"},{"author":{"_account_id":16310,"name":"Flávio","email":"framalho@suse.com","username":"flaviosr"},"change_message_id":"3a3d346f88b465ae8d0723e6fe735d4d976b2980","unresolved":false,"context_lines":[{"line_number":14,"context_line":"- name: Install sahara server"},{"line_number":15,"context_line":"  hosts: sahara_all"},{"line_number":16,"context_line":"  gather_facts: \"{{ gather_facts | default(True) }}\""},{"line_number":17,"context_line":"  max_fail_percentage: 20"},{"line_number":18,"context_line":"  user: root"},{"line_number":19,"context_line":"  pre_tasks:"},{"line_number":20,"context_line":"    - include: common-tasks/rabbitmq-servers-sort.yml"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_2759435b","line":17,"in_reply_to":"fa7ab95a_2ca13595","updated":"2016-08-26 16:23:39.000000000","message":"As most of the playbooks have this, I though it was some kind of pattern. Seems it is not.","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"060b39f5048b4f4b77ba91db5606100003bd3b32","unresolved":false,"context_lines":[{"line_number":48,"context_line":"      sahara_venv_download_url: \"{{ openstack_repo_url }}/venvs/{{ openstack_release }}/{{ ansible_distribution | lower }}/sahara-{{ openstack_release }}-{{ ansible_architecture | lower }}.tgz\""},{"line_number":49,"context_line":"      tags:"},{"line_number":50,"context_line":"        - \"os-sahara\""},{"line_number":51,"context_line":"    - { role: \"openstack_openrc\", tags: [ \"openstack-openrc\" ] }"},{"line_number":52,"context_line":"    - role: \"rsyslog_client\""},{"line_number":53,"context_line":"      rsyslog_client_log_rotate_file: sahara_log_rotate"},{"line_number":54,"context_line":"      rsyslog_client_log_dir: \"/var/log/sahara\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_0c40f1a4","line":51,"updated":"2016-08-26 13:49:54.000000000","message":"nit, but this should be the same yaml format as the other roles in this list","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"},{"author":{"_account_id":16310,"name":"Flávio","email":"framalho@suse.com","username":"flaviosr"},"change_message_id":"3a3d346f88b465ae8d0723e6fe735d4d976b2980","unresolved":false,"context_lines":[{"line_number":48,"context_line":"      sahara_venv_download_url: \"{{ openstack_repo_url }}/venvs/{{ openstack_release }}/{{ ansible_distribution | lower }}/sahara-{{ openstack_release }}-{{ ansible_architecture | lower }}.tgz\""},{"line_number":49,"context_line":"      tags:"},{"line_number":50,"context_line":"        - \"os-sahara\""},{"line_number":51,"context_line":"    - { role: \"openstack_openrc\", tags: [ \"openstack-openrc\" ] }"},{"line_number":52,"context_line":"    - role: \"rsyslog_client\""},{"line_number":53,"context_line":"      rsyslog_client_log_rotate_file: sahara_log_rotate"},{"line_number":54,"context_line":"      rsyslog_client_log_dir: \"/var/log/sahara\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fa7ab95a_87368f01","line":51,"in_reply_to":"fa7ab95a_0c40f1a4","updated":"2016-08-26 16:23:39.000000000","message":"Done","commit_id":"c536022da2a679753028b1d01cec5fbc3f1113a1"}],"releasenotes/notes/sahara-data-processing-service-8e63ebed6baf08bc.yaml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"13bed95775f915e31ebf974f5ec01f9121889075","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Experimental support has been added to allow the deployment of the"},{"line_number":4,"context_line":"    the Sahara data-processing service. To deploy sahara hosts should be"},{"line_number":5,"context_line":"    present in the host group ``sahara-infra_hosts``. "}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1ac06dbe_4e7c62eb","line":5,"range":{"start_line":5,"start_character":53,"end_line":5,"end_character":54},"updated":"2016-08-25 17:52:13.000000000","message":"remove the trailing whitespace","commit_id":"c90e7fff3bb7be7be2a807bf99002d9a2b10d5ec"},{"author":{"_account_id":16310,"name":"Flávio","email":"framalho@suse.com","username":"flaviosr"},"change_message_id":"9b12087ed754cd61c9fe8303697768df7c08a679","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Experimental support has been added to allow the deployment of the"},{"line_number":4,"context_line":"    the Sahara data-processing service. To deploy sahara hosts should be"},{"line_number":5,"context_line":"    present in the host group ``sahara-infra_hosts``. "}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1ac06dbe_b141f70d","line":5,"range":{"start_line":5,"start_character":53,"end_line":5,"end_character":54},"in_reply_to":"1ac06dbe_4e7c62eb","updated":"2016-08-25 18:11:33.000000000","message":"Done","commit_id":"c90e7fff3bb7be7be2a807bf99002d9a2b10d5ec"}],"tests/roles/bootstrap-host/tasks/prepare_aio_config.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"13bed95775f915e31ebf974f5ec01f9121889075","unresolved":false,"context_lines":[{"line_number":40,"context_line":"      override: \"{{ ceilometer_conf_overrides | default({}) }}\""},{"line_number":41,"context_line":"    - name: gnocchi.yml.aio"},{"line_number":42,"context_line":"      override: \"{{ gnocchi_conf_overrides | default({}) }}\""},{"line_number":43,"context_line":"    - name: sahara.yml.aio"},{"line_number":44,"context_line":"      override: \"{{ sahara_conf_overrides | default({}) }}\""},{"line_number":45,"context_line":"  tags:"},{"line_number":46,"context_line":"    - deploy-confd"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1ac06dbe_6e603e36","line":44,"range":{"start_line":43,"start_character":0,"end_line":44,"end_character":59},"updated":"2016-08-25 17:52:13.000000000","message":"This will force deployment into the AIO. Please don\u0027t do that right now - we\u0027re already struggling to complete a build within the 90 min window.\n\nWith everything else someone is able to add sahara to their own AIO easily by simply copying this file into conf.d themselves.","commit_id":"c90e7fff3bb7be7be2a807bf99002d9a2b10d5ec"},{"author":{"_account_id":16310,"name":"Flávio","email":"framalho@suse.com","username":"flaviosr"},"change_message_id":"9b12087ed754cd61c9fe8303697768df7c08a679","unresolved":false,"context_lines":[{"line_number":40,"context_line":"      override: \"{{ ceilometer_conf_overrides | default({}) }}\""},{"line_number":41,"context_line":"    - name: gnocchi.yml.aio"},{"line_number":42,"context_line":"      override: \"{{ gnocchi_conf_overrides | default({}) }}\""},{"line_number":43,"context_line":"    - name: sahara.yml.aio"},{"line_number":44,"context_line":"      override: \"{{ sahara_conf_overrides | default({}) }}\""},{"line_number":45,"context_line":"  tags:"},{"line_number":46,"context_line":"    - deploy-confd"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1ac06dbe_71e93f1a","line":44,"range":{"start_line":43,"start_character":0,"end_line":44,"end_character":59},"in_reply_to":"1ac06dbe_6e603e36","updated":"2016-08-25 18:11:33.000000000","message":"Sorry! Had no idea about that.","commit_id":"c90e7fff3bb7be7be2a807bf99002d9a2b10d5ec"}]}