)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"e5015a22200f29a5ab77d9e2e5bdb246d78b393b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"af3011de_a8ef3104","updated":"2022-11-16 05:58:59.000000000","message":"Looking at the log, the os-brick code is still old. We need to add \"Depends-On:\" tag or rebase?\n\nhttps://zuul.opendev.org/t/openstack/build/809e4e7317de4091b8dbc93639f2a489/log/controller/logs/screen-c-vol.txt#8115-8116\n~~~\n...\nNov 16 02:49:24.784212 ubuntu-focal-ovh-bhs1-0032050649 cinder-volume[104428]: ERROR oslo_messaging.rpc.server             if symlink and symlink.startswith(CUSTOM_LINK_PREFIX):\nNov 16 02:49:24.784212 ubuntu-focal-ovh-bhs1-0032050649 cinder-volume[104428]: ERROR oslo_messaging.rpc.server         AttributeError: \u0027RBDVolumeIOWrapper\u0027 object has no attribute \u0027startswith\u0027\n...\n~~~","commit_id":"a7419a24a6cac5e18671c65a8084af9fb91948f8"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"d0f79b0ba5646f8d5c4de1fd428dd258c0aba123","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"387c0d4c_1a5a649f","updated":"2022-11-15 01:47:06.000000000","message":"recheck","commit_id":"a7419a24a6cac5e18671c65a8084af9fb91948f8"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a69adbe1b69ec14395e27f05047b41a8c8be120c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a6a1d1fe_42751070","updated":"2022-11-16 01:38:41.000000000","message":"recheck\n\n\nThe job still failed.. Looking at the log, it seems that some of tests which creates a volume passed with no issue[1].\nAnd the number of failures increased from 3[2] to 36, so I guess the failures happen randomly.\n\nTo check the number of failures, I\u0027d like to recheck this again.\n\n[1] https://zuul.opendev.org/t/openstack/build/96d8d2e5063542f798c3fa7c3f167661/log/job-output.txt#28403\n~~~\n{2} tempest.api.compute.volumes.test_volumes_get.VolumesGetTestJSON.test_volume_create_get_delete [1.740196s] ... ok\n~~~\n[2] https://zuul.opendev.org/t/openstack/build/d2192284592b4c6fb20ef0610f372556\n[3] https://zuul.opendev.org/t/openstack/build/96d8d2e5063542f798c3fa7c3f167661/log/job-output.txt#44660","commit_id":"a7419a24a6cac5e18671c65a8084af9fb91948f8"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"31872d5c93a973c2e70d5ca77eb5207e5b5f7dc1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4b6793ca_a42c7a2b","updated":"2022-11-15 10:48:34.000000000","message":"recheck\n\n\nspurious cinder-plugin-ceph-tempest failure","commit_id":"a7419a24a6cac5e18671c65a8084af9fb91948f8"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"8c3e28f4800026ffe9c4566655a6dfedc544eb2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"354d1095_cfa584a6","updated":"2022-11-15 17:37:56.000000000","message":"recheck\n\ndependant patch https://review.opendev.org/c/openstack/os-brick/+/864479 that resolves the zuul failure is now merged.","commit_id":"a7419a24a6cac5e18671c65a8084af9fb91948f8"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6cd378203280a77f378a67892c9bf6c48033efa6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8f22ed4d_9b65fe7c","updated":"2022-11-16 23:50:45.000000000","message":"Still failing :(\nThe os-brick code looks like still old[1].. I\u0027m not sure why but rebasing is necessary?\n\n\n[1] https://zuul.opendev.org/t/openstack/build/7329c68ede7f4f7e9dbf19acea0b258a/log/controller/logs/screen-c-vol.txt#7158\n~~~\nNov 16 07:10:58.939247 ubuntu-focal-ovh-bhs1-0032059019 cinder-volume[104857]:           File \"/usr/local/lib/python3.8/dist-packages/os_brick/utils.py\", line 226, in _device_path_from_symlink\nNov 16 07:10:58.939247 ubuntu-focal-ovh-bhs1-0032059019 cinder-volume[104857]:             if symlink and symlink.startswith(CUSTOM_LINK_PREFIX):\nNov 16 07:10:58.939247 ubuntu-focal-ovh-bhs1-0032059019 cinder-volume[104857]:         AttributeError: \u0027RBDVolumeIOWrapper\u0027 object has no attribute \u0027startswith\u0027\n~~~","commit_id":"5ca770fe0f5a2cd42f185a4b866fabf61e3cc6a2"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"268891d63fe3e488dabb39de37620d13fb1af015","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d509c90d_7a0d5920","in_reply_to":"8f22ed4d_9b65fe7c","updated":"2022-11-17 01:38:43.000000000","message":"hmm, it seems os-brick-4.3.4 is installed from pypi(cache). So, do we need to release to use it? or do we have a way to use unreleased os-brick version?\n\n\nhttps://zuul.opendev.org/t/openstack/build/7329c68ede7f4f7e9dbf19acea0b258a/log/controller/logs/devstacklog.txt#6422\n~~~\n2022-11-16 06:16:26.103 |   Downloading https://mirror.bhs1.ovh.opendev.org/pypifiles/packages/e6/21/d9bab89db08a1e2a9f4c7365a9012630c1b30ada2257cf240953e6932de2/os_brick-4.3.4-py3-none-any.whl (259 kB)\n~~~\n\nhttps://zuul.opendev.org/t/openstack/build/7329c68ede7f4f7e9dbf19acea0b258a/log/controller/logs/devstacklog.txt#6485\n~~~\n2022-11-16 06:16:29.002 | Successfully installed argparse-1.4.0 glance-store-2.5.1 httplib2-0.19.0 os-brick-4.3.4 os-win-5.4.0 oslo.privsep-2.5.1 oslo.rootwrap-6.3.1 python-cinderclient-7.4.1 python-swiftclient-3.11.1 simplejson-3.17.2\n~~~","commit_id":"5ca770fe0f5a2cd42f185a4b866fabf61e3cc6a2"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"dc6711c84b8f631e6c7618bc1a15dc5a511f8c54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"477be022_1a2c24a9","in_reply_to":"d509c90d_7a0d5920","updated":"2022-11-17 16:12:50.000000000","message":"You\u0027re right, we need to release os-brick to see changes in other projects since it\u0027s a library.\nUnfortunately, wallaby has transitioned into EM so after discussing this with release team, I\u0027ve proposed another way to fix this situation. This patch should probably fix it.\n\nhttps://review.opendev.org/c/openstack/cinder/+/864916","commit_id":"5ca770fe0f5a2cd42f185a4b866fabf61e3cc6a2"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"baf696dcc5079a10572de01c058e584e7a1aab15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f7afe489_b54a65d4","updated":"2022-11-21 19:42:29.000000000","message":"@Rajat, my interpretation is the same has Brian\u0027s.  Config options get a little confusing, as they do border on adding new functionality and do need a case by case review.  In this case it is a more or less minor improvement and the config option won\u0027t break anything, as Brian indicated.","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2761805fbe6d79175e01e973a4d7e8105f3b3c9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e5c8b307_abb21362","updated":"2022-11-21 15:39:09.000000000","message":"@Rajat: my reading of the guidelines is that a new config option is only a problem if it\u0027s incompatible with current configuration (i.e., if someone upgraded to a release containing the new option, something could break).  This option doesn\u0027t change the default behavior and won\u0027t break current configurations, so I think it\u0027s OK.\n\nSo I\u0027m +2 on this backport, but will hold off +A to allow for other opinions.","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"70931281ea03d682db0327015a72f86749b69b46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b15153f4_79e39faf","updated":"2022-11-21 19:48:40.000000000","message":"Based on stable cores feedback, this looks appropriate for backporting. merged in xena (master at that time) so looks good for wallaby.","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"a41e29be976ad2568ef9ce15c01d80f6f5dcd325","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4122991d_2c8f4e96","updated":"2022-11-18 18:34:56.000000000","message":"Looks like this was properly cherry-picked.","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"53c489503095d08c9c8c52b0410bb27bf7061805","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a4c3456c_9da643d6","updated":"2022-11-17 22:26:57.000000000","message":"Note: This will never pass the NVMe driver as it isn\u0027t available in Wallaby.","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a86e0243b480a3b42ecfd4c494503158cc35408f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4bc038bb_f75d73de","updated":"2022-11-17 22:17:25.000000000","message":"Rebased on the patch that uses os-brick from stable/wallaby (instead of from released os-brick) in the failing cinder-plugin-ceph-tempest job.","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"96fb5e28b36317155bbb81469848342fd6b53027","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"07a4b7c8_cf4b3881","updated":"2022-11-18 03:02:40.000000000","message":"Thanks Rajat and Brian!\nLGTM.","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ddd1b94c22f0a4835c94fc54485ab63fc6ac7988","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8733fdbd_fa360487","updated":"2022-11-21 06:36:10.000000000","message":"This introduces a new config option that might not comply with stable branch policies. However, i was unable to find anything related to it in the stable branch guidelines[1]. Will wait for opinions of other stable branch maintainers.\n\n[1] https://docs.openstack.org/project-team-guide/stable-branches.html","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"f0324804e15de46c580df09a8cc37c624f5c8397","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"25ea9ff5_5589d603","updated":"2022-11-22 01:21:59.000000000","message":"recheck","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"8e7290838725f9388777aba2c45e34c537dbb4c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a5a57d3e_35a0de53","updated":"2022-11-22 04:44:00.000000000","message":"recheck cinder-plugin-ceph-tempest\n\nunrelated failure in DB\n\nNov 21 20:35:56.701109 ubuntu-focal-ovh-gra1-0032156513 cinder-volume[111603]: ERROR oslo_db.sqlalchemy.engines oslo_db.exception.DBConnectionError: (pymysql.err.OperationalError) (2013, \u0027Lost connection to MySQL server during query\u0027)\n","commit_id":"84dc4fb0599743ad19eac43ca2e10dedb5d5386b"}]}