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
opendevreviewTakashi Kajinami proposed openstack/cinder master: Remove usage of undefined CONF.quota_metadata_items
opendevreviewTakashi Kajinami proposed openstack/cinder master: Fix deprecation warnings caused by invalid UUIDs
opendevreviewMargaritaShakhova proposed openstack/cinder master: RBD: more safe revert-to-snapshot
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Return HTTPBadRequest instead of HTTPNotFound
*** 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
opendevreviewTushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Add Python 3 only classifier
opendevreviewTushar Trambak Gite proposed openstack/python-cinderclient master: Add Python 3 only classifier
*** akekane__ is now known as abhishekk09:26
opendevreviewyuval proposed openstack/cinder master: Followup - remove "hostnqn" extra key
opendevreviewyuval proposed openstack/os-brick master: Reuse get_host_nqn code
opendevreviewyuval proposed openstack/os-brick master: Reuse get_host_nqn code
*** dviroel|out is now known as dviroel|ruck11:10
opendevreviewMounika Sreeram proposed openstack/cinder master: [SVf] update rccg name property to metadata
opendevreviewGorka Eguileor proposed openstack/cinder master: Fix request id mess on Cinder-API
rosmaitageguileo: thanks for fixing that13:23
rosmaitageguileo: when you have a few minutes, could you look at the discussion on 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
opendevreviewyuval proposed openstack/cinder master: [Lightos] standardize connector usage
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
akawairosmaita: I would like to ask you to approve following four blueprints :
rosmaitaakawai: ack15:06
tkajinamdoes anybody mind reviewing this tiny cleanup, sitting there for a while ?
tkajinamand this deprecation cleanup
rosmaitaakawai: please verify that your BPs are showing up:
rosmaitatkajinam: i'm not sure what to do about ... 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
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
akawairosmaita: ty! I confirmed to approved our BPs.15:37
opendevreviewHarsh Ailani proposed openstack/cinder master: [SVf] Manage host attachment using portsets
opendevreviewMerged openstack/cinder stable/victoria: Volume transfers: Remove duplicate policy check
opendevreviewMerged openstack/cinder stable/victoria: Fix detach notification
opendevreviewMerged openstack/cinder stable/victoria: Delete attachment on remove_export failure
opendevreviewGorka Eguileor proposed openstack/python-cinderclient master: Add support for collect-timing option
opendevreviewMerged openstack/cinder stable/victoria: Fix old attach method
opendevreviewMerged openstack/cinder stable/victoria: Remove unnecessary save call
opendevreviewMerged openstack/cinder stable/victoria: Fix volume OVO create method
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm
*** dviroel|ruck is now known as dviroel|out22:33

Generated by 2.17.3 by Marius Gedminas - find it at!