)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"42de12aa4a5f17b943f3d9eee4b5b62e821fbd21","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Using floating-ip-{associate|disassociate}"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This path implement a blueprint to add more consistent into"},{"line_number":10,"context_line":"nova command, especially for the subcommand \"floating-ip-*\""},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Currently when we want to associate or disassociate an ip with"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F80Is%3D","line":9,"updated":"2013-12-20 01:46:16.000000000","message":"path \u003d\u003e patch.\n\nThis commit message is a little confusing and can use better phrasing","commit_id":"f47d4ade4cbf46fe44ae410279122342a794490a"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"7cc68baf31386a30be4f6817d0a8f5dadd8df991","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Using floating-ip-{associate|disassociate}"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This path implement a blueprint to add more consistent into"},{"line_number":10,"context_line":"nova command, especially for the subcommand \"floating-ip-*\""},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Currently when we want to associate or disassociate an ip with"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F8y4s%3D","line":9,"in_reply_to":"AAAATn%2F80Is%3D","updated":"2013-12-20 08:39:16.000000000","message":"Done","commit_id":"f47d4ade4cbf46fe44ae410279122342a794490a"},{"author":{"_account_id":7770,"name":"Abhishek Chanda","email":"abhishek.becs@gmail.com","username":"abhishek"},"change_message_id":"cd7f19a1657c7e0ff07cf157345dc30362e7a095","unresolved":false,"context_lines":[{"line_number":22,"context_line":"add-floating-ip or remove-floating-ip."},{"line_number":23,"context_line":"In the client v3 this patch replace add-floating-ip with"},{"line_number":24,"context_line":"associate-floating-ip and remove-floating-ip with disassociate-floating-ip."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Implement blueprint commands-floating-ip"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"ubuntu@workspace:~/python-novaclient$ nova help | grep floating"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F893A%3D","line":25,"updated":"2013-12-18 10:09:51.000000000","message":"Needs a DocImpact tag","commit_id":"f47d4ade4cbf46fe44ae410279122342a794490a"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"7f962d173e31b7d3488530438d1c119a8c472858","unresolved":false,"context_lines":[{"line_number":22,"context_line":"add-floating-ip or remove-floating-ip."},{"line_number":23,"context_line":"In the client v3 this patch replace add-floating-ip with"},{"line_number":24,"context_line":"associate-floating-ip and remove-floating-ip with disassociate-floating-ip."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Implement blueprint commands-floating-ip"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"ubuntu@workspace:~/python-novaclient$ nova help | grep floating"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F89wU%3D","line":25,"in_reply_to":"AAAATn%2F893A%3D","updated":"2013-12-18 10:26:17.000000000","message":"Actually I think there is no impact with the documentation. \nThe doc is based on the command \u0027nova help\u0027\n\nhttps://github.com/openstack/python-novaclient/tree/master/doc\n\nWhat do you think?","commit_id":"f47d4ade4cbf46fe44ae410279122342a794490a"},{"author":{"_account_id":7770,"name":"Abhishek Chanda","email":"abhishek.becs@gmail.com","username":"abhishek"},"change_message_id":"2d014741281cc541289ab636347cb3b145229aab","unresolved":false,"context_lines":[{"line_number":22,"context_line":"add-floating-ip or remove-floating-ip."},{"line_number":23,"context_line":"In the client v3 this patch replace add-floating-ip with"},{"line_number":24,"context_line":"associate-floating-ip and remove-floating-ip with disassociate-floating-ip."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Implement blueprint commands-floating-ip"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"ubuntu@workspace:~/python-novaclient$ nova help | grep floating"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F89p4%3D","line":25,"in_reply_to":"AAAATn%2F89wU%3D","updated":"2013-12-18 10:50:55.000000000","message":"My bad. Removing the -1.","commit_id":"f47d4ade4cbf46fe44ae410279122342a794490a"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"a0ce9bc16b1b9ee427ef8731983576fdc84f3297","unresolved":false,"context_lines":[{"line_number":23,"context_line":" + In the client v3 this patch replaces add-floating-ip with"},{"line_number":24,"context_line":"   floating-ip-associate and remove-floating-ip with floating-ip-disassociate."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Implement blueprint commands-floating-ip"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"$:~/python-novaclient$ nova help | grep floating"},{"line_number":29,"context_line":"    add-floating-ip     DEPRECATED, use floating-ip-associate instead."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F8qEg%3D","line":26,"updated":"2013-12-23 16:07:16.000000000","message":"This should be in the same block of text as the Change-Id line.  All machine parsable lines should be stuck together.","commit_id":"7d8a6c37734ddd7b7b00fbdd3207b5d9f0d9d066"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"6b6dbe6f25baa4e1dccf87297f37a86a043053bd","unresolved":false,"context_lines":[{"line_number":23,"context_line":" + In the client v3 this patch replaces add-floating-ip with"},{"line_number":24,"context_line":"   floating-ip-associate and remove-floating-ip with floating-ip-disassociate."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Implement blueprint commands-floating-ip"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"$:~/python-novaclient$ nova help | grep floating"},{"line_number":29,"context_line":"    add-floating-ip     DEPRECATED, use floating-ip-associate instead."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F8pdg%3D","line":26,"in_reply_to":"AAAATn%2F8qEg%3D","updated":"2013-12-23 18:34:14.000000000","message":"Done","commit_id":"7d8a6c37734ddd7b7b00fbdd3207b5d9f0d9d066"}],"novaclient/v1_1/shell.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"42de12aa4a5f17b943f3d9eee4b5b62e821fbd21","unresolved":false,"context_lines":[{"line_number":1911,"context_line":"           default\u003dNone,"},{"line_number":1912,"context_line":"           help\u003d\u0027Fixed IP Address to associate with.\u0027)"},{"line_number":1913,"context_line":"def do_add_floating_ip(cs, args):"},{"line_number":1914,"context_line":"    \"\"\"DEPRECATED, use floating-ip-associate instead.\"\"\""},{"line_number":1915,"context_line":"    _associate_floating_ip(cs, args)"},{"line_number":1916,"context_line":""},{"line_number":1917,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F80II%3D","line":1914,"updated":"2013-12-20 01:46:16.000000000","message":"this should also print that this is deprecated, so users will see it.\n\nperhaps print to stderr so that any bash scripts that use this won\u0027t break","commit_id":"f47d4ade4cbf46fe44ae410279122342a794490a"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"7cc68baf31386a30be4f6817d0a8f5dadd8df991","unresolved":false,"context_lines":[{"line_number":1911,"context_line":"           default\u003dNone,"},{"line_number":1912,"context_line":"           help\u003d\u0027Fixed IP Address to associate with.\u0027)"},{"line_number":1913,"context_line":"def do_add_floating_ip(cs, args):"},{"line_number":1914,"context_line":"    \"\"\"DEPRECATED, use floating-ip-associate instead.\"\"\""},{"line_number":1915,"context_line":"    _associate_floating_ip(cs, args)"},{"line_number":1916,"context_line":""},{"line_number":1917,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F8y3M%3D","line":1914,"in_reply_to":"AAAATn%2F80II%3D","updated":"2013-12-20 08:39:16.000000000","message":"I\u0027m not sure that the deprecated message should be printed into stderr, since it is not generated by an error that occurred. \nIHMO, the deprecated warning should be part of the whole help message like in other clients.","commit_id":"f47d4ade4cbf46fe44ae410279122342a794490a"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"f9f6b5e7918b71ef93217bb0369b35fb09e9d61c","unresolved":false,"context_lines":[{"line_number":1911,"context_line":"           default\u003dNone,"},{"line_number":1912,"context_line":"           help\u003d\u0027Fixed IP Address to associate with.\u0027)"},{"line_number":1913,"context_line":"def do_add_floating_ip(cs, args):"},{"line_number":1914,"context_line":"    \"\"\"DEPRECATED, use floating-ip-associate instead.\"\"\""},{"line_number":1915,"context_line":"    _associate_floating_ip(cs, args)"},{"line_number":1916,"context_line":""},{"line_number":1917,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F8wXI%3D","line":1914,"updated":"2013-12-20 16:36:40.000000000","message":"stderr is the traditional stream for reporting warnings, such as the warning that a given command or command argument is deprecated and that you should use _x_.  That said, I don\u0027t feel strongly that novaclient MUST emit a warning when you use a deprecated command, but there\u0027s no reason not to.","commit_id":"7d8a6c37734ddd7b7b00fbdd3207b5d9f0d9d066"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"6b6dbe6f25baa4e1dccf87297f37a86a043053bd","unresolved":false,"context_lines":[{"line_number":1911,"context_line":"           default\u003dNone,"},{"line_number":1912,"context_line":"           help\u003d\u0027Fixed IP Address to associate with.\u0027)"},{"line_number":1913,"context_line":"def do_add_floating_ip(cs, args):"},{"line_number":1914,"context_line":"    \"\"\"DEPRECATED, use floating-ip-associate instead.\"\"\""},{"line_number":1915,"context_line":"    _associate_floating_ip(cs, args)"},{"line_number":1916,"context_line":""},{"line_number":1917,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F8pdc%3D","line":1914,"in_reply_to":"AAAATn%2F8qFk%3D","updated":"2013-12-23 18:34:14.000000000","message":"Done","commit_id":"7d8a6c37734ddd7b7b00fbdd3207b5d9f0d9d066"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"a0ce9bc16b1b9ee427ef8731983576fdc84f3297","unresolved":false,"context_lines":[{"line_number":1911,"context_line":"           default\u003dNone,"},{"line_number":1912,"context_line":"           help\u003d\u0027Fixed IP Address to associate with.\u0027)"},{"line_number":1913,"context_line":"def do_add_floating_ip(cs, args):"},{"line_number":1914,"context_line":"    \"\"\"DEPRECATED, use floating-ip-associate instead.\"\"\""},{"line_number":1915,"context_line":"    _associate_floating_ip(cs, args)"},{"line_number":1916,"context_line":""},{"line_number":1917,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F8qFk%3D","line":1914,"in_reply_to":"AAAATn%2F8uDE%3D","updated":"2013-12-23 16:07:16.000000000","message":"If a command is going to be deprecated I think it should notify a user, but I think that the help string is enough to satisfy this requirement.","commit_id":"7d8a6c37734ddd7b7b00fbdd3207b5d9f0d9d066"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"e84c08af12a751c2463ee5031487f68b6b7512be","unresolved":false,"context_lines":[{"line_number":1911,"context_line":"           default\u003dNone,"},{"line_number":1912,"context_line":"           help\u003d\u0027Fixed IP Address to associate with.\u0027)"},{"line_number":1913,"context_line":"def do_add_floating_ip(cs, args):"},{"line_number":1914,"context_line":"    \"\"\"DEPRECATED, use floating-ip-associate instead.\"\"\""},{"line_number":1915,"context_line":"    _associate_floating_ip(cs, args)"},{"line_number":1916,"context_line":""},{"line_number":1917,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F8uDE%3D","line":1914,"in_reply_to":"AAAATn%2F8wXI%3D","updated":"2013-12-21 08:39:26.000000000","message":"Thanks for all of this information. I\u0027m waiting for the community to know what I have todo.","commit_id":"7d8a6c37734ddd7b7b00fbdd3207b5d9f0d9d066"}]}