)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f0808cceed7b3fc9597996fa336e5597ce8e7736","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"62345e83_81043f23","updated":"2023-12-13 15:30:36.000000000","message":"I think this looks good. I wouldn\u0027t mind seeing the logging improved a little bit, but this should definitely do the needful at this point. Thanks Dmitry!","commit_id":"1e771fa9f34040e1b1af2b1e1f05c56ab90a9314"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ca548855db3e63d2ab9da9b1d25529f73c167a3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"85f97700_1fed0d5c","updated":"2023-12-13 16:27:13.000000000","message":"This is worth it for the unit test changes alone. I hated those things.","commit_id":"1e771fa9f34040e1b1af2b1e1f05c56ab90a9314"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"87f1b02a91a5ce1b14c4e02b3c744bc2f4c8a8c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4814d625_ff6aca9a","updated":"2023-12-13 16:35:08.000000000","message":"Just added a release note, approving as single core.","commit_id":"2ab83646495e993940d14ec368c703854e6fb7f6"}],"ironic_python_agent/agent.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"b70a1b27ada672d8e1a29a8bc915b23a541c3544","unresolved":true,"context_lines":[{"line_number":151,"context_line":"                # Reschedule the heartbeat so that it comes much later"},{"line_number":152,"context_line":"                self.heartbeat_forced \u003d False"},{"line_number":153,"context_line":"                self.previous_heartbeat \u003d _time()"},{"line_number":154,"context_line":"            self.unavailable \u003d True"},{"line_number":155,"context_line":"        except Exception:"},{"line_number":156,"context_line":"            LOG.exception(\u0027error sending heartbeat to %s\u0027, self.agent.api_url)"},{"line_number":157,"context_line":"            self.unavailable \u003d False"}],"source_content_type":"text/x-python","patch_set":2,"id":"7bc88b38_df2fe0b2","line":154,"updated":"2023-10-09 19:19:05.000000000","message":"I guess I feel like we should be leveraging the knowledge the API returned a 500 as a stronger hint instead of just resetting heartbeat_forced and the time of the previous heartbeat, which sends us down the \"not forced, and we just did it successfully\" so we\u0027re to use an english idiom, we\u0027re sort of kicking the can down the road once. \n\nDunno, Might make it easier, since I know I\u0027ve personally kind of raised an eyebrow on the prior state in the past, to just go ahead and put inline notes to share context to reviewers/future developers down the road.","commit_id":"e74d87e3aa9b1d5dd89563a768e560689ec08e60"}]}