)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"212c3067a7d24d623755061afb8f9cd3b6c72674","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix, default prefix length,"},{"line_number":10,"context_line":"minimun prefix length and maximun prefix length for a subnet pool."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_c3738ab3","line":9,"updated":"2016-03-07 07:53:42.000000000","message":"Too long, lets keep it at 72 chars per line","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"e7db3ead0d20ee5817c1e96adcc9b010722c9059","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix, default prefix length,"},{"line_number":10,"context_line":"minimun prefix length and maximun prefix length for a subnet pool."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_8634f0de","line":9,"in_reply_to":"5aef4532_c3738ab3","updated":"2016-03-07 08:25:48.000000000","message":"Done","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"212c3067a7d24d623755061afb8f9cd3b6c72674","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix, default prefix length,"},{"line_number":10,"context_line":"minimun prefix length and maximun prefix length for a subnet pool."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397"},{"line_number":13,"context_line":"Partial-Bug: #1544591"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_a378fe90","line":10,"updated":"2016-03-07 07:53:42.000000000","message":"minimun -\u003e minimum","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"e7db3ead0d20ee5817c1e96adcc9b010722c9059","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix, default prefix length,"},{"line_number":10,"context_line":"minimun prefix length and maximun prefix length for a subnet pool."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397"},{"line_number":13,"context_line":"Partial-Bug: #1544591"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_0609002d","line":10,"in_reply_to":"5aef4532_a378fe90","updated":"2016-03-07 08:25:48.000000000","message":"Done","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"a2a0ce37cfd6ee66735d1f1e7e17dfa3adbd9c73","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix,"},{"line_number":10,"context_line":"default prefix length, minimun prefix length,"},{"line_number":11,"context_line":" and maximum prefix length for a subnet pool."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_46d148fb","line":9,"updated":"2016-03-07 08:29:00.000000000","message":"nit: This patch supports setting a new name, pool prefix ...\n\n\nSorry I should have seen it earlier","commit_id":"a4d72faa84bb240c3143a2f0a07bd2915dbf4a95"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"3deb3aa397fc75b01f92ad89aa8b666454e63e00","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix,"},{"line_number":10,"context_line":"default prefix length, minimun prefix length,"},{"line_number":11,"context_line":" and maximum prefix length for a subnet pool."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_b13048a0","line":9,"in_reply_to":"5aef4532_46d148fb","updated":"2016-03-07 08:52:48.000000000","message":"Done","commit_id":"a4d72faa84bb240c3143a2f0a07bd2915dbf4a95"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"a2a0ce37cfd6ee66735d1f1e7e17dfa3adbd9c73","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix,"},{"line_number":10,"context_line":"default prefix length, minimun prefix length,"},{"line_number":11,"context_line":" and maximum prefix length for a subnet pool."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_a6080c93","line":10,"range":{"start_line":10,"start_character":23,"end_line":10,"end_character":30},"updated":"2016-03-07 08:29:00.000000000","message":"Another one I missed earlier\n\nMinimun -\u003e Minimum","commit_id":"a4d72faa84bb240c3143a2f0a07bd2915dbf4a95"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"3deb3aa397fc75b01f92ad89aa8b666454e63e00","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix,"},{"line_number":10,"context_line":"default prefix length, minimun prefix length,"},{"line_number":11,"context_line":" and maximum prefix length for a subnet pool."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_d1d23434","line":10,"range":{"start_line":10,"start_character":23,"end_line":10,"end_character":30},"in_reply_to":"5aef4532_a6080c93","updated":"2016-03-07 08:52:48.000000000","message":"So sorry that I was in a hurry. :)","commit_id":"a4d72faa84bb240c3143a2f0a07bd2915dbf4a95"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"876f37bc15825694f6d9fb476c9aa046e32a1b5d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[Subnet pool] Add \u0027subnet pool set\u0027 command support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch supports set new name, pool prefix,"},{"line_number":10,"context_line":"default prefix length, minimun prefix length,"},{"line_number":11,"context_line":" and maximum prefix length for a subnet pool."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I65bd71e0f54f2f65acefbc542df67a1b1ec26397"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_d17874fb","line":10,"range":{"start_line":10,"start_character":23,"end_line":10,"end_character":30},"in_reply_to":"5aef4532_d1d23434","updated":"2016-03-07 08:54:55.000000000","message":"Ok, relax :)","commit_id":"a4d72faa84bb240c3143a2f0a07bd2915dbf4a95"}],"doc/source/command-objects/subnet-pool.rst":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"d51b939906eb810929875e4f35ecfa095569927d","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    Set subnet pool prefixes (in CIDR notation)."},{"line_number":61,"context_line":"    Repeat this option to set multiple prefixes."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":".. option:: --default-prefixlen \u003cdefault-prefixlen\u003e"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    Set subnet pool default prefix length"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"5aef4532_cd1c0cc4","line":63,"range":{"start_line":63,"start_character":22,"end_line":63,"end_character":31},"updated":"2016-03-08 12:25:12.000000000","message":"i feel like all of this \"prefixlen\" arguments should be \"prefix-len\" since they are two separate words, or even \"prefix-length\" to make it more clear.","commit_id":"ebde407b05e9c759c9fdec6ffbd7043c81f227ad"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"f27d5d27b5eb5f6d8be6fc0329c14768028538ca","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    Set subnet pool prefixes (in CIDR notation)."},{"line_number":61,"context_line":"    Repeat this option to set multiple prefixes."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":".. option:: --default-prefixlen \u003cdefault-prefixlen\u003e"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    Set subnet pool default prefix length"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"5aef4532_03936550","line":63,"range":{"start_line":63,"start_character":22,"end_line":63,"end_character":31},"in_reply_to":"5aef4532_68132667","updated":"2016-03-08 13:02:32.000000000","message":"Let\u0027s rename it using full spelled words from now on. And let\u0027s fix all the other options if they are not fully spelled.","commit_id":"ebde407b05e9c759c9fdec6ffbd7043c81f227ad"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"31d8b1c2ca97b8c6a18611b0276c615712229c3d","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    Set subnet pool prefixes (in CIDR notation)."},{"line_number":61,"context_line":"    Repeat this option to set multiple prefixes."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":".. option:: --default-prefixlen \u003cdefault-prefixlen\u003e"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    Set subnet pool default prefix length"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"5aef4532_68132667","line":63,"range":{"start_line":63,"start_character":22,"end_line":63,"end_character":31},"in_reply_to":"5aef4532_c8735a14","updated":"2016-03-08 12:50:49.000000000","message":"i don\u0027t see if being used anywhere as an *option* just yet. https://github.com/openstack/python-openstackclient/blob/d01525f5e53f0651973ed735449b632552d6d55d/openstackclient/network/v2/subnet_pool.py just when we are getting the list field","commit_id":"ebde407b05e9c759c9fdec6ffbd7043c81f227ad"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"749dcda029e40dd70906bd7b91797a7dba016769","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    Set subnet pool prefixes (in CIDR notation)."},{"line_number":61,"context_line":"    Repeat this option to set multiple prefixes."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":".. option:: --default-prefixlen \u003cdefault-prefixlen\u003e"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    Set subnet pool default prefix length"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"5aef4532_c8735a14","line":63,"range":{"start_line":63,"start_character":22,"end_line":63,"end_character":31},"in_reply_to":"5aef4532_cd1c0cc4","updated":"2016-03-08 12:44:06.000000000","message":"I also had the same feeling, but I thought that this is being used everywhere, so avoided a change in previously committed patches","commit_id":"ebde407b05e9c759c9fdec6ffbd7043c81f227ad"}],"openstackclient/network/v2/subnet_pool.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"212c3067a7d24d623755061afb8f9cd3b6c72674","unresolved":false,"context_lines":[{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"def _get_attrs(client_manager, parsed_args):"},{"line_number":36,"context_line":"    attrs \u003d {}"},{"line_number":37,"context_line":"    if parsed_args.name is not None:"},{"line_number":38,"context_line":"        attrs[\u0027name\u0027] \u003d str(parsed_args.name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_a33bde9b","line":35,"updated":"2016-03-07 07:53:42.000000000","message":"Where is the client_manager used in this function?\nIf it is not used, do we need to pass it?","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"e7db3ead0d20ee5817c1e96adcc9b010722c9059","unresolved":false,"context_lines":[{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"def _get_attrs(client_manager, parsed_args):"},{"line_number":36,"context_line":"    attrs \u003d {}"},{"line_number":37,"context_line":"    if parsed_args.name is not None:"},{"line_number":38,"context_line":"        attrs[\u0027name\u0027] \u003d str(parsed_args.name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_23098e32","line":35,"in_reply_to":"5aef4532_a33bde9b","updated":"2016-03-07 08:25:48.000000000","message":"Oh, I just copied it and forgot to remove it. You are right, we don\u0027t need it at all.","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"212c3067a7d24d623755061afb8f9cd3b6c72674","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            dest\u003d\u0027prefixes\u0027,"},{"line_number":138,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":139,"context_line":"            help\u003d\u0027Set subnet pool prefixes (in CIDR notation). \u0027"},{"line_number":140,"context_line":"                 \u0027(This option can be repeated)\u0027,"},{"line_number":141,"context_line":"        )"},{"line_number":142,"context_line":"        parser.add_argument("},{"line_number":143,"context_line":"            \u0027--default-prefixlen\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_43201a4a","line":140,"range":{"start_line":140,"start_character":47,"end_line":140,"end_character":48},"updated":"2016-03-07 07:53:42.000000000","message":"the doc file has a Full stop/period after \"(This option can be repeated)\"\nLets keep it consistent","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"e7db3ead0d20ee5817c1e96adcc9b010722c9059","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            dest\u003d\u0027prefixes\u0027,"},{"line_number":138,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":139,"context_line":"            help\u003d\u0027Set subnet pool prefixes (in CIDR notation). \u0027"},{"line_number":140,"context_line":"                 \u0027(This option can be repeated)\u0027,"},{"line_number":141,"context_line":"        )"},{"line_number":142,"context_line":"        parser.add_argument("},{"line_number":143,"context_line":"            \u0027--default-prefixlen\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_63039610","line":140,"range":{"start_line":140,"start_character":47,"end_line":140,"end_character":48},"in_reply_to":"5aef4532_43201a4a","updated":"2016-03-07 08:25:48.000000000","message":"Done","commit_id":"7fa26cda021d6358fdc1072908d99a1efe7176b3"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"f0a617b8ce241664b702174007b04817210915ab","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        help\u003d\u0027Set subnet pool maximum prefix length\u0027,"},{"line_number":77,"context_line":"    )"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    return parser"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"class DeleteSubnetPool(command.Command):"}],"source_content_type":"text/x-python","patch_set":6,"id":"5aef4532_9b308bea","line":79,"updated":"2016-03-08 07:27:59.000000000","message":"You dont need to return the parser.\nIt would be updated in-memory.\nIt wont lose its contents when accessed after the function completes executing","commit_id":"2bdc0683fd76d6e8943065cf33792b14aeb700cc"}]}