)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"80485b71233176160eab15313d1d977dcbed544c","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Omit --global to set the identity only in this repository."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ibf5bdd583c10c134d10de4893bb345f66fe03431"},{"line_number":20,"context_line":"fatal: empty ident name (for \u003cswift@aio-storage\u003e) not allowed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5a890539_62585c8b","line":20,"updated":"2014-11-26 18:40:58.000000000","message":"This looks like it\u0027s in the wrong spot, should be above Change-Id.","commit_id":"1cb0255a7aa3d4cbb91300a94d3cae0a8ce6bf88"}],"recipes/ring-repo.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"6e74e08e89980b916e1cd0600c48e1887c826e1a","unresolved":false,"context_lines":[{"line_number":41,"context_line":"execute \u0027create empty git repo\u0027 do"},{"line_number":42,"context_line":"  cwd \u0027/tmp\u0027"},{"line_number":43,"context_line":"  umask 022"},{"line_number":44,"context_line":"  command \"mkdir $$; cd $$; git init; echo \\\"backups\\\" \\\u003e .gitignore; git config user.email \u0027chef@openstack.org\u0027 ; git config user.name \u0027Chef\u0027; git add .gitignore; git commit -m \u0027initial commit\u0027 --author\u003d\u0027chef \u003cchef@openstack\u003e\u0027; git push file:///#{platform_options[\"git_dir\"]}/rings master\""},{"line_number":45,"context_line":"  user \u0027swift\u0027"},{"line_number":46,"context_line":"  action :nothing"},{"line_number":47,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":4,"id":"5a890539_8bf7cef4","line":44,"updated":"2014-11-12 20:23:37.000000000","message":"Maybe some DRY here, push this out into a var and include here and below with #{var}  ?","commit_id":"ae69d9e14f99dc5d99c93b70aceb7070562657d5"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"6e74e08e89980b916e1cd0600c48e1887c826e1a","unresolved":false,"context_lines":[{"line_number":129,"context_line":"                 \"min_part_hours\u003d#{min_part_hours}, replicas\u003d#{replicas}\")"},{"line_number":130,"context_line":"  execute \"add #{ring_type}.builder\" do"},{"line_number":131,"context_line":"    cwd \u0027/etc/swift/ring-workspace/rings\u0027"},{"line_number":132,"context_line":"    command \"git add #{ring_type}.builder \u0026\u0026  git config user.email \u0027chef@openstack.org\u0027 ; git config user.name \u0027Chef\u0027 \u0026\u0026 git commit -m \u0027initial ring builders\u0027 --author\u003d\u0027chef \u003cchef@openstack\u003e\u0027\""},{"line_number":133,"context_line":"    user \u0027swift\u0027"},{"line_number":134,"context_line":"    action :nothing"},{"line_number":135,"context_line":"  end"}],"source_content_type":"text/x-ruby","patch_set":4,"id":"5a890539_4bd40647","line":132,"updated":"2014-11-12 20:23:37.000000000","message":"Looks like spec tests are fairly light for this cookbook, maybe add something?","commit_id":"ae69d9e14f99dc5d99c93b70aceb7070562657d5"},{"author":{"_account_id":9488,"name":"chenzhiwei","email":"zhiweik@gmail.com","username":"chenzhiwei"},"change_message_id":"46cb6dea429ae715432992576b2cd0c9ca5dcb27","unresolved":false,"context_lines":[{"line_number":43,"context_line":"execute \u0027create empty git repo\u0027 do"},{"line_number":44,"context_line":"  cwd \u0027/tmp\u0027"},{"line_number":45,"context_line":"  umask 022"},{"line_number":46,"context_line":"  command \"mkdir $$; cd $$; git init; echo \\\"backups\\\" \\\u003e .gitignore; #{git_config_email} ; #{git_config_name} ; git add .gitignore; git commit -m \u0027initial commit\u0027 --author\u003d\u0027chef \u003cchef@openstack\u003e\u0027; git push file:///#{platform_options[\"git_dir\"]}/rings master\""},{"line_number":47,"context_line":"  user \u0027swift\u0027"},{"line_number":48,"context_line":"  action :nothing"},{"line_number":49,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":6,"id":"3a961159_250939dd","line":46,"updated":"2015-01-16 08:21:11.000000000","message":"Can you use bash lwrp instead of command? Since this command is too complex and hard to read.","commit_id":"dd13a4004f1e603afac146f8f476eade7ac8a2dc"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"a29d5d2e89e9d6f37a81863e5caf9611d3701533","unresolved":false,"context_lines":[{"line_number":43,"context_line":"execute \u0027create empty git repo\u0027 do"},{"line_number":44,"context_line":"  cwd \u0027/tmp\u0027"},{"line_number":45,"context_line":"  umask 022"},{"line_number":46,"context_line":"  command \"mkdir $$; cd $$; git init; echo \\\"backups\\\" \\\u003e .gitignore; #{git_config_email} ; #{git_config_name} ; git add .gitignore; git commit -m \u0027initial commit\u0027 --author\u003d\u0027chef \u003cchef@openstack\u003e\u0027; git push file:///#{platform_options[\"git_dir\"]}/rings master\""},{"line_number":47,"context_line":"  user \u0027swift\u0027"},{"line_number":48,"context_line":"  action :nothing"},{"line_number":49,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":6,"id":"fa81d914_0a17b0ee","line":46,"in_reply_to":"3a961159_250939dd","updated":"2015-01-27 23:20:28.000000000","message":"Since execute is used all over the place in here, I would rather is a different patch to cover that type of refactoring.","commit_id":"dd13a4004f1e603afac146f8f476eade7ac8a2dc"}]}