| *** amar7ibm is now known as amar7_ | 04:40 | |
| zenkuro | hi! can anybody please check: https://review.opendev.org/c/openstack/cinder/+/794962 ? | 08:06 |
|---|---|---|
| lyarwood | if anyone has time today I'm seeing a weird c-bak failure in a CentOS 8 stream job https://bugs.launchpad.net/cinder/+bug/1936909 | 09:50 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Fix for renaming GVG https://review.opendev.org/c/openstack/cinder/+/801354 | 11:05 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Fix for legacy PowerMax OS around generations https://review.opendev.org/c/openstack/cinder/+/778907 | 11:18 |
| opendevreview | Merged openstack/python-cinderclient master: Remove v2 support from the shell https://review.opendev.org/c/openstack/python-cinderclient/+/791834 | 12:17 |
| rosmaita | \o/ -- 1 down 3 to go! | 12:17 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: mypy: db/sqlalchemy/api.py https://review.opendev.org/c/openstack/cinder/+/788369 | 12:53 |
| opendevreview | Merged openstack/python-cinderclient master: Remove v2 classes https://review.opendev.org/c/openstack/python-cinderclient/+/792959 | 13:49 |
| rosmaita | \o/ | 13:50 |
| rosmaita | geguileo: you looked at an earlier version of this, do you have some time to take a look today? https://review.opendev.org/c/openstack/python-cinderclient/+/783628 | 13:52 |
| rosmaita | would probably be good to include in the M-2 cinderclient release | 13:52 |
| *** dviroel is now known as dviroel|lunch | 14:54 | |
| geguileo | rosmaita: done | 15:44 |
| rosmaita | geguileo: ty | 15:45 |
| geguileo | rosmaita: ty for the reminder | 15:45 |
| rosmaita | btw, i noticed that 'abc' you pointed out on the unit test, but i was wondering if the shell accepts either volume name or id on the command line, and then i forgot to go look | 15:47 |
| geguileo | it probably does accepts both | 15:54 |
| geguileo | but the volume is not even created with that name lol | 15:55 |
| rosmaita | i didn't look that far! | 15:56 |
| *** dviroel|lunch is now known as dviroel | 16:00 | |
| opendevreview | Merged openstack/python-cinderclient master: Make instance_uuid optional in attachment create https://review.opendev.org/c/openstack/python-cinderclient/+/783628 | 17:06 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: [rbd] Fix creating encrypted volume from unencrypted image https://review.opendev.org/c/openstack/cinder/+/801522 | 18:12 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: [rbd] Fix creating encrypted volume from unencrypted image https://review.opendev.org/c/openstack/cinder/+/801522 | 18:17 |
| hemna | mep | 19:38 |
| hemna | just found a bug in the volume manager | 19:38 |
| hemna | extra_capabilities aren't set in the pool stats for a volume dirver | 19:39 |
| hemna | driver | 19:39 |
| hemna | and so the capabilities filter doesn't see them | 19:39 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L2657-L2658 | 19:40 |
| hemna | they only get shoved in the dict that comes back from stats update | 19:41 |
| hemna | and aren't inserted into the pools capabilities records. | 19:41 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Add cinder permissions matrix https://review.opendev.org/c/openstack/cinder/+/763306 | 23:00 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Allow removing NFS snapshots in error status https://review.opendev.org/c/openstack/cinder/+/679138 | 23:39 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!