Thursday, 2023-01-26

opendevreviewFelipe Rodrigues proposed openstack/cinder master: NetApp: Add NVMe/TCP driver  https://review.opendev.org/c/openstack/cinder/+/87000400:29
*** JasonF is now known as JayF00:57
abdiwhoami-rajat: Updated https://review.opendev.org/c/openstack/cinder/+/815582 with a response to your question from the Cinder meeting.  Please advise, so we can take next step.01:46
opendevreviewGorka Eguileor proposed openstack/cinder-tempest-plugin master: WIP: zuul: Test LVM/nvmet additional job  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/87088909:18
zigorosmaita: Hi there!10:15
zigoI was wondering, are you going to make a backport of the CVE-2022-47951 patch to Rocky and Stein?10:15
zigoI'm having a hard time doing the backport to Rocky, which I would like to have for Debian 10 (which is LTS).10:16
zigoThinking about it, I did a bad job doing this backport, I'm tryiing again now ...10:41
zigoHum... I think I got a Rocky patch now! :)11:11
zigoOnly, test_mount_quobyte_should_reraise_already_mounted_error() is failing, but this really looks like unrelated.11:11
*** dviroel|out is now known as dviroel11:32
*** dviroel_ is now known as dviroel11:45
opendevreviewPeter Penchev proposed openstack/cinder master: Add the clone_across_pools driver capability  https://review.opendev.org/c/openstack/cinder/+/84713112:35
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: declare the clone_across_pools capability  https://review.opendev.org/c/openstack/cinder/+/84713212:35
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: drop _attach_volume() and _detach_volume()  https://review.opendev.org/c/openstack/cinder/+/78729612:35
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool driver: remove the obsolete backup_volume()  https://review.opendev.org/c/openstack/cinder/+/80382312:35
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: drop copy_image_to_volume() and copy_volume_to_image()  https://review.opendev.org/c/openstack/cinder/+/87055012:35
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: fix the "rename volume" unit test emulation  https://review.opendev.org/c/openstack/cinder/+/87068112:35
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: fix the retype volume flow  https://review.opendev.org/c/openstack/cinder/+/87055112:35
opendevreviewPeter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements  https://review.opendev.org/c/openstack/cinder/+/84327712:35
opendevreviewThomas Goirand proposed openstack/cinder stable/rocky: Check VMDK subformat against an allowed list  https://review.opendev.org/c/openstack/cinder/+/87181712:38
zigorosmaita: ^12:50
zigoThe Stein patch looks like nearly the same. I'm running unit tests to see if that works...12:51
rosmaitazigo: how far back do you plan to backport?13:13
opendevreviewMerged openstack/devstack-plugin-nfs master: Add zed and yoga jobs  https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/87107213:37
rosmaitazigo: left comments on your rocky patch13:42
opendevreviewBrian Rosmaita proposed openstack/cinder stable/wallaby: [stable-em only] move grenade jobs to experimental  https://review.opendev.org/c/openstack/cinder/+/87179014:12
opendevreviewEric Harney proposed openstack/python-cinderclient master: remove simplejson requirement  https://review.opendev.org/c/openstack/python-cinderclient/+/86898814:30
eharneyhttps://review.opendev.org/c/openstack/cinder/+/871284  is needed for our pylint job to run at all14:53
whoami-rajatgmann, hey, one question, any reason why we're releasing antelope early?16:08
opendevreviewPeter Penchev proposed openstack/cinder master: Add iSCSI export support to the StorPool driver  https://review.opendev.org/c/openstack/cinder/+/84753616:18
gmannwhoami-rajat: that is intermediate release to mark stable/wallaby end of support. for tempest plugin we can release final release for antelope at he end also16:42
whoami-rajatrosmaita, enriquetaso hey, can you take a look at this feature? I was skeptical to review it at first but it's pretty straightforward (as mentioned in my review), it's blocking the HPE XP driver merge https://review.opendev.org/c/openstack/cinder/+/84697716:42
opendevreviewGorka Eguileor proposed openstack/os-brick master: Support separate privsep logging levels  https://review.opendev.org/c/openstack/os-brick/+/87183516:43
geguileoeharney: ^ that's related to the conversation we had the other day16:44
geguileoeharney: about having different log levels for different privsep contexts16:44
eharneygeguileo: cool16:44
geguileoeharney: thank you for reminding me that I hadn't finished fixing that bug   lol16:44
whoami-rajatgmann, ah ok, looks good then, added my +116:46
gmannwhoami-rajat: thanks16:48
whoami-rajatgmann, np, thanks for releasing it16:48
rosmaitawhoami-rajat: ack about https://review.opendev.org/c/openstack/cinder/+/846977 , will look at it today16:51
whoami-rajatrosmaita, thanks!17:06
rosmaitawhoami-rajat: if you have a few minutes, see what you think of this change to the wallaby gate: https://review.opendev.org/c/openstack/cinder/+/87179017:25
whoami-rajatrosmaita, looking good, is the grenade job failing on wallaby?17:55
opendevreviewMerged openstack/python-cinderclient master: Move print operations to shell_utils  https://review.opendev.org/c/openstack/python-cinderclient/+/86330518:13
rosmaitawhoami-rajat: yes, and all earlier branches18:23
andrewbogott_rosmaita: anything blocking +w on https://review.opendev.org/c/openstack/cinder/+/861566 ?19:41
rosmaitaandrewbogott_: not from me, just needs another +219:42
andrewbogott_ok, I didn't know it needed two +2s19:42
andrewbogott_I will find others to bother!19:44
rosmaitayeah, except for doc-only changes, we require two +2s19:44
* andrewbogott_ nods19:44
spatelquick question what could be the issue if cinder-backup volume getting stuck in backing-up?  This is ceph backend and logs not showing any kind of indication or error20:26
andrewbogott_spatel: in my experience if a backup fails the api often reports it as backing up forever :(  I'd look at the cinder-backup logs for starters.20:33
spatelandrewbogott_ I am splunking all the logs but none saying any issue. 20:34
spatelvolume service is up 20:34
spatelThis is what i am getting in logs when i start backup - https://paste.opendev.org/show/bqwCrp30bCIUb5XCFwn7/20:35
spatelafter that no logs 20:35
andrewbogott_then I don't  have a good guess, sorry20:36
spatelThis is my cephx  - caps: [osd] allow class-read object_prefix rbd_children, allow rwx pool=backups20:36
spateldo i need to give permission to volume pool ?20:37
andrewbogott_I haven't used the ceph backend, only posix. I'd certainly hope that something would log a warning before just failing20:37
spatelbut if that is the case then i should get error message of it20:37
spatelI can turn on debug and see 20:37
spatelwhat you said posix means NFS correct?20:37
andrewbogott_not necessarily NFS, just backing up straight to a filesystem rather than rdb or object storage or whatever20:38
spatelCan we do that?20:38
spatelI can give it a try then let me see what plugin it use20:39
spatelhttps://docs.openstack.org/cinder/rocky/configuration/block-storage/backup/posix-backup-driver.html20:39
andrewbogott_Oh man, the posix backend does /not/ work well right now, I think ceph is the most widely used backend.20:39
andrewbogott_Because this is a dev channel you aren't likely to get a lot of good help with active deployments.  It's somewhat possible that #openstack-operators might be more help20:40
spatelTotally posix isn't solution for production but atleast i can see if it work or not20:40
spatelI can reachout to folks in operator 20:41
andrewbogott_yeah,  you could try posix just to see if the scheduler is getting called at all.20:42
spatelYes20:43
spatellet me give it a shot 20:43
spatelandrewbogott_ now i am getting error in logs - https://paste.opendev.org/show/bQD93N7m74NYshHOWJSv/20:55
andrewbogott_oh yeah, that looks like due to trying to delete the backup you made before switching to posix.  Try creating a fresh one from a different volume?20:56
spatelok20:57
*** dviroel is now known as dviroel|out20:58
spatelandrewbogott_ same behavior :( getting stuck into Creating...21:39
spatellook like something else going on. I am on yoga release 21:39
andrewbogott_hm, interesting, maybe a communication issue between the scheduler and the backup agent.21:40
andrewbogott_does 'openstack volume service list' show your cinder-backup services as 'up'?21:41
spatelYes its showing up 21:47
spatelThis is single box where all service running21:48
spatelandrewbogott_ https://paste.opendev.org/show/bWOJkrtlle4Ve9R7IMvn/21:48
andrewbogott_I'm out of ideas, sorry.  Poring over the debug logs is all I would know to do next.22:04
spatelandrewbogott_ DEBUG is also useless.. no sign of any indication. Anyway i will re-build environment and try again. 22:05
spatelI am running out of ideas now :)22:05
opendevreviewBrian Rosmaita proposed openstack/cinder stable/victoria: [stable-em only] move grenade jobs to experimental  https://review.opendev.org/c/openstack/cinder/+/87179122:15
opendevreviewBrian Rosmaita proposed openstack/cinder stable/ussuri: [stable-em only] move grenade jobs to experimental  https://review.opendev.org/c/openstack/cinder/+/87179222:15
opendevreviewBrian Rosmaita proposed openstack/cinder stable/train: [stable-em only] move grenade jobs to experimental  https://review.opendev.org/c/openstack/cinder/+/87179422:16

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