)]}'
{"tools/check_valid_gerrit_config.sh":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"e668621bdad2ebcbcc4c687a56fc3cb3f7090f2d","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            \"Project $config does not have a core/admins team defined, please revise!\""},{"line_number":21,"context_line":"            exit 1"},{"line_number":22,"context_line":"        fi"},{"line_number":23,"context_line":"        $OLDPWD/tools/normalize_acl.py $config  0 1 2 3 4 5 \u003e $TMPDIR/normalized"},{"line_number":24,"context_line":"        if ! diff -u $config $TMPDIR/normalized;"},{"line_number":25,"context_line":"        then"},{"line_number":26,"context_line":"            echo \"Project $config is not normalized\""}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a961159_eb02f46d","line":23,"updated":"2014-12-10 19:51:37.000000000","message":"It misses transformation 6 (openstack-ci-admins and openstack-ci-core vs infra-core), but I suppose that\u0027s unlikely to come up in the future anyway.","commit_id":"a0fca6892dba2248da7bb288700f9ce6d9c4fde3"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"c1eaf1b6ca7e3b15c542ec9b2b2c6cd4f39832ef","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            \"Project $config does not have a core/admins team defined, please revise!\""},{"line_number":21,"context_line":"            exit 1"},{"line_number":22,"context_line":"        fi"},{"line_number":23,"context_line":"        $OLDPWD/tools/normalize_acl.py $config  0 1 2 3 4 5 \u003e $TMPDIR/normalized"},{"line_number":24,"context_line":"        if ! diff -u $config $TMPDIR/normalized;"},{"line_number":25,"context_line":"        then"},{"line_number":26,"context_line":"            echo \"Project $config is not normalized\""}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a961159_cb3c58e5","line":23,"in_reply_to":"3a961159_eb02f46d","updated":"2014-12-10 20:00:12.000000000","message":"Added it just for completeness","commit_id":"a0fca6892dba2248da7bb288700f9ce6d9c4fde3"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"a87afa6664780f687c72e987a52f1953254d9875","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            \"Project $config does not have a core/admins team defined, please revise!\""},{"line_number":21,"context_line":"            exit 1"},{"line_number":22,"context_line":"        fi"},{"line_number":23,"context_line":"        $OLDPWD/tools/normalize_acl.py $config  0 1 2 3 4 5 6 \\"},{"line_number":24,"context_line":"            \u003e $TMPDIR/normalized"},{"line_number":25,"context_line":"        if ! diff -u $config $TMPDIR/normalized;"},{"line_number":26,"context_line":"        then"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3a961159_b7011f08","line":23,"updated":"2014-12-11 06:17:06.000000000","message":"Anita and I were wondering if we should consolidate all tests in a single file. Having said that, shall we have this patch and the parent one go in as is (after rebase of this one), and refine in another iteration?","commit_id":"072b637b9e2d6d5363245c0d3489eddc64163826"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"0260b231ccf7888bdc3fcfc18ab5bc2e00298cc6","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            \"Project $config does not have a core/admins team defined, please revise!\""},{"line_number":21,"context_line":"            exit 1"},{"line_number":22,"context_line":"        fi"},{"line_number":23,"context_line":"        $OLDPWD/tools/normalize_acl.py $config  0 1 2 3 4 5 6 \\"},{"line_number":24,"context_line":"            \u003e $TMPDIR/normalized"},{"line_number":25,"context_line":"        if ! diff -u $config $TMPDIR/normalized;"},{"line_number":26,"context_line":"        then"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3a961159_549d40bb","line":23,"in_reply_to":"3a961159_6df8cac3","updated":"2014-12-11 17:41:22.000000000","message":"I can take over this one, if you like. I\u0027ll post a patch shortly","commit_id":"072b637b9e2d6d5363245c0d3489eddc64163826"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"0a9eb567f8bfa8decca014fcf67f7c6b0486a5f6","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            \"Project $config does not have a core/admins team defined, please revise!\""},{"line_number":21,"context_line":"            exit 1"},{"line_number":22,"context_line":"        fi"},{"line_number":23,"context_line":"        $OLDPWD/tools/normalize_acl.py $config  0 1 2 3 4 5 6 \\"},{"line_number":24,"context_line":"            \u003e $TMPDIR/normalized"},{"line_number":25,"context_line":"        if ! diff -u $config $TMPDIR/normalized;"},{"line_number":26,"context_line":"        then"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3a961159_6df8cac3","line":23,"in_reply_to":"3a961159_b7011f08","updated":"2014-12-11 07:24:32.000000000","message":"Armando, feel free to propose a different patch or take this over.","commit_id":"072b637b9e2d6d5363245c0d3489eddc64163826"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"54e31af2db81067debf7f67c3abd266094ef7f4c","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    for config in $configs_list; do"},{"line_number":17,"context_line":"        echo \"Checking $config file...\""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"        $OLDPWD/tools/normalize_acl.py $config  0 1 2 3 4 5 6 7 \\"},{"line_number":20,"context_line":"            \u003e $TMPDIR/normalized"},{"line_number":21,"context_line":"        if ! diff -u $config $TMPDIR/normalized;"},{"line_number":22,"context_line":"        then"}],"source_content_type":"text/x-sh","patch_set":5,"id":"3a961159_f969b2be","line":19,"updated":"2014-12-12 07:59:18.000000000","message":"Just one consideration:\nWhat about just saying \"all\" here and enhancing normalize_acl to handle \"all\"?","commit_id":"6d51954e550c7fff1a93fe8101caa4b4d6973abb"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"e86bb253c8b5d0681ee53a98f851ce09c14b9b56","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    for config in $configs_list; do"},{"line_number":17,"context_line":"        echo \"Checking $config file...\""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"        $OLDPWD/tools/normalize_acl.py $config  0 1 2 3 4 5 6 7 \\"},{"line_number":20,"context_line":"            \u003e $TMPDIR/normalized"},{"line_number":21,"context_line":"        if ! diff -u $config $TMPDIR/normalized;"},{"line_number":22,"context_line":"        then"}],"source_content_type":"text/x-sh","patch_set":5,"id":"3a961159_598ba667","line":19,"in_reply_to":"3a961159_f969b2be","updated":"2014-12-12 08:00:49.000000000","message":"good one. I\u0027ll enhance","commit_id":"6d51954e550c7fff1a93fe8101caa4b4d6973abb"}]}