Wednesday, 2017-02-01

marksturYeah. I'll hang on to it. It'll probably come in handy eventually.  :)00:00
*** alonma has joined #openstack-cinder00:01
* jungleboyj starts making my own sign00:01
*** ducttape_ has joined #openstack-cinder00:01
hemnahttp://paste.openstack.org/show/597113/00:02
hemna^^ latest CI reporting results00:02
hemnanow it shows the last success for each job in each CI00:03
ildikovmarkstur: free me from multi-attach ;)00:04
hemnaso it exposes some jobs that are not running now for whatever reason00:04
hemnalike00:04
hemnadsvm-tempest-cinder-hnas-iscsi00:05
hemnamaybe they changed the name of the job?00:05
*** alonma has quit IRC00:05
hemnaildikov, so, now that the new API is in, shouldn't multi-attach be 'easy' ? :)00:05
marksturildikov: That works. I guess it is better than incarceration00:06
hemnamarkstur, are you sure about that?00:06
ildikovhemna: it's not fully figured out yet still, but that's my hope too, yes :)00:07
*** alonma has joined #openstack-cinder00:07
*** dims_ has joined #openstack-cinder00:10
*** dims has quit IRC00:11
*** alonma has quit IRC00:11
*** Julien-zte has joined #openstack-cinder00:16
*** kaisers has joined #openstack-cinder00:18
*** alonma has joined #openstack-cinder00:18
jgriffithhemna might suggest filtering out the gate-* jobs when computing averages at least00:22
*** kaisers has quit IRC00:22
*** alonma has quit IRC00:23
jgriffithhemna it would also be interesting to get a tally/comparison of jobs run out of last n succesful gate runs00:23
jgriffithlooking at those numbers just a quick glance and things don't quite add up00:23
jgriffithsmcginnis so are we going to do anything like anything with a success rate < n% gets marked as unsupported?00:24
jungleboyjjgriffith, There was a comment about anything that was less than 40% the other day getting marked unsupported.00:25
jungleboyjThat was just said in passing though.00:25
jgriffithhehe... 40%, setting the bar high :)00:26
smcginnisjgriffith: I think for now it's good just seeing who stops passing for a few weeks.00:27
jgriffith@smcginnis yeah00:27
smcginnisjgriffith: We probably can discuss at the PTG what we want for a next step of setting a minimum passing rate.00:27
jgriffithscary data IMO00:27
smcginnisTru00:28
jgriffithI'm trying to figure out how the "out of 'n' runs " is calculated00:29
jungleboyjjgriffith, Yeah, 40 is high.  It was thrown out there though.  :-)00:30
jgriffithjungleboyj I was being sarcastic00:30
jgriffith:)00:31
jgriffithI think I'll bump my average up and not report on failure :)00:31
jgriffiththen I can get 100% :)00:31
jungleboyjjgriffith, Well, there you go!  That is being industrious.00:33
jgriffithjungleboyj :)00:33
jungleboyjSadly, that is kind of a high bar though, isn't it right now?00:33
jgriffithjungleboyj I think it should be at least a passing grade honestly00:33
jgriffith70%00:33
jgriffithno curve00:33
jgriffithand I also want to understand the 100% ers that I can't actually find any votes for out of the last 15 merged reviews I've checked00:34
*** jdurgin has quit IRC00:34
jgriffithhehe.. found my answer00:37
jungleboyjjgriffith, Ok, a C is a fair expectation.  ;-)  What was the answer?00:42
*** zhurong has joined #openstack-cinder00:42
jgriffithdon't run anything but add a comment under the CI name00:43
jgriffithso you won't get anything in the ci results column up top, but if you toggle on ci comments there are comments logged with no info00:43
*** zhurong has quit IRC00:44
jungleboyjjgriffith, Oh, interesting.00:44
smcginnisOh lovely.00:44
*** crushil has quit IRC00:45
jgriffithwe should probably do something like implement testr-results and dump everything off into an ELK stack00:45
smcginnisIf it's working the way my script was, I was checking the last 1000 comments. If they only reported 10, and all 10 of them passed, it would show up as 100%.00:45
*** crushil has joined #openstack-cinder00:45
*** jose-phillips has quit IRC00:45
jgriffith@smcginnis I know that's how I came up with my diabolical plan to achieve 100% :)00:46
*** gouthamr has joined #openstack-cinder00:46
smcginnisjgriffith: That's not a bad idea. But another pile of work. :/00:46
smcginnisOh, definitely. There are plenty of ways to cheat at this game.00:46
jgriffithsmcginnis LOL, I know.. and I don't want to do it so never mind :)00:46
*** rajinir has quit IRC00:46
smcginnis;)00:46
*** jdurgin has joined #openstack-cinder00:48
*** mriedem has joined #openstack-cinder00:48
*** diablo_rojo has joined #openstack-cinder00:50
jgriffith@smcginnis hemna jungleboyj to be clear, I think what you guys have been working on there and putting together is great work and very useful00:51
jgriffithdidn't mean to detract from that00:51
smcginnisjgriffith: Hah, not taken that way at all.00:51
jgriffithjust being cynical about 3'rd party CI00:51
smcginnisI've worked on it - and I'm still cynical about 3rd party CI. ;)00:52
smcginnisMostly because I know what it's like trying to run one!00:52
jgriffith@smcginnis :)00:52
* asselin__ rises from the dead00:52
smcginnisasselin__: Hey! Haven't seen you in a long time. :)00:52
asselin__what's up :)00:52
jgriffithI honestly don't know how to get much better, most of the failures I've checked as of late on mine have been things completely outside of the driver00:52
jgriffithwhich is weird00:53
*** ducttape_ has quit IRC00:55
smcginnisjgriffith: Same here. Though I do have a persistent issue with iSCSI that I swear is some day going to be discovered as the other guys in the lab stomping on my IP addresses.00:56
smcginnisOtherwise most are patches that need to be rebased, weird intermitent failures, or who knows what.00:56
*** liverpooler has quit IRC00:58
jgriffith@smcginnis LOL01:01
jgriffithI do have a problem after my virtual cluster gets about 50K volumes on it01:01
smcginnisjgriffith: At least no one has walked off with my gear. Yet.01:01
jgriffith@smcginnis consider yourself lucky!!!01:02
smcginnis50k? How many CI jobs you runnin? :)01:02
jgriffithhehe01:02
jgriffithwell, it's a long story01:02
*** ducttape_ has joined #openstack-cinder01:06
*** jperry has quit IRC01:07
*** xyang has joined #openstack-cinder01:07
*** xyang has quit IRC01:08
*** diablo_rojo has quit IRC01:14
*** ducttape_ has quit IRC01:39
*** dpak has joined #openstack-cinder01:46
dpak?01:47
*** dpak has left #openstack-cinder01:50
*** ducttape_ has joined #openstack-cinder01:54
*** knomura has joined #openstack-cinder02:01
*** knomura has quit IRC02:05
*** Apoorva has quit IRC02:06
*** Apoorva has joined #openstack-cinder02:06
*** Apoorva has quit IRC02:07
*** Jack_Iv has joined #openstack-cinder02:09
*** Apoorva has joined #openstack-cinder02:12
*** Apoorva has quit IRC02:13
*** ducttape_ has quit IRC02:13
*** Jack_Iv has quit IRC02:14
*** Apoorva has joined #openstack-cinder02:15
*** Apoorva has quit IRC02:17
*** Apoorva has joined #openstack-cinder02:18
*** kaisers has joined #openstack-cinder02:19
*** alonma has joined #openstack-cinder02:20
*** diablo_rojo_phon has quit IRC02:20
*** ducttape_ has joined #openstack-cinder02:20
*** ducttape_ has quit IRC02:20
*** ducttape_ has joined #openstack-cinder02:20
*** Apoorva has quit IRC02:22
*** kaisers has quit IRC02:23
*** alonma has quit IRC02:24
*** alonma has joined #openstack-cinder02:26
*** alonma has quit IRC02:30
*** Julien-z_ has joined #openstack-cinder02:32
*** Julien-zte has quit IRC02:34
*** Julien-z_ has quit IRC02:40
openstackgerritMerged openstack/cinder: Cluster volume group fix in lvm  https://review.openstack.org/39626902:47
*** dhinesh has quit IRC02:48
*** raunak has joined #openstack-cinder03:00
openstackgerritjiamin proposed openstack/cinder: change the design of CG in DS8K driver  https://review.openstack.org/41932103:01
*** xyang has joined #openstack-cinder03:18
*** xyang has quit IRC03:19
*** crushil has quit IRC03:28
openstackgerritMerged openstack/cinder: clean up replication volume on an xiv backend  https://review.openstack.org/42676303:28
*** sticker_ has joined #openstack-cinder03:29
*** sticker has quit IRC03:32
*** mtanino has quit IRC03:43
openstackgerritSean McGinnis proposed openstack/cinder: Move release notes to correct location  https://review.openstack.org/42754203:43
*** nicolasbock has quit IRC03:46
*** tlian has quit IRC03:59
*** crushil has joined #openstack-cinder04:02
*** psachin has joined #openstack-cinder04:03
*** ducttape_ has quit IRC04:05
smcginnisIf anyone, like me, ignored this in the ML and now has stacking failures: http://lists.openstack.org/pipermail/openstack-dev/2017-January/111295.html04:07
*** sudipto_ has joined #openstack-cinder04:08
*** sudipto has joined #openstack-cinder04:08
*** raunak has quit IRC04:10
*** mdnadeem has joined #openstack-cinder04:10
*** crushil has quit IRC04:20
*** crushil has joined #openstack-cinder04:21
*** arch-nemesis has quit IRC04:24
*** ducttape_ has joined #openstack-cinder04:28
*** crushil has quit IRC04:31
*** alonma has joined #openstack-cinder04:33
openstackgerritMerged openstack/cinder: Add create volume from image policy  https://review.openstack.org/42672804:36
*** alonma has quit IRC04:38
*** ducttape_ has quit IRC04:38
*** pbandark has joined #openstack-cinder04:46
*** ducttape_ has joined #openstack-cinder04:47
*** nikeshm has quit IRC04:53
openstackgerritVijay Ladani proposed openstack/cinder: 3PAR: rally test create-volume-from-snapshot fails  https://review.openstack.org/42173904:58
*** kwathore has joined #openstack-cinder05:03
*** pgadiya has joined #openstack-cinder05:09
*** kaisers has joined #openstack-cinder05:10
*** ducttape_ has quit IRC05:13
*** kaisers has quit IRC05:15
*** raunak has joined #openstack-cinder05:15
*** lpetrut has joined #openstack-cinder05:16
*** kaisers has joined #openstack-cinder05:18
*** dave-mccowan has quit IRC05:18
*** gouthamr has quit IRC05:21
*** dave-mccowan has joined #openstack-cinder05:22
*** kaisers has quit IRC05:22
*** pdeore has joined #openstack-cinder05:22
*** nikeshm has joined #openstack-cinder05:32
*** dave-mcc_ has joined #openstack-cinder05:59
*** Jack_Iv has joined #openstack-cinder06:00
*** dave-mccowan has quit IRC06:01
*** raunak has quit IRC06:01
*** lkuchlan has joined #openstack-cinder06:04
*** knomura has joined #openstack-cinder06:07
*** knomura has quit IRC06:11
*** lpetrut has quit IRC06:13
openstackgerritDinesh Bhor proposed openstack/cinder: Fix 500 error while listing manageable volumes and snapshots  https://review.openstack.org/42756006:16
*** alonmarx has joined #openstack-cinder06:21
*** ducttape_ has joined #openstack-cinder06:27
*** rcernin has joined #openstack-cinder06:29
*** ducttape_ has quit IRC06:31
*** mjura has joined #openstack-cinder06:33
vladaniPlease review these changes...06:33
vladanihttps://review.openstack.org/#/c/421739/06:34
vladanihttps://review.openstack.org/#/c/424454/06:34
*** alonma has joined #openstack-cinder06:41
*** nkrinner_afk is now known as nkrinner06:44
*** alonma has quit IRC06:46
*** arne_r has joined #openstack-cinder06:53
*** lpetrut has joined #openstack-cinder06:56
*** fnordahl has quit IRC06:57
*** fnordahl has joined #openstack-cinder06:57
*** kaisers has joined #openstack-cinder07:09
*** alexey_weyl has joined #openstack-cinder07:13
alexey_weylHi Guys..07:13
alexey_weylI am a core developer in the Vitrage project, and recently (from about 30 hours ago) we have a problem in our gate in gerrit due to Cinder and I wanted to ask your help07:14
alexey_weylThe problem goes like that..07:14
alexey_weylIn our tempest we subscribe to the cinder.conf the following data:07:15
alexey_weylnotification_topics = notifications,vitrage_notifications07:15
alexey_weylDue to that, when we create volumes and attached them etc.. Vitrage receives those events from the oslo bus and uses them.07:16
alexey_weylBut approximatelly from 30 hours ago we don't receive those events anymore, although I see in the logs that the volume is created and attached (and after wards deleted of course)07:17
alexey_weylSo I wanted to ask you guys if maybe you know due to what this problem happens?07:17
*** alonma has joined #openstack-cinder07:17
*** tesseract has joined #openstack-cinder07:18
alexey_weyl(it is happening only in when creating cinder, because when creating ports in neutron, instances in nova, alarms in aodh etc.. we receive the events)07:18
*** pcaruana has joined #openstack-cinder07:19
*** alonma has quit IRC07:22
*** fnordahl has quit IRC07:27
*** fnordahl has joined #openstack-cinder07:27
*** Jack_Iv has quit IRC07:31
*** xek__ is now known as xek07:31
*** Jack_Iv has joined #openstack-cinder07:32
*** alonmarx has quit IRC07:34
*** alonma has joined #openstack-cinder07:35
*** sudipto has quit IRC07:37
*** sudipto_ has quit IRC07:37
*** Jack_Iv has quit IRC07:38
*** Jack_I has joined #openstack-cinder07:39
*** alonma has quit IRC07:40
*** ChubYann has quit IRC07:42
openstackgerritMerged openstack/cinder: Add cluster_name to temporary volumes  https://review.openstack.org/42736107:44
*** alonma has joined #openstack-cinder07:45
*** shubham0d has joined #openstack-cinder07:45
*** lpetrut has quit IRC07:48
*** adisky_ has joined #openstack-cinder07:54
*** ducttape_ has joined #openstack-cinder07:57
*** ducttape_ has quit IRC08:02
*** lpetrut has joined #openstack-cinder08:06
*** bkopilov has quit IRC08:14
*** lpetrut has quit IRC08:20
*** dave-mcc_ has quit IRC08:24
*** bkopilov has joined #openstack-cinder08:24
*** s-shiono has joined #openstack-cinder08:48
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
*** s-shiono has quit IRC09:00
*** alexey_weyl has quit IRC09:06
*** alexey_weyl has joined #openstack-cinder09:12
dulekalexey_weyl: 24 h ago this was merged: https://review.openstack.org/#/c/426547/09:15
dulekgeguileo: ^09:15
geguileoalexey_weyl: I'll have a look, as it should be working...09:16
geguileodulek: thanks for the ping09:16
geguileoalexey_weyl: Can you point me to a sample config in the gate that is failing, please?09:17
alexey_weylgeguileo: yes09:19
alexey_weyli am sending you a link to a tempest that failed09:20
geguileoalexey_weyl: thanks09:20
alexey_weylyou can go to this link http://logs.openstack.org/28/427128/4/check/gate-vitrage-dsvm-datasources-ubuntu-xenial/7189cf2/09:20
alexey_weyland here you can see in the vitrage-graph.log that we don't receive any cinder.volume notifications although we create a volume and attach it09:21
alexey_weyldulek09:22
alexey_weyldulek: yes, thats what I saw and thought that it is connected09:22
geguileoalexey_weyl: I see the problem09:22
alexey_weylthats why I decided to ask you guys about that09:22
*** e0ne has joined #openstack-cinder09:23
geguileoalexey_weyl: there's no transport_url defined in oslo_messaging_notifications09:23
alexey_weylgeguileo09:23
alexey_weylgeguileo09:23
alexey_weylgeguileo: i am very curious09:23
geguileoprobably oslo_messaging takes the value from the default when you define a driver in that section09:24
*** mdnadeem has quit IRC09:24
*** shubham0d has quit IRC09:24
*** mdnadeem has joined #openstack-cinder09:24
geguileoalexey_weyl: I'll investigate how oslo messaging is actually getting the value for oslo_messaging_motifications.transport_url and fix the check09:24
geguileoalexey_weyl: OK, apparently they decoupled the notification transport url in Mitaka, and Vitrage is using the old cfg09:27
alexey_weylgeguileo: what should be written in the transport_url?09:27
geguileoalexey_weyl: Oh, it's not your fault09:27
alexey_weylI see..09:28
geguileoalexey_weyl: I have to make the check compatible with the old configuration09:28
alexey_weylin cinder you mean?09:28
*** ducttape_ has joined #openstack-cinder09:28
geguileoalexey_weyl: Sure you can fix it, but it was my mistake for not considering how oslo messaging was configured before Mitaka09:28
geguileoalexey_weyl: Yes, I have to fix it in Cinder09:28
alexey_weylgeguileo: That will be great.09:29
alexey_weylgeguileo: when do you think it will be possible to see the new change?09:30
geguileoalexey_weyl: Could you create a LP bug for me?09:30
geguileoalexey_weyl: I should be able to get it fixed this morning09:30
*** stakeda has quit IRC09:31
alexey_weylgeguileo: ok, I am opening an LP bug.09:31
geguileoalexey_weyl: thanks09:31
*** lpetrut has joined #openstack-cinder09:32
alexey_weylgeguileo: but just that I will know, what is supposed to be the transport_url value in the conf?09:32
geguileoalexey_weyl: it allows you to send the notifications to a different transport than the rpc09:32
*** ducttape_ has quit IRC09:32
geguileoalexey_weyl: As you can see, defining it in the default section is deprecated: https://github.com/openstack/oslo.messaging/blob/afaa4d935dd77760492f36dba622f6a01fe14fd9/oslo_messaging/notify/notifier.py#L4409:33
alexey_weylgeguileo: ok, we are all waiting for your fix, because this is the end of the Ocata cycle and we have to push all our changes.09:33
alexey_weylgeguileo: thanks in advance09:33
geguileoalexey_weyl: so no pressure, right?  ;-)09:34
alexey_weylgeguileo: hhhhhhhhh :)09:35
*** zhonghua has quit IRC09:36
*** zhonghua has joined #openstack-cinder09:36
*** sudipto_ has joined #openstack-cinder09:38
*** sudipto has joined #openstack-cinder09:38
alexey_weylgeguileo: I have opened a new bug. Hope it is ok the way I wrote it. here is the link: https://bugs.launchpad.net/cinder/+bug/166092809:39
openstackLaunchpad bug 1660928 in Cinder "Backwards compatibility for oslo messaging in Cinder" [Undecided,New]09:39
geguileoalexey_weyl: Thanks09:42
openstackgerritzhangsong proposed openstack/cinder: RBD:Move RBDVolume calls to a separate threads  https://review.openstack.org/42318409:42
*** jordanP has joined #openstack-cinder09:46
*** yumiriam has joined #openstack-cinder09:48
*** xiangxinyong has quit IRC09:51
*** xiangxinyong has joined #openstack-cinder09:52
openstackgerritMarc Koderer proposed openstack/cinder: Add check if copyoffload tool is available  https://review.openstack.org/42761309:52
*** shubham0d has joined #openstack-cinder09:55
*** e0ne has quit IRC09:56
*** ebalduf has quit IRC10:08
*** ebalduf has joined #openstack-cinder10:11
erlongeguileo: hey Gorka, when you can, please give a look on these: https://review.openstack.org/#/c/422733/, https://review.openstack.org/425100, https://review.openstack.org/42734810:12
*** shausy has joined #openstack-cinder10:18
*** Jack_Iv has joined #openstack-cinder10:20
*** Jack_I has quit IRC10:21
*** Jack_Iv has quit IRC10:23
openstackgerritjiamin proposed openstack/cinder: change the design of CG in DS8K driver  https://review.openstack.org/41932110:25
*** Zack2 has joined #openstack-cinder10:26
Zack2hi all. how come there is a --tenant option for "cinder list" and "cinder snapshot-list", but no such option for "cinder backup-list"?10:28
Zack2not having that option for backups hardens ones efforts to create any sort of "per tenant backups" overview script/report10:29
kwathorePlease review below cheery pick patch10:36
kwathore https://review.openstack.org/#/c/407928/10:36
*** alexey_weyl has quit IRC10:37
*** sudipto has quit IRC10:40
*** sudipto_ has quit IRC10:40
*** alonma has quit IRC10:45
*** alonma has joined #openstack-cinder10:45
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix attachments after attached migration  https://review.openstack.org/42734810:45
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix volume retype with migration as non-admin  https://review.openstack.org/42273310:45
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix multibackend-matrix confs  https://review.openstack.org/42763210:45
*** dalgaaf has quit IRC10:48
*** dalgaaf has joined #openstack-cinder10:49
*** alyson_ has joined #openstack-cinder10:49
*** alonma has quit IRC10:49
*** mvk has quit IRC10:49
*** alonma has joined #openstack-cinder10:51
*** shubham0d has quit IRC10:51
*** alonma has quit IRC10:55
*** alexey_weyl has joined #openstack-cinder10:58
*** ducttape_ has joined #openstack-cinder10:58
Zack2kwathore: so this is the case only when HP 3par is in question or it does not work in general. I mean I use ceph for volumes, snapshots and backups and I don't have any problems creating it10:59
kwathoreyes10:59
kwathoreit is releated to HP 3par10:59
*** alexey_weyl has quit IRC11:01
*** kaisers1 has quit IRC11:01
*** kaisers1 has joined #openstack-cinder11:02
*** ducttape_ has quit IRC11:03
*** Jack_Iv has joined #openstack-cinder11:03
*** nicolasbock has joined #openstack-cinder11:04
*** shubham0d has joined #openstack-cinder11:06
*** alonma has joined #openstack-cinder11:07
*** jbernard has quit IRC11:11
*** alonma has quit IRC11:12
*** alonma has joined #openstack-cinder11:15
*** jbernard has joined #openstack-cinder11:17
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - ignore service level and workload in xml  https://review.openstack.org/42750611:18
*** mvk has joined #openstack-cinder11:19
openstackgerritGorka Eguileor proposed openstack/cinder: Fix notification short-circuit  https://review.openstack.org/42764711:21
geguileodulek: ^11:22
*** e0ne has joined #openstack-cinder11:22
*** jamielennox is now known as jamielennox|away11:25
*** shubham0d has quit IRC11:27
dulekgeguileo: Looking.11:27
*** sdague has joined #openstack-cinder11:28
geguileoI have verified it in my env with the driver set to logging and setting a hard breakpoint in the code11:29
*** mdnadeem has quit IRC11:36
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix attachments after attached migration  https://review.openstack.org/42734811:41
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix multibackend-matrix confs  https://review.openstack.org/42763211:41
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix volume retype with migration as non-admin  https://review.openstack.org/42273311:41
*** mdnadeem has joined #openstack-cinder11:42
erlonhemma: indeed, dsvm-tempest-cinder-hnas-iscsi -> hitachi-hnas-iscsi11:44
*** ociuhandu has joined #openstack-cinder11:46
*** ociuhandu has quit IRC11:51
*** pdeore has quit IRC11:52
lyarwoodMorning all, hopefully a stupid question but none of the connectors in os-brick update the connection_info / connection_properties passed in by Nova/Cinder right?11:57
lyarwoodI've found some code that's going out of it's way in nova to save connection_info after a call to connect_volume but I can't think why11:57
openstackgerritjiamin proposed openstack/cinder: support bulk create for CG and VG  https://review.openstack.org/42494511:58
*** adrianofr has quit IRC11:59
dulekgeguileo: What do you think about this: https://github.com/openstack/tempest/blob/bf73e0cb1c9eba5d62b9230b0535c970651dc1fa/tempest/api/volume/admin/v2/test_backends_capabilities.py#L36-L5212:00
dulekgeguileo: Tempest assumes that get-pools returns list that can be used as argument to get-capabilities.12:00
dulekgeguileo: I'm leaning to the side that this is a rightful assumption and get-capabilities should accept clusters as well.12:01
dulekgeguileo: I'll be happy to know what do you think about it.12:01
*** ociuhandu has joined #openstack-cinder12:01
* geguileo checking12:02
*** Jack_Iv has quit IRC12:03
*** knomura has joined #openstack-cinder12:05
geguileodulek: You mean that it should it should look by host and cluster to find the match?12:08
*** ociuhandu has quit IRC12:08
*** adrianofr has joined #openstack-cinder12:09
*** cdelatte has joined #openstack-cinder12:11
dulekgeguileo: Hm, I thought get-capabilities goes down to the c-vol, let my check this.12:12
geguileodulek: It goes12:12
geguileodulek: But first checks if the service exists12:12
geguileodulek: And it's checking by host12:13
dulekgeguileo: Oh right, this is where it fails. I don't see an issue with letting it check for cluster too.12:13
dulekMaybe only the fact that host means both things now.12:13
dulekmdulko:tempest/ (master) $ cinder get-capabilities12:14
dulekusage: cinder get-capabilities <host>12:14
dulekThis <host>12:14
geguileodulek: well, truth be told I don't think we have documentation for that12:14
geguileodulek: Except in the client, and we can change that12:14
geguileodulek: So it says host/cluster12:14
dulekgeguileo: Right - URL has capabilities/chrzaszcz@nfs, so it's unnamed.12:15
geguileodulek: yup  ;-)12:15
geguileodulek: So we can say it was always backend_id12:15
dulekchrząszcz is a beetle BTW. ;)12:16
geguileolol12:16
dulekgeguileo: Now there will be two ways of accessing same data - through cluster and host URL. I don't exactly see a serious issue with that too.12:16
geguileodulek: I think that's ok, since they are not contradicting one another12:17
geguileodulek: Though we'll send the request to the cluster if it's clustered12:17
geguileodulek: You may ask for the capabilities of a specific host, but if it's clustered the request will go to the cluster12:18
dulekgeguileo: This isn't that bad, unless someone is using this call to healthcheck individual nodes.12:18
geguileodulek: Well, that will still work for A/P deployments12:19
geguileodulek: And for A/A they'll have to do it using the new log level checking API12:19
dulekgeguileo: Good point, if you're moving to A/A, then you expect your get-capabilities call to be A/A as well.12:19
*** alonma has quit IRC12:20
geguileodulek: I can write a patch for this12:20
geguileodulek: Or do you want to do it?12:20
*** alonma has joined #openstack-cinder12:20
dulekgeguileo: I have some free cycles, so I can do it. I'll also upload a Tempest patch that fixes get-pools tests when using A/A.12:21
geguileodulek: I was talking about the Cinder changes12:22
dulekgeguileo: To get-capabilities? I meant I have some time, so I can write Cinder changes as well. :)12:22
geguileodulek: Oh, OK, please add me to the review, since I already have the context in my head to review the patch12:22
dulekOkay!12:23
*** alonma has quit IRC12:24
openstackgerritjiamin proposed openstack/cinder: support bulk create for CG and VG  https://review.openstack.org/42494512:25
*** alonma has joined #openstack-cinder12:26
*** catintheroof has joined #openstack-cinder12:28
*** ducttape_ has joined #openstack-cinder12:29
*** alonma has quit IRC12:31
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix attachments after attached migration  https://review.openstack.org/42734812:31
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix multibackend-matrix confs  https://review.openstack.org/42763212:31
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix volume retype with migration as non-admin  https://review.openstack.org/42273312:31
*** ociuhandu has joined #openstack-cinder12:31
*** ducttape_ has quit IRC12:33
*** pbandark has quit IRC12:33
*** ociuhandu has quit IRC12:36
Dinesh_Bhorgeguileo, smcginnis: Hi, whenever you get time please take a look at it: https://review.openstack.org/#/c/427560/12:40
geguileoDinesh_Bhor: lol, I had just opened it for review12:40
Dinesh_Bhorgeguileo: aah, thanks12:41
*** alonma has joined #openstack-cinder12:42
*** alonma has quit IRC12:45
*** alonma has joined #openstack-cinder12:46
*** gaurangt has joined #openstack-cinder12:46
*** mvk has quit IRC12:47
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42674712:48
*** test123 has joined #openstack-cinder12:48
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42674812:49
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Use storage profile name in extra-spec  https://review.openstack.org/42725712:49
*** test123 has quit IRC12:49
*** eharney has joined #openstack-cinder12:50
nikeshmhi12:50
nikeshmany one facing the issue in their CIs https://review.openstack.org/#/c/427561/12:51
*** lkuchlan_ has joined #openstack-cinder12:52
*** lkuchlan has quit IRC12:53
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42674912:53
openstackgerritPooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants  https://review.openstack.org/42675012:55
*** ociuhandu has joined #openstack-cinder12:56
*** mtanino has joined #openstack-cinder12:56
arne_rhi channel, quick question12:57
*** ducttape_ has joined #openstack-cinder12:57
arne_ris there al list of who participated in the PTGs, or who wants to be in the next one?12:57
*** mvk has joined #openstack-cinder13:00
*** ociuhandu has quit IRC13:01
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Fall back to full clone if extend needed  https://review.openstack.org/42768313:02
*** nikeshm has quit IRC13:02
*** wN has quit IRC13:02
*** Jack_Iv has joined #openstack-cinder13:03
*** knomura has quit IRC13:04
*** knomura has joined #openstack-cinder13:04
*** cdelatte has quit IRC13:05
*** wN has joined #openstack-cinder13:07
*** wN has quit IRC13:07
*** wN has joined #openstack-cinder13:07
*** bapalm has quit IRC13:07
*** cdelatte has joined #openstack-cinder13:07
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Use full clone if extend needed  https://review.openstack.org/42768313:08
*** Jack_Iv has quit IRC13:10
*** bapalm has joined #openstack-cinder13:12
*** shausy has quit IRC13:12
*** mtanino has quit IRC13:13
*** ociuhandu has joined #openstack-cinder13:16
*** pgadiya has quit IRC13:21
*** akapil has joined #openstack-cinder13:21
*** bapalm has quit IRC13:24
*** jordanP has quit IRC13:27
smcginnisarne_r: This is a the first PTG, so it's all new.13:28
*** jordanP has joined #openstack-cinder13:28
smcginnisarne_r: Last I heard we were close to registration being full. Something like 550-600 people.13:28
openstackgerritMerged openstack/cinder: Revert "Mark the sheepdog driver as unsupported"  https://review.openstack.org/42558613:29
jungleboyjsmcginnis, We have been talking about that a lot at Lenovo.  Really wonder how this is going to go.13:29
jungleboyjOooh, Sheepdog is back in huh?13:30
smcginnisjungleboyj: Yep, yep. :)13:30
*** alonmarx has joined #openstack-cinder13:31
jungleboyjsmcginnis, Plus have you seen that they have made the statement that they are not doing events in the US due to the plitical crap.13:32
smcginnisjungleboyj: Yeah, after Boston, at least the next three summits are going to be international.13:32
smcginnisjungleboyj: I wonder if that means Australia, Canada, somewhere-in-Europe.13:33
smcginnisjungleboyj: Though south would be nice. Haven't gone that way yet. ;)13:33
jungleboyjsmcginnis, That would be my guess.  I hope Canada is still taking refugees.13:33
jungleboyjBrazil?13:33
*** gouthamr has joined #openstack-cinder13:34
smcginnisjungleboyj: That could work. We have a few contributors there I know of.13:34
*** Jack_Iv has joined #openstack-cinder13:34
jungleboyjsmcginnis, I am surprised we haven't done South America yet.13:34
arne_rsmcginnis: Any chance that the registration list for the PTG is public?13:35
dulekarne_r: I don't think so, but maybe diablo_rojo can give you some limited info if you need it.13:35
smcginnisarne_r: I highly doubt it.13:36
dulekgeguileo: Vitrage gate is blocked until bug 1660928 gets fixed. I think it's worth it to just hit recheck on the patch and solve the unit tests problem in another patch.13:36
openstackbug 1660928 in Cinder "Backwards compatibility for oslo messaging in Cinder" [Undecided,In progress] https://launchpad.net/bugs/1660928 - Assigned to Gorka Eguileor (gorka)13:36
*** dustins has joined #openstack-cinder13:37
*** knomura_ has joined #openstack-cinder13:37
arne_rdulek: That would probably be a bit overkill. I am just curious if IBM folks will be present at the cinder-PTG13:37
jungleboyjarne_r, They will be.  Why do you ask?13:38
*** knomura has quit IRC13:39
*** gaurangt has left #openstack-cinder13:40
*** eharney has quit IRC13:41
dulekgeguileo: Uh-oh, I've just reproduced this recursion problem locally.13:41
*** gaurangt has joined #openstack-cinder13:41
arne_rjungleboyj: to put it diplomatic: my boss is considering wether he can afford to send guys from our team to the summit, and if other IBM teams do so as well, that would be an incentive13:42
*** ducttape_ has quit IRC13:42
erlonildikov: jgriffith: shouldn't the attach APIs receive the volume name as optional? Im having this error: ERROR (BadRequest): Invalid input for field/attribute volumeId. Value: v1. u'v1' is not a 'uuid' (HTTP 400) (Request-ID: req-204b6b47-c6ce-4826-a87a-b3d961423c86)13:43
*** mriedem has quit IRC13:44
dulekarne_r: FYI: Eventbrite says that there are only 18 tickets left.13:46
openstackgerritMerged openstack/cinder: 3PAR: Inconsistency in copied and source CG  https://review.openstack.org/41933513:48
arne_rdulek: that's probably why i was asked to learn it today :p13:49
*** gaurangt has quit IRC13:51
*** gaurangt has joined #openstack-cinder13:54
*** tongli has joined #openstack-cinder13:55
*** kfarr has joined #openstack-cinder14:01
jungleboyjarne_r, Yes, there will be a number of people from the Cloud Division there.14:02
jungleboyjhemna and scottda I believe are both going.14:02
arne_rjungleboyj: good to hear :)14:03
arne_rdo you also know from people of the systems group?14:04
jungleboyjAh, I know that there were a number that were planning to go.  I am pretty sure that people from under Anne Funai will be going.14:05
jungleboyjPing Joe Cropper or Gerald McBrearty for details on that side of the org.14:05
jungleboyjAFK for a bit.  Need to get a boy to school.14:05
*** tlian has joined #openstack-cinder14:07
*** dustins has quit IRC14:08
*** arne_r has quit IRC14:08
*** arne_r has joined #openstack-cinder14:09
arne_rjungleboyj: my client crashed, could you resend your last message?14:09
*** ducttape_ has joined #openstack-cinder14:09
jungleboyjarne_r, Ping Joe Cropper or Gerald McBrearty for details on that side of the org.14:11
arne_rthank you very much :)14:11
smcginnisdulek, geguileo: Mind taking care of this one? https://review.openstack.org/#/c/427542/14:11
*** xyang1 has joined #openstack-cinder14:13
smcginnisdulek: Thank you.14:14
duleksmcginnis: It's funny we haven't noticed earlier.14:14
openstackgerritMichal Dulko proposed openstack/cinder: Make get_capabilities look for clusters  https://review.openstack.org/42772414:14
smcginnisdulek: I know! :)14:15
geguileosmcginnis: Sorry, I was out of my desk and forgot to set it to away14:15
geguileodulek: I was able to reproduce it locally when I wrote the comment, I'll fix it after lunch14:15
geguileodulek: That's how I know it's an interdependency, because it's caused by a specific order14:16
geguileodulek: I need to pinpoint the exact tests that are interacting14:16
*** mriedem has joined #openstack-cinder14:16
*** xyang has joined #openstack-cinder14:17
smcginnisgeguileo: Hah, no problem.14:18
smcginnisgeguileo: I never set mine to away. Keep people guessing if I'm actually here or not. :D14:18
duleksmcginnis: It might be even better to keep it set to away all the time… :D14:19
smcginnisdulek: Hah, there you go. ;)14:20
*** eharney has joined #openstack-cinder14:22
*** ducttape_ has quit IRC14:22
*** ducttape_ has joined #openstack-cinder14:22
*** ducttape_ has quit IRC14:23
*** liverpooler has joined #openstack-cinder14:24
*** porrua has joined #openstack-cinder14:24
*** mkoderer has joined #openstack-cinder14:24
*** cdelatte has quit IRC14:25
*** gaurangt has left #openstack-cinder14:26
xyanggeguileo: hi, we got this error in CI's:  ERROR oslo_db.sqlalchemy.exc_filters [-] DBAPIError exception wrapped from (pymysql.err.ProgrammingError) (1064, u"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(6)' at line 1") [SQL: u'\nALTER TABLE workers CHANGE COLU14:26
xyangMN updated_at updated_at DATETIME(6)']14:26
xyanggeguileo: any idea what might be the problem?14:26
*** ducttape_ has joined #openstack-cinder14:28
*** gaurangt has joined #openstack-cinder14:30
*** ducttape_ has quit IRC14:31
xyanggeguileo: upgrade to mysql > v5.5?14:32
*** alonmarx has quit IRC14:32
*** crushil has joined #openstack-cinder14:32
*** gaurangt has left #openstack-cinder14:33
scottdalyarwood: regarding Nova saving connection_info after calling connect_volume: is that during Nova live migration? It seems I recall doing that to keep track of old_connection while creating new_connection.14:33
scottdalyarwood: I think saving that info prevents some trickery to get the old info after the new connection is made.14:34
xyanggeguileo: looking at your comments in the migration script, sounds like this error can be safely ignored?  v5.5 should still work?14:34
*** gaurangt has joined #openstack-cinder14:36
lyarwoodscottda: it was during swap_volume, turns out that nova is stashing the device['path'] that os_brick returns in the connection_info14:40
lyarwoodscottda: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/iscsi.py#L6714:40
scottdalyarwood: OK, I *believe* that's only called for a Cinder migration to a new backend, if the volume is attached...14:43
scottdalyarwood: So, maybe a similar situation, to save the old_connection info and use it after the new connection is made, in order to call terminate_connection on cinder properly.14:44
scottdalyarwood: But I'm not certain of that.14:44
lyarwoodscottda: we can also call it via `nova volume-update` that switches two volumes, it isn't an issue really, we were just double saving the connection_info on the Nova side for no reason14:45
lyarwoodscottda: do you know if retyping volumes multiple times is supported btw?14:45
lyarwoodscottda: it's continously breaking for me with the second attempt14:46
*** dave-mccowan has joined #openstack-cinder14:47
scottdalyarwood: It seems I recall a bug for that...not sure if it has been fixed...just a minute..14:47
dulekgeguileo: Okay, so here's another fun tempest assumption: https://github.com/openstack/tempest/blob/9e6ee394184aedc777226b0723e5fe16d2b33ebe/tempest/api/volume/admin/v2/test_volume_pools.py#L29-L3414:49
dulekgeguileo: I thought I could fix this by looking for a cluster_name in volume-show output, but that's missing from there.14:50
dulekgeguileo: Maybe this test should call service-show to decide if it should pass cluster_name or host to get-pools?14:50
scottdalyarwood: Old bug, it looks like it was fixed: https://bugs.launchpad.net/nova/+bug/149023614:52
openstackLaunchpad bug 1490236 in OpenStack Compute (nova) "Swapping volume can't been swap again" [Low,Fix released] - Assigned to Takashi NATSUME (natsume-takashi)14:52
lyarwoodscottda: sweet, thanks14:52
lyarwoodscottda: ah that's the one we reverted14:53
*** guyr-infinidat has joined #openstack-cinder14:53
lyarwoodscottda: introduced a regression14:53
scottdalyarwood: Yeah14:53
*** cdelatte has joined #openstack-cinder14:53
scottdaFYI, we've introduced new APIs for Cinder attach/detach in Ocata. Trying to improve situations like this..14:54
scottdalyarwood: With new APIs, Cinder will keep track of connection info, so less burden on Nova (or whomever) to track all that.14:54
scottdastay tuned...14:55
*** dustins has joined #openstack-cinder14:55
lyarwoodscottda: yeah that's cool, looking forward to using them in Pike :)14:56
geguileoxyang: Yes v5.5 will still work15:04
xyanggeguileo: ok, thanks15:05
geguileoxyang: But That exception should have been swallowed by the exception catching...15:05
*** liverpooler has quit IRC15:05
geguileoxyang: That happened during migration 85?15:06
geguileodulek: I think we should be returning the value of cluster15:06
geguileodulek: I saw it yesterday15:06
*** arne_r has quit IRC15:06
xyanggeguileo: I have not got the full logs yet.  I only saw a section of it15:06
geguileodulek: But I haven't had time to add it15:06
xyanggeguileo: by looking at the migration script, it should be from 8515:07
geguileoxyang: I'd like to see it to be sure15:07
xyanggeguileo: sure.  I have asked for it.  will let you know15:07
geguileoxyang: thanks15:07
xyanggeguileo: np15:07
dulekgeguileo: Yup, I have fun with microversions now. It's v2 API tests that are failing, so I don't really have any info about clusters available.15:08
*** liverpooler has joined #openstack-cinder15:08
*** tongli has quit IRC15:09
geguileodulek: I'm having troubles running devstack, but I want to check one thing in case there's a workaround...15:09
*** jperry has joined #openstack-cinder15:11
dulekgeguileo: I don't have an idea how to solve this, besides removing any asserts from the test.15:18
*** cyberik has joined #openstack-cinder15:18
*** akapil has quit IRC15:26
*** liverpooler has quit IRC15:30
*** akapil has joined #openstack-cinder15:33
*** breitz has quit IRC15:33
*** mdnadeem has quit IRC15:33
*** breitz has joined #openstack-cinder15:34
*** ducttape_ has joined #openstack-cinder15:34
*** ducttape_ has quit IRC15:34
*** ducttape_ has joined #openstack-cinder15:35
ildikoverlon: jgriffith: is volume name unique?15:37
jungleboyjildikov, The display name?15:39
ildikovjungleboyj: just got a question earlier whether the new attach API should receive the volume name or not15:41
ildikovjungleboyj: does the volume have other name than the display name?15:42
jungleboyjildikov, The volume name is not unique.  Just the UUID is.  You can have multiple volumes with the same name or no name at all.15:42
ildikovjungleboyj: tnx, that's what I thought15:42
jungleboyjildikov, The 'display name' is the only name I am aware of.15:42
jungleboyjildikov, You are welcome.  :-)15:42
openstackgerritJustin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi  https://review.openstack.org/42539215:43
ildikovjungleboyj: haha, cool, thanks :)15:43
jungleboyjnp15:44
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Cloned volume with different volume type  https://review.openstack.org/42687815:44
ildikoverlon: jgriffith: so I don't have a strong preference on the name question, although I don't really see that would add too much value to the API as we wouldn't use it to identify the volume anyhow15:44
*** liverpooler has joined #openstack-cinder15:44
xyanggeguileo: http://publiclogs.emc.com/xio_ostack/EMC_XtremIO_FC/1444/logs/devstacklog.txt.gz15:45
jungleboyjildikov, Is there anything helpful that it could be used for?15:45
xyanggeguileo: looks like it only happened during migration 8515:45
ildikovjungleboyj: I don't see anything at this point honestly15:45
jungleboyjWas there an explanation for the request?  :-)15:46
ildikovjust the question on this channel earlier15:47
jungleboyjI suppose it could be used in Logging which could be helpful.15:48
geguileoxyang: Yes that can be ignored15:49
xyanggeguileo: thanks!15:49
geguileoxyang: We are catching the exception (because we anticipate it can fail) so it doesn't break the migration15:49
geguileoxyang: But SQLAlchemy still logs the error15:50
xyanggeguileo: sure.  I can see in the logs it actually continues with 86, 87, ...15:50
geguileoxyang: The only solution to prevent it from displaying would be to disable the SQLA logging durint that call15:50
*** rcernin has quit IRC15:51
*** liverpooler has quit IRC15:51
xyanggeguileo: sure15:51
*** liverpooler has joined #openstack-cinder15:53
*** mtanino has joined #openstack-cinder15:54
*** ircuser-1 has joined #openstack-cinder15:56
hemnasmcginnis, http://paste.openstack.org/show/597113/15:59
hemnadidn't know if you saw that yesterday15:59
*** sv7577 has joined #openstack-cinder15:59
scottdasmcginnis: I just updated the meeting agenda , FYI15:59
smcginnisscottda: ack16:00
smcginnishemna: Looks great!16:00
*** lkuchlan_ has quit IRC16:01
openstackgerritDanny Al-Gaaf proposed openstack/cinder: doc: verify all rst files  https://review.openstack.org/31059116:04
*** stvnoyes has quit IRC16:05
*** stvnoyes has joined #openstack-cinder16:06
*** sv7577 has quit IRC16:07
openstackgerritJustin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi  https://review.openstack.org/42539216:09
*** _ducttape_ has joined #openstack-cinder16:09
erlonildikov: jgriffith: that is just the behaviour of most of Cinder commands, so, users expect consistency16:10
*** JoseMello has joined #openstack-cinder16:10
openstackgerritJustin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi  https://review.openstack.org/42539216:10
*** ducttape_ has quit IRC16:13
ildikoverlon: jgriffith: if there's a good use of that attribute I'm not against; although this one is a completely new API, therefore if we see it completely unnecessary we might choose not to add it16:13
ildikoverlon: jgriffith: currently the only thing we might use is logging, but as it's not unique the UUID is still needed16:14
*** sv7577 has joined #openstack-cinder16:15
erlonildikov: jgriffith: the clients (not sure if is really the clients who check that) try to do an operation and if the name is unique for the tenant, the operation succeeds, its not only for logging, its for convenience I believe16:16
*** sv7577 has quit IRC16:17
erlonildikov: jgriffith: if the checking is done in the API, so that is something we should work on as OSC heavily rely on that to keep consistency with the current behaviour16:17
*** windmoon has joined #openstack-cinder16:20
pjm6Hi there16:22
pjm6anyone here tried to run cinder in a Dell EqualLogic? I'm getting an error: "cli-settings confirmation off"16:22
*** windmoon has quit IRC16:22
pjm6Bad or unexpected resposne from the storage volume backend API: SSH Command failed after '6' attemtps: 'cli-settings confirmation off'16:22
pjm6if I do this in the Dell via SSH i have no problems16:23
*** diablo_rojo_phon has joined #openstack-cinder16:24
*** alonma has quit IRC16:24
smcginnispjm6: Our meeting is taking place right now, but I might be able to help, or at least get you going in the right direction, in a little while.16:24
*** alonma has joined #openstack-cinder16:25
pjm6thanks smcginnis :)16:25
pjm6I think that just found the problem, I wasn't using the default Group name16:25
pjm6and I did specified it, I just corrected and that error gone, will try to see if its working :)16:26
smcginnispjm6: Great!16:27
*** xinli has joined #openstack-cinder16:27
*** alonma has quit IRC16:29
pjm6so far, so good :) it seems working smcginnis16:31
jgriffithildikov erlon huh?16:31
jgriffithNames are not unique, and should be banned if you ask me :)16:31
jgriffithUUID's FTW :)16:31
ildikovjgriffith: +1 :)16:32
smcginnispjm6: Excellent! CI jobs run on that, so should be good. Feel free to ask if anything comes up though.16:32
jgriffithbut I'm not following the conversation, and my scroll back seems truncated16:32
erlonjgriffith: O_o how that came to exist so far then?16:32
* ildikov is not user friendly this morning :)16:32
jgriffitherlon how what came to exist?16:32
erlonjgriffith: calling resources by their names16:32
jgriffitherlon don't16:33
jgriffithpeople whined and added that in but it's bad16:33
openstackgerritGorka Eguileor proposed openstack/cinder: Fix notification short-circuit  https://review.openstack.org/42764716:33
openstackgerritGorka Eguileor proposed openstack/cinder: Fix CopyVolumeToImageTestCase max recursion depth  https://review.openstack.org/42780016:33
jgriffithbecause we don't require/guarantee unique names and the whole thing sucks16:33
jgriffithjust adds confusion16:33
erlonjgriffith: from a UPOV its a good idea16:33
pjm6smcginnis: thank you :)16:33
jgriffithI don't know what UPOV is16:33
geguileodulek: ^ That's the fix for the mas recursion error16:34
jgriffithI don't even know what your'e asking still :)16:34
erlonjgriffith: see? you know nothing about user experience :)16:34
erlonjgriffith: jgriffith: user point of view16:34
jgriffithbut historically I will lose any argument along the lines of "banning names" :)16:34
dulekgeguileo: I've already opened it. :)16:35
jgriffitherlon Yeah, I just actually talk to them instead of sitting on panels or making up silly groups and mettings of OS devs that talk about them and work in a vacuum16:35
geguileodulek: thanks16:35
erlonjgriffith: but seriously, we should keep consistency as much as possible with the behaviour of the other commands of the CLI16:35
ildikoverlon: well, in smaller environments if users are careful enough to choose unique names it might make life easier, but anything different than that I agree with jgriffith here16:35
erlonusers rely on that, it was the first thing I notice using the new command16:36
jgriffitherlon hoenstly if you want to go names, go names I don't care any more; but you need to add uniqueness requirements if you're going to do it16:36
jgriffitherlon and yes, what we have now thanks to a certain individual SUCKS because some Client things allow it, some don't16:36
jgriffithand that's crappy16:36
ildikovjgriffith: or an extra check that fails when the name is not unique and then the user can retry with the UUID16:37
erlonildikov: even in big deployments, some tenants rely in the names to better management of their resources16:37
jgriffitherlon still unclear what you're proposing16:38
erlonildikov: +1  thats how its done so far16:38
jgriffitherlon you want to convert everything to use names instead of UUIDs?16:38
*** tongli has joined #openstack-cinder16:38
erlonjgriffith: no no, you can still use the UUID but if the user passes volume-name and its is unique it needs to work16:39
ildikovjgriffith: to try to use volume name as an alternative to UUID to identify the volume, so the user does not have to look up the UUID, but use the nice and shiny names16:39
jgriffitherlon yeah you mean if they pass it instead of UUID?16:39
*** jdurgin1 has joined #openstack-cinder16:40
erlonjgriffith: yes16:40
ildikovjgriffith: so UUID stays, but if only the name is specified, look up the volumes by name, if you found multiple or none you fail, if one the request goes as if you would've had UUID specified there16:40
erlonjgriffith: try: cinder create 1 --name v116:40
erlonjgriffith: then cinder delete v116:40
*** alonma has joined #openstack-cinder16:40
jgriffitherlon I don't like it but whatever16:41
erlonildikov: jgriffith, yes I believe by the way that there is already a function on cinder that does that16:41
jgriffitherlon keep in mind you're bighting off a pretty large chunk of work16:41
*** nikeshm has joined #openstack-cinder16:41
jgriffithsnapshots, backups, volume-types, consistency-groups etc etc16:41
jgriffitherlon there's a function intended to do it; and it's half done work16:41
jgriffithso would be cool to finish it or remove it (I'm guessing the vote is finish it)16:42
jgriffitherlon FYI, I was AWS a good bit and have never had an issue with the fact that my resources don't have "names" to access them by16:42
jgriffithin fact I prefer it16:43
jgriffiths/was/use/16:43
ildikovwe can report a bug for this and add wishlist or higher priority, cores here can decide16:43
jgriffithand I'm not sure, but last time I checked there are a few people using that and have a pretty descent UPOV16:43
*** knomura_ has quit IRC16:43
nikeshmhi do any obe face CI failures due to this https://review.openstack.org/#/c/427561/16:43
nikeshms/obe/one16:44
erlonjgriffith: its not the end of life if you dont have this at all, but have in half of the commands and not the others is a big problem16:44
jgriffitherlon I'll agree with you on that16:44
*** alonma has quit IRC16:45
erlonabout the chunk of work, that actually already works for snapshots, types, volumes, most of stuff16:46
erlonjgriffith: not sure about the consistency groups16:46
*** knomura has joined #openstack-cinder16:47
ildikoverlon: jgriffith: my vote is on creating a bug report to document this and then we can decide when to get it done16:49
ildikoverlon: jgriffith: unless either of you wants to look into it now16:49
*** alonmarx has joined #openstack-cinder16:50
jgriffithildikov seems reasonable to me16:50
*** tesseract has quit IRC16:56
*** knomura has quit IRC16:56
*** alonma has joined #openstack-cinder16:56
*** diablo_rojo has joined #openstack-cinder16:59
*** tongli has quit IRC17:00
*** psachin has quit IRC17:01
*** alonma has quit IRC17:01
potssmcginnis i think you'd suggested you would revert the patch depending on how many CIs we got running before the freeze17:01
potssmcginnis i can submit a patch though, if you like17:01
smcginnispots: Last I recall, I had said submitting a patch for the one that is reporting would be accetable.17:02
smcginnispots: But the bigger issue here is there's going to need to be a new driver submitted for Pike, because the inherited DotHill one will be removed.17:02
smcginnisIf CI isn't fixed on that very very soon.17:02
potssmcginnis the other CIs are very close, one is running in the sandbox now17:03
smcginnispots: OK, great.17:04
potssmcginnis but there is probably no more need for a "dot hill" branded driver since that company was acquired17:04
jungleboyjpots, So, what is the status on the Lenovo CI?17:04
potssmcginnis so we need to remove the documentation, but 99.9% of the driver has to remain to support the other drivers.17:05
potsthe Lenovo CI is running against the sandbox but I'm having trouble with the FC passthrough17:05
potsi think the docs are out of date for that?17:05
openstackgerritJustin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi  https://review.openstack.org/42539217:06
potsSo I'd really like to keep the Lenovo driver in the 'supported' column since we're committed to getting the CI up and obviously to supporting the driver17:06
* jungleboyj would like that too. :-)17:07
jungleboyjpots, Can we at least start reporting the iSCSI results?17:07
potsyes, i can get that going today17:08
*** e0ne has quit IRC17:08
*** guyr-infinidat has quit IRC17:08
*** jordanP has quit IRC17:08
jungleboyjsmcginnis, Thoughts?17:09
*** kfarr has quit IRC17:09
potssmcginnis sorry, i put too much stock in the fact that the drivers are all identical and didn't realize how much work it would be to duplicate the same CI with the latest OS release17:10
smcginnisjungleboyj: We can get the one marked supported, but we need to see CI on the rest.17:11
erlonildikov: jgriffith: ok: https://bugs.launchpad.net/cinder/+bug/166104317:11
openstackLaunchpad bug 1661043 in Cinder "Cinder attachment-* commands does not support names" [Undecided,New]17:11
smcginnisbbl17:11
jungleboyjsmcginnis, Which one?17:12
openstackgerritSzymon Wróblewski proposed openstack/cinder: [WIP][PoC] RCPAPI microversion constants  https://review.openstack.org/42781717:13
*** nkrinner is now known as nkrinner_afk17:14
*** akapil has quit IRC17:14
*** akapil has joined #openstack-cinder17:14
*** lpetrut has quit IRC17:17
*** ggrabows has joined #openstack-cinder17:17
*** akapil_ has joined #openstack-cinder17:19
hemnasmcginnis, https://github.com/hemna/third-party-ci-tools/tree/hemna/monitoring/lastcomment-scoreboard17:20
*** akapil has quit IRC17:21
jungleboyjhemna, Nice.17:22
erlonsmcginnis: Sean, can you give a look on this when you have a change? https://review.openstack.org/#/c/414076/17:24
ildikoverlon: tnx17:25
*** akapil_ has quit IRC17:25
hemnasoo......17:26
hemnaI just ran the generate_driver_list17:27
geguileodulek: How about adding an attribute to show command like we have now the volume extension volume_host_attribute but for cluster?17:27
hemnaand I'm seeing some drivers not having a CI_WIKI_NAME associated with them17:27
nikeshm hi do any one face CI failures due to this https://review.openstack.org/#/c/427561/17:28
nikeshmi am facing it in Kaminario CI17:29
nikeshmpots: you might try nova passthrough instead of passthrough using scripts17:30
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: ZTE: Added CI_WIKI_NAME  https://review.openstack.org/42782317:30
nikeshmwhen i was maintaining those CIs, passthrough was failing for higher versions of Ubuntu17:31
erlongeguileo: dulek: guys, can you give a look in those changes when you have a chance? The migration-matrix is only failing on ceph (which apparently does not support attached migration): https://review.openstack.org/#/q/status:open+project:openstack/cinder+branch:master+topic:fix-migration-bugs17:31
nikeshmwith the passthrough scripts17:31
erlonildikov: np17:31
hemnanikeshm, the FC passthrough was failing ?17:32
*** Zack2 has left #openstack-cinder17:32
nikeshmhemna: when i was trying seting up CI with FC passthrough scripts (more than one year back), i faced issues with higher versions of Ubuntu17:36
*** raunak has joined #openstack-cinder17:36
nikeshmnot failing17:37
*** raunak has quit IRC17:37
nikeshmpassthrough did not work17:37
nikeshmfor me that time17:37
*** ericyoung has joined #openstack-cinder17:39
hemnaI think we ran into that problem at HPE too for a while.17:40
*** ericyoung has quit IRC17:45
*** ericyoung has joined #openstack-cinder17:45
openstackgerritSilvan Kaiser proposed openstack/cinder: Removes getfattr from Quobyte Cinder driver  https://review.openstack.org/42783317:45
*** ericyoung has quit IRC17:47
*** ericyoung has joined #openstack-cinder17:48
*** ericyoung has quit IRC17:49
*** ericyoung has joined #openstack-cinder17:49
*** esker has joined #openstack-cinder17:50
*** cyberik has quit IRC17:52
*** _ducttape_ has quit IRC17:54
*** sv7577 has joined #openstack-cinder17:54
*** ducttape_ has joined #openstack-cinder17:54
*** kaisers has quit IRC17:54
*** ducttape_ has quit IRC17:54
*** ducttape_ has joined #openstack-cinder17:55
*** ducttape_ has quit IRC17:55
*** ducttape_ has joined #openstack-cinder17:56
*** gaurangt has left #openstack-cinder17:56
*** aarefiev is now known as aarefiev_afk17:57
*** sv7577 has quit IRC18:01
*** harlowja_ has joined #openstack-cinder18:05
*** harlowja has quit IRC18:06
*** mkoderer has quit IRC18:06
openstackgerritMerged openstack/cinder: Move release notes to correct location  https://review.openstack.org/42754218:07
*** Apoorva has joined #openstack-cinder18:10
*** xyang has quit IRC18:12
*** Jack_Iv has quit IRC18:12
*** kaisers has joined #openstack-cinder18:13
jgriffithscottda umm, has anybody even tried running tempest tests with the Cinder API endpoint set to V3?18:14
jgriffithscottda it's kinda ugly18:14
*** xyang has joined #openstack-cinder18:15
*** lpetrut has joined #openstack-cinder18:16
jgriffithscottda I think I'm going to give up on this honestly18:17
geguileosmcginnis: If you have time for a couple of easy reviews: spureous erros in unit tests (https://review.openstack.org/#/c/427800/), and this one has broken vitrage CI (https://review.openstack.org/#/c/427647/)18:17
smcginnishemna: Filtered out to the non-reporting CIs that we have not aleady marked nonsupported.18:17
smcginnishemna: http://paste.openstack.org/show/597255/18:17
smcginnisgeguileo: Sure, I can take a look now.18:17
smcginnisjgriffith: Bloodbath?18:17
geguileosmcginnis: Awesome!, Thanks!18:17
smcginnisgeguileo: Oh good, had meant to get back to checking on the short circuit one.18:18
*** xyang has quit IRC18:18
*** dhinesh has joined #openstack-cinder18:19
geguileosmcginnis: patrickeast Thanks!18:19
geguileopatrickeast: This one already had your +2 before I changed the commit message: https://review.openstack.org/#/c/424747/18:20
geguileo;-)18:20
*** diablo_rojo has quit IRC18:21
geguileopatrickeast: thanks!18:22
patrickeastgeguileo: np18:22
hemnasmcginnis, cool, that's a short list18:23
hemnawhich is good18:23
smcginnishemna: Yes, much better than I had originally feared.18:24
*** diablo_rojo has joined #openstack-cinder18:24
*** raunak has joined #openstack-cinder18:24
*** xyang has joined #openstack-cinder18:27
potsnikeshm: thanks, i've tried both so far, not sure how to troubleshoot the nova passthrough18:27
potshemna: do you remember how it was solved at HPE?18:28
*** pcaruana has quit IRC18:28
*** xyang has quit IRC18:29
*** xyang has joined #openstack-cinder18:29
hemnaI think I had rewritten most of the FC passthrough stuff18:30
hemnathe problem was that it wasn't correctly unbinding the HBA18:30
hemnaI don't have access to any of that code now, unless asselin__ put it upstream18:30
asselin__it's all in the 3rd party ci tools repo18:32
potshemna: i think i tried those scripts but they didn't work for me and i couldn't find any doc to double-check my work.  we use a version of them on the HP MSA CI successfully but that's an older version of everything.18:32
hemnaasselin__, ok that's what I was hoping for18:32
asselin__http://git.openstack.org/cgit/openstack/third-party-ci-tools/tree/provisioning_scripts/fibre_channel18:32
potsis anyone using the (current?) method of FC pass-through in Nova?18:33
asselin__but....openstack nova *should* support this now out of the box.18:33
asselin__those scripts were used back in kilo18:33
*** sv7577 has joined #openstack-cinder18:34
smcginnispatrickeast: Aren't you doing something with this? ^^18:35
smcginnisFor those interested - some old reviews sitting out there: http://cinderstats-dellstorage.rhcloud.com/cinder-openreviews.html18:35
potsasselin__: i followed the PCI passthrough instructions here: http://docs.openstack.org/admin-guide/compute-pci-passthrough.html18:36
asselin__looks promising18:37
sv7577Can someone review https://review.openstack.org/#/c/407123/18:38
openstackgerritMerged openstack/cinder: Reduced the complexity of the main() method  https://review.openstack.org/25091418:47
*** sv7577 has quit IRC18:51
*** mvk has quit IRC18:57
*** mjura has quit IRC18:58
*** alonma has joined #openstack-cinder18:58
*** ksumit has joined #openstack-cinder18:59
*** yumiriam has quit IRC19:00
jungleboyjOh, look at that pots you are already talking to asselin__ :-)19:00
jungleboyjSo, talk to asselin__ :-)19:01
*** ducttape_ has quit IRC19:02
*** alonma has quit IRC19:02
*** ducttape_ has joined #openstack-cinder19:03
*** alonma has joined #openstack-cinder19:04
openstackgerritRohan Arora proposed openstack/cinder: Nosec audit url open issue from Bandit  https://review.openstack.org/42435619:06
*** knomura has joined #openstack-cinder19:08
*** alonma has quit IRC19:08
*** lpetrut has quit IRC19:08
*** adisky_ has quit IRC19:09
*** lpetrut has joined #openstack-cinder19:11
*** Jack_Iv has joined #openstack-cinder19:12
*** knomura has quit IRC19:13
*** kfarr has joined #openstack-cinder19:13
openstackgerritAdriano Freires Rosso proposed openstack/cinder: Hitachi VSP: Specify compute nodes and copy ports  https://review.openstack.org/41407619:15
*** Jack_Iv has quit IRC19:17
openstackgerritTom Swanson proposed openstack/cinder: Dell EMC: Moved SC driver under dell_emc  https://review.openstack.org/42743519:19
*** harlowja_ has quit IRC19:19
openstackgerritMitsuhiro Tanino proposed openstack/python-cinderclient: Add --metadata option to API v2 cinder list command again  https://review.openstack.org/42787519:25
patrickeastasselin__: pots: yea the nova pci passthrough is working pretty well for me and the Pure CI, we still have some problems but its mostly switch related19:25
*** ggrabows has quit IRC19:25
*** ggrabows has joined #openstack-cinder19:25
patrickeasti definitely recommend doing that over using the passthrough scripts19:25
openstackgerritAdriano Freires Rosso proposed openstack/cinder: Hitachi VSP: Specify compute nodes and copy ports  https://review.openstack.org/41407619:26
openstackgerritTom Swanson proposed openstack/cinder: Dell EMC: Moved SC driver under dell_emc  https://review.openstack.org/42743519:29
*** e0ne has joined #openstack-cinder19:30
*** ggrabows has quit IRC19:31
*** ociuhandu has quit IRC19:33
*** mvk has joined #openstack-cinder19:34
*** akapil has joined #openstack-cinder19:36
*** alonmarx has quit IRC19:36
*** xyang has quit IRC19:37
*** Apoorva has quit IRC19:38
*** xyang has joined #openstack-cinder19:39
*** esker has quit IRC19:40
*** ggrabows has joined #openstack-cinder19:40
*** akapil has quit IRC19:41
*** ggrabows has quit IRC19:43
*** ggrabows has joined #openstack-cinder19:44
*** ChubYann has joined #openstack-cinder19:45
openstackgerritChris M proposed openstack/cinder: Add SUPPORTED flag to HP MSA driver  https://review.openstack.org/42788119:46
*** ggrabows_ has joined #openstack-cinder19:49
*** ggrabows has quit IRC19:50
*** viks has joined #openstack-cinder19:51
*** mtanino has quit IRC19:51
*** ociuhandu has joined #openstack-cinder19:53
*** cknight has joined #openstack-cinder19:53
*** diablo_rojo has quit IRC19:54
*** e0ne has quit IRC19:55
openstackgerritMitsuhiro Tanino proposed openstack/python-cinderclient: Add --metadata option to API v2 cinder list command again  https://review.openstack.org/42787519:55
*** mtanino has joined #openstack-cinder19:59
*** e0ne has joined #openstack-cinder19:59
*** Jack_Iv has joined #openstack-cinder20:00
openstackgerritJustin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi  https://review.openstack.org/42539220:00
openstackgerritChris M proposed openstack/cinder: Add SUPPORTED flag to Lenovo iSCSI driver  https://review.openstack.org/42788620:00
openstackgerritTom Swanson proposed openstack/cinder: Dell EMC: Moved SC driver under dell_emc  https://review.openstack.org/42743520:01
potspatrickeast: thanks.  do you ever see "Claim pci failed" errors when trying to pass through a device?  Do you know what the disposition of the PCI device needs to be before Nova grabs it (e.g. bound to a stub driver or unbound)?20:03
*** gouthamr has quit IRC20:03
*** crushil has quit IRC20:06
*** crushil has joined #openstack-cinder20:06
*** mtanino has quit IRC20:09
*** mtanino has joined #openstack-cinder20:12
*** ericyoung has quit IRC20:13
*** e0ne has quit IRC20:17
*** crushil has quit IRC20:20
*** crushil has joined #openstack-cinder20:20
*** diablo_rojo_phon has quit IRC20:20
openstackgerritSean McGinnis proposed openstack/cinder: Clean up release notes for Ocata  https://review.openstack.org/42789620:20
*** porrua has quit IRC20:21
patrickeastpots: I haven't seen that particular error before, but as far as state goes you just need to make sure that it *can* be stubbed out20:21
patrickeastpots: for mine, when something bad happens I just re-bind them all to their qla2xxx drivers20:21
*** e0ne has joined #openstack-cinder20:21
openstackgerritSean McGinnis proposed openstack/cinder: Clean up release notes for Ocata  https://review.openstack.org/42789620:21
patrickeastpots: and nova/libvirt take care of it from there20:21
potspatrickeast: thanks20:22
*** nikeshm has quit IRC20:26
*** timcl has quit IRC20:28
*** e0ne has quit IRC20:28
*** jamielennox|away is now known as jamielennox20:30
*** e0ne has joined #openstack-cinder20:32
*** timcl has joined #openstack-cinder20:33
*** JoseMello has quit IRC20:35
openstackgerritSean McGinnis proposed openstack/cinder: Clean up release notes for Ocata  https://review.openstack.org/42789620:36
*** e0ne has quit IRC20:37
*** harlowja has joined #openstack-cinder20:39
*** e0ne has joined #openstack-cinder20:41
*** e0ne has quit IRC20:47
*** e0ne has joined #openstack-cinder20:51
*** esker has joined #openstack-cinder20:53
*** e0ne has quit IRC20:55
*** diablo_rojo has joined #openstack-cinder20:55
*** Jack_Iv has quit IRC20:57
*** Jack_Iv has joined #openstack-cinder20:57
*** Jack_Iv_ has joined #openstack-cinder20:58
*** _ducttape_ has joined #openstack-cinder20:58
smcginnispots: Not that you need another thing to worry about, but when you get some time, can you update your CI trigger: https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#How_do_I_trigger_my_CI_to_rerun_on_gerrit_comments.3F20:58
smcginnispots: Otherwise that rechecks the infra tests too. Which then rechecks all CIs that trigger on Jenkins +1.20:59
*** e0ne has joined #openstack-cinder21:00
*** ggrabows_ has quit IRC21:01
potswill do--i think i got it right on the Lenovo CI21:01
*** liverpooler has quit IRC21:01
smcginnispots: Thanks!21:01
*** ggrabows has joined #openstack-cinder21:01
*** ducttape_ has quit IRC21:02
*** Jack_Iv has quit IRC21:02
*** gouthamr has joined #openstack-cinder21:02
openstackgerritSean McGinnis proposed openstack/cinder: Clean up release notes for Ocata  https://review.openstack.org/42789621:03
*** jose-phillips has joined #openstack-cinder21:05
*** diablo_rojo has quit IRC21:11
*** alonmarx has joined #openstack-cinder21:18
*** erlon has quit IRC21:20
*** cknight has quit IRC21:20
*** alonma has joined #openstack-cinder21:20
*** ggrabows has quit IRC21:23
*** alonma has quit IRC21:25
*** eharney has quit IRC21:28
*** erlon has joined #openstack-cinder21:30
openstackgerritMerged openstack/cinder: Fix CopyVolumeToImageTestCase max recursion depth  https://review.openstack.org/42780021:30
openstackgerritYucong Feng proposed openstack/cinder: Resolve Brocade HTTPS connection error  https://review.openstack.org/41637121:30
*** kfarr has quit IRC21:32
*** _ducttape_ has quit IRC21:37
*** catintheroof has quit IRC21:37
*** ducttape_ has joined #openstack-cinder21:37
*** catintheroof has joined #openstack-cinder21:37
*** Apoorva has joined #openstack-cinder21:38
*** Apoorva has quit IRC21:40
*** Apoorva has joined #openstack-cinder21:41
*** eharney has joined #openstack-cinder21:44
*** ggrabows has joined #openstack-cinder21:45
*** e0ne has quit IRC21:53
*** catintheroof has quit IRC21:57
*** catintheroof has joined #openstack-cinder21:58
*** diablo_rojo_phon has joined #openstack-cinder22:01
*** alyson_ has quit IRC22:07
*** gouthamr has quit IRC22:07
*** knomura has joined #openstack-cinder22:08
*** ducttape_ has quit IRC22:09
*** ducttape_ has joined #openstack-cinder22:10
*** ksumit has quit IRC22:10
*** knomura has quit IRC22:13
*** ggrabows has quit IRC22:13
*** ducttape_ has quit IRC22:15
*** lpetrut has quit IRC22:19
*** akapil has joined #openstack-cinder22:19
*** akapil has quit IRC22:24
*** nileo has joined #openstack-cinder22:28
*** gouthamr has joined #openstack-cinder22:33
*** ksumit has joined #openstack-cinder22:34
*** mtbar has joined #openstack-cinder22:35
*** catintheroof has quit IRC22:35
*** mtbar has quit IRC22:37
*** Jack_Iv_ has quit IRC22:38
*** crushil has quit IRC22:39
*** viks has quit IRC22:46
*** ducttape_ has joined #openstack-cinder22:47
*** ducttape_ has quit IRC22:49
*** _ducttape_ has joined #openstack-cinder22:49
*** tlian has quit IRC22:51
*** _ducttape_ has quit IRC22:53
*** dustins has quit IRC22:54
*** crushil has joined #openstack-cinder22:57
*** tlian has joined #openstack-cinder22:58
*** alonmarx has quit IRC22:59
hemnasmcginnis, https://github.com/hemna/third-party-ci-tools/tree/hemna/monitoring/lastcomment-scoreboard23:04
*** raunak has quit IRC23:04
hemnamore reworking/refactoring23:04
hemnanew cireporter.py script23:04
hemnathat generates JSON output23:04
smcginnishemna: Sweet23:05
hemnait can dump it to stdout or a file too23:05
smcginnishemna: That should be handy.23:05
hemnathinking of writing a cinder script that runs the generate_driver_list, sucks out the CI names, and then uses that to populate the file needed to kick off the cireporter.py script and output as son.23:05
hemnajson23:05
smcginnisThat really would be ideal. Then we don't have to try to maintain a separate yaml file for things.23:06
hemnayah23:06
smcginnisAdded this to my backlog to try to automate publishing results so anyone can look whenever they want.23:07
hemnathe json output is nice to view in chrome w/ a json formatter23:07
*** jperry has quit IRC23:09
*** tlian2 has joined #openstack-cinder23:10
*** tlian has quit IRC23:13
hemnasmcginnis, http://paste.openstack.org/show/597284/    and JSON   http://paste.openstack.org/show/597285/23:15
*** tlian2 has quit IRC23:15
*** tlian has joined #openstack-cinder23:24
*** alonma has joined #openstack-cinder23:30
*** alonma has quit IRC23:34
*** xyang has quit IRC23:35
*** raunak has joined #openstack-cinder23:43
*** xyang has joined #openstack-cinder23:46
*** s-shiono has joined #openstack-cinder23:47
*** s-shiono_ has joined #openstack-cinder23:50
*** ducttape_ has joined #openstack-cinder23:51
*** s-shiono has quit IRC23:52
*** xyang has quit IRC23:54
*** xyang has joined #openstack-cinder23:57
*** xinli has quit IRC23:58

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