)]}'
{"nova/context.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"a433b822df152e296e31f0629a28ede6b3ee8cfa","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        if service_catalog:"},{"line_number":119,"context_line":"            # Only include required parts of service_catalog"},{"line_number":120,"context_line":"            self.service_catalog \u003d [s for s in service_catalog"},{"line_number":121,"context_line":"                if s.get(\u0027type\u0027) in (\u0027volume\u0027, \u0027volumev2\u0027, \u0027volumev3\u0027,"},{"line_number":122,"context_line":"                                     \u0027key-manager\u0027, \u0027placement\u0027)]"},{"line_number":123,"context_line":"        else:"},{"line_number":124,"context_line":"            # if list is empty or none"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_d873fb15","side":"PARENT","line":121,"range":{"start_line":121,"start_character":48,"end_line":121,"end_character":56},"updated":"2018-06-04 20:32:28.000000000","message":"So I *think* this has broken volume operations on Pike n-cpu under Queens n-api as it strips out volumev2 from the service_catalog back when it was the only volume endpoint available:\n\nLM between OSP 12 and OSP 13 computes fail during major upgrade\nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d1585656\n\nI\u0027ll get an upstream bug created for this now.","commit_id":"253aad457fa3cc7000572ec7a33378c5f6264a1f"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"e8cf6eb06b714834e9ed33adf4282ef3c31eafbf","unresolved":false,"context_lines":[{"line_number":116,"context_line":"        self.timestamp \u003d timestamp"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"        if service_catalog:"},{"line_number":119,"context_line":"            # Only include required parts of service_catalog"},{"line_number":120,"context_line":"            self.service_catalog \u003d [s for s in service_catalog"},{"line_number":121,"context_line":"                if s.get(\u0027type\u0027) in (\u0027block-storage\u0027, \u0027volumev3\u0027,"},{"line_number":122,"context_line":"                                     \u0027key-manager\u0027, \u0027placement\u0027)]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4e5783_75d53db9","line":119,"updated":"2017-10-10 17:08:32.000000000","message":"Is it worth considering not doing this trimming at all, and keeping the entire catalog?\n\nI guess there must have been a reason for this in the past.","commit_id":"a03976ee892333720b2227f607a5ddbb77163fea"}]}