)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"2ed73d0918d7d1a84a1f836a845da6133df1c5ed","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Convert existing service-* to coe-service-*"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLI are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and potentially"},{"line_number":11,"context_line":"become coe-specific, going forward. Thus moving them to be coe-service-*,"},{"line_number":12,"context_line":"instead."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa1b9901_4b8f9a75","line":10,"updated":"2015-08-21 21:20:10.000000000","message":"Minor : potentially -\u003e will potentially","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":15311,"name":"Surojit Pathak","email":"surojit.pathak@gmail.com","username":"suropathak"},"change_message_id":"6ec805fbbe60d1a95ca57c5268a6d29161f0dd81","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Convert existing service-* to coe-service-*"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLI are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and potentially"},{"line_number":11,"context_line":"become coe-specific, going forward. Thus moving them to be coe-service-*,"},{"line_number":12,"context_line":"instead."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa1b9901_25ad6a88","line":10,"in_reply_to":"fa1b9901_4b8f9a75","updated":"2015-08-24 19:19:11.000000000","message":"Done","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"9703b9f8ae9742c02a2c863686fe351f265b769d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Convert existing service-* to coe-service-*"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLI are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and potentially"},{"line_number":11,"context_line":"become coe-specific, going forward. Thus moving them to be coe-service-*,"},{"line_number":12,"context_line":"instead."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa1b9901_cba96a92","line":10,"in_reply_to":"fa1b9901_4b8f9a75","updated":"2015-08-21 21:26:26.000000000","message":"or and potentially *becomes* coe-specific","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":15311,"name":"Surojit Pathak","email":"surojit.pathak@gmail.com","username":"suropathak"},"change_message_id":"6ec805fbbe60d1a95ca57c5268a6d29161f0dd81","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Convert existing service-* to coe-service-*"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLI are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and potentially"},{"line_number":11,"context_line":"become coe-specific, going forward. Thus moving them to be coe-service-*,"},{"line_number":12,"context_line":"instead."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa1b9901_45a89e77","line":10,"in_reply_to":"fa1b9901_cba96a92","updated":"2015-08-24 19:19:11.000000000","message":"Done","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"2ed73d0918d7d1a84a1f836a845da6133df1c5ed","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLI are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and potentially"},{"line_number":11,"context_line":"become coe-specific, going forward. Thus moving them to be coe-service-*,"},{"line_number":12,"context_line":"instead."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia4ee95e324087309b7f771b3a9d4930b8d8589cf"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa1b9901_0b81a24d","line":11,"updated":"2015-08-21 21:20:10.000000000","message":"1. Minor : Please follow the resomendations here https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure\n\n2. IMHO \u0027,\u0027 is unnecessary","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":15311,"name":"Surojit Pathak","email":"surojit.pathak@gmail.com","username":"suropathak"},"change_message_id":"6ec805fbbe60d1a95ca57c5268a6d29161f0dd81","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLI are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and potentially"},{"line_number":11,"context_line":"become coe-specific, going forward. Thus moving them to be coe-service-*,"},{"line_number":12,"context_line":"instead."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia4ee95e324087309b7f771b3a9d4930b8d8589cf"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa1b9901_05b946cb","line":11,"in_reply_to":"fa1b9901_0b81a24d","updated":"2015-08-24 19:19:11.000000000","message":"Done","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"56d4c72e12302694bcd13ebe1d5ba98e7658e1a2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Convert existing service-* to coe-service-*"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLIs are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and may"},{"line_number":11,"context_line":"potentially become coe-specific, going forward. Thus, moving them to be"},{"line_number":12,"context_line":"coe-service-*."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa1b9901_61c6ec4a","line":10,"updated":"2015-08-25 00:32:03.000000000","message":"i think you should write related to or specific to coe or coe type instead of writing related to or specific to Kubernetes. because that\u0027s confusing we will have coe types swarm, kubernetes and mesos, may be more in future.","commit_id":"5a5d6d5720edc766787c3530f2617cc97b4b2591"},{"author":{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"change_message_id":"0e89b8a891f3d853166701a2f8be09dd3b05eeaf","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Convert existing service-* to coe-service-*"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As part of the blueprint, we concluded that all the service-* CLIs are"},{"line_number":10,"context_line":"related to or specific to Kubernetes at this moment, and may"},{"line_number":11,"context_line":"potentially become coe-specific, going forward. Thus, moving them to be"},{"line_number":12,"context_line":"coe-service-*."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa1b9901_873b10eb","line":10,"in_reply_to":"fa1b9901_61c6ec4a","updated":"2015-08-25 01:52:54.000000000","message":"I think it is right at now, service-* CLIs are kubernetes specific now.","commit_id":"5a5d6d5720edc766787c3530f2617cc97b4b2591"},{"author":{"_account_id":7049,"name":"Kai Qiang Wu(Kennan)","email":"wkq5325@gmail.com","username":"Kennan"},"change_message_id":"0e89b8a891f3d853166701a2f8be09dd3b05eeaf","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia4ee95e324087309b7f771b3a9d4930b8d8589cf"},{"line_number":15,"context_line":"Partially-Implements: blueprint magnum-service-list"},{"line_number":16,"context_line":"Closes-bug: 1487580"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa1b9901_0759405a","line":16,"updated":"2015-08-25 01:52:54.000000000","message":"Use this format Closes-Bug:  #number","commit_id":"5a5d6d5720edc766787c3530f2617cc97b4b2591"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"fa529f22be466e5da9f3d284150df7dfe2f66624","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"At present, all \u0027magnum service-*\u0027 CLIs are specific to Kubernetes."},{"line_number":10,"context_line":"As a community, we anticipate that other COEs may introduce \u0027service\u0027"},{"line_number":11,"context_line":"concept too, going forward. Thus, moving \u0027service-*\u0027 CLIs to be \u0027coe-service-*\u0027,"},{"line_number":12,"context_line":"instead of \u0027k8s-service-*\u0027."},{"line_number":13,"context_line":"This is also required to model \u0027magnum service-list\u0027 after \u0027nova"},{"line_number":14,"context_line":"service-list\u0027.[1]"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa1b9901_db8d397d","line":11,"updated":"2015-08-25 18:43:08.000000000","message":"Please follow guidelines here\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"e92fc0220db3f372ae56f2cd1cf715852d0f1d6f"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"8c67600183b7363dd1cfba512f94a2c8d4a49746","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"At present, all \u0027magnum service-*\u0027 CLIs are specific to Kubernetes."},{"line_number":10,"context_line":"As a community, we anticipate that other COEs may introduce \u0027service\u0027"},{"line_number":11,"context_line":"concept too, going forward. Thus, moving \u0027service-*\u0027 CLIs to be \u0027coe-service-*\u0027,"},{"line_number":12,"context_line":"instead of \u0027k8s-service-*\u0027."},{"line_number":13,"context_line":"This is also required to model \u0027magnum service-list\u0027 after \u0027nova"},{"line_number":14,"context_line":"service-list\u0027.[1]"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa1b9901_e1316aee","line":11,"in_reply_to":"fa1b9901_3e3ea328","updated":"2015-08-25 19:25:10.000000000","message":"72 char limit","commit_id":"e92fc0220db3f372ae56f2cd1cf715852d0f1d6f"},{"author":{"_account_id":15311,"name":"Surojit Pathak","email":"surojit.pathak@gmail.com","username":"suropathak"},"change_message_id":"259ab60c099956085d476df10e88de81a1f565d4","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"At present, all \u0027magnum service-*\u0027 CLIs are specific to Kubernetes."},{"line_number":10,"context_line":"As a community, we anticipate that other COEs may introduce \u0027service\u0027"},{"line_number":11,"context_line":"concept too, going forward. Thus, moving \u0027service-*\u0027 CLIs to be \u0027coe-service-*\u0027,"},{"line_number":12,"context_line":"instead of \u0027k8s-service-*\u0027."},{"line_number":13,"context_line":"This is also required to model \u0027magnum service-list\u0027 after \u0027nova"},{"line_number":14,"context_line":"service-list\u0027.[1]"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa1b9901_3e3ea328","line":11,"in_reply_to":"fa1b9901_db8d397d","updated":"2015-08-25 19:15:01.000000000","message":"Can you please be specific on the guideline violation?\nAre you suggesting each line should be equally wrapped at 72 char?\nOr do you object the sequence of Change-Id, Closes-bug, blueprint?","commit_id":"e92fc0220db3f372ae56f2cd1cf715852d0f1d6f"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"fa529f22be466e5da9f3d284150df7dfe2f66624","unresolved":false,"context_lines":[{"line_number":13,"context_line":"This is also required to model \u0027magnum service-list\u0027 after \u0027nova"},{"line_number":14,"context_line":"service-list\u0027.[1]"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[1] - https://blueprints.launchpad.net/magnum/+spec/magnum-service-list"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ia4ee95e324087309b7f771b3a9d4930b8d8589cf"},{"line_number":19,"context_line":"Partially-Implements: blueprint magnum-service-list"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa1b9901_9bd1015a","line":16,"updated":"2015-08-25 18:43:08.000000000","message":"Since you have already included name of blueprint under \"Partially-Implements\" IMHO we don\u0027t need to mention it here again.","commit_id":"e92fc0220db3f372ae56f2cd1cf715852d0f1d6f"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"1cc13ff179e38111fc2b445fbd8abb6b0fa75d26","unresolved":false,"context_lines":[{"line_number":13,"context_line":"This is also required to model \u0027magnum service-list\u0027 after \u0027nova"},{"line_number":14,"context_line":"service-list\u0027.[1]"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[1] - https://blueprints.launchpad.net/magnum/+spec/magnum-service-list"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ia4ee95e324087309b7f771b3a9d4930b8d8589cf"},{"line_number":19,"context_line":"Partially-Implements: blueprint magnum-service-list"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa1b9901_015d7e14","line":16,"in_reply_to":"fa1b9901_7e7e5b6b","updated":"2015-08-25 19:26:34.000000000","message":"No concerns about Partially-Implements but not sure we need to include the link explicitly.","commit_id":"e92fc0220db3f372ae56f2cd1cf715852d0f1d6f"},{"author":{"_account_id":15311,"name":"Surojit Pathak","email":"surojit.pathak@gmail.com","username":"suropathak"},"change_message_id":"259ab60c099956085d476df10e88de81a1f565d4","unresolved":false,"context_lines":[{"line_number":13,"context_line":"This is also required to model \u0027magnum service-list\u0027 after \u0027nova"},{"line_number":14,"context_line":"service-list\u0027.[1]"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[1] - https://blueprints.launchpad.net/magnum/+spec/magnum-service-list"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ia4ee95e324087309b7f771b3a9d4930b8d8589cf"},{"line_number":19,"context_line":"Partially-Implements: blueprint magnum-service-list"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa1b9901_7e7e5b6b","line":16,"in_reply_to":"fa1b9901_9bd1015a","updated":"2015-08-25 19:15:01.000000000","message":"From the previous set of review comments, I felt it would be helpful to provide an explicit link to indicate that the blueprint captures the details of the design decision.\u0027\n\nIt seems \u0027Partially-Implements\u0027 is implicit in nature.","commit_id":"e92fc0220db3f372ae56f2cd1cf715852d0f1d6f"}],"magnumclient/v1/shell.py":[{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"b1efa584eab411f0d3b2d66e6eed76ceb5c4860a","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    utils.print_dict(rc._info)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def _show_coe_service(service):"},{"line_number":55,"context_line":"    utils.print_dict(service._info)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_c48e1827","line":54,"updated":"2015-08-24 03:42:42.000000000","message":"I think that we should keep \"magnum sevice-xxx\" for at least one release to keep backward compatibility. Can we just mark \"magnum service-xxx\" as deprecated and then remove those command in next release?","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":15311,"name":"Surojit Pathak","email":"surojit.pathak@gmail.com","username":"suropathak"},"change_message_id":"2c43a845d10e260281db6debfbe01a3b543ebf99","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    utils.print_dict(rc._info)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def _show_coe_service(service):"},{"line_number":55,"context_line":"    utils.print_dict(service._info)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_a5089a29","line":54,"in_reply_to":"fa1b9901_1a2540c1","updated":"2015-08-24 19:14:53.000000000","message":"+1, Tom\n\nJay,\nIf required we can append the following for a release at the end of output of \u0027magnum service-list\u0027\n\n\"NOTE: For kubernetes services, check \u0027magnum coe-service-list\u0027\"","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"5ff5409e8ab8751471d15aa317b56447788f3eb1","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    utils.print_dict(rc._info)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def _show_coe_service(service):"},{"line_number":55,"context_line":"    utils.print_dict(service._info)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_81cf3555","line":54,"in_reply_to":"fa1b9901_a5089a29","updated":"2015-08-26 08:27:17.000000000","message":"OK, I think that magnum may not have user using in production, so we can delete the command directly.","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"},{"author":{"_account_id":11650,"name":"Tom Cammann","email":"tom.cammann@hpe.com","username":"cammann"},"change_message_id":"f3822a284f6e41c281e6593f1680986ab788fcc5","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    utils.print_dict(rc._info)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def _show_coe_service(service):"},{"line_number":55,"context_line":"    utils.print_dict(service._info)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_1a2540c1","line":54,"in_reply_to":"fa1b9901_c48e1827","updated":"2015-08-24 11:14:43.000000000","message":"Isn\u0027t this the whole point, so that we can then use \"magnum service-list\" to get info on services running in magnum and not display a deprecated message?","commit_id":"ef61a98174388bceb6c4c808c35b9e0b1be38f98"}]}