*** geguileo is now known as Guest6416 | 07:30 | |
opendevreview | Josephine Seifert proposed openstack/cinder master: WIP: API for volume type metadata https://review.opendev.org/c/openstack/cinder/+/928794 | 09:13 |
---|---|---|
opendevreview | Josephine Seifert proposed openstack/cinder master: Add volume type metadata table to database https://review.opendev.org/c/openstack/cinder/+/918316 | 09:27 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report thin provisioning correctly https://review.opendev.org/c/openstack/cinder/+/924504 | 09:45 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: [HPE Nimble] Report max_oversubscription_rati via backend capabilities https://review.opendev.org/c/openstack/cinder/+/924505 | 10:02 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: [HPE Nimble] Report max_oversubscription_rati via backend capabilities https://review.opendev.org/c/openstack/cinder/+/924505 | 10:11 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report max_oversubscription_ratio via backend capabilities https://review.opendev.org/c/openstack/cinder/+/924505 | 10:22 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report max_oversubscription_ratio via backend capabilities https://review.opendev.org/c/openstack/cinder/+/924505 | 10:24 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report max_oversubscription_ratio via backend capabilities https://review.opendev.org/c/openstack/cinder/+/924505 | 12:28 |
rosmaita | tkajinam: left a comment for you on https://review.opendev.org/c/openstack/cinder/+/929932 | 12:39 |
tkajinam | rosmaita, thanks ! will check it and update the change | 12:43 |
rosmaita | ty | 12:43 |
rosmaita | tkajinam: also, https://review.opendev.org/c/openstack/cinder/+/929390 | 12:46 |
tkajinam | rosmaita, ack | 12:48 |
tkajinam | rosmaita, do you mind also reviewing https://review.opendev.org/c/openstack/cinder/+/929400 ? I hope we can merge this early before additional wrong usage is added. | 12:49 |
rosmaita | sure | 12:49 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: datacore: Mock is_valid_ipv6 to fix unit tests with netaddr >= 1.0.0 https://review.opendev.org/c/openstack/cinder/+/929932 | 13:08 |
tkajinam | I'm afraid mypy job is showhow broken https://paste.opendev.org/show/bjeYBpFJaBAnOoMHotbE/ | 14:42 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix mypy errors https://review.opendev.org/c/openstack/cinder/+/932422 | 14:49 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix mypy errors https://review.opendev.org/c/openstack/cinder/+/932422 | 15:08 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix mypy errors https://review.opendev.org/c/openstack/cinder/+/932422 | 15:34 |
eharney | yeah, new mypy version released, thanks for looking at that | 15:39 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix mypy errors https://review.opendev.org/c/openstack/cinder/+/932422 | 15:42 |
tkajinam | hmm so the problem is that mypy now check possible types quite strictly but is not smart enough to check if-else logic to drop possibility of None. | 15:55 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix mypy errors https://review.opendev.org/c/openstack/cinder/+/932422 | 15:55 |
tkajinam | we really should consider internal volume_type is None check to avoid such confusions ... | 15:55 |
opendevreview | Merged openstack/cinder master: tox: Drop envdir https://review.opendev.org/c/openstack/cinder/+/932257 | 15:56 |
opendevreview | Eric Harney proposed openstack/cinder master: mypy: Print mypy version used https://review.opendev.org/c/openstack/cinder/+/932429 | 15:56 |
*** spy is now known as antonym | 16:41 | |
jbernard | eharney: have you seen https://bugs.launchpad.net/cinder/+bug/2084476 ? | 18:00 |
eharney | i saw it go by but haven't really dug into it yet | 18:00 |
jbernard | is tempest not running with this configuration? | 18:01 |
eharney | probably not glance s3, not sure about image cache | 18:03 |
jbernard | it sounds like the cached volume is created as a snapshot, but not delete when the instance is deleted, leaving it to hang around | 18:04 |
*** elodilles is now known as elodilles_pto | 19:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!