*** henriqueof has quit IRC | 00:01 | |
*** henriqueof has joined #openstack-cinder | 00:02 | |
*** sapd1 has quit IRC | 00:16 | |
*** hoonetorg has quit IRC | 00:24 | |
*** hoonetorg has joined #openstack-cinder | 00:27 | |
*** brinzhang has joined #openstack-cinder | 00:31 | |
*** dviroel_ has quit IRC | 00:37 | |
*** markvoelker has joined #openstack-cinder | 00:49 | |
*** itlinux has joined #openstack-cinder | 00:57 | |
*** mriedem has quit IRC | 01:00 | |
*** whoami-rajat has joined #openstack-cinder | 01:09 | |
*** mmethot has joined #openstack-cinder | 01:15 | |
*** markvoelker has quit IRC | 01:22 | |
*** sdake has quit IRC | 01:29 | |
*** itlinux_ has joined #openstack-cinder | 01:38 | |
*** itlinux_ has quit IRC | 01:40 | |
*** itlinux has quit IRC | 01:41 | |
*** sdake has joined #openstack-cinder | 01:45 | |
*** gnufied has quit IRC | 01:53 | |
*** sdake has quit IRC | 02:05 | |
*** markvoelker has joined #openstack-cinder | 02:19 | |
*** sdake has joined #openstack-cinder | 02:28 | |
*** sdake has quit IRC | 02:42 | |
*** psachin has joined #openstack-cinder | 02:42 | |
*** markvoelker has quit IRC | 02:52 | |
*** sdake has joined #openstack-cinder | 03:20 | |
*** sdake has quit IRC | 03:23 | |
*** sdake_ has joined #openstack-cinder | 03:24 | |
*** _alastor_ has quit IRC | 03:39 | |
*** zhubx has quit IRC | 03:46 | |
*** zhubx has joined #openstack-cinder | 03:47 | |
*** markvoelker has joined #openstack-cinder | 03:49 | |
*** zhubx has quit IRC | 03:57 | |
*** sdake_ has quit IRC | 03:58 | |
*** zhubx has joined #openstack-cinder | 03:58 | |
*** markvoelker has quit IRC | 04:22 | |
*** dave-mccowan has quit IRC | 05:03 | |
*** vishakha has joined #openstack-cinder | 05:11 | |
*** bhagyashris_ has joined #openstack-cinder | 05:11 | |
*** ianychoi_ has joined #openstack-cinder | 05:24 | |
*** ianychoi has quit IRC | 05:28 | |
*** sdake has joined #openstack-cinder | 05:31 | |
*** sdake has quit IRC | 05:31 | |
*** hoonetorg has quit IRC | 05:33 | |
*** sdake has joined #openstack-cinder | 05:38 | |
*** bhagyashris_ has quit IRC | 05:43 | |
*** hoonetorg has joined #openstack-cinder | 05:50 | |
*** udesale has joined #openstack-cinder | 06:07 | |
*** gkadam has quit IRC | 06:13 | |
*** e0ne has joined #openstack-cinder | 06:31 | |
*** v12aml has quit IRC | 06:32 | |
*** v12aml has joined #openstack-cinder | 06:39 | |
*** markvoelker has joined #openstack-cinder | 06:49 | |
*** Luzi has joined #openstack-cinder | 06:53 | |
*** artem_vasilyev has joined #openstack-cinder | 06:58 | |
*** markvoelker has quit IRC | 07:21 | |
*** ondrejme has joined #openstack-cinder | 07:35 | |
ondrejme | How could I remove block storage backend from cinder and also from database in Kolla? | 07:36 |
---|---|---|
*** bhagyashris has joined #openstack-cinder | 07:38 | |
whoami-rajat | ondrejme: you can remove the entry of backend from cinder.conf and restart the cinder services (also delete the associated volume type). For kolla database you can directly go to database (mysql) and remove the entry but that's not recommended. you can ask for the same in kolla channel. | 07:41 |
openstackgerrit | Abhishek Kekane proposed openstack/cinder-specs master: Support multiple backend of Glance https://review.openstack.org/641267 | 07:46 |
*** e0ne has quit IRC | 07:47 | |
ondrejme | whoami-rajat: so the only thing needed is to delete volumes, volume type and restart cinder services? | 07:49 |
ondrejme | whoami-rajat: no need to work with database? | 07:49 |
*** e0ne has joined #openstack-cinder | 07:54 | |
*** avishay has joined #openstack-cinder | 08:07 | |
*** tkajinam has quit IRC | 08:09 | |
avishay | Hi all, in our tests (running Ocata) we are seeing CleanableInUse errors sporadically in create_volume and create_snapshot | 08:13 |
avishay | I saw a few bugs such as this one: https://bugs.launchpad.net/cinder/+bug/1712272 | 08:13 |
openstack | Launchpad bug 1712272 in Cinder "cinder get worker failed " [Undecided,Invalid] - Assigned to wlfightup (wlfightup) | 08:13 |
*** gkadam has joined #openstack-cinder | 08:14 | |
avishay | @geguileo wrote there that it is caused by an old version of pymysql, but we are running 0.7.9 and I verified that it contains the referenced fix | 08:14 |
avishay | (Running one instance of cinder-volume) | 08:15 |
*** e0ne has quit IRC | 08:16 | |
*** markvoelker has joined #openstack-cinder | 08:19 | |
openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Always run chmod privileged in remotefs drivers https://review.openstack.org/641277 | 08:19 |
*** pcaruana has joined #openstack-cinder | 08:29 | |
*** markvoelker has quit IRC | 08:52 | |
whoami-rajat | ondrejme: no, the important part is to delete the entry in cinder.conf | 08:54 |
whoami-rajat | something like this | 08:54 |
whoami-rajat | [lvmdriver-1] | 08:54 |
whoami-rajat | volume_group=cinder-volumes-1 | 08:54 |
whoami-rajat | volume_driver=cinder.volume.drivers.lvm.LVMVolumeDriver | 08:54 |
whoami-rajat | volume_backend_name=LVM | 08:54 |
whoami-rajat | ondrejme: deleting volumes and volume types related to that backend will automatically wipe out entries in database. | 08:54 |
*** dpawlik has joined #openstack-cinder | 08:57 | |
*** dpawlik has quit IRC | 09:17 | |
*** lemko has joined #openstack-cinder | 09:27 | |
*** markvoelker has joined #openstack-cinder | 09:49 | |
*** bhagyashris has quit IRC | 09:54 | |
*** luizbag has joined #openstack-cinder | 10:06 | |
*** markvoelker has quit IRC | 10:22 | |
*** e0ne has joined #openstack-cinder | 10:35 | |
openstackgerrit | Rajat Dhasmana proposed openstack/python-cinderclient master: Correct the 'get resource' behavior for non-admin user https://review.openstack.org/641310 | 10:37 |
*** artem_vasilyev has quit IRC | 10:41 | |
*** dviroel_ has joined #openstack-cinder | 10:43 | |
*** udesale has quit IRC | 10:57 | |
*** zhubx has quit IRC | 10:59 | |
*** zhubx has joined #openstack-cinder | 10:59 | |
*** thgcorrea has joined #openstack-cinder | 11:00 | |
*** FlorianFa has quit IRC | 11:01 | |
*** FlorianFa has joined #openstack-cinder | 11:02 | |
*** pcaruana has quit IRC | 11:04 | |
*** erlon has joined #openstack-cinder | 11:09 | |
*** erlon has quit IRC | 11:16 | |
*** erlon has joined #openstack-cinder | 11:17 | |
*** markvoelker has joined #openstack-cinder | 11:19 | |
*** ianychoi_ is now known as ianychoi | 11:20 | |
*** pcaruana has joined #openstack-cinder | 11:32 | |
*** dave-mccowan has joined #openstack-cinder | 11:45 | |
*** markvoelker has quit IRC | 11:52 | |
*** erlon has quit IRC | 12:03 | |
*** erlon has joined #openstack-cinder | 12:04 | |
*** carlos_silva has joined #openstack-cinder | 12:08 | |
*** enriquetaso has joined #openstack-cinder | 12:09 | |
*** mvkr has joined #openstack-cinder | 12:13 | |
*** henriqueof has quit IRC | 12:16 | |
*** brinzhang has quit IRC | 12:31 | |
*** udesale has joined #openstack-cinder | 12:38 | |
*** markvoelker has joined #openstack-cinder | 12:49 | |
*** avishay has quit IRC | 12:52 | |
*** mchlumsky has joined #openstack-cinder | 13:02 | |
*** raghavendrat has joined #openstack-cinder | 13:03 | |
*** TheJulia_sick is now known as TheJulia | 13:05 | |
raghavendrat | https://review.openstack.org/#/c/634119/ I have replied to reviewers comments. Not sure if I was able to convince with my explanation. Waiting for response from core reviewers so that we can logically conclude. If any core reviewer is available, appreciate if you can look and respond. | 13:12 |
*** lemko has quit IRC | 13:15 | |
*** markvoelker has quit IRC | 13:22 | |
hemna | mornin | 13:25 |
openstackgerrit | sahid proposed openstack/os-brick master: multipathd: fix support for find_multipaths set to yes https://review.openstack.org/638639 | 13:28 |
openstackgerrit | sahid proposed openstack/os-brick master: multipath: add debug messages of connection threads https://review.openstack.org/641350 | 13:28 |
*** mriedem has joined #openstack-cinder | 13:34 | |
*** arne_wiebalck has quit IRC | 13:41 | |
*** arne_wiebalck_ has joined #openstack-cinder | 13:42 | |
whoami-rajat | raghavendrat: Hey, I think eharney and hemna had queries regarding your patch. It would be better if you ask them. | 13:43 |
openstackgerrit | Merged openstack/cinder stable/queens: Fix multiattach set to false after retype https://review.openstack.org/641068 | 13:45 |
raghavendrat | Thanks Rajat. I am waiting for eharney and hemna. | 13:48 |
hemna | sup | 13:48 |
*** pcaruana has quit IRC | 13:51 | |
*** eharney has joined #openstack-cinder | 13:51 | |
*** sdake has quit IRC | 13:54 | |
whoami-rajat | raghavendrat: you can discuss it here too :) | 13:56 |
*** sdake has joined #openstack-cinder | 14:00 | |
*** pcaruana has joined #openstack-cinder | 14:01 | |
*** FlorianFa has quit IRC | 14:09 | |
*** gkadam_ has joined #openstack-cinder | 14:16 | |
*** sdake has quit IRC | 14:17 | |
*** gkadam has quit IRC | 14:19 | |
*** markvoelker has joined #openstack-cinder | 14:19 | |
*** sdake has joined #openstack-cinder | 14:21 | |
*** sdake has quit IRC | 14:24 | |
openstackgerrit | jacky06 proposed openstack/python-cinderclient master: Update hacking version https://review.openstack.org/641395 | 14:35 |
*** arne_wiebalck_ is now known as arne_wiebalck | 14:35 | |
*** arne_wiebalck has quit IRC | 14:35 | |
*** arne_wiebalck_ has joined #openstack-cinder | 14:37 | |
*** sdake has joined #openstack-cinder | 14:38 | |
*** markvoelker has quit IRC | 14:52 | |
*** gkadam__ has joined #openstack-cinder | 15:00 | |
*** sdake has quit IRC | 15:00 | |
*** gkadam_ has quit IRC | 15:03 | |
*** sdake has joined #openstack-cinder | 15:06 | |
*** lseki has joined #openstack-cinder | 15:06 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: Revert "Use native python truncate for privsep" https://review.openstack.org/641413 | 15:09 |
openstackgerrit | Eric Harney proposed openstack/cinder master: Revert "Remove truncate from rootwrap filters" https://review.openstack.org/641414 | 15:09 |
*** Luzi has quit IRC | 15:09 | |
*** gnufied has joined #openstack-cinder | 15:27 | |
*** udesale has quit IRC | 15:37 | |
openstackgerrit | Matthew Booth proposed openstack/cinder master: DNM: Getting extra logs for a volume type delete race https://review.openstack.org/641425 | 15:50 |
*** lpetrut has joined #openstack-cinder | 15:50 | |
*** markvoelker has joined #openstack-cinder | 15:50 | |
*** pcaruana has quit IRC | 15:53 | |
*** lseki has quit IRC | 16:00 | |
*** erlon has quit IRC | 16:00 | |
*** luizbag has quit IRC | 16:00 | |
*** thgcorrea has quit IRC | 16:01 | |
*** luizbag has joined #openstack-cinder | 16:01 | |
*** erlon has joined #openstack-cinder | 16:03 | |
*** pcaruana has joined #openstack-cinder | 16:06 | |
*** sdake has quit IRC | 16:09 | |
*** sdake has joined #openstack-cinder | 16:13 | |
*** _alastor_ has joined #openstack-cinder | 16:17 | |
*** markvoelker has quit IRC | 16:22 | |
*** arne_wiebalck_ has quit IRC | 16:32 | |
*** arne_wiebalck_ has joined #openstack-cinder | 16:32 | |
*** imacdonn has quit IRC | 16:36 | |
*** imacdonn has joined #openstack-cinder | 16:36 | |
*** gkadam__ has quit IRC | 16:42 | |
*** sdake has quit IRC | 16:48 | |
*** sdake has joined #openstack-cinder | 16:48 | |
*** pcaruana has quit IRC | 16:55 | |
*** sdake has quit IRC | 16:56 | |
*** arne_wiebalck_ has quit IRC | 16:56 | |
hemna | so whoami-rajat fwiw, we have a customer that's running into problems with the 3PAR driver related to standing up a lot of VMs (100) and creating volumes from snaps | 17:02 |
hemna | because by default the 3PAR driver creates clones for every volume from snap operation | 17:02 |
hemna | which is why I hated that idea from the beginning | 17:03 |
*** henriqueof has joined #openstack-cinder | 17:03 | |
hemna | it's crazy fast to create a vol from snap in 3PAR | 17:03 |
hemna | but cloning is very slow | 17:03 |
hemna | the problem with vols from snaps is that you can't delete the snap, while those volumes exist (dependency) | 17:03 |
hemna | but I never understood why that was a problem from cinder's perspective | 17:04 |
whoami-rajat | hemna: ok. | 17:05 |
hemna | and the main usecase is to create a lot of new vms at once | 17:05 |
hemna | which means vols from snaps | 17:05 |
hemna | and doing a convert to base on every vol from snap is....s l o w | 17:05 |
hemna | then you run into ssh connection issues | 17:06 |
hemna | and lots of stuff stops working | 17:06 |
*** eharney has quit IRC | 17:06 | |
hemna | so, that's why I think the default in that new patch should be set to False. | 17:06 |
hemna | meaning don't convert to base by default. | 17:06 |
hemna | and if you REALLY want to have a base volume, then you can create a new vol type and set that new attribute in there. | 17:07 |
hemna | our customer wants to create 100 140G volumes from snaps | 17:08 |
hemna | that's a lot of bits to copy (for convert_to_base_volume) | 17:08 |
hemna | it looks like eharney disagrees though | 17:09 |
*** angela-s has joined #openstack-cinder | 17:10 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method https://review.openstack.org/635255 | 17:10 |
whoami-rajat | hemna: so if we create a volume type with convert_to_base as False and use it instead to create volumes. what problems do we face in this case? | 17:13 |
*** eharney has joined #openstack-cinder | 17:13 | |
angela-s | smcginnis, jungleboyj: Hi Sean, Jay, could you take a look at this cherry pick for Pike? Thanks! https://review.openstack.org/#/c/640568/ | 17:13 |
*** lpetrut has quit IRC | 17:14 | |
*** lpetrut has joined #openstack-cinder | 17:17 | |
hemna | the problem with not creating a base volume, is one of dependency | 17:17 |
hemna | you can't delete the snapshot | 17:18 |
hemna | until all of the volumes from that snap have either been deleted or made into a base volume. | 17:18 |
hemna | IMHO I think that's a better tradeoff | 17:18 |
hemna | as then you can get the 3PAR to act more cloud like. | 17:18 |
hemna | such as being able to create 100 volumes very quickly | 17:19 |
*** markvoelker has joined #openstack-cinder | 17:19 | |
hemna | this is how I originally wrote the driver, because I knew of the 3PAR limitations for creating cloned volumes. | 17:20 |
hemna | but this patch changed all that, in an attempt to make rally happy during delete. | 17:21 |
hemna | https://github.com/openstack/cinder/commit/5f2074e1092f07c3420877fa00d1c49efde6e420 | 17:21 |
whoami-rajat | hemna: ohh | 17:22 |
*** ntpttr has quit IRC | 17:22 | |
*** lseki has joined #openstack-cinder | 17:22 | |
whoami-rajat | hemna: i thought the initial functionality while implementing the driver was to convert to base volume | 17:23 |
hemna | but that patch causes all sorts of problems with rally if you try and create 100 volumes of any size | 17:23 |
whoami-rajat | so i remember raghavendrat argued to keep support of the current functionality that wasn't intended at all. | 17:24 |
hemna | originally the 3PAR driver didn't clone the volume during create from snap. | 17:24 |
hemna | because of all the stuff I mentioned above | 17:25 |
hemna | it creates a rw snapshot | 17:25 |
hemna | which is basically just a metadata change on the parent snap | 17:25 |
hemna | crazy fast | 17:25 |
hemna | and it would only convert to a base volume if the new volume was larger than the original snap | 17:25 |
hemna | https://github.com/openstack/cinder/blob/liberty-eol/cinder/volume/drivers/san/hp/hp_3par_common.py#L1684 | 17:26 |
whoami-rajat | hemna: makes sense. now i understand your concerns regarding the patch. | 17:27 |
hemna | unforunately I have a lot of history with the 3PAR :P | 17:27 |
whoami-rajat | hemna: probably the author is just trying to keep the current (but not intended) functionality. | 17:28 |
hemna | yah | 17:28 |
hemna | I think so | 17:28 |
whoami-rajat | hemna: yes, a lot | 17:29 |
whoami-rajat | hemna: so, the fast solution coming into my mind is | 17:29 |
whoami-rajat | hemna: since the patch is just adding a new extra spec, and your concern is a little bit different from the point of patch, we merge this and then change the default to false in the next PS. | 17:30 |
hemna | PS ? | 17:30 |
whoami-rajat | hemna: or add a comment about all the things you explained currently for the usecase and set the option to false as default with that comment. | 17:31 |
whoami-rajat | hemna: oh sorry, followup. | 17:31 |
hemna | we have a customer that can make use of that patch today :P | 17:31 |
hemna | I just wanted the default to be changed, such that large volume creations work out of the box | 17:31 |
hemna | as they used to. | 17:32 |
*** lpetrut has quit IRC | 17:32 | |
*** e0ne has quit IRC | 17:34 | |
whoami-rajat | hemna: yes, but the change that shouldn't have merged and this patch brought up the same Case again. Most probably we can summarize the benefit for setting the convert_to_base to false as default in a comment, and add the following to current patch. does that seem like the right approach? | 17:35 |
hemna | either way there should be some documentation associated with this patch to explain this | 17:36 |
hemna | and the cost benefits of both True and False for customers. | 17:37 |
hemna | I have to go get some lunch..bbiab | 17:37 |
whoami-rajat | hemna: is this worth discussing why the cloning is slower compared to other drivers? | 17:37 |
whoami-rajat | hemna: ok see ya! Thanks for your insights. | 17:37 |
hemna | well, that's a 3PAR specific comparison vs. other backends | 17:38 |
hemna | 3par just is very slow in cloning | 17:38 |
whoami-rajat | hemna: ok, better not to dig more into that. | 17:40 |
*** mmethot has quit IRC | 17:44 | |
*** mmethot has joined #openstack-cinder | 17:45 | |
*** markvoelker has quit IRC | 17:52 | |
*** psachin has quit IRC | 18:05 | |
*** irclogbot_0 has joined #openstack-cinder | 18:11 | |
whoami-rajat | hemna: I'm really not sure why rally jobs were failing, it is initially deleting the volumes then snapshot[1] so the dependency shouldn't cause problem, do we need any wait_for_resource_delete func to make these operations synchronous, not sure. the bug doesn't seem like a bug now, sometime it may or may not fail IMO. | 18:20 |
whoami-rajat | Also i found your review regarding the patch[2], looking at the rally job makes me think we can revert this change. | 18:20 |
whoami-rajat | [1] https://github.com/openstack/rally-openstack/blob/master/rally_openstack/scenarios/cinder/volumes.py#L834-L835 | 18:20 |
whoami-rajat | [2] https://review.openstack.org/#/c/421739/3/cinder/volume/drivers/hpe/hpe_3par_common.py@2215 | 18:20 |
*** luizbag_ has joined #openstack-cinder | 18:21 | |
*** luizbag__ has joined #openstack-cinder | 18:25 | |
whoami-rajat | hemna: and if we are going forward with reverting this change, we can easily set the value default=False for convert_to_base. | 18:27 |
*** luizbag_ has quit IRC | 18:27 | |
*** luizbag has quit IRC | 18:27 | |
*** irclogbot_0 has quit IRC | 18:36 | |
openstackgerrit | Merged openstack/cinder master: Update minimal acceptable glanceclient https://review.openstack.org/641037 | 18:46 |
openstackgerrit | Merged openstack/cinder master: Update required version of taskflow https://review.openstack.org/641065 | 18:46 |
*** markvoelker has joined #openstack-cinder | 18:50 | |
*** enriquetaso has quit IRC | 18:54 | |
*** markvoelker has quit IRC | 19:22 | |
*** zhubx has quit IRC | 19:59 | |
*** zhubx007 has joined #openstack-cinder | 19:59 | |
*** luizbag__ has quit IRC | 20:01 | |
*** e0ne has joined #openstack-cinder | 20:03 | |
*** arne_wiebalck has joined #openstack-cinder | 20:06 | |
openstackgerrit | Merged openstack/cinder stable/pike: Fix for auth version change in Brcd HTTP https://review.openstack.org/640568 | 20:06 |
*** markvoelker has joined #openstack-cinder | 20:19 | |
*** e0ne has quit IRC | 20:21 | |
*** lemko has joined #openstack-cinder | 20:29 | |
*** vishakha has quit IRC | 20:35 | |
hemna | did we get a bug filed about the filter issue? | 20:51 |
*** markvoelker has quit IRC | 20:53 | |
smcginnis | Not that I've seen. | 20:57 |
smcginnis | There may be an existing one for a very specific side effect of all of that, but not the actual real issue. | 20:57 |
*** erlon has quit IRC | 21:05 | |
*** irclogbot_0 has joined #openstack-cinder | 21:08 | |
*** e0ne has joined #openstack-cinder | 21:13 | |
*** whoami-rajat has quit IRC | 21:18 | |
*** lseki has quit IRC | 21:21 | |
*** irclogbot_0 has quit IRC | 21:28 | |
*** eharney has quit IRC | 21:29 | |
angela-s | smcginnis, jungleboyj, hemna: thanks guys, Pike is done. Now how about Ocata, man is this tedious. https://review.openstack.org/#/c/640567/ Thank you! | 21:36 |
*** markvoelker has joined #openstack-cinder | 21:50 | |
*** mchlumsky has quit IRC | 21:53 | |
*** angela-s has quit IRC | 21:56 | |
*** henriqueof has quit IRC | 21:57 | |
*** e0ne has quit IRC | 21:58 | |
*** e0ne has joined #openstack-cinder | 22:01 | |
*** e0ne has quit IRC | 22:01 | |
*** carlos_silva has quit IRC | 22:07 | |
*** bal has quit IRC | 22:11 | |
*** irclogbot_0 has joined #openstack-cinder | 22:20 | |
*** markvoelker has quit IRC | 22:22 | |
*** irclogbot_0 has quit IRC | 22:34 | |
*** irclogbot_0 has joined #openstack-cinder | 22:36 | |
*** lemko has quit IRC | 22:38 | |
*** irclogbot_0 has quit IRC | 22:44 | |
*** rcernin has joined #openstack-cinder | 22:52 | |
*** tkajinam has joined #openstack-cinder | 23:01 | |
*** dave-mccowan has quit IRC | 23:06 | |
*** markvoelker has joined #openstack-cinder | 23:19 | |
*** markvoelker has quit IRC | 23:53 | |
*** gnufied has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!