)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fbf94335821a438508547c8f080e0c44cd6b0a3d","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-12-20 16:25:48 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix parent of ChassisAgentTypeChangeEvent"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Event ChassisAgentTypeChangeEvent was not getting"},{"line_number":10,"context_line":"logged, updating parent to ChassisAgentEvent to get"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8964fad3_d2f73250","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"updated":"2021-12-20 11:18:43.000000000","message":"Fix? It is working right now.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0db5abe90665a5f0ed64e7a7f3eb6e826ef2e3db","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-12-20 16:25:48 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix parent of ChassisAgentTypeChangeEvent"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Event ChassisAgentTypeChangeEvent was not getting"},{"line_number":10,"context_line":"logged, updating parent to ChassisAgentEvent to get"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ad0e95b5_5ab99710","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"8964fad3_d2f73250","updated":"2021-12-20 12:46:28.000000000","message":"Yes functionality wise it\u0027s working as actions are getting performed on the events as seen in functional tests, but fixing the parent also fixes the mentioned logging issue. I tried to find why ChassisAgentTypeChangeEvent parented to ChassisEvent unlike other ChassisAgent*Event so thought of better fixing it. \n\nMay be can just fix commit message to reflect it like Update parent of ChassisAgentTypeChangeEvent like other Agent Events ?","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fbf94335821a438508547c8f080e0c44cd6b0a3d","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix parent of ChassisAgentTypeChangeEvent"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Event ChassisAgentTypeChangeEvent was not getting"},{"line_number":10,"context_line":"logged, updating parent to ChassisAgentEvent to get"},{"line_number":11,"context_line":"it fixed."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Also add debug log for AgentCache reset in OVN"},{"line_number":14,"context_line":"Functional Test base class."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"f1bfbdba_04e301b4","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":9},"updated":"2021-12-20 11:18:43.000000000","message":"Sorry, I don\u0027t understand what does it means.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0db5abe90665a5f0ed64e7a7f3eb6e826ef2e3db","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix parent of ChassisAgentTypeChangeEvent"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Event ChassisAgentTypeChangeEvent was not getting"},{"line_number":10,"context_line":"logged, updating parent to ChassisAgentEvent to get"},{"line_number":11,"context_line":"it fixed."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Also add debug log for AgentCache reset in OVN"},{"line_number":14,"context_line":"Functional Test base class."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"371e1ec1_ed9137ea","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":9},"in_reply_to":"f1bfbdba_04e301b4","updated":"2021-12-20 12:46:28.000000000","message":"actually ChassisAgentWriteEvent is getting logged but not ChassisAgentTypeChangeEvent in logs, for example:- https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_621/821727/5/check/neutron-functional-with-uwsgi/6217962/controller/logs/dsvm-functional-logs/neutron.tests.functional.plugins.ml2.drivers.ovn.mech_driver.ovsdb.test_ovsdb_monitor.TestAgentMonitor.test_agent_change_controller/testrun.txt\n\nCurrent run failed as likely i missed something https://f42751d2b17b54492a51-3336c09562cf18ca80eac3b0e8bfb3f2.ssl.cf1.rackcdn.com/822307/1/check/neutron-functional-with-uwsgi/f40f313/testr_results.html","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"e9cef224d8265267e29bfce686398e74c425378a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c299712e_08481425","updated":"2021-12-20 12:14:55.000000000","message":"Hi Yatin! Thanks a lot for your patch, I\u0027ve left one comment below. \n\nIf I\u0027m wrong I will remove my -1 but I think it\u0027s important to be sure about it so as to not create a regression. :)","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py":[{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"e9cef224d8265267e29bfce686398e74c425378a","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1b95d465_8a8758ee","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"updated":"2021-12-20 12:14:55.000000000","message":"Have you made sure this will work? I think this was not being triggered if the parent was ChassisAgent because the change we wanted to monitor is in the ovn-cms-options property. IIRC tha property is only present in the chassis table. Therefore the driver won\u0027t notice the Agent type change if you monitor the agent_chassis_table and won\u0027t update the Neutron DB correctly.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"9f782475fdf1c7608efb0d49e6d2c51333db341f","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b12a352e_bee8b33e","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"0f7800bf_821158f6","updated":"2021-12-20 13:48:47.000000000","message":"I\u0027m not sure if it is correct to inherit from ChassisAgentEvent but then change the table and read from ChassisEvent... In my opinion, it is misleading if you inherit from ChassisAgentEvent but then change the table that you want to monitor to chassis. To me it sounds like inheriting from ChassisEvent is the sensible thing to do when we need to check the chassis table. \n\nMaybe you can see which part of the ChassisAgentEvent code you need in ChassisAgentTypeChangeEvent and add that to the class.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0db5abe90665a5f0ed64e7a7f3eb6e826ef2e3db","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4f11d8b4_016889c7","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"1b95d465_8a8758ee","updated":"2021-12-20 12:46:28.000000000","message":"Seems you are right, as there are failures in the patch, also thanks for sharing the background. https://f42751d2b17b54492a51-3336c09562cf18ca80eac3b0e8bfb3f2.ssl.cf1.rackcdn.com/822307/1/check/neutron-functional-with-uwsgi/f40f313/testr_results.html\n\nI thought of pinging you post CI results, but since it\u0027s failed i have to re look what\u0027s wrong.\n\nWhile investigating the other bug, i have seen after fixing parent it had logged the info, but it\u0027s not doing now. May be i did some additional changes that time, will relook.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"ec552b3add65a509a45c4b4adaae9bba0ea103cb","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"31657f64_1f6cc4ac","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"21130ad4_2816452d","updated":"2021-12-21 19:51:53.000000000","message":"I\u0027m not sure if I understood correctly. When you say \"other option is to define a match function...\" Wouldn\u0027t that make the event to be logged twice when not being in a testing environment?","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"9611573d873d18da2ad15176a22fdf57bded72a3","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"58260df8_b842ad90","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"31657f64_1f6cc4ac","updated":"2021-12-22 05:18:04.000000000","message":"No it would be logged just once whether testing or non testing enviornment as the matches function in ChassisEvent will just override the matches function from RowEvent class. Because of [3] debug log from ovsdbapp are disabled, so i can think of only this option or re enable ovsdbapp debug but that may make the subunit issue reapper(i am not sure if that issue fixed already and revert is safe)","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"3cd43c87f53a5dc3df8b8823adb3287274f9c757","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"0f7800bf_821158f6","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"4f11d8b4_016889c7","updated":"2021-12-20 12:58:47.000000000","message":"Ohkk with this patch + additionally setting table as \u0027Chassis\u0027 i can see event logs like below in local tests, i totally don\u0027t recall what i did last time but should have retested before sending the patch to avoid such changes:- \n2021-12-20 12:49:59.033 886961 DEBUG neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb.ovsdb_monitor [-] ChassisAgentTypeChangeEvent : Matched Chassis, update, None None matches /opt/stack/neutron/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py:61","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"1cec38d86f7a1d1ecb31857c03b80fb1ec1eb3df","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb43b72_8836b00c","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"58260df8_b842ad90","updated":"2021-12-22 19:48:14.000000000","message":"I think it either of them could be fine, but since (as you said) we don\u0027t know the status of that issue the first option might be safer from our perspective, right?","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"87522af3cc586cc2fd338ae930960c99375c8017","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"26f0f54f_f7c17493","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"823db58a_a9951e7c","updated":"2021-12-23 06:20:20.000000000","message":"Also rowevent event logs row like below which makes more sense as debug log then the one used by ChassisAgentEvent here, and duplicating same in ChassisEvent doesn\u0027t looks good. \n\nSo for now i will just abandon this. And will check subunit issue as if that\u0027s fixed then we are all good and would not need duplication.\n\nThanks Elvira, Rodolfo.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"bd25a4975d757f195319dd4725b86571f0c10a7d","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b2cbd0a2_bb495ae2","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"b12a352e_bee8b33e","updated":"2021-12-20 14:41:55.000000000","message":"Ok Thanks, will explore alternative options later.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"7fa2447cf4df256609fd40d66a63de195c17fc50","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"21130ad4_2816452d","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"b2cbd0a2_bb495ae2","updated":"2021-12-21 13:40:58.000000000","message":"So i looked further to found why some Events are logged and some not.\n\nEvent\u0027s with Parent class ChassisAgentEvent are logged because of [1]. But RowEvent(parent of ChassisEvent) also have debug logs set[2], still those event\u0027s are not logged in functional tests. The reason is DEBUG logs are disabled for ovsdbapp in tests due to [3]. Those were disabled due to subunit.parser error so moving back to debug may make that issue reappear. Other option is to define matches function in ChassisEvent too so event\u0027s get logged in functional tests too same as like CHassisAgentEvent. Tested both ways and events are logged fine.\n\nwhat you guys think just abandon the patch or add the matches function?\n\n[1] https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py#L61-L62\n[2] https://github.com/openstack/ovsdbapp/blob/master/ovsdbapp/backend/ovs_idl/event.py#L43-L44\n[3] https://github.com/openstack/neutron/commit/9359366d93316afb8e774f69fbdb5832934e1607","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"aa1e79cd459633762b869311afde3c588bab2a0d","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                                    clear_down\u003devent \u003d\u003d self.ROW_CREATE)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"}],"source_content_type":"text/x-python","patch_set":1,"id":"823db58a_a9951e7c","line":337,"range":{"start_line":337,"start_character":34,"end_line":337,"end_character":51},"in_reply_to":"bfb43b72_8836b00c","updated":"2021-12-23 06:01:13.000000000","message":"yes right.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fbf94335821a438508547c8f080e0c44cd6b0a3d","unresolved":true,"context_lines":[{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"},{"line_number":341,"context_line":""},{"line_number":342,"context_line":"    def match_fn(self, event, row, old\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5d6936_ff856eef","line":339,"range":{"start_line":339,"start_character":4,"end_line":339,"end_character":17},"updated":"2021-12-20 11:18:43.000000000","message":"If you change the parent to \"ChassisAgentEvent\", then this line is redundant.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0db5abe90665a5f0ed64e7a7f3eb6e826ef2e3db","unresolved":true,"context_lines":[{"line_number":336,"context_line":""},{"line_number":337,"context_line":"class ChassisAgentTypeChangeEvent(ChassisAgentEvent):"},{"line_number":338,"context_line":"    \"\"\"Chassis Agent class change event\"\"\""},{"line_number":339,"context_line":"    GLOBAL \u003d True"},{"line_number":340,"context_line":"    events \u003d (BaseEvent.ROW_UPDATE,)"},{"line_number":341,"context_line":""},{"line_number":342,"context_line":"    def match_fn(self, event, row, old\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ec848ef_de40da7a","line":339,"range":{"start_line":339,"start_character":4,"end_line":339,"end_character":17},"in_reply_to":"7a5d6936_ff856eef","updated":"2021-12-20 12:46:28.000000000","message":"ohkk will update this.","commit_id":"762c0e8b0d21ff858adc8f225bee79b71c257a93"}]}