Monday, 2021-03-01

« Sunday, 2021-02-28 Index Tuesday, 2021-03-02 »
*** tosky has quit IRC00:11
*** zzzeek has quit IRC00:35
*** zzzeek has joined #openstack-cinder00:36
*** n-saito has joined #openstack-cinder00:51
*** zenkuro has quit IRC01:34
*** rcernin has quit IRC02:19
*** rcernin has joined #openstack-cinder02:35
*** brtknr has quit IRC03:00
*** brtknr has joined #openstack-cinder03:00
*** tinwood has quit IRC03:10
*** tinwood has joined #openstack-cinder03:13
*** akrpan-pure has joined #openstack-cinder03:34
*** psachin has joined #openstack-cinder03:44
*** Xuchu has joined #openstack-cinder03:58
*** rcernin has quit IRC04:14
*** rcernin has joined #openstack-cinder04:21
*** venkatakrishnath has joined #openstack-cinder04:30
*** akrpan-pure has quit IRC04:34
*** rcernin has quit IRC05:01
openstackgerritVenkata krishna Thumu proposed openstack/cinder master: [SVF] Set volume IOPS based on volume size  https://review.opendev.org/c/openstack/cinder/+/76480405:03
*** udesale has joined #openstack-cinder05:04
*** rcernin has joined #openstack-cinder05:08
*** akrpan-pure has joined #openstack-cinder05:20
*** m75abrams has joined #openstack-cinder05:25
*** akrpan-pure has quit IRC05:49
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/cinder/+/77682206:15
*** e0ne has joined #openstack-cinder06:49
*** e0ne has quit IRC06:49
*** e0ne has joined #openstack-cinder06:51
*** e0ne has quit IRC06:51
*** venkatakrishnath has quit IRC07:01
*** rcernin has quit IRC07:04
*** e0ne has joined #openstack-cinder07:07
*** e0ne has quit IRC07:11
*** whoami-rajat has joined #openstack-cinder07:11
*** GirishChilukuri has joined #openstack-cinder07:27
openstackgerritTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398507:34
*** brinzhang has joined #openstack-cinder07:37
*** rcernin has joined #openstack-cinder07:48
*** rcernin has quit IRC07:53
*** tobias-urdin has joined #openstack-cinder07:55
*** pcaruana has joined #openstack-cinder08:01
*** rpittau|afk is now known as rpittau08:04
*** tosky has joined #openstack-cinder08:20
*** rcernin has joined #openstack-cinder08:21
*** rcernin has quit IRC08:26
*** n-saito has quit IRC08:49
*** rcernin has joined #openstack-cinder08:55
*** ociuhandu has joined #openstack-cinder08:56
*** rcernin has quit IRC09:00
*** zoharm has joined #openstack-cinder09:07
*** ociuhandu has quit IRC09:10
*** rcernin has joined #openstack-cinder09:11
*** brinzhang_ has joined #openstack-cinder09:15
*** ociuhandu has joined #openstack-cinder09:15
*** rcernin has quit IRC09:16
*** brinzhang has quit IRC09:18
openstackgerritwushiming proposed openstack/cinder-tempest-plugin master: remove unicode from code  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/77794309:22
*** e0ne has joined #openstack-cinder09:37
*** e0ne has quit IRC09:37
*** martinkennelly has joined #openstack-cinder09:56
*** GirishChilukuri has quit IRC10:00
*** e0ne has joined #openstack-cinder10:10
*** e0ne has quit IRC10:10
*** dsariel has joined #openstack-cinder10:11
*** n-saito has joined #openstack-cinder10:13
*** jawad_axd has joined #openstack-cinder10:21
*** GirishChilukuri has joined #openstack-cinder10:25
jawad_axdHi folks, a quick question. Are encrypted cinder volumes supported with ceph rbd backend ?  Couldt find any update on this one. I am using stein/centos7.10:25
*** ociuhandu has quit IRC10:36
*** ociuhandu has joined #openstack-cinder10:38
*** manoj_katari has joined #openstack-cinder10:38
*** dsariel has quit IRC10:38
*** dsariel has joined #openstack-cinder10:39
*** ociuhandu has quit IRC10:43
*** rcernin has joined #openstack-cinder10:51
*** udesale_ has joined #openstack-cinder11:05
*** carloss has joined #openstack-cinder11:05
*** udesale has quit IRC11:08
*** ociuhandu has joined #openstack-cinder11:15
*** ociuhandu has quit IRC11:19
*** ociuhandu has joined #openstack-cinder11:31
*** rcernin has quit IRC11:31
*** rcernin has joined #openstack-cinder11:38
*** ociuhandu has quit IRC11:44
*** zenkuro has joined #openstack-cinder11:47
*** rcernin has quit IRC11:55
*** ociuhandu has joined #openstack-cinder11:56
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: WIP Revert "Skip a few c-bak tests broken by the new ceph"  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/77776611:56
*** rcernin has joined #openstack-cinder12:00
*** ociuhandu has quit IRC12:01
*** ociuhandu has joined #openstack-cinder12:12
*** yoctozepto has quit IRC12:17
*** yoctozepto has joined #openstack-cinder12:17
*** ociuhandu has quit IRC12:18
*** rcernin has quit IRC12:21
*** lpetrut has joined #openstack-cinder12:30
*** ociuhandu has joined #openstack-cinder12:45
*** ociuhandu_ has joined #openstack-cinder12:47
*** ociuhandu has quit IRC12:50
*** tbarron|out is now known as tbarron12:51
*** Luzi has joined #openstack-cinder12:55
*** m75abrams has quit IRC12:57
*** m75abrams has joined #openstack-cinder12:57
*** hoonetorg has joined #openstack-cinder13:03
*** rosmaita has quit IRC13:19
*** GirishChilukuri has quit IRC13:26
*** ociuhandu_ has quit IRC13:26
*** ociuhandu has joined #openstack-cinder13:27
*** ociuhandu has quit IRC13:27
*** ociuhandu has joined #openstack-cinder13:30
*** m75abrams has quit IRC13:34
*** eharney has joined #openstack-cinder13:35
*** ociuhandu has quit IRC13:35
openstackgerritTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398513:37
*** rosmaita has joined #openstack-cinder13:38
openstackgerritEric Harney proposed openstack/cinder master: mypy: annotate volume_utils / utils / exc  https://review.opendev.org/c/openstack/cinder/+/73685513:48
*** hemna has quit IRC13:48
*** hemna has joined #openstack-cinder13:49
openstackgerritEric Harney proposed openstack/cinder master: mypy: annotate api.py  https://review.opendev.org/c/openstack/cinder/+/73362213:50
*** m75abrams has joined #openstack-cinder13:51
*** psachin has quit IRC13:51
openstackgerritVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Update ceph default version to octopus  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/77723214:00
*** GirishChilukuri has joined #openstack-cinder14:03
*** GirishChilukuri has quit IRC14:17
carlossHello folks o/14:24
carlossCan you tell me if cinder supports NFS v4.2 or if you know someone that already tried to use a NFS driver using it?14:24
carlossAs per I saw in a note in this documentation: https://docs.openstack.org/cinder/latest/configuration/block-storage/drivers/nfs-volume-driver.html, cinder will always try to mount shares using v4.1. But I haven't seen anything telling this is not possible yet  :)14:25
jungleboyjcarloss: Hello.  I am not aware of it not working with v4.2 but eharney is probably the best resource here.14:26
carlossthanks jungleboyj :)14:27
openstackgerritBrian Rosmaita proposed openstack/rbd-iscsi-client master: Format as a Cinder-related OpenStack project  https://review.opendev.org/c/openstack/rbd-iscsi-client/+/77474814:27
whoami-rajatrosmaita: hey, i missed that we still have one patch remaining in victoria, can you take a look? https://review.opendev.org/c/openstack/os-brick/+/77709214:31
rosmaitawhoami-rajat: sure thing14:31
whoami-rajatthanks!14:32
rosmaitayeah, i think we want that in victoria!14:32
whoami-rajatyep, looks important14:32
rosmaitajungleboyj: if you have a minute, here's a backport to look at: https://review.opendev.org/c/openstack/os-brick/+/77709214:33
jungleboyjrosmaita: Looks good.  Thanks.14:34
whoami-rajatjungleboyj: rosmaita thanks for the quick merge, will propose victoria release patch after this merges14:35
jungleboyjwhoami-rajat:  Sounds good.  Thanks!14:35
*** Luzi has quit IRC14:44
toskytalking about backports, there is also https://review.opendev.org/c/openstack/os-brick/+/77740614:46
toskyif you want to ensure cinder-tempest-plugin is not going to break soon (after a refactoring in the not-yet public tempest API), please take a look at https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/77707414:47
jungleboyj+2 on the backport.14:47
jungleboyjAnd the fix for cinder-tempest-plugin14:52
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/62057714:53
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: DNM WIP Try to import cinder tests from barbican-tempest-plugin  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/74349214:54
*** m75abrams has quit IRC14:55
*** m75abrams has joined #openstack-cinder14:56
toskythanks!15:15
toskygates are kind of green, luckily the workaround to readd "member" to tempest-created users merged over the weekend15:15
*** __ministry1 has joined #openstack-cinder15:25
*** __ministry1 has quit IRC15:26
*** lpetrut has quit IRC15:29
*** jbernard_ is now known as jbernard15:31
*** eharney_ has joined #openstack-cinder15:45
*** eharney has quit IRC15:46
*** udesale_ has quit IRC15:48
*** zoharm has quit IRC15:50
*** m75abrams has quit IRC15:52
openstackgerritGorka Eguileor proposed openstack/cinder master: Remove unused quota python classes  https://review.opendev.org/c/openstack/cinder/+/77801715:55
*** eharney_ is now known as eharney15:58
*** frickler has joined #openstack-cinder16:17
fricklermicroversion 3.31 seems to break "cinder show volume_name", is that a known issue? using an id instead of the name still works16:18
*** venkatakrishnath has joined #openstack-cinder16:21
venkatakrishnathHi, Please someone review these patches(Already got one +2 )16:27
venkatakrishnathhttps://review.opendev.org/c/openstack/cinder/+/75019916:27
venkatakrishnathhttps://review.opendev.org/c/openstack/cinder/+/77156316:27
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - RDF checks  https://review.opendev.org/c/openstack/cinder/+/76810516:31
*** venkatakrishnath has quit IRC16:37
*** manoj_katari has quit IRC16:38
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: WIP Revert "Skip a few c-bak tests broken by the new ceph"  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/77776616:41
openstackgerritTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398516:47
*** jdillaman has joined #openstack-cinder16:55
openstackgerritMerged openstack/cinder-tempest-plugin master: Wait for CG to be available before creating volume in the CG  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/75466016:57
*** rpittau is now known as rpittau|afk17:00
*** e0ne has joined #openstack-cinder17:05
legochenhey team, I have a question about the settings “image_conversion_cpu_limit” and “image_conversion_address_space_limit” in /etc/cinder/cinder.conf17:14
legochenbecause I defined the variables/values in /etc/cinder/cinder.conf, but still encounter errors raised from this command - Command: /opt/openstack/venv/cinder/bin/python -m oslo_concurrency.prlimit --as=1073741824 --cpu=60 -- sudo cinder-rootwrap /etc/cinder/rootwrap.conf env LC_ALL=C qemu-img info /var/conversion/tmp3g9s6xe_test.com@block-sas17:15
legochenlooks like my defined values are not actually applied to the code. it is still using default values in code: https://github.com/openstack/cinder/blob/master/cinder/image/image_utils.py#L66-L7217:16
legochenrefer to this doc - https://access.redhat.com/solutions/280011117:17
legochenI manually to update the default values in the code  https://github.com/openstack/cinder/blob/master/cinder/image/image_utils.py#L66-L72 . I can see it works well, but if I define the variables in /etc/cinder/cinder.conf. it won’t work.17:18
legochenjungleboyj ^^ if you know this :)17:19
*** enriquetaso has joined #openstack-cinder17:25
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/62057717:30
*** MrClayPole_ has quit IRC17:53
*** MrClayPole has joined #openstack-cinder17:53
*** felipe_rodrigues has joined #openstack-cinder17:55
*** e0ne has quit IRC18:22
*** e0ne has joined #openstack-cinder18:22
*** e0ne has quit IRC18:22
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix old attach method  https://review.opendev.org/c/openstack/cinder/+/77804718:32
openstackgerritGorka Eguileor proposed openstack/cinder master: Remove unnecessary save call  https://review.opendev.org/c/openstack/cinder/+/77804818:32
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix volume OVO create method  https://review.opendev.org/c/openstack/cinder/+/77804918:32
rosmaitalegochen: forgive me for asking, but did you restart the service?18:41
legochenrosmaita, yes, I did that18:49
rosmaitalegochen: and you are expecting to see the new values only in the 'qemu-img info' call, not 'qemu-img convert', right?18:51
legochenright18:51
legochenuhh, sorry, I misunderstood.18:51
legochenbecaue the default values caused the failure from this command - Command: /opt/openstack/venv/cinder/bin/python -m oslo_concurrency.prlimit --as=1073741824 --cpu=60 -- sudo cinder-rootwrap /etc/cinder/rootwrap.conf env LC_ALL=C qemu-img info /var/conversion/tmp3g9s6xe_test.com@block-sas18:52
legochenSo, that’s why I was trying to configure “image_conversion_cpu_limit” and “image_conversion_address_space_limit” in /etc/cinder/cinder.conf18:52
legochenbut, after configured, seems like the values didn’t get effect.18:53
rosmaitaok, give me a few more minutes to check something18:53
legochenbut, if I directly to modify the code https://github.com/openstack/cinder/blob/master/cinder/image/image_utils.py#L66-L72 , I can see the new values applied.18:53
legochenthank you18:53
rosmaitalegochen: just want to verify that you restarted the cinder-volume service, not the api?18:57
legochenyes, I restarted all cinder-* services - scheduler, api, volume18:57
rosmaitaok, thanks18:57
legochenbtw, I tried to dump the value in the code - CONF.image_conversion_cpu_limit and CONF.image_conversion_address_space_limit , I can see the values are the same as what I configured in /etc/cinder/cinder.conf.18:59
legochenbut, looks like the values dont apply to QEMU_IMG_LIMITS = processutils.ProcessLimits(19:00
legochen    cpu_time=CONF.image_conversion_cpu_limit,19:00
legochen    address_space=CONF.image_conversion_address_space_limit * units.Gi)19:00
rosmaitalegochen: while you are experimenting, see what happens if you move lines 77-79 (definition of QEMU_IMG_LIMITS) down to after line 123 (beginning of qemu_img_info function) and see if it picks up the values from the config file19:02
legochenokay, let me try try19:03
rosmaitathanks!19:03
legochenrosmaita, you mean inside of qemu_img_info function or outside?19:06
rosmaitainside the function, right at the beginning19:07
legochenok19:07
*** stand has joined #openstack-cinder19:12
legochenrosmaita, bingo, it works fine after move lines.19:18
rosmaitaok, can you file a bug?  looks like that ProcessLimits object is being defined too soon, but i'm not sure off the top of my head where we should fix it19:19
*** hamalq has joined #openstack-cinder19:24
legochenrosmaita, https://bugs.launchpad.net/cinder/+bug/191735319:34
openstackLaunchpad bug 1917353 in Cinder "image_conversion_cpu_limit and image_conversion_address_space_limit settings cannot not work in /etc/cinder/cinder.conf" [Undecided,New]19:34
legochenthis is my first time to create bug, please have a look. thanks19:34
rosmaitalegochen: i will, thank you for filing it19:34
*** enriquetaso has quit IRC19:35
legochenthank you rosmaita. btw, if you have some time, I’d like to consult from you about the questions regarding boot instance from cinder volume.  maybe we could schedule a time for that. I can follow your time :)19:36
rosmaitaif you have time now, we can discuss here19:37
legochenmy company is going to adopt to use cinder block storage as boot volume of instance. this would be a first step to implement instance live migration.19:37
legochenOur architecture - glance images are stored on NetApp NFS volume19:38
legochencinder backend (block storage) adopt EMC VxFlex OS19:39
legochenPer my understanding to boot instance from volume, the steps are: 1) create a bootable image volume 2) use this bootable volume to create instance.19:40
*** dsariel has quit IRC19:40
*** dsariel has joined #openstack-cinder19:40
legochenBut, I feel there is a concern is about disk loading on the control plane.19:42
legochenbecause this process needs to copy image from glance to image_conversion_dir, mount cinder volume on control plane node for creating image volume from this temp copied image19:42
legochenthen use this image volume to create an instance.19:42
legochenI’d like to know what’s the better and lighter way to perform this.19:43
legochenrosmaita ^ :)19:43
rosmaitalegochen: ok, give me a few min to read through19:44
legochensure, thanks19:44
legochenI thought there is an option to let KVM can configure “cinder volume” as vda in instance’s kvm xml. And boot instance from “image” as the normal process as how we create an instance on hypervisor local store.19:48
*** dsariel has quit IRC19:58
rosmaitalegochen: i'm not sure about the KVM option, you will have to check with the nova docs about that20:00
*** jawad_axd has quit IRC20:01
rosmaitalegochen: there is a image_volume_cache in cinder where the glance image can be stored as volume in the backend, and then any requests to create volumes from that image will use the cache, so will be very fast (after the first one)20:02
rosmaitathat's all i can think of at the moment20:03
rosmaitayou might want to ask on the mailing list for suggestions20:04
rosmaitafrom other operators20:04
legochenrosmaita, cool, do you know where is the image_volume_cache path?20:04
*** e0ne has joined #openstack-cinder20:05
rosmaitalegochen: here are the docs on it: https://docs.openstack.org/cinder/latest/admin/blockstorage-image-volume-cache.html20:06
*** jawad_axd has joined #openstack-cinder20:07
rosmaitato see the actual code path, probably look in cinder/volume/flows/manager/create_volume.py and in cinder/volume/manager.py20:07
legochenokay, thanks for the guidance. I just wanted to make sure the cache path has enough disk space to do cache.20:08
legochenif we could specify the image_volume_cache, then even better.20:09
*** jawad_axd has quit IRC20:10
*** dsariel has joined #openstack-cinder20:25
legochenrosmaita, if our glance and cinder are running on the same host, and both of them are able to access to the image source dir. Do you think it is possible to ignore image cache/copy/download behaviors?20:28
legochenbtw, thanks for your information. I’ll investigate mroe about that.20:29
*** mgagne has joined #openstack-cinder20:41
*** e0ne has quit IRC21:30
*** rcernin has joined #openstack-cinder21:33
*** rcernin has quit IRC21:54
*** rcernin has joined #openstack-cinder21:55
*** rcernin has quit IRC21:55
*** rcernin has joined #openstack-cinder21:55
*** e0ne has joined #openstack-cinder22:03
*** whoami-rajat has quit IRC22:07
*** dsariel has quit IRC22:10
openstackgerritEric Harney proposed openstack/cinder master: zuul: add mypy experimental job  https://review.opendev.org/c/openstack/cinder/+/73685722:14
openstackgerritLee Yarwood proposed openstack/devstack-plugin-ceph master: Added missing libvirt keyring for Cinder  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/69499722:21
openstackgerritzhaoleilc proposed openstack/cinder master: Get rid of renaming error logs under special context  https://review.opendev.org/c/openstack/cinder/+/77808522:33
*** enriquetaso has joined #openstack-cinder22:36
*** rcernin has quit IRC22:39
*** e0ne has quit IRC22:40
*** rcernin has joined #openstack-cinder22:42
hemnaanyone use xonsh ?  python command line shell.   just checking it out.22:43
« Sunday, 2021-02-28 Index Tuesday, 2021-03-02 »

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