Wednesday, 2022-04-06

whoami-rajatsean-k-mooney, sure, please add the topic to the planning etherpad and i will look for a slot to disucss it on friday https://etherpad.opendev.org/p/zed-ptg-cinder-planning04:40
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/82648406:07
opendevreviewBence Romsics proposed openstack/cinder stable/victoria: Fix: Schema validation for attachment create API  https://review.opendev.org/c/openstack/cinder/+/83675309:12
opendevreviewGorka Eguileor proposed openstack/cinder master: Report shared_targets for NVMe volumes  https://review.opendev.org/c/openstack/cinder/+/83607010:04
opendevreviewGorka Eguileor proposed openstack/cinder master: LVM-nvmet: Use nvmetcli as library instead of CLI  https://review.opendev.org/c/openstack/cinder/+/83607110:04
opendevreviewGorka Eguileor proposed openstack/cinder master: nvmeof: Support new connection properties  https://review.opendev.org/c/openstack/cinder/+/83607210:04
opendevreviewGorka Eguileor proposed openstack/cinder master: LVM: terminate_connection fails if no initiator  https://review.opendev.org/c/openstack/cinder/+/83607310:04
opendevreviewGorka Eguileor proposed openstack/cinder master: LVM nvmet: Add support for shared subsystems  https://review.opendev.org/c/openstack/cinder/+/83607410:04
opendevreviewRajat Dhasmana proposed openstack/cinder master: Update docstring for copy_image_to_volume method  https://review.opendev.org/c/openstack/cinder/+/83676910:05
sean-k-mooneywhoami-rajat: added on line 16010:08
whoami-rajatsean-k-mooney, thanks! I've added your topic for friday EOD10:11
opendevreviewGorka Eguileor proposed openstack/cinder master: Join do_setup and check_for_setup_error  https://review.opendev.org/c/openstack/cinder/+/81565910:11
opendevreviewGorka Eguileor proposed openstack/cinder master: Docs: Document clone_image driver method  https://review.opendev.org/c/openstack/cinder/+/81566010:11
sean-k-mooneyperfect10:14
*** dviroel|out is now known as dviroel11:21
opendevreviewRajat Dhasmana proposed openstack/cinder master: Update docstring for copy_image_to_volume method  https://review.opendev.org/c/openstack/cinder/+/83676911:50
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add missing copy_image_to_encrypted_volume in interface  https://review.opendev.org/c/openstack/cinder/+/83680411:59
opendevreviewWalt proposed openstack/cinder master: [Ceph-iscsi]  Disable multiattach  https://review.opendev.org/c/openstack/cinder/+/82755412:07
rubasovhi Cinder folks, I just added a topic proposal to the ptg etherpad, line 24512:25
rubasovcould you please let me know if you have time to fit this topic into the ptg? and if yes, which day?12:26
whoami-rajatrubasov, hi, which openstack release did you use for the testing?12:30
rubasovhi, downstream colleagues used stable/victoria, but I was able to reproduce in a master devstack12:30
whoami-rajatack, can you also specify the exact error you're facing?12:32
whoami-rajata error trace would be helpful12:32
rubasovshall I add it to https://bugs.launchpad.net/cinder/+bug/1940466?12:32
whoami-rajatrubasov, is it the same error? 12:33
whoami-rajatERROR cinder.volume.api [req-41b9c765-827d-4c75-ac95-331c8fa0c840 cd15c27229234675ac78f3f83af1c60c 3a2aec7e5c5b46c2a366e471fa133066 - 3f2f1264e29e4cc9913aa2a63197866a 3f2f1264e29e4cc9913aa2a63197866a] Volume status must be available or in-use to reserve, but the status is attaching.12:33
rubasovI believe it has the same root cause, though my reproduction is a bit different12:34
whoami-rajatok, i think I've implemented a solution for this12:34
whoami-rajatare you using single or multi stores on glance side?12:34
rubasovI use cinder with multiattach if that's what you mean12:35
whoami-rajatI mean in your glance-api.conf file, how many stores are you configuring? I understand one store is cinder but are there other stores?12:35
rubasovstores = cinder,file,http and default_store = cinder12:37
rubasov(the devstack defaults with cinder)12:37
whoami-rajatok12:37
whoami-rajatand are these in [glance_store] group?12:37
rubasovyes12:37
whoami-rajatok so it's single store,12:37
whoami-rajati guess you have a multiattach volume type already created?12:38
rubasovyes12:38
whoami-rajatok12:38
whoami-rajatjust add this line in the same glance_store group12:38
whoami-rajatcinder_volume_type=<your multiattach type name>12:38
whoami-rajatrestart glance service (devstack@g-api -- if devstack)12:39
whoami-rajatand the issue should resolve (hopefully)12:40
rubasovthat sounds great12:40
rubasovI'm giving it a try and will come back with the outcome12:40
rubasovthanks a lot12:41
whoami-rajatrubasov, but you need to test it in master since i added that feature in Xena12:41
whoami-rajatnp, let me know about the results and I can work on further issues (if you face any)12:41
rubasovokay, will fetch the latest master12:43
opendevreviewRajat Dhasmana proposed openstack/cinder master: Update docstring for copy_volume_to_image  https://review.opendev.org/c/openstack/cinder/+/83682212:51
rubasovwhoami-rajat: I pulled latest master from cinder, glance and glance_store, added cinder_volume_type=lvmdriver-1-multiattach to glance-api.conf in section [glance_store]12:56
jungleboyjwhoami-rajat: rosmaita  I will need to join a little late this morning.12:56
rubasovthen I'm testing with manually clearing the image from glance cache and starting two "openstack image save"s in parallel12:57
whoami-rajatjungleboyj, sure np12:57
rosmaitajungleboyj: acl12:57
jungleboyjThanks!12:57
jungleboyjDo you guys have the recordings from yesterday?12:57
whoami-rajatjungleboyj, yes, I've added the link in the etherpad L#69-70 https://etherpad.opendev.org/p/zed-ptg-cinder12:58
rubasovout of the two image saves one exits with: Unable to download image: InvalidResponse12:58
rubasovin the meanwhile in glance logs I see: cinderclient.exceptions.BadRequest: Invalid volume: Volume e5ad5acd-c20f-4c9e-a6c5-0771af3056ac status must be available or in-use or downloading to reserve, but the current status is reserved. (HTTP 400)12:58
whoami-rajatrubasov, hmm, i see that as a potential issue my code may not be covering, can you try with simultaneous image create operations?12:59
whoami-rajatCourtesy reminder for PTG day 2 jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval13:00
rubasovsure, will try and get back13:00
hemnaare we still on bluejeans?13:01
whoami-rajatmeeting link https://bluejeans.com/55668129013:01
whoami-rajathemna, yes13:01
whoami-rajatrubasov, ack13:01
hemnathanks13:01
whoami-rajatetherpad link https://etherpad.opendev.org/p/zed-ptg-cinder13:03
opendevreviewEric Harney proposed openstack/cinder master: mypy: cinder/volume/volume_types.py  https://review.opendev.org/c/openstack/cinder/+/83588113:08
rubasovwhoami-rajat: don't want to disturb you during the ptg, but leaving it here for the record: two parallel image creates worked (openstack volume create --type lvmdriver-1-multiattach --size 1 --image cirros-0.5.2-x86_64-disk volume0)13:17
jungleboyjwhoami-rajat: Awesome.  Thank you.  I will get those uploaded.13:20
whoami-rajatjungleboyj, thanks!13:20
whoami-rajatrubasov, the command should be openstack image create right? the command you mentioned creates a bootable volume from image13:22
opendevreviewPete Zaitcev proposed openstack/cinder master: Retry _get_ig() when Initiator Group already exists.  https://review.opendev.org/c/openstack/cinder/+/81713513:35
do3melihi there, i have a 15 line yoga backport that requires some approvals, might someone able to view https://review.opendev.org/c/openstack/cinder/+/835133? 13:36
rubasovwhoami-rajat: sorry for the misunderstanding, now I tested two parallel actual image creates and I got no error back and no errors in glance logs (openstack image create --public --disk-format qcow2 --file cirros-0.5.2-x86_64-disk.img image0)13:43
whoami-rajatrubasov, ok, was it failing before?13:44
jungleboyjwhoami-rajat: I don't see an option to download the recordings.  I normally pull them down and add them to our YouTube channel.13:45
whoami-rajatjungleboyj, ah, maybe i forgot to enable that option, checking...13:45
rubasovI did not test this before (only image saves), but I can comment out cinder_volume_type=lvmdriver-1-multiattach in glance-api.conf restart glance and test again, if that's what you mean13:45
jungleboyjThank you.13:46
whoami-rajatrubasov, no no, only if you had tested that before, as far as i can see, the image download should work as well, let's discuss this after the PTG if you're available at that time or tomorrow13:47
whoami-rajatjungleboyj, enabled it, can you try now?13:48
jungleboyjwhoami-rajat:  Thank you.  Looks good now!13:49
whoami-rajatjungleboyj, great, thanks13:49
rubasovwhoami-rajat: later today I have other things to attend, but I can contact you tomorrow, just let me know when13:51
whoami-rajatrubasov, sure, I'm UTC+5:30 and will be available since 10-11 AM my time till the PTG starts13:52
rubasovwhoami-rajat: okay, I will ping you then tomorrow13:53
rubasovI'm in utc+213:53
whoami-rajatack13:53
rubasovthanks a lot for your help!13:53
whoami-rajatnp, thanks for raising it13:54
*** dviroel is now known as dviroel|ptg14:13
geguileomkatari: courtesy ping for the Documenting the Driver Interface topic at 14:16
geguileohttps://bluejeans.com/55668129014:16
mkatariThanks Gorka, iam already in the meeting:)14:17
geguileomkatari: sorry, I didn't notice14:39
opendevreviewEric Harney proposed openstack/cinder master: Bump pylint to 2.13.4  https://review.opendev.org/c/openstack/cinder/+/83684215:25
opendevreviewGorka Eguileor proposed openstack/cinder master: Join do_setup and check_for_setup_error  https://review.opendev.org/c/openstack/cinder/+/81565915:28
opendevreviewGorka Eguileor proposed openstack/cinder master: Docs: Document clone_image driver method  https://review.opendev.org/c/openstack/cinder/+/81566015:28
*** dviroel|ptg is now known as dviroel|ptg|lunch15:42
opendevreviewEric Harney proposed openstack/cinder master: Tests: Add alembic to pylint ignores  https://review.opendev.org/c/openstack/cinder/+/83684815:49
opendevreviewEric Harney proposed openstack/cinder master: cmd/manage: Correct db_sync() return  https://review.opendev.org/c/openstack/cinder/+/83685015:56
*** dviroel|ptg|lunch is now known as dviroel|ptg16:03
jungleboyjhttps://usercontent.irccloud-cdn.com/file/m2AzjpBq/Cinder-Zed-PTG.png16:22
jungleboyjhttps://usercontent.irccloud-cdn.com/file/KRAMn1li/Cinder-Zed-PTG-2.png16:22
jungleboyjwhoami-rajat:  ^^ The screen shots that I got.  :-)16:22
whoami-rajatjungleboyj, cool, thanks!16:23
jungleboyjWelcome.16:23
opendevreviewEric Harney proposed openstack/cinder master: Remove privsep/hscli  https://review.opendev.org/c/openstack/cinder/+/83685916:30
geguileorosmaita: how is this generated? http://cinderstats.ivehearditbothways.com/cireport.txt16:58
rosmaitageguileo: smcginnis has some scripts16:58
* geguileo wonders where the source data is coming from r:-??16:59
geguileommmm, probably gerrit queries...16:59
eharneyIIRC it queries gerrit16:59
whoami-rajatjungleboyj, sharing links to the recording17:17
whoami-rajatFirst recording: https://bluejeans.com/s/xYqjCgPJ0p317:17
whoami-rajatSecond recording: https://bluejeans.com/s/xYqjCgPJ0p317:17
whoami-rajatFinal recording: https://bluejeans.com/s/xYqjCgPJ0p317:18
abishopwhoami-rajat: those urls are all the same17:18
whoami-rajatabishop, ah, noticed just now, i guess for a particular day (different chapters) the same link works17:19
whoami-rajatabishop, thanks for pointing it out17:19
*** dviroel|ptg is now known as dviroel17:22
*** dviroel is now known as dviroel|mtg17:47
geguileorosmaita: could you send me the quota survey results_18:27
geguileo?18:27
rosmaitasorry, i completely forgot!18:27
geguileonp18:27
geguileothanks to the TZ difference I have all morning of tomorrow to go trough it :-)18:28
rosmaitageguileo: we are up to 10 responses!18:28
geguileolol18:28
geguileocrazy numbers!!! ;-)18:28
smcginnisgeguileo: I just have a cron job that runs a slightly modified https://opendev.org/x/third-party-ci-tools18:36
rosmaitageguileo: https://rosmaita.wufoo.com/reports/cinder-resource-survey-202218:36
*** dviroel|mtg is now known as dviroel18:43
geguileosmcginnis: thanks!19:01
geguileorosmaita: that's not visible to the public19:01
*** dviroel is now known as dviroel|afk20:34

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