*** dviroel|ruck|afk is now known as dviroel|ruck | 00:48 | |
*** dviroel|ruck is now known as dviroel|ruck|out | 00:57 | |
*** dviroel|ruck|out is now known as dviroel|out | 00:57 | |
rosmaita | tkajinam: ty! | 01:55 |
---|---|---|
opendevreview | Merged openstack/cinder master: Use LOG.warning instead of deprecated LOG.warn https://review.opendev.org/c/openstack/cinder/+/819568 | 03:34 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove usage of undefined CONF.quota_metadata_items https://review.opendev.org/c/openstack/cinder/+/798783 | 03:59 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix deprecation warnings caused by invalid UUIDs https://review.opendev.org/c/openstack/cinder/+/801923 | 05:26 |
opendevreview | MargaritaShakhova proposed openstack/cinder master: RBD: more safe revert-to-snapshot https://review.opendev.org/c/openstack/cinder/+/826036 | 06:27 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Return HTTPBadRequest instead of HTTPNotFound https://review.opendev.org/c/openstack/cinder/+/828459 | 06:47 |
*** hemna1 is now known as hemna | 07:37 | |
*** _pewp__ is now known as _pewp_ | 07:38 | |
opendevreview | Lior Friedman proposed openstack/os-brick master: nvmeof connector utilize replica_count https://review.opendev.org/c/openstack/os-brick/+/811717 | 08:14 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Add Python 3 only classifier https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/828475 | 08:31 |
opendevreview | Tushar Trambak Gite proposed openstack/python-cinderclient master: Add Python 3 only classifier https://review.opendev.org/c/openstack/python-cinderclient/+/828479 | 08:37 |
*** akekane__ is now known as abhishekk | 09:26 | |
opendevreview | yuval proposed openstack/cinder master: Followup - remove "hostnqn" extra key https://review.opendev.org/c/openstack/cinder/+/828085 | 09:35 |
opendevreview | yuval proposed openstack/os-brick master: Reuse get_host_nqn code https://review.opendev.org/c/openstack/os-brick/+/828087 | 09:41 |
opendevreview | yuval proposed openstack/os-brick master: Reuse get_host_nqn code https://review.opendev.org/c/openstack/os-brick/+/828087 | 10:01 |
*** dviroel|out is now known as dviroel|ruck | 11:10 | |
opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] update rccg name property to metadata https://review.opendev.org/c/openstack/cinder/+/810917 | 11:55 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix request id mess on Cinder-API https://review.opendev.org/c/openstack/cinder/+/827860 | 12:15 |
rosmaita | geguileo: thanks for fixing that | 13:23 |
rosmaita | geguileo: 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 |
yuval | Thanks rosmaita and geguileo | 13:31 |
rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^ | 13:58 |
fabiooliveira | thanks | 14:01 |
opendevreview | yuval proposed openstack/cinder master: [Lightos] standardize connector usage https://review.opendev.org/c/openstack/cinder/+/828085 | 14:42 |
rosmaita | no bug squad meeting today | 15:01 |
rosmaita | use your free time to review os-brick patches! | 15:02 |
yuval | is there a way to rigger zuul after he failed? | 15:03 |
yuval | it* | 15:03 |
yuval | :) | 15:03 |
akawai | rosmaita: I would like to ask you to approve following four blueprints : https://blueprints.launchpad.net/cinder/+spec/hitachi-gad-support | 15:05 |
akawai | https://blueprints.launchpad.net/cinder/+spec/hitachi-vsp-aix-os-type | 15:05 |
akawai | https://blueprints.launchpad.net/cinder/+spec/hitachi-vsp-tgt-port-asgn | 15:06 |
akawai | https://blueprints.launchpad.net/cinder/+spec/hitachi-vsp-port-scheduler | 15:06 |
rosmaita | akawai: ack | 15:06 |
akawai | yes | 15:07 |
tkajinam | does anybody mind reviewing this tiny cleanup, sitting there for a while ? https://review.opendev.org/c/openstack/cinder/+/798783 | 15:15 |
tkajinam | and this deprecation cleanup https://review.opendev.org/c/openstack/cinder/+/799663 | 15:15 |
rosmaita | akawai: please verify that your BPs are showing up: https://blueprints.launchpad.net/cinder/yoga | 15:16 |
rosmaita | tkajinam: 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 |
tkajinam | rosmaita, it looks like that code was imported from nova | 15:19 |
rosmaita | yeah, i was surprised recently when i saw that we don't have a limit on that | 15:20 |
rosmaita | but i guess the thing to do now is merge your patch, your bug is specific enough | 15:20 |
tkajinam | rosmaita, nova has that option. cinder does not. we can implement that if needed but I tend to complete clean-up first | 15:20 |
rosmaita | tkajinam: agree | 15:21 |
tkajinam | fact is that nobody has complained about missing quota opts so... might not be something required in wild | 15:23 |
rosmaita | tkajinam: maybe ... let's not talk about it out loud | 15:24 |
rosmaita | :) | 15:24 |
tkajinam | :-) | 15:24 |
rosmaita | tkajinam: btw, thanks for all these cleanup patches, particularly the deprecation-related ones | 15:24 |
tkajinam | rosmaita, np :-) I saw several deprecation warnings while looking at unit test logs and fixed what though I could | 15:26 |
tkajinam | there 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 them | 15:26 |
tkajinam | s/lift/left/ | 15:26 |
akawai | rosmaita: ty! I confirmed to approved our BPs. | 15:37 |
opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf] Manage host attachment using portsets https://review.opendev.org/c/openstack/cinder/+/817351 | 16:12 |
opendevreview | Merged openstack/cinder stable/victoria: Volume transfers: Remove duplicate policy check https://review.opendev.org/c/openstack/cinder/+/827242 | 16:37 |
opendevreview | Merged openstack/cinder stable/victoria: Fix detach notification https://review.opendev.org/c/openstack/cinder/+/825100 | 16:37 |
opendevreview | Merged openstack/cinder stable/victoria: Delete attachment on remove_export failure https://review.opendev.org/c/openstack/cinder/+/825101 | 16:38 |
opendevreview | Gorka Eguileor proposed openstack/python-cinderclient master: Add support for collect-timing option https://review.opendev.org/c/openstack/python-cinderclient/+/828380 | 16:38 |
opendevreview | Merged openstack/cinder stable/victoria: Fix old attach method https://review.opendev.org/c/openstack/cinder/+/825102 | 16:51 |
opendevreview | Merged openstack/cinder stable/victoria: Remove unnecessary save call https://review.opendev.org/c/openstack/cinder/+/825103 | 16:51 |
opendevreview | Merged openstack/cinder stable/victoria: Fix volume OVO create method https://review.opendev.org/c/openstack/cinder/+/825104 | 16:51 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 17:40 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 17:43 |
opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 18:40 |
*** dviroel|ruck is now known as dviroel|out | 22:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!