)]}'
{"manifests/backend/dellsc_iscsi.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"54270410e77a2ac7726c4532de8b1aa014962407","unresolved":false,"context_lines":[{"line_number":31,"context_line":"#   (optional) Name of the server folder to use on the Storage Center."},{"line_number":32,"context_line":"#   Defaults to \u0027srv\u0027"},{"line_number":33,"context_line":"#"},{"line_number":34,"context_line":"# [*dell_sc_verify_cert*]"},{"line_number":35,"context_line":"#   (optional) Enable HTTPS SC ceritifcate verification"},{"line_number":36,"context_line":"#   Defaults to $::os_service_default"},{"line_number":37,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"3a7e1126_41872fd7","line":34,"range":{"start_line":34,"start_character":4,"end_line":34,"end_character":23},"updated":"2015-12-18 10:22:05.000000000","message":"why you did\u0027nt included this option to related volume manifest?\n\nThis parameter is not present here:\n\nhttps://github.com/openstack/puppet-cinder/blob/master/manifests/volume/dellsc_iscsi.pp","commit_id":"2eddd8c277650c8c656f7751199ff3502ae2c62b"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"641bfab9befefe17201356fb77dc02021c3be877","unresolved":false,"context_lines":[{"line_number":31,"context_line":"#   (optional) Name of the server folder to use on the Storage Center."},{"line_number":32,"context_line":"#   Defaults to \u0027srv\u0027"},{"line_number":33,"context_line":"#"},{"line_number":34,"context_line":"# [*dell_sc_verify_cert*]"},{"line_number":35,"context_line":"#   (optional) Enable HTTPS SC ceritifcate verification"},{"line_number":36,"context_line":"#   Defaults to $::os_service_default"},{"line_number":37,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"3a7e1126_f7ab6073","line":34,"range":{"start_line":34,"start_character":4,"end_line":34,"end_character":23},"in_reply_to":"3a7e1126_41872fd7","updated":"2015-12-18 17:48:19.000000000","message":"IMHO the volume classes are redundant and I\u0027m not even sure why we have them since they don\u0027t provide anything except creating a \u0027DEFAULT\u0027 version of this resource.  But i\u0027ll update them anyway...","commit_id":"2eddd8c277650c8c656f7751199ff3502ae2c62b"}],"manifests/backend/glusterfs.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"54270410e77a2ac7726c4532de8b1aa014962407","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#"},{"line_number":16,"context_line":"# [*glusterfs_backup_mount_point*]"},{"line_number":17,"context_line":"#   (optional) Base dir containing mount point for gluster share."},{"line_number":18,"context_line":"#   Defaults to $::os_service_default"},{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":"# [*glusterfs_backup_share*]"},{"line_number":21,"context_line":"#   (optonal) GlusterFS share in \u003chostname|ipv4addr|ipv6addr\u003e:\u003cgluster_vol_name\u003e"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"3a7e1126_81f2f76d","line":18,"range":{"start_line":18,"start_character":13,"end_line":18,"end_character":15},"updated":"2015-12-18 10:22:05.000000000","message":"same","commit_id":"2eddd8c277650c8c656f7751199ff3502ae2c62b"}],"manifests/backend/nfs.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"54270410e77a2ac7726c4532de8b1aa014962407","unresolved":false,"context_lines":[{"line_number":12,"context_line":"#"},{"line_number":13,"context_line":"# [*nfs_mount_attempts*]"},{"line_number":14,"context_line":"#   (optional) The number of attempts to mount nfs shares before raising an"},{"line_number":15,"context_line":"#   error. At least one attempt will be made to mount an nfs share, regardless"},{"line_number":16,"context_line":"#   of the value specified."},{"line_number":17,"context_line":"#   Defaults to $::os_service_default"},{"line_number":18,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"3a7e1126_018607b9","line":15,"range":{"start_line":15,"start_character":14,"end_line":15,"end_character":19},"updated":"2015-12-18 10:22:05.000000000","message":"and here","commit_id":"2eddd8c277650c8c656f7751199ff3502ae2c62b"}],"manifests/backend/rbd.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"54270410e77a2ac7726c4532de8b1aa014962407","unresolved":false,"context_lines":[{"line_number":51,"context_line":"#"},{"line_number":52,"context_line":"# [*rados_connection_interval*]"},{"line_number":53,"context_line":"#   (optional) Interval value (in seconds) between connection retries to ceph"},{"line_number":54,"context_line":"#   cluster."},{"line_number":55,"context_line":"#   Defaults to $::os_service_default"},{"line_number":56,"context_line":"#"},{"line_number":57,"context_line":"# [*rados_connection_retries*]"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"3a7e1126_41716fdf","line":54,"range":{"start_line":54,"start_character":4,"end_line":54,"end_character":11},"updated":"2015-12-18 10:22:05.000000000","message":"these options also are not present in volume manifest","commit_id":"2eddd8c277650c8c656f7751199ff3502ae2c62b"}]}