| *** leeantho has quit IRC | 00:02 | |
| *** yhayashi has joined #openstack-cinder | 00:03 | |
| *** yangyapeng has quit IRC | 00:04 | |
| *** suyash has quit IRC | 00:05 | |
| *** IlyaG has joined #openstack-cinder | 00:15 | |
| *** laughterwym has joined #openstack-cinder | 00:18 | |
| *** laughterwym has joined #openstack-cinder | 00:19 | |
| *** IlyaG has quit IRC | 00:23 | |
| *** xingyang has joined #openstack-cinder | 00:24 | |
| *** xingyang has quit IRC | 00:24 | |
| *** sdake_ has joined #openstack-cinder | 00:27 | |
| *** mtanino_ has joined #openstack-cinder | 00:27 | |
| *** mtanino has quit IRC | 00:29 | |
| *** sdake has quit IRC | 00:30 | |
| *** bill_az has quit IRC | 00:33 | |
| *** bardia has quit IRC | 00:36 | |
| *** jamielennox|away is now known as jamielennox | 00:37 | |
| *** diogogmt has joined #openstack-cinder | 00:42 | |
| *** ducttape_ has joined #openstack-cinder | 00:43 | |
| *** ircuser-1 has quit IRC | 00:46 | |
| *** mriedem has joined #openstack-cinder | 00:47 | |
| *** IlyaG has joined #openstack-cinder | 00:54 | |
| *** EinstCrazy has joined #openstack-cinder | 00:54 | |
| *** cheneydc has joined #openstack-cinder | 00:55 | |
| *** alkhodos has quit IRC | 01:05 | |
| *** raunak has quit IRC | 01:13 | |
| *** akerr has quit IRC | 01:13 | |
| *** yangyapeng has joined #openstack-cinder | 01:14 | |
| *** jwcroppe has joined #openstack-cinder | 01:15 | |
| *** yangyape_ has joined #openstack-cinder | 01:16 | |
| *** yangyape_ has quit IRC | 01:16 | |
| *** yangyapeng has quit IRC | 01:16 | |
| *** yangyapeng has joined #openstack-cinder | 01:17 | |
| *** yangyapeng has quit IRC | 01:17 | |
| *** yangyapeng has joined #openstack-cinder | 01:17 | |
| *** Apoorva_ has joined #openstack-cinder | 01:27 | |
| *** krtaylor has joined #openstack-cinder | 01:28 | |
| *** Lee1092 has joined #openstack-cinder | 01:28 | |
| *** krtaylor has quit IRC | 01:30 | |
| *** Apoorva has quit IRC | 01:31 | |
| *** Apoorva_ has quit IRC | 01:32 | |
| *** jamielennox is now known as jamielennox|away | 01:59 | |
| *** mtanino has joined #openstack-cinder | 02:00 | |
| *** Julien-zte has joined #openstack-cinder | 02:01 | |
| *** mtanino_ has quit IRC | 02:02 | |
| *** mtanino has quit IRC | 02:06 | |
| *** ducttape_ has quit IRC | 02:08 | |
| *** houming has joined #openstack-cinder | 02:11 | |
| *** raunak has joined #openstack-cinder | 02:23 | |
| *** iceyao has joined #openstack-cinder | 02:24 | |
| *** mriedem has quit IRC | 02:28 | |
| *** iceyao_ has joined #openstack-cinder | 02:29 | |
| *** Julien-zte has quit IRC | 02:29 | |
| smcginnis | jgriffith: Glad the conversation got around to stats reporting. | 02:31 |
|---|---|---|
| smcginnis | jgriffith: That's where I was going with it. | 02:31 |
| smcginnis | jgriffith: config option causes problems for multibackend. | 02:31 |
| smcginnis | jgriffith: So I think we need the driver to report something like "allocation_unit_size: 8" or something to report its size in increments. | 02:32 |
| *** iceyao has quit IRC | 02:32 | |
| smcginnis | Or maybe that and minimum_allocation_size to report the smallest size it supports, which may or may not be the increments from there on. | 02:32 |
| jgriffith | smcginnis: yeah, that seems like a reasonable way to go to me, the only real sane answer anyway | 02:36 |
| *** enriquetaso has quit IRC | 02:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/316343 | 02:55 |
| *** duobei has joined #openstack-cinder | 02:59 | |
| *** itlinux has quit IRC | 03:09 | |
| *** EinstCra_ has joined #openstack-cinder | 03:10 | |
| *** EinstCrazy has quit IRC | 03:14 | |
| *** alekseyruban has quit IRC | 03:18 | |
| *** wxy has joined #openstack-cinder | 03:21 | |
| *** ducttape_ has joined #openstack-cinder | 03:24 | |
| *** raunak has quit IRC | 03:27 | |
| *** links has joined #openstack-cinder | 03:31 | |
| *** sdake has joined #openstack-cinder | 03:36 | |
| *** itlinux has joined #openstack-cinder | 03:38 | |
| *** sdake_ has quit IRC | 03:38 | |
| *** itlinux has quit IRC | 03:39 | |
| *** itzdilip has quit IRC | 03:41 | |
| *** gouthamr_ has quit IRC | 03:46 | |
| *** duobei has quit IRC | 03:52 | |
| *** gcb has quit IRC | 03:53 | |
| *** sgotliv_ has joined #openstack-cinder | 03:54 | |
| *** jamielennox|away is now known as jamielennox | 03:55 | |
| *** sgotliv_ has quit IRC | 03:57 | |
| *** sgotliv has joined #openstack-cinder | 03:57 | |
| *** bill_az has joined #openstack-cinder | 03:58 | |
| *** laughterwym has quit IRC | 03:59 | |
| *** ducttape_ has quit IRC | 04:00 | |
| *** laughterwym has joined #openstack-cinder | 04:05 | |
| *** gcb has joined #openstack-cinder | 04:06 | |
| *** sheel has joined #openstack-cinder | 04:06 | |
| *** duobei has joined #openstack-cinder | 04:08 | |
| *** salv-orlando has joined #openstack-cinder | 04:15 | |
| *** salv-orl_ has quit IRC | 04:18 | |
| *** sdake_ has joined #openstack-cinder | 04:20 | |
| *** sdake has quit IRC | 04:20 | |
| *** laughter_ has joined #openstack-cinder | 04:20 | |
| *** duobei has quit IRC | 04:22 | |
| openstackgerrit | venkatamahesh proposed openstack/python-cinderclient: Update the home-page with developer documentation https://review.openstack.org/317232 | 04:22 |
| *** laughterwym has quit IRC | 04:23 | |
| *** itzdilip has joined #openstack-cinder | 04:23 | |
| *** delatte has quit IRC | 04:27 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:29 | |
| *** lpetrut has joined #openstack-cinder | 04:37 | |
| *** dave-mccowan has quit IRC | 04:37 | |
| *** IlyaG_ has joined #openstack-cinder | 04:50 | |
| *** IlyaG has quit IRC | 04:54 | |
| *** GB21 has joined #openstack-cinder | 04:56 | |
| *** sdake_ has quit IRC | 04:56 | |
| *** asselin has joined #openstack-cinder | 05:03 | |
| *** GB21 has quit IRC | 05:04 | |
| *** avishay has joined #openstack-cinder | 05:05 | |
| *** asselin__ has quit IRC | 05:06 | |
| *** e0ne has joined #openstack-cinder | 05:09 | |
| *** GB21 has joined #openstack-cinder | 05:09 | |
| *** e0ne has quit IRC | 05:11 | |
| *** shausy has joined #openstack-cinder | 05:14 | |
| *** savihou has joined #openstack-cinder | 05:19 | |
| *** harlowja_at_home has quit IRC | 05:20 | |
| *** lpetrut has quit IRC | 05:21 | |
| *** lpetrut has joined #openstack-cinder | 05:22 | |
| *** laughter_ has quit IRC | 05:23 | |
| *** laughterwym has joined #openstack-cinder | 05:23 | |
| *** nkrinner has joined #openstack-cinder | 05:26 | |
| *** ChubYann has quit IRC | 05:28 | |
| *** GB21 has quit IRC | 05:31 | |
| *** gcb has quit IRC | 05:36 | |
| *** lpetrut has quit IRC | 05:44 | |
| *** iceyao_ has quit IRC | 05:46 | |
| *** iceyao has joined #openstack-cinder | 05:46 | |
| *** duobei has joined #openstack-cinder | 05:49 | |
| *** gcb has joined #openstack-cinder | 05:50 | |
| *** mjura has joined #openstack-cinder | 05:51 | |
| *** iceyao_ has joined #openstack-cinder | 05:53 | |
| *** iceyao has quit IRC | 05:55 | |
| *** savihou has quit IRC | 05:58 | |
| *** mjura has quit IRC | 05:59 | |
| *** mjura has joined #openstack-cinder | 05:59 | |
| *** rcernin has joined #openstack-cinder | 06:03 | |
| *** liverpooler has quit IRC | 06:04 | |
| *** liverpooler has joined #openstack-cinder | 06:04 | |
| *** daneyon has quit IRC | 06:08 | |
| *** savihou has joined #openstack-cinder | 06:11 | |
| *** rcernin has quit IRC | 06:18 | |
| *** savihou has quit IRC | 06:18 | |
| *** rcernin has joined #openstack-cinder | 06:21 | |
| *** savihou has joined #openstack-cinder | 06:22 | |
| *** bill_az has quit IRC | 06:26 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 06:28 |
| *** cdelatte has joined #openstack-cinder | 06:30 | |
| *** jamielennox is now known as jamielennox|away | 06:41 | |
| *** ircuser-1 has joined #openstack-cinder | 06:45 | |
| *** belmoreira has joined #openstack-cinder | 06:53 | |
| *** arecknag has joined #openstack-cinder | 06:54 | |
| *** duobei has quit IRC | 06:57 | |
| *** duobei has joined #openstack-cinder | 07:05 | |
| *** ozialien10 has quit IRC | 07:07 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 07:23 |
| *** duobei has quit IRC | 07:26 | |
| *** iceyao_ has quit IRC | 07:27 | |
| *** sayalilunkad has joined #openstack-cinder | 07:28 | |
| *** gcb has quit IRC | 07:29 | |
| *** sayalilunkad has quit IRC | 07:36 | |
| *** sayalilunkad has joined #openstack-cinder | 07:36 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Updated volume, version, limit, metadata API-ref https://review.openstack.org/316587 | 07:39 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 07:40 |
| *** gcb has joined #openstack-cinder | 07:41 | |
| DuncanT | smcginnis: I think it is a very sane answer to say 'the cinder API works in 1 gb increments and backends that don't support it are not suitable of use with cinder' - it might not be an answer we eventually go with, but it should certainly be an acknowledged option on the table. Whenever we put weirdness like this in our API it has a cost forever, and some of | 07:42 |
| DuncanT | those costs are undoubtedly not worth it | 07:42 |
| *** savihou has quit IRC | 07:48 | |
| *** savihou has joined #openstack-cinder | 07:48 | |
| *** ozialien10 has joined #openstack-cinder | 07:52 | |
| openstackgerrit | Bin Zhou proposed openstack/cinder: add function call chain https://review.openstack.org/317295 | 07:52 |
| *** iceyao has joined #openstack-cinder | 07:53 | |
| *** cloudpuppy has joined #openstack-cinder | 07:54 | |
| *** ozialien10 has quit IRC | 07:57 | |
| *** ozialien10 has joined #openstack-cinder | 07:58 | |
| *** duobei has joined #openstack-cinder | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** liverpooler has quit IRC | 08:01 | |
| *** liverpooler has joined #openstack-cinder | 08:01 | |
| *** divab has quit IRC | 08:06 | |
| *** iceyao_ has joined #openstack-cinder | 08:07 | |
| lixiaoy1 | DuncanT: Could you kindly review https://review.openstack.org/#/c/247577? | 08:07 |
| *** iceyao has quit IRC | 08:08 | |
| *** savihou has quit IRC | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** savihou has joined #openstack-cinder | 08:10 | |
| *** cristicalin has joined #openstack-cinder | 08:13 | |
| *** caspinol has joined #openstack-cinder | 08:15 | |
| *** jistr has joined #openstack-cinder | 08:19 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage snapshot in ScaleIO driver Add support for manage/unmanage snapshot in the ScaleIO driver. https://review.openstack.org/316742 | 08:20 |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage snapshot in ScaleIO driver https://review.openstack.org/316742 | 08:21 |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO actual volume size model update https://review.openstack.org/316811 | 08:26 |
| *** e0ne has joined #openstack-cinder | 08:30 | |
| *** cristicalin has quit IRC | 08:31 | |
| *** cristicalin has joined #openstack-cinder | 08:31 | |
| *** mvk has joined #openstack-cinder | 08:33 | |
| *** laughterwym has quit IRC | 08:43 | |
| *** cristicalin has quit IRC | 08:44 | |
| *** laughterwym has joined #openstack-cinder | 08:44 | |
| *** laughterwym has quit IRC | 08:46 | |
| *** laughterwym has joined #openstack-cinder | 08:46 | |
| openstackgerrit | LisaLi proposed openstack/cinder-specs: Retype encrypted volumes https://review.openstack.org/248593 | 08:46 |
| *** savihou has quit IRC | 08:47 | |
| *** savihou has joined #openstack-cinder | 08:48 | |
| *** laughterwym has quit IRC | 08:51 | |
| *** salv-orlando has quit IRC | 08:54 | |
| *** caspinol has quit IRC | 09:00 | |
| *** yhayashi has quit IRC | 09:01 | |
| *** caspinol has joined #openstack-cinder | 09:08 | |
| *** jordanP has joined #openstack-cinder | 09:10 | |
| *** caspinol has quit IRC | 09:11 | |
| *** cloudpuppy has quit IRC | 09:32 | |
| *** cloudpuppy has joined #openstack-cinder | 09:32 | |
| *** cloudpuppy has quit IRC | 09:33 | |
| *** cloudpuppy has joined #openstack-cinder | 09:33 | |
| *** cloudpuppy has quit IRC | 09:34 | |
| *** cloudpuppy has joined #openstack-cinder | 09:34 | |
| *** dave-mccowan has joined #openstack-cinder | 09:35 | |
| *** mvk has quit IRC | 09:35 | |
| *** mvk has joined #openstack-cinder | 09:37 | |
| yuriy_n17 | dulek: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/315673/ whenever you have free time. | 09:43 |
| *** dobson has quit IRC | 09:56 | |
| *** cheneydc has quit IRC | 10:01 | |
| *** laughterwym has joined #openstack-cinder | 10:01 | |
| *** duobei has quit IRC | 10:02 | |
| *** duobei has joined #openstack-cinder | 10:03 | |
| *** laughterwym has quit IRC | 10:06 | |
| *** caspinol has joined #openstack-cinder | 10:07 | |
| *** caspinol has quit IRC | 10:09 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 10:12 |
| *** salv-orl_ has joined #openstack-cinder | 10:13 | |
| *** salv-orl_ has quit IRC | 10:13 | |
| *** salv-orl_ has joined #openstack-cinder | 10:13 | |
| *** iceyao_ has quit IRC | 10:13 | |
| *** sdague has joined #openstack-cinder | 10:18 | |
| *** EinstCra_ has quit IRC | 10:25 | |
| *** e0ne has quit IRC | 10:25 | |
| *** dobson has joined #openstack-cinder | 10:26 | |
| *** e0ne has joined #openstack-cinder | 10:26 | |
| *** yangyapeng has quit IRC | 10:29 | |
| *** duobei has quit IRC | 10:39 | |
| *** dobson has quit IRC | 10:43 | |
| *** lpetrut has joined #openstack-cinder | 10:48 | |
| *** jordanP has quit IRC | 10:49 | |
| *** jwcroppe has quit IRC | 10:51 | |
| *** gcb has quit IRC | 10:51 | |
| *** jwcroppe has joined #openstack-cinder | 10:53 | |
| *** jamielennox|away is now known as jamielennox | 10:53 | |
| *** dobson has joined #openstack-cinder | 10:55 | |
| *** jwcroppe_ has joined #openstack-cinder | 10:56 | |
| *** jwcroppe has quit IRC | 10:59 | |
| yuriy_n17 | geguileo: Hi! Very sorry for disturbing. Please review https://review.openstack.org/#/c/315673/ whenever you have free time. | 11:16 |
| *** gcb has joined #openstack-cinder | 11:18 | |
| *** jistr has quit IRC | 11:24 | |
| *** ccesario has quit IRC | 11:32 | |
| *** salv-orl_ has quit IRC | 11:32 | |
| *** diogogmt has quit IRC | 11:35 | |
| *** salv-orlando has joined #openstack-cinder | 11:35 | |
| *** diogogmt has joined #openstack-cinder | 11:37 | |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: List manageable volumes and snapshots https://review.openstack.org/285296 | 11:41 |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3 https://review.openstack.org/307324 | 11:41 |
| avishay | DuncanT: added "DocImpact" ^^ | 11:42 |
| *** caspinol has joined #openstack-cinder | 11:47 | |
| *** dave-mccowan has quit IRC | 11:47 | |
| *** vnogin has quit IRC | 11:47 | |
| *** caspinol has quit IRC | 11:48 | |
| *** diogogmt has quit IRC | 11:54 | |
| *** caspinol has joined #openstack-cinder | 11:55 | |
| *** laughterwym has joined #openstack-cinder | 11:55 | |
| *** laughterwym has quit IRC | 11:57 | |
| *** jistr has joined #openstack-cinder | 12:00 | |
| *** gouthamr has joined #openstack-cinder | 12:02 | |
| *** julim has joined #openstack-cinder | 12:05 | |
| *** caspinol has quit IRC | 12:05 | |
| *** laughterwym has joined #openstack-cinder | 12:06 | |
| *** duobei has joined #openstack-cinder | 12:07 | |
| *** ducttape_ has joined #openstack-cinder | 12:09 | |
| *** houming has quit IRC | 12:10 | |
| *** dave-mccowan has joined #openstack-cinder | 12:13 | |
| openstackgerrit | zhangguoqing proposed openstack/cinder: Violin: resize volume if cloned lun is larger https://review.openstack.org/295094 | 12:14 |
| dulek | guitarzan: I've just read your talk with geguileo. Basically we've incremented RPC versions and made managers to accept both of them in Mitaka RC1. | 12:15 |
| *** duobei has quit IRC | 12:15 | |
| dulek | guitarzan: a0686c4641f0a22182b3b7ee3d0c7b70d00fd9c4 is the place we've did that. | 12:15 |
| *** raildo-afk is now known as raildo | 12:15 | |
| dulek | guitarzan: Now we're dropping that compat code. Current N's master has scheduler compat code removed. | 12:16 |
| dulek | guitarzan: So to do L->N upgrade you need to go through this commit (actually through something between that and d0bddb19dc508265274a1193c7d5c1d19e35e441 where we're removing first compatibility code), and then to master. | 12:17 |
| *** JoseMello has joined #openstack-cinder | 12:22 | |
| *** krtaylor has joined #openstack-cinder | 12:25 | |
| *** ducttape_ has quit IRC | 12:25 | |
| *** timcl has joined #openstack-cinder | 12:33 | |
| *** adrianofr has joined #openstack-cinder | 12:34 | |
| openstackgerrit | Tomaz Vieira proposed openstack/cinder: Improve hitachi hnas volume drivers log messages https://review.openstack.org/310785 | 12:36 |
| openstackgerrit | Merged openstack/cinder: Manage existing volume with invalid host https://review.openstack.org/314482 | 12:37 |
| *** ccesario has joined #openstack-cinder | 12:39 | |
| *** gcb has quit IRC | 12:40 | |
| *** EinstCrazy has joined #openstack-cinder | 12:45 | |
| *** sheel has quit IRC | 12:45 | |
| *** edmondsw has joined #openstack-cinder | 12:48 | |
| *** salv-orlando has quit IRC | 12:49 | |
| *** gcb has joined #openstack-cinder | 12:53 | |
| *** salv-orlando has joined #openstack-cinder | 12:55 | |
| openstackgerrit | Cedric Brandily proposed openstack/python-cinderclient: Support --os-key option https://review.openstack.org/300243 | 13:02 |
| *** xyang1 has joined #openstack-cinder | 13:02 | |
| *** Michael-ZTE has joined #openstack-cinder | 13:05 | |
| *** Michael-ZTE has quit IRC | 13:06 | |
| *** eharney has joined #openstack-cinder | 13:07 | |
| guitarzan | dulek: it's no problem, I just didn't understand what was going on | 13:08 |
| *** cknight has joined #openstack-cinder | 13:09 | |
| *** rlrossit has joined #openstack-cinder | 13:10 | |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: List manageable volumes and snapshots https://review.openstack.org/285296 | 13:11 |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3 https://review.openstack.org/307324 | 13:11 |
| avishay | DuncanT: and a rebase... ^^ | 13:11 |
| *** tpsilva has joined #openstack-cinder | 13:12 | |
| *** yangyapeng has joined #openstack-cinder | 13:12 | |
| *** mriedem has joined #openstack-cinder | 13:14 | |
| *** ducttape_ has joined #openstack-cinder | 13:15 | |
| openstackgerrit | Merged openstack/cinder: Use example.com in unit tests https://review.openstack.org/315131 | 13:15 |
| openstackgerrit | Merged openstack/cinder: Whitelist 'rm' in api-ref tox environment https://review.openstack.org/316363 | 13:16 |
| *** akerr has joined #openstack-cinder | 13:17 | |
| *** jordanP has joined #openstack-cinder | 13:18 | |
| *** ducttape_ has quit IRC | 13:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/316343 | 13:26 |
| *** diogogmt has joined #openstack-cinder | 13:26 | |
| *** avishay has quit IRC | 13:26 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way https://review.openstack.org/312853 | 13:27 |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Make api_vicroversion_dev more explicit https://review.openstack.org/317460 | 13:27 |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Make api_microversion_dev more explicit https://review.openstack.org/317460 | 13:28 |
| *** porrua has joined #openstack-cinder | 13:29 | |
| *** bill_az has joined #openstack-cinder | 13:29 | |
| *** openstackgerrit has quit IRC | 13:32 | |
| *** diogogmt has quit IRC | 13:33 | |
| *** openstackgerrit has joined #openstack-cinder | 13:34 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way https://review.openstack.org/312853 | 13:35 |
| *** links has quit IRC | 13:36 | |
| *** cknight1 has joined #openstack-cinder | 13:38 | |
| *** cknight has quit IRC | 13:40 | |
| *** shausy has quit IRC | 13:45 | |
| *** crose has joined #openstack-cinder | 13:45 | |
| *** takedakn has joined #openstack-cinder | 13:45 | |
| *** dustins has joined #openstack-cinder | 13:49 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Add sample config file to cinder docs https://review.openstack.org/314220 | 13:52 |
| *** mjura has quit IRC | 13:53 | |
| *** jgregor has joined #openstack-cinder | 13:54 | |
| *** crose has quit IRC | 13:54 | |
| *** ametts has joined #openstack-cinder | 13:58 | |
| *** iceyao has joined #openstack-cinder | 13:59 | |
| *** mpjetta has quit IRC | 14:01 | |
| *** ducttape_ has joined #openstack-cinder | 14:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/316343 | 14:02 |
| *** jordanP has quit IRC | 14:03 | |
| *** med_ has joined #openstack-cinder | 14:04 | |
| *** med_ has quit IRC | 14:04 | |
| *** med_ has joined #openstack-cinder | 14:04 | |
| *** Julien-zte has joined #openstack-cinder | 14:05 | |
| *** caspinol has joined #openstack-cinder | 14:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/317493 | 14:08 |
| *** avishay has joined #openstack-cinder | 14:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/317186 | 14:09 |
| *** kfarr has joined #openstack-cinder | 14:14 | |
| *** fnordahl has quit IRC | 14:17 | |
| *** fnordahl has joined #openstack-cinder | 14:17 | |
| kfarr | Hi all, if I'd like to add a spec to the meeting agenda for the 18th, do I just add it to the wiki, or do I need to get it informally approved here first? | 14:17 |
| smcginnis | kfarr: You can just add it to the agenda. No permission required. :) | 14:17 |
| kfarr | Thanks smcginnis! | 14:18 |
| smcginnis | kfarr: No problem, thanks for checking. | 14:18 |
| *** jordanP has joined #openstack-cinder | 14:24 | |
| ildikov | smcginnis: hi | 14:28 |
| ildikov | smcginnis: I had an epic fail with the Cinder-Nova meeting schedule you might already saw on the mailing list | 14:29 |
| ildikov | smcginnis: I suggested this channel as a fall back only for today | 14:29 |
| *** baumann has joined #openstack-cinder | 14:29 | |
| ildikov | smcginnis: do you have any concern regarding this idea? | 14:30 |
| *** mtanino has joined #openstack-cinder | 14:30 | |
| smcginnis | ildikov: Sure, we can try that. | 14:34 |
| smcginnis | ildikov: I think it should work out OK. | 14:35 |
| *** markvoelker has joined #openstack-cinder | 14:35 | |
| *** markvoelker has quit IRC | 14:35 | |
| ildikov | smcginnis: cool, thanks for confirmation | 14:35 |
| *** diablo_rojo has joined #openstack-cinder | 14:36 | |
| *** markvoelker has joined #openstack-cinder | 14:36 | |
| *** karthikp has joined #openstack-cinder | 14:37 | |
| smcginnis | ildikov: No problem. Thanks for pulling it together. Meeting timing can be tricky. | 14:38 |
| *** markvoelker has quit IRC | 14:39 | |
| *** markvoelker has joined #openstack-cinder | 14:39 | |
| scottda | ildikov: What's the official time of the meeting? 1700 UTC or 1800 UTC? | 14:42 |
| ildikov | scottda: 1700UTC | 14:42 |
| ildikov | scottda: I put up the question whether it should be 1800UTC on Tuesdays or 1700UTC on Mondays | 14:43 |
| scottda | ildikov: OK, thanks. I believe the latest email in the thread had a typo (which was 1800)... | 14:43 |
| *** Julien-zte has quit IRC | 14:45 | |
| *** Julien-zte has joined #openstack-cinder | 14:45 | |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add pagination support for Messages https://review.openstack.org/315769 | 14:45 |
| ildikov | scottda: I sent out a follow up mail to the smaller group to find another slot that works for the meeting channels as well | 14:46 |
| ildikov | scottda: I wasn't sure how much Monday 1700UTC would be popular in the US, so I wanted ask around before have that option as a fall back from next week | 14:46 |
| scottda | ildikov: OK. Thanks for all the work arranging this | 14:47 |
| ildikov | scottda: np, I'm sorry I annoy you guys so much with this :( | 14:48 |
| *** markvoelker has quit IRC | 14:51 | |
| *** lprice has joined #openstack-cinder | 14:51 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 14:52 |
| *** markvoelker has joined #openstack-cinder | 14:53 | |
| *** xinli has joined #openstack-cinder | 14:55 | |
| *** dramakri has quit IRC | 14:55 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 14:56 |
| *** diogogmt has joined #openstack-cinder | 14:56 | |
| *** zhonghua-lee has quit IRC | 14:56 | |
| smcginnis | ildikov: Just to make sure I have things straight - we're talking meeting today in 1 hour? | 14:57 |
| *** dramakri has joined #openstack-cinder | 14:57 | |
| ildikov | smcginnis: 1700UTC is two hours from now | 14:57 |
| *** markvoelker has quit IRC | 14:57 | |
| *** iceyao has quit IRC | 14:57 | |
| *** markvoelker has joined #openstack-cinder | 14:58 | |
| ildikov | smcginnis: although it must be a magical time as many people asked me last week as well when we start exactly :) | 14:58 |
| *** IlyaG_ has quit IRC | 14:58 | |
| *** takedakn has quit IRC | 14:59 | |
| *** salv-orlando has quit IRC | 14:59 | |
| smcginnis | ildikov: Thanks! :) | 14:59 |
| *** salv-orlando has joined #openstack-cinder | 15:00 | |
| *** salv-orlando has quit IRC | 15:00 | |
| *** salv-orlando has joined #openstack-cinder | 15:00 | |
| *** salv-orlando has quit IRC | 15:01 | |
| *** salv-orlando has joined #openstack-cinder | 15:02 | |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add v3 user messages https://review.openstack.org/299052 | 15:03 |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add pagination support for Messages https://review.openstack.org/315769 | 15:03 |
| *** Julien-zte has quit IRC | 15:04 | |
| ameade | scottda: ^^ thanks | 15:04 |
| *** laughterwym has quit IRC | 15:04 | |
| *** markvoelker has quit IRC | 15:04 | |
| *** laughterwym has joined #openstack-cinder | 15:04 | |
| scottda | ameade: I don't see the changes in PS#4. Is there another patch coming? | 15:07 |
| ameade | scottda: i made the exact changes in the prior patch so it just shows as a rebase | 15:07 |
| *** sheel has joined #openstack-cinder | 15:09 | |
| ameade | scottda: in the patch it depends on i mean | 15:09 |
| sheel | Hello All, could someone please review on https://review.openstack.org/#/c/316587/ | 15:10 |
| ameade | sheel: I'll check that out soon, even though i dont have +2 powers | 15:10 |
| sheel | ameade: +1 attracts +2... | 15:11 |
| sheel | ameade: so it will help for sure... ;0 | 15:11 |
| sheel | ;) | 15:11 |
| ameade | sheel: oh I was going to -1 it | 15:11 |
| ameade | sheel: just kidding :P | 15:11 |
| sheel | ameade: haha.... | 15:11 |
| *** yangyapeng has quit IRC | 15:12 | |
| buhman | dumb question: I thought I read in the austin summit notes that cinderclient was deprecated--is actively adding new features/etc.. still allowed? | 15:14 |
| sheel | buhman: no its still not deprecated... AFAIK its ok to add features in V2... | 15:16 |
| Swanson | I thought they were de-deprecating it. | 15:16 |
| buhman | O.o | 15:17 |
| sheel | Swanson: :) | 15:17 |
| sheel | buhman: you can go ahead..its still alive for this release at least... | 15:18 |
| buhman | so, what I've really wanted to do is refactor *client's extension architecture to be more flexible; when I asked about making a generic oslo.apiclientextensions or similar in #openstack-oslo, I was discouraged | 15:18 |
| scottda | ameade: I'm still confused... Your first patch looks like it's picked up the not-yet-merged change: https://review.openstack.org/#/c/299052/12/cinderclient/base.py | 15:19 |
| scottda | ameade: And your second patch doesn't have a Depends-On for first patch (It has Depends-On for server-side code). | 15:19 |
| sheel | buhman: I think that is fair enough... may be smcginnis can through some more light on it... | 15:20 |
| ameade | scottda: whats the net yet merged change ur referring to? | 15:25 |
| ameade | i made those changes so that the api_versions.wraps decorator works properly | 15:26 |
| ameade | the second patch's parent is the first patch | 15:26 |
| scottda | ameade: https://review.openstack.org/#/c/309283/ | 15:26 |
| *** nkrinner is now known as nkrinner_afk | 15:26 | |
| ameade | haha awesome | 15:27 |
| ameade | i'll just rebase onto that | 15:27 |
| ameade | i didnt even know about that | 15:27 |
| scottda | ameade: That explains everything! Maybe just put a Depends-On for that 303283? | 15:27 |
| ameade | yeah | 15:27 |
| *** belmoreira has quit IRC | 15:28 | |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add v3 user messages https://review.openstack.org/299052 | 15:29 |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add pagination support for Messages https://review.openstack.org/315769 | 15:29 |
| *** caspinol has quit IRC | 15:31 | |
| *** leeantho has joined #openstack-cinder | 15:32 | |
| *** sticker has quit IRC | 15:32 | |
| *** alkhodos has joined #openstack-cinder | 15:32 | |
| mriedem | ildikov: so we're doing the nova/cinder meeting at 1800 UTC in this channel today? | 15:33 |
| *** lprice1 has joined #openstack-cinder | 15:34 | |
| *** lprice has quit IRC | 15:34 | |
| *** enriquetaso has joined #openstack-cinder | 15:35 | |
| ildikov | mriedem: the plan for today is/was to keep the 1700UTC that I announced on the ML, but have it here as a fall back | 15:36 |
| savihou | Gents, apologies for nudging (again.. :-( ) - will you guys be able to review https://review.openstack.org/#/c/263026/ ? | 15:36 |
| mriedem | ah ok | 15:36 |
| mriedem | works for me | 15:36 |
| ildikov | mriedem: and in the meantime find another slot that would work for the small group and the IRC channels as well | 15:36 |
| ildikov | mriedem: that's why I sent out the follow up mail | 15:36 |
| ildikov | mriedem: sorry for the confusion | 15:37 |
| *** jistr has quit IRC | 15:38 | |
| *** akshai has joined #openstack-cinder | 15:38 | |
| *** mvk has quit IRC | 15:39 | |
| openstackgerrit | Xinli Guan proposed openstack/cinder: IBM FlashSystem: Cleanup host resource leaking https://review.openstack.org/307451 | 15:45 |
| *** daneyon has joined #openstack-cinder | 15:48 | |
| *** asselin__ has joined #openstack-cinder | 15:52 | |
| *** caspinol has joined #openstack-cinder | 15:53 | |
| *** earlephilhower has joined #openstack-cinder | 15:53 | |
| *** kfarr has quit IRC | 15:53 | |
| *** daneyon_ has joined #openstack-cinder | 15:54 | |
| *** daneyon has quit IRC | 15:55 | |
| *** daneyon has joined #openstack-cinder | 15:55 | |
| *** daneyon_ has quit IRC | 15:55 | |
| *** asselin has quit IRC | 15:56 | |
| *** lprice1 has quit IRC | 16:00 | |
| *** savihou has quit IRC | 16:02 | |
| *** lprice has joined #openstack-cinder | 16:05 | |
| smcginnis | buhman, sheel: Sorry, had stepped away. Yes, new functionality can and should still be added to python-cinderclient. | 16:07 |
| smcginnis | buhman, sheel: Long term goal is to have osc support all the same functionality. Once it does, then we can visit the idea of deprecating cinderclient cli. | 16:07 |
| *** sdague has quit IRC | 16:07 | |
| buhman | smcginnis: hmm, so how does adding new functionality to cinderclient get osc closer to having feature parity? | 16:10 |
| smcginnis | buhman: It doesn't. It actually makes it harder. | 16:11 |
| smcginnis | buhman: But we need to continue to support end users with CLI access until we can switch. | 16:11 |
| smcginnis | buhman: So there is a lot of work making sure whatever we add to cinderclient also gets added to osc. | 16:11 |
| *** laughterwym has quit IRC | 16:15 | |
| *** laughterwym has joined #openstack-cinder | 16:16 | |
| ameade | if i'm not mistaken, horizon uses cinderclient | 16:17 |
| david-lyle | ameade: yes, but not via CLI | 16:17 |
| ameade | david-lyle: yeah, but the CLI layer in cinderclient is near trivial | 16:18 |
| david-lyle | ameade: sure, just deprecating CLI is not problematic for Horizon in and of itself | 16:19 |
| ameade | oh we're only deprecating the CLI part? does OSC use cinderclient libraries? | 16:20 |
| smcginnis | ameade: Yes. | 16:22 |
| smcginnis | ameade: The deprecation talk is only for the CLI part. | 16:22 |
| *** caspinol has quit IRC | 16:22 | |
| smcginnis | ameade: The client library part would remain and I believe is what is ultimately used by osc and others. | 16:22 |
| ameade | so we have to add a feature to cinderclient in order to get it in osc anyways, k cool i'm caught up | 16:23 |
| *** jordanP has quit IRC | 16:24 | |
| *** laughterwym has quit IRC | 16:24 | |
| smcginnis | ameade: Yep, pretty much. :) | 16:26 |
| *** Apoorva has joined #openstack-cinder | 16:27 | |
| *** daneyon has quit IRC | 16:28 | |
| *** e0ne has quit IRC | 16:28 | |
| sheel | smcginnis: thanks for giving thoughts... | 16:28 |
| *** daneyon has joined #openstack-cinder | 16:29 | |
| *** yuriy_n17 has quit IRC | 16:31 | |
| *** xinli has quit IRC | 16:38 | |
| *** diablo_rojo has quit IRC | 16:38 | |
| *** raunak has joined #openstack-cinder | 16:40 | |
| *** diablo_rojo has joined #openstack-cinder | 16:41 | |
| jgriffith | patrickeast: ping | 16:41 |
| *** kfarr has joined #openstack-cinder | 16:45 | |
| patrickeast | jgriffith: pong | 16:49 |
| jgriffith | patrickeast: not sure if you've seen my drama lately :) | 16:50 |
| jgriffith | https://bugs.launchpad.net/cinder/+bug/1582806 | 16:50 |
| openstack | Launchpad bug 1582806 in Cinder "virtual_size property in image no longer usable for caching on SF driver" [High,Triaged] - Assigned to John Griffith (john-griffith) | 16:50 |
| jgriffith | patrickeast: so the virtual_size property is R/O | 16:50 |
| patrickeast | Uh oh, no I've missed it (have been out the last week) | 16:51 |
| jgriffith | patrickeast: in other words, users/admins can't set that any longer (I dunno wtf it's for then) | 16:51 |
| patrickeast | Shitty | 16:51 |
| jgriffith | So my local caching broke after a customer upgraded... and I think the general mechanism is now f'd too | 16:51 |
| jgriffith | I'm working a patch right now and was going to just change the name to "min_volsz" as that's a custom property | 16:51 |
| patrickeast | I'll have to double check, but I think we fixed that part of the generic cache, it doesn't look for that property from glance | 16:51 |
| jgriffith | patrickeast: oh... even better | 16:52 |
| jgriffith | if that's the case then life is ok | 16:52 |
| jgriffith | hmm... it grabs it in taskflow create | 16:52 |
| patrickeast | We download the image first and look at the IMG info output then decide on the size | 16:52 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Add pagination support to messages https://review.openstack.org/313113 | 16:52 |
| patrickeast | Yea | 16:52 |
| jgriffith | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L746 | 16:53 |
| patrickeast | Yep yep | 16:53 |
| patrickeast | That's the one | 16:53 |
| jgriffith | coolio | 16:53 |
| *** daneyon has quit IRC | 16:57 | |
| *** bill_az has quit IRC | 16:58 | |
| *** bill_az has joined #openstack-cinder | 16:59 | |
| *** daneyon has joined #openstack-cinder | 16:59 | |
| ildikov | ping scottda ildikov DuncanT ameade cFouts johnthetubaguy e0ne jgriffith andrearosa hemna mriedem smcginnis | 17:00 |
| scottda | ildikov: Hi | 17:00 |
| ildikov | so as suggested on the ML earlier I would like to hijack the Cinder channel for a quick sync | 17:01 |
| johnthetubaguy | cool | 17:01 |
| * ameade lurking | 17:01 | |
| cFouts | hi | 17:01 |
| mriedem | oh it's here | 17:01 |
| hemna | yough | 17:01 |
| ildikov | jgriffith: around? | 17:02 |
| jgriffith | ildikov: hi there | 17:02 |
| DuncanT | hi | 17:02 |
| ildikov | jgriffith: I hoped we can start with your plans on initialize_connection | 17:02 |
| ildikov | jgriffith: any progress with that item? | 17:03 |
| jgriffith | ildikov: sure. | 17:03 |
| jgriffith | ildikov: yes.. that was pretty easy with the exception of our crazy unit tests :( | 17:03 |
| jgriffith | I plan to get that fixed up | 17:03 |
| jgriffith | One thing that is troubling for me though.... | 17:03 |
| jgriffith | on volume.manager.attach we've gotten ourselves in a pretty bad state | 17:04 |
| jgriffith | we have a number of arguments that are "optional" which is kinda bad IMO | 17:04 |
| *** bill_az has quit IRC | 17:04 | |
| jgriffith | and we don't really document in the docstrings what and why | 17:04 |
| jgriffith | we don't even do something like set a default=None to signify things | 17:05 |
| jgriffith | I'd like to propose we fix that now while we're at this | 17:05 |
| jgriffith | In other words, I'd like to add a follow up patch to what I'm working on that just eliminates the whole "optional" arguments thing | 17:05 |
| *** jgregor has quit IRC | 17:05 | |
| jgriffith | That would mean a dependent patch to Nova. So the alternative here.... | 17:06 |
| jgriffith | Rewrite the flow using "new names" and then we don't have version problems, compatability issues etc | 17:06 |
| jgriffith | deprecate the old calls | 17:06 |
| jgriffith | thoughts? | 17:07 |
| scottda | If the changes are significant enough, I say re-write with new names. | 17:07 |
| mriedem | nova is only calling the api | 17:07 |
| hemna | os-attach ? | 17:07 |
| mriedem | the cinder api is eventually calling the volume manager | 17:07 |
| *** diablo_rojo has quit IRC | 17:08 | |
| mriedem | so any changes there are rpc changes in cinder | 17:08 |
| mriedem | not exposed to nova | 17:08 |
| jgriffith | mriedem: I can hide them :) | 17:08 |
| jgriffith | mriedem: the only thing is I'd like to submit a change to Nova for the current release that just sets standard args for attach | 17:08 |
| jgriffith | mriedem: no more "maybe I'll get this, maybe I'll get that" | 17:09 |
| johnthetubaguy | so there is stuff we should be sending? | 17:09 |
| johnthetubaguy | do you have an example? | 17:09 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_actions.py#L58-L66 | 17:09 |
| *** baumann has quit IRC | 17:09 | |
| hemna | jgriffith, that stuffs ? | 17:09 |
| jgriffith | johnthetubaguy: yeah, that would've been useful eh :) | 17:09 |
| *** baumann has joined #openstack-cinder | 17:09 | |
| jgriffith | hemna: yep | 17:09 |
| hemna | coolio | 17:09 |
| johnthetubaguy | I am wondering also if sending that stuff to old versions would be OK? I guess its fine as it was optional before? | 17:09 |
| jgriffith | johnthetubaguy: yes | 17:09 |
| johnthetubaguy | so thats doesn't seem too bad | 17:10 |
| hemna | I think the older versions would just ignore em | 17:10 |
| jgriffith | johnthetubaguy: so my plan was to put the logic in the new calls to grab the old call if need be | 17:10 |
| jgriffith | all handled on Cinder's side | 17:10 |
| johnthetubaguy | so old nova talking to new cinder needs care | 17:10 |
| johnthetubaguy | but its an edge case | 17:10 |
| jgriffith | johnthetubaguy: yes, but I would address that case on the Cinder side | 17:10 |
| jgriffith | so my change would be compatible either way | 17:11 |
| jgriffith | johnthetubaguy: if it's an old nova then it just uses the existing calls (that I'll deprecate) | 17:11 |
| *** earlephilhower has quit IRC | 17:11 | |
| jgriffith | johnthetubaguy: if it's the new format and I just route it to the appropriate method in the manager | 17:11 |
| johnthetubaguy | sounds like its (1) get nova to send stuff it should be sending (2) tidy cinder code so its clear: a) what is optional, b) what is optional because we did silly things in the past, c) whats always been required | 17:11 |
| *** vasanthi has quit IRC | 17:12 | |
| jgriffith | kind of the other way around actually.... always goes to the new, and it decides what to do | 17:12 |
| johnthetubaguy | hmm, so new nova, old cinder? you eventually drop that API? | 17:12 |
| jgriffith | johnthetubaguy: you got it | 17:12 |
| jgriffith | johnthetubaguy: after another release at least | 17:12 |
| jgriffith | johnthetubaguy: cuz if you're trying to do n-2 you're hosed anyway most likely | 17:12 |
| johnthetubaguy | and new Nova, old cinder? has to know about both cases? | 17:12 |
| jgriffith | johnthetubaguy: yup, that one is covered as well | 17:13 |
| *** marcin12345 has quit IRC | 17:13 | |
| johnthetubaguy | how is it covered? I am missing something I think | 17:13 |
| jgriffith | johnthetubaguy: since the volume_actions module uses a kwargs we have some safety there | 17:13 |
| johnthetubaguy | so I am confused | 17:14 |
| jgriffith | johnthetubaguy: so even if new nova talks to old cinder and sends something in the body that old cinder doesn't understand it's just going to ignore it anyway | 17:14 |
| smcginnis | So we stay flexible with being able to parse kwargs, but in normal cases everything is explicitly passed in. | 17:14 |
| johnthetubaguy | there is a new format? | 17:14 |
| jgriffith | smcginnis: correct | 17:14 |
| jgriffith | johnthetubaguy: not a new format, just make the args required instead of "optional" | 17:14 |
| jgriffith | johnthetubaguy: I don't explain code well, it may be best if I just finish the proposed patch and throw it out for everyone to check | 17:15 |
| jgriffith | johnthetubaguy: if it gets barfed on that's ok | 17:15 |
| ildikov | will we actually pass in new arguments or we just pass everything from now on that's expected to be there? | 17:15 |
| johnthetubaguy | OK, so that breaks old nova talking to new cinder right? | 17:15 |
| jgriffith | johnthetubaguy: nope, I can address that | 17:15 |
| ildikov | as if nothing is fundamentally new than new Nova old Cinder should work | 17:15 |
| johnthetubaguy | so then they are not required still? | 17:15 |
| jgriffith | johnthetubaguy: I'm saying I've thought of both cases old-nova:new-cinder and new-nova:old-cinder and it works | 17:15 |
| jgriffith | johnthetubaguy: well... yeah, I see your point | 17:16 |
| johnthetubaguy | lets back up a sec... | 17:16 |
| jgriffith | johnthetubaguy: so technically YES, you could not change Nova and things would still work | 17:16 |
| johnthetubaguy | I am cool with nova starting to fill in some optional params it didn't before, that works | 17:16 |
| jgriffith | johnthetubaguy: it's not as bad as it sounds I promise :) | 17:16 |
| johnthetubaguy | its the required on cinder side, or new format stuff that sounds dangerous | 17:16 |
| jgriffith | johnthetubaguy: and actually at least in all of the iSCSI cases I checked Nova is giving everything I want consistently. I just want to add one more thing for multi-attach :) | 17:17 |
| johnthetubaguy | thats not the hostname right? | 17:17 |
| jgriffith | johnthetubaguy: either that or initiator | 17:18 |
| jgriffith | so here's what we have at intialize-connection time: | 17:18 |
| jgriffith | http://paste.openstack.org/show/497394/ | 17:18 |
| jgriffith | johnthetubaguy: sorry.. I said initiator; I mean either host or ip | 17:19 |
| jgriffith | initiator might be better due to multipath | 17:19 |
| jgriffith | but I'm not sure yet | 17:19 |
| jgriffith | johnthetubaguy: basically we have MORE than enough info in the connector | 17:20 |
| jgriffith | johnthetubaguy: including duplicate info IMO | 17:20 |
| johnthetubaguy | my worry is live-migrate | 17:20 |
| hemna | the FC case adds 2 more items wwpns, wwnns | 17:20 |
| hemna | which are lists of wwns | 17:20 |
| jgriffith | well... just host and ip | 17:20 |
| jgriffith | hemna: yeah, I noticed that | 17:20 |
| hemna | (in the connector) | 17:20 |
| jgriffith | johnthetubaguy: why's that? | 17:20 |
| johnthetubaguy | if you need that, we need to start updating that across live-migrate, somehow | 17:20 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 17:21 |
| jgriffith | johnthetubaguy: I mean, what's missing on the live-migration piece? | 17:21 |
| johnthetubaguy | we haven't got the neturon version of this straight yet | 17:21 |
| johnthetubaguy | jgriffith: so we would need to work out when to update the host during the live-migrate, so it doesn't break things | 17:21 |
| jgriffith | johnthetubaguy: oh... got ya | 17:21 |
| johnthetubaguy | you basically have a connection on both hosts for a while | 17:22 |
| jgriffith | johnthetubaguy: so currently I believe we reinitialize, that would cover it I believe | 17:22 |
| ildikov | jgriffith: if we remove the second call | 17:22 |
| *** earlephilhower has joined #openstack-cinder | 17:22 | |
| ildikov | jgriffith: to initialize_connection I mean | 17:22 |
| hemna | yah, this is part of the effort to eliminate the need for the 2nd call to refetch the connection info that nova lost | 17:23 |
| johnthetubaguy | right, so if we have matched pairs of initialize / destroy(?) we are in a better place | 17:23 |
| hemna | nova had the connection_info at the start of the live migration process and writes over it in the bdm. | 17:23 |
| johnthetubaguy | and each of the initialize calls passes the correct host, so we might be good | 17:23 |
| *** daneyon has quit IRC | 17:23 | |
| hemna | https://review.openstack.org/#/c/312773/ | 17:23 |
| ildikov | hemna: correct | 17:23 |
| mriedem | hemna: btw, i know why the live migration job is busted there, it's unrelated | 17:24 |
| hemna | that was a WIP to 'save' the original connection_info, to avoid the 2nd call. | 17:24 |
| jgriffith | johnthetubaguy: ildikov so what I'm getting at is that as long as I have the connector object and knew what to do at attach time I can cover those cases | 17:24 |
| hemna | mriedem, ok cool | 17:24 |
| jgriffith | hemna: you don't even need to do that if i have the initiator or host info at attach | 17:24 |
| jgriffith | hemna: I'll already know what is connected where | 17:24 |
| jgriffith | hemna: then *we* can determine what to do or not to do when initialize_connection is called | 17:24 |
| hemna | I like the idea of not having nova call cinder a 2nd time. just saves a useless round trip to Cinder...fwiw. | 17:24 |
| jgriffith | hemna: yeah, I'm fine with that | 17:25 |
| johnthetubaguy | jgriffith: if we set the host on initiailize_connection, then we are good, I guess? | 17:25 |
| smcginnis | hemna: +1 | 17:25 |
| jgriffith | hemna: I've just been trying to minimize impact on Nova API interactions | 17:25 |
| jgriffith | hemna: I'd certainly be down with following up with something like that | 17:25 |
| hemna | jgriffith, ok cool. lets figure out what's best. | 17:25 |
| jgriffith | johnthetubaguy: yeah | 17:25 |
| hemna | I kinda think it's a bug in nova that it calls cinder a 2nd time right now. | 17:25 |
| hemna | but I'm open to whatevs | 17:25 |
| johnthetubaguy | hemna: yeah, that extra call is totally a bug | 17:26 |
| jgriffith | hemna: nah, I don't disagree with you. I've viewed some of that as a secondary effort though | 17:26 |
| ildikov | hemna: I think we should remove that 2nd call, it's very confusing there... | 17:26 |
| hemna | jgriffith, but I think having cinder be smart in init_connection is also good in the case where it's not nova calling cinder. | 17:26 |
| johnthetubaguy | I mean, I wish we didn't need that information when calling cinder, but thats getting ahead of ourselves | 17:26 |
| jgriffith | johnthetubaguy: :) | 17:26 |
| mriedem | so passing the connector to attach, that's going to store the connector in the bdm attachment table? | 17:26 |
| jgriffith | don't forget, I'm trying to also solve the "old-nova:new-cinder" case | 17:27 |
| jgriffith | so I'm going to have that logic initially regardless | 17:27 |
| jgriffith | mriedem: you'll have to talk with hemna on that | 17:27 |
| jgriffith | mriedem: my proposal is not to do that, at least not right now | 17:27 |
| mriedem | "so what I'm getting at is that as long as I have the connector object and knew what to do at attach time I can cover those cases" | 17:27 |
| mriedem | i thought it came from ^ | 17:27 |
| mriedem | or is the connector stored when we call init_connection? | 17:28 |
| jgriffith | mriedem: as long as I have the connector at initialize_connection, and then eithter, host, IP or initiator on the attach call we can handle all of it on the Cinder side | 17:28 |
| hemna | jgriffith, +1 | 17:29 |
| mriedem | ok, nova passes the connector to init today | 17:29 |
| mriedem | but not the host/connector/ip to attach | 17:29 |
| mriedem | but looks like we can, it's been in the cinder api as optional | 17:29 |
| jgriffith | mriedem: correct | 17:29 |
| mriedem | point i was trying to get to was older nova wouldn't be passing those, but it sounds like we've already covered that | 17:29 |
| jgriffith | mriedem: my only outstanding question was *which* of those is the best to use | 17:30 |
| jgriffith | mriedem: to cover FC, Ceph etc etc etc | 17:30 |
| ildikov | wasn't passing the host name the one created issues for us? or that was another call? | 17:30 |
| jgriffith | mriedem: as well as multi-path | 17:30 |
| scottda | ildikov: We had originally tried passing both host and Instance_id | 17:30 |
| jgriffith | ildikov: yes, *host* was the one that you and hemna added | 17:30 |
| mriedem | jgriffith: nova has the connector when it calls attach, it just doesn't pass it, and the os-attach api doesn't take connector | 17:31 |
| hemna | ildikov, I think jgriffith is going to pull the hostname out of the connector though | 17:31 |
| jgriffith | ildikov: I'm not sure that's the right one though | 17:31 |
| hemna | as that's always been there. | 17:31 |
| scottda | ildikov: Passing both caused problems with older cinder | 17:31 |
| ildikov | yeah, I remember the check for that | 17:31 |
| jgriffith | hemna: it's there but it's not used... that's my point about the optional stuff :) | 17:31 |
| hemna | yah | 17:31 |
| johnthetubaguy | scottda: it was stuff older cinder didn't accept, that time, I guess? | 17:32 |
| scottda | johnthetubaguy: yes | 17:32 |
| mriedem | how old? when was https://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_actions.py#L58-L66 added? | 17:32 |
| hemna | for whatever reason when the host was added to the API it was enforced that it had to be either host or instance_uuid | 17:32 |
| jgriffith | johnthetubaguy: correct | 17:32 |
| hemna | not both. which....seems silly | 17:32 |
| jgriffith | johnthetubaguy: the way it was implemented it broke a bunch of drivers | 17:32 |
| jgriffith | because the param wasn't added to them | 17:32 |
| ildikov | hemna: correct, we can fix that now with an API microversion though if needed | 17:33 |
| mriedem | ah https://github.com/openstack/cinder/commit/728f9838d2ca44b66fc240cac211fcae4b286416 | 17:33 |
| hemna | ildikov, we could, but I would rather see what jgriffith has in mind and see if we can fix all this shit once and for all :) | 17:33 |
| johnthetubaguy | with microversion, Nova can detect what it needs to send, based on the detected cinder version, older system just keep using their older microversion | 17:33 |
| *** IlyaG has joined #openstack-cinder | 17:34 | |
| *** lpetrut has quit IRC | 17:34 | |
| smcginnis | jgriffith: Any idea how far along until we can take a look at a WIP patch? | 17:34 |
| smcginnis | I think that may help looking at a little code at this point. | 17:34 |
| jgriffith | smcginnis: end of day today or tomorrow morning? | 17:34 |
| ildikov | hemna: I'm not against, I just don't want more CIs broken this time :) | 17:34 |
| smcginnis | jgriffith: Oh, awesome! | 17:35 |
| jgriffith | smcginnis: or course I always say "today or tomorrow morning" :) | 17:35 |
| jgriffith | of | 17:35 |
| jgriffith | but honestly I need to get this done and move on to other things | 17:36 |
| hemna | jgriffith, anything I can help with ? | 17:36 |
| jgriffith | hemna: you can review it for the FC pieces :) | 17:36 |
| jgriffith | hemna: I'll likely need some help on that part since I can't test it | 17:37 |
| hemna | ok :) Just let me know if you need anything else. | 17:37 |
| jgriffith | will do | 17:37 |
| ildikov | jgriffith: it's fine in an iterative way as well, I mean just upload what you have and then the next patches later, we're all eager to chime in :) | 17:37 |
| smcginnis | We need to make sure jbernard or one of those types looks at it too. | 17:37 |
| *** salv-orl_ has joined #openstack-cinder | 17:37 | |
| jgriffith | smcginnis: +1 jbernard and also eharney | 17:37 |
| scottda | smcginnis: Hopefully you mean "Ceph type" | 17:37 |
| smcginnis | scottda: Haha, yep. :) | 17:37 |
| jgriffith | smcginnis: smcginnis ceph, nfs and FC are the ones I'll want to get others to look at more closely | 17:38 |
| smcginnis | jgriffith: +1 Let's not get an iSCSI only solution if we can help it. | 17:38 |
| hemna | wait, people use non iSCSI ? | 17:39 |
| smcginnis | :) | 17:39 |
| smcginnis | Not if jgriffith could help it. :P | 17:40 |
| jgriffith | smcginnis: untrue | 17:40 |
| jgriffith | smcginnis: RBD is cool | 17:40 |
| *** salv-orlando has quit IRC | 17:40 | |
| hemna | rm -rf cinder/zonemanager ...... | 17:40 |
| jgriffith | that's about it though :) | 17:40 |
| openstackgerrit | Parashuram Hallur proposed openstack/cinder: Implements : Blueprint CoprHD Cinder Drivers - https://blueprints.launchpad.net/cinder/+spec/coprhd-cinder-drivers https://review.openstack.org/317643 | 17:40 |
| smcginnis | Hah! | 17:40 |
| ildikov | anything else to this item? :) | 17:41 |
| *** avishay has quit IRC | 17:43 | |
| ildikov | mriedem: johnthetubaguy: did you have a chance to look at these: https://review.openstack.org/#/c/312773/ , https://review.openstack.org/#/c/315789/ ? | 17:43 |
| johnthetubaguy | I did look at the first one, not properly looked at the second one | 17:44 |
| johnthetubaguy | there is a todo for me right there | 17:44 |
| ildikov | johnthetubaguy: cool, thanks | 17:44 |
| johnthetubaguy | we should make sure we get the bugs marked with the correct priority, etc | 17:44 |
| mriedem | ildikov: i haven't yet | 17:44 |
| ildikov | johnthetubaguy: basically it would be good to see whether the direction is good or not, etc. | 17:45 |
| johnthetubaguy | added a quick question on the second one | 17:45 |
| ildikov | johnthetubaguy: can you check that? | 17:45 |
| jgriffith | hemna: ahh... cool | 17:46 |
| ildikov | johnthetubaguy: I mean the prios, it can only be set by cores | 17:46 |
| hemna | johnthetubaguy, I think other places were calling check_attach for some reason outside of the normal volume attach process | 17:46 |
| hemna | I didn't understand those | 17:46 |
| hemna | hence, leaving it and the small refactor | 17:47 |
| hemna | I can try finding those other cases and ask about those fwiw. | 17:47 |
| ildikov | hemna: I can try to check those | 17:47 |
| *** e0ne has joined #openstack-cinder | 17:47 | |
| johnthetubaguy | hemna: ildikov: I remember we spoke about them at some point, but I don't remember what they were now... | 17:48 |
| *** ducttape_ has quit IRC | 17:48 | |
| ildikov | hemna: I spent some time with check_attach during the last cycle, although I have to admit I did not understand everything :) | 17:48 |
| hemna | ok | 17:48 |
| *** earlephilhower has quit IRC | 17:48 | |
| ildikov | hemna: we can both take a look and then sync up on the ones in question | 17:49 |
| johnthetubaguy | so the remove check_attach seems sound, just want to dig a little into the other use cases, just in case | 17:49 |
| hemna | sounds good | 17:49 |
| ildikov | hemna: it should not be that many cases | 17:49 |
| ildikov | johnthetubaguy: cool! | 17:49 |
| johnthetubaguy | the saving of the connection_info seems too hacky, but I am failing to suggest a nice alternative right now, I also want to check what other drivers are doing, as it feels odd this is libvirt specific | 17:50 |
| hemna | johnthetubaguy, yah it does seem hacky | 17:50 |
| hemna | I just threw that up there as a starting point for discussion, hoping to get nova cores to look at it and provide some direction. | 17:51 |
| mriedem | has there been any progress on volume migration / swap volume testing? | 17:51 |
| ildikov | scottda: ^^ | 17:51 |
| *** mtanino has quit IRC | 17:51 | |
| scottda | I've got it all working for retype-with-migration, but I need to get a patch up for the config changes | 17:51 |
| johnthetubaguy | hemna: ack | 17:51 |
| scottda | We've had Tempest tests for this in-tree for a while, but I'm not sure why multi-backend is not configured to have them run. | 17:52 |
| mriedem | scottda: patch in tempest? | 17:52 |
| scottda | yes | 17:52 |
| *** ChubYann has joined #openstack-cinder | 17:53 | |
| mriedem | and last week you said devstack removed multi backend support? | 17:53 |
| scottda | mriedem: No, they just changed the way you do it. | 17:53 |
| mriedem | or are you going to do this in the multinode job? | 17:53 |
| mriedem | oh | 17:53 |
| scottda | mriedem: I saw the patch to remove ENABLE_MULTIBACKEND and was confused. | 17:53 |
| scottda | I want to add multi-node job, there's a patch up for this, but the dev has moved on and I'll pick it up. | 17:54 |
| *** ducttape_ has joined #openstack-cinder | 17:55 | |
| scottda | https://review.openstack.org/#/c/195443/ | 17:56 |
| scottda | That does not work ATM ^^^ | 17:57 |
| *** jgregor has joined #openstack-cinder | 17:57 | |
| scottda | Does anyone know how to change owners on a patch? | 17:57 |
| *** rlrossit has quit IRC | 17:57 | |
| *** mpjetta has joined #openstack-cinder | 17:57 | |
| mriedem | scottda: you don't | 17:58 |
| mriedem | but you can contribute to it | 17:58 |
| scottda | OK, but I need to be able to WIP it, so maybe best to have someone with the powers abandon it, and I'll push up a new one. | 17:58 |
| ildikov | scottda: you can only change Author, but that will not fix the issues you have with it :( | 17:58 |
| mriedem | scottda: just prefix the commit title with WIP: | 17:59 |
| mriedem | while you're working on it | 17:59 |
| scottda | Alright, that's just details. I'll be back to work on this and keep the tests moving forward. | 17:59 |
| mriedem | it already has a -2 | 17:59 |
| ildikov | scottda: if you have a new one up can you add the link to the etherpad? I will add the current for now | 18:00 |
| scottda | sure, that works. | 18:00 |
| * mriedem jumps to another meeting | 18:00 | |
| jgriffith | scottda: you should use the existing one and be a co-author | 18:00 |
| jgriffith | scottda: you're completely enabled to be a comitter and mark as wip etc | 18:01 |
| scottda | jgriffith: ok, I'll do it that way then, thanks | 18:01 |
| *** bardia has joined #openstack-cinder | 18:01 | |
| *** akshai has quit IRC | 18:01 | |
| *** bill_az has joined #openstack-cinder | 18:01 | |
| jgriffith | scottda: unless you're actually going to rewrite everything. If you're basing it on what somebody else started they should get credit | 18:02 |
| *** bill_az has quit IRC | 18:02 | |
| scottda | jgriffith: Yes, no problem with that. I think the existing patch mostly works, something is currently broken but that seems to be a regression. | 18:02 |
| *** timcl1 has joined #openstack-cinder | 18:03 | |
| *** akshai has joined #openstack-cinder | 18:03 | |
| jgriffith | scottda: darn bugs :) | 18:03 |
| scottda | Yeah, if it weren't for those..... | 18:03 |
| smcginnis | scottda: Thanks for picking that one up. I thing we need that coverage regardless. | 18:04 |
| scottda | smcginnis: Yeah, everywhere I look we need coverage. But this also helps with testing Nova swap_volume, so double-good. | 18:04 |
| ildikov | hemna: scottda smcginnis johnthetubaguy jgriffith mriedem: regarding the new meeting slot, would Monday 1700UTC work for everyone? | 18:05 |
| ildikov | Tuesday seems colliding for a few | 18:05 |
| jgriffith | ildikov: sure why not :) | 18:05 |
| johnthetubaguy | that works for me | 18:05 |
| hemna | fine with me | 18:05 |
| *** timcl has quit IRC | 18:05 | |
| scottda | Works for me | 18:05 |
| mriedem | ildikov: i'm on vacation next week | 18:05 |
| smcginnis | ildikov: That would work for me. | 18:05 |
| ildikov | mriedem: that I know, I meant in general | 18:06 |
| smcginnis | mriedem: You can still connect in. Come on, where's your commitment? :P | 18:06 |
| mriedem | keep in mind monday 5/30 is a US holiday | 18:06 |
| * johnthetubaguy goes to make dinner | 18:06 | |
| ildikov | mriedem: I already asked johnthetubaguy to participate as you'll be out | 18:06 |
| mriedem | otherwise monday at 1700 works for me | 18:06 |
| ildikov | mriedem: ok, I officially give up meeting organization... | 18:07 |
| ildikov | :) | 18:07 |
| *** rlrossit has joined #openstack-cinder | 18:07 | |
| ildikov | ok, then let's go with Mondays in general and we will have a backup slot for 5/30 | 18:07 |
| ildikov | mriedem: thanks for the heads up! | 18:08 |
| smcginnis | ildikov: Sounds like a plan. | 18:08 |
| ildikov | mriedem: and also have a nice vacation | 18:08 |
| *** asselin__ has quit IRC | 18:08 | |
| mriedem | thanks | 18:08 |
| ildikov | thank you all, I will register the slot for Mondays 1700UTC | 18:09 |
| scottda | thanks ildikov | 18:09 |
| ildikov | ok, patch proposed, fingers crossed :) | 18:13 |
| *** akshai has quit IRC | 18:16 | |
| *** mtanino has joined #openstack-cinder | 18:18 | |
| *** akshai has joined #openstack-cinder | 18:18 | |
| *** rcernin has quit IRC | 18:31 | |
| *** lpetrut has joined #openstack-cinder | 18:34 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: DO NOT MERGE. Testing job https://review.openstack.org/317668 | 18:36 |
| *** salv-orl_ has quit IRC | 18:37 | |
| *** timcl1 has quit IRC | 18:41 | |
| *** akshai has quit IRC | 18:43 | |
| *** bill_az has joined #openstack-cinder | 18:43 | |
| *** akerr has quit IRC | 18:49 | |
| *** akerr has joined #openstack-cinder | 18:50 | |
| *** sdague has joined #openstack-cinder | 18:51 | |
| *** akshai has joined #openstack-cinder | 18:59 | |
| *** timcl has joined #openstack-cinder | 19:01 | |
| *** akshai_ has joined #openstack-cinder | 19:02 | |
| *** timcl has quit IRC | 19:03 | |
| smcginnis | Grumble grumble grumble gerrit grumble... | 19:04 |
| *** akshai has quit IRC | 19:04 | |
| openstackgerrit | Merged openstack/cinder: Make api_microversion_dev more explicit https://review.openstack.org/317460 | 19:05 |
| openstackgerrit | Merged openstack/cinder: Updated from global requirements https://review.openstack.org/316343 | 19:05 |
| Swanson | smcginnis, living the gerrit dream? | 19:07 |
| *** timcl has joined #openstack-cinder | 19:11 | |
| smcginnis | Always... | 19:11 |
| *** lpetrut has quit IRC | 19:14 | |
| *** lcurtis has joined #openstack-cinder | 19:14 | |
| *** lpetrut has joined #openstack-cinder | 19:16 | |
| *** Lee1092 has quit IRC | 19:22 | |
| *** akshai_ has quit IRC | 19:25 | |
| *** sdake has joined #openstack-cinder | 19:26 | |
| *** daneyon has joined #openstack-cinder | 19:30 | |
| *** sdake has quit IRC | 19:32 | |
| *** sdake_ has joined #openstack-cinder | 19:32 | |
| *** mriedem has quit IRC | 19:35 | |
| *** salv-orlando has joined #openstack-cinder | 19:35 | |
| *** mriedem has joined #openstack-cinder | 19:41 | |
| *** e0ne has quit IRC | 19:48 | |
| *** e0ne has joined #openstack-cinder | 19:51 | |
| *** sdake_ has quit IRC | 19:53 | |
| *** e0ne has quit IRC | 19:54 | |
| *** ccesario has quit IRC | 19:54 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Cleanup for Volume Object(Part 3) https://review.openstack.org/317698 | 19:55 |
| *** sdake has joined #openstack-cinder | 19:58 | |
| *** rcernin has joined #openstack-cinder | 19:58 | |
| *** e0ne has joined #openstack-cinder | 20:01 | |
| *** e0ne has quit IRC | 20:04 | |
| *** sheel has quit IRC | 20:15 | |
| *** lpetrut has quit IRC | 20:18 | |
| smcginnis | eharney: ping | 20:22 |
| *** daneyon_ has joined #openstack-cinder | 20:23 | |
| *** rlrossit has quit IRC | 20:24 | |
| smcginnis | eharney: nm, was going to ask why you approved/unapproved this one: https://review.openstack.org/#/c/316348/ | 20:24 |
| smcginnis | eharney: But no CI results yet, I'm guessing. | 20:24 |
| *** akerr has quit IRC | 20:25 | |
| *** daneyon_ has quit IRC | 20:25 | |
| *** akshai has joined #openstack-cinder | 20:25 | |
| *** akshai_ has joined #openstack-cinder | 20:26 | |
| *** daneyon has quit IRC | 20:27 | |
| *** daneyon has joined #openstack-cinder | 20:29 | |
| *** akshai has quit IRC | 20:30 | |
| *** daneyon has quit IRC | 20:31 | |
| *** ntpttr_ has quit IRC | 20:32 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Skip dict conversion when no volumes are updated https://review.openstack.org/317728 | 20:33 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for failover_host failback https://review.openstack.org/310064 | 20:37 |
| *** timcl has quit IRC | 20:37 | |
| *** daneyon has joined #openstack-cinder | 20:40 | |
| *** rcernin has quit IRC | 20:41 | |
| *** cknight1 has quit IRC | 20:45 | |
| *** gouthamr has quit IRC | 20:46 | |
| *** daneyon has quit IRC | 20:47 | |
| *** raildo is now known as raildo-afk | 20:47 | |
| *** daneyon_ has joined #openstack-cinder | 20:49 | |
| dulek | Can someone take look on https://review.openstack.org/#/c/297699/ ? It's pretty trivial and already got +2s. | 20:50 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove support for 1.x volume RPC API https://review.openstack.org/297701 | 20:51 |
| *** daneyon has joined #openstack-cinder | 20:52 | |
| *** ntpttr_ has joined #openstack-cinder | 20:53 | |
| *** daneyon_ has quit IRC | 20:55 | |
| *** sdake_ has joined #openstack-cinder | 20:59 | |
| *** salv-orlando has quit IRC | 20:59 | |
| *** salv-orlando has joined #openstack-cinder | 21:00 | |
| eharney | smcginnis: yeah, it looks pretty safe, but i figured i'd see if any showed up | 21:00 |
| *** sdake has quit IRC | 21:00 | |
| *** julim has quit IRC | 21:01 | |
| *** bill_az has quit IRC | 21:03 | |
| smcginnis | eharney: Yeah, good plan. | 21:05 |
| *** bill_az has joined #openstack-cinder | 21:05 | |
| *** porrua has quit IRC | 21:06 | |
| *** raunak has quit IRC | 21:07 | |
| *** dustins has quit IRC | 21:12 | |
| *** raunak has joined #openstack-cinder | 21:13 | |
| *** sdague has quit IRC | 21:14 | |
| *** ccesario has joined #openstack-cinder | 21:25 | |
| *** baumann has left #openstack-cinder | 21:26 | |
| *** jgregor has quit IRC | 21:28 | |
| *** jamielennox is now known as jamielennox|away | 21:30 | |
| ntpttr_ | smcginnis: when you have a minute mind taking a look at https://review.openstack.org/#/c/230622/ ? | 21:33 |
| *** haplo37 has joined #openstack-cinder | 21:37 | |
| *** jamielennox|away is now known as jamielennox | 21:39 | |
| *** caspinol has joined #openstack-cinder | 21:39 | |
| *** ametts has quit IRC | 21:41 | |
| *** edmondsw has quit IRC | 21:45 | |
| smcginnis | ntpttr_: Sure, I'll take a look. | 21:51 |
| *** lprice has quit IRC | 21:53 | |
| *** ametts has joined #openstack-cinder | 21:54 | |
| *** sdake_ is now known as sdake | 21:54 | |
| *** mriedem has quit IRC | 21:55 | |
| *** kongwei has joined #openstack-cinder | 22:00 | |
| *** ozialien10 has quit IRC | 22:04 | |
| *** ozialien10 has joined #openstack-cinder | 22:04 | |
| kongwei | eharney: Thank you for the code review (https://review.openstack.org/#/c/312355). We will fix ASAP. | 22:07 |
| *** ametts has quit IRC | 22:07 | |
| *** markvoelker has joined #openstack-cinder | 22:14 | |
| *** ducttape_ has quit IRC | 22:17 | |
| *** tpsilva has quit IRC | 22:17 | |
| *** xyang1 has quit IRC | 22:19 | |
| *** markvoelker has quit IRC | 22:21 | |
| *** markvoelker has joined #openstack-cinder | 22:22 | |
| *** dansmith has quit IRC | 22:23 | |
| *** dansmith has joined #openstack-cinder | 22:23 | |
| *** JoseMello has quit IRC | 22:29 | |
| *** aorourke__ is now known as aorourke | 22:30 | |
| *** aunnam_ has joined #openstack-cinder | 22:33 | |
| *** kongwei has quit IRC | 22:38 | |
| *** karthikp has quit IRC | 22:42 | |
| *** sdake_ has joined #openstack-cinder | 22:45 | |
| openstackgerrit | Merged openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 22:46 |
| *** sdake has quit IRC | 22:48 | |
| *** haplo37 has quit IRC | 22:49 | |
| *** sticker has joined #openstack-cinder | 22:49 | |
| *** alkhodos has quit IRC | 22:52 | |
| *** kfarr has quit IRC | 22:54 | |
| *** sdake_ has quit IRC | 22:54 | |
| *** r-daneel has joined #openstack-cinder | 23:01 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR Clean up VLUN deletion on detach https://review.openstack.org/317761 | 23:02 |
| *** diogogmt has quit IRC | 23:03 | |
| *** diogogmt has joined #openstack-cinder | 23:05 | |
| openstackgerrit | Jay Mehta proposed openstack/cinder: Implemented unit test cases for HPE 3PAR driver https://review.openstack.org/317763 | 23:07 |
| *** markvoelker has quit IRC | 23:12 | |
| *** karthikp has joined #openstack-cinder | 23:18 | |
| *** lcurtis has quit IRC | 23:19 | |
| *** salv-orlando has quit IRC | 23:21 | |
| *** caspinol has quit IRC | 23:22 | |
| *** sdake has joined #openstack-cinder | 23:24 | |
| *** dansmith has quit IRC | 23:31 | |
| *** dansmith has joined #openstack-cinder | 23:32 | |
| *** daneyon has quit IRC | 23:35 | |
| *** IlyaG has quit IRC | 23:50 | |
| *** EinstCrazy has quit IRC | 23:55 | |
| *** karthikp has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!