)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36870,"name":"Lex Li","display_name":"Lex Li","email":"support@lextm.com","username":"lextm"},"change_message_id":"78bcffbd6dd7f210f1befc03aaec4005d6372d9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8a049ebc_3834fc6f","updated":"2024-03-09 08:03:13.000000000","message":"More details can be found in https://github.com/lextudio/pysnmp/issues/49","commit_id":"1a007409fbedf35dd1fd9b45f17e870d15a67f2c"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"0960b402747bd915175fe934735cab32a2dd58ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0b1dbc40_c3350478","updated":"2024-03-11 07:44:39.000000000","message":"hi! We have already a patch open for changes to lextudio dependencies here https://review.opendev.org/c/openstack/ironic/+/910342\nThis will be changed during next cycle anyway as currently lextudio is capped on upper-constraints https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L399","commit_id":"1a007409fbedf35dd1fd9b45f17e870d15a67f2c"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"5221614cb1297be57d99de0daee9b84c6ec99823","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"98672e73_58d5e99b","updated":"2024-04-12 10:36:29.000000000","message":"recheck","commit_id":"1a007409fbedf35dd1fd9b45f17e870d15a67f2c"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"080fa599ac69df461a0fac7701122737310d6838","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9c320df5_9397150f","updated":"2024-03-28 13:55:22.000000000","message":"we can move forward with this now but we\u0027ll have to wait for upper-constraints to be updated first as it\u0027s still pinning 5.0.33 https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L399\nalso blocking merge until final caracal release\nthanks!","commit_id":"1a007409fbedf35dd1fd9b45f17e870d15a67f2c"},{"author":{"_account_id":36870,"name":"Lex Li","display_name":"Lex Li","email":"support@lextm.com","username":"lextm"},"change_message_id":"5553e17c650aee456c8e228796b1d98bbb5da93d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"35dcbf79_489539a1","in_reply_to":"0b1dbc40_c3350478","updated":"2024-03-13 22:58:49.000000000","message":"OK. Two things.\n1. The version number for pysnmp-lextudio must be bumped to 6.0. I can see that your patch will ultimately achieve the same.\n2. My changes in ironic/drivers/modules/snmp.py are required, because pysnmp-lextudio 6.0.x either uses a different function signature (so next() won\u0027t be needed) or switches to another function (nextCmd to walkCmd). Your patch does not contain this.","commit_id":"1a007409fbedf35dd1fd9b45f17e870d15a67f2c"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"86929a8703e1865865cb756526cf34dcfc9d6a49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4f43f8df_ab6718f5","in_reply_to":"35dcbf79_489539a1","updated":"2024-03-14 13:30:20.000000000","message":"thanks for the comment, my patch will include all the needed changes once we decide to move to the next version of pysnmp-lextudio during the next cycle\nwe can keep the discussion there","commit_id":"1a007409fbedf35dd1fd9b45f17e870d15a67f2c"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"56372242805e58f44af53df7a6bfbf767a57c759","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"202405fe_c1056a73","updated":"2024-05-28 08:21:39.000000000","message":"apologies for the delay with this, we\u0027re now ready to merge this as soon as https://review.opendev.org/c/openstack/requirements/+/915830 merges","commit_id":"70af7b954823df7453f042b70f3fa416bb107af0"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"b1d71b6d4d7203543769a0dcc4a2077c2b8daaed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7df067e1_e4d82c01","updated":"2024-06-17 08:37:33.000000000","message":"recheck uc patch merged","commit_id":"70af7b954823df7453f042b70f3fa416bb107af0"},{"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":"0f741601be8ad9fbae6e4dba0929dde81db841b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b65d1498_a59b3d65","updated":"2024-06-17 21:46:46.000000000","message":"We install virtualpdu and Ironic in the same venv; so they have to use versions compatible with each other. We\u0027ll need to land something making snmp job non-voting to land this (and we should)","commit_id":"9adf9ff77610dfc538f6c6d43df2dc5cd59c6711"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"dac77798d7a5d48366f355f82361bcf177bfeefe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e0f57e58_39af3d17","updated":"2024-06-19 09:21:17.000000000","message":"recheck metal3 job should work now","commit_id":"7e463f21dcc1b09e18f186c45f1978dcd1599c5c"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"d00aedd0dfd2d8a9f74cd7ed0a86098756570066","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4668e85f_51ceda70","updated":"2025-11-22 19:16:34.000000000","message":"what are we doing with pysnmp?","commit_id":"7e463f21dcc1b09e18f186c45f1978dcd1599c5c"}]}