)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"a3547213c29984828e49049b3608e20cb93193b0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add regions and zones in ring-builder.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"swift-ring-builder CLI usage does not specify about regions and"},{"line_number":10,"context_line":"zones. This patch adds a description about them. It also adds"},{"line_number":11,"context_line":"a brief description of the flags (report parameter)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: If51a15c0904857abf62636b90ac0c7ed63300871"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a89bdaa_28c6069a","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":5},"updated":"2016-09-07 14:18:46.000000000","message":"nit: may have been mentioned before but the patch does not add anything about zones","commit_id":"568434331e25ed7c722d26ac789cf36991c8ea8e"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"8b3ad84f5d36998d8f6c3180e1f9d25444d89341","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add regions and zones in ring-builder.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"swift-ring-builder CLI usage does not specify about regions and"},{"line_number":10,"context_line":"zones. This patch adds a description about them. It also adds"},{"line_number":11,"context_line":"a brief description of the flags (report parameter)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: If51a15c0904857abf62636b90ac0c7ed63300871"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a89bdaa_3d58018a","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":5},"in_reply_to":"9a89bdaa_28c6069a","updated":"2016-09-07 15:23:29.000000000","message":"Done","commit_id":"568434331e25ed7c722d26ac789cf36991c8ea8e"}],"swift/cli/ringbuilder.py":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"09ac1301b226d0505dd3b9f68c2fa039cfc0b3a7","unresolved":false,"context_lines":[{"line_number":415,"context_line":"    region and are physical hardware whose failure isolates from one"},{"line_number":416,"context_line":"    another. Regions separate the parts of cluster from one another"},{"line_number":417,"context_line":"    with physical boundaries. Cluster sharing multiple regions are"},{"line_number":418,"context_line":"    Multi Region Cluster. Both zone and regions are referenced by a"},{"line_number":419,"context_line":"    positive integer."},{"line_number":420,"context_line":"    Flags:"},{"line_number":421,"context_line":"        DEL - marked for removal and will be removed next rebalance."}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_d95006d7","line":418,"range":{"start_line":418,"start_character":31,"end_line":418,"end_character":35},"updated":"2016-08-04 11:40:39.000000000","message":"nits) s/zone/zones/","commit_id":"a48ff3ec18c386c12469bd939ed220e16a31ea8a"},{"author":{"_account_id":22255,"name":"Tuan","email":"tuanla@vn.fujitsu.com","username":"Tuan"},"change_message_id":"046ae3e7c67a9692046e580f016c2a29b9247025","unresolved":false,"context_lines":[{"line_number":413,"context_line":"    Shows information about the ring and the devices within. Output"},{"line_number":414,"context_line":"    includes a table that describes the report parameters(id, region,"},{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    Flags:"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"},{"line_number":418,"context_line":"              ring and will be removed in next rebalance."},{"line_number":419,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_67e3ef33","line":416,"range":{"start_line":416,"start_character":4,"end_line":416,"end_character":9},"updated":"2016-08-29 15:46:43.000000000","message":"I thinks it should be flags for more connective with you comments.","commit_id":"b3d744609ec3ca17d9464f477beca69dc9a327b8"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"9f652c11ddf16ecc11d30238cb7250dac4059d9c","unresolved":false,"context_lines":[{"line_number":413,"context_line":"    Shows information about the ring and the devices within. Output"},{"line_number":414,"context_line":"    includes a table that describes the report parameters(id, region,"},{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    Flags:"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"},{"line_number":418,"context_line":"              ring and will be removed in next rebalance."},{"line_number":419,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_1b68beb8","line":416,"range":{"start_line":416,"start_character":4,"end_line":416,"end_character":9},"in_reply_to":"fa7ab95a_67e3ef33","updated":"2016-08-29 16:34:47.000000000","message":"Done","commit_id":"b3d744609ec3ca17d9464f477beca69dc9a327b8"},{"author":{"_account_id":12261,"name":"Mahati","email":"mahati.chamarthy@gmail.com","username":"Mahati"},"change_message_id":"f87f5722b2e4a118608d5d19fd5881ef33cf5e57","unresolved":false,"context_lines":[{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    Flags:"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"},{"line_number":418,"context_line":"              ring and will be removed in next rebalance."},{"line_number":419,"context_line":"        \"\"\""},{"line_number":420,"context_line":"        print(\u0027%s, build version %d\u0027 % (builder_file, builder.version))"},{"line_number":421,"context_line":"        regions \u003d 0"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_7d960040","line":418,"updated":"2016-08-23 06:49:34.000000000","message":"Maybe you can include other value for flags which is \u0027\u0027. Something like:\n\nFlags: values are \u0027DEL\u0027 or \u0027\u0027\n    DEL - \u003ctext\u003e","commit_id":"b3d744609ec3ca17d9464f477beca69dc9a327b8"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"9f652c11ddf16ecc11d30238cb7250dac4059d9c","unresolved":false,"context_lines":[{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    Flags:"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"},{"line_number":418,"context_line":"              ring and will be removed in next rebalance."},{"line_number":419,"context_line":"        \"\"\""},{"line_number":420,"context_line":"        print(\u0027%s, build version %d\u0027 % (builder_file, builder.version))"},{"line_number":421,"context_line":"        regions \u003d 0"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_3b6102d4","line":418,"in_reply_to":"1ac06dbe_7d960040","updated":"2016-08-29 16:34:47.000000000","message":"Done","commit_id":"b3d744609ec3ca17d9464f477beca69dc9a327b8"},{"author":{"_account_id":18334,"name":"Nandini Tata","email":"nandini.tata@intel.com","username":"ntata"},"change_message_id":"cc7b726ebd2030128443b2597f284bfd29fcd72b","unresolved":false,"context_lines":[{"line_number":413,"context_line":"    Shows information about the ring and the devices within. Output"},{"line_number":414,"context_line":"    includes a table that describes the report parameters(id, region,"},{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    flags: possible values are \u0027DEL\u0027 and \u0027 \u0027"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"},{"line_number":418,"context_line":"              ring and will be removed in next rebalance."},{"line_number":419,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_3c53516e","line":416,"updated":"2016-08-31 02:42:56.000000000","message":"nit: flags will be an empty string (single quotes without a space in between) when not set to anything","commit_id":"3319084ad3fd50e3befb3d77a3886c850f4fd70f"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"7e79971c4d731b1ca227ca36f48dd7296f63d422","unresolved":false,"context_lines":[{"line_number":413,"context_line":"    Shows information about the ring and the devices within. Output"},{"line_number":414,"context_line":"    includes a table that describes the report parameters(id, region,"},{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    flags: possible values are \u0027DEL\u0027 and \u0027 \u0027"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"},{"line_number":418,"context_line":"              ring and will be removed in next rebalance."},{"line_number":419,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_d96410e7","line":416,"in_reply_to":"fa7ab95a_3c53516e","updated":"2016-08-31 14:10:51.000000000","message":"Done","commit_id":"3319084ad3fd50e3befb3d77a3886c850f4fd70f"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"a3547213c29984828e49049b3608e20cb93193b0","unresolved":false,"context_lines":[{"line_number":411,"context_line":"        \"\"\""},{"line_number":412,"context_line":"swift-ring-builder \u003cbuilder_file\u003e"},{"line_number":413,"context_line":"    Shows information about the ring and the devices within. Output"},{"line_number":414,"context_line":"    includes a table that describes the report parameters(id, region,"},{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    flags: possible values are \u0027DEL\u0027 and \u0027\u0027"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_a883f654","line":414,"range":{"start_line":414,"start_character":47,"end_line":414,"end_character":59},"updated":"2016-09-07 14:18:46.000000000","message":"missing space between parameters and (","commit_id":"568434331e25ed7c722d26ac789cf36991c8ea8e"},{"author":{"_account_id":20508,"name":"Mohit Motiani","email":"mohit.motiani@intel.com","username":"mohit.motiani"},"change_message_id":"8b3ad84f5d36998d8f6c3180e1f9d25444d89341","unresolved":false,"context_lines":[{"line_number":411,"context_line":"        \"\"\""},{"line_number":412,"context_line":"swift-ring-builder \u003cbuilder_file\u003e"},{"line_number":413,"context_line":"    Shows information about the ring and the devices within. Output"},{"line_number":414,"context_line":"    includes a table that describes the report parameters(id, region,"},{"line_number":415,"context_line":"    port, flags, etc)."},{"line_number":416,"context_line":"    flags: possible values are \u0027DEL\u0027 and \u0027\u0027"},{"line_number":417,"context_line":"        DEL - indicates that the device is marked for removal from"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_7d56095d","line":414,"range":{"start_line":414,"start_character":47,"end_line":414,"end_character":59},"in_reply_to":"9a89bdaa_a883f654","updated":"2016-09-07 15:23:29.000000000","message":"Done","commit_id":"568434331e25ed7c722d26ac789cf36991c8ea8e"}]}