)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"32f10d4db485bbbcd2bc823184e0458bf9427c29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8e1081c3_d382e248","updated":"2026-03-09 14:57:22.000000000","message":"HPE 3PAR Storage FC CI and HPE 3PAR Storage iSCSI CI have passed","commit_id":"a1cbdfb9f3bef0c248dae86bc575af9904314c6f"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5aa18a711bc42f8ad0453af63ad008282b3afcc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c44c028e_559ad83d","updated":"2026-03-18 14:53:52.000000000","message":"Code and tests LGTM.","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"e4eb9308fcce9f548cffa68e4af9f39c1e5fdc78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2b84fa9b_7f4089c7","updated":"2026-03-18 17:23:51.000000000","message":"Small request for clarification, code otherwise looks good.","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"ba90166e48ce9fe50f8a52aff7070fc971bbdc8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1bcb20d7_6c5e0245","updated":"2026-03-18 15:02:05.000000000","message":"Thank you Brian","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"8c92e37264d8d2b311a8590283bfd76c32a28887","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"af5c3d92_2343965f","updated":"2026-03-10 16:27:36.000000000","message":"The HPE 3PAR Storage CI (iSCSI \u0026 FC) have passed","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9d014075872d92e3bb8571314c17c5f052e6effb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"bbaa978d_305a0320","updated":"2026-03-10 16:07:10.000000000","message":"recheck cinder-tempest-plugin-protection-functional failed","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":38274,"name":"Jyotsna Lothe","display_name":"Jyotsna Lothe","email":"jyotsna.lothe@hpe.com","username":"jlothe"},"change_message_id":"bec44ae0852799412db5a4c2184771f5f289fcf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"aa9709d7_50c63044","updated":"2026-03-20 04:44:55.000000000","message":"@jobernar@redhat.com as per your feedback we updated both the commit message as well as release notes","commit_id":"aa0d7266a8926d6da86f7866cb2e51849c2dc0a9"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"454cfe0b203acb7fcbb0c8ba20386a5289e8afe8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"5a54d4d6_306761a7","updated":"2026-03-20 05:04:20.000000000","message":"Can you consistently use QoS. I am seeing QOS and QoS is used in different places.","commit_id":"aa0d7266a8926d6da86f7866cb2e51849c2dc0a9"},{"author":{"_account_id":38274,"name":"Jyotsna Lothe","display_name":"Jyotsna Lothe","email":"jyotsna.lothe@hpe.com","username":"jlothe"},"change_message_id":"9d0358a7c08c3b75c4bc678dccbdc3cdd6777a74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a0331170_0d1b883c","in_reply_to":"5a54d4d6_306761a7","updated":"2026-03-20 09:13:49.000000000","message":"@jayaanand.borra@netapp.com, corrected it and made it uniform by using \"QOS\" all places","commit_id":"aa0d7266a8926d6da86f7866cb2e51849c2dc0a9"},{"author":{"_account_id":38274,"name":"Jyotsna Lothe","display_name":"Jyotsna Lothe","email":"jyotsna.lothe@hpe.com","username":"jlothe"},"change_message_id":"9d0358a7c08c3b75c4bc678dccbdc3cdd6777a74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"2d216d09_60b66825","updated":"2026-03-20 09:13:49.000000000","message":"@jayaanand.borra@netapp.com as per the feedback, addressed all the comments.","commit_id":"e174cf70fac057a4f430494d7972a78badb5856d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"65172cf82f01751841e3876e2e46e4ae6a9452f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"bae3b1e2_6a2d9d14","updated":"2026-03-25 15:56:49.000000000","message":"Revisions LGTM.  Good suggestion from jayaanand to add the invalid input test.","commit_id":"e174cf70fac057a4f430494d7972a78badb5856d"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"5052b678302e7d2e03d4e1a82f204d937616ad68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ef5672c1_bc7b35cf","updated":"2026-03-25 16:48:55.000000000","message":"Thank you Brian","commit_id":"e174cf70fac057a4f430494d7972a78badb5856d"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"1761a6fbc95b68f11c79a7152a015709f0eb04c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"08cb8c83_c0c34f18","updated":"2026-03-26 06:20:45.000000000","message":"Thank you Jon, Brian \u0026 Jayaanand. The patch has merged","commit_id":"e174cf70fac057a4f430494d7972a78badb5856d"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"2aed88067d6c611044c0b3cbf1098da3c4f8c8b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c1d84db2_434e3ab3","updated":"2026-03-21 12:38:22.000000000","message":"recheck openstack-tox-py310, openstack-tox-py313 \u0026 devstack-plugin-nfs-tempest-full failed","commit_id":"e174cf70fac057a4f430494d7972a78badb5856d"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"5436b9ee2a77ea961a3956b73c677d2d3398013b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e6222afa_1f9cdbac","updated":"2026-03-20 14:12:38.000000000","message":"recheck openstack-tox-py313 and tempest-integrated-storage failed","commit_id":"e174cf70fac057a4f430494d7972a78badb5856d"}],"cinder/tests/unit/volume/drivers/hpe/test_hpe3par.py":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"e4eb9308fcce9f548cffa68e4af9f39c1e5fdc78","unresolved":true,"context_lines":[{"line_number":1246,"context_line":""},{"line_number":1247,"context_line":"        When the backend API version indicates Alletra MP, maxBWS should be"},{"line_number":1248,"context_line":"        converted using a 1000-based factor into bwMaxLimitKB, and only the"},{"line_number":1249,"context_line":"        max limits should be sent to createQoSRules."},{"line_number":1250,"context_line":"        \"\"\""},{"line_number":1251,"context_line":""},{"line_number":1252,"context_line":"        # Create a common driver instance with Alletra MP API_VERSION."}],"source_content_type":"text/x-python","patch_set":6,"id":"edc1c65e_1321c08f","line":1249,"updated":"2026-03-18 17:23:51.000000000","message":"Ahh, I think it would be good to mention this in the commit message and/or the release note, as this patch not only adds support for AlletraMP but also adjusts the QoS values as well - if I\u0027m understanding correctly.","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":38274,"name":"Jyotsna Lothe","display_name":"Jyotsna Lothe","email":"jyotsna.lothe@hpe.com","username":"jlothe"},"change_message_id":"bec44ae0852799412db5a4c2184771f5f289fcf3","unresolved":false,"context_lines":[{"line_number":1246,"context_line":""},{"line_number":1247,"context_line":"        When the backend API version indicates Alletra MP, maxBWS should be"},{"line_number":1248,"context_line":"        converted using a 1000-based factor into bwMaxLimitKB, and only the"},{"line_number":1249,"context_line":"        max limits should be sent to createQoSRules."},{"line_number":1250,"context_line":"        \"\"\""},{"line_number":1251,"context_line":""},{"line_number":1252,"context_line":"        # Create a common driver instance with Alletra MP API_VERSION."}],"source_content_type":"text/x-python","patch_set":6,"id":"7e630e2d_ba1899a9","line":1249,"in_reply_to":"edc1c65e_1321c08f","updated":"2026-03-20 04:44:55.000000000","message":"Done","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"}],"cinder/volume/drivers/hpe/hpe_3par_common.py":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"e4eb9308fcce9f548cffa68e4af9f39c1e5fdc78","unresolved":true,"context_lines":[{"line_number":2120,"context_line":"                    qosRule[\u0027ioMinGoal\u0027] \u003d int(max_io)"},{"line_number":2121,"context_line":"            if min_bw:"},{"line_number":2122,"context_line":"                # 3PAR/Primera/Alletra 9k expect bandwidth in kB/s"},{"line_number":2123,"context_line":"                qosRule[\u0027bwMinGoalKB\u0027] \u003d int(min_bw) * units.k"},{"line_number":2124,"context_line":"                if max_bw is None:"},{"line_number":2125,"context_line":"                    qosRule[\u0027bwMaxLimitKB\u0027] \u003d int(min_bw) * units.k"},{"line_number":2126,"context_line":"            if max_bw:"}],"source_content_type":"text/x-python","patch_set":6,"id":"3872f043_81c94479","line":2123,"updated":"2026-03-18 17:23:51.000000000","message":"The removed code multiples by units.Ki (1024), but this is 1000 - is that intended?","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":38274,"name":"Jyotsna Lothe","display_name":"Jyotsna Lothe","email":"jyotsna.lothe@hpe.com","username":"jlothe"},"change_message_id":"bec44ae0852799412db5a4c2184771f5f289fcf3","unresolved":false,"context_lines":[{"line_number":2120,"context_line":"                    qosRule[\u0027ioMinGoal\u0027] \u003d int(max_io)"},{"line_number":2121,"context_line":"            if min_bw:"},{"line_number":2122,"context_line":"                # 3PAR/Primera/Alletra 9k expect bandwidth in kB/s"},{"line_number":2123,"context_line":"                qosRule[\u0027bwMinGoalKB\u0027] \u003d int(min_bw) * units.k"},{"line_number":2124,"context_line":"                if max_bw is None:"},{"line_number":2125,"context_line":"                    qosRule[\u0027bwMaxLimitKB\u0027] \u003d int(min_bw) * units.k"},{"line_number":2126,"context_line":"            if max_bw:"}],"source_content_type":"text/x-python","patch_set":6,"id":"743c4017_45e0d2b9","line":2123,"in_reply_to":"3872f043_81c94479","updated":"2026-03-20 04:44:55.000000000","message":"@jobernar@redhat.com It is intended as backend expected the QOS params in KB.","commit_id":"87c809edbaa4b99bb5d362df2715b24a232309e3"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"454cfe0b203acb7fcbb0c8ba20386a5289e8afe8","unresolved":true,"context_lines":[{"line_number":2087,"context_line":"                    \"For Alletra MP, at least one of maxIOPS or maxBWS \""},{"line_number":2088,"context_line":"                    \"QoS parameters must be provided.\")"},{"line_number":2089,"context_line":"                LOG.error(err)"},{"line_number":2090,"context_line":"                raise exception.InvalidInput(reason\u003derr)"},{"line_number":2091,"context_line":"            # Alletra MP: Only set max limits, min goals are deprecated"},{"line_number":2092,"context_line":"            if max_io:"},{"line_number":2093,"context_line":"                qosRule[\u0027ioMaxLimit\u0027] \u003d int(max_io)"}],"source_content_type":"text/x-python","patch_set":7,"id":"8365aaf4_aafa40ad","line":2090,"updated":"2026-03-20 05:04:20.000000000","message":"i didn\u0027t find any tests for InvalidInput path. Good to have test. Test are covering happy path.","commit_id":"aa0d7266a8926d6da86f7866cb2e51849c2dc0a9"},{"author":{"_account_id":38274,"name":"Jyotsna Lothe","display_name":"Jyotsna Lothe","email":"jyotsna.lothe@hpe.com","username":"jlothe"},"change_message_id":"9d0358a7c08c3b75c4bc678dccbdc3cdd6777a74","unresolved":false,"context_lines":[{"line_number":2087,"context_line":"                    \"For Alletra MP, at least one of maxIOPS or maxBWS \""},{"line_number":2088,"context_line":"                    \"QoS parameters must be provided.\")"},{"line_number":2089,"context_line":"                LOG.error(err)"},{"line_number":2090,"context_line":"                raise exception.InvalidInput(reason\u003derr)"},{"line_number":2091,"context_line":"            # Alletra MP: Only set max limits, min goals are deprecated"},{"line_number":2092,"context_line":"            if max_io:"},{"line_number":2093,"context_line":"                qosRule[\u0027ioMaxLimit\u0027] \u003d int(max_io)"}],"source_content_type":"text/x-python","patch_set":7,"id":"de394ecc_b9d2d486","line":2090,"in_reply_to":"8365aaf4_aafa40ad","updated":"2026-03-20 09:13:49.000000000","message":"Done, added invalid input test case","commit_id":"aa0d7266a8926d6da86f7866cb2e51849c2dc0a9"}]}