Tuesday, 2024-10-15

*** geguileo is now known as Guest641607:30
opendevreviewJosephine Seifert proposed openstack/cinder master: WIP: API for volume type metadata  https://review.opendev.org/c/openstack/cinder/+/92879409:13
opendevreviewJosephine Seifert proposed openstack/cinder master: Add volume type metadata table to database  https://review.opendev.org/c/openstack/cinder/+/91831609:27
opendevreviewRaghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report thin provisioning correctly  https://review.opendev.org/c/openstack/cinder/+/92450409:45
opendevreviewRaghavendra Tilay proposed openstack/cinder master: [HPE Nimble] Report max_oversubscription_rati via backend capabilities  https://review.opendev.org/c/openstack/cinder/+/92450510:02
opendevreviewRaghavendra Tilay proposed openstack/cinder master: [HPE Nimble] Report max_oversubscription_rati via backend capabilities  https://review.opendev.org/c/openstack/cinder/+/92450510:11
opendevreviewRaghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report max_oversubscription_ratio via backend capabilities  https://review.opendev.org/c/openstack/cinder/+/92450510:22
opendevreviewRaghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report max_oversubscription_ratio via backend capabilities  https://review.opendev.org/c/openstack/cinder/+/92450510:24
opendevreviewRaghavendra Tilay proposed openstack/cinder master: HPE Nimble: Report max_oversubscription_ratio via backend capabilities  https://review.opendev.org/c/openstack/cinder/+/92450512:28
rosmaitatkajinam: left a comment for you on https://review.opendev.org/c/openstack/cinder/+/92993212:39
tkajinamrosmaita, thanks ! will check it and update the change12:43
rosmaitaty12:43
rosmaitatkajinam: also, https://review.opendev.org/c/openstack/cinder/+/92939012:46
tkajinamrosmaita, ack12:48
tkajinamrosmaita, 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
rosmaitasure12:49
opendevreviewTakashi 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/+/92993213:08
tkajinamI'm afraid mypy job is showhow broken https://paste.opendev.org/show/bjeYBpFJaBAnOoMHotbE/14:42
opendevreviewTakashi Kajinami proposed openstack/cinder master: Fix mypy errors  https://review.opendev.org/c/openstack/cinder/+/93242214:49
opendevreviewTakashi Kajinami proposed openstack/cinder master: Fix mypy errors  https://review.opendev.org/c/openstack/cinder/+/93242215:08
opendevreviewTakashi Kajinami proposed openstack/cinder master: Fix mypy errors  https://review.opendev.org/c/openstack/cinder/+/93242215:34
eharneyyeah, new mypy version released, thanks for looking at that15:39
opendevreviewTakashi Kajinami proposed openstack/cinder master: Fix mypy errors  https://review.opendev.org/c/openstack/cinder/+/93242215:42
tkajinamhmm 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
opendevreviewTakashi Kajinami proposed openstack/cinder master: Fix mypy errors  https://review.opendev.org/c/openstack/cinder/+/93242215:55
tkajinamwe really should consider internal volume_type is None check to avoid such confusions ...15:55
opendevreviewMerged openstack/cinder master: tox: Drop envdir  https://review.opendev.org/c/openstack/cinder/+/93225715:56
opendevreviewEric Harney proposed openstack/cinder master: mypy: Print mypy version used  https://review.opendev.org/c/openstack/cinder/+/93242915:56
*** spy is now known as antonym16:41
jbernardeharney: have you seen https://bugs.launchpad.net/cinder/+bug/2084476 ?18:00
eharneyi saw it go by but haven't really dug into it yet18:00
jbernardis tempest not running with this configuration?18:01
eharneyprobably not glance s3, not sure about image cache18:03
jbernardit sounds like the cached volume is created as a snapshot, but not delete when the instance is deleted, leaving it to hang around18:04
*** elodilles is now known as elodilles_pto19:41

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!