)]}'
{"/PATCHSET_LEVEL":[{"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":"003ebcf1359fcb9d9c32bbbd0d77d30887dd48d6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5130a8d2_3c9d6e74","updated":"2024-11-05 21:51:05.000000000","message":"Itamar and I are working together to fix the SNMP driver too, so we can actually end-to-end test all this before it lands.","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"},{"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":"12bd34ef3caac59febeee94fdce69f3036b07345","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bb2d058a_623ba332","updated":"2024-10-31 18:49:49.000000000","message":"So this is seemingly working, but the Ironic SNMP driver broke in the interim when pysnmp was bumped to 7.0.1 and CI was down.\n\nWe\u0027ll be looking to fix that too, then I\u0027ll do the proper job disable/re-enable to get things hooked back up for SNMP.","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"91831638a2a0a91e03935fcd191eec8f284870bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"89ae3ff2_f6a81ebc","updated":"2024-11-05 09:14:58.000000000","message":"also let\u0027s link this to https://bugs.launchpad.net/virtualpdu/+bug/2069700","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"},{"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":"f3384711db13b5df8e77702370311051e72f1b95","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dd3f66c3_064be14d","updated":"2024-10-31 18:48:37.000000000","message":"requirements.txt issue has to be fixed for CI requirements-check job to be happy; also some pep8 violations here -- `tox -epep8` should help with those. \n\nAs I said in DM though, if we can get a passing integration job between this and Ironic I\u0027m happy to do any cleanups.","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"e2707343154d4451eaf5eb2df60dc9b5b9ded43b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7d6e7022_3fe6bc4d","updated":"2024-11-04 08:26:26.000000000","message":"this looks good, please add a release note, thanks!:)","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"}],"requirements.txt":[{"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":"f3384711db13b5df8e77702370311051e72f1b95","unresolved":true,"context_lines":[{"line_number":1,"context_line":"libvirt-python\u003e\u003d6.0.0 # LGPLv2+"},{"line_number":2,"context_line":"pysnmp\u003d\u003d7.0.1 # BSD"},{"line_number":3,"context_line":"pyasn1\u003e\u003d0.5.1 # BSD"},{"line_number":4,"context_line":"pyasn1-modules\u003e\u003d0.3.0 # BSD"}],"source_content_type":"text/plain","patch_set":1,"id":"34301386_c919c854","line":2,"updated":"2024-10-31 18:48:37.000000000","message":"this needs to be \u003e\u003d in order to pass requirements.txt (we use upper-constraints.txt to lock versions to an upper bound)","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"},{"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":"0416d8083c35c9416d991948dd1b2edb2c931f8f","unresolved":true,"context_lines":[{"line_number":1,"context_line":"libvirt-python\u003e\u003d6.0.0 # LGPLv2+"},{"line_number":2,"context_line":"pysnmp\u003d\u003d7.0.1 # BSD"},{"line_number":3,"context_line":"pyasn1\u003e\u003d0.5.1 # BSD"},{"line_number":4,"context_line":"pyasn1-modules\u003e\u003d0.3.0 # BSD"}],"source_content_type":"text/plain","patch_set":1,"id":"25d5250f_5f0582b6","line":2,"in_reply_to":"34301386_c919c854","updated":"2024-10-31 18:50:25.000000000","message":"**in order to pass requirements-check CI job","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"}],"virtualpdu/pdu/pysnmp_handler.py":[{"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":"f3384711db13b5df8e77702370311051e72f1b95","unresolved":true,"context_lines":[{"line_number":273,"context_line":"                              .format(self.listen_address, self.listen_port,"},{"line_number":274,"context_line":"                                      self.power_unit.name))"},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"            # It\u0027s important to create this here, so the loop and registered"},{"line_number":277,"context_line":"            # sockets created in create_snmp_engine() are created in the"},{"line_number":278,"context_line":"            # correct thread."},{"line_number":279,"context_line":"            self.loop \u003d new_event_loop()"}],"source_content_type":"text/x-python","patch_set":1,"id":"cbacb678_a41b8be2","line":276,"updated":"2024-10-31 18:48:37.000000000","message":"nit: it\u0027s customary in OpenStack to label this with your handle, e.g.\n\n\\# NOTE(itamarst): It\u0027s important to...","commit_id":"778d7314745774ca0998495a5b81705c6bc7df05"}]}
