Wednesday, 2022-02-09

*** dviroel|ruck|afk is now known as dviroel|ruck00:48
*** dviroel|ruck is now known as dviroel|ruck|out00:57
*** dviroel|ruck|out is now known as dviroel|out00:57
rosmaitatkajinam: ty!01:55
opendevreviewMerged openstack/cinder master: Use LOG.warning instead of deprecated LOG.warn  https://review.opendev.org/c/openstack/cinder/+/81956803:34
opendevreviewTakashi Kajinami proposed openstack/cinder master: Remove usage of undefined CONF.quota_metadata_items  https://review.opendev.org/c/openstack/cinder/+/79878303:59
opendevreviewTakashi Kajinami proposed openstack/cinder master: Fix deprecation warnings caused by invalid UUIDs  https://review.opendev.org/c/openstack/cinder/+/80192305:26
opendevreviewMargaritaShakhova proposed openstack/cinder master: RBD: more safe revert-to-snapshot  https://review.opendev.org/c/openstack/cinder/+/82603606:27
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Return HTTPBadRequest instead of HTTPNotFound  https://review.opendev.org/c/openstack/cinder/+/82845906:47
*** hemna1 is now known as hemna07:37
*** _pewp__ is now known as _pewp_07:38
opendevreviewLior Friedman proposed openstack/os-brick master: nvmeof connector utilize replica_count  https://review.opendev.org/c/openstack/os-brick/+/81171708:14
opendevreviewTushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Add Python 3 only classifier  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/82847508:31
opendevreviewTushar Trambak Gite proposed openstack/python-cinderclient master: Add Python 3 only classifier  https://review.opendev.org/c/openstack/python-cinderclient/+/82847908:37
*** akekane__ is now known as abhishekk09:26
opendevreviewyuval proposed openstack/cinder master: Followup - remove "hostnqn" extra key  https://review.opendev.org/c/openstack/cinder/+/82808509:35
opendevreviewyuval proposed openstack/os-brick master: Reuse get_host_nqn code  https://review.opendev.org/c/openstack/os-brick/+/82808709:41
opendevreviewyuval proposed openstack/os-brick master: Reuse get_host_nqn code  https://review.opendev.org/c/openstack/os-brick/+/82808710:01
*** dviroel|out is now known as dviroel|ruck11:10
opendevreviewMounika Sreeram proposed openstack/cinder master: [SVf] update rccg name property to metadata  https://review.opendev.org/c/openstack/cinder/+/81091711:55
opendevreviewGorka Eguileor proposed openstack/cinder master: Fix request id mess on Cinder-API  https://review.opendev.org/c/openstack/cinder/+/82786012:15
rosmaitageguileo: thanks for fixing that13:23
rosmaitageguileo: when you have a few minutes, could you look at the discussion on https://review.opendev.org/c/openstack/os-brick/+/827496/4/os_brick/initiator/connectors/lightos.py and say if your concerns have been addressed?13:23
yuvalThanks rosmaita and geguileo 13:31
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:58
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^13:58
fabiooliveirathanks14:01
opendevreviewyuval proposed openstack/cinder master: [Lightos] standardize connector usage  https://review.opendev.org/c/openstack/cinder/+/82808514:42
rosmaitano bug squad meeting today15:01
rosmaitause your free time to review os-brick patches!15:02
yuvalis there a way to rigger zuul after he failed?15:03
yuvalit*15:03
yuval:)15:03
akawairosmaita: I would like to ask you to approve following four blueprints : https://blueprints.launchpad.net/cinder/+spec/hitachi-gad-support15:05
akawaihttps://blueprints.launchpad.net/cinder/+spec/hitachi-vsp-aix-os-type15:05
akawaihttps://blueprints.launchpad.net/cinder/+spec/hitachi-vsp-tgt-port-asgn15:06
akawaihttps://blueprints.launchpad.net/cinder/+spec/hitachi-vsp-port-scheduler15:06
rosmaitaakawai: ack15:06
akawaiyes15:07
tkajinamdoes anybody mind reviewing this tiny cleanup, sitting there for a while ? https://review.opendev.org/c/openstack/cinder/+/79878315:15
tkajinamand this deprecation cleanup https://review.opendev.org/c/openstack/cinder/+/79966315:15
rosmaitaakawai: please verify that your BPs are showing up: https://blueprints.launchpad.net/cinder/yoga15:16
rosmaitatkajinam: i'm not sure what to do about https://review.opendev.org/c/openstack/cinder/+/798783 ... you are correct that we don't actually have that config option, but i think we really should!15:19
tkajinamrosmaita, it looks like that code was imported from nova15:19
rosmaitayeah, i was surprised recently when i saw that we don't have a limit on that15:20
rosmaitabut i guess the thing to do now is merge your patch, your bug is specific enough15:20
tkajinamrosmaita, nova has that option. cinder does not. we can implement that if needed but I tend to complete clean-up first15:20
rosmaitatkajinam: agree15:21
tkajinamfact is that nobody has complained about missing quota opts so... might not be something required in wild15:23
rosmaitatkajinam: maybe ... let's not talk about it out loud15:24
rosmaita:)15:24
tkajinam:-)15:24
rosmaitatkajinam: btw, thanks for all these cleanup patches, particularly the deprecation-related ones15:24
tkajinamrosmaita, np :-) I saw several deprecation warnings while looking at unit test logs and fixed what though I could15:26
tkajinamthere are more items lift like SAWarning which I don't have clear idea about but, ideally we should check these items and get rid of them15:26
tkajinams/lift/left/15:26
akawairosmaita: ty! I confirmed to approved our BPs.15:37
opendevreviewHarsh Ailani proposed openstack/cinder master: [SVf] Manage host attachment using portsets  https://review.opendev.org/c/openstack/cinder/+/81735116:12
opendevreviewMerged openstack/cinder stable/victoria: Volume transfers: Remove duplicate policy check  https://review.opendev.org/c/openstack/cinder/+/82724216:37
opendevreviewMerged openstack/cinder stable/victoria: Fix detach notification  https://review.opendev.org/c/openstack/cinder/+/82510016:37
opendevreviewMerged openstack/cinder stable/victoria: Delete attachment on remove_export failure  https://review.opendev.org/c/openstack/cinder/+/82510116:38
opendevreviewGorka Eguileor proposed openstack/python-cinderclient master: Add support for collect-timing option  https://review.opendev.org/c/openstack/python-cinderclient/+/82838016:38
opendevreviewMerged openstack/cinder stable/victoria: Fix old attach method  https://review.opendev.org/c/openstack/cinder/+/82510216:51
opendevreviewMerged openstack/cinder stable/victoria: Remove unnecessary save call  https://review.opendev.org/c/openstack/cinder/+/82510316:51
opendevreviewMerged openstack/cinder stable/victoria: Fix volume OVO create method  https://review.opendev.org/c/openstack/cinder/+/82510416:51
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398517:40
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398517:43
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/82648418:40
*** dviroel|ruck is now known as dviroel|out22:33

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