Tuesday, 2017-07-11

*** bkopilov has joined #openstack-cinder00:00
*** jessegler has quit IRC00:04
*** wxy- has joined #openstack-cinder00:07
*** Apoorva_ has quit IRC00:07
*** Apoorva has joined #openstack-cinder00:08
*** wxy- has quit IRC00:11
*** Apoorva has quit IRC00:12
*** gcb has quit IRC00:21
*** stakeda has joined #openstack-cinder00:27
*** edmondsw has joined #openstack-cinder00:49
*** edmondsw has quit IRC00:53
*** wxy- has joined #openstack-cinder00:54
*** bkopilov has quit IRC00:54
openstackgerritzheng yin proposed openstack/cinder master: Ceph backup: eventlet.green.subprocess replace subprocess  https://review.openstack.org/46660700:56
*** bkopilov has joined #openstack-cinder00:57
*** wxy- has quit IRC00:58
*** zzzeek_ has quit IRC01:09
*** zzzeek_ has joined #openstack-cinder01:11
*** wanghao_ has joined #openstack-cinder01:11
*** wanghao has quit IRC01:15
*** bkopilov has quit IRC01:16
*** bkopilov has joined #openstack-cinder01:16
*** sdague has quit IRC01:17
*** Apoorva has joined #openstack-cinder01:18
*** wanghao has joined #openstack-cinder01:20
*** wanghao_ has quit IRC01:23
openstackgerritliuyamin proposed openstack/cinder master: Use QoSConsumerValues enum field  https://review.openstack.org/47967401:26
*** wanghao_ has joined #openstack-cinder01:28
*** wanghao has quit IRC01:31
*** zzzeek_ has quit IRC01:33
*** zzzeek_ has joined #openstack-cinder01:34
*** zzzeek_ has quit IRC01:36
*** Shunli has joined #openstack-cinder01:41
*** zzzeek_ has joined #openstack-cinder01:47
*** Apoorva has quit IRC01:49
*** gkadam has joined #openstack-cinder01:49
*** Apoorva has joined #openstack-cinder01:49
*** abhitechie has joined #openstack-cinder01:52
*** Apoorva has quit IRC01:54
*** bkopilov has quit IRC01:59
*** zhaochao has joined #openstack-cinder02:00
*** gcb has joined #openstack-cinder02:03
*** zhurong has joined #openstack-cinder02:03
openstackgerritTommyLike proposed openstack/cinder master: Enable backend specific async migration  https://review.openstack.org/33492302:15
jgriffithmriedem sorry, I didn't see your follow up02:21
jgriffithmriedem so yes, my point was that as it stands we should probably add the reserved status to the expected list02:21
jgriffithotherwise there are windows of failure02:21
jgriffithmriedem if we're not comfortable with doing that that's fine as well, but we need to comment the code and make sure it's obvious in the message02:22
jgriffithI would hope that people wouldn't be extending without some thought, but I guess you never know02:22
jgriffithThe existing exception mesg is probably good enough, but would be helpful to have some more detail.02:23
jgriffithI'll update the patch, but I'm curious what your thoughts are around allowing extend on reserved?02:23
jgriffithI'm not sure there's a real risk to it?02:23
jgriffithI'll remove my -1 and +2 when you and Mathieu let me know which behavior we want and we document it.02:27
jgriffithmgagne ^^02:34
*** edmondsw has joined #openstack-cinder02:37
*** edmondsw has quit IRC02:42
*** abhitechie has quit IRC02:57
*** wanghao_ has quit IRC02:58
openstackgerritLiu Qing proposed openstack/cinder master: Fix SSHPool current_size not work correctly  https://review.openstack.org/46863402:58
*** wanghao has joined #openstack-cinder02:59
openstackgerritBin Zhou proposed openstack/cinder master: Support utf-8 decode  https://review.openstack.org/48238903:03
openstackgerritBin Zhou proposed openstack/cinder master: Support utf-8 decode  https://review.openstack.org/48238903:07
openstackgerritBin Zhou proposed openstack/cinder master: Support utf-8 decode  https://review.openstack.org/48238903:09
*** bkopilov has joined #openstack-cinder03:18
*** abhitechie has joined #openstack-cinder03:21
*** diablo_rojo has quit IRC03:25
*** wanghao has quit IRC03:27
*** wanghao has joined #openstack-cinder03:28
*** abhitechie has quit IRC03:35
*** abhitechie has joined #openstack-cinder03:36
*** links has joined #openstack-cinder03:41
*** abhitechie has quit IRC03:46
*** jmlowe has joined #openstack-cinder03:58
*** psachin has joined #openstack-cinder04:00
*** dave-mccowan has quit IRC04:00
*** isaacb has joined #openstack-cinder04:08
*** isaacb has quit IRC04:10
*** abhitechie has joined #openstack-cinder04:15
openstackgerritkongxiangyun proposed openstack/cinder master: add exception check for lvm_type  https://review.openstack.org/48145404:18
*** edmondsw has joined #openstack-cinder04:25
*** edmondsw has quit IRC04:30
*** abhitechie has quit IRC04:30
*** abhitechie has joined #openstack-cinder04:30
*** arnewiebalck_ has joined #openstack-cinder04:33
*** Julien-zte has quit IRC04:41
*** Julien-zte has joined #openstack-cinder04:42
*** markstur has quit IRC04:55
openstackgerritiswarya vakati proposed openstack/cinder master: Stop using deprecated 'message' attribute in Exception  https://review.openstack.org/48212504:57
*** abhitechie has quit IRC04:57
*** abhinavtechie has joined #openstack-cinder04:57
*** abhitechie has joined #openstack-cinder05:28
*** abhinavtechie has quit IRC05:28
*** abhitechie has quit IRC05:32
*** abhitechie has joined #openstack-cinder05:34
*** Shunli has quit IRC05:39
*** Shunli has joined #openstack-cinder05:40
*** rcernin has joined #openstack-cinder05:41
*** lpetrut has joined #openstack-cinder05:41
*** arnewiebalck_ has quit IRC05:49
*** nicolasbock has joined #openstack-cinder05:55
*** gkadam has quit IRC05:56
*** gkadam has joined #openstack-cinder05:58
*** pcaruana has joined #openstack-cinder06:04
*** ChubYann has quit IRC06:07
*** abhitechie has quit IRC06:09
*** zhonghua has quit IRC06:11
*** zhonghua has joined #openstack-cinder06:12
*** edmondsw has joined #openstack-cinder06:14
*** abhitechie has joined #openstack-cinder06:15
*** gkadam has quit IRC06:17
*** gkadam has joined #openstack-cinder06:17
*** edmondsw has quit IRC06:18
*** gkadam has quit IRC06:18
*** gkadam has joined #openstack-cinder06:19
*** gkadam has quit IRC06:21
*** jmlowe has quit IRC06:25
*** laurent\ has quit IRC06:35
*** laurent\ has joined #openstack-cinder06:36
*** abhitechie has quit IRC06:36
*** abhitechie has joined #openstack-cinder06:36
*** lpetrut has quit IRC06:39
*** zhurong has quit IRC06:40
*** gkadam has joined #openstack-cinder06:44
*** zhurong has joined #openstack-cinder06:44
*** arnewiebalck_ has joined #openstack-cinder06:45
*** belmoreira has joined #openstack-cinder06:45
*** gkadam has quit IRC06:46
*** lhx__ has joined #openstack-cinder06:46
*** arnewiebalck_ has quit IRC06:46
*** Poornima has joined #openstack-cinder06:47
*** arnewiebalck_ has joined #openstack-cinder06:49
*** Poornima has quit IRC06:49
*** lhx__ has quit IRC06:50
*** lhx__ has joined #openstack-cinder06:50
*** tesseract has joined #openstack-cinder07:02
*** lkuchlan has joined #openstack-cinder07:09
*** zhurong has quit IRC07:30
*** mriedem has quit IRC07:30
*** openstackgerrit has quit IRC07:33
*** openstackgerrit has joined #openstack-cinder07:42
openstackgerritkongxiangyun proposed openstack/cinder master: add exception check for lvm_type  https://review.openstack.org/48145407:42
*** aarefiev_pto is now known as aarefiev07:44
*** markvoelker_ has quit IRC07:47
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** openstackgerrit has quit IRC08:03
*** abhitechie has quit IRC08:12
*** openstackgerrit has joined #openstack-cinder08:17
openstackgerritArnon Yaari proposed openstack/cinder master: INFINIDAT: Add QoS support  https://review.openstack.org/48213208:17
*** Nishant_ has joined #openstack-cinder08:18
*** e0ne has joined #openstack-cinder08:19
*** abhitechie has joined #openstack-cinder08:23
*** isaacb has joined #openstack-cinder08:23
*** gkadam has joined #openstack-cinder08:25
*** mvk has quit IRC08:26
*** gkadam is now known as gkadam-afk08:26
*** gkadam-afk has quit IRC08:26
*** zhurong has joined #openstack-cinder08:27
*** mvk has joined #openstack-cinder08:55
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - seamless upgrade from SMI-S to REST  https://review.openstack.org/48213809:00
*** lpetrut has joined #openstack-cinder09:04
*** pgadiya has joined #openstack-cinder09:13
*** arnewiebalck_ has quit IRC09:15
*** zhurong has quit IRC09:27
*** Shunli has quit IRC09:29
*** e0ne has quit IRC09:31
*** e0ne has joined #openstack-cinder09:34
*** mvk has quit IRC09:37
*** links has quit IRC09:37
*** links has joined #openstack-cinder09:39
*** links has quit IRC09:44
*** dalgaaf has quit IRC09:47
*** dalgaaf has joined #openstack-cinder09:47
*** markvoelker has joined #openstack-cinder09:48
*** links has joined #openstack-cinder09:49
*** edmondsw has joined #openstack-cinder09:50
*** mvk has joined #openstack-cinder09:51
*** edmondsw has quit IRC09:54
*** kongwei has joined #openstack-cinder09:58
*** kongwei has quit IRC10:00
openstackgerritTommyLike proposed openstack/cinder master: [WIP] Implement policies in code, reviews are welcome  https://review.openstack.org/48249210:03
*** wxy- has joined #openstack-cinder10:04
*** sdague has joined #openstack-cinder10:05
*** wanghao has quit IRC10:12
*** wanghao has joined #openstack-cinder10:13
*** gcb has quit IRC10:13
*** links has quit IRC10:13
*** wanghao has quit IRC10:17
*** wanghao has joined #openstack-cinder10:17
*** markvoelker has quit IRC10:22
*** wxy-_ has joined #openstack-cinder10:29
*** wxy- has quit IRC10:29
*** gouthamr has quit IRC10:33
openstackgerritYu Shengzuo proposed openstack/cinder master: Fix no feature to enable 'exclusive-lock' of image in ceph  https://review.openstack.org/48147210:38
*** gouthamr has joined #openstack-cinder10:41
kaisersHey! Anyone with exp in encrypted volumes and a spare moment might help with taking a look at the questions in this bug: https://bugs.launchpad.net/tempest/+bug/1702843 , please10:58
openstackLaunchpad bug 1702843 in tempest "test_boot_server_from_encrypted_volume_luks has insufficient skip decorator" [Undecided,New]10:58
*** gkadam has joined #openstack-cinder11:00
*** dave-mccowan has joined #openstack-cinder11:10
*** wxy- has joined #openstack-cinder11:10
*** bkopilov has quit IRC11:11
*** Julien-zte has quit IRC11:12
*** wxy- has quit IRC11:13
*** Julien-zte has joined #openstack-cinder11:13
*** wxy-_ has quit IRC11:14
*** markvoelker has joined #openstack-cinder11:20
*** gkadam has quit IRC11:24
*** edmondsw has joined #openstack-cinder11:38
*** wanghao has quit IRC11:38
*** abishop has joined #openstack-cinder11:40
*** edmondsw has quit IRC11:42
*** links has joined #openstack-cinder11:46
openstackgerritVivek Soni proposed openstack/cinder master: 3PAR: Add volume to consistency/generic group  https://review.openstack.org/48025511:47
*** markvoelker has quit IRC11:53
*** links has quit IRC11:59
*** gkadam has joined #openstack-cinder12:05
*** Nishant_ has quit IRC12:07
*** felipemonteiro_ has joined #openstack-cinder12:08
*** felipemonteiro__ has joined #openstack-cinder12:09
*** dustins has joined #openstack-cinder12:09
*** felipemonteiro_ has quit IRC12:12
*** liverpooler has joined #openstack-cinder12:18
*** markvoelker has joined #openstack-cinder12:19
*** MVenesio has joined #openstack-cinder12:32
*** bkopilov has joined #openstack-cinder12:33
*** Poornima has joined #openstack-cinder12:35
*** abhitechie has quit IRC12:38
*** mlakat has quit IRC12:46
*** edmondsw has joined #openstack-cinder12:46
*** pgadiya has quit IRC12:47
*** openstackgerrit has quit IRC12:47
*** gkadam_ has joined #openstack-cinder12:51
*** lhx__ has quit IRC12:52
*** gkadam has quit IRC12:54
*** chhavi has joined #openstack-cinder13:03
*** gkadam__ has joined #openstack-cinder13:04
arnewiebalckWe have just upgraded our Cinder deployment here at CERN to Ocata.13:04
arnewiebalckThe c-vol for one of our Ceph pools did not want to start.13:05
*** openstackgerrit has joined #openstack-cinder13:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements  https://review.openstack.org/47807113:06
*** gkadam_ has quit IRC13:06
*** lkuchlan has quit IRC13:07
arnewiebalckThe problem was the gathering of the provisioning stats.13:07
*** gkadam__ has quit IRC13:11
arnewiebalckFrom what I see, the code loops over all volumes in a periodic task every 60 secs.13:11
arnewiebalckWith more than 4’000 volumes in the pool, this takes too long and it basically enters an infinite loop querying all volumes.13:12
arnewiebalckWe have now taken out this line: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L41513:15
Dinesh_Bhorarnewiebalck: I think this is related https://bugs.launchpad.net/cinder/+bug/168137413:15
openstackLaunchpad bug 1681374 in Cinder "cinder-volume can't be inited with large number of volume" [Medium,Confirmed]13:15
arnewiebalckDinesh_Bhor: I think this is different.13:16
arnewiebalckDinesh_Bhor: Thanks for the pointer, btw ;)13:16
Dinesh_Bhorarnewiebalck: ok, hope someone will be answer13:18
arnewiebalckarnewiebalck: The loop is also triggered on deletion, it seems, not only on startup.13:18
openstackgerritRawan Herzallah proposed openstack/cinder master: Adding NVMEoF target for LVM driver  https://review.openstack.org/48256213:21
smcginnisSwanson: ^13:22
arnewiebalckIn addition to the fact this does not scale, I struggle to understand the underlying idea.13:24
arnewiebalckFrom what I see, Cinder is asking Ceph for the allocated sizes of all volumes. Cinder knows this already, no?13:24
smcginnisarnewiebalck: I wonder if that's the actual space used, not the logical space configured.13:25
arnewiebalcke0ne: ^13:26
smcginnisarnewiebalck: It was added as part of the thin provisioning work: https://github.com/openstack/cinder/commit/d4fd5660736a1363a4e78480b116532c71b5ce4913:26
arnewiebalcksmcginnis: not it’s not13:26
arnewiebalcksmcginnis: I checked experimentally (and had our Ceph expert check the code)13:26
arnewiebalcksmcginnis: I’m certainly missing sth.13:26
openstackgerritChaynika Saikia proposed openstack/python-cinderclient master: Add cinder create --poll  https://review.openstack.org/47591913:26
e0nearnewiebalck:  what issue are you talking about?13:27
smcginnisarnewiebalck: Quite possible the logic may be wrong.13:27
*** josecastroleon has joined #openstack-cinder13:27
arnewiebalcke0ne: the one I described ^13:27
*** lhx_ has joined #openstack-cinder13:29
*** gcb has joined #openstack-cinder13:32
*** Poornima has quit IRC13:34
*** wxy- has joined #openstack-cinder13:34
arnewiebalcksmcginnis: my “no, it’s not” referred to the actual/allocated space; you’re correct about the commit, we’ve found this as well13:38
smcginnisarnewiebalck: Hah, that's how I took it. ;)13:39
*** wxy- has quit IRC13:39
e0nearnewiebalck:  here is PoC how to make it faster13:39
arnewiebalcksmcginnis: In the corresponding review, there were dicusssions about actual vs allocated space, but I didn’t get the idea in the end.13:40
e0nearnewiebalck, smcginnis: sorry, forgot to paste the link: https://review.openstack.org/#/c/476079/13:46
arnewiebalcke0ne: :-D13:48
arnewiebalcke0ne: this also changes from allocated space to used space?13:48
e0nearnewiebalck:  yes13:48
*** eharney has joined #openstack-cinder13:48
arnewiebalcke0ne: so, the current logic is wrong?13:49
e0nebut jbernard has some valid concerns on it13:49
e0nearnewiebalck:  I'm not sure, TBH13:49
*** abhitechie has joined #openstack-cinder13:49
arnewiebalcke0ne: yes, his point 1) is what I just asked13:50
e0nearnewiebalck: we can discuss it tomorrow at the meeting13:50
arnewiebalcke0ne: sure13:51
*** mlakat has joined #openstack-cinder13:52
arnewiebalcke0ne: our local Ceph expert promised to check if there is a way to address Jon’s point 2)13:53
e0nearnewiebalck: I didn't find any API in current ceph/rbd/rados python bindings13:54
e0nearnewiebalck: I hope, you have a better luck with it13:54
arnewiebalcke0ne: he thinks “no”, but wanted to double check, let’s see :)13:55
arnewiebalcksmcginnis: ok with you if we put this on the agenda for tomorrow?13:57
*** jistr is now known as jistr|call13:58
smcginnisarnewiebalck: Yes, absolutely.14:02
arnewiebalcksmcginnis: done14:02
_pewp_hemna (ه’́⌣’̀ه )/14:03
smcginnishemna: guten morgen14:13
hemnagetting up to speed on the new gig14:14
smcginnishemna: I had been wondering how that was going. Glad to see the news.14:14
hemnaIt's a relief14:14
smcginnishemna: I bet14:15
eharneyhemna: glad to hear!14:15
e0nehemna: evening:)14:15
*** dfisher has joined #openstack-cinder14:16
hemnaat least I'll still be doing OpenStack for sure now too14:16
*** zzzeek_ has quit IRC14:18
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST  https://review.openstack.org/46602714:18
ildikovhemna: awesome news! :)14:19
hemnathank you :)14:20
scottdaCongrats on your escape hemna :)14:20
*** zzzeek_ has joined #openstack-cinder14:21
hemna:)  thanks scottda14:21
smcginnisWe'll have to all drive up and kidnap scottda and bring him to Rosa Mexicana for some Manberries.14:22
hemnaooh yah14:22
scottdaHa. Where's Rosa Mexicana? Denver?14:23
scottdaFor the PTG: http://www.richardsandoval.com/tamayo/14:23
smcginnisscottda: Isn't that the place in Ft Collins we went to where kmartin got the drink?14:24
scottdaNo, that was The Rio. There is one in downtown Denver BTW14:25
dfisherThe Rio is horrible.14:25
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Live Migration, replacing SMI-S with REST  https://review.openstack.org/47227914:25
smcginnisRio, rosa, whatever. :)14:25
smcginnisdfisher: Hah!14:25
dfishermargaritas hit like trucks14:25
dfisherbut that's about all14:25
dfishercajun food in Denver:  Bayou Bob's.  *amazing*14:25
dfisherhttp://www.bayoubobs.com/ — slow internet connection but the food is outstanding.14:26
scottdasmcginnis: What is up with https://review.openstack.org/#/c/471792/?14:26
scottdaDoes it really take 48+ hours to get through the merge queue?14:27
smcginnisscottda: One of the dependent patches hasn't merged yet.14:27
scottdaOr did Jenkins have to reboot or something, and it got lost?14:27
scottdaahh......silly me.14:27
*** xyang1 has joined #openstack-cinder14:27
smcginnisscottda: And I've got a -1 on it from someone on the tempest team because I decided not to rewrite half of tempest to fix this specific issue.14:27
scottdasmcginnis: Yeah, I think that's called "The Tempest Blues".14:28
*** xyang has joined #openstack-cinder14:28
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX - Live Migration, replacing SMI-S with REST  https://review.openstack.org/47227914:28
smcginnisscottda: Not really sure at this point if I should start down that path of redoing everything, or just wait to see if another tempest core is willing to let it pass.14:28
jgriffithdfisher you don't go to Bayou Bob's to surf the net or work, you go for the cajun goodness, my god man, the CAJUN GOODNESS!14:28
smcginnisSounds like we have something on our evening agenda already. :)14:29
dfishertryin' to help whoever's an out-of-towner to not go to the Rio.14:29
dfisherseriously, just go chug cuervo instead.14:29
smcginnisdfisher: Don't put it past this group.14:29
jgriffithsmcginnis +114:29
dfisheryou all work on *cinder*14:29
dfisheri'd be stunned if you weren't hammered right now.14:30
dfisherat least it's not Neutron ...14:30
smcginnisThat's the truth.14:30
smcginnisjgriffith: Hey, wrt reserved state in that patch...14:31
smcginnisjgriffith: If it's reserved, that's some kind of indication that there's some work in progress, right. So if we allow extend with reserve, does that open up a race window where something starts with the old size, it gets extended mid-process, and it completes with stale information.14:32
smcginnisjgriffith: Not sure if that's even a possibility or something to be concerned about.14:32
ildikovsmcginnis: in the current Nova implementation, which is still under review, you have the volume reserved in the regular attach scenarios, which can open the race window you refer to14:35
*** gheorghe_ has quit IRC14:35
ildikovsmcginnis: I have no idea what happens if we do attach and extend at the same time, and I'm not sure I would want to try, but that's just my taste :)14:35
smcginnisildikov: That's what I was thinking of. But will it cause problems if the volume ends up being bigger by the time it is all done than when it started off? That part I'm not sure of.14:35
ildikovsmcginnis: the other scenario when you'll find the volume in reserved state is shelved offloaded14:36
smcginnisildikov: Yep.14:36
smcginnisMaybe it's not a big deal, but probably something we should think about.14:36
ildikovsmcginnis: in case of swap_volume that can be nasty I think14:36
ildikovas that operation specifically checks the volume size to do that operation14:37
*** rcernin has quit IRC14:38
ildikovsmcginnis: I don't know what happens if a shelved offloaded server is put back to operation and gets back a bigger volume, that might not be an issue, I don't really know that part14:38
ildikovsmcginnis: in case of swap I would think the volume has a state change in the danger zone, but when it comes to race conditions, I'm not sure whether or not I would depend on that14:39
ildikovsmcginnis: my 2 cents :)14:39
smcginnisildikov: Probably an extremely unlikely case anyway.14:39
ildikovsmcginnis: and that's what guarantees you that it'll happen in production... :)14:40
ildikovsmcginnis: this or something very similar14:40
*** rcernin has joined #openstack-cinder14:40
ildikovsmcginnis: does the reserved state have to be included now or it can be a decision later as well to add?14:42
dfisherquick question on image-to-volume:  If my Glance image is gzip'd, does qemu-img decompress it when streaming the image to the volume?14:43
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - implement Generic Volume Group feature  https://review.openstack.org/47454614:43
smcginnisdfisher: Yes, it has to be downloaded locally, then uncompressed.14:45
* dfisher nods14:45
smcginnisildikov: I'd actually rather wait, but the problem with that is I think we would need to bump the microversion if we changed it afterwards.14:45
jgriffithsmcginnis ildikov just add it if you ask me; I just wanted to get mgagne 's input first14:46
smcginnisjgriffith: Any thoughts on race conditions there?14:47
jgriffithwe could always spend some time and "look" at the nova code and make sure we do a fresh volume-get when finalizing the attach14:47
jgriffithwhich I believe we do now14:47
*** openstackgerrit has quit IRC14:48
jgriffithI'm not as concerned about races, I raised it in the review as a potential concern, but I'm not too excited about it14:48
smcginnisjgriffith: It does seem like (assuming it could be a problem) it would be a very small window and very unlikely to ever happen.14:48
jgriffithpersonally I don't care which way it goes, just want it to be excplicit and documented one way or the other14:49
smcginnisBut like ildikov said, that almost guarantees it would.14:49
smcginnisjgriffith: +114:49
jgriffithsmcginnis ildikov haha, yeah, well there is that!14:49
jgriffithmriedemann responded to my comments and prefers NOT to add it, so that's fine by me14:50
smcginnisjgriffith: Doesn't seem like we lose any major usability by not allowing it.14:51
jgriffithYeah, honestly it's a corner case, not supported, get over it and move along :)14:52
jgriffiththat's me responding to a bug in LP :)14:52
jgriffithfinsh the attach, then extend it14:52
jgriffithit's so easy to deal with it's almost a non-issue IMO.  I just want to make sure we state clearly in the api doc and the cli help that you're not allowed to do it14:53
ildikovsmcginnis: jgriffith: I just felt someone needs to represent the worrying side14:55
ildikovsmcginnis: jgriffith: it feels better to have the volume in a stable state, which I would not consider 'reserved' to be14:55
ildikovsmcginnis: jgriffith: but we don't code based on feelings so you might as well ignore me :)14:56
*** jmlowe has joined #openstack-cinder14:56
jgriffithildikov yeah, and the name "reserved" that implies you shouldn't be screwing with it anyway; somebody "reserve" the right to do that later :)14:56
ildikovjgriffith: exactly :)14:57
*** isaacb has quit IRC14:59
*** Julien-zte has quit IRC15:03
*** markstur has joined #openstack-cinder15:06
*** diablo_rojo has joined #openstack-cinder15:08
*** zzzeek_ has quit IRC15:12
lpetruthi, I have two small bug fixes targeting the Windows drivers. Could anyone please take a look? Our CI is passing. https://review.openstack.org/#/c/464659/ https://review.openstack.org/#/c/462902/15:12
*** abhitechie has quit IRC15:16
*** abhitechie has joined #openstack-cinder15:17
*** psachin has quit IRC15:18
lpetrutthat was fast, thanks :)15:18
smcginnislpetrut: :)15:18
jungleboyjAsk and ye shall receive.15:18
*** rcernin has quit IRC15:18
smcginnislpetrut: I had actually reviewed the update on the first one but I think at the time I was just waiting for the Windows CI results. Never got back to it.15:18
*** jistr|call is now known as jistr15:22
*** isaacb has joined #openstack-cinder15:23
jungleboyjsmcginnis: Morning sir.15:24
xyangjungleboyj: I have a question regarding the doc15:25
smcginnisjungleboyj: Hey15:25
xyangjungleboyj: what happened to config reference?15:25
jungleboyjxyang:  Sure.15:25
jungleboyjxyang:  Is it gone?15:25
xyangjungleboyj: yes15:26
xyangjungleboyj: no longer in openstack-manuals15:26
xyangbut I don't see it under cinder repo either15:26
jungleboyjxyang:  Hmmm, ok, let me look here.15:27
*** abhinavtechie has joined #openstack-cinder15:27
xyangjungleboyj: ok, thanks15:27
jungleboyjsmcginnis: I have a related question on docs give me a minute to figure out what happened to the config reference.15:27
*** catintheroof has joined #openstack-cinder15:28
smcginnisjungleboyj: Doesn't that need to get merged and the redirect put in place?15:28
smcginnisI thought most of the docs.o.o links were broken for now until everything gets moved over.15:28
jungleboyjsmcginnis:  Yuck, the config-reference wasn't in the list of things we needed to move over.15:29
smcginnisRuh roh15:29
jungleboyjsmcginnis:  But it was removed.15:29
jungleboyjLet me check with Doug but it looks like I will need to migrate that as well.15:30
jungleboyjxyang:  ^^^15:30
*** zzzeek_ has joined #openstack-cinder15:30
*** abhitechie has quit IRC15:30
*** e0ne has quit IRC15:34
xyangjungleboyj: thanks:)15:35
*** belmoreira has quit IRC15:38
jungleboyjxyang:  It is now supposed to be autogenerated.  Doug pointed me to an example.  I will try to do the same for Cinder here today.15:40
xyangjungleboyj: Great.  thanks15:42
jungleboyjxyang:  Thanks for catching it.  Let me know as you find other things.15:43
*** abhinavtechie has quit IRC15:44
*** liverpooler has quit IRC15:44
*** lpetrut has quit IRC15:56
*** aarefiev is now known as aarefiev_afk15:56
*** jmlowe has quit IRC15:57
*** abishop_ has joined #openstack-cinder15:58
*** abishop has quit IRC15:59
*** eharney has quit IRC16:00
*** jmlowe has joined #openstack-cinder16:00
*** markstur has quit IRC16:01
*** markstur has joined #openstack-cinder16:02
*** Apoorva has joined #openstack-cinder16:08
*** openstackgerrit has joined #openstack-cinder16:10
openstackgerritYu Shengzuo proposed openstack/cinder master: Fix no feature to enable 'exclusive-lock' of image in ceph  https://review.openstack.org/48147216:10
openstackgerritApoorva Deshpande proposed openstack/cinder master: Tintri: Fix config option access  https://review.openstack.org/48227816:11
*** mriedem has joined #openstack-cinder16:11
*** isaacb has quit IRC16:16
*** edmondsw has quit IRC16:17
*** lhx_ has quit IRC16:17
*** edmondsw_ has joined #openstack-cinder16:19
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - implement Generic Volume Group feature  https://review.openstack.org/47454616:21
*** jmlowe has quit IRC16:22
*** edmondsw_ has quit IRC16:23
*** liverpooler has joined #openstack-cinder16:24
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - bootable volume errors  https://review.openstack.org/47267416:24
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - None connector object in a terminate_connection in Pike  https://review.openstack.org/48213016:27
* jungleboyj sighs16:27
jungleboyjxyang:  So, right now we have doc/source/sample_config.rst that creates a web page that has the sample.conf file in it.16:28
jungleboyjxyang:  That isn't what you were looking for though.  Right?16:29
xyangjungleboyj: we just found out this is missing: https://github.com/openstack/openstack-manuals/blob/master/doc/config-reference/source/block-storage/drivers/emc-vmax-driver.rst16:30
xyangjungleboyj: so I don't know where it goes16:31
openstackgerritJohn Griffith proposed openstack/cinder master: Add ability to extend 'in-use' volume  https://review.openstack.org/45428716:31
xyangjungleboyj: I assume it should be moved to cinder doc folder eventually.  it has all driver docs16:31
xyangjungleboyj: https://github.com/openstack/openstack-manuals/tree/stable/ocata/doc/config-reference16:32
jungleboyjYeah, smcginnis pointed that out to Doug and it sounds like I will need to move that over16:32
xyangjungleboyj: it is still in ocata16:32
xyangjungleboyj: ok16:32
jungleboyjRight, they deleted all that stuff from Pike.16:32
xyangjungleboyj: sure16:33
jungleboyjThanks for finding that.  Let me figure out how to integrate that into our local tree and push up a patch.16:33
xyangjungleboyj: thanks!16:33
jgriffithxyang You're KILLING me!!  :)16:34
jungleboyjxyang:  Welcome.16:34
jgriffithxyang we're going to have to put a LOC limit on your patches from now on :)16:34
xyangjgriffith: what did I do:)16:34
jungleboyjjgriffith:  I feel your pain?  ;-)16:34
jungleboyjjgriffith:  We have been trying to do that for years!16:34
xyangjgriffith: I don't have big LOC patches any more:)16:34
jgriffiththen why am I reviewing one right now :)16:34
xyangjgriffith: oh that one, please review it!16:35
jgriffithNope, you said you didn't have any16:35
xyangjgriffith: that's a small one:)16:35
jgriffithI'm going to play PingPong :)16:35
xyangjgriffith: :)16:35
*** ChubYann has joined #openstack-cinder16:36
jgriffithxyang you get the "best Cinder contributor" award by the way, for being so thorough and updating all of the drivers!!16:37
xyangjgriffith: I have to because otherwise I can't pass Jenkins:)16:37
jgriffithlots of people would've cheated and made a kwargs or something there16:37
xyangjgriffith: oh, I didn't think of that.  will try next time:)16:38
jgriffithxyang Ohhh, so what you're saying is you're NOT really that good, you just didn't have any choice :)16:38
xyangjgriffith: :)16:38
jgriffithxyang don't you go changing16:38
jgriffithit's not as bad as I thought to review given that all the drivers are pretty consistent here16:39
xyangjgriffith: see, I told you it's a small patch:)16:40
jgriffithI know who to ping now when I need help with unit tests :)16:40
xyangjgriffith: thanks!16:41
*** itlinux has joined #openstack-cinder17:09
ericyoungAnyone have some time to review a ScaleIO driver patch... https://review.openstack.org/#/c/478644/17:13
jungleboyjericyoung:  I will try to take a look later.17:14
ericyoungjungleboyj: much appreciated17:14
ericyoungjungleboyj: quite a bit of cleanup in that patch...feel free to ask any questions17:15
jungleboyjericyoung:  Will do.17:17
*** lpetrut has joined #openstack-cinder17:19
openstackgerritRawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI  https://review.openstack.org/48264217:26
*** gheorghe_ has joined #openstack-cinder17:29
*** edmondsw has joined #openstack-cinder17:35
*** eharney has joined #openstack-cinder17:40
mriedemsmcginnis: ildikov: jgriffith: just reading scrollback, i still don't think we want to support extending the size of a reserved volume,17:42
mriedemwe know that's going to be unpredictable on the nova side, so why even bother risking it?17:42
smcginnismriedem: Yep, i think that's where we ended up.17:42
mriedemok, whew17:43
jgriffithmriedem correct, that's what we decided17:43
jgriffithmriedem I just added some notes in the docs to make it clear and gave it a +217:43
*** chhavi has quit IRC17:43
smcginnismriedem: jgriffith Just looking at that latest one. mriedem want to take a look before I +W?17:43
mriedemlet me take a glance quick17:44
hemnasomeone wants to be able to extend a reserved volume ?!17:44
smcginnishemna: No, not really.17:45
hemnaok phew17:45
*** zzzeek_ has quit IRC17:46
*** zzzeek_ has joined #openstack-cinder17:49
mriedemjgriffith: lgtm17:50
*** lpetrut has quit IRC17:51
*** raunak has joined #openstack-cinder18:01
*** raunak has quit IRC18:02
*** eharney has quit IRC18:05
*** diablo_rojo has quit IRC18:06
*** lpetrut has joined #openstack-cinder18:13
*** eharney has joined #openstack-cinder18:18
*** edmondsw_ has joined #openstack-cinder18:23
*** tesseract has quit IRC18:25
*** edmondsw has quit IRC18:25
*** lpetrut has quit IRC18:47
*** crushil has quit IRC18:49
*** crushil has joined #openstack-cinder18:50
*** nicolasbock has quit IRC18:50
patrickeastwould be curious to hear if others have opinions on https://review.openstack.org/#/c/479392 for where we would prefer to turn on cache limits for cinder testing and stuff18:57
openstackgerritPatrick East proposed openstack/cinder master: Add support for shared "backend_defaults" config  https://review.openstack.org/33529918:58
*** sdague has quit IRC19:00
smcginnispatrickeast: So we should all switch to systemd because it's closer to real deployment, but we shouldn't use realistic limits because why do developers need that? :|19:02
patrickeastsmcginnis: i guess so, yea19:04
*** arnewiebalck_ has joined #openstack-cinder19:06
geguileosmcginnis: eharney I'll post a solution to the os-brick problem by tomorrow19:09
smcginnisgeguileo: Great! Does that mean you've found the root problem?19:10
geguileosmcginnis: I have a poc working now, but I want to make it more robust, add unit tests, and test it with multipath as well19:10
geguileosmcginnis: yes19:10
smcginnisgeguileo: Awesome, thanks for doing that.19:10
*** mvk has quit IRC19:11
*** itlinux has quit IRC19:47
*** diablo_rojo has joined #openstack-cinder19:56
*** diablo_rojo has quit IRC20:05
*** diablo_rojo has joined #openstack-cinder20:05
*** edmondsw_ is now known as edmondsw20:07
*** itlinux has joined #openstack-cinder20:10
*** Nel1x has joined #openstack-cinder20:20
*** MVenesio has quit IRC20:25
*** adisky__ has quit IRC20:36
*** arnewiebalck_ has quit IRC20:44
*** zzzeek_ has quit IRC20:49
*** catintheroof has quit IRC21:04
*** jmlowe has joined #openstack-cinder21:05
openstackgerritEric Harney proposed openstack/cinder master: Fix python 3.6 escape char warnings in strings  https://review.openstack.org/48270421:05
smcginniseharney: 3.6?! :)21:05
eharneysmcginnis: i upgraded to Fedora 26 today, now my unit tests are noisy :)21:06
smcginniseharney: You have to put that on docstrings even?21:07
smcginniseharney: Does sphynx like that?21:07
eharneysmcginnis: yeah, it's a string issue, not a regex issue21:07
eharneyi don't know why it wouldn't, the "r" just affects the parsing, they still end up as strings21:08
smcginniseharney: Not sure how that parses source.21:08
eharneysmcginnis: converting them to raw seemed easier than having to redraw ascii diagrams that used backslashes for arrows :)21:09
*** zzzeek_ has joined #openstack-cinder21:11
*** dustins has quit IRC21:22
*** pcaruana has quit IRC21:23
*** abishop_ has quit IRC21:23
diablo_rojosmcginnis, so, been looking at condensing configuration options for a large chunk of today, and man...that will take a very long time. Its very very tedious.21:37
smcginnisdiablo_rojo: No doubt.21:37
diablo_rojoAlso, it occurs to me that in order to make a set that everyone can use, we will have more options during the deprecation period..21:38
openstackgerritEric Harney proposed openstack/cinder master: Dell SC: Fix python 3.6 'async' keyword warning  https://review.openstack.org/48271321:39
patrickeastwhat was the motivation for condensing the options again?21:39
diablo_rojosmcginnis, if you want me to put up a patch to start condensing them- even a few options at a time it's going to take some time.21:39
diablo_rojopatrickeast, uhhh, I don't remember who lead the charge..21:39
diablo_rojojgriffith, ?21:40
diablo_rojogeguileo, was in the conversation too if I remember21:40
smcginnisdiablo_rojo: Was the diea behind that that we have so many options that are similar but different for each driver, so collapse some of them into some more commonly applicable ones?21:40
smcginnispatrickeast: I think it was out of the DISCO driver trying to add new san_ip, san_user, etc. options.21:41
diablo_rojoI think so? But wasn't that kind of the original intention with the SAN ones that don't get used almost ever?21:41
patrickeastsmcginnis: gotcha21:41
patrickeastso whats the benefit of doing all that?21:41
eharneyyes that was the idea21:41
patrickeastor more importantly do i need to change my drivers? :D21:41
patrickeastlike... is this to save us headaches? simplify the deployments?21:42
eharneymain benefit IMO is that it's not a giant pain for deployers to figure out how to write a correct config file21:42
smcginniseharney: +121:43
diablo_rojoThat makes sense.21:43
patrickeastis it really going to help though? like.. if we have one thing called "management_ip" or something (an an example)21:43
smcginnisThis seems like it would be a really long term thing before everyone would be able to really take advantage of it, but if we could semi-standardize one some things, it would make it a little simpler.21:43
patrickeastisn't it the same amount of documentation required to read for a deployer to figure out what that does for their storage array versus something called "pure_management_vip"21:44
patrickeastor whatever21:44
diablo_rojoShould we agree on a general set of options and how they are defined and then collapse? Cause, looking at ALL THE OPTIOSN FOR ALL THE THINGS is making me want to claw my eyes out.21:44
eharneyit's not the same amount of reading if they have two different arrays21:44
smcginnispatrickeast: Well, if everyone used management_ip, then if you've had some experience, you wouldn't need to look up specific options each time you work with a driver.21:44
smcginnisAt least that's the hope.21:45
patrickeastmm i guess21:45
patrickeasti dunno, there is enough subtle differences that i'm skeptical it would really be much less confusing21:45
eharneythis consolidation will also help with efforts where we (someone) have to wire in every new option to tripleo for deployment tools to work for more drivers21:45
patrickeastlook at like "san_ip" today21:45
patrickeastits not even an ip on the san for us21:45
jgriffithpatrickeast diablo_rojo I was bitching because somebody had a patch up for their driver that had "device_xyz_api_endpoint", "device_xyz_admin_login"... etc etc21:45
jgriffithI suggested they just use san_ip, san_login etc21:46
jgriffiththe response was "it's not really a san"21:46
jgriffith"I don't know what those are"21:46
jgriffithblah blah blah21:46
smcginnisOr "it doesn21:46
smcginnist use the san protocol", whatever that means.21:46
jgriffiththen I went on a rant on how f'ing ridiculous it is that every driver has it's own custom variable/conf entries and exception entries that are 99.99999% exactly the same21:47
*** eharney has quit IRC21:47
diablo_rojojgriffith, lol noted :)21:47
jgriffithSo then I said in the meeting:  "generate a sample-conf and open the file, take a look and tell me how to sort through that"21:47
jgriffithdiablo_rojo said something like "OMG"21:47
jgriffithor "that's terrible" or "I"m blind!!!!"21:48
jgriffithcan't remember exactly21:48
jgriffithsomebody said "we should fix that" I laughed, and said "yes we should"21:48
jgriffiththat's about all I remember ;)21:48
jgriffithdiablo_rojo ok... so now seriously :)21:49
diablo_rojojgriffith, sounds about right. I remember being led into volunteering. Not sure exactly why I agreed to this torture.21:49
jgriffithdiablo_rojo yes, the problem with doing something like this and I do believe we discussed it was that it sucks because if you follow deprecation rules etc that means you implement MORE duplicates to remove duplicates which is stupid21:49
*** Apoorva has quit IRC21:50
jgriffithI thought we came up with an idea of trying to just consolidate some of the most obvious ones or something, but I don't know21:50
patrickeastthat makes sense, but i guess i'm not really sure that fewer abstract options are better than more specific-to-backend ones21:50
jgriffithpatrickeast seems about right coming from YOU!!!21:50
jgriffithJust kidding21:50
patrickeasthey, we use like 50/50 shared versus pure_* options :P21:50
jgriffithpatrickeast haha21:51
patrickeastand the problem i always have is with the vague shared ones21:51
jgriffithwell then you're in the top 90 percentile21:51
patrickeastthat no one can understand21:51
jgriffithyeah, I get it; it's just kinda sad when you look at our conf-sample21:51
patrickeastthe ones that are like "pure_do_the_thing = true" are way more self documenting21:51
jgriffithit's not really a sample for anything other than "look how messy this is"21:51
smcginnisdiablo_rojo: Maybe hold off on doing anything. This seems like something we should have a PTG session on.21:51
jgriffithhey.. here's another idea21:51
smcginnisOr a Friday afternoon rant session.21:52
patrickeastsounds like maybe we just need to organize it better?21:52
jgriffithdiablo_rojo I'm not saying you should do this (don't want you blaming me later)....21:52
jgriffithwhat if we tuned up a config-generator that you could specify what backend you wanted?21:52
jgriffithand what options were in your cloud?21:52
jgriffithso like if you don't run c-backup or swift you don't get those options, and if you run pure you don't get SolidFire options?21:53
jgriffithOr people can just use grep21:53
jgriffithand their eyes21:53
jgriffithor whatever21:53
* jgriffith shuts up now21:53
smcginnisjgriffith: Like this? https://review.openstack.org/#/c/474307/21:53
patrickeasthaha yea i was just looking for that patch21:54
patrickeastif we had them organized somehow it wouldn't be so scary21:54
jgriffithOH, well there ya go!21:54
patrickeastjust group the backend options21:54
jgriffithdiablo_rojo is so quick she actually implemented it 2 weeks before I mentioned it :)21:54
jgriffithdiablo_rojo I said "DONT GO DO IT YET"21:54
jgriffithOh, that was Doug21:55
jgriffithnever mind21:55
diablo_rojojgriffith, I will take credit for it since I did +1 ;)21:55
jgriffithdiablo_rojo works for me :)21:55
diablo_rojoMoral support is worth a lot you know :)21:55
jgriffithsometimes it's worth *everything*21:56
openstackgerritJohn Griffith proposed openstack/cinder master: organize options based on backend drivers  https://review.openstack.org/47430721:56
diablo_rojoHe needs some +1 and +2 love21:56
diablo_rojojgriffith, was that on purpose? lol21:56
jgriffithbut it won't hurt anything21:57
jgriffithI hope21:57
jgriffithdiablo_rojo he had a -1 on it and jenkins fail21:57
diablo_rojoI thought not. Now it's gonna look kinda weird when you +221:57
*** jmlowe has quit IRC21:57
jgriffithI was looking to see if the patch you mentioned that landed fixes jenkins up21:57
diablo_rojosmcginnis, sounds good. I can add it to the ptg agenda.21:57
jgriffithdiablo_rojo why would that look weird?21:57
jgriffithI just hit rebase on it21:58
jgriffithif that's weird I'm in super big trouble :)21:58
diablo_rojoI dunno, just a passing thought. Nevermind me :)21:58
diablo_rojoI guess it won't21:58
jgriffithI'm also assuming doug ws going to come back to do something else with it21:58
diablo_rojoLol, blame the config options. My brain is becoming mush.21:58
diablo_rojoAlso booking flights for the other chunk of today..21:59
jgriffithconfig options will most certainly ruin your brain21:59
jgriffithmaybe that can be my excuse :)21:59
diablo_rojojgriffith, I just need one more config option. I CAN STOP WHENEVER I WANT21:59
dfisherini-style config is much better than ifdef's all over .c/.h code.21:59
jgriffithdfisher says who?21:59
jgriffithbully for you22:00
dfisherthat's right.22:00
diablo_rojojgriffith, +1 to tshirts with that and the cinder logo22:00
jgriffithdfisher I'm just kidding by the way22:00
dfisheri figured ;)22:00
dfisherbuilding Erlang for SPARC is the opposite of fun.22:00
* jgriffith resists tempation to comment on building any C program on SPARC22:01
*** jmlowe has joined #openstack-cinder22:01
jgriffithit was nice when it was running :)22:01
dfisherpkg install gcc.  gcc foo.c ;)22:01
jgriffithonly thing worse was AIX and old Smitty22:01
jgriffithdfisher well.. yeah, depending on what sort of libs foo.c pulled in, and if you used so's22:02
dfisherit's better now.22:02
dfisherbut, I understand.22:02
jgriffithBUT to be fair, this was 15 years ago and I worked on Linux and HPUX22:02
jgriffithso anything else was just "hard"22:02
jgriffithI'm sure everybody running SPARC said the same about crappy HPUX22:03
dfisherwe still say it.22:03
ildikovjgriffith: what's the problem with AIX? :)22:03
jgriffithetc etc22:03
jgriffithildikov I knew I was going to catch it from somebody!22:03
dfisheri know this is the wrong community for this, but it's really too bad Solaris isn't more prevalent.  It's really really good.  :/22:03
jgriffithNothing, TBC nothing at all!22:03
jgriffithexcept Smitty was kind of an awkard sysadmin tool IMO22:04
ildikovjgriffith: my pleasure then :)22:04
dfisheralas, Oracle had to go be dbags and re-close it.22:04
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - seamless upgrade from SMI-S to REST  https://review.openstack.org/48213822:04
jgriffithbut again, probably because I was coming from my sheltered HPUX world at the time22:04
*** Apoorva has joined #openstack-cinder22:04
*** felipemonteiro__ has quit IRC22:04
jgriffithI just remember being at a customer site in their DC at 3 in the morning watching that stupid running man graphic over and over and over, only to have it fail22:05
jgriffithkept thinking they should have made him explode or something when a task failed22:05
jgriffithor run into a wall22:05
ildikovI think running over and over again and not to explode when the task failed was kind of a bigger torture :)22:07
dfisheror trip over Clippy?22:08
*** jmlowe has quit IRC22:09
ildikovoooh the good old days, when Clippy started to snore on the projector one of my university classes... :)22:10
ildikovthat teacher was hopeless...22:10
*** dave-mccowan has quit IRC22:14
*** markvoelker has quit IRC22:16
*** markvoelker has joined #openstack-cinder22:17
jungleboyjdiablo_rojo:  jgriffith Just finally catching up with the backlog here.22:17
jungleboyjdiablo_rojo:  jgriffith  The direction we were looking to go was to take existing config options and try to standardize around them so that we could not have to add a bunch more during the deprecation period.22:18
*** gnufied has quit IRC22:18
jungleboyjDo it little by little starting with easy/obvious ones like san_ip.22:18
jungleboyjAnyway, if we can go into the PTG with some idea of what those common options are so that it is not just a bitch session it would be good.  I really think we need to try to slow our config option sprawl though.22:19
ildikovjungleboyj: that and the proposal about adding only those to the config file which driver is configured could result in a kind of condensed output by the end IMHO22:21
jungleboyjildikov:  Agreed.  Being able to create a more focused sample config file would also be a good improvement.22:23
ildikovjungleboyj: +122:24
jungleboyjildikov:  Also hoping to help the situation somewhat with the documentation improvements.22:25
ildikovjungleboyj: yeah, that's a tough one22:26
*** edmondsw has quit IRC22:26
jungleboyjildikov:  How so>22:30
*** dr_gogeta86 has quit IRC22:30
ildikovjungleboyj: that's what comes to my mind when I think about config options and docs at the same time :)22:30
ildikovjungleboyj: spent a little time with the scripts back at the time that generate the Config Guide22:31
ildikovjungleboyj: I still have all my respect to the team to put all that automation together22:31
*** gnufied has joined #openstack-cinder22:31
jungleboyjildikov: Well, I am just hoping to be able to keep things equally good as I bring the docs over and then hopefully improve it over time.22:34
ildikovjungleboyj: it will be all good22:34
jungleboyjildikov: Thanks.  Keep sending me moral support.  :-)22:35
ildikovjungleboyj: moving the docs over has many advantages, so keep those in mind :)22:36
jungleboyjildikov:  Thanks.22:36
ildikovjungleboyj: you can sort out the rest step by step22:36
jungleboyjildikov: That is the plan.  :-)22:36
ildikovjungleboyj: sounds good then :)22:37
*** dfisher has quit IRC22:44
*** itlinux has quit IRC23:10
openstackgerritMerged openstack/cinder master: Windows iSCSI: use configured iSCSI portals  https://review.openstack.org/46465923:28
openstackgerritMerged openstack/cinder master: Allow extension of volumes with snapshots for SMBfs driver  https://review.openstack.org/46290223:30
*** diablo_rojo has quit IRC23:46

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!