)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"dbc5dd538814255e54af546153536c657ad985d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1ac06dbe_ed3cb094","updated":"2016-08-24 07:19:10.000000000","message":"Yes, please link the BP in commit msg.","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"949ec2f5f957f060b44fd0dc33fc4e0944e1527a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1ac06dbe_bacca235","in_reply_to":"1ac06dbe_ed3cb094","updated":"2016-08-24 09:51:31.000000000","message":"Done","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"}],"doc/source/command-objects/port.rst":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        [--host \u003chost-id\u003e]"},{"line_number":27,"context_line":"        [--enable | --disable]"},{"line_number":28,"context_line":"        [--mac-address \u003cmac-address\u003e]"},{"line_number":29,"context_line":"        [--security-group \u003csecurity-group\u003e | --no-security-groups]"},{"line_number":30,"context_line":"        [--project \u003cproject\u003e [--project-domain \u003cproject-domain\u003e]]"},{"line_number":31,"context_line":"        \u003cname\u003e"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_b76defcd","line":29,"range":{"start_line":29,"start_character":45,"end_line":29,"end_character":65},"updated":"2016-09-16 23:47:47.000000000","message":"This should be --no-security-group","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        [--host \u003chost-id\u003e]"},{"line_number":27,"context_line":"        [--enable | --disable]"},{"line_number":28,"context_line":"        [--mac-address \u003cmac-address\u003e]"},{"line_number":29,"context_line":"        [--security-group \u003csecurity-group\u003e | --no-security-groups]"},{"line_number":30,"context_line":"        [--project \u003cproject\u003e [--project-domain \u003cproject-domain\u003e]]"},{"line_number":31,"context_line":"        \u003cname\u003e"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a8bc5a2_dae016fd","line":29,"range":{"start_line":29,"start_character":45,"end_line":29,"end_character":65},"in_reply_to":"7a8ec9b2_b76defcd","updated":"2016-09-26 09:26:17.000000000","message":"Done","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":".. option:: --security-group \u003csecurity-group\u003e"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    Security group name or ID associated with the port. (You can repeat this option)"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. option::  --no-security-groups"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_57b4ab33","line":81,"updated":"2016-09-16 23:47:47.000000000","message":"I suggest\n\nSecurity group to associate with this port (name or ID)\n(repeat option to set multiple security groups)","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":".. option:: --security-group \u003csecurity-group\u003e"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    Security group name or ID associated with the port. (You can repeat this option)"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. option::  --no-security-groups"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a8bc5a2_fae952e7","line":81,"in_reply_to":"7a8ec9b2_57b4ab33","updated":"2016-09-26 09:26:17.000000000","message":"Done. Thanks for your suggestion.","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. option::  --no-security-groups"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    Associate no security groups with the port."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":".. option:: --project \u003cproject\u003e"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_d7a7db08","line":85,"updated":"2016-09-16 23:47:47.000000000","message":"General comment: we normally don\u0027t add periods to single sentence help text.","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":".. option::  --no-security-groups"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    Associate no security groups with the port."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":".. option:: --project \u003cproject\u003e"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a8bc5a2_5aede6ed","line":85,"in_reply_to":"7a8ec9b2_d7a7db08","updated":"2016-09-26 09:26:17.000000000","message":"Done","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":156,"context_line":"        [--host \u003chost-id\u003e]"},{"line_number":157,"context_line":"        [--enable | --disable]"},{"line_number":158,"context_line":"        [--name \u003cname\u003e]"},{"line_number":159,"context_line":"        [--security-group \u003csecurity-group\u003e | --no-security-groups]"},{"line_number":160,"context_line":"        \u003cport\u003e"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":".. option:: --fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_d770bba6","line":159,"range":{"start_line":159,"start_character":45,"end_line":159,"end_character":65},"updated":"2016-09-16 23:47:47.000000000","message":"This should be --no-security-group","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":156,"context_line":"        [--host \u003chost-id\u003e]"},{"line_number":157,"context_line":"        [--enable | --disable]"},{"line_number":158,"context_line":"        [--name \u003cname\u003e]"},{"line_number":159,"context_line":"        [--security-group \u003csecurity-group\u003e | --no-security-groups]"},{"line_number":160,"context_line":"        \u003cport\u003e"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":".. option:: --fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a8bc5a2_9ac12e57","line":159,"range":{"start_line":159,"start_character":45,"end_line":159,"end_character":65},"in_reply_to":"7a8ec9b2_d770bba6","updated":"2016-09-26 09:26:17.000000000","message":"Done","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":211,"context_line":""},{"line_number":212,"context_line":".. option:: --security-group \u003csecurity-group\u003e"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    Security group name or ID associated with the port. (You can repeat this option)"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":".. option::  --no-security-groups"},{"line_number":217,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_77852758","line":214,"updated":"2016-09-16 23:47:47.000000000","message":"Similar comment as above","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":211,"context_line":""},{"line_number":212,"context_line":".. option:: --security-group \u003csecurity-group\u003e"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    Security group name or ID associated with the port. (You can repeat this option)"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":".. option::  --no-security-groups"},{"line_number":217,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a8bc5a2_fad07221","line":214,"in_reply_to":"7a8ec9b2_77852758","updated":"2016-09-26 09:26:17.000000000","message":"Done","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":215,"context_line":""},{"line_number":216,"context_line":".. option::  --no-security-groups"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"    Associate no security groups with the port."},{"line_number":219,"context_line":""},{"line_number":220,"context_line":".. _port_set-port:"},{"line_number":221,"context_line":".. describe:: \u003cport\u003e"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a8ec9b2_3762bfd8","line":218,"range":{"start_line":218,"start_character":4,"end_line":218,"end_character":47},"updated":"2016-09-16 23:47:47.000000000","message":"I suggest \"Clear existing security groups associated with this port.\".","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":215,"context_line":""},{"line_number":216,"context_line":".. option::  --no-security-groups"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"    Associate no security groups with the port."},{"line_number":219,"context_line":""},{"line_number":220,"context_line":".. _port_set-port:"},{"line_number":221,"context_line":".. describe:: \u003cport\u003e"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5a8bc5a2_3ac73a65","line":218,"range":{"start_line":218,"start_character":4,"end_line":218,"end_character":47},"in_reply_to":"7a8ec9b2_3762bfd8","updated":"2016-09-26 09:26:17.000000000","message":"Done. Thanks for your suggestion.","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"cb9a32cc3d676be5b3df75b5d8a5068a4a6e0bf2","unresolved":false,"context_lines":[{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    Port to display (name or ID)"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"port unset"},{"line_number":255,"context_line":"----------"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"Unset port properties"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a98d1f4_2fc7f864","line":254,"updated":"2016-09-30 18:44:25.000000000","message":"Please add unset support.","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"f2f8c522a0f0ef9c3cc3cdaddfc8afa01f7cbd2f","unresolved":false,"context_lines":[{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    Port to display (name or ID)"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"port unset"},{"line_number":255,"context_line":"----------"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"Unset port properties"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a98d1f4_f9005336","line":254,"in_reply_to":"3a98d1f4_2fc7f864","updated":"2016-10-03 09:29:42.000000000","message":"Done","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"81baf6ecb63a7c312baeb10bb3a236e7fa1298c4","unresolved":false,"context_lines":[{"line_number":164,"context_line":"        [--host \u003chost-id\u003e]"},{"line_number":165,"context_line":"        [--enable | --disable]"},{"line_number":166,"context_line":"        [--name \u003cname\u003e]"},{"line_number":167,"context_line":"        [--security-group \u003csecurity-group\u003e | --no-security-group]"},{"line_number":168,"context_line":"        \u003cport\u003e"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":".. option:: --fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa6399be_67e50e36","line":167,"updated":"2016-10-11 18:00:35.000000000","message":"These are not mutually exclusive.","commit_id":"ae12b9757a98203d101345ed9ce2ce383cac8236"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"90fff4b6b5a6de6e613b3863ed483252723c689a","unresolved":false,"context_lines":[{"line_number":164,"context_line":"        [--host \u003chost-id\u003e]"},{"line_number":165,"context_line":"        [--enable | --disable]"},{"line_number":166,"context_line":"        [--name \u003cname\u003e]"},{"line_number":167,"context_line":"        [--security-group \u003csecurity-group\u003e | --no-security-group]"},{"line_number":168,"context_line":"        \u003cport\u003e"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":".. option:: --fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa6399be_300183af","line":167,"in_reply_to":"fa6399be_67e50e36","updated":"2016-10-12 02:09:59.000000000","message":"Done! Thanks.","commit_id":"ae12b9757a98203d101345ed9ce2ce383cac8236"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"81baf6ecb63a7c312baeb10bb3a236e7fa1298c4","unresolved":false,"context_lines":[{"line_number":262,"context_line":"    os port unset"},{"line_number":263,"context_line":"        [--fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e [...]]"},{"line_number":264,"context_line":"        [--binding-profile \u003cbinding-profile-key\u003e [...]]"},{"line_number":265,"context_line":"        [--security-group \u003csecurity-group\u003e]"},{"line_number":266,"context_line":"        \u003cport\u003e"},{"line_number":267,"context_line":""},{"line_number":268,"context_line":".. option:: --fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa6399be_4791ea98","line":265,"updated":"2016-10-11 18:00:35.000000000","message":"Please include [...]","commit_id":"ae12b9757a98203d101345ed9ce2ce383cac8236"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"90fff4b6b5a6de6e613b3863ed483252723c689a","unresolved":false,"context_lines":[{"line_number":262,"context_line":"    os port unset"},{"line_number":263,"context_line":"        [--fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e [...]]"},{"line_number":264,"context_line":"        [--binding-profile \u003cbinding-profile-key\u003e [...]]"},{"line_number":265,"context_line":"        [--security-group \u003csecurity-group\u003e]"},{"line_number":266,"context_line":"        \u003cport\u003e"},{"line_number":267,"context_line":""},{"line_number":268,"context_line":".. option:: --fixed-ip subnet\u003d\u003csubnet\u003e,ip-address\u003d\u003cip-address\u003e"}],"source_content_type":"text/x-rst","patch_set":8,"id":"fa6399be_502c374a","line":265,"in_reply_to":"fa6399be_4791ea98","updated":"2016-10-12 02:09:59.000000000","message":"Done","commit_id":"ae12b9757a98203d101345ed9ce2ce383cac8236"}],"openstackclient/network/v2/port.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ede057a667269b9a52874c22259b8191415be0ed","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        client \u003d client_manager.network"},{"line_number":144,"context_line":"        attrs[\u0027security_groups\u0027] \u003d ["},{"line_number":145,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":146,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":147,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":148,"context_line":"            parsed_args.no_security_groups is not None):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_78f3f79e","line":145,"updated":"2016-08-16 09:49:48.000000000","message":"client is not used anywhere.\nWhy not directly use client_manager.network here ?","commit_id":"1a153df4192a79f5fc7e0af8da8d76071b0294dd"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"991780ad4117d0f25845f352f8942f9c41e0d9cd","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        client \u003d client_manager.network"},{"line_number":144,"context_line":"        attrs[\u0027security_groups\u0027] \u003d ["},{"line_number":145,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":146,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":147,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":148,"context_line":"            parsed_args.no_security_groups is not None):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_00ffb3aa","line":145,"in_reply_to":"3ac371cc_78f3f79e","updated":"2016-08-17 08:02:31.000000000","message":"Thanks for your review.","commit_id":"1a153df4192a79f5fc7e0af8da8d76071b0294dd"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ede057a667269b9a52874c22259b8191415be0ed","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":146,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":147,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":148,"context_line":"            parsed_args.no_security_groups is not None):"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        attrs[\u0027security_groups\u0027] \u003d []"},{"line_number":151,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_9815139d","line":148,"updated":"2016-08-16 09:49:48.000000000","message":"cant we use hasattr ?","commit_id":"1a153df4192a79f5fc7e0af8da8d76071b0294dd"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"991780ad4117d0f25845f352f8942f9c41e0d9cd","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":146,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":147,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":148,"context_line":"            parsed_args.no_security_groups is not None):"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        attrs[\u0027security_groups\u0027] \u003d []"},{"line_number":151,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_40e40be0","line":148,"in_reply_to":"3ac371cc_9815139d","updated":"2016-08-17 08:02:31.000000000","message":"I\u0027m not sure which best way we should use for that. Should we use hasattr at here?","commit_id":"1a153df4192a79f5fc7e0af8da8d76071b0294dd"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"ede057a667269b9a52874c22259b8191415be0ed","unresolved":false,"context_lines":[{"line_number":296,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":297,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":298,"context_line":"            help\u003d_(\"Security group associated with the port. \""},{"line_number":299,"context_line":"                   \"You can repeat this option\")"},{"line_number":300,"context_line":"        )"},{"line_number":301,"context_line":"        secgroups.add_argument("},{"line_number":302,"context_line":"            \u0027--no-security-groups\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_581c3bc1","line":299,"range":{"start_line":299,"start_character":20,"end_line":299,"end_character":46},"updated":"2016-08-16 09:49:48.000000000","message":"Should this be in Parenthesis ? ()","commit_id":"1a153df4192a79f5fc7e0af8da8d76071b0294dd"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"991780ad4117d0f25845f352f8942f9c41e0d9cd","unresolved":false,"context_lines":[{"line_number":296,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":297,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":298,"context_line":"            help\u003d_(\"Security group associated with the port. \""},{"line_number":299,"context_line":"                   \"You can repeat this option\")"},{"line_number":300,"context_line":"        )"},{"line_number":301,"context_line":"        secgroups.add_argument("},{"line_number":302,"context_line":"            \u0027--no-security-groups\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_400c0bed","line":299,"range":{"start_line":299,"start_character":20,"end_line":299,"end_character":46},"in_reply_to":"3ac371cc_581c3bc1","updated":"2016-08-17 08:02:31.000000000","message":"Done","commit_id":"1a153df4192a79f5fc7e0af8da8d76071b0294dd"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"dbc5dd538814255e54af546153536c657ad985d3","unresolved":false,"context_lines":[{"line_number":231,"context_line":"        default\u003d[],"},{"line_number":232,"context_line":"        action\u003d\u0027append\u0027,"},{"line_number":233,"context_line":"        dest\u003d\u0027security_groups\u0027,"},{"line_number":234,"context_line":"        help\u003d_(\"Security group associated with the port. \""},{"line_number":235,"context_line":"               \"(You can repeat this option)\")"},{"line_number":236,"context_line":"    )"},{"line_number":237,"context_line":"    secgroups.add_argument("}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_6d49e030","line":234,"range":{"start_line":234,"start_character":16,"end_line":234,"end_character":29},"updated":"2016-08-24 07:19:10.000000000","message":"name or ID ?","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"949ec2f5f957f060b44fd0dc33fc4e0944e1527a","unresolved":false,"context_lines":[{"line_number":231,"context_line":"        default\u003d[],"},{"line_number":232,"context_line":"        action\u003d\u0027append\u0027,"},{"line_number":233,"context_line":"        dest\u003d\u0027security_groups\u0027,"},{"line_number":234,"context_line":"        help\u003d_(\"Security group associated with the port. \""},{"line_number":235,"context_line":"               \"(You can repeat this option)\")"},{"line_number":236,"context_line":"    )"},{"line_number":237,"context_line":"    secgroups.add_argument("}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_5ad48eea","line":234,"range":{"start_line":234,"start_character":16,"end_line":234,"end_character":29},"in_reply_to":"1ac06dbe_6d49e030","updated":"2016-08-24 09:51:31.000000000","message":"Done","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"dbc5dd538814255e54af546153536c657ad985d3","unresolved":false,"context_lines":[{"line_number":248,"context_line":"        attrs[\u0027security_groups\u0027] \u003d ["},{"line_number":249,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":250,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":251,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":252,"context_line":"            parsed_args.no_security_groups is not None):"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        attrs[\u0027security_groups\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_6d1e000d","line":251,"range":{"start_line":251,"start_character":10,"end_line":251,"end_character":46},"updated":"2016-08-24 07:19:10.000000000","message":"I don\u0027t think we need this check here. no_security_group will be False by default.","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"949ec2f5f957f060b44fd0dc33fc4e0944e1527a","unresolved":false,"context_lines":[{"line_number":248,"context_line":"        attrs[\u0027security_groups\u0027] \u003d ["},{"line_number":249,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":250,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":251,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":252,"context_line":"            parsed_args.no_security_groups is not None):"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        attrs[\u0027security_groups\u0027] \u003d []"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_7ad50ae3","line":251,"range":{"start_line":251,"start_character":10,"end_line":251,"end_character":46},"in_reply_to":"1ac06dbe_6d1e000d","updated":"2016-08-24 09:51:31.000000000","message":"Done","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"dbc5dd538814255e54af546153536c657ad985d3","unresolved":false,"context_lines":[{"line_number":249,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":250,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":251,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":252,"context_line":"            parsed_args.no_security_groups is not None):"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        attrs[\u0027security_groups\u0027] \u003d []"},{"line_number":255,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_0d2f8c3f","line":252,"range":{"start_line":252,"start_character":46,"end_line":252,"end_character":54},"updated":"2016-08-24 07:19:10.000000000","message":"Incorrect here. False is not None as well.","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"949ec2f5f957f060b44fd0dc33fc4e0944e1527a","unresolved":false,"context_lines":[{"line_number":249,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":250,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":251,"context_line":"    elif (\u0027no_security_groups\u0027 in parsed_args and"},{"line_number":252,"context_line":"            parsed_args.no_security_groups is not None):"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        attrs[\u0027security_groups\u0027] \u003d []"},{"line_number":255,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_da06be84","line":252,"range":{"start_line":252,"start_character":46,"end_line":252,"end_character":54},"in_reply_to":"1ac06dbe_0d2f8c3f","updated":"2016-08-24 09:51:31.000000000","message":"Done","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"dbc5dd538814255e54af546153536c657ad985d3","unresolved":false,"context_lines":[{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":"        # TODO(singhj): Add support for extended options:"},{"line_number":316,"context_line":"        # qos,security groups,dhcp, address pairs"},{"line_number":317,"context_line":"        _add_secgroup_option(parser)"},{"line_number":318,"context_line":"        return parser"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_edd83055","line":317,"range":{"start_line":317,"start_character":8,"end_line":317,"end_character":28},"updated":"2016-08-24 07:19:10.000000000","message":"Not necessary to add this function, I think.","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"949ec2f5f957f060b44fd0dc33fc4e0944e1527a","unresolved":false,"context_lines":[{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":"        # TODO(singhj): Add support for extended options:"},{"line_number":316,"context_line":"        # qos,security groups,dhcp, address pairs"},{"line_number":317,"context_line":"        _add_secgroup_option(parser)"},{"line_number":318,"context_line":"        return parser"},{"line_number":319,"context_line":""},{"line_number":320,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_ba65c245","line":317,"range":{"start_line":317,"start_character":8,"end_line":317,"end_character":28},"in_reply_to":"1ac06dbe_edd83055","updated":"2016-08-24 09:51:31.000000000","message":"I add this function to reuse code for \u0027port create\u0027 and \u0027port set\u0027 commands. How do you think?","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":243,"context_line":""},{"line_number":244,"context_line":"def _get_secgroup_attr(client, parsed_args, attrs):"},{"line_number":245,"context_line":"    if parsed_args.security_groups:"},{"line_number":246,"context_line":"        attrs[\u0027security_groups\u0027] \u003d ["},{"line_number":247,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":248,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":249,"context_line":"    elif parsed_args.no_security_groups:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a8ec9b2_77ec472a","line":246,"updated":"2016-09-16 23:47:47.000000000","message":"Please follow \"https://github.com/openstack/python-openstackclient/blob/master/doc/source/command-options.rst\" to ensure options are appended on set.","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":243,"context_line":""},{"line_number":244,"context_line":"def _get_secgroup_attr(client, parsed_args, attrs):"},{"line_number":245,"context_line":"    if parsed_args.security_groups:"},{"line_number":246,"context_line":"        attrs[\u0027security_groups\u0027] \u003d ["},{"line_number":247,"context_line":"            client.find_security_group(sg, ignore_missing\u003dFalse).id"},{"line_number":248,"context_line":"            for sg in parsed_args.security_groups]"},{"line_number":249,"context_line":"    elif parsed_args.no_security_groups:"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a8bc5a2_1a951e3e","line":246,"in_reply_to":"7a8ec9b2_77ec472a","updated":"2016-09-26 09:26:17.000000000","message":"Done","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":309,"context_line":"            help\u003d_(\"Name of this port\")"},{"line_number":310,"context_line":"        )"},{"line_number":311,"context_line":"        # TODO(singhj): Add support for extended options:"},{"line_number":312,"context_line":"        # qos,security groups,dhcp, address pairs"},{"line_number":313,"context_line":"        _add_secgroup_option(parser)"},{"line_number":314,"context_line":"        return parser"},{"line_number":315,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"7a8ec9b2_f74e9739","line":312,"updated":"2016-09-16 23:47:47.000000000","message":"Please remove security groups from the TODO comment.","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":309,"context_line":"            help\u003d_(\"Name of this port\")"},{"line_number":310,"context_line":"        )"},{"line_number":311,"context_line":"        # TODO(singhj): Add support for extended options:"},{"line_number":312,"context_line":"        # qos,security groups,dhcp, address pairs"},{"line_number":313,"context_line":"        _add_secgroup_option(parser)"},{"line_number":314,"context_line":"        return parser"},{"line_number":315,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"5a8bc5a2_3a9e5a63","line":312,"in_reply_to":"7a8ec9b2_f74e9739","updated":"2016-09-26 09:26:17.000000000","message":"Done","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"cb9a32cc3d676be5b3df75b5d8a5068a4a6e0bf2","unresolved":false,"context_lines":[{"line_number":286,"context_line":"        secgroups.add_argument("},{"line_number":287,"context_line":"            \u0027--security-group\u0027,"},{"line_number":288,"context_line":"            metavar\u003d\u0027\u003csecurity-group\u003e\u0027,"},{"line_number":289,"context_line":"            default\u003d[],"},{"line_number":290,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":291,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":292,"context_line":"            help\u003d_(\"Security group associated with the port (name or ID) \""}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_0f95dc3d","line":289,"updated":"2016-09-30 18:44:25.000000000","message":"Is a default needed?","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"f2f8c522a0f0ef9c3cc3cdaddfc8afa01f7cbd2f","unresolved":false,"context_lines":[{"line_number":286,"context_line":"        secgroups.add_argument("},{"line_number":287,"context_line":"            \u0027--security-group\u0027,"},{"line_number":288,"context_line":"            metavar\u003d\u0027\u003csecurity-group\u003e\u0027,"},{"line_number":289,"context_line":"            default\u003d[],"},{"line_number":290,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":291,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":292,"context_line":"            help\u003d_(\"Security group associated with the port (name or ID) \""}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_d970f70d","line":289,"in_reply_to":"3a98d1f4_0f95dc3d","updated":"2016-10-03 09:29:42.000000000","message":"Done! Thanks.","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"cb9a32cc3d676be5b3df75b5d8a5068a4a6e0bf2","unresolved":false,"context_lines":[{"line_number":495,"context_line":"            default\u003d[],"},{"line_number":496,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":497,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":498,"context_line":"            help\u003d_(\"Security group associated with the port (name or ID) \""},{"line_number":499,"context_line":"                   \"(repeat option to set multiple security groups)\")"},{"line_number":500,"context_line":"        )"},{"line_number":501,"context_line":"        secgroups.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_af8068f8","line":498,"range":{"start_line":498,"start_character":51,"end_line":498,"end_character":54},"updated":"2016-09-30 18:44:25.000000000","message":"this","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"cb9a32cc3d676be5b3df75b5d8a5068a4a6e0bf2","unresolved":false,"context_lines":[{"line_number":495,"context_line":"            default\u003d[],"},{"line_number":496,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":497,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":498,"context_line":"            help\u003d_(\"Security group associated with the port (name or ID) \""},{"line_number":499,"context_line":"                   \"(repeat option to set multiple security groups)\")"},{"line_number":500,"context_line":"        )"},{"line_number":501,"context_line":"        secgroups.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_8f8f2c29","line":498,"range":{"start_line":498,"start_character":35,"end_line":498,"end_character":45},"updated":"2016-09-30 18:44:25.000000000","message":"to associate","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"f2f8c522a0f0ef9c3cc3cdaddfc8afa01f7cbd2f","unresolved":false,"context_lines":[{"line_number":495,"context_line":"            default\u003d[],"},{"line_number":496,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":497,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":498,"context_line":"            help\u003d_(\"Security group associated with the port (name or ID) \""},{"line_number":499,"context_line":"                   \"(repeat option to set multiple security groups)\")"},{"line_number":500,"context_line":"        )"},{"line_number":501,"context_line":"        secgroups.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_d9571756","line":498,"range":{"start_line":498,"start_character":35,"end_line":498,"end_character":45},"in_reply_to":"3a98d1f4_8f8f2c29","updated":"2016-10-03 09:29:42.000000000","message":"Done","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"f2f8c522a0f0ef9c3cc3cdaddfc8afa01f7cbd2f","unresolved":false,"context_lines":[{"line_number":495,"context_line":"            default\u003d[],"},{"line_number":496,"context_line":"            action\u003d\u0027append\u0027,"},{"line_number":497,"context_line":"            dest\u003d\u0027security_groups\u0027,"},{"line_number":498,"context_line":"            help\u003d_(\"Security group associated with the port (name or ID) \""},{"line_number":499,"context_line":"                   \"(repeat option to set multiple security groups)\")"},{"line_number":500,"context_line":"        )"},{"line_number":501,"context_line":"        secgroups.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_f9521366","line":498,"range":{"start_line":498,"start_character":51,"end_line":498,"end_character":54},"in_reply_to":"3a98d1f4_af8068f8","updated":"2016-10-03 09:29:42.000000000","message":"Done! Thanks for your suggestion.","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"cb9a32cc3d676be5b3df75b5d8a5068a4a6e0bf2","unresolved":false,"context_lines":[{"line_number":533,"context_line":"        elif parsed_args.no_fixed_ip:"},{"line_number":534,"context_line":"            attrs[\u0027fixed_ips\u0027] \u003d []"},{"line_number":535,"context_line":""},{"line_number":536,"context_line":"        if parsed_args.security_groups and parsed_args.no_security_group:"},{"line_number":537,"context_line":"            attrs[\u0027security_groups\u0027] \u003d [client.find_security_group(sg,"},{"line_number":538,"context_line":"                                        ignore_missing\u003dFalse).id"},{"line_number":539,"context_line":"                                        for sg in parsed_args.security_groups]"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_4fa9a47a","line":536,"updated":"2016-09-30 18:44:25.000000000","message":"These are currently mutually exclusive and should be set together.  Although adding overwrite support would be good.","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"f2f8c522a0f0ef9c3cc3cdaddfc8afa01f7cbd2f","unresolved":false,"context_lines":[{"line_number":533,"context_line":"        elif parsed_args.no_fixed_ip:"},{"line_number":534,"context_line":"            attrs[\u0027fixed_ips\u0027] \u003d []"},{"line_number":535,"context_line":""},{"line_number":536,"context_line":"        if parsed_args.security_groups and parsed_args.no_security_group:"},{"line_number":537,"context_line":"            attrs[\u0027security_groups\u0027] \u003d [client.find_security_group(sg,"},{"line_number":538,"context_line":"                                        ignore_missing\u003dFalse).id"},{"line_number":539,"context_line":"                                        for sg in parsed_args.security_groups]"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_b9d32bd4","line":536,"in_reply_to":"3a98d1f4_4fa9a47a","updated":"2016-10-03 09:29:42.000000000","message":"Thanks for teaching me. I\u0027ve removed mutually exclusive.","commit_id":"1b222d08fffa4b5ab87187001eb0909d20f1bce6"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"81baf6ecb63a7c312baeb10bb3a236e7fa1298c4","unresolved":false,"context_lines":[{"line_number":635,"context_line":"                        sg, ignore_missing\u003dFalse).id"},{"line_number":636,"context_line":"                    tmp_secgroups.remove(sg_id)"},{"line_number":637,"context_line":"            except ValueError:"},{"line_number":638,"context_line":"                msg \u003d _(\"Port dose not contain security group id %s\") % sg"},{"line_number":639,"context_line":"                raise exceptions.CommandError(msg)"},{"line_number":640,"context_line":"            attrs[\u0027security_groups\u0027] \u003d tmp_secgroups"},{"line_number":641,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"fa6399be_8705c2c3","line":638,"range":{"start_line":638,"start_character":30,"end_line":638,"end_character":34},"updated":"2016-10-11 18:00:35.000000000","message":"does","commit_id":"ae12b9757a98203d101345ed9ce2ce383cac8236"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"81baf6ecb63a7c312baeb10bb3a236e7fa1298c4","unresolved":false,"context_lines":[{"line_number":635,"context_line":"                        sg, ignore_missing\u003dFalse).id"},{"line_number":636,"context_line":"                    tmp_secgroups.remove(sg_id)"},{"line_number":637,"context_line":"            except ValueError:"},{"line_number":638,"context_line":"                msg \u003d _(\"Port dose not contain security group id %s\") % sg"},{"line_number":639,"context_line":"                raise exceptions.CommandError(msg)"},{"line_number":640,"context_line":"            attrs[\u0027security_groups\u0027] \u003d tmp_secgroups"},{"line_number":641,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"fa6399be_67018eae","line":638,"range":{"start_line":638,"start_character":62,"end_line":638,"end_character":64},"updated":"2016-10-11 18:00:35.000000000","message":"you can remove \"id\" since this may be a name.","commit_id":"ae12b9757a98203d101345ed9ce2ce383cac8236"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"90fff4b6b5a6de6e613b3863ed483252723c689a","unresolved":false,"context_lines":[{"line_number":635,"context_line":"                        sg, ignore_missing\u003dFalse).id"},{"line_number":636,"context_line":"                    tmp_secgroups.remove(sg_id)"},{"line_number":637,"context_line":"            except ValueError:"},{"line_number":638,"context_line":"                msg \u003d _(\"Port dose not contain security group id %s\") % sg"},{"line_number":639,"context_line":"                raise exceptions.CommandError(msg)"},{"line_number":640,"context_line":"            attrs[\u0027security_groups\u0027] \u003d tmp_secgroups"},{"line_number":641,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"fa6399be_d020474c","line":638,"range":{"start_line":638,"start_character":30,"end_line":638,"end_character":34},"in_reply_to":"fa6399be_8705c2c3","updated":"2016-10-12 02:09:59.000000000","message":"Done","commit_id":"ae12b9757a98203d101345ed9ce2ce383cac8236"}],"openstackclient/tests/network/v2/test_port.py":[{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"dbc5dd538814255e54af546153536c657ad985d3","unresolved":false,"context_lines":[{"line_number":259,"context_line":"        self.assertEqual(ref_columns, columns)"},{"line_number":260,"context_line":"        self.assertEqual(ref_data, data)"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"    def test_create_port_without_secgroups(self):"},{"line_number":263,"context_line":"        arglist \u003d ["},{"line_number":264,"context_line":"            \u0027--network\u0027, self._port.network_id,"},{"line_number":265,"context_line":"            \u0027--no-security-groups\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_cd6bb483","line":262,"range":{"start_line":262,"start_character":8,"end_line":262,"end_character":42},"updated":"2016-08-24 07:19:10.000000000","message":"Please add a test for both --security-group and --no-security-group","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"949ec2f5f957f060b44fd0dc33fc4e0944e1527a","unresolved":false,"context_lines":[{"line_number":259,"context_line":"        self.assertEqual(ref_columns, columns)"},{"line_number":260,"context_line":"        self.assertEqual(ref_data, data)"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"    def test_create_port_without_secgroups(self):"},{"line_number":263,"context_line":"        arglist \u003d ["},{"line_number":264,"context_line":"            \u0027--network\u0027, self._port.network_id,"},{"line_number":265,"context_line":"            \u0027--no-security-groups\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ac06dbe_1aafb67d","line":262,"range":{"start_line":262,"start_character":8,"end_line":262,"end_character":42},"in_reply_to":"1ac06dbe_cd6bb483","updated":"2016-08-24 09:51:31.000000000","message":"Done","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"}],"releasenotes/notes/bug-1612136-051b5f94796e3b51.yaml":[{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"dbc5dd538814255e54af546153536c657ad985d3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add ``security-group`` option to ``os port create`` command."},{"line_number":5,"context_line":"    [Bug `1612136 \u003chttps://bugs.launchpad.net/python-openstackclient/+bug/1612136\u003e`_]"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1ac06dbe_2db6c8f8","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":26},"updated":"2016-08-24 07:19:10.000000000","message":"--security-group and --no-security-group, right?","commit_id":"c82e657e8d1c3ecea15fca28b0c435b04fe583f3"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"612c7626a05b88a4c5110b68d7a5706f55924c1a","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add ``--security-group`` and ``--no-security-groups`` options to"},{"line_number":5,"context_line":"    ``port create`` command."},{"line_number":6,"context_line":"    [Bug `1612136 \u003chttps://bugs.launchpad.net/python-openstackclient/+bug/1612136\u003e`_]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7a8ec9b2_f7e0572b","line":5,"updated":"2016-09-16 23:47:47.000000000","message":"\"port set\" was updated.","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"},{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"87cd7fa9cdd1e48b2439a5e4052ef331ace74d08","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add ``--security-group`` and ``--no-security-groups`` options to"},{"line_number":5,"context_line":"    ``port create`` command."},{"line_number":6,"context_line":"    [Bug `1612136 \u003chttps://bugs.launchpad.net/python-openstackclient/+bug/1612136\u003e`_]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5a8bc5a2_5ac0862a","line":5,"in_reply_to":"7a8ec9b2_f7e0572b","updated":"2016-09-26 09:26:17.000000000","message":"Done","commit_id":"68593089dacf3bebec6e033ae9b34664f9e3b1e7"}]}