)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"297d7c9e834b0f36cb6f8d330602ccf80c38b092","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Minor change for typo and indentation for patch 80b11279b"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the patch 80b11279b, there is a typo and indentation issue,"},{"line_number":10,"context_line":"This patch just fix it."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ie84d83183b13f99ffb33213684b9d9b8a4e59b42"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2FEio%3D","line":9,"updated":"2013-11-14 20:13:33.000000000","message":"For other reviewers that weren\u0027t on the patch, it\u0027s probably more useful to point this out in the commit message for context, i.e. this patch is addressing Gary\u0027s comments from a cherry pick patch so that wasn\u0027t the place to address them, this is.\n\nhttps://review.openstack.org/#/c/53528/","commit_id":"d8163378ba6986b0a275135cc5818dc9a5181b28"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d79e6984a8989d0a81eb124767e5ef5c006fcb66","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Minor change for typo and indentation for patch 80b11279b"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the patch 80b11279b, there is a typo and indentation issue,"},{"line_number":10,"context_line":"This patch just fix it."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ie84d83183b13f99ffb33213684b9d9b8a4e59b42"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2FDbs%3D","line":9,"in_reply_to":"AAAATn%2F%2FDcY%3D","updated":"2013-11-15 04:46:19.000000000","message":"No, I don\u0027t mean you need to call out Gary specifically, I just meant linking to that stable branch review where the comments were made.","commit_id":"d8163378ba6986b0a275135cc5818dc9a5181b28"},{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"86fae8f9f51ea35bd34ffde207f9ae5431de1694","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Minor change for typo and indentation for patch 80b11279b"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the patch 80b11279b, there is a typo and indentation issue,"},{"line_number":10,"context_line":"This patch just fix it."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ie84d83183b13f99ffb33213684b9d9b8a4e59b42"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2FDcY%3D","line":9,"in_reply_to":"AAAATn%2F%2FEio%3D","updated":"2013-11-15 04:32:58.000000000","message":"I don\u0027t think adds the \"addressing Gary\u0027s comments\" in commit message is a good idea, but I agree to make it clear, so I will add a comments in this patchset.","commit_id":"d8163378ba6986b0a275135cc5818dc9a5181b28"}],"nova/api/openstack/xmlutil.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"d1de1d1010cb04a234492b622091d69e81563c04","unresolved":false,"context_lines":[{"line_number":160,"context_line":"                            This is used to further refine the datum"},{"line_number":161,"context_line":"                            object returned by selector in the event"},{"line_number":162,"context_line":"                            that it is a list of objects."},{"line_number":163,"context_line":"        :colon_ns: An optional flag indicates whether to support k:v"},{"line_number":164,"context_line":"                    type tagname, if so the k:v type tagname will"},{"line_number":165,"context_line":"                    be supported by adding the k into namespace."},{"line_number":166,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2BIlQ%3D","line":163,"updated":"2013-12-02 07:06:27.000000000","message":"the wording for this is still a little awkward. While fixing this perhaps its worth rewording the whole sentence.","commit_id":"39984a85760c3901e187280e7f8693dc266b3f89"},{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"63439e7fe4d6f13be8cecdd4aa45a3f2d473d21c","unresolved":false,"context_lines":[{"line_number":160,"context_line":"                            This is used to further refine the datum"},{"line_number":161,"context_line":"                            object returned by selector in the event"},{"line_number":162,"context_line":"                            that it is a list of objects."},{"line_number":163,"context_line":"        :colon_ns: An optional flag indicates whether to support k:v"},{"line_number":164,"context_line":"                    type tagname, if so the k:v type tagname will"},{"line_number":165,"context_line":"                    be supported by adding the k into namespace."},{"line_number":166,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2BAug%3D","line":163,"in_reply_to":"AAAATn%2F%2BBgs%3D","updated":"2013-12-03 14:21:03.000000000","message":"Done, will address it in new patch, thanks.","commit_id":"39984a85760c3901e187280e7f8693dc266b3f89"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"b8e62933fa8f4f53e0594ad501ad00e23c778691","unresolved":false,"context_lines":[{"line_number":160,"context_line":"                            This is used to further refine the datum"},{"line_number":161,"context_line":"                            object returned by selector in the event"},{"line_number":162,"context_line":"                            that it is a list of objects."},{"line_number":163,"context_line":"        :colon_ns: An optional flag indicates whether to support k:v"},{"line_number":164,"context_line":"                    type tagname, if so the k:v type tagname will"},{"line_number":165,"context_line":"                    be supported by adding the k into namespace."},{"line_number":166,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2BA7Q%3D","line":163,"in_reply_to":"AAAATn%2F%2BBgs%3D","updated":"2013-12-03 13:31:59.000000000","message":"I like it.","commit_id":"39984a85760c3901e187280e7f8693dc266b3f89"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"5d7ca2928aecf52cb59f19e0aa469471749e49b6","unresolved":false,"context_lines":[{"line_number":160,"context_line":"                            This is used to further refine the datum"},{"line_number":161,"context_line":"                            object returned by selector in the event"},{"line_number":162,"context_line":"                            that it is a list of objects."},{"line_number":163,"context_line":"        :colon_ns: An optional flag indicates whether to support k:v"},{"line_number":164,"context_line":"                    type tagname, if so the k:v type tagname will"},{"line_number":165,"context_line":"                    be supported by adding the k into namespace."},{"line_number":166,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2BBgs%3D","line":163,"in_reply_to":"AAAATn%2F%2BIlQ%3D","updated":"2013-12-03 10:50:32.000000000","message":"How about:\n\n\"An optional flag indicating whether to support k:v type tagnames. If True, the k:v type tagname will be supported by adding k into the namespace.\"","commit_id":"39984a85760c3901e187280e7f8693dc266b3f89"}],"nova/tests/api/openstack/test_xmlutil.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"8b746d35bbed9ccacad92b842c42ab64cc2b0f97","unresolved":false,"context_lines":[{"line_number":715,"context_line":"                    \u0027\u003cextra_specs\u003e\u003cfoo:bar xmlns:foo\u003d\"foo\"\u003e999\u003c/foo:bar\u003e\u0027"},{"line_number":716,"context_line":"                    \u0027\u003c/extra_specs\u003e\u0027))"},{"line_number":717,"context_line":"        # Set up our master template"},{"line_number":718,"context_line":"        tag_name \u003d \u0027extra_specs\u0027"},{"line_number":719,"context_line":"        root \u003d xmlutil.TemplateElement(tag_name, selector\u003d\u0027extra_specs\u0027,"},{"line_number":720,"context_line":"                                       colon_ns\u003dTrue)"},{"line_number":721,"context_line":"        value \u003d xmlutil.SubTemplateElement(root, \u0027foo:bar\u0027, selector\u003d\u0027foo:bar\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FE3k%3D","line":718,"updated":"2013-11-14 18:31:16.000000000","message":"huh? why this change?","commit_id":"d8163378ba6986b0a275135cc5818dc9a5181b28"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"297d7c9e834b0f36cb6f8d330602ccf80c38b092","unresolved":false,"context_lines":[{"line_number":715,"context_line":"                    \u0027\u003cextra_specs\u003e\u003cfoo:bar xmlns:foo\u003d\"foo\"\u003e999\u003c/foo:bar\u003e\u0027"},{"line_number":716,"context_line":"                    \u0027\u003c/extra_specs\u003e\u0027))"},{"line_number":717,"context_line":"        # Set up our master template"},{"line_number":718,"context_line":"        tag_name \u003d \u0027extra_specs\u0027"},{"line_number":719,"context_line":"        root \u003d xmlutil.TemplateElement(tag_name, selector\u003d\u0027extra_specs\u0027,"},{"line_number":720,"context_line":"                                       colon_ns\u003dTrue)"},{"line_number":721,"context_line":"        value \u003d xmlutil.SubTemplateElement(root, \u0027foo:bar\u0027, selector\u003d\u0027foo:bar\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FEjA%3D","line":718,"in_reply_to":"AAAATn%2F%2FE3k%3D","updated":"2013-11-14 20:13:33.000000000","message":"Agreed, not needed.","commit_id":"d8163378ba6986b0a275135cc5818dc9a5181b28"},{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"86fae8f9f51ea35bd34ffde207f9ae5431de1694","unresolved":false,"context_lines":[{"line_number":715,"context_line":"                    \u0027\u003cextra_specs\u003e\u003cfoo:bar xmlns:foo\u003d\"foo\"\u003e999\u003c/foo:bar\u003e\u0027"},{"line_number":716,"context_line":"                    \u0027\u003c/extra_specs\u003e\u0027))"},{"line_number":717,"context_line":"        # Set up our master template"},{"line_number":718,"context_line":"        tag_name \u003d \u0027extra_specs\u0027"},{"line_number":719,"context_line":"        root \u003d xmlutil.TemplateElement(tag_name, selector\u003d\u0027extra_specs\u0027,"},{"line_number":720,"context_line":"                                       colon_ns\u003dTrue)"},{"line_number":721,"context_line":"        value \u003d xmlutil.SubTemplateElement(root, \u0027foo:bar\u0027, selector\u003d\u0027foo:bar\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FDcM%3D","line":718,"in_reply_to":"AAAATn%2F%2FE3k%3D","updated":"2013-11-15 04:32:58.000000000","message":"Done","commit_id":"d8163378ba6986b0a275135cc5818dc9a5181b28"}]}