)]}'
{"openstack/cloud/_compute.py":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"b949a3a4b5310c82ba089dcb925ded5eadf4117d","unresolved":true,"context_lines":[{"line_number":1594,"context_line":"        aggregate \u003d self.compute.find_aggregate("},{"line_number":1595,"context_line":"            name_or_id, ignore_missing\u003dFalse)"},{"line_number":1596,"context_line":"        if aggregate:"},{"line_number":1597,"context_line":"            return aggregate._to_munch()"},{"line_number":1598,"context_line":""},{"line_number":1599,"context_line":"    def create_aggregate(self, name, availability_zone\u003dNone):"},{"line_number":1600,"context_line":"        \"\"\"Create a new host aggregate."}],"source_content_type":"text/x-python","patch_set":2,"id":"bb78e9b9_4299818b","line":1597,"range":{"start_line":1597,"start_character":28,"end_line":1597,"end_character":38},"updated":"2020-12-16 18:27:53.000000000","message":"Do we really even need _to_munch here? Shouldn\u0027t the Resource object behave like a munch since it\u0027s a subclass?","commit_id":"32e9730ace26edccd67508aa82af82cdfff36c67"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"f83c8166fe5b227c6c8fe4ce39a6ecfef3f94c33","unresolved":true,"context_lines":[{"line_number":1594,"context_line":"        aggregate \u003d self.compute.find_aggregate("},{"line_number":1595,"context_line":"            name_or_id, ignore_missing\u003dFalse)"},{"line_number":1596,"context_line":"        if aggregate:"},{"line_number":1597,"context_line":"            return aggregate._to_munch()"},{"line_number":1598,"context_line":""},{"line_number":1599,"context_line":"    def create_aggregate(self, name, availability_zone\u003dNone):"},{"line_number":1600,"context_line":"        \"\"\"Create a new host aggregate."}],"source_content_type":"text/x-python","patch_set":2,"id":"e8728350_efa3e20a","line":1597,"range":{"start_line":1597,"start_character":28,"end_line":1597,"end_character":38},"in_reply_to":"bb78e9b9_4299818b","updated":"2020-12-16 18:37:02.000000000","message":"It behaves like munch, but it is not munch. That causes weird things on the client side. Additionally in some cases we need original_names, while in others - not. And then, when we have caching we must have munch, otherwise dogpile exits trying to make use of it. I do not like it either, but for consistency we should have it. In the long run (after switching all cloud functions to proxy) we can replace it everywhere, but perhaps breaking our contract (R1.0?)","commit_id":"32e9730ace26edccd67508aa82af82cdfff36c67"}]}