Tuesday, 2022-02-01

*** dviroel|brb is now known as dviroel00:03
*** dviroel is now known as dviroel_00:19
*** dviroel_ is now known as dviroel00:19
*** dviroel is now known as dviroel|out01:47
opendevreviewVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/82648407:03
*** hemna4 is now known as hemna07:38
*** akekane_ is now known as abhishekk08:51
opendevreviewAtsushi Kawai proposed openstack/cinder master: This patch is bugfixing and refactoring for the following patch: https://review.opendev.org/c/openstack/cinder/+/786873  https://review.opendev.org/c/openstack/cinder/+/82725910:23
*** dviroel|out is now known as dviroel11:27
yuvaltempest random volume type nameing12:33
yuvalI upload a patch12:33
yuvalbut then I noticed there is also a problem in cinder-tempet-plugin12:34
opendevreviewyuval proposed openstack/cinder-tempest-plugin master: test_create_from_image - add random prefix  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/82727812:37
yuvalthere is some duplication in the two patches - so if its a problem please point me to where I should put the duplicated code12:38
rosmaitayuval: ty, will take a look later13:28
opendevreviewRajat Dhasmana proposed openstack/cinder stable/victoria: Volume transfers: Remove duplicate policy check  https://review.opendev.org/c/openstack/cinder/+/82724214:37
*** bhagyashris_ is now known as bhagyashris14:42
rosmaitawhoami-rajat: i put together this dashboard because i keep missing patches proposed by the release team, you may find it useful: http://tiny.cc/cinder-release-patches14:43
whoami-rajatcool, thanks!14:44
eharneyhemna: https://review.opendev.org/c/openstack/cinder/+/800703 could use another look when you get a chance14:53
Adri2000hi :) anyone familiar with the netapp/nfs driver codebase could have a look at this patch https://review.opendev.org/c/openstack/cinder/+/820578 ? it's been in review for the last 2 months, the change itself looks pretty simple and as a user I confirmed it fixes the problem (resizing attached volume now works). thanks!15:14
*** dviroel is now known as dviroel|lunch15:40
opendevreviewyuval proposed openstack/os-brick master: Lightbits LightOS driver  https://review.opendev.org/c/openstack/os-brick/+/82160316:01
yuval<geguileo> following your comments ^16:02
*** abishop_ is now known as abishop16:11
*** dviroel|lunch is now known as dviroel16:35
geguileorosmaita: there are a couple of changes that I wanted to see in yuval connector info, but we are too close to the deadline16:41
geguileorosmaita: can we merge what we have now and make the changes as a copule of simple follow up patches?16:41
geguileorosmaita: it will be ok as long as we don't do an actual release of os-brick or cinder before those 2 merge16:41
rosmaitai am ok with that16:41
rosmaitabrick release is scheduled for next friday, i think16:42
geguileorosmaita: so we can merge these 2 patches that use the new hostnqn entry in the connector properties, and then we add a couple of patches that change it to use the existing nqn instead16:42
rosmaita"Changes to be included in the Yoga release of the os-brick library must be merged by Thursday 10 February 2022 (20:00 UTC)"16:43
rosmaitageguileo: i am ok with that, let's be clear on the gerrit review what followups we are requiring16:44
geguileoyuval: I +2 the os-brick patch and added the comment with a summary of what we discussed about the hostnqn and nqn16:48
geguileoyuval: feel free to ping me for review when you have the follow up cinder and os-brick patches16:48
yuvalfixe for nqn till 10 Feb16:49
yuvalgot it16:49
smcginnis"brick release is scheduled for next friday, i think" Hey jungleboyj, do you want to tell rosmaita what day release deadlines happen on? XD16:51
rosmaitasmcginnis: we are actually early by a week from official openstack16:52
rosmaitai'm trying an experiment so we can have 2 weeks between os-brick release and python-cinderclient release16:52
smcginnisHah, OK. Just gave jungleboyj a hard time once over release deadlines. ;)16:52
rosmaitasmcginnis: i know, *thursday* is the day116:53
smcginnisIt was more fun giving Jay a hard time about it!16:53
rosmaitawell, Jay is just more fun overall, i think16:54
rosmaitaif someone has a few minutes, this is a pretty quick review that will help with CI for nvme, which as you know, is so popular nowadays: https://review.opendev.org/c/openstack/cinder/+/79192917:05
jungleboyjsmcginnis:  So, you go and disappear for a long time and then show up to harass me !?!  ;-)17:07
smcginnisOf course! :P17:08
jungleboyjI don't even remember what that whole fiasco was.17:08
smcginnisProbably just met harassing you on something.17:09
rosmaitasmcginnis: ty!17:10
jungleboyjIndeed.  I probably tried to move a release day to Monday or something.17:11
jungleboyjI put my two cents but didn't vote since I saw smcginnis started the merge.17:12
rosmaitacool, thanks for looking at it17:14
*** dviroel is now known as dviroel|biab17:55
*** dviroel|biab is now known as dviroel18:08
opendevreviewMerged openstack/cinder master: LVM: Added NVMe TCP support for the nvmet target  https://review.opendev.org/c/openstack/cinder/+/79192920:24
*** dviroel is now known as dviroel|afk20:35
opendevreviewMerged openstack/cinder master: [SVf] Add cleanrate in volume-type extra specs  https://review.opendev.org/c/openstack/cinder/+/81187720:41
jungleboyjmsg smcginnis Are you going to the Berlin Summit?20:41
jungleboyjWell, that was a fail.20:41
opendevreviewAtsushi Kawai proposed openstack/cinder master: Hitachi: bugfix and refactoring for add maintenance parameters  https://review.opendev.org/c/openstack/cinder/+/82725923:31
*** dviroel|afk is now known as dviroel23:44

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