)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"b83ce97491e757369a13f65c9e942a8cf606a2ca","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     karimb \u003ckarimboumedhel@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-18 17:03:05 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Set correct options path for bind9 agent"},{"line_number":8,"context_line":"Closes-Bug: #1534309"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I546aceeb257954afe768bb1ee702a277ed0a2a5a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba63e191_092697ba","line":7,"updated":"2016-01-19 00:42:24.000000000","message":"There needs to be a newline between the subject and the rest of the commit message. Please also briefly explain what the change is and why it is needed, don\u0027t just rely on us finding the launchpad bug.","commit_id":"fe47c00fc8fb485277cf08bff28345d0ad554320"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"fb8a5d4267512b10b04615ba8fe68d9d13d13d1e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"When using Red Hat systems and bind9 backend,"},{"line_number":10,"context_line":"/etc/named/options.conf should be used so"},{"line_number":11,"context_line":"this patch implements this change  to accomodate"},{"line_number":12,"context_line":"the parameters used in the dns module.t review -v"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1534309"},{"line_number":15,"context_line":"Change-Id: I546aceeb257954afe768bb1ee702a277ed0a2a5a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"9a68dd71_d9aeb284","line":12,"updated":"2016-01-22 04:25:08.000000000","message":"nit: Looks like you have some extra random stuff at the end. Might need to s/t review -v//","commit_id":"763c6a90b0f7c98b03ba7f09142725b3ac02ddaf"}],"manifests/backend/bind9.pp":[{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"8f7b07accd83c33f59456c12cbf22f8fee37c019","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"  file_line {\u0027dns allow-new-zones\u0027:"},{"line_number":40,"context_line":"    ensure  \u003d\u003e present,"},{"line_number":41,"context_line":"    path    \u003d\u003e \"${::dns::params::optionspath}\","},{"line_number":42,"context_line":"    line    \u003d\u003e \u0027allow-new-zones yes;\u0027,"},{"line_number":43,"context_line":"    require \u003d\u003e Class[\u0027::designate\u0027],"},{"line_number":44,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":7,"id":"9a68dd71_c81a9017","line":41,"updated":"2016-01-21 10:40:36.000000000","message":"there is no need of double-quotes and curly-braces here.\n\nJust\n\n  path \u003d\u003e $::dns::params::optionspath,","commit_id":"f0c6a4bd2a08e001b6783bf625f23c80afc03f5a"},{"author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"change_message_id":"334402b50693f2fb56b967efc81decd9b2ce909d","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"  file_line {\u0027dns allow-new-zones\u0027:"},{"line_number":40,"context_line":"    ensure  \u003d\u003e present,"},{"line_number":41,"context_line":"    path    \u003d\u003e $dns::params::optionspath,"},{"line_number":42,"context_line":"    line    \u003d\u003e \u0027allow-new-zones yes;\u0027,"},{"line_number":43,"context_line":"    require \u003d\u003e Class[\u0027::designate\u0027],"},{"line_number":44,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":9,"id":"9a68dd71_3f2484ad","line":41,"updated":"2016-01-22 09:59:50.000000000","message":"Should be $::dns::params::optionspath","commit_id":"bc97acfa11b09b0034eef060439f54f7e57aa66a"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"3a3225112a7ee8a3b5550adfe082d4a416ed54f2","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"  file_line {\u0027dns allow-new-zones\u0027:"},{"line_number":40,"context_line":"    ensure  \u003d\u003e present,"},{"line_number":41,"context_line":"    path    \u003d\u003e $::dns::params::optionspath,"},{"line_number":42,"context_line":"    line    \u003d\u003e \u0027allow-new-zones yes;\u0027,"},{"line_number":43,"context_line":"    require \u003d\u003e Class[\u0027::designate\u0027],"},{"line_number":44,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":11,"id":"5a5ae5dd_23445848","line":41,"range":{"start_line":41,"start_character":15,"end_line":41,"end_character":42},"updated":"2016-02-05 09:47:42.000000000","message":"it will be nice, if you will update tests also","commit_id":"7fb932ed6e69b84ab55b79b436c452a30f303e4c"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"6ce7969944aca28481497a3a378dcbd4850e6506","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"  file_line {\u0027dns allow-new-zones\u0027:"},{"line_number":40,"context_line":"    ensure  \u003d\u003e present,"},{"line_number":41,"context_line":"    path    \u003d\u003e $::dns::params::optionspath,"},{"line_number":42,"context_line":"    line    \u003d\u003e \u0027allow-new-zones yes;\u0027,"},{"line_number":43,"context_line":"    require \u003d\u003e Class[\u0027::designate\u0027],"},{"line_number":44,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":11,"id":"bae84128_1ab9782e","line":41,"range":{"start_line":41,"start_character":15,"end_line":41,"end_character":42},"in_reply_to":"3a57f1b5_7ecbfc53","updated":"2016-02-24 09:02:12.000000000","message":"here:\n\nhttps://github.com/openstack/puppet-designate/blob/master/spec/classes/designate_backend_bind9_spec.rb#L15","commit_id":"7fb932ed6e69b84ab55b79b436c452a30f303e4c"},{"author":{"_account_id":9500,"name":"Matt Fischer","email":"matt@mattfischer.com","username":"mfisch"},"change_message_id":"19bb4734b3899db1cd209c4e83a05828cf0731dc","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"  file_line {\u0027dns allow-new-zones\u0027:"},{"line_number":40,"context_line":"    ensure  \u003d\u003e present,"},{"line_number":41,"context_line":"    path    \u003d\u003e $::dns::params::optionspath,"},{"line_number":42,"context_line":"    line    \u003d\u003e \u0027allow-new-zones yes;\u0027,"},{"line_number":43,"context_line":"    require \u003d\u003e Class[\u0027::designate\u0027],"},{"line_number":44,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":11,"id":"3a57f1b5_7ecbfc53","line":41,"range":{"start_line":41,"start_character":15,"end_line":41,"end_character":42},"in_reply_to":"5a5ae5dd_23445848","updated":"2016-02-23 11:37:05.000000000","message":"where in the tests would this get updated? I didnt see it.","commit_id":"7fb932ed6e69b84ab55b79b436c452a30f303e4c"}]}