)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11297,"name":"Nisha Agarwal","email":"agarwalnisha1980@gmail.com","username":"Nisha"},"change_message_id":"77b723938b526c2366bf2fd91642ee2f5f53148e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6fec8336_6eaf01c7","updated":"2024-03-13 12:18:21.000000000","message":"Ironic is pinning to the lower version of proliantutils because of this(pyasn1-lextudio) library not available. We do not intend to add or modify any functionality in proliantutils with respect to snmp. SNMP in proliantutils is used in a corner case to handle one scenario and the current snmp code doesn\u0027t need any change with respect to that scenario. We understand that pysnmp is not maintained, but it is available and we can rely on the functionality. \n\neven ironic stable branches are still relying on pysnmp library in driver-requirements. \nExample: https://review.opendev.org/c/openstack/ironic/+/910528\n\nif we change proliantutils to use pyasn1 instead of pysnmp, it will require lot of code changes and testing and specific hardware to test. And that is not required if the functionality can be met with pysnmp.","commit_id":"68e2d8a51cfdc325837647a6a70a9c872295857f"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"5192bb5c9e4bb939059c534c1ecbecdab9336138","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e5b73110_8771a918","updated":"2024-03-11 09:30:18.000000000","message":"what\u0027s the reason of the revert?","commit_id":"68e2d8a51cfdc325837647a6a70a9c872295857f"},{"author":{"_account_id":11297,"name":"Nisha Agarwal","email":"agarwalnisha1980@gmail.com","username":"Nisha"},"change_message_id":"a6dd9e0e163b892b984521c943bb11bf7afa16bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ba685b93_e3f7360c","in_reply_to":"36778023_c4442b7d","updated":"2024-03-14 05:14:41.000000000","message":"This patch is for just removing the non-supported library pyasn1-lextudio from proliantutils else proliantutils will not load in conductor at all and ilo drivers will not work. \n\nMoving to pyasn1 can be a seperate work item. we can work on this later but for now for 2024.1 release of ironic this quick change is must for \u0027ilo\u0027 drivers.\n\nI tried installing proliantutils 2.16.0, it fails:\n---\nERROR: Could not find a version that satisfies the requirement pyasn1-lextudio\u003e\u003d1.1.0 (from proliantutils) (from versions: 0.4.9, 1.1.2)\nERROR: No matching distribution found for pyasn1-lextudio\u003e\u003d1.1.0\n---\n\nWhile if i install proliantutils 2.16.1 (which we released today after reverting back to pysnmp) and the installation of proliantutils go through.\n\nSo i will raise the ironic patch for pointing the proliantutils release to 2.16.1 so that ilo drivers can work without any issue.","commit_id":"68e2d8a51cfdc325837647a6a70a9c872295857f"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"20fffcddb09fbd6021c14940aac87323f9fa7a5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"36778023_c4442b7d","in_reply_to":"6fec8336_6eaf01c7","updated":"2024-03-13 13:33:54.000000000","message":"thank you for your answer Nisha\nI would just like to point out that ironic uses pysnmp-lextudio since almost a year as you can see from https://opendev.org/openstack/ironic/src/branch/master/driver-requirements.txt#L8 and we don\u0027t plan to switch it back to pysnmp in the next future\non the contrary, we\u0027re going to move out from pyasn1-lextudio during the next cycle  as planned here https://review.opendev.org/c/openstack/ironic/+/910342","commit_id":"68e2d8a51cfdc325837647a6a70a9c872295857f"},{"author":{"_account_id":11297,"name":"Nisha Agarwal","email":"agarwalnisha1980@gmail.com","username":"Nisha"},"change_message_id":"f71f40c96ed621e112c642456ae1492cd688475a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f08f5fa1_fce1457b","in_reply_to":"e5b73110_8771a918","updated":"2024-03-11 10:19:10.000000000","message":"There is a bug raised by lexstudio. \n- [Bug 2052395] [NEW] pyasn1 packages from LeXtudio are no longer maintained\nAnd there is a patch in ironic for pinning proliantutils version as pyasn1-lexstudio package imports are not available. and there we can see pysnmp is still there as a requirement.","commit_id":"68e2d8a51cfdc325837647a6a70a9c872295857f"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"f3da6f53820805ca4215c157a9bfd0d433cd20d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8a6b1fba_751fb14e","in_reply_to":"f08f5fa1_fce1457b","updated":"2024-03-13 08:26:29.000000000","message":"yes but that is about pyasn1 only, not the entire pysnmp suite","commit_id":"68e2d8a51cfdc325837647a6a70a9c872295857f"}]}