)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"8dcb56211601ab669542546448f2817b524bce3d","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removing the upgrade code from Kuryr-Kubernetes repo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When moving from annotation to CRD a compatibilitz code was added"},{"line_number":10,"context_line":"to allow user to upgrade form 4.5 to 4.6 version. Now we are going"},{"line_number":11,"context_line":"to upgrade to ocp 4.8, so everyone should already use CRD and we"},{"line_number":12,"context_line":"can remove the code."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b8055609_9333bc4d","line":9,"range":{"start_line":9,"start_character":37,"end_line":9,"end_character":50},"updated":"2021-02-11 18:46:12.000000000","message":"typo","commit_id":"20f9c0b3c42b1dc3a4beea066723128927318b74"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"8dcb56211601ab669542546448f2817b524bce3d","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Removing the upgrade code from Kuryr-Kubernetes repo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When moving from annotation to CRD a compatibilitz code was added"},{"line_number":10,"context_line":"to allow user to upgrade form 4.5 to 4.6 version. Now we are going"},{"line_number":11,"context_line":"to upgrade to ocp 4.8, so everyone should already use CRD and we"},{"line_number":12,"context_line":"can remove the code."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"208d95ed_4ebaeaa2","line":10,"range":{"start_line":10,"start_character":25,"end_line":10,"end_character":29},"updated":"2021-02-11 18:46:12.000000000","message":"ditto","commit_id":"20f9c0b3c42b1dc3a4beea066723128927318b74"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"3c1ec639e6b238ff8e7b512007c6f68e6cbd999b","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2021-02-12 12:44:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removing the upgrade code from Kuryr-Kubernetes repo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When moving from annotation to CRD a compatibility code was added"},{"line_number":10,"context_line":"to allow user to upgrade from 4.5 to 4.6 version. Now we are going"},{"line_number":11,"context_line":"to upgrade to ocp 4.8, so everyone should already use CRD and we"},{"line_number":12,"context_line":"can remove the code."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2acea7f6b3e1f02edd89f2f08127065a6556d367"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"4ee3bfb2_8f4bec87","line":12,"range":{"start_line":8,"start_character":0,"end_line":12,"end_character":20},"updated":"2021-02-12 16:09:16.000000000","message":"The OCP versions are not upstream kuryr-kubernetes. This should mention OpenStack versions: ussuri-\u003evictoria upgrade and then that in wallaby (current release) we can remove it.","commit_id":"f76f0517f1c1f933ee5b7914b774e146b04614a1"}],"kuryr_kubernetes/controller/drivers/network_policy.py":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"3c1ec639e6b238ff8e7b512007c6f68e6cbd999b","unresolved":true,"context_lines":[{"line_number":159,"context_line":"            constants.K8S_API_BASE, pod_namespace))"},{"line_number":160,"context_line":"        return pods.get(\u0027items\u0027)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    def _convert_old_sg_rule(self, rule):"},{"line_number":163,"context_line":"        del rule[\u0027security_group_rule\u0027][\u0027id\u0027]"},{"line_number":164,"context_line":"        del rule[\u0027security_group_rule\u0027][\u0027security_group_id\u0027]"},{"line_number":165,"context_line":"        result \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"efc98551_90b7d107","side":"PARENT","line":162,"range":{"start_line":162,"start_character":0,"end_line":162,"end_character":41},"updated":"2021-02-12 16:09:16.000000000","message":"This can be removed as well now.","commit_id":"2611dc3b3ab034dc850222e968930e0ce0f92861"}],"kuryr_kubernetes/controller/handlers/kuryrnetworkpolicy.py":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"8dcb56211601ab669542546448f2817b524bce3d","unresolved":true,"context_lines":[{"line_number":66,"context_line":"            return"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        for netpolicy in netpolicies.get(\u0027items\u0027, []):"},{"line_number":69,"context_line":"            new_networkpolicy \u003d self._drv_policy.get_from_old_crd(netpolicy)"},{"line_number":70,"context_line":"            url \u003d (f\"{constants.K8S_API_CRD_NAMESPACES}/\""},{"line_number":71,"context_line":"                   f\"{netpolicy[\u0027metadata\u0027][\u0027namespace\u0027]}/\""},{"line_number":72,"context_line":"                   f\"kuryrnetworkpolicies\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"e2c40f72_4f31985e","side":"PARENT","line":69,"updated":"2021-02-11 18:46:12.000000000","message":"If the get_from_old_crd was only used here, shouldn\u0027t the definition of it also get removed?","commit_id":"0f2925347449a5e0c4ac7baac385e11934835b2c"}],"kuryr_kubernetes/controller/handlers/lbaas.py":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"8dcb56211601ab669542546448f2817b524bce3d","unresolved":true,"context_lines":[{"line_number":448,"context_line":""},{"line_number":449,"context_line":"            for path, name in to_remove:"},{"line_number":450,"context_line":"                try:"},{"line_number":451,"context_line":"                    k8s.remove_annotations(path, name)"},{"line_number":452,"context_line":"                except k_exc.K8sClientException:"},{"line_number":453,"context_line":"                    LOG.warning(\u0027Error removing %s annotation from %s\u0027, name,"},{"line_number":454,"context_line":"                                path)"}],"source_content_type":"text/x-python","patch_set":3,"id":"b94beda5_12e1b542","side":"PARENT","line":451,"updated":"2021-02-11 18:46:12.000000000","message":"ditto","commit_id":"0f2925347449a5e0c4ac7baac385e11934835b2c"}]}