)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"ef1c57b83a55855af58644f2ffe90e8f999a9d93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2152e6a7_1bd3a922","updated":"2024-10-02 19:06:16.000000000","message":"Good catch","commit_id":"0fa2aac44b3c98486c8db48df450fad07e123df3"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"ec41948b9becb82bb6a5d1ad0eb1fcd6c6984f75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4fde3050_46fc1803","updated":"2024-10-08 06:47:24.000000000","message":"Hi Pablo,\n\ni am replying to your query about release note.\nIn commit message, \"Closes-Bug\" can be specified.\nFor release note, please refer below link [1].\nIf it helps, you can check my patch [2] for commit message and release note yaml file.\n\n[1] https://docs.openstack.org/cinder/latest/contributor/releasenotes.html#drivers\n[2] https://review.opendev.org/c/openstack/cinder/+/928599","commit_id":"0fa2aac44b3c98486c8db48df450fad07e123df3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b04dbe6902de58d886b848480a34ac3a77aa3550","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"33c3c02e_92710086","updated":"2024-10-02 21:15:04.000000000","message":"That\u0027s a bad typo! It was introduced by change I3deae1cc6bbe5ef5820224 during Dalmatian development, so it will need to be backported to stable/2024.2\n\nThis is going to sound stupid, but I think it needs a bug and a release note, so operators can know if they have a fixed version of the driver.","commit_id":"0fa2aac44b3c98486c8db48df450fad07e123df3"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"910553416ba0db4e753f8a36aa47647e21c2a2c7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e2b02368_482f1c3a","in_reply_to":"33c3c02e_92710086","updated":"2024-10-03 09:14:20.000000000","message":"Here is the bug report https://bugs.launchpad.net/cinder/+bug/2083587\n\nHow to you do realease note and backport ? (sorry first merge request on any openstack project)","commit_id":"0fa2aac44b3c98486c8db48df450fad07e123df3"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"202a142295b6518d2a3dac497a3539087a9908b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d88015d8_e190c8f2","in_reply_to":"4fde3050_46fc1803","updated":"2024-10-08 12:33:23.000000000","message":"I think I\u0027m all good now","commit_id":"0fa2aac44b3c98486c8db48df450fad07e123df3"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"00afefcf45b5557133a31aed5f2d5f62ddfd9f21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2e4110b2_4fd1aa82","in_reply_to":"e2b02368_482f1c3a","updated":"2024-10-09 08:10:04.000000000","message":"Done","commit_id":"0fa2aac44b3c98486c8db48df450fad07e123df3"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f64c050a64c0e4b549481beebdb1c7ceef1d5d18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"21d38e98_8de35336","updated":"2024-10-12 14:07:39.000000000","message":"Also, I\u0027d be nice if you can add a few more unit test cases to verify volume creation with different QoS patterns (the only existing test case test_create_volume_with_qos assumes both iops and bws is set), to avoid similar regressions in the future.","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"d448c1e3261eb566a947df1f61eef867e7da017c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b48a9607_452c6831","updated":"2024-10-08 13:31:13.000000000","message":"LGTM","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"ea8e02df8d3d4108e1598d58d7056548854e69cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f2b84606_07607b31","updated":"2024-10-09 03:40:13.000000000","message":"Looks good now.","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"e3297ebd764e80c549fd89688cb848d948c42eab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0c8dd445_7a1a1ffd","updated":"2024-10-08 16:03:52.000000000","message":"recheck","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"cf478b298fa016d0df248325ecca62e301a041a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b616bc13_1eafa0f3","updated":"2024-10-08 16:03:46.000000000","message":"run Pure Storage CI","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"c52ea50d68819b8f61adf1891291bf49b9865fae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0c4c295c_cea8a25a","updated":"2024-10-23 12:13:08.000000000","message":"Pure will supply approriate unit tests as requested.","commit_id":"9f6d445647ea1aaa299f70e72b3ca7080138753f"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"c78786792d6c65a95646c18959e66db353def6c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c9e3cb73_1c7c2f19","updated":"2024-11-19 17:12:59.000000000","message":"Any news on this one ? all good for everyone ?","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"11413d2330e160d0e4c5f363443396b25cdad158","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"31515449_68599a69","updated":"2024-11-05 06:37:06.000000000","message":"Changes look good. Zuul and Pure CI have passed.","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"b9f94b2d7499df2d3cffcfde86975f3be8ec23a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"06ffbd0b_223c1c7e","updated":"2024-11-23 11:50:23.000000000","message":"FC failed: http://openstack-logs.purestorage.com/14/931214/4/thirdparty-check/pure-devstack-cinder-tempest-fc-aio-jammy/cba0874/testr_results.html.gz\n\nIs this OK?","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"904aaf4772d6f11c701197a042c586a0b684792b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5d71beaa_400d9f64","updated":"2024-11-20 16:00:05.000000000","message":"None of Pure CI links is accessible.","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"779ab9699a7c0fbd0d40877dcc5a26f51475dce8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5d71186f_fe68587c","updated":"2024-11-21 17:44:37.000000000","message":"Publically accesible logs are here: http://openstack-logs.purestorage.com/14/931214/4/thirdparty-check/","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"15d9929e743c1d7da55fb8f6cfdafcc14589d830","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"66ec56a1_255252a7","updated":"2024-11-22 14:40:16.000000000","message":"Results for iSCSI: http://openstack-logs.purestorage.com/14/931214/4/thirdparty-check/pure-devstack-cinder-tempest-iscsi-aio-jammy/1596c17/testr_results.html.gz\n\nResults for NVMe: http://openstack-logs.purestorage.com/14/931214/4/thirdparty-check/pure-devstack-cinder-tempest-nvme-tcp-aio-jammy/9641cd3/testr_results.html.gz","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"559ae364f5817c8657ab228962b08df01c7a31fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"90807ff4_4a790c59","updated":"2024-11-21 17:32:46.000000000","message":"Simon, are you able to make the CI logs available as a one-off for this patch to get it moving?  We need to have accessible logs and a broader discussion on the requirements and how to configure and deploy, but if you\u0027re able to make results available in some way, this patch could continue review.","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"9ecb233dfe14e4027dc578bf714d182c89a8e67c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"df6de46e_c059e976","updated":"2024-11-22 14:57:45.000000000","message":"Simple fix, improved unit tests, CI reports green, and CI log is accessible.  LGTM.","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"41e51a86ccd4a36fa4e13d4f507f18ca01287a56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2d900f36_d5db0702","updated":"2024-11-23 18:56:51.000000000","message":"This looks okay to me, given the provided CI results.  Code changes are ok.","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"ad880ae6f6f4504d2d90c64215df69636d07051d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"07be010d_406480aa","updated":"2024-10-29 18:26:53.000000000","message":"run Pure Storage CI","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"5bd418cf579762568dd8312c6eed1c7db81d4057","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1a54970c_a68ee227","updated":"2024-10-30 12:43:47.000000000","message":"run Pure Storage CI","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"1aa2d7e49737fd175dc2e9c97cadef6c2ec3967b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"91c8d561_ee4b6e52","updated":"2024-10-29 20:06:01.000000000","message":"run Pure Storage CI","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"f70068e1d838cc685669247549e404ce5b576451","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"feb76886_29c18454","updated":"2024-10-29 15:25:29.000000000","message":"run Pure Storage CI","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"d22199ff178060ac88768222fa2f8f8f90282180","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"808b670f_3be80ced","in_reply_to":"5d71186f_fe68587c","updated":"2024-11-23 12:44:03.000000000","message":"Note - only the iSCSI and NVMe-TCP jobs are voting due to ongoing hardware issues with the other two jobs","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"d22199ff178060ac88768222fa2f8f8f90282180","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"325e7524_4d6c28ea","in_reply_to":"5d71beaa_400d9f64","updated":"2024-11-23 12:44:03.000000000","message":"Public links to results added below","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"1106f5bf3cb8f96908419189304c7c2579302978","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2b187e25_07d14bda","in_reply_to":"c9e3cb73_1c7c2f19","updated":"2024-11-19 23:57:48.000000000","message":"All looks good - awaiting Core votes","commit_id":"dfb233b06ec2f1d111a7f7fb1cb7aac806a360cd"}],"releasenotes/notes/pure-storage-driver-typo-a24d19021f25a4f8.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d391b562d00547316f4c419c65bbcc008599e1d0","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #2083532 \u003chttps://bugs.launchpad.net/cinder/+bug/2083532\u003e`_:"},{"line_number":5,"context_line":"    There is a typo in the pure driver provisoned instead of provisioned on two line 389 et 483."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"78ff4ebd_892bc5f3","line":5,"range":{"start_line":5,"start_character":4,"end_line":5,"end_character":96},"updated":"2024-10-12 14:05:16.000000000","message":"Release note is provided for users, and users don\u0027t care how the problem is fixed, but the problem actually fixed by the change. So this needs to describe something like\n\n```\n[Pure Storage] Fixed creation of volumes with only IOPS qos.\n```","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"f8ba09fc02b13ae794763502528f0b26e51a711a","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #2083532 \u003chttps://bugs.launchpad.net/cinder/+bug/2083532\u003e`_:"},{"line_number":5,"context_line":"    There is a typo in the pure driver provisoned instead of provisioned on two line 389 et 483."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d80bdc99_df88a358","line":5,"range":{"start_line":5,"start_character":4,"end_line":5,"end_character":96},"in_reply_to":"44c4a572_90da3f56","updated":"2024-10-14 10:03:00.000000000","message":"Done","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"61f5996d3ee06b9e0efcb2a8fdb9d6ce89c633b4","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #2083532 \u003chttps://bugs.launchpad.net/cinder/+bug/2083532\u003e`_:"},{"line_number":5,"context_line":"    There is a typo in the pure driver provisoned instead of provisioned on two line 389 et 483."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"44c4a572_90da3f56","line":5,"range":{"start_line":5,"start_character":4,"end_line":5,"end_character":96},"in_reply_to":"78ff4ebd_892bc5f3","updated":"2024-10-14 10:00:12.000000000","message":"Ok i\u0027m doing it 😊","commit_id":"5f5cf6e97dd5caf9e2882ce0a313be03aafebf49"}]}
