)]}'
{"nova/api/openstack/compute/views/servers.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e0674a84b4827222432f30025a76bc461c292b71","unresolved":false,"context_lines":[{"line_number":118,"context_line":"    def show(self, request, instance, extend_address\u003dTrue,"},{"line_number":119,"context_line":"             show_extra_specs\u003dNone, show_AZ\u003dTrue, show_config_drive\u003dTrue,"},{"line_number":120,"context_line":"             show_extended_attr\u003dNone, show_host_status\u003dNone,"},{"line_number":121,"context_line":"             show_keypair\u003dTrue):"},{"line_number":122,"context_line":"        \"\"\"Detailed view of a single instance.\"\"\""},{"line_number":123,"context_line":"        ip_v4 \u003d instance.get(\u0027access_ip_v4\u0027)"},{"line_number":124,"context_line":"        ip_v6 \u003d instance.get(\u0027access_ip_v6\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3f79a3b5_7436d905","line":121,"range":{"start_line":121,"start_character":18,"end_line":121,"end_character":25},"updated":"2018-08-23 21:40:14.000000000","message":"show_key_name would be a better variable name IMO since a keypair is an object while the key name is just an attribute of that object.","commit_id":"5dd8622f92aedacbc128bf7197607f1bea777647"}],"nova/tests/unit/api/openstack/compute/test_serversV21.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"3d3edc26f94d2e5fad4bc2f453d3be9bc9fdd363","unresolved":false,"context_lines":[{"line_number":6262,"context_line":"                \"OS-EXT-SRV-ATTR:host\": None,"},{"line_number":6263,"context_line":"                \"OS-EXT-SRV-ATTR:hypervisor_hostname\": None,"},{"line_number":6264,"context_line":"                \"OS-EXT-SRV-ATTR:instance_name\": \"instance-00000001\","},{"line_number":6265,"context_line":"                \"key_name\": \u0027\u0027"},{"line_number":6266,"context_line":"            }"},{"line_number":6267,"context_line":"        }"},{"line_number":6268,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"3f79a3b5_a8797cc5","line":6265,"range":{"start_line":6265,"start_character":28,"end_line":6265,"end_character":30},"updated":"2018-08-23 17:48:44.000000000","message":"nit: It would be better to change here to \"my-key\" or something instead of \u0027\u0027 to clarify that should be present, but it is fine to do that with another patch.","commit_id":"5dd8622f92aedacbc128bf7197607f1bea777647"}]}
