)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"37cf24b91ad39b0ac62951d823f195a9bb934feb","unresolved":false,"context_lines":[{"line_number":12,"context_line":"WARNING: before merging into \"stable\", currently testing non-regression."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: If05eefcf1c91e4d564ec32d12d144f47564162d6"},{"line_number":15,"context_line":"Implements: external-ceph"},{"line_number":16,"context_line":"Signed-off-by: Sebastien Fuchs \u003csebastien@emmene-moi.fr\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a122d0e_b3a1cfab","line":15,"range":{"start_line":15,"start_character":12,"end_line":15,"end_character":25},"updated":"2016-04-27 11:41:31.000000000","message":"fix the link","commit_id":"3d1abe0b62dde7a5ae3d52a6bdf181c2c476bbb4"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8f433d750da97f56adfc4fc761aa9ebf2cbb4075","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-04-27 19:14:30 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Introduced some config values into globals.yml and passwords.yml"},{"line_number":8,"context_line":"to be able to use the ceph driver without deploying a ceph cluster."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: If05eefcf1c91e4d564ec32d12d144f47564162d6"},{"line_number":11,"context_line":"Implements: external-ceph"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a122d0e_a362fd3b","line":8,"updated":"2016-04-28 13:32:51.000000000","message":"this is too long. Please use a short title and a detailed\ndescription.\nthere should be one empty line between title and desc.","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"72c9d2a68784a7bd3c1cb5c7c6862de4facdf8f8","unresolved":false,"context_lines":[{"line_number":8,"context_line":"to be able to use the ceph driver without deploying a ceph cluster."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: If05eefcf1c91e4d564ec32d12d144f47564162d6"},{"line_number":11,"context_line":"Implements: external-ceph"},{"line_number":12,"context_line":"Signed-off-by: Sebastien Fuchs \u003csebastien@emmene-moi.fr\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a122d0e_1ecd0466","line":11,"range":{"start_line":11,"start_character":12,"end_line":11,"end_character":25},"updated":"2016-04-29 18:29:19.000000000","message":"fix the link","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"}],"ansible/group_vars/all.yml":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8f433d750da97f56adfc4fc761aa9ebf2cbb4075","unresolved":false,"context_lines":[{"line_number":186,"context_line":"enable_central_logging: \"no\""},{"line_number":187,"context_line":"enable_ceph: \"no\""},{"line_number":188,"context_line":"enable_ceph_rgw: \"no\""},{"line_number":189,"context_line":"enable_ceph_external: \"no\""},{"line_number":190,"context_line":"enable_cinder: \"no\""},{"line_number":191,"context_line":"enable_heat: \"yes\""},{"line_number":192,"context_line":"enable_horizon: \"yes\""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1a122d0e_e358758c","line":189,"updated":"2016-04-28 13:32:51.000000000","message":"the enable_ceph and enable_ceph_external are confused.\n\nhow about add a third option for enable_ceph,\n\nenable_ceph: yes/no/external","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":20596,"name":"Sebastien Fuchs","email":"sebastien@emmene-moi.fr","username":"emmenemoi"},"change_message_id":"46113d60e592acc7144395f1d9b1a794a74836fe","unresolved":false,"context_lines":[{"line_number":186,"context_line":"enable_central_logging: \"no\""},{"line_number":187,"context_line":"enable_ceph: \"no\""},{"line_number":188,"context_line":"enable_ceph_rgw: \"no\""},{"line_number":189,"context_line":"enable_ceph_external: \"no\""},{"line_number":190,"context_line":"enable_cinder: \"no\""},{"line_number":191,"context_line":"enable_heat: \"yes\""},{"line_number":192,"context_line":"enable_horizon: \"yes\""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1a122d0e_21baf70a","line":189,"in_reply_to":"1a122d0e_e358758c","updated":"2016-05-03 14:02:17.000000000","message":"I did follow what has been suggested in a former proposition by a maintainer.\nDo you confirm that if enable_ceph \u003d \"external\", all bool conditions will fail without errors?\nIs it compliant with the future setup manual (yes/no/other value for \"enable_service\" params) ?\n\nAnother possibility: enable_ceph\u003dyes and set ceph_external_mons. If ceph_external_mons is defined and valid will set the deployment as enable_ceph_external.\n\nWhich is best?","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"667096a34678328e9662bed70926bc61d96b76d6","unresolved":false,"context_lines":[{"line_number":259,"context_line":"###################"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"# mandatory if enable_ceph\u003dno and cinder_volume_driver or glance_volume_driver is set to \"ceph\""},{"line_number":262,"context_line":"#ceph_mon_hosts: \"host1, host2\""},{"line_number":263,"context_line":""},{"line_number":264,"context_line":"# Ceph can be setup with a caching to improve performance. To use the cache you"},{"line_number":265,"context_line":"# must provide separate disks than those for the OSDs"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3aaa91ec_8bcae64d","line":262,"updated":"2016-06-28 13:45:33.000000000","message":"So I\u0027m thinking if we could fill this var from our ceph deployment if we deploy ceph. That would allow us to collapse logic in nova/glance/cinder into one (external-ceph)","commit_id":"bfcbbd3dcb7ca2e4f31d2305c09af91212e7de60"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"8f4401c6aa9d9e8f8b18760e4616c9044d1cdab4","unresolved":false,"context_lines":[{"line_number":251,"context_line":"#################################"},{"line_number":252,"context_line":"# Glance - Block Storage options"},{"line_number":253,"context_line":"#################################"},{"line_number":254,"context_line":"glance_volume_driver: \"{{ \u0027ceph\u0027 if enable_ceph | bool else \u0027file\u0027 }}\""},{"line_number":255,"context_line":""},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"###################"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3aaa91ec_eb972ae9","line":254,"updated":"2016-06-28 13:48:32.000000000","message":"this ia a test comment for trianing purpoes.","commit_id":"cbe98631c49af43c9f2ce1d68db240d1699eef6c"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"042d5aae1f1fa4e3c84a05dcc95b6690c61c5de5","unresolved":false,"context_lines":[{"line_number":251,"context_line":"#################################"},{"line_number":252,"context_line":"# Glance - Block Storage options"},{"line_number":253,"context_line":"#################################"},{"line_number":254,"context_line":"glance_volume_driver: \"{{ \u0027ceph\u0027 if enable_ceph | bool else \u0027file\u0027 }}\""},{"line_number":255,"context_line":""},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"###################"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3aaa91ec_ae5d3017","line":254,"in_reply_to":"3aaa91ec_eb972ae9","updated":"2016-06-28 13:51:17.000000000","message":"Done","commit_id":"cbe98631c49af43c9f2ce1d68db240d1699eef6c"}],"ansible/roles/ceph/templates/ceph.conf.j2":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8f433d750da97f56adfc4fc761aa9ebf2cbb4075","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fsid \u003d {{ ceph_cluster_fsid }}"},{"line_number":3,"context_line":"mon initial members \u003d {% for host in groups[\u0027ceph-mon\u0027] %}{{ hostvars[host][\u0027ansible_\u0027 + hostvars[host][\u0027storage_interface\u0027]][\u0027ipv4\u0027][\u0027address\u0027] }}{% if not loop.last %}, {% endif %}{% endfor %}"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"mon host \u003d {% if not (ceph_mon_hosts|trim \u003d\u003d \u0027\u0027) %}{{ ceph_mon_hosts }}{% else %}{% for host in groups[\u0027ceph-mon\u0027] %}{{ hostvars[host][\u0027ansible_\u0027 + hostvars[host][\u0027storage_interface\u0027]][\u0027ipv4\u0027][\u0027address\u0027] }}{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"auth cluster required \u003d cephx"},{"line_number":8,"context_line":"auth service required \u003d cephx"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"1a122d0e_23570d59","line":5,"updated":"2016-04-28 13:32:51.000000000","message":"since we support external ceph, the ceph_mon_hosts must be configured\nwhen the enable_ceph_external is true.","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8f433d750da97f56adfc4fc761aa9ebf2cbb4075","unresolved":false,"context_lines":[{"line_number":9,"context_line":"auth client required \u003d cephx"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[mon]"},{"line_number":12,"context_line":"# NOTE(SebastienFuchs): Shall be put in [mon] sections."},{"line_number":13,"context_line":"mon addr \u003d {% if not (ceph_mon_ips|trim \u003d\u003d \u0027\u0027) %}{{ ceph_mon_ips }}{% else %}{% for host in groups[\u0027ceph-mon\u0027] %}{{ hostvars[host][\u0027ansible_\u0027 + hostvars[host][\u0027storage_interface\u0027]][\u0027ipv4\u0027][\u0027address\u0027] }}:6789{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# NOTE(SamYaple): The monitor files have been known to grow very large. The"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"1a122d0e_0352d166","line":12,"updated":"2016-04-28 13:32:51.000000000","message":"Shall be? typo?\nAdding this NOTE is confuesed to the newbie.","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":20596,"name":"Sebastien Fuchs","email":"sebastien@emmene-moi.fr","username":"emmenemoi"},"change_message_id":"5cc8cf5b8c894602ed9c8a37bc22685cc25a68af","unresolved":false,"context_lines":[{"line_number":9,"context_line":"auth client required \u003d cephx"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[mon]"},{"line_number":12,"context_line":"# NOTE(SebastienFuchs): Shall be put in [mon] sections."},{"line_number":13,"context_line":"mon addr \u003d {% if not (ceph_mon_ips|trim \u003d\u003d \u0027\u0027) %}{{ ceph_mon_ips }}{% else %}{% for host in groups[\u0027ceph-mon\u0027] %}{{ hostvars[host][\u0027ansible_\u0027 + hostvars[host][\u0027storage_interface\u0027]][\u0027ipv4\u0027][\u0027address\u0027] }}:6789{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# NOTE(SamYaple): The monitor files have been known to grow very large. The"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"1a122d0e_7c03f2be","line":12,"in_reply_to":"1a122d0e_0352d166","updated":"2016-05-23 13:54:15.000000000","message":"Done","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8f433d750da97f56adfc4fc761aa9ebf2cbb4075","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[mon]"},{"line_number":12,"context_line":"# NOTE(SebastienFuchs): Shall be put in [mon] sections."},{"line_number":13,"context_line":"mon addr \u003d {% if not (ceph_mon_ips|trim \u003d\u003d \u0027\u0027) %}{{ ceph_mon_ips }}{% else %}{% for host in groups[\u0027ceph-mon\u0027] %}{{ hostvars[host][\u0027ansible_\u0027 + hostvars[host][\u0027storage_interface\u0027]][\u0027ipv4\u0027][\u0027address\u0027] }}:6789{% if not loop.last %}, {% endif %}{% endfor %}{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# NOTE(SamYaple): The monitor files have been known to grow very large. The"},{"line_number":16,"context_line":"#                 only fix for that is to compact the files."}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"1a122d0e_634d85c3","line":13,"updated":"2016-04-28 13:32:51.000000000","message":"ditto. ceph_mon_ips must be configured when enable_ceph_external is true.","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"}],"ansible/roles/cinder/tasks/deploy.yml":[{"author":{"_account_id":16233,"name":"Mauricio Lima","email":"mauriciolimab@gmail.com","username":"limamauricio"},"change_message_id":"37cf24b91ad39b0ac62951d823f195a9bb934feb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: ceph.yml"},{"line_number":3,"context_line":"  when:"},{"line_number":4,"context_line":"    - (enable_ceph | bool) or (enable_ceph_external | bool)"},{"line_number":5,"context_line":"    - inventory_hostname in groups[\u0027ceph-mon\u0027] or"},{"line_number":6,"context_line":"        inventory_hostname in groups[\u0027cinder-api\u0027] or"},{"line_number":7,"context_line":"        inventory_hostname in groups[\u0027cinder-volume\u0027] or"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1a122d0e_b3dd4f16","line":4,"range":{"start_line":4,"start_character":6,"end_line":4,"end_character":59},"updated":"2016-04-27 11:41:31.000000000","message":"(enable_ceph | bool or enable_ceph_external | bool)","commit_id":"3d1abe0b62dde7a5ae3d52a6bdf181c2c476bbb4"},{"author":{"_account_id":20596,"name":"Sebastien Fuchs","email":"sebastien@emmene-moi.fr","username":"emmenemoi"},"change_message_id":"f6debc76f358089b117ef71f9d088681db16bfbb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: ceph.yml"},{"line_number":3,"context_line":"  when:"},{"line_number":4,"context_line":"    - (enable_ceph | bool) or (enable_ceph_external | bool)"},{"line_number":5,"context_line":"    - inventory_hostname in groups[\u0027ceph-mon\u0027] or"},{"line_number":6,"context_line":"        inventory_hostname in groups[\u0027cinder-api\u0027] or"},{"line_number":7,"context_line":"        inventory_hostname in groups[\u0027cinder-volume\u0027] or"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1a122d0e_f5a167f8","line":4,"range":{"start_line":4,"start_character":6,"end_line":4,"end_character":59},"in_reply_to":"1a122d0e_b3dd4f16","updated":"2016-04-27 17:15:54.000000000","message":"Done","commit_id":"3d1abe0b62dde7a5ae3d52a6bdf181c2c476bbb4"}],"ansible/roles/glance/tasks/ceph.yml":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8f433d750da97f56adfc4fc761aa9ebf2cbb4075","unresolved":false,"context_lines":[{"line_number":42,"context_line":"#    content: \"{{ cephx_key_glance.stdout }}\\n\\r\""},{"line_number":43,"context_line":"#    dest: \"{{ node_config_directory }}/glance-api/ceph.client.glance.keyring\""},{"line_number":44,"context_line":"#    mode: \"0600\""},{"line_number":45,"context_line":"#  when: inventory_hostname in groups[\u0027glance-api\u0027]"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"# TODO: find why cephx_key_glance.stdout is undefined the direct way (above) when pulling keys"},{"line_number":48,"context_line":"- name: Pushing cephx keyring"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1a122d0e_83654146","line":45,"updated":"2016-04-28 13:32:51.000000000","message":"why this is commented out?","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":20596,"name":"Sebastien Fuchs","email":"sebastien@emmene-moi.fr","username":"emmenemoi"},"change_message_id":"46113d60e592acc7144395f1d9b1a794a74836fe","unresolved":false,"context_lines":[{"line_number":42,"context_line":"#    content: \"{{ cephx_key_glance.stdout }}\\n\\r\""},{"line_number":43,"context_line":"#    dest: \"{{ node_config_directory }}/glance-api/ceph.client.glance.keyring\""},{"line_number":44,"context_line":"#    mode: \"0600\""},{"line_number":45,"context_line":"#  when: inventory_hostname in groups[\u0027glance-api\u0027]"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"# TODO: find why cephx_key_glance.stdout is undefined the direct way (above) when pulling keys"},{"line_number":48,"context_line":"- name: Pushing cephx keyring"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1a122d0e_5c50b60a","line":45,"in_reply_to":"1a122d0e_83654146","updated":"2016-05-03 14:02:17.000000000","message":"Because this is the proper way to do it... but it doesn\u0027t work (cf TODO).\nIf you see any reason...","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"}],"ansible/roles/rabbitmq/templates/rabbitmq-env.conf.j2":[{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"667096a34678328e9662bed70926bc61d96b76d6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"RABBITMQ_NODENAME\u003drabbit"},{"line_number":2,"context_line":"RABBITMQ_BOOT_MODULE\u003drabbit_clusterer"},{"line_number":3,"context_line":"#RABBITMQ_USE_LONGNAME\u003dtrue"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"RABBITMQ_LOG_BASE\u003d/var/log/kolla/rabbitmq"},{"line_number":6,"context_line":"RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS\u003d\"-pa /usr/lib/rabbitmq/lib/rabbitmq_server-3.5.7/plugins/rabbitmq_clusterer-3.5.x-189b3a81.ez/rabbitmq_clusterer-3.5.x-189b3a81/ebin\""}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"3aaa91ec_0b2b96ec","line":3,"updated":"2016-06-28 13:45:33.000000000","message":"why do you change it?","commit_id":"bfcbbd3dcb7ca2e4f31d2305c09af91212e7de60"}],"etc/kolla/passwords.yml":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8f433d750da97f56adfc4fc761aa9ebf2cbb4075","unresolved":false,"context_lines":[{"line_number":11,"context_line":"cephx_keys:"},{"line_number":12,"context_line":"    cinder:"},{"line_number":13,"context_line":"    cinder-backup:"},{"line_number":14,"context_line":"    glance:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"###################"},{"line_number":17,"context_line":"# Database options"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1a122d0e_c35bb984","line":14,"updated":"2016-04-28 13:32:51.000000000","message":"when adding this, please ingore the cephx_keys in genpwd.py file\n\nbtw, I think this make thing more complicated. why not create the\npool automatically like now when using external ceph?","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"},{"author":{"_account_id":20596,"name":"Sebastien Fuchs","email":"sebastien@emmene-moi.fr","username":"emmenemoi"},"change_message_id":"46113d60e592acc7144395f1d9b1a794a74836fe","unresolved":false,"context_lines":[{"line_number":11,"context_line":"cephx_keys:"},{"line_number":12,"context_line":"    cinder:"},{"line_number":13,"context_line":"    cinder-backup:"},{"line_number":14,"context_line":"    glance:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"###################"},{"line_number":17,"context_line":"# Database options"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1a122d0e_3cb85a2a","line":14,"in_reply_to":"1a122d0e_c35bb984","updated":"2016-05-03 14:02:17.000000000","message":"This would require the admin keys. I\u0027m not sure everybody wants to give the admin key to an external user/script.\nIt\u0027s a security decision.","commit_id":"25140db5057eb9c381692a5dc08d8f243e444798"}]}
