)]}'
{"openstackclient/network/v2/network.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"f6f3f3b50fd91c66507e4003829986cb566514ec","unresolved":false,"context_lines":[{"line_number":82,"context_line":"            action\u003d\u0027store_false\u0027,"},{"line_number":83,"context_line":"            help\u003d\u0027Do not share the network between projects\u0027,"},{"line_number":84,"context_line":"        )"},{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project-id\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project_id\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project ID\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_35e62e17","line":85,"updated":"2015-02-17 15:15:04.000000000","message":"To me, it makes a lot more sense to use --os-project-id or --os-project-name.  Why not authtenticate with the project you want to modify?","commit_id":"e44aca61117f5d5e5f18c2685998ad8209833223"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"b66ad2777e6039861deebc580a274d01d2784f66","unresolved":false,"context_lines":[{"line_number":82,"context_line":"            action\u003d\u0027store_false\u0027,"},{"line_number":83,"context_line":"            help\u003d\u0027Do not share the network between projects\u0027,"},{"line_number":84,"context_line":"        )"},{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project-id\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project_id\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project ID\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_1f3a1bcd","line":85,"in_reply_to":"da86d52c_35e62e17","updated":"2015-02-17 19:26:05.000000000","message":"That prevents an admin user from being able to add a network for a project that the are not a member of:\n\nhttp://paste.openstack.org/show/176219/\n\nThis contrasts with, for example, adding a user, where the admin has the power to add users to projects the admin is not a member of:\n\nhttp://paste.openstack.org/show/176218/","commit_id":"e44aca61117f5d5e5f18c2685998ad8209833223"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"0848b0319cbb11a4593df0a3e7b22e2048eff70c","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        )"},{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project name\u0027)"},{"line_number":89,"context_line":"        return parser"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_dc5ebbe8","line":87,"updated":"2015-02-18 07:48:23.000000000","message":"metavar\u003d\u0027\u003cproject\u003e\u0027","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"7109f3bdfdf8e7ea7b13ad94e52e40aa2cf45ffb","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project name\u0027)"},{"line_number":89,"context_line":"        return parser"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_41ad849a","line":88,"updated":"2015-02-18 07:20:59.000000000","message":"This is not required, we can just use the project the user is scoped to.","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"5b675f2d4a2b2e6d206f2191ad3eeaef339e8f92","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project name\u0027)"},{"line_number":89,"context_line":"        return parser"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_fdfb3478","line":88,"in_reply_to":"da86d52c_3c435714","updated":"2015-02-19 17:14:59.000000000","message":"Does it matter if the user is using the v2 API? The help text would display --domain even if it\u0027s not valid for the API they\u0027re using.\n\nTrying to find examples of how to deal with this, I came across \u0027image create\u0027 and \u0027quota set\u0027 which take a project but don\u0027t seem to be able to accept a domain. Is that a bug?","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"0306135099ad8452ade84a37dc9b5cae250dc3c9","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project name\u0027)"},{"line_number":89,"context_line":"        return parser"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_41b2e4e5","line":88,"in_reply_to":"da86d52c_41ad849a","updated":"2015-02-18 07:28:07.000000000","message":"Just read the commit details in more detail.\n\nUsing the project name will not always work, the project name is not globally unique.  \n\nA project name is only unique within a domain","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"0848b0319cbb11a4593df0a3e7b22e2048eff70c","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project name\u0027)"},{"line_number":89,"context_line":"        return parser"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_3c435714","line":88,"in_reply_to":"da86d52c_41b2e4e5","updated":"2015-02-18 07:48:23.000000000","message":"grumble grumble... yes, lin\u0027s right project names may not be enough... \n\nadd an additional parameter --domain (like:\nhttps://github.com/openstack/python-openstackclient/blob/master/openstackclient/identity/v3/project.py#L303-L307\n\nand you can determine the projects ID by doing something like this:\nhttps://github.com/openstack/python-openstackclient/blob/master/openstackclient/identity/v3/project.py#L314-L321\n\n\nalso change the help text to:\n\n  Owner\u0027s project (name or ID)\n\nor some variation, just try to keep (name or ID)","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"b4244331a1fc3cdc27e1310a7db4dcc4159ca96c","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        parser.add_argument("},{"line_number":86,"context_line":"            \u0027--project\u0027,"},{"line_number":87,"context_line":"            metavar\u003d\u0027\u003cnetwork_project\u003e\u0027,"},{"line_number":88,"context_line":"            help\u003d\u0027The owner project name\u0027)"},{"line_number":89,"context_line":"        return parser"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_c323ee55","line":88,"in_reply_to":"da86d52c_fdfb3478","updated":"2015-02-19 18:01:05.000000000","message":"Agree, it would be cleaner if the argument would only be displayed if the user is using a Keystone V3 endpoint.\n\nYou might be able to check the keystone version using:\nauth_ref \u003d self.app.client_manager.auth_ref\nauth_ref.version\n\nI haven\u0027t looked though how to make the CLI argument conditionally displayed.\n\nYes, that sounds like a bug, those might only work when Project ID is passed.","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"0848b0319cbb11a4593df0a3e7b22e2048eff70c","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                \u0027admin_state_up\u0027: parsed_args.admin_state}"},{"line_number":106,"context_line":"        if parsed_args.shared is not None:"},{"line_number":107,"context_line":"            body[\u0027shared\u0027] \u003d parsed_args.shared"},{"line_number":108,"context_line":"        if parsed_args.project is not None:"},{"line_number":109,"context_line":"            identity_client \u003d self.app.client_manager.identity"},{"line_number":110,"context_line":"            project_id \u003d utils.find_resource("},{"line_number":111,"context_line":"                identity_client.tenants,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_3c5a77f2","line":108,"updated":"2015-02-18 07:48:23.000000000","message":"needs to be updated for domain","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"7109f3bdfdf8e7ea7b13ad94e52e40aa2cf45ffb","unresolved":false,"context_lines":[{"line_number":111,"context_line":"                identity_client.tenants,"},{"line_number":112,"context_line":"                parsed_args.project,"},{"line_number":113,"context_line":"            ).id"},{"line_number":114,"context_line":"            body[\u0027tenant_id\u0027] \u003d project_id"},{"line_number":115,"context_line":"        return {\u0027network\u0027: body}"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_01a70cbe","line":114,"updated":"2015-02-18 07:20:59.000000000","message":"The project id is already available when the user logged in.\n\nYou can get the project_id by something like:\n\nauth_ref \u003d self.app.client_manager.auth_ref\nbody[\u0027tenant_id\u0027] \u003d auth_ref.project_id","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"0306135099ad8452ade84a37dc9b5cae250dc3c9","unresolved":false,"context_lines":[{"line_number":111,"context_line":"                identity_client.tenants,"},{"line_number":112,"context_line":"                parsed_args.project,"},{"line_number":113,"context_line":"            ).id"},{"line_number":114,"context_line":"            body[\u0027tenant_id\u0027] \u003d project_id"},{"line_number":115,"context_line":"        return {\u0027network\u0027: body}"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_a1f8004b","line":114,"in_reply_to":"da86d52c_01a70cbe","updated":"2015-02-18 07:28:07.000000000","message":"ignore the comment.\n\nprobably need to ask the user for the domain name too.","commit_id":"9bf32c1eeffb7ddd9be525204898838968e557ca"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"d515be8539c8de41acc3df7dfd26304c794f8440","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from openstackclient.common import exceptions"},{"line_number":24,"context_line":"from openstackclient.common import utils"},{"line_number":25,"context_line":"from openstackclient.network import common"},{"line_number":26,"context_line":"from openstackclient.identity import common as identity_common"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"def _prep_network_detail(net):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_7e56705d","line":26,"updated":"2015-03-14 03:48:32.000000000","message":"need to move this one up","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"6181eefd6857be64cfe22b40a2172c7387f3289d","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":89,"context_line":"            help\u003d\"Owner\u0027s project (name or ID)\")"},{"line_number":90,"context_line":"        api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":91,"context_line":"        if api_version \u003d\u003d \u00273\u0027:"},{"line_number":92,"context_line":"            parser.add_argument("},{"line_number":93,"context_line":"                \u0027--domain\u0027,"},{"line_number":94,"context_line":"                metavar\u003d\u0027\u003cdomain\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_08c0e34e","line":91,"updated":"2015-03-13 22:04:33.000000000","message":"So how do we know if/when neutron requires a domain_id?  If neutron uses v3 but the user wants to use v2 for whatever reason, this will fail.\n\ndo we have a way to discover which Identity API version Neutron is using?","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"5e0371be4c29656afea54fc94a424eaf07ecb072","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":89,"context_line":"            help\u003d\"Owner\u0027s project (name or ID)\")"},{"line_number":90,"context_line":"        api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":91,"context_line":"        if api_version \u003d\u003d \u00273\u0027:"},{"line_number":92,"context_line":"            parser.add_argument("},{"line_number":93,"context_line":"                \u0027--domain\u0027,"},{"line_number":94,"context_line":"                metavar\u003d\u0027\u003cdomain\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_a82d17aa","line":91,"in_reply_to":"9a80dd14_08c0e34e","updated":"2015-03-13 22:10:47.000000000","message":"This will be true if the user specifies `--os-identity-api-version 3` on the command line. I don\u0027t have enough experience to know how neutron interacts with it. Would love to have guidance here.","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ba49f73f2c2098a66187d1e9c56584c3f00fa420","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":89,"context_line":"            help\u003d\"Owner\u0027s project (name or ID)\")"},{"line_number":90,"context_line":"        api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":91,"context_line":"        if api_version \u003d\u003d \u00273\u0027:"},{"line_number":92,"context_line":"            parser.add_argument("},{"line_number":93,"context_line":"                \u0027--domain\u0027,"},{"line_number":94,"context_line":"                metavar\u003d\u0027\u003cdomain\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_6e8b27c3","line":91,"in_reply_to":"9a80dd14_53a7ec0a","updated":"2015-03-13 23:05:22.000000000","message":"In patchset 2 we agreed that it would be cleaner to only add this parameter conditionally.","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"d21659d65f245e1c6889e9580e50bc5aaeb37355","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":89,"context_line":"            help\u003d\"Owner\u0027s project (name or ID)\")"},{"line_number":90,"context_line":"        api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":91,"context_line":"        if api_version \u003d\u003d \u00273\u0027:"},{"line_number":92,"context_line":"            parser.add_argument("},{"line_number":93,"context_line":"                \u0027--domain\u0027,"},{"line_number":94,"context_line":"                metavar\u003d\u0027\u003cdomain\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_21a77dbe","line":91,"in_reply_to":"9a80dd14_6e8b27c3","updated":"2015-03-14 05:20:07.000000000","message":"yeah, sorry if you received incorrect guidance before, i think having the parameter always be there is the right approach.","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"26d168389d6ec373521eeb7ff157d9f59a5fa677","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":89,"context_line":"            help\u003d\"Owner\u0027s project (name or ID)\")"},{"line_number":90,"context_line":"        api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":91,"context_line":"        if api_version \u003d\u003d \u00273\u0027:"},{"line_number":92,"context_line":"            parser.add_argument("},{"line_number":93,"context_line":"                \u0027--domain\u0027,"},{"line_number":94,"context_line":"                metavar\u003d\u0027\u003cdomain\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_53a7ec0a","line":91,"in_reply_to":"9a80dd14_a82d17aa","updated":"2015-03-13 22:17:13.000000000","message":"Seems like this parameter should always be added and let the user decide if it is needed.","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"26d168389d6ec373521eeb7ff157d9f59a5fa677","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        if parsed_args.project is not None:"},{"line_number":116,"context_line":"            identity_client \u003d self.app.client_manager.identity"},{"line_number":117,"context_line":"            api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":118,"context_line":"            if api_version \u003d\u003d \u00273\u0027 and parsed_args.domain is not None:"},{"line_number":119,"context_line":"                domain \u003d identity_common.find_domain(identity_client, parsed_args.domain)"},{"line_number":120,"context_line":"                project_id \u003d utils.find_resource(identity_client.projects,"},{"line_number":121,"context_line":"                                                 parsed_args.project,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_939324a2","line":118,"updated":"2015-03-13 22:17:13.000000000","message":"why don\u0027t we get rid of api_version checking here again and expect the user to know if they need a domain or not.  If they provide one, use it.","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"545f48504a2972d22a985abb7185f5c4fd28d1e7","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        if parsed_args.project is not None:"},{"line_number":116,"context_line":"            identity_client \u003d self.app.client_manager.identity"},{"line_number":117,"context_line":"            api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":118,"context_line":"            if api_version \u003d\u003d \u00273\u0027 and parsed_args.domain is not None:"},{"line_number":119,"context_line":"                domain \u003d identity_common.find_domain(identity_client, parsed_args.domain)"},{"line_number":120,"context_line":"                project_id \u003d utils.find_resource(identity_client.projects,"},{"line_number":121,"context_line":"                                                 parsed_args.project,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_f49d1c0b","line":118,"in_reply_to":"9a80dd14_939324a2","updated":"2015-03-14 19:46:20.000000000","message":"I think that\u0027s the right approach, but we need to be sure the user gets a useful error message when it fails.  We will need the api version check then, and I need to work out a not-private way to get that","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"d515be8539c8de41acc3df7dfd26304c794f8440","unresolved":false,"context_lines":[{"line_number":116,"context_line":"            identity_client \u003d self.app.client_manager.identity"},{"line_number":117,"context_line":"            api_version \u003d self.app.client_manager._api_version.get(\u0027identity\u0027)"},{"line_number":118,"context_line":"            if api_version \u003d\u003d \u00273\u0027 and parsed_args.domain is not None:"},{"line_number":119,"context_line":"                domain \u003d identity_common.find_domain(identity_client, parsed_args.domain)"},{"line_number":120,"context_line":"                project_id \u003d utils.find_resource(identity_client.projects,"},{"line_number":121,"context_line":"                                                 parsed_args.project,"},{"line_number":122,"context_line":"                                                 domain_id\u003ddomain.id).id"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_5e5c4c7d","line":119,"updated":"2015-03-14 03:48:32.000000000","message":"pep8 not happy about this line, it is more than 80 chars","commit_id":"781c342252f7df58a2739e2ded72489597a2a95a"}],"openstackclient/tests/fakes.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"b3e1fcaf40ddcdc60f40d007ddc7b1475dbbf4cb","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        self.session \u003d None"},{"line_number":100,"context_line":"        self.auth_ref \u003d None"},{"line_number":101,"context_line":"        self.auth_plugin_name \u003d None"},{"line_number":102,"context_line":"        self._api_version \u003d {}"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"class FakeModule(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_793f3c87","line":102,"updated":"2015-03-16 15:19:06.000000000","message":"Is this still needed?","commit_id":"3eb2a4170c5ea60317caa8a3f27c7edd1476a315"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"36b36e365b7433e2d7dd237c305ac999b8aa9d17","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        self.session \u003d None"},{"line_number":100,"context_line":"        self.auth_ref \u003d None"},{"line_number":101,"context_line":"        self.auth_plugin_name \u003d None"},{"line_number":102,"context_line":"        self._api_version \u003d {}"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"class FakeModule(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_73b4ab82","line":102,"in_reply_to":"9a80dd14_793f3c87","updated":"2015-03-16 20:09:33.000000000","message":"Looks like not, I\u0027ll remove it.","commit_id":"3eb2a4170c5ea60317caa8a3f27c7edd1476a315"}]}