)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"7d75be6ed33b7959286a0683987c7d1790b395cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2b6ec479_e829e1f0","updated":"2023-06-21 12:57:28.000000000","message":"recheck all cover tests passed both on the check pipeline and on my local machine","commit_id":"3fbe97c638f929c97e739900ec6da10f8eb86a5c"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"ae3c3d0f8d58423e252149b4946dd0bedf193ae2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7f8ee410_37e85898","updated":"2023-06-21 08:15:03.000000000","message":"recheck looks like check pipelines are working now","commit_id":"3fbe97c638f929c97e739900ec6da10f8eb86a5c"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"63e92034ee0842287fe1a0211cbde984ce43cfb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ac38db24_4366c60c","updated":"2023-07-24 12:13:50.000000000","message":"I just ran tox -e cover on my machine and it passed. Not sure why on u/s there were 8 that failed. Anyway, I would like to see if rerunning make them fail again.","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"14d5cfff6cca1f7b45b1897c5ca5cd96b9e3fe01","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"50544d60_728d2324","updated":"2023-08-22 11:41:58.000000000","message":"Thanks Greg, I was got some \"inspiration\" from the existing pool children quota tests and I covered those missing tests","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"a58e6e799c17aa71b922aa201a1f7830172053a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"640e1b68_d654d923","updated":"2023-08-09 14:05:39.000000000","message":"recheck tox -e cover passes on my laptop","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"925fa4d212f7cbf79c64fcd177707b90c4370e1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c7f4c876_e7c4e979","updated":"2023-08-22 15:14:25.000000000","message":"WIP","commit_id":"5859a21b22f755ec362124a9bda19fbcb90f5b1f"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"c6628232c93b33848c5b818f7fefa7624689528e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8b4fc432_5e5911b2","updated":"2023-10-27 15:50:00.000000000","message":"recheck to get recent CI results and logs","commit_id":"f0d66b561b0c84a4953721fba1e5a3b1b2444332"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"dfcb21c5a5525c2c2357ab0c32e8f07724388634","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"fad7e985_3d5e64a3","updated":"2023-12-19 12:58:07.000000000","message":"recheck some upstream jobs were fixed","commit_id":"2d2c144fde9d5ff1821719dd1fad1e7b85f95bc9"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"989b24daf6596f777738c97a2115ce16cebfd0c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"27691131_a95c62bb","updated":"2023-12-21 20:32:02.000000000","message":"recheck just in case","commit_id":"0fdd3fa185cc3310d9d814a932117d8502dd64de"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"6227367a03e86b6c24dc8c8de8a1bb4beb84b15f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"89663a9d_61e554cf","updated":"2023-12-21 16:47:41.000000000","message":"recheck looks like a regular timeout","commit_id":"0fdd3fa185cc3310d9d814a932117d8502dd64de"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"fbf632515e579baf0cec212de9446e138d8d3100","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"644888f8_46ba5a00","updated":"2024-10-14 09:15:37.000000000","message":"https://bugs.launchpad.net/octavia/+bug/2015050","commit_id":"54c110e78ec868fb2621ffea1a8683d8de829208"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"bd84e530ed0c306cac1fb81f85769d8e2de38a68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"9f62f20e_769768b7","updated":"2024-10-14 01:37:21.000000000","message":"is there a bug for this? I am seeing similar issues in health_monitor too - deleting an LB with --cascade will decrement LB and pool, but not health_monitor.","commit_id":"54c110e78ec868fb2621ffea1a8683d8de829208"}],"octavia/controller/worker/v2/tasks/database_tasks.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"35365d2a861573bdb683be101f2da9d2e86ffd71","unresolved":true,"context_lines":[{"line_number":2560,"context_line":""},{"line_number":2561,"context_line":"            # Listeners cascade delete l7policies and l7rules"},{"line_number":2562,"context_line":"            # update the quota for those items as well."},{"line_number":2563,"context_line":"            if listener_child_count[constants.L7POLICY] \u003e 0:"},{"line_number":2564,"context_line":"                self.repos.decrement_quota("},{"line_number":2565,"context_line":"                    lock_session,"},{"line_number":2566,"context_line":"                    data_models.L7Policy,"},{"line_number":2567,"context_line":"                    project_id,"},{"line_number":2568,"context_line":"                    quantity\u003dlistener_child_count[constants.L7POLICY])"},{"line_number":2569,"context_line":"            if listener_child_count[constants.L7RULE] \u003e 0:"},{"line_number":2570,"context_line":"                self.repos.decrement_quota("},{"line_number":2571,"context_line":"                    lock_session,"},{"line_number":2572,"context_line":"                    data_models.L7Rule,"},{"line_number":2573,"context_line":"                    project_id,"},{"line_number":2574,"context_line":"                    quantity\u003dlistener_child_count[constants.L7RULE])"},{"line_number":2575,"context_line":""},{"line_number":2576,"context_line":"            lock_session.commit()"},{"line_number":2577,"context_line":"        except Exception:"},{"line_number":2578,"context_line":"            with excutils.save_and_reraise_exception():"}],"source_content_type":"text/x-python","patch_set":5,"id":"19394dd5_c468bb0d","line":2575,"range":{"start_line":2563,"start_character":0,"end_line":2575,"end_character":0},"updated":"2023-08-22 06:28:53.000000000","message":"not covered:\nhttps://943d39391aa3bf76e709-f73785700cacef583f29e4dcac8fae98.ssl.cf5.rackcdn.com/886111/5/check/openstack-tox-cover/db36813/cover/d_b37783f7bc1df236_database_tasks_py.html#t2566","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"14d5cfff6cca1f7b45b1897c5ca5cd96b9e3fe01","unresolved":false,"context_lines":[{"line_number":2560,"context_line":""},{"line_number":2561,"context_line":"            # Listeners cascade delete l7policies and l7rules"},{"line_number":2562,"context_line":"            # update the quota for those items as well."},{"line_number":2563,"context_line":"            if listener_child_count[constants.L7POLICY] \u003e 0:"},{"line_number":2564,"context_line":"                self.repos.decrement_quota("},{"line_number":2565,"context_line":"                    lock_session,"},{"line_number":2566,"context_line":"                    data_models.L7Policy,"},{"line_number":2567,"context_line":"                    project_id,"},{"line_number":2568,"context_line":"                    quantity\u003dlistener_child_count[constants.L7POLICY])"},{"line_number":2569,"context_line":"            if listener_child_count[constants.L7RULE] \u003e 0:"},{"line_number":2570,"context_line":"                self.repos.decrement_quota("},{"line_number":2571,"context_line":"                    lock_session,"},{"line_number":2572,"context_line":"                    data_models.L7Rule,"},{"line_number":2573,"context_line":"                    project_id,"},{"line_number":2574,"context_line":"                    quantity\u003dlistener_child_count[constants.L7RULE])"},{"line_number":2575,"context_line":""},{"line_number":2576,"context_line":"            lock_session.commit()"},{"line_number":2577,"context_line":"        except Exception:"},{"line_number":2578,"context_line":"            with excutils.save_and_reraise_exception():"}],"source_content_type":"text/x-python","patch_set":5,"id":"b5287764_d4d8e405","line":2575,"range":{"start_line":2563,"start_character":0,"end_line":2575,"end_character":0},"in_reply_to":"19394dd5_c468bb0d","updated":"2023-08-22 11:41:58.000000000","message":"Done","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"35365d2a861573bdb683be101f2da9d2e86ffd71","unresolved":true,"context_lines":[{"line_number":2610,"context_line":"                except Exception:"},{"line_number":2611,"context_line":"                    lock_session.rollback()"},{"line_number":2612,"context_line":""},{"line_number":2613,"context_line":"                # Attempt to increment back the l7policy quota"},{"line_number":2614,"context_line":"                # This is separate calls to maximize the correction"},{"line_number":2615,"context_line":"                # should other factors have increased the in use quota"},{"line_number":2616,"context_line":"                # before this point in the revert flow"},{"line_number":2617,"context_line":"                for i in range(listener_child_count[constants.L7POLICY]):"},{"line_number":2618,"context_line":"                    lock_session \u003d db_apis.get_session(autocommit\u003dFalse)"},{"line_number":2619,"context_line":"                    try:"},{"line_number":2620,"context_line":"                        self.repos.check_quota_met(session,"},{"line_number":2621,"context_line":"                                                   lock_session,"},{"line_number":2622,"context_line":"                                                   data_models.L7Policy,"},{"line_number":2623,"context_line":"                                                   project_id)"},{"line_number":2624,"context_line":"                        lock_session.commit()"},{"line_number":2625,"context_line":"                    except Exception:"},{"line_number":2626,"context_line":"                        lock_session.rollback()"},{"line_number":2627,"context_line":""},{"line_number":2628,"context_line":"                # Attempt to increment back the l7rule quota"},{"line_number":2629,"context_line":"                # This is separate calls to maximize the correction"},{"line_number":2630,"context_line":"                # should other factors have increased the in use quota"},{"line_number":2631,"context_line":"                # before this point in the revert flow"},{"line_number":2632,"context_line":"                for i in range(listener_child_count[constants.L7RULE]):"},{"line_number":2633,"context_line":"                    lock_session \u003d db_apis.get_session(autocommit\u003dFalse)"},{"line_number":2634,"context_line":"                    try:"},{"line_number":2635,"context_line":"                        self.repos.check_quota_met(session,"},{"line_number":2636,"context_line":"                                                   lock_session,"},{"line_number":2637,"context_line":"                                                   data_models.L7Rule,"},{"line_number":2638,"context_line":"                                                   project_id)"},{"line_number":2639,"context_line":"                        lock_session.commit()"},{"line_number":2640,"context_line":"                    except Exception:"},{"line_number":2641,"context_line":"                        lock_session.rollback()"},{"line_number":2642,"context_line":"            except Exception:"},{"line_number":2643,"context_line":"                # Don\u0027t fail the revert flow"},{"line_number":2644,"context_line":"                pass"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a495c39_f73a71a1","line":2641,"range":{"start_line":2613,"start_character":0,"end_line":2641,"end_character":47},"updated":"2023-08-22 06:28:53.000000000","message":"not covered\nhttps://943d39391aa3bf76e709-f73785700cacef583f29e4dcac8fae98.ssl.cf5.rackcdn.com/886111/5/check/openstack-tox-cover/db36813/cover/d_b37783f7bc1df236_database_tasks_py.html#t2620","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"14d5cfff6cca1f7b45b1897c5ca5cd96b9e3fe01","unresolved":false,"context_lines":[{"line_number":2610,"context_line":"                except Exception:"},{"line_number":2611,"context_line":"                    lock_session.rollback()"},{"line_number":2612,"context_line":""},{"line_number":2613,"context_line":"                # Attempt to increment back the l7policy quota"},{"line_number":2614,"context_line":"                # This is separate calls to maximize the correction"},{"line_number":2615,"context_line":"                # should other factors have increased the in use quota"},{"line_number":2616,"context_line":"                # before this point in the revert flow"},{"line_number":2617,"context_line":"                for i in range(listener_child_count[constants.L7POLICY]):"},{"line_number":2618,"context_line":"                    lock_session \u003d db_apis.get_session(autocommit\u003dFalse)"},{"line_number":2619,"context_line":"                    try:"},{"line_number":2620,"context_line":"                        self.repos.check_quota_met(session,"},{"line_number":2621,"context_line":"                                                   lock_session,"},{"line_number":2622,"context_line":"                                                   data_models.L7Policy,"},{"line_number":2623,"context_line":"                                                   project_id)"},{"line_number":2624,"context_line":"                        lock_session.commit()"},{"line_number":2625,"context_line":"                    except Exception:"},{"line_number":2626,"context_line":"                        lock_session.rollback()"},{"line_number":2627,"context_line":""},{"line_number":2628,"context_line":"                # Attempt to increment back the l7rule quota"},{"line_number":2629,"context_line":"                # This is separate calls to maximize the correction"},{"line_number":2630,"context_line":"                # should other factors have increased the in use quota"},{"line_number":2631,"context_line":"                # before this point in the revert flow"},{"line_number":2632,"context_line":"                for i in range(listener_child_count[constants.L7RULE]):"},{"line_number":2633,"context_line":"                    lock_session \u003d db_apis.get_session(autocommit\u003dFalse)"},{"line_number":2634,"context_line":"                    try:"},{"line_number":2635,"context_line":"                        self.repos.check_quota_met(session,"},{"line_number":2636,"context_line":"                                                   lock_session,"},{"line_number":2637,"context_line":"                                                   data_models.L7Rule,"},{"line_number":2638,"context_line":"                                                   project_id)"},{"line_number":2639,"context_line":"                        lock_session.commit()"},{"line_number":2640,"context_line":"                    except Exception:"},{"line_number":2641,"context_line":"                        lock_session.rollback()"},{"line_number":2642,"context_line":"            except Exception:"},{"line_number":2643,"context_line":"                # Don\u0027t fail the revert flow"},{"line_number":2644,"context_line":"                pass"}],"source_content_type":"text/x-python","patch_set":5,"id":"6f4b8c8e_d6ff9c77","line":2641,"range":{"start_line":2613,"start_character":0,"end_line":2641,"end_character":47},"in_reply_to":"1a495c39_f73a71a1","updated":"2023-08-22 11:41:58.000000000","message":"Done","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"35365d2a861573bdb683be101f2da9d2e86ffd71","unresolved":true,"context_lines":[{"line_number":2659,"context_line":"        :param listener: listener object to count children on"},{"line_number":2660,"context_line":"        :returns: None"},{"line_number":2661,"context_line":"        \"\"\""},{"line_number":2662,"context_line":"        session \u003d db_apis.get_session()"},{"line_number":2663,"context_line":""},{"line_number":2664,"context_line":"        l7policy_count, l7rule_count \u003d ("},{"line_number":2665,"context_line":"            self.listener_repo.get_children_count(session\u003dsession,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9b315f12_30169b64","line":2662,"updated":"2023-08-22 06:28:53.000000000","message":"not covered\nhttps://943d39391aa3bf76e709-f73785700cacef583f29e4dcac8fae98.ssl.cf5.rackcdn.com/886111/5/check/openstack-tox-cover/db36813/cover/d_b37783f7bc1df236_database_tasks_py.html#t2665","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"14d5cfff6cca1f7b45b1897c5ca5cd96b9e3fe01","unresolved":false,"context_lines":[{"line_number":2659,"context_line":"        :param listener: listener object to count children on"},{"line_number":2660,"context_line":"        :returns: None"},{"line_number":2661,"context_line":"        \"\"\""},{"line_number":2662,"context_line":"        session \u003d db_apis.get_session()"},{"line_number":2663,"context_line":""},{"line_number":2664,"context_line":"        l7policy_count, l7rule_count \u003d ("},{"line_number":2665,"context_line":"            self.listener_repo.get_children_count(session\u003dsession,"}],"source_content_type":"text/x-python","patch_set":5,"id":"6237774f_18ed033b","line":2662,"in_reply_to":"9b315f12_30169b64","updated":"2023-08-22 11:41:58.000000000","message":"Done","commit_id":"71430598fb3dfea97cd3976b8b30019c3efdf020"}]}
