| *** simondodsley_ is now known as simondodsley | 09:31 | |
| *** erlon_ is now known as erlon | 09:32 | |
| *** TheJulia_ is now known as TheJulia | 09:32 | |
| *** manpreetk_ is now known as manpreetk | 09:32 | |
| *** dviroel|out is now known as dviroel | 10:54 | |
| opendevreview | Xuan Yandong proposed openstack/cinder master: Prohibit volume manage to an encrypted volume type https://review.opendev.org/c/openstack/cinder/+/768458 | 11:38 |
|---|---|---|
| caiqilong | Hi core reviewers, if you have time, please look at https://review.opendev.org/c/openstack/cinder/+/816443 | 12:55 |
| opendevreview | Xuan Yandong proposed openstack/cinder master: Prohibit volume manage to an encrypted volume type https://review.opendev.org/c/openstack/cinder/+/768458 | 13:02 |
| xuany | whoami-rajat: hello | 13:22 |
| opendevreview | mitya-eremeev-2 proposed openstack/cinder master: Fix simultaneous volume creation error in multiattach mode. https://review.opendev.org/c/openstack/cinder/+/805072 | 13:23 |
| xuany | whoami-rajat: the bp spec revert-any-snapshot looks ok, When will it merge | 13:24 |
| caiqilong | rosmaita:Hi, if you have time, please look at https://review.opendev.org/c/openstack/cinder/+/816443 | 13:43 |
| xuany | rosmaita: hello, i was too busy some time ago | 14:19 |
| xuany | . and i was update the patch for manage encrypt volume_type patch | 14:19 |
| rosmaita | xuany: hello | 14:20 |
| xuany | and the the bp spec revert-any-snapshot looks ok, When will it merge | 14:20 |
| rosmaita | spec freeze is Friday 17 December 2021 (23:59 UTC) | 14:22 |
| rosmaita | i have some draft comments on your spec, but haven't finished reviewing it yet | 14:23 |
| rosmaita | i saw your update on "Prohibit volume manage to an encrypted volume type", thank you! | 14:24 |
| xuany | Ok, do i need add a unit tests for manage_to_default_volumetype? | 14:25 |
| rosmaita | xuany: i think that would be a good test to add ... looking at your code, i can see that you handle that case, but it would be good to have the test in case your code gets refactored at some point ... it will protect against a regression | 14:28 |
| xuany | ok, i will try to add it, Must I have the same patch ? | 14:33 |
| rosmaita | i think it would be good to have it in this patch | 14:34 |
| rosmaita | when you have time to add it | 14:34 |
| xuany | OK, I'll update it later | 14:35 |
| opendevreview | huawei volume/driver-huawei HUAWEI DRIVER CI proposed openstack/cinder stable/ussuri: fix 'huawei dorado v6' create new volume form snapshot error https://review.opendev.org/c/openstack/cinder/+/818778 | 14:35 |
| rosmaita | xuany: that will be fine | 14:37 |
| caiqilong | rosmaita: Hi, if you have time, please look at https://review.opendev.org/c/openstack/cinder/+/816443 | 14:42 |
| rosmaita | caiqilong: ack | 14:45 |
| caiqilong | rosmaita: thanks | 14:46 |
| *** dviroel is now known as dviroel|lunch | 15:11 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 15:52 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group os-reset_status https://review.opendev.org/c/openstack/cinder/+/804735 | 15:52 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group-snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804757 | 15:53 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 15:53 |
| *** dviroel|lunch is now known as dviroel | 16:24 | |
| opendevreview | Leonardo Campelo proposed openstack/cinder master: API-REF: Add volume_id at snapshot list and detail https://review.opendev.org/c/openstack/cinder/+/818815 | 17:52 |
| *** dviroel_ is now known as dviroel|afk | 20:52 | |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: RBD driver https://review.opendev.org/c/openstack/cinder/+/774288 | 21:23 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD driver: clean up _trash_purge exception handling https://review.opendev.org/c/openstack/cinder/+/818830 | 21:23 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: Add encryption and decryption for files for Image Encryption https://review.opendev.org/c/openstack/os-brick/+/709432 | 21:45 |
| opendevreview | Goutham Pacha Ravi proposed openstack/cinder master: replace deprecated pyparsing method https://review.opendev.org/c/openstack/cinder/+/818834 | 22:17 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!