)]}'
{"playbooks/os-cinder-install.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"10dd498c0bd1744bd6790dc28073c8be0ac7757a","unresolved":false,"context_lines":[{"line_number":196,"context_line":"    - role: \"ceph_client\""},{"line_number":197,"context_line":"      cinder_venv_bin: \"/openstack/venvs/cinder-{{ openstack_release }}/bin\""},{"line_number":198,"context_line":"      openstack_service_system_user: \"{{ cinder_system_user_name }}\""},{"line_number":199,"context_line":"      openstack_service_venv_bin: \"{{ (cinder_venv_enabled|default(true)) | ternary(cinder_venv_bin,\u0027\u0027) }}\""},{"line_number":200,"context_line":"      tags:"},{"line_number":201,"context_line":"        - \"cinder-ceph-client\""},{"line_number":202,"context_line":"        - \"ceph-client\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_f3a92a92","line":199,"range":{"start_line":199,"start_character":38,"end_line":199,"end_character":103},"updated":"2016-05-23 19:39:33.000000000","message":"should we go for a simpler way like \nvariable if venv_enabled else None ?","commit_id":"a1d8a6494318d770e462a75daf5f1934fb33fc43"}],"playbooks/roles/ceph_client/tasks/ceph_all.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"10dd498c0bd1744bd6790dc28073c8be0ac7757a","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  tags:     ceph-install"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- include: ceph_install_python_libs.yml"},{"line_number":43,"context_line":"  when:"},{"line_number":44,"context_line":"    - openstack_service_venv_bin is defined"},{"line_number":45,"context_line":"    - openstack_service_venv_bin !\u003d \u0027\u0027"},{"line_number":46,"context_line":"  tags:"},{"line_number":47,"context_line":"    - ceph-install"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bab6814e_f333ea60","line":45,"range":{"start_line":43,"start_character":0,"end_line":45,"end_character":38},"updated":"2016-05-23 19:39:33.000000000","message":"this would probably require a change with ansible2","commit_id":"a1d8a6494318d770e462a75daf5f1934fb33fc43"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"2e03adbf3f3d0579e491610ffa1b8cbfa346347b","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  tags:     ceph-install"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- include: ceph_install_python_libs.yml"},{"line_number":43,"context_line":"  when: openstack_service_venv_bin !\u003d \u0027\u0027"},{"line_number":44,"context_line":"  tags:"},{"line_number":45,"context_line":"    - ceph-install"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bab6814e_51229c02","line":43,"updated":"2016-05-24 07:40:53.000000000","message":"Should we be checking for None here, rather than \u0027\u0027?","commit_id":"213d028348f424c9bb471ebb927b4d0e24fdae17"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"da3b5c33c31ff1cae1f1c9e6048e41702a60b85b","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  tags:     ceph-install"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- include: ceph_install_python_libs.yml"},{"line_number":43,"context_line":"  when: openstack_service_venv_bin !\u003d \u0027\u0027"},{"line_number":44,"context_line":"  tags:"},{"line_number":45,"context_line":"    - ceph-install"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bab6814e_b79e5861","line":43,"in_reply_to":"bab6814e_51229c02","updated":"2016-05-24 08:16:27.000000000","message":"Looks like \u0027\u0027 works fine.","commit_id":"213d028348f424c9bb471ebb927b4d0e24fdae17"}],"playbooks/roles/ceph_client/tasks/ceph_install_python_libs.yml":[{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"4743db3755c929bf400c773fafa058c1da377bc7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Register rados module path"},{"line_number":17,"context_line":"  command: python -c \u0027import rados; print rados.__file__\u0027"},{"line_number":18,"context_line":"  register: rados_module_path"},{"line_number":19,"context_line":"  tags:"},{"line_number":20,"context_line":"    - ceph-python-libs"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bab6814e_f0d475be","line":17,"range":{"start_line":17,"start_character":48,"end_line":17,"end_character":56},"updated":"2016-05-24 00:57:26.000000000","message":"could be __path__ ?","commit_id":"213d028348f424c9bb471ebb927b4d0e24fdae17"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"00c4287d6c4ff2988df5f694d945a657baccd77a","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Register rados module path"},{"line_number":17,"context_line":"  command: python -c \u0027import rados; print rados.__file__\u0027"},{"line_number":18,"context_line":"  register: rados_module_path"},{"line_number":19,"context_line":"  tags:"},{"line_number":20,"context_line":"    - ceph-python-libs"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bab6814e_f1d11031","line":17,"range":{"start_line":17,"start_character":48,"end_line":17,"end_character":56},"in_reply_to":"bab6814e_f0d475be","updated":"2016-05-24 07:34:01.000000000","message":"AttributeError: \u0027module\u0027 object has no attribute \u0027__path__\u0027\n\nDo you mean rados.__path__ or something else?","commit_id":"213d028348f424c9bb471ebb927b4d0e24fdae17"},{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"99c811f4a19c43165b20c5abc21653ede763113b","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Register rados module path"},{"line_number":17,"context_line":"  command: python -c \u0027import rados; print rados.__file__\u0027"},{"line_number":18,"context_line":"  register: rados_module_path"},{"line_number":19,"context_line":"  tags:"},{"line_number":20,"context_line":"    - ceph-python-libs"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bab6814e_2ccfbeb4","line":17,"range":{"start_line":17,"start_character":48,"end_line":17,"end_character":56},"in_reply_to":"bab6814e_f1d11031","updated":"2016-05-24 08:54:16.000000000","message":"yes, I meant rados.__path__\n\nif that is doable, we can save the basename call on line 31 and line 40\n\njust a suggestion","commit_id":"213d028348f424c9bb471ebb927b4d0e24fdae17"},{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"4743db3755c929bf400c773fafa058c1da377bc7","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    - ceph-python-libs"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"- name: Register rbd module path"},{"line_number":23,"context_line":"  command: python -c \u0027import rbd; print rbd.__file__\u0027"},{"line_number":24,"context_line":"  register: rbd_module_path"},{"line_number":25,"context_line":"  tags:"},{"line_number":26,"context_line":"    - ceph-python-libs"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bab6814e_50daa1ea","line":23,"range":{"start_line":23,"start_character":44,"end_line":23,"end_character":52},"updated":"2016-05-24 00:57:26.000000000","message":"__path__ ?","commit_id":"213d028348f424c9bb471ebb927b4d0e24fdae17"}]}