)]}'
{"playbooks/roles/os_cinder/tasks/cinder_backends.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"848d73b280cb10624a71edacee1f5e074aeadf28","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Ensure cinder api is available"},{"line_number":17,"context_line":"  get_url:"},{"line_number":18,"context_line":"    url: \"{{ cinder_service_internaluri }}\""},{"line_number":19,"context_line":"    dest: /dev/null"},{"line_number":20,"context_line":"    force: yes"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a740942_9ed99724","line":17,"updated":"2015-12-14 11:14:33.000000000","message":"A more reliable check could be to use the uri module and check for \u0027versions\u0027 in the response, as that\u0027s part of the API:\n\nhttp://developer.openstack.org/api-ref-blockstorage-v2.html#listVersions\n\nhttp://developer.openstack.org/api-ref-blockstorage-v1.html#listVersionsv1","commit_id":"3cef374ffa62852df2c239b5b2b6db1b83bb795a"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"848d73b280cb10624a71edacee1f5e074aeadf28","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    dest: /dev/null"},{"line_number":20,"context_line":"    force: yes"},{"line_number":21,"context_line":"  register: result"},{"line_number":22,"context_line":"  until: result |success"},{"line_number":23,"context_line":"  retries: 10"},{"line_number":24,"context_line":"  delay: 30"},{"line_number":25,"context_line":"  tags:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a740942_7ea06bf4","line":22,"updated":"2015-12-14 11:14:33.000000000","message":"nit: \u0027result | success\u0027 \u003cextra space\u003e","commit_id":"3cef374ffa62852df2c239b5b2b6db1b83bb795a"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"848d73b280cb10624a71edacee1f5e074aeadf28","unresolved":false,"context_lines":[{"line_number":21,"context_line":"  register: result"},{"line_number":22,"context_line":"  until: result |success"},{"line_number":23,"context_line":"  retries: 10"},{"line_number":24,"context_line":"  delay: 30"},{"line_number":25,"context_line":"  tags:"},{"line_number":26,"context_line":"    - cinder-backends"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a740942_de6abfb2","line":24,"updated":"2015-12-14 11:14:33.000000000","message":"This results in a minimum wait of 30s, versus the current minimum wait of 10s. -1 on this.","commit_id":"3cef374ffa62852df2c239b5b2b6db1b83bb795a"},{"author":{"_account_id":7219,"name":"git-harry","email":"git-harry@live.co.uk","username":"git-harry"},"change_message_id":"ef200a7e63af1027c860e582b879a96de0bac9a6","unresolved":false,"context_lines":[{"line_number":16,"context_line":"- name: Ensure cinder api is available"},{"line_number":17,"context_line":"  uri:"},{"line_number":18,"context_line":"    url: \"{{ cinder_service_internaluri }}\""},{"line_number":19,"context_line":"    return_content: yes"},{"line_number":20,"context_line":"  register: response"},{"line_number":21,"context_line":"  until:"},{"line_number":22,"context_line":"    - response.content is defined"},{"line_number":23,"context_line":"    - \"\u0027version\u0027 in response.content\""},{"line_number":24,"context_line":"  retries: 10"},{"line_number":25,"context_line":"  delay: 10"},{"line_number":26,"context_line":"  tags:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"fa69d971_51d763ed","line":23,"range":{"start_line":19,"start_character":0,"end_line":23,"end_character":37},"updated":"2016-01-07 13:21:27.000000000","message":"I don\u0027t think this is required and would be better to replace it with the line:\n\nstatus_code: 200","commit_id":"fba698fdd4a99e47c59bcd2455c6deb913067fc1"}]}