)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"95aba313391290f08ea67931a1cb4ef99d469ecb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Donovan Francesco \u003cdonovan.francesco@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-08 17:24:11 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add Magnum support to build repository."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds Magnum support to the build repository"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I32dee168d1005572510f630a21f7d7a7a05640d9"},{"line_number":12,"context_line":"Implements: blueprint role-magnum"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9ad45d7e_4e1dd10e","line":9,"range":{"start_line":7,"start_character":0,"end_line":9,"end_character":54},"updated":"2016-08-08 15:37:35.000000000","message":"Update the commit message to cover the scope of the patch.","commit_id":"cb7cf70acd54844edcdc935931701a435c1518e5"},{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"dcfc887a5adcc89a8063f3d96fd7384a3e4af32a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Donovan Francesco \u003cdonovan.francesco@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-08 19:59:55 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding support for Magnum to be built as part of the repo build process."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In addition to adding the magnum role to the repo build process we also"},{"line_number":10,"context_line":"include the os-magnum-install.yml to deploy magnum to hosts tagged with"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"9ad45d7e_8c54ec54","line":7,"range":{"start_line":7,"start_character":26,"end_line":7,"end_character":71},"updated":"2016-08-08 18:02:44.000000000","message":"Remove this part of the title line, or refactor to be more appropriate to the patch set.","commit_id":"fa171c75498c5e1a6e949120fa7c40882713a192"}],"ansible-role-requirements.yml":[{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"b03fc8636a08906c1b3c609bd8ac683218ef1261","unresolved":false,"context_lines":[{"line_number":80,"context_line":"  version: master"},{"line_number":81,"context_line":"- name: os_magnum"},{"line_number":82,"context_line":"  scm: git"},{"line_number":83,"context_line":"  src: https://github.com/openstack/openstack-ansible-os_magnum"},{"line_number":84,"context_line":"  version: master"},{"line_number":85,"context_line":"- name: os_neutron"},{"line_number":86,"context_line":"  scm: git"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9ad45d7e_73135e5d","line":83,"range":{"start_line":83,"start_character":7,"end_line":83,"end_character":63},"updated":"2016-08-08 15:13:00.000000000","message":"should be git.openstack.org, not githuh.com","commit_id":"9ddef6b8f04826666129ac2347bf7abfd77f4886"}],"doc/source/index.rst":[{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"678933adf597b2cd0b5f5cbf2381c0772c8f79e3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"OpenStack-Ansible Magnum"},{"line_number":2,"context_line":"##########################"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"Ansible role that installs and configures OpenStack Magnum. Magnum is"}],"source_content_type":"text/x-rst","patch_set":10,"id":"9ad45d7e_23d86e53","line":1,"updated":"2016-08-08 18:40:56.000000000","message":"Not sure why this is here, but it should be in a patch on openstack-ansible-os_magnum if you are looking to change the os_magnum role doc","commit_id":"f492d40e462d874276c24ee40880cfb8dc9547e7"}],"playbooks/inventory/env.d/magnum.yml":[{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"b03fc8636a08906c1b3c609bd8ac683218ef1261","unresolved":false,"context_lines":[{"line_number":30,"context_line":"      - magnum_api\r"},{"line_number":31,"context_line":"    properties:\r"},{"line_number":32,"context_line":"      service_name: magnum\r"},{"line_number":33,"context_line":"      container_release: trusty\r"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9ad45d7e_286df504","line":33,"range":{"start_line":33,"start_character":6,"end_line":33,"end_character":31},"updated":"2016-08-08 15:13:00.000000000","message":"Container release is not longer used.  Is this still mentioned in the docs somewhere?","commit_id":"9ddef6b8f04826666129ac2347bf7abfd77f4886"},{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"95aba313391290f08ea67931a1cb4ef99d469ecb","unresolved":false,"context_lines":[{"line_number":24,"context_line":"container_skel:\r"},{"line_number":25,"context_line":"  magnum_container:\r"},{"line_number":26,"context_line":"    belongs_to:\r"},{"line_number":27,"context_line":"      - infra_containers\r"},{"line_number":28,"context_line":"      - os-infra_containers\r"},{"line_number":29,"context_line":"    contains:\r"},{"line_number":30,"context_line":"      - magnum_api\r"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9ad45d7e_2efe9d6b","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":24},"updated":"2016-08-08 15:37:35.000000000","message":"Remove this.","commit_id":"cb7cf70acd54844edcdc935931701a435c1518e5"},{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"5f9240ded70df04956d0ffc7aac4d522be96b407","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#\r"},{"line_number":16,"context_line":"# (c) 2016 Donovan Francesco \u003cdonovan.francesco@is.co.za\u003e\r"},{"line_number":17,"context_line":"# (c) 2016 Paul Stevens \u003cpaul.stevens@is.co.za\u003e\r"},{"line_number":18,"context_line":"#\r"},{"line_number":19,"context_line":"component_skel:\r"},{"line_number":20,"context_line":"  magnum_api:\r"},{"line_number":21,"context_line":"    belongs_to:\r"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"9ad45d7e_e71663a6","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":1},"updated":"2016-08-08 18:24:21.000000000","message":"delete, should be an empty line here.","commit_id":"f47a34c2c3baca6f471da2fcd8363ead436c5571"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"66516c4041dc89fc76a58266dea2efa84a34082a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---\r"},{"line_number":2,"context_line":"# Copyright 2016 Internet Solutions (Pty) Ltd\r"},{"line_number":3,"context_line":"#\r"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");\r"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"9ad45d7e_ac3c4b30","line":1,"updated":"2016-08-08 19:52:43.000000000","message":"Could we get UNIX line feeds (\\n) in this file?","commit_id":"89ebe64d8f3a3240f03b984f358cbfe9c3caf9d7"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"82c0928198eba526e9b46d98009c531315ab0c88","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"container_skel:"},{"line_number":25,"context_line":"  magnum_container:"},{"line_number":26,"context_line":"    belongs_to:"},{"line_number":27,"context_line":"      - os-infra_containers"},{"line_number":28,"context_line":"    contains:"},{"line_number":29,"context_line":"      - magnum_api"},{"line_number":30,"context_line":"    properties:"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"9ad45d7e_0400ac00","line":27,"range":{"start_line":26,"start_character":0,"end_line":27,"end_character":27},"updated":"2016-08-09 11:54:37.000000000","message":"This will automatically deploy Magnum to all environments, which I don\u0027t think is appropriate. This should be an opt-in feature and it should therefore have its own group.\n\nIt would seem that this is missing a host group too - ie magnum could be deployed to containers or hosts.\n\nAlso, is only an API deployed?\n\nSee https://github.com/openstack/openstack-ansible/blob/master/playbooks/inventory/env.d/ironic.yml for a more comprehensive set of group specification.","commit_id":"623c5b7f39ab7f241f2fea6e53ef3930ad5ef4b6"}],"playbooks/inventory/group_vars/magnum_all.yml":[{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"b03fc8636a08906c1b3c609bd8ac683218ef1261","unresolved":false,"context_lines":[{"line_number":1,"context_line":"## Magnum\r"},{"line_number":2,"context_line":"magnum_service_port: 9511\r"},{"line_number":3,"context_line":"magnum_service_proto: http\r"},{"line_number":4,"context_line":"magnum_service_publicuri_proto: \"{{ openstack_service_adminuri_proto | default(magnum_service_proto) }}\"\r"},{"line_number":5,"context_line":"magnum_service_internaluri_proto: \"{{ openstack_service_adminuri_proto | default(magnum_service_proto) }}\"\r"},{"line_number":6,"context_line":"magnum_service_adminuri_proto: \"{{ openstack_service_adminuri_proto | default(magnum_service_proto) }}\"\r"},{"line_number":7,"context_line":"magnum_service_user_name: magnum\r"},{"line_number":8,"context_line":"magnum_service_project_name: service\r"},{"line_number":9,"context_line":"magnum_service_project_domain_id: default\r"},{"line_number":10,"context_line":"magnum_service_user_domain_id: default\r"},{"line_number":11,"context_line":"magnum_service_adminuri: \"{{ magnum_service_adminuri_proto }}://{{ internal_lb_vip_address }}:{{ magnum_service_port }}\"\r"},{"line_number":12,"context_line":"magnum_service_adminurl: \"{{ magnum_service_adminuri }}\"\r"},{"line_number":13,"context_line":"magnum_service_region: \"{{ service_region }}\"\r"},{"line_number":14,"context_line":"magnum_rabbitmq_userid: magnum\r"},{"line_number":15,"context_line":"magnum_rabbitmq_vhost: /magnum\r"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9ad45d7e_28121563","line":15,"range":{"start_line":1,"start_character":0,"end_line":15,"end_character":30},"updated":"2016-08-08 15:13:00.000000000","message":"These look like they can be covered by role defaults, I don\u0027t think we necessarily need this file.","commit_id":"9ddef6b8f04826666129ac2347bf7abfd77f4886"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"d04624e8f39a09e2b218fc3039899bfcf7b93bc2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---\r"},{"line_number":2,"context_line":"# Copyright 2016 Internet Solutions (Pty) Ltd\r"},{"line_number":3,"context_line":"#\r"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");\r"},{"line_number":5,"context_line":"# you may not use this file except in compliance with the License.\r"},{"line_number":6,"context_line":"# You may obtain a copy of the License at\r"},{"line_number":7,"context_line":"#\r"},{"line_number":8,"context_line":"#     http://www.apache.org/licenses/LICENSE-2.0\r"},{"line_number":9,"context_line":"#\r"},{"line_number":10,"context_line":"# Unless required by applicable law or agreed to in writing, software\r"},{"line_number":11,"context_line":"# distributed under the License is distributed on an \"AS IS\" BASIS,\r"},{"line_number":12,"context_line":"# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r"},{"line_number":13,"context_line":"# See the License for the specific language governing permissions and\r"},{"line_number":14,"context_line":"# limitations under the License.\r"},{"line_number":15,"context_line":"\r"},{"line_number":16,"context_line":"magnum_galera_user: magnum\r"},{"line_number":17,"context_line":"magnum_galera_database_name: magnum_service\r"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"9ad45d7e_2b69258c","line":14,"range":{"start_line":1,"start_character":0,"end_line":14,"end_character":32},"updated":"2016-08-09 18:41:09.000000000","message":"Please add this license boilerplate back.","commit_id":"623c5b7f39ab7f241f2fea6e53ef3930ad5ef4b6"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"d04624e8f39a09e2b218fc3039899bfcf7b93bc2","unresolved":false,"context_lines":[{"line_number":15,"context_line":"\r"},{"line_number":16,"context_line":"magnum_galera_user: magnum\r"},{"line_number":17,"context_line":"magnum_galera_database_name: magnum_service\r"},{"line_number":18,"context_line":"magnum_galera_address: \"{{ galera_address }}\"\r"},{"line_number":19,"context_line":"\r"},{"line_number":20,"context_line":"magnum_rabbitmq_userid: magnum\r"},{"line_number":21,"context_line":"magnum_rabbitmq_vhost: /magnum\r"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"9ad45d7e_ebf54dcb","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":45},"updated":"2016-08-09 18:41:09.000000000","message":"Please ensure that this is added back to ensure that the MAriaDB address used is consistent.","commit_id":"623c5b7f39ab7f241f2fea6e53ef3930ad5ef4b6"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"d04624e8f39a09e2b218fc3039899bfcf7b93bc2","unresolved":false,"context_lines":[{"line_number":19,"context_line":"\r"},{"line_number":20,"context_line":"magnum_rabbitmq_userid: magnum\r"},{"line_number":21,"context_line":"magnum_rabbitmq_vhost: /magnum\r"},{"line_number":22,"context_line":"magnum_rabbitmq_port: \"{{ rabbitmq_port }}\"\r"},{"line_number":23,"context_line":"magnum_rabbitmq_servers: \"{{ rabbitmq_servers }}\"\r"},{"line_number":24,"context_line":"magnum_rabbitmq_use_ssl: \"{{ rabbitmq_use_ssl }}\"\r"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"9ad45d7e_ab13550c","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":49},"updated":"2016-08-09 18:41:09.000000000","message":"Please ensure that these vars are implemented in group_vars to ensure that the rabbit servers and config are consistent.","commit_id":"623c5b7f39ab7f241f2fea6e53ef3930ad5ef4b6"}],"playbooks/os-magnum-install.yml":[{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"b03fc8636a08906c1b3c609bd8ac683218ef1261","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Copyright 2016, Ian Cordasco"},{"line_number":3,"context_line":"# Copyright 2016, Rackspace"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":6,"context_line":"# you may not use this file except in compliance with the License."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9ad45d7e_73419e39","line":3,"range":{"start_line":2,"start_character":0,"end_line":3,"end_character":27},"updated":"2016-08-08 15:13:00.000000000","message":"Should only be one copyright line.","commit_id":"9ddef6b8f04826666129ac2347bf7abfd77f4886"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"9f8e5347ee5b86d3e02f4ea083f3531d086b9cc4","unresolved":false,"context_lines":[{"line_number":52,"context_line":"    - name: Set rabbitmq servers"},{"line_number":53,"context_line":"      set_fact:"},{"line_number":54,"context_line":"        rabbitmq_servers: \"{{ servers.sorted_list }}\""},{"line_number":55,"context_line":"    - name: Ensure the Magnum RabbitMQ virtualhost exists"},{"line_number":56,"context_line":"      rabbitmq_vhost:"},{"line_number":57,"context_line":"        name: \"{{ magnum_rabbitmq_vhost }}\""},{"line_number":58,"context_line":"        state: \"present\""}],"source_content_type":"text/x-yaml","patch_set":11,"id":"9ad45d7e_a63c6c30","line":55,"updated":"2016-08-08 19:08:12.000000000","message":"This should use the common_tasks includes","commit_id":"82cf21c42b1a2f081a1605682671add41cb62c36"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"cd92bb104a46baf87b1407bed0733863877cd350","unresolved":false,"context_lines":[{"line_number":60,"context_line":"      tags:"},{"line_number":61,"context_line":"        - \"system-crontab-coordination\""},{"line_number":62,"context_line":"  vars:"},{"line_number":63,"context_line":"    magnum_galera_address: \"{{ internal_lb_vip_address }}\""},{"line_number":64,"context_line":"    magnum_galera_database_name: magnum_service"},{"line_number":65,"context_line":"    magnum_galera_user: magnum"},{"line_number":66,"context_line":"    magnum_rabbitmq_userid: magnum"},{"line_number":67,"context_line":"    magnum_rabbitmq_vhost: /magnum"},{"line_number":68,"context_line":"    ansible_hostname: \"{{ container_name }}\""},{"line_number":69,"context_line":"    is_metal: \"{{ properties.is_metal|default(false) }}\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"9ad45d7e_ec063e1f","line":67,"range":{"start_line":63,"start_character":0,"end_line":67,"end_character":34},"updated":"2016-08-08 22:09:01.000000000","message":"These should be moved to group_vars once this merges.","commit_id":"541d89c9694edc47ab476069f3a11b26535505f2"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"cd92bb104a46baf87b1407bed0733863877cd350","unresolved":false,"context_lines":[{"line_number":65,"context_line":"    magnum_galera_user: magnum"},{"line_number":66,"context_line":"    magnum_rabbitmq_userid: magnum"},{"line_number":67,"context_line":"    magnum_rabbitmq_vhost: /magnum"},{"line_number":68,"context_line":"    ansible_hostname: \"{{ container_name }}\""},{"line_number":69,"context_line":"    is_metal: \"{{ properties.is_metal|default(false) }}\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"9ad45d7e_4f0e5c35","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":44},"updated":"2016-08-08 22:09:01.000000000","message":"I\u0027m not entirely sure that this is needed.","commit_id":"541d89c9694edc47ab476069f3a11b26535505f2"}],"playbooks/vars/configs/haproxy_config.yml":[{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"b03fc8636a08906c1b3c609bd8ac683218ef1261","unresolved":false,"context_lines":[{"line_number":242,"context_line":"      haproxy_balance_type: http"},{"line_number":243,"context_line":"      haproxy_balance_alg: leastconn"},{"line_number":244,"context_line":"      haproxy_backend_options:"},{"line_number":245,"context_line":"        - \"forwardfor\""},{"line_number":246,"context_line":"        - \"httpchks\""},{"line_number":247,"context_line":"        - \"httplog\""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9ad45d7e_1314a2f3","line":247,"range":{"start_line":245,"start_character":0,"end_line":247,"end_character":19},"updated":"2016-08-08 15:13:00.000000000","message":"These options don\u0027t seem to be used on other services here.  Any particular reason for including them on this service?","commit_id":"9ddef6b8f04826666129ac2347bf7abfd77f4886"},{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"95aba313391290f08ea67931a1cb4ef99d469ecb","unresolved":false,"context_lines":[{"line_number":235,"context_line":"        - 192.168.0.0/16"},{"line_number":236,"context_line":"        - 172.16.0.0/12"},{"line_number":237,"context_line":"        - 10.0.0.0/8"},{"line_number":238,"context_line":"  - service:"},{"line_number":239,"context_line":"      haproxy_service_name: magnum"},{"line_number":240,"context_line":"      haproxy_backend_nodes: \"{{ groups[\u0027magnum_all\u0027] | default([]) }}\""},{"line_number":241,"context_line":"      haproxy_port: 9511"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9ad45d7e_ce99c121","line":238,"range":{"start_line":238,"start_character":2,"end_line":238,"end_character":12},"updated":"2016-08-08 15:37:35.000000000","message":"looks like other services are using haproxy_ssl: \"{{ haproxy_ssl }}\"\n\nWould that be appropriate here?","commit_id":"cb7cf70acd54844edcdc935931701a435c1518e5"},{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"95aba313391290f08ea67931a1cb4ef99d469ecb","unresolved":false,"context_lines":[{"line_number":240,"context_line":"      haproxy_backend_nodes: \"{{ groups[\u0027magnum_all\u0027] | default([]) }}\""},{"line_number":241,"context_line":"      haproxy_port: 9511"},{"line_number":242,"context_line":"      haproxy_balance_type: http"},{"line_number":243,"context_line":"      haproxy_balance_alg: leastconn"},{"line_number":244,"context_line":"      haproxy_backend_options:"},{"line_number":245,"context_line":"        - \"httpchk HEAD /\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9ad45d7e_6e541554","line":243,"range":{"start_line":243,"start_character":6,"end_line":243,"end_character":36},"updated":"2016-08-08 15:37:35.000000000","message":"Do we need this here?","commit_id":"cb7cf70acd54844edcdc935931701a435c1518e5"}],"releasenotes/notes/add-magnum-to-repo-548f243b3a253b04.yaml":[{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"b03fc8636a08906c1b3c609bd8ac683218ef1261","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Adding support for Magnum to be built as part of the"},{"line_number":4,"context_line":"    repo build process. "}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9ad45d7e_73f99ec8","line":4,"range":{"start_line":4,"start_character":23,"end_line":4,"end_character":24},"updated":"2016-08-08 15:13:00.000000000","message":"Extra white space.","commit_id":"9ddef6b8f04826666129ac2347bf7abfd77f4886"},{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"b03fc8636a08906c1b3c609bd8ac683218ef1261","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Adding support for Magnum to be built as part of the"},{"line_number":4,"context_line":"    repo build process. "}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9ad45d7e_132b0232","line":4,"range":{"start_line":3,"start_character":2,"end_line":4,"end_character":22},"updated":"2016-08-08 15:13:00.000000000","message":"The scope of this patch is larger than just this.  You\u0027re also adding a play to deploy magnum.","commit_id":"9ddef6b8f04826666129ac2347bf7abfd77f4886"},{"author":{"_account_id":20038,"name":"Michael Gugino","email":"michael.gugino@walmart.com","username":"michaelgugino"},"change_message_id":"95aba313391290f08ea67931a1cb4ef99d469ecb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Adding support for Magnum to be built as part of the repo build process."},{"line_number":4,"context_line":"    In addition to adding the magnum role to the repo build process we also"},{"line_number":5,"context_line":"    include the os-magnum-install.yml to deploy magnum to hosts tagged with"},{"line_number":6,"context_line":"    os_magnum. "}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9ad45d7e_2e269d68","line":6,"range":{"start_line":4,"start_character":3,"end_line":6,"end_character":13},"updated":"2016-08-08 15:37:35.000000000","message":"Make this another item in the list.  IE:\n- Adding play to install magnum, os-magnum-install.yml","commit_id":"cb7cf70acd54844edcdc935931701a435c1518e5"},{"author":{"_account_id":15993,"name":"Amy Marrich","display_name":"Amy Marrich (spotz)","email":"amy@demarco.com","username":"amarrich"},"change_message_id":"146691cdb3fd3a86b3244de2bef8d4246c1169d0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Adding support for Magnum to be built as part of the repo build process."},{"line_number":4,"context_line":"  - In addition to adding the magnum role to the repo build process we also"},{"line_number":5,"context_line":"    include the os-magnum-install.yml to deploy magnum to hosts tagged with"},{"line_number":6,"context_line":"    os_magnum."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9ad45d7e_45d64fc6","line":6,"range":{"start_line":4,"start_character":4,"end_line":6,"end_character":14},"updated":"2016-08-08 16:14:38.000000000","message":"Adding the os-magnum-install.yml file to deploy magnum to hosts tagged with os_magnum.","commit_id":"f43a216bf13839b3446fc1cd68fdf7a357893179"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"66516c4041dc89fc76a58266dea2efa84a34082a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Adding support for Magnum to be built as part of the repo build process."},{"line_number":4,"context_line":"  - Adding the os-magnum-install.yml file to deploy magnum to hosts tagged"},{"line_number":5,"context_line":"    with hostgroup magnum_all."}],"source_content_type":"text/x-yaml","patch_set":12,"id":"9ad45d7e_acce0b58","line":4,"updated":"2016-08-08 19:52:43.000000000","message":"Make this monospaced: ``os-magnum-install.yml``","commit_id":"89ebe64d8f3a3240f03b984f358cbfe9c3caf9d7"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"66516c4041dc89fc76a58266dea2efa84a34082a","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Adding support for Magnum to be built as part of the repo build process."},{"line_number":4,"context_line":"  - Adding the os-magnum-install.yml file to deploy magnum to hosts tagged"},{"line_number":5,"context_line":"    with hostgroup magnum_all."}],"source_content_type":"text/x-yaml","patch_set":12,"id":"9ad45d7e_ecd80320","line":5,"updated":"2016-08-08 19:52:43.000000000","message":"host group ``magnum_all``.","commit_id":"89ebe64d8f3a3240f03b984f358cbfe9c3caf9d7"}]}
