Tuesday, 2022-06-14

opendevreviewBrian Rosmaita proposed openstack/cinderlib master: Add local upper constraints support  https://review.opendev.org/c/openstack/cinderlib/+/84560703:02
HappyStackerHello everyone08:40
HappyStackerin cinder when creating a volume, what does model_updates?08:41
opendevreviewGorka Eguileor proposed openstack/os-brick master: NVMe-oF: Consolidate code paths  https://review.opendev.org/c/openstack/os-brick/+/83606010:32
opendevreviewGorka Eguileor proposed openstack/os-brick master: NVMe-oF: Disconnect subsystems  https://review.opendev.org/c/openstack/os-brick/+/83606210:32
opendevreviewGorka Eguileor proposed openstack/os-brick master: Support shared_targets tristate value  https://review.opendev.org/c/openstack/os-brick/+/83606310:32
opendevreviewGorka Eguileor proposed openstack/cinder master: Doc: Mention tick-tock upgrades  https://review.opendev.org/c/openstack/cinder/+/83028310:40
opendevreviewGorka Eguileor proposed openstack/os-brick master: NVMe-oF: Get system uuid in privsep  https://review.opendev.org/c/openstack/os-brick/+/83605710:57
*** dviroel|out is now known as dviroel11:22
HappyStackerquick question: we have a new storage plafform coming in based upon our PowerMax serie. As we need to add support for it, is a blueprint required?12:56
opendevreviewBrian Rosmaita proposed openstack/cinderlib master: Add local upper constraints support  https://review.opendev.org/c/openstack/cinderlib/+/84560712:57
rosmaitaHappyStacker: model_update ... see https://docs.openstack.org/cinder/latest/contributor/migration.html#required-methods and https://docs.openstack.org/cinder/latest/contributor/groups.html and https://docs.openstack.org/cinder/latest/contributor/attach_detach_conventions.html#driver-create-export13:13
rosmaitaHappyStacker: new storage plafform sounds like either a new driver or a new feature, so yes, file a blueprint and make sure whoami-rajat knows about it13:13
HappyStackerThanks 13:27
HappyStackerI have submitted a new blueprint for supporting our new PowerMax V4 platform https://blueprints.launchpad.net/cinder/+spec/powermax-v4-support13:39
whoami-rajatrosmaita, thanks for guidance and thanks HappyStacker for filing the blueprint14:01
kgubeHi all! I opened this bug and am working on it: https://bugs.launchpad.net/cinder/+bug/197829414:50
kgubeBut since I'm completely new to cinder development, I wanted to ask if I am taking the right approach here.14:50
kgubeAny suggestions or comments are very welcome!14:50
enriquetasokgube**15:06
enriquetasokgube++15:06
*** dviroel is now known as dviroel|lunch15:58
opendevreviewLuisa proposed openstack/cinder master: NetApp ONTAP: Fix type error exception in get_volume_state  https://review.opendev.org/c/openstack/cinder/+/82635016:43
opendevreviewMerged openstack/cinder master: db: Move comment  https://review.opendev.org/c/openstack/cinder/+/84496416:49
opendevreviewMerged openstack/cinder master: cmd/manage and coordination: Clean up exception handling  https://review.opendev.org/c/openstack/cinder/+/84214216:58
opendevreviewMerged openstack/cinder master: pylint: volume/api: Fix E0601 error  https://review.opendev.org/c/openstack/cinder/+/84214816:58
opendevreviewMerged openstack/cinder master: pylint: ibm_storage: Fix E0601 used-before-assignment error  https://review.opendev.org/c/openstack/cinder/+/84214916:59
opendevreviewMerged openstack/cinder master: Ceph: Remove unnecessary convert_str() calls  https://review.opendev.org/c/openstack/cinder/+/83925716:59
opendevreviewMerged openstack/cinder master: Groups: remove unneeded "status" variable  https://review.opendev.org/c/openstack/cinder/+/84049317:08
opendevreviewMerged openstack/cinder stable/victoria: [stable-only] Cap cinder-tempest-plugin version for stable/victoria  https://review.opendev.org/c/openstack/cinder/+/84330517:08
*** dviroel|lunch is now known as dviroel17:17
dc_arunHello core reviewers, please review patch https://review.opendev.org/c/openstack/cinder/+/83699617:29
opendevreviewLuigi Toscano proposed openstack/cinder stable/wallaby: NetApp ONTAP: Fix check QoS min support for SVM account  https://review.opendev.org/c/openstack/cinder/+/84579917:54
opendevreviewGorka Eguileor proposed openstack/cinder master: Revert "Cleanup code duplication in cinder.cmd.backup module"  https://review.opendev.org/c/openstack/cinder/+/84580618:37
geguileorosmaita: whoami-rajat abishop  that's the cinder part to reduce memory consumption of the cinder-backup service18:48
geguileothis is the devstack one: https://review.opendev.org/c/openstack/devstack/+/84580518:49
rosmaitageguileo: nice!18:49
geguileoand this is the tripleo one (which I haven't tested): https://review.opendev.org/c/openstack/tripleo-common/+/84580718:49
*** dviroel is now known as dviroel|afk20:46
opendevreviewSofia Enriquez proposed openstack/cinder-tempest-plugin master: Add tests for Backup Restore Multibackend  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/83551221:05
opendevreviewSofia Enriquez proposed openstack/cinder-tempest-plugin master: Add tests for Backup Restore Multibackend  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/83551221:05
simondodsleyAnyone else seeing tempest test `tempest.api.compute.admin.test_volume_swap.TestMultiAttachVolumeSwap` failing with `ValueError: Multiple pingable or sshable servers not supported at this stage.`21:16
simondodsleyLooks like this patch has caused it... https://review.opendev.org/c/openstack/tempest/+/84292121:40
simondodsleyAnyone else seeing this?21:40
opendevreviewPeter Penchev proposed openstack/cinder master: Send the correct location URI to the Glance v2 API  https://review.opendev.org/c/openstack/cinder/+/84517823:00
opendevreviewPeter Penchev proposed openstack/cinder master: Send the correct location URI to the Glance v2 API  https://review.opendev.org/c/openstack/cinder/+/84517823:28
opendevreviewSofia Enriquez proposed openstack/cinder master: RBD: Flattening of child volumes during deletion  https://review.opendev.org/c/openstack/cinder/+/83538423:34

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