)]}'
{"docker/collectd/collectd_sudoers":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"15047623fea816c342fab5b36c96260b8af69c91","unresolved":false,"context_lines":[{"line_number":1,"context_line":"%kolla ALL\u003d(root) NOPASSWD: /usr/bin/chown -R collectd\\: /var/lib/collectd/, /usr/bin/chown -R collectd\\: /var/lib/collectd/"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"1a95cdbc_d961e03a","line":1,"updated":"2016-10-08 02:47:22.000000000","message":"your never running chown -R that I see in extend_start, so I think this won\u0027t have the desired result.","commit_id":"408900108cecbdfcddd06b47b82971d5b772a7c3"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"85721b2f932e434a5c715889f7652ff2e65c46ca","unresolved":false,"context_lines":[{"line_number":1,"context_line":"%kolla ALL\u003d(root) NOPASSWD: /usr/bin/chown -R collectd\\: /var/lib/collectd/, /usr/bin/chown -R collectd\\: /var/lib/collectd/"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"1a95cdbc_74c37389","line":1,"in_reply_to":"1a95cdbc_d961e03a","updated":"2016-10-08 03:18:31.000000000","message":"i have test passed\nhttps://github.com/openstack/kolla/blob/master/docker/ceilometer/ceilometer-base/ceilometer_sudoers#L1","commit_id":"408900108cecbdfcddd06b47b82971d5b772a7c3"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"dabccebc7da09ccb9526d570268b0e5785ccd1e2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"%kolla ALL\u003d(root) NOPASSWD: /bin/chown -R collectd\\: /var/lib/collectd/, /usr/bin/chown -R collectd\\: /var/lib/collectd/"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"1a95cdbc_949d5f68","line":1,"updated":"2016-10-08 03:30:33.000000000","message":"I do not think the fix is correct.\n\n-R is never used in extend_start.sh\n\nThe root cause may be the last \u0027/\u0027 in the path.\n\n... /bin/chown collectd\\: /var/lib/collectd, ...","commit_id":"8a750401576f08035de449dd045f0e1ae1e0c46c"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"39fecc06a57e36f87df4b6dba05df5b4ac0abe8e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"%kolla ALL\u003d(root) NOPASSWD: /bin/chown -R collectd\\: /var/lib/collectd/, /usr/bin/chown -R collectd\\: /var/lib/collectd/"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"1a95cdbc_60bf23aa","line":1,"updated":"2016-10-09 08:04:58.000000000","message":"this file is useless now.","commit_id":"287de09f7f3ecbbb56c3883e34696bd9bab8b29b"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"37a4f8648628af7cb45f79175c4cafe5bde225a7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"%kolla ALL\u003d(root) NOPASSWD: /bin/chown -R collectd\\: /var/lib/collectd/, /usr/bin/chown -R collectd\\: /var/lib/collectd/"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"1a95cdbc_a00a0b88","line":1,"in_reply_to":"1a95cdbc_60bf23aa","updated":"2016-10-09 08:06:25.000000000","message":"i fix it now","commit_id":"287de09f7f3ecbbb56c3883e34696bd9bab8b29b"},{"author":{"_account_id":979,"name":"Dave Walker","email":"email@daviey.com","username":"davewalker"},"change_message_id":"5883beb16d7633a157d73335ddb3d4539c97633c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"%kolla ALL\u003d(root) NOPASSWD: /bin/chown  collectd\\: /var/lib/collectd/, /usr/bin/chown collectd\\: /var/lib/collectd/"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"1a95cdbc_20251b08","line":1,"range":{"start_line":1,"start_character":38,"end_line":1,"end_character":40},"updated":"2016-10-09 08:11:54.000000000","message":"why the new whitespace?","commit_id":"aa87d33aaf089c9f2b89666d021a698f84316620"}],"docker/collectd/extend_start.sh":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"15047623fea816c342fab5b36c96260b8af69c91","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Only update permissions if permissions need to be updated"},{"line_number":4,"context_line":"if [[ $(stat -c %U:%G /var/lib/collectd) !\u003d \"collectd:collectd\" ]]; then"},{"line_number":5,"context_line":"    chown collectd: /var/lib/collectd"},{"line_number":6,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a95cdbc_796c4c35","line":5,"updated":"2016-10-08 02:47:22.000000000","message":"this process does not run as root - that is why the sudo part exists.","commit_id":"408900108cecbdfcddd06b47b82971d5b772a7c3"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"85721b2f932e434a5c715889f7652ff2e65c46ca","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Only update permissions if permissions need to be updated"},{"line_number":4,"context_line":"if [[ $(stat -c %U:%G /var/lib/collectd) !\u003d \"collectd:collectd\" ]]; then"},{"line_number":5,"context_line":"    chown collectd: /var/lib/collectd"},{"line_number":6,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a95cdbc_19426868","line":5,"in_reply_to":"1a95cdbc_796c4c35","updated":"2016-10-08 03:18:31.000000000","message":"This should be the legacy of the problem before","commit_id":"408900108cecbdfcddd06b47b82971d5b772a7c3"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"dabccebc7da09ccb9526d570268b0e5785ccd1e2","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Only update permissions if permissions need to be updated"},{"line_number":4,"context_line":"if [[ $(stat -c %U:%G /var/lib/collectd) !\u003d \"collectd:collectd\" ]]; then"},{"line_number":5,"context_line":"    chown collectd: /var/lib/collectd"},{"line_number":6,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":3,"id":"1a95cdbc_74945342","line":5,"updated":"2016-10-08 03:30:33.000000000","message":"you need sudo here, otherwise the collectd_sudoers is useless.","commit_id":"8a750401576f08035de449dd045f0e1ae1e0c46c"}]}