)]}'
{"playbooks/os-aodh-install.yml":[{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"9594b656b47271879691d9680428750a5944ef0b","unresolved":false,"context_lines":[{"line_number":66,"context_line":"      rabbitmq_userid: \"{{ aodh_rabbitmq_userid }}\""},{"line_number":67,"context_line":"      rabbitmq_password: \"{{ aodh_rabbitmq_password }}\""},{"line_number":68,"context_line":"      rabbitmq_delegate_to: \"{{ groups[\u0027rabbitmq_all\u0027][0] }}\""},{"line_number":69,"context_line":"      when:"},{"line_number":70,"context_line":"        - \"inventory_hostname \u003d\u003d groups[\u0027aodh_all\u0027][0]\""},{"line_number":71,"context_line":"    - role: \"os_aodh\""},{"line_number":72,"context_line":"      aodh_venv_tag: \"{{ openstack_release }}\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7a5de9d1_ad6dc805","line":69,"updated":"2016-01-28 19:42:18.000000000","message":"I would prefer to switch to the pattern of  \"run_once: true\" unless there is a good reason not to.\n\nThis recommendation applies to all playbooks in this ps","commit_id":"fd3a15337289d8a81b5815d9e23a4d362b32ddc1"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"f8c43441484a3b7a5aa576d3cf1f85fbde22d509","unresolved":false,"context_lines":[{"line_number":66,"context_line":"      rabbitmq_userid: \"{{ aodh_rabbitmq_userid }}\""},{"line_number":67,"context_line":"      rabbitmq_password: \"{{ aodh_rabbitmq_password }}\""},{"line_number":68,"context_line":"      rabbitmq_delegate_to: \"{{ groups[\u0027rabbitmq_all\u0027][0] }}\""},{"line_number":69,"context_line":"      when:"},{"line_number":70,"context_line":"        - \"inventory_hostname \u003d\u003d groups[\u0027aodh_all\u0027][0]\""},{"line_number":71,"context_line":"    - role: \"os_aodh\""},{"line_number":72,"context_line":"      aodh_venv_tag: \"{{ openstack_release }}\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7a5de9d1_4a3ac5cd","line":69,"in_reply_to":"7a5de9d1_27578259","updated":"2016-01-29 16:36:00.000000000","message":"On second thought IDK if the \"run_once\" command has any real benefit when I review the docs on how it works. http://docs.ansible.com/ansible/playbooks_delegation.html#run-once -- that said, i\u0027d be happy to experiment with the option to see if its a better option.","commit_id":"fd3a15337289d8a81b5815d9e23a4d362b32ddc1"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"fb6b6b7e59de669e5c7ac2c2dc25f2cb72603701","unresolved":false,"context_lines":[{"line_number":66,"context_line":"      rabbitmq_userid: \"{{ aodh_rabbitmq_userid }}\""},{"line_number":67,"context_line":"      rabbitmq_password: \"{{ aodh_rabbitmq_password }}\""},{"line_number":68,"context_line":"      rabbitmq_delegate_to: \"{{ groups[\u0027rabbitmq_all\u0027][0] }}\""},{"line_number":69,"context_line":"      when:"},{"line_number":70,"context_line":"        - \"inventory_hostname \u003d\u003d groups[\u0027aodh_all\u0027][0]\""},{"line_number":71,"context_line":"    - role: \"os_aodh\""},{"line_number":72,"context_line":"      aodh_venv_tag: \"{{ openstack_release }}\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7a5de9d1_27578259","line":69,"in_reply_to":"7a5de9d1_ad6dc805","updated":"2016-01-29 16:30:29.000000000","message":"this is a great idea. Ill pull that through on the next update to the WIP.","commit_id":"fd3a15337289d8a81b5815d9e23a4d362b32ddc1"}],"playbooks/os-cinder-install.yml":[{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"61f8035b78c4bf8260d78d556f23713d255f29c8","unresolved":false,"context_lines":[{"line_number":194,"context_line":"      tags:"},{"line_number":195,"context_line":"        - \"system-crontab-coordination\""},{"line_number":196,"context_line":"  vars:"},{"line_number":197,"context_line":"    galera_address: \"{{ internal_lb_vip_address }}\""},{"line_number":198,"context_line":"    ansible_hostname: \"{{ container_name }}\""},{"line_number":199,"context_line":"    is_metal: \"{{ properties.is_metal|default(false) }}\""},{"line_number":200,"context_line":"    cinder_galera_user: cinder"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7a5de9d1_b2752499","line":197,"updated":"2016-01-28 19:56:24.000000000","message":"any thoughts on using vars_files at this point? (pointing to playbooks/vars/...)","commit_id":"fd3a15337289d8a81b5815d9e23a4d362b32ddc1"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"fb6b6b7e59de669e5c7ac2c2dc25f2cb72603701","unresolved":false,"context_lines":[{"line_number":194,"context_line":"      tags:"},{"line_number":195,"context_line":"        - \"system-crontab-coordination\""},{"line_number":196,"context_line":"  vars:"},{"line_number":197,"context_line":"    galera_address: \"{{ internal_lb_vip_address }}\""},{"line_number":198,"context_line":"    ansible_hostname: \"{{ container_name }}\""},{"line_number":199,"context_line":"    is_metal: \"{{ properties.is_metal|default(false) }}\""},{"line_number":200,"context_line":"    cinder_galera_user: cinder"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7a5de9d1_476dde2f","line":197,"in_reply_to":"7a5de9d1_b2752499","updated":"2016-01-29 16:30:29.000000000","message":"I guess we could use a var file however i\u0027d be hesitant of using a var-file here because of the precedence impacts var-files have when a user attempts to override a pre-defined variable. That said, I\u0027m open to giving it a go.","commit_id":"fd3a15337289d8a81b5815d9e23a4d362b32ddc1"}]}