)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e7ac3624f1b0f54ce4c5c1a132a878c92c444e7d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Disable configuration of tgtd on RHEL 7"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- This is a stage I fix for bug #1400958. First disable the related"},{"line_number":10,"context_line":"resources for tgtd."},{"line_number":11,"context_line":"- Bump version to 10.1.0 to indicate this is a start point of new"},{"line_number":12,"context_line":"platform support"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a961159_de4a8d26","line":10,"updated":"2014-12-17 12:04:31.000000000","message":"These two sentences don\u0027t make too much sense to me. Do you mean to say that this is only a partial fix for the bug you are referencing? Then you should not use \"Closes-bug:\"","commit_id":"cf519cf4f77e3cc07ff321354ad4ac298ab32353"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"63ac2448909568fe41e4e5e3df09228c8c757596","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Disable configuration of tgtd on RHEL 7"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- This is a stage I fix for bug #1400958. First disable the related"},{"line_number":10,"context_line":"resources for tgtd."},{"line_number":11,"context_line":"- Bump version to 10.1.0 to indicate this is a start point of new"},{"line_number":12,"context_line":"platform support"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a961159_efecd62b","line":10,"in_reply_to":"3a961159_c87c7bb8","updated":"2014-12-18 08:43:05.000000000","message":"That would be Partial-Bug: then, see https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references.","commit_id":"cf519cf4f77e3cc07ff321354ad4ac298ab32353"},{"author":{"_account_id":9492,"name":"Yi Ming Yin","email":"yinym@cn.ibm.com","username":"yym"},"change_message_id":"075d2f4255396a859c84673aabfe9475291c275f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Disable configuration of tgtd on RHEL 7"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- This is a stage I fix for bug #1400958. First disable the related"},{"line_number":10,"context_line":"resources for tgtd."},{"line_number":11,"context_line":"- Bump version to 10.1.0 to indicate this is a start point of new"},{"line_number":12,"context_line":"platform support"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a961159_c87c7bb8","line":10,"in_reply_to":"3a961159_de4a8d26","updated":"2014-12-18 02:56:28.000000000","message":"Thank you for bringing this up. Yes, this fix only make volume recipe can run without error on RHEL7, but not a complete fix for the bug. I was concerning on this tag \"closes-bug\" as well.\n\nIs there any other tag I can use here? My intention is to notify launch pad to add the link to the bug, but not indicate that this will fix the bug all.","commit_id":"cf519cf4f77e3cc07ff321354ad4ac298ab32353"}],"attributes/default.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"3c2c3f398821831400196fedff92fea143fc6d58","unresolved":false,"context_lines":[{"line_number":318,"context_line":"    \u0027cinder_svc_packages\u0027 \u003d\u003e [\u0027sysfsutils\u0027],"},{"line_number":319,"context_line":"    \u0027package_overrides\u0027 \u003d\u003e \u0027\u0027"},{"line_number":320,"context_line":"  }"},{"line_number":321,"context_line":"  if platform_family \u003d\u003d \u0027rhel\u0027 \u0026\u0026 platform_version.start_with?(\u00277\u0027)"},{"line_number":322,"context_line":"    default[\u0027openstack\u0027][\u0027block-storage\u0027][\u0027platform\u0027][\u0027cinder_iscsitarget_packages\u0027] \u003d []"},{"line_number":323,"context_line":"    default[\u0027openstack\u0027][\u0027block-storage\u0027][\u0027platform\u0027][\u0027cinder_iscsitarget_service\u0027] \u003d []"},{"line_number":324,"context_line":"  end"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"3a961159_66f92b4e","line":321,"updated":"2015-01-12 14:47:38.000000000","message":"I would like to see some comments here to help doc why this was needed.","commit_id":"e015eec7c7c47684c9a947f89198c805879fb37d"}],"recipes/volume.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"3c2c3f398821831400196fedff92fea143fc6d58","unresolved":false,"context_lines":[{"line_number":233,"context_line":"  subscribes :restart, \u0027template[/etc/cinder/cinder.conf]\u0027"},{"line_number":234,"context_line":"end"},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"unless node[\u0027platform_family\u0027] \u003d\u003d \u0027rhel\u0027 \u0026\u0026 node[\u0027platform_version\u0027].start_with?(\u00277\u0027)"},{"line_number":237,"context_line":"  service \u0027iscsitarget\u0027 do"},{"line_number":238,"context_line":"    service_name platform_options[\u0027cinder_iscsitarget_service\u0027]"},{"line_number":239,"context_line":"    supports status: true, restart: true"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"3a961159_268643bb","line":236,"updated":"2015-01-12 14:47:38.000000000","message":"Again, a comment would be nice to clarify this logic","commit_id":"e015eec7c7c47684c9a947f89198c805879fb37d"}]}
