)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"6edfc32c19e7bc7fbb8e894395dc85e1fa6727c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"64a3a5db_412dff84","updated":"2022-11-11 04:36:45.000000000","message":"recheck unrelated error","commit_id":"d3c5ddd40fde9018962d700a06c2bea16459f5f3"}],"plugins/modules/compute_flavor.py":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"405f62007f1cb6ea5d292f17472a2e030ad11236","unresolved":true,"context_lines":[{"line_number":187,"context_line":""},{"line_number":188,"context_line":"class ComputeFlavorModule(OpenStackModule):"},{"line_number":189,"context_line":"    argument_spec \u003d dict("},{"line_number":190,"context_line":"        disk\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":191,"context_line":"        ephemeral\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":192,"context_line":"        extra_specs\u003ddict(type\u003d\u0027dict\u0027),"},{"line_number":193,"context_line":"        id\u003ddict(aliases\u003d[\u0027flavorid\u0027]),"}],"source_content_type":"text/x-python","patch_set":2,"id":"44b399be_1fe10bce","line":190,"range":{"start_line":190,"start_character":8,"end_line":190,"end_character":30},"updated":"2022-11-09 13:26:38.000000000","message":"# required when state is \u0027present\u0027","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"aa1e16b29fab2269cff4e501fef7518cceaac807","unresolved":false,"context_lines":[{"line_number":187,"context_line":""},{"line_number":188,"context_line":"class ComputeFlavorModule(OpenStackModule):"},{"line_number":189,"context_line":"    argument_spec \u003d dict("},{"line_number":190,"context_line":"        disk\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":191,"context_line":"        ephemeral\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":192,"context_line":"        extra_specs\u003ddict(type\u003d\u0027dict\u0027),"},{"line_number":193,"context_line":"        id\u003ddict(aliases\u003d[\u0027flavorid\u0027]),"}],"source_content_type":"text/x-python","patch_set":2,"id":"ca1e6104_2566a874","line":190,"range":{"start_line":190,"start_character":8,"end_line":190,"end_character":30},"in_reply_to":"44b399be_1fe10bce","updated":"2022-11-09 13:50:18.000000000","message":"Dropped the comment here because module_kwargs.required_if is enough and because i added it to DOCUMENTATION above.","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"405f62007f1cb6ea5d292f17472a2e030ad11236","unresolved":true,"context_lines":[{"line_number":192,"context_line":"        extra_specs\u003ddict(type\u003d\u0027dict\u0027),"},{"line_number":193,"context_line":"        id\u003ddict(aliases\u003d[\u0027flavorid\u0027]),"},{"line_number":194,"context_line":"        is_public\u003ddict(type\u003d\u0027bool\u0027),"},{"line_number":195,"context_line":"        name\u003ddict(required\u003dTrue),  # required when state is \u0027present\u0027"},{"line_number":196,"context_line":"        ram\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":197,"context_line":"        rxtx_factor\u003ddict(type\u003d\u0027float\u0027),"},{"line_number":198,"context_line":"        state\u003ddict(default\u003d\u0027present\u0027, choices\u003d[\u0027absent\u0027, \u0027present\u0027]),"}],"source_content_type":"text/x-python","patch_set":2,"id":"b72b334c_b8d7ddfe","line":195,"range":{"start_line":195,"start_character":36,"end_line":195,"end_character":68},"updated":"2022-11-09 13:26:38.000000000","message":"I suppose it\u0027s relevant for row below with \u0027ram\u0027, not for \"name\"?","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"aa1e16b29fab2269cff4e501fef7518cceaac807","unresolved":false,"context_lines":[{"line_number":192,"context_line":"        extra_specs\u003ddict(type\u003d\u0027dict\u0027),"},{"line_number":193,"context_line":"        id\u003ddict(aliases\u003d[\u0027flavorid\u0027]),"},{"line_number":194,"context_line":"        is_public\u003ddict(type\u003d\u0027bool\u0027),"},{"line_number":195,"context_line":"        name\u003ddict(required\u003dTrue),  # required when state is \u0027present\u0027"},{"line_number":196,"context_line":"        ram\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":197,"context_line":"        rxtx_factor\u003ddict(type\u003d\u0027float\u0027),"},{"line_number":198,"context_line":"        state\u003ddict(default\u003d\u0027present\u0027, choices\u003d[\u0027absent\u0027, \u0027present\u0027]),"}],"source_content_type":"text/x-python","patch_set":2,"id":"37020c34_30a0012d","line":195,"range":{"start_line":195,"start_character":36,"end_line":195,"end_character":68},"in_reply_to":"b72b334c_b8d7ddfe","updated":"2022-11-09 13:50:18.000000000","message":"Oh wow, i dont know what i did here. This comment was actually attached to ram and vcpus because those do not have a default like disk has. this comment was not meant to be attached to name, because name is required always... \n\nSo in the end, i simply dropped that comment because its \"documented\" in module_kwargs.required_if below and now also in DOCUMENTATION above.","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"405f62007f1cb6ea5d292f17472a2e030ad11236","unresolved":true,"context_lines":[{"line_number":197,"context_line":"        rxtx_factor\u003ddict(type\u003d\u0027float\u0027),"},{"line_number":198,"context_line":"        state\u003ddict(default\u003d\u0027present\u0027, choices\u003d[\u0027absent\u0027, \u0027present\u0027]),"},{"line_number":199,"context_line":"        swap\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":200,"context_line":"        vcpus\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":201,"context_line":"    )"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    module_kwargs \u003d dict("}],"source_content_type":"text/x-python","patch_set":2,"id":"c59a8c3f_77af7179","line":200,"range":{"start_line":200,"start_character":8,"end_line":200,"end_character":13},"updated":"2022-11-09 13:26:38.000000000","message":"# required when state is \u0027present\u0027","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"aa1e16b29fab2269cff4e501fef7518cceaac807","unresolved":false,"context_lines":[{"line_number":197,"context_line":"        rxtx_factor\u003ddict(type\u003d\u0027float\u0027),"},{"line_number":198,"context_line":"        state\u003ddict(default\u003d\u0027present\u0027, choices\u003d[\u0027absent\u0027, \u0027present\u0027]),"},{"line_number":199,"context_line":"        swap\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":200,"context_line":"        vcpus\u003ddict(type\u003d\u0027int\u0027),"},{"line_number":201,"context_line":"    )"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    module_kwargs \u003d dict("}],"source_content_type":"text/x-python","patch_set":2,"id":"bc2f17bc_603ed580","line":200,"range":{"start_line":200,"start_character":8,"end_line":200,"end_character":13},"in_reply_to":"c59a8c3f_77af7179","updated":"2022-11-09 13:50:18.000000000","message":"In module_kwargs.required_if and now also in DOCUMENTATION above.","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"405f62007f1cb6ea5d292f17472a2e030ad11236","unresolved":true,"context_lines":[{"line_number":346,"context_line":"        elif state \u003d\u003d \u0027present\u0027 and flavor:"},{"line_number":347,"context_line":"            return bool(self._build_update(flavor))"},{"line_number":348,"context_line":"        elif state \u003d\u003d \u0027absent\u0027 and flavor:"},{"line_number":349,"context_line":"            return False"},{"line_number":350,"context_line":"        else:"},{"line_number":351,"context_line":"            # state \u003d\u003d \u0027absent\u0027 and not flavor:"},{"line_number":352,"context_line":"            return True"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":""},{"line_number":355,"context_line":"def main():"}],"source_content_type":"text/x-python","patch_set":2,"id":"d0820ec0_9aab4dab","line":352,"range":{"start_line":349,"start_character":0,"end_line":352,"end_character":23},"updated":"2022-11-09 13:26:38.000000000","message":"seems like an opposite?\n       \n        elif state \u003d\u003d \u0027absent\u0027 and flavor:\n            return True\n        else:\n            # state \u003d\u003d \u0027absent\u0027 and not flavor:\n            return False","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"aa1e16b29fab2269cff4e501fef7518cceaac807","unresolved":false,"context_lines":[{"line_number":346,"context_line":"        elif state \u003d\u003d \u0027present\u0027 and flavor:"},{"line_number":347,"context_line":"            return bool(self._build_update(flavor))"},{"line_number":348,"context_line":"        elif state \u003d\u003d \u0027absent\u0027 and flavor:"},{"line_number":349,"context_line":"            return False"},{"line_number":350,"context_line":"        else:"},{"line_number":351,"context_line":"            # state \u003d\u003d \u0027absent\u0027 and not flavor:"},{"line_number":352,"context_line":"            return True"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":""},{"line_number":355,"context_line":"def main():"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f6633b6_473c6d93","line":352,"range":{"start_line":349,"start_character":0,"end_line":352,"end_character":23},"in_reply_to":"d0820ec0_9aab4dab","updated":"2022-11-09 13:50:18.000000000","message":"Yeah 🙈","commit_id":"788f97f41e431d9040d8ce53e078338238dde89e"}]}