Friday, 2017-12-01

*** felipemonteiro has quit IRC00:01
*** rcernin has quit IRC00:02
*** rcernin has joined #openstack-cinder00:02
*** mchlumsky has quit IRC00:04
*** ntpttr_laptop has joined #openstack-cinder00:06
*** chhavi has joined #openstack-cinder00:07
*** rcernin has quit IRC00:07
*** rcernin has joined #openstack-cinder00:07
*** chhavi has quit IRC00:11
*** david-lyle has quit IRC00:15
*** sdague has quit IRC00:22
*** jmlowe has quit IRC00:23
*** david-lyle has joined #openstack-cinder00:29
*** felipemonteiro_ has quit IRC00:34
*** david-lyle has quit IRC00:34
*** salv-orlando has joined #openstack-cinder00:42
openstackgerritTommyLike proposed openstack/python-cinderclient master: Remove 'end_version' parameter in backup update  https://review.openstack.org/52443200:46
*** salv-orlando has quit IRC00:46
matt_ejgriffith: thanks, will check that out00:49
*** armax has quit IRC00:54
*** peter-wang has joined #openstack-cinder01:08
*** yangyapeng has joined #openstack-cinder01:09
*** ntpttr_laptop has quit IRC01:10
*** Apoorva_ has joined #openstack-cinder01:17
*** Apoorva has quit IRC01:21
*** Apoorva_ has quit IRC01:22
*** peter-wang has quit IRC01:23
*** sticker has joined #openstack-cinder01:25
*** zhurong has joined #openstack-cinder01:35
*** salv-orlando has joined #openstack-cinder01:43
openstackgerritMerged openstack/cinder master: VMware: Backend driver for VStorageObject  https://review.openstack.org/52121301:43
*** salv-orlando has quit IRC01:48
*** v12aml has quit IRC01:53
*** gnufied has quit IRC02:05
*** gmann_afk is now known as gmann02:07
*** _dalgaaf has quit IRC02:08
openstackgerritTommyLike proposed openstack/cinder-specs master: Support restore backup to volumes simultaneously  https://review.openstack.org/52444102:12
*** AlexeyAbashkin has joined #openstack-cinder02:12
*** vivsoni_ has joined #openstack-cinder02:12
*** niedbalski_ has joined #openstack-cinder02:13
*** dalgaaf has joined #openstack-cinder02:13
*** rcernin_ has joined #openstack-cinder02:14
*** EmilienM_ has joined #openstack-cinder02:15
*** slunkad_ has joined #openstack-cinder02:16
*** AlexeyAbashkin has quit IRC02:16
*** Anticime1 has joined #openstack-cinder02:18
*** rcernin has quit IRC02:21
*** niedbalski has quit IRC02:21
*** vivsoni has quit IRC02:21
*** Dinesh__Bhor has joined #openstack-cinder02:21
*** zhurong has quit IRC02:22
*** abhi has quit IRC02:22
*** eharney has quit IRC02:22
*** markvoelker has quit IRC02:22
*** EmilienM has quit IRC02:22
*** Dinesh_Bhor has quit IRC02:22
*** Anticimex has quit IRC02:22
*** zigo has quit IRC02:22
*** tomh__ has quit IRC02:22
*** slunkad has quit IRC02:22
*** EmilienM_ is now known as EmilienM02:22
*** EmilienM has quit IRC02:22
*** EmilienM has joined #openstack-cinder02:22
*** dr_gogeta86_ has quit IRC02:24
*** dr_gogeta86 has joined #openstack-cinder02:27
*** dr_gogeta86 has quit IRC02:27
*** dr_gogeta86 has joined #openstack-cinder02:27
*** lkwan is now known as 17SAA4WSA02:27
*** zigo has joined #openstack-cinder02:27
*** lkwan has joined #openstack-cinder02:27
*** abhi has joined #openstack-cinder02:27
*** markvoelker has joined #openstack-cinder02:27
*** tomh__ has joined #openstack-cinder02:27
*** cheneydc has joined #openstack-cinder02:27
*** zigo is now known as Guest1326802:29
*** rhedlind has quit IRC02:29
*** dhinesh has quit IRC02:30
*** nicolasbock has quit IRC02:30
*** rhedlind has joined #openstack-cinder02:31
*** dave-mccowan has joined #openstack-cinder02:31
*** mriedem has quit IRC02:33
openstackgerritJack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector  https://review.openstack.org/51077602:37
*** jmlowe has joined #openstack-cinder02:42
*** salv-orlando has joined #openstack-cinder02:44
*** salv-orlando has quit IRC02:49
*** dave-mccowan has quit IRC02:51
*** itlinux has joined #openstack-cinder02:53
*** v12aml has joined #openstack-cinder03:07
tommylikehujgriffith: , smcginnis : please take a look: https://review.openstack.org/#/c/524432/03:11
*** masber has joined #openstack-cinder03:14
*** felipemonteiro has joined #openstack-cinder03:23
openstackgerritMerged openstack/cinder master: Remove extra test variable assignment  https://review.openstack.org/52439803:24
openstackgerritMerged openstack/cinder master: Discrepancy in api-ref and code for create group snapshot API  https://review.openstack.org/52412003:24
*** cheneydc has quit IRC03:24
*** felipemonteiro_ has joined #openstack-cinder03:26
*** felipemonteiro has quit IRC03:29
*** felipemonteiro_ has quit IRC03:31
*** abhishekk has joined #openstack-cinder03:34
openstackgerritMerged openstack/cinder master: Add shared_targets flag to Volumes  https://review.openstack.org/52067603:43
*** salv-orlando has joined #openstack-cinder03:45
*** salv-orlando has quit IRC03:49
openstackgerritJohnnyChou proposed openstack/cinder master: Re-add Infortrend Cinder volume driver  https://review.openstack.org/52365903:50
*** rcernin has joined #openstack-cinder03:50
*** rcernin_ has quit IRC03:51
*** namnh has joined #openstack-cinder03:52
*** links has joined #openstack-cinder03:55
*** psachin has joined #openstack-cinder04:23
*** threestrands_ has joined #openstack-cinder04:24
*** threestrands_ has joined #openstack-cinder04:24
*** threestrands has quit IRC04:26
vivsoni_Hi Team, please review patch https://review.openstack.org/#/c/524161/04:40
*** itlinux has quit IRC04:40
*** pgadiya has joined #openstack-cinder04:46
*** salv-orlando has joined #openstack-cinder04:46
*** salv-orlando has quit IRC04:50
*** rcernin_ has joined #openstack-cinder05:09
*** rcernin has quit IRC05:09
*** david-lyle has joined #openstack-cinder05:13
*** itlinux has joined #openstack-cinder05:17
*** chhavi has joined #openstack-cinder05:35
*** sticker has quit IRC05:36
*** gouthamr has quit IRC05:44
*** salv-orlando has joined #openstack-cinder05:47
*** salv-orlando has quit IRC05:51
*** armax has joined #openstack-cinder06:01
*** junboli has joined #openstack-cinder06:02
openstackgerritNguyen Van Trung proposed openstack/cinder-specs master: Update bindep.txt for doc builds  https://review.openstack.org/52241806:06
*** abhitechie has quit IRC06:10
*** pcaruana has joined #openstack-cinder06:10
*** salv-orlando has joined #openstack-cinder06:14
*** itlinux has quit IRC06:22
*** threestrands_ has quit IRC06:24
*** armax has quit IRC06:27
*** david-lyle has quit IRC06:30
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/52448306:33
*** dhinesh has joined #openstack-cinder06:39
*** dhinesh has quit IRC06:44
openstackgerritPooja Jadhav proposed openstack/cinder master: Discrepancy in api-ref and code for create group type API  https://review.openstack.org/52449206:47
*** wxy_ is now known as wxy06:51
pooja_jadhavabhishekk: Hi06:56
abhishekkpooja_jadhav: Hi06:56
pooja_jadhavabhishekk: Want to discuss about review comments, is_public_1 is for generic volume group.06:57
pooja_jadhavabhishekk: So I have added new parameter.06:57
abhishekkpooja_jadhav: still you can use that as there is no difference between description of 1 and 206:58
abhishekkpooja_jadhav: if something is different then yes is_public_2 is required but as I can see no change between is_public_1 and is_public_2 then no need to add is_public_206:59
abhishekkpooja_jadhav: hope your doubts are clear06:59
pooja_jadhavabhishekk: yes u r right, but I have changed description from group to group type.07:00
pooja_jadhavabhishekk: please correct me if i am wrong.07:00
abhishekkpooja_jadhav: my bad :)07:00
abhishekkpooja_jadhav: thank you for correcting me07:02
pooja_jadhavabhishekk: :)07:03
pooja_jadhavabhishekk: Thanks for review :)07:06
openstackgerritNiraj Singh proposed openstack/cinder master: Add service_token for cinder-nova interaction  https://review.openstack.org/52449707:09
*** e0ne has joined #openstack-cinder07:27
*** rcernin_ has quit IRC07:29
*** adisky_ has joined #openstack-cinder07:43
gmannsmcginnis: this is ready now - https://review.openstack.org/#/c/523354/07:44
*** diablo_rojo has quit IRC08:01
*** e0ne has quit IRC08:03
openstackgerritJohnnyChou proposed openstack/cinder master: Re-add Infortrend Cinder volume driver  https://review.openstack.org/52365908:03
*** salv-orlando has quit IRC08:06
*** alexchadin has joined #openstack-cinder08:10
*** AlexeyAbashkin has joined #openstack-cinder08:15
*** gkadam has joined #openstack-cinder08:24
*** kwathore has joined #openstack-cinder08:29
*** kwathore_ has quit IRC08:31
*** armaan has joined #openstack-cinder08:32
*** salv-orlando has joined #openstack-cinder08:33
*** amoralej|off is now known as amoralej08:34
*** armaan has quit IRC08:42
openstackgerritTadas Ustinavičius proposed openstack/cinder master: Fix for volume from image snapshot free space issue  https://review.openstack.org/52433108:43
*** moshele has quit IRC08:46
*** gmann is now known as gmann_afk08:54
*** armaan has joined #openstack-cinder08:57
*** psachin` has joined #openstack-cinder08:59
openstackgerritPooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Group Snapshots  https://review.openstack.org/52452808:59
openstackgerritNeha Alhat proposed openstack/cinder master: workers cleanup api raises TypeError for invalid resource_type value  https://review.openstack.org/52453009:00
*** psachin has quit IRC09:00
*** pgadiya has quit IRC09:07
openstackgerrityuyafei proposed openstack/cinder master: Pass in the parameters multipath when migrating volume  https://review.openstack.org/52453609:10
openstackgerritNiraj Singh proposed openstack/cinder master: Add service_token for cinder-nova interaction  https://review.openstack.org/52449709:16
*** e0ne has joined #openstack-cinder09:16
*** Dinesh__Bhor has quit IRC09:18
*** Dinesh__Bhor has joined #openstack-cinder09:19
*** pgadiya has joined #openstack-cinder09:23
*** lhx_ has joined #openstack-cinder09:24
*** psachin`` has joined #openstack-cinder09:29
*** psachin` has quit IRC09:30
*** alexchadin has quit IRC09:37
openstackgerritTommyLike proposed openstack/cinder-specs master: Support restore backup to volumes simultaneously  https://review.openstack.org/52444109:49
*** ganso has joined #openstack-cinder09:51
openstackgerritTommyLike proposed openstack/cinder-specs master: Support restore backup to volumes simultaneously  https://review.openstack.org/52444109:53
*** markvoelker has quit IRC09:55
*** alexchadin has joined #openstack-cinder09:55
*** junboli has quit IRC09:56
*** sri_ has joined #openstack-cinder09:58
*** adisky_ has quit IRC10:00
*** abhishekk has quit IRC10:02
*** alexchadin has quit IRC10:07
*** armaan has quit IRC10:10
*** armaan has joined #openstack-cinder10:10
*** hoangcx has quit IRC10:12
*** alexchadin has joined #openstack-cinder10:13
*** pgadiya has quit IRC10:14
*** namnh has quit IRC10:20
*** pgadiya has joined #openstack-cinder10:28
*** openstackgerrit has quit IRC10:33
*** gaurangt has quit IRC10:39
*** gaurangt has joined #openstack-cinder10:40
*** armaan has quit IRC10:44
*** armaan has joined #openstack-cinder10:44
*** markvoelker has joined #openstack-cinder10:55
*** kaisers_ has joined #openstack-cinder11:13
*** lhx_ has quit IRC11:14
*** lhx_ has joined #openstack-cinder11:14
*** lhx__ has joined #openstack-cinder11:19
*** lhx_ has quit IRC11:20
*** alexchadin has quit IRC11:26
*** alexchadin has joined #openstack-cinder11:27
*** Dinesh__Bhor has quit IRC11:30
*** zhonghua2 has quit IRC11:31
*** zhonghua2 has joined #openstack-cinder11:31
*** alexchadin has quit IRC11:34
*** alexchadin has joined #openstack-cinder11:34
*** seitan has joined #openstack-cinder11:38
*** nicolasbock has joined #openstack-cinder11:43
*** salv-orlando has quit IRC11:49
*** salv-orlando has joined #openstack-cinder11:50
*** salv-orlando has quit IRC11:55
*** alexchadin has quit IRC12:10
*** eharney has joined #openstack-cinder12:14
*** schmots has joined #openstack-cinder12:18
*** armaan has quit IRC12:20
*** Jack_Iv has joined #openstack-cinder12:27
*** alexchadin has joined #openstack-cinder12:29
*** psachin`` has quit IRC12:33
*** dr_gogeta86 has left #openstack-cinder12:38
*** lhx__ has quit IRC12:41
*** lhx__ has joined #openstack-cinder12:42
*** armaan has joined #openstack-cinder12:46
*** abishop has joined #openstack-cinder12:47
*** yangyapeng has quit IRC12:53
*** yangyapeng has joined #openstack-cinder12:57
*** yangyapeng has quit IRC13:01
*** liverpooler has joined #openstack-cinder13:04
*** seitan has quit IRC13:10
*** lhx__ has quit IRC13:12
*** pckizer has quit IRC13:15
*** links has quit IRC13:17
*** pck has joined #openstack-cinder13:23
*** moshele has joined #openstack-cinder13:25
*** markvoelker has quit IRC13:25
*** markvoelker has joined #openstack-cinder13:25
*** moshele has quit IRC13:30
*** pck has quit IRC13:30
*** pck has joined #openstack-cinder13:30
*** yangyapeng has joined #openstack-cinder13:34
nsinghi am using "noauth" auth_strategy. Getting 400 bad request Dec 01 18:40:27 openstack-VirtualBox devstack@c-api.service[30723]: DEBUG cinder.api.openstack.wsgi [None req-1e0fbbee-357f-4198-866c-a4c4e10baff8 None None] Empty body provided in request {{(pid=30725) get_body /opt/stack/cinder/cinder/api/openstack/wsgi.py:718}}13:36
*** lhx_ has joined #openstack-cinder13:39
*** pgadiya has quit IRC13:40
*** stephenfin is now known as finucannot13:40
*** openstackgerrit has joined #openstack-cinder13:43
openstackgerritAndreas Jaeger proposed openstack/os-brick master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52460313:43
*** mriedem has joined #openstack-cinder13:48
-openstackstatus- NOTICE: gerrit has been restarted to get it back to its normal speed.13:50
*** salv-orlando has joined #openstack-cinder13:52
*** itlinux has joined #openstack-cinder13:53
*** salv-orlando has quit IRC13:57
*** links has joined #openstack-cinder13:58
*** nicolasbock has quit IRC13:59
*** AlexeyAbashkin has quit IRC14:02
*** liverpooler has quit IRC14:03
*** MeltedLux has quit IRC14:03
*** zhaochao has quit IRC14:07
*** erlon has joined #openstack-cinder14:13
*** amoralej is now known as amoralej|lunch14:14
*** salv-orlando has joined #openstack-cinder14:15
*** jmlowe has quit IRC14:16
*** ildikov is now known as coffee_cat14:20
*** liverpooler has joined #openstack-cinder14:22
*** dansmith is now known as superdan14:25
*** alexchadin has quit IRC14:25
*** links has quit IRC14:26
*** Jack_Iv has quit IRC14:30
*** armaan has quit IRC14:34
*** armaan has joined #openstack-cinder14:35
*** dustins has joined #openstack-cinder15:02
*** crose has joined #openstack-cinder15:03
*** amoralej|lunch is now known as amoralej15:05
*** david-lyle has joined #openstack-cinder15:07
*** gnufied has joined #openstack-cinder15:08
openstackgerritMerged openstack/python-cinderclient master: Add cluster support in migration and manage  https://review.openstack.org/38750015:10
*** gouthamr has joined #openstack-cinder15:11
*** moshele has joined #openstack-cinder15:18
*** MeltedLux has joined #openstack-cinder15:21
*** armaan has quit IRC15:24
*** schmots_ has joined #openstack-cinder15:25
*** schmots has quit IRC15:25
*** schmots_ is now known as schmots15:25
*** felipemonteiro has joined #openstack-cinder15:26
*** felipemonteiro_ has joined #openstack-cinder15:28
*** armaan has joined #openstack-cinder15:31
*** felipemonteiro has quit IRC15:31
*** armax has joined #openstack-cinder15:32
*** moshele has quit IRC15:33
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - support for manage/unmanage snapshot  https://review.openstack.org/51873515:36
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Incorrect stats reporting  https://review.openstack.org/51938315:38
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - revert a volume to last snapshot  https://review.openstack.org/52065915:40
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Fix error handling and checks for generic volume groups  https://review.openstack.org/52108015:42
*** ntpttr_laptop has joined #openstack-cinder15:43
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Errors extending replicated volumes  https://review.openstack.org/52421515:44
*** jmlowe has joined #openstack-cinder15:47
*** armaan has quit IRC15:49
*** armaan has joined #openstack-cinder15:49
*** david-lyle has quit IRC15:53
*** Apoorva has joined #openstack-cinder16:04
*** imacdonn has quit IRC16:11
*** imacdonn has joined #openstack-cinder16:11
*** erlon has quit IRC16:22
*** david-lyle has joined #openstack-cinder16:22
*** armax has quit IRC16:27
*** e0ne has quit IRC16:39
*** itlinux has quit IRC16:53
*** ntpttr_laptop has quit IRC16:55
jgriffithstupid f'ing piece of s*& worst design ever f'ing grumble grumble grumble17:02
* jungleboyj pats jgriffith on the back17:03
jungleboyjThere there.17:03
*** armax has joined #openstack-cinder17:11
jungleboyjjgriffith:  So you are just going to leave it at that and no more details?17:13
*** AlexeyAbashkin has joined #openstack-cinder17:13
jgriffithjungleboyj: yeah, I'm quiting at this point17:17
jgriffithjust not worth it any more17:17
*** AlexeyAbashkin has quit IRC17:17
jungleboyjThis the multi-attach stuff?17:18
jgriffithjungleboyj: no, it's Cinder in general17:18
* jgriffith is shutting up now17:18
jungleboyjjgriffith: Ok.  Sorry you are feeling that way.17:19
jgriffithjungleboyj: me too :)17:19
jungleboyjjgriffith:  This too shall pass.  It has been a long week.17:19
coffee_catjgriffith: we could rewrite it over the holidays, when no one sees :)17:19
*** itlinux has joined #openstack-cinder17:20
* jungleboyj looks the other way17:21
*** crose has quit IRC17:23
*** crose has joined #openstack-cinder17:24
_alastor__jgriffith:  It's still a better design than pretty much any external persistent storage implementation in the other ecosystems.  I was reading over the Kube CSI spec and had a bit of an ugly cry17:26
coffee_catjgriffith: ok, sorry, back to serious17:26
coffee_catjgriffith: is it the tests again or the microversions or everything?17:26
_alastor__I've been working with mesos all week though.  It's been rough...17:30
openstackgerritMerged openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/52448317:31
jungleboyj_alastor__:  Thank you.  I agree.  When I went and tried to find another solution that does what we can do ... nada.17:32
_alastor__jungleboyj:  Weird bumps and oddities aside, I think Cinder is where it's at for total functionality.  I'm still running into ecosystems that have three volume operations: Get, Mount, Unmount and expect that to work for everybody17:34
_alastor__jungleboyj:  read through this if you haven't for an example: https://dcos.io/docs/1.10/storage/external-storage/17:35
_alastor__jungleboyj:  It gets really good around this point : https://dcos.io/docs/1.10/storage/external-storage/#implicit-volumes17:36
*** AlexeyAbashkin has joined #openstack-cinder17:37
jungleboyj_alastor__:  OMG and my container guy was really excited to share that with me the other day.17:37
_alastor__jungleboyj:  I have hacks on top of hacks just trying to get basic functionality into that volume lifecycle17:38
jgriffith_alastor__: good luck :)17:38
_alastor__jgriffith: Thanks, I'll need it17:39
jgriffith_alastor__: might I interest you in something like this:  https://github.com/kubernetes-incubator/external-storage/tree/master/openstack/standalone-cinder17:39
jgriffith_alastor__: and BTW there's a CSI shim in the works, CSI--->Cinder as well17:40
_alastor__jgriffith: Adapters and adapters for your adapters ;)17:40
jgriffith_alastor__: ha!  Well, I have an idea for a Cinder API module17:40
jgriffithbut I may never get around to it17:41
jgriffithsort of like the old ec2 and ebs modules we used to have17:41
smcginnisjgriffith: Direct CSI interface in Cinder?17:41
jungleboyj_alastor__: Thanks for sharing that.  I need to look closer at that.17:41
jgriffithsmcginnis: yeah, in the Cinder tree17:41
*** AlexeyAbashkin has quit IRC17:41
smcginnisI've thought about the same thing. Maybe that would be a good thing for us to do.17:42
jgriffithsmcginnis: and best of all NO FING MicroVersions!!!17:42
smcginnis:)17:42
jgriffithsmcginnis: I think it's *the* thing for us to do17:42
_alastor__jgriffith:  does it look like this? https://i.stack.imgur.com/9LY6l.jpg17:42
*** Apoorva has quit IRC17:42
smcginnis_alastor__: A couple more adapters in there at least. :)17:42
jgriffith_alastor__: LOL17:42
SwansonPutting up a spec for microversion replication.17:42
jgriffithLooks better than the one under my desk :)17:43
jgriffithSwanson: don't think that I won't kill you17:43
jgriffithSwanson: you won't be the first (or last)17:43
jgriffithI have a wood chipper and I'm not afraid to use it17:43
smcginnisSome nice secluded areas around there too.17:44
jungleboyjWell, this just got dark.17:45
jgriffithYeah, and just for reference, there are plenty of viable options :)17:45
jgriffithFun fact.. > 40% of homicides go unsolved17:46
jgriffithThat's just the ones where the body is found17:46
jgriffithjust sayin17:46
smcginnisHah17:46
_alastor__kek17:46
jungleboyjWow.17:46
jgriffithKeep that in mind jungleboyj17:47
jgriffith:)17:47
coffee_catjgriffith: why do you know that number exactly?17:47
smcginnisBetter not to ask.17:47
coffee_catI guess no one will be surprised if I disappear now :)17:48
jgriffithcoffee_cat: nah... you're safe, but don't ask too many questions :)17:48
* jungleboyj is not as scared of jgriffith as I used to be. ;-)17:49
coffee_catjgriffith: yeah, well, that's never been my strong suite :)17:49
jgriffithHa!17:50
jgriffithCome on now... I'm the "nicest" person in Cinder, after....17:50
jgriffithwell...17:50
jgriffitheveryone17:50
coffee_catI think I got stuck with multi-attach as I asked 'why?' too many times...17:50
jgriffithcoffee_cat: are you volunteering to write the code?17:51
coffee_catjgriffith: are you asking me after the 165th revision of the new attach patch?17:51
SwansonJust picturing the moment in a customer meeting: "The dell driver does microversion replication. Do you?" *boom*17:52
jgriffithcoffee_cat: as I've said for 6 months, that's not even multi-attach17:52
coffee_catjgriffith: I guess we can catfight about who starts the policy implementation in Cinder17:53
coffee_catjgriffith: oh, I put one of the multi-attach patches on top of that now17:53
jgriffithbut lets not ruin what could otherwise be a tolerable day by bringing up that discussion :)17:53
coffee_catjgriffith: so I think it qualifies ;)17:53
coffee_catjgriffith: so you say that cursing several times on the channel is a tolerable day, but if I bring up policies then it's suddenly ruined?17:55
* coffee_cat asked another question, that might be dangerous...17:55
jgriffithhaha17:58
jgriffithnah17:58
*** lhx_ has quit IRC17:59
*** david-lyle has quit IRC18:02
openstackgerritPeter Penchev proposed openstack/os-brick master: Add the StorPool brick connector  https://review.openstack.org/19263918:04
*** david-lyle has joined #openstack-cinder18:04
*** Apoorva has joined #openstack-cinder18:05
openstackgerritPeter Penchev proposed openstack/cinder master: Reintroduce the StorPool driver  https://review.openstack.org/22015518:05
*** Apoorva has quit IRC18:07
*** chhavi has quit IRC18:10
*** Apoorva has joined #openstack-cinder18:14
*** ntpttr_laptop has joined #openstack-cinder18:19
lbragstado/ smcginnis just a heads up - https://github.com/openstack/cinder/blob/master/setup.cfg#L27 can probably be removed soon, too18:23
*** gkadam has quit IRC18:23
smcginnislbragstad: Yep, have that removed in the policy clean up patch I'm working on. Thanks!18:23
lbragstadsmcginnis: oh - cool, i looked for that review but i didn't see it18:23
*** armaan has quit IRC18:23
lbragstadsmcginnis: do you have a link?18:24
smcginnislbragstad: Yeah, still having issues. Without the policy.json file I am getting a bunch of errors. So I haven't pushed up anything yet.18:24
lbragstadaha - ok, sounds good18:24
smcginnislbragstad: I'm hoping to at least put up a WIP by the end of the day.18:24
smcginnislbragstad: I'll add you on that.18:24
openstackgerritJohn Griffith proposed openstack/cinder master: Add shared_targets and service_uuid to volumes  https://review.openstack.org/52469718:29
*** amoralej is now known as amoralej|off18:29
jgriffithcoffee_cat: ^^ there ya go; 3.4818:30
coffee_catjgriffith: sweet, thanks! :)18:30
coffee_catjgriffith: I hope it'll still be 3.48 when it merges :)18:30
jgriffiththat's up to jungleboyj :)18:30
coffee_catjungleboyj: are you following what happened to the other three microversions in the client: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/api_versions.py#L32 ?18:31
*** crose has quit IRC18:33
jgriffithNothing like an extremely tightly coupled collection of independent modules18:33
*** armaan has joined #openstack-cinder18:34
jgriffithcoffee_cat: jungleboyj FWIW I am now working the patch to update the client with .4818:34
coffee_catjungleboyj: just because I'll be pretty restless to get a client cut with the very latest microversion when jgriffith's patch gets merged; just sayin' :)18:34
smcginnisS&W .48?18:34
coffee_catjgriffith: that's ok, if the other three bumps didn't need anything in the client we can fix the version bumps easily18:35
coffee_catjgriffith: I just don't want to end up with the same discussion as the last time18:35
coffee_catjgriffith: as it seems you're the only one who bumps the client... :/18:35
* jungleboyj is trying to catch up.18:36
jungleboyjOk, so we want to get jgriffith 's patch merged sooner than later.18:37
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Bump Max API version to 3.48  https://review.openstack.org/52470018:37
jungleboyjClient change is coming which we will also want to merge and then rool a new Cinderclient.18:37
coffee_catjungleboyj: that's always the case ;)18:37
jgriffithjungleboyj: just +2/A https://review.openstack.org/#/c/524697/1, then do the same with that last one ^^18:37
jgriffith:)18:37
jungleboyjjgriffith:  Ok.18:37
coffee_catjungleboyj: the current client max_version is 3.4418:37
jgriffithcoffee_cat: https://review.openstack.org/#/c/524700/18:38
jungleboyjsmcginnis: Do we have the microversions between 3.44 and 3.48 fixed?18:38
coffee_catjungleboyj: jgriffith's change bumps it to 3.4818:38
jgriffithjungleboyj: smcginnis there's no impact on the client18:38
jungleboyjYou had just pushed a patch to rever something.18:38
jgriffithjungleboyj: so you don't need to worry about that too much, unless I missed something18:38
coffee_catjungleboyj: the revert was from 3.45, because that was supposed to be the shared_targets bump, but that didn't get merged at that time18:39
jgriffithoh... hmm18:39
smcginnisjungleboyj: Another patch did merge I believe that bumped up the max version.18:39
smcginnisSo I think we are OK, but I can try to take a look later and see if we are missing any actual functionality.18:39
coffee_catjgriffith: you remember the discussion maybe at the PTG about not being able to use things that bumped the version to any between 3.44 and 3.48 in this case?18:39
jgriffithsmcginnis: jungleboyj so not sure but the server side was up to 3.47 prior to my patch so in theory everything is there no?18:40
jungleboyjjgriffith:  In theory.  :-)18:40
coffee_catsmcginnis: well, by now it should be 3.4718:40
jgriffithcoffee_cat: I remember parts of that conversation, and that's also FWIW the reasoning behind me waiting to do the API version bump until the very last step18:40
*** mkoderer_ has joined #openstack-cinder18:41
coffee_catjgriffith: that's fine, you don't need to bump the client patch all the time that way18:41
jgriffiththis lock-step dependency with outstanding changes etc can be challenging, so hopefully this way it's a one-day thing18:41
jgriffithcoffee_cat: well.. you do18:41
jgriffithcoffee_cat: that was the point18:41
coffee_catjgriffith: I mean you don't need to update the patch that much probably due to conflicts if that's the very last step18:42
jgriffithanyway, whatever... if you don't need that patch it can be ignored, if you do that's cool too.  Moving on here18:42
coffee_catjgriffith: we need your patch18:42
jgriffithcoffee_cat: ok... good, it's up :)18:43
*** diablo_rojo has joined #openstack-cinder18:43
coffee_catjgriffith: I just wanted to raise that someone would need to check whether there's any functionality in 3.45, 3.46 and 3.47 that should be in the client before we jump 4 versions again with your patch18:43
coffee_catjungleboyj: smcginnis: ^^18:43
jgriffithgot ya18:44
jungleboyjcoffee_cat:  Right.18:44
coffee_catjgriffith: and I was just reflecting to your comment earlier on doing the API version bump as the last step that it's a good idea, so you have less version conflicts on the way while your patch gets merged18:45
jgriffithshit18:45
coffee_catjgriffith: that was just a side comment18:45
jgriffiththere are missing changes18:45
* coffee_cat hates being right... :/18:46
jungleboyjjgriffith:  That was just what I was seeing too.18:46
jungleboyj3.46 doesn't have any client changes but 3.47 does18:47
coffee_catone way to avoid this is asking for client patch(es) before merging the server side changes18:47
smcginnisI think we talked about that at the train depot (aka Denver PTG)18:48
coffee_catand then get the client changes land as well once the server side parts are in18:48
jungleboyjcoffee_cat:  May have to do that.18:48
jungleboyjThough, this is ok ... right.18:48
*** moshele has joined #openstack-cinder18:48
jungleboyjIt will match the server side.  Problem is there just isn't client support for 3.45 and 3.47 to actually be used.18:49
coffee_catjungleboyj: you might also want to look into the testing that Nova does to avoid some of the inconveniences with this18:49
jungleboyjI will need to go track those people down.18:49
coffee_catsmcginnis: we did and we said it would be at least a change in the review practice18:50
coffee_catjungleboyj: that would be nice, as I said, I will be restless for a new client once jgriffith's server side API change landed :)18:50
smcginnisThat's the problem with the "Hey, reviewers should really remember to check for X" decisions. X gets forgotten in the list of A-V to look for in reviews.18:51
coffee_catsmcginnis: we can write a checkbook like pilots use if you think that would help18:51
jungleboyjAnyway, does what I said above make sense?18:52
*** moshele has quit IRC18:52
coffee_catjungleboyj: we cannot bump to 3.48 before the changes needed for 3.45 and 3.47 land in the client18:53
coffee_catjungleboyj: so I think hunting down the people to fix up the client does make sense18:53
jungleboyjcoffee_cat:  Why can't we?  I know we shouldn't ...18:56
coffee_catjungleboyj: yeah, we physically can, but then you sabotage the availability of the functionality added by 3.45 and 3.4718:57
jungleboyjRight, which will hopefully encourage them to get their code in.  :-)18:59
*** dhinesh has joined #openstack-cinder18:59
coffee_catjungleboyj: tell them to do it quick otherwise I will harass them and they won't like it :)19:00
jungleboyjcoffee_cat:  I guess that works for me.19:01
coffee_catjungleboyj: ok, cool19:01
jungleboyjcoffee_cat:  Let me go figure out who did those two MVs.19:02
jungleboyjI need to eat something firs though.19:02
*** dhinesh has quit IRC19:03
coffee_catjungleboyj: thanks and bon appetite :)19:04
*** Apoorva has quit IRC19:04
*** david-lyle has quit IRC19:05
*** moshele has joined #openstack-cinder19:06
*** Apoorva has joined #openstack-cinder19:07
*** dhinesh has joined #openstack-cinder19:09
jgriffithwe should write a hacking check that scans for MV bumps and if found requires a depends-on client patch in the commit message19:10
coffee_catjgriffith: I like the idea :)19:12
*** dhinesh has quit IRC19:15
jungleboyjYeah, that isn't a bad idea.19:18
*** david-lyle has joined #openstack-cinder19:21
*** schmots_ has joined #openstack-cinder19:29
*** schmots has quit IRC19:29
*** schmots_ is now known as schmots19:29
jungleboyjcoffee_cat: jgriffith Ok, the first change was tommylikehu .  I just pinged him.19:31
jungleboyjSending a note to luqitao now19:31
coffee_catjungleboyj: thanks19:32
*** armaan has quit IRC19:35
*** moshele has quit IRC19:36
*** dklyle has joined #openstack-cinder19:36
*** david-lyle has quit IRC19:37
jungleboyjWe will see if those make a difference.19:39
*** moshele has joined #openstack-cinder19:41
*** dhinesh has joined #openstack-cinder19:42
coffee_catjungleboyj: you can ask best practices from mriedem if not :)19:43
jungleboyjcoffee_cat:  Will do when I see him Monday19:44
mriedem?19:45
mriedemoh19:46
mriedemi see19:46
coffee_catmriedem: also, I know it's over morning but if you could leave a few comments on the new attach patch today that would be great :)19:50
*** dklyle has quit IRC19:57
*** schmots_ has joined #openstack-cinder20:05
*** schmots has quit IRC20:05
*** schmots_ is now known as schmots20:05
*** moshele has quit IRC20:09
*** AlexeyAbashkin has joined #openstack-cinder20:11
*** AlexeyAbashkin has quit IRC20:15
*** MeltedLux has quit IRC20:16
mriedemwhat was it you wanted me to review?20:19
mriedemsomething about attach?20:19
coffee_catmriedem: what? attach? :)20:22
coffee_catmriedem: https://review.openstack.org/#/c/330285/16520:22
coffee_catmriedem: I think the emphasis is on the 165 at the end of that url, so I'm eager to add just a few, preferably not more than 2 or 3 to it and get it landed finally :)20:23
coffee_catmriedem: so get a review now could be a pre-Christmas present for me, which I surely don't deserve, but let's pretend for now that I do20:24
coffee_catmriedem: pretty please :)20:24
*** MeltedLux has joined #openstack-cinder20:24
mriedemlet's say i'm +2 on it, do we then hold it for the shared_targets mv in cinder?20:24
*** armaan has joined #openstack-cinder20:25
*** pcaruana has quit IRC20:25
*** itlinux has quit IRC20:25
coffee_catwe said we would20:25
mriedemok20:25
coffee_catat this point I would say that doing it in a follow up patch wouldn't hurt?20:25
mriedemnot really how it works20:25
mriedemif someone picks up that code in between it's not really the right thing20:26
mriedemanyway, not a bad problem to have, once we're happy20:26
mriedemso let me do that20:26
coffee_catI can harass jungleboyj to get a client out quick20:26
*** itlinux has joined #openstack-cinder20:26
jungleboyjYes dear.20:27
coffee_cator you can advise on how to handle things if we do it in two steps20:27
mriedemit's not the client that we care about20:27
coffee_catbefore the client I can't do a version bump in Nova20:27
coffee_catat this point the shared_targets change is almost there, it's only the API patch and version bump on the server side along with the version bump in the client20:29
coffee_catand without a client release I cannot bump the version we require in Nova20:30
coffee_catwhich is why I referred to the client20:30
jungleboyjAnd I am stuck waiting for people who didn't get their client changes proposed to propose them so we can make progress.20:31
coffee_catmriedem: so if we would end up on volume status check and flag check and microversion check because we don't hold now, then let's hold20:32
mriedem"before the client I can't do a version bump in Nova" ?20:33
mriedemnova just checks the highest version from the server20:33
mriedemthe client gets the version20:33
mriedemnova says, "yes that's good"20:33
coffee_catnot really...20:34
mriedemoh,20:34
mriedemwait,20:34
mriedemit's becaues the version check in the client is capped by what the client supports?20:34
openstackgerritAlan Bishop proposed openstack/cinder master: Migrate fixed_key encryption keys to Barbican  https://review.openstack.org/52472020:34
mriedemhmm, ok20:34
coffee_catmriedem: this is what gets checked at least by one of the Nova tests: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/api_versions.py#L3220:34
mriedemwell,20:34
*** armaan_ has joined #openstack-cinder20:34
mriedemwe could always use ksa to just do a GET / to the block storage API to get the version document and parse that20:35
coffee_catyep, it's all about what the client supports by the end20:35
*** moshele has joined #openstack-cinder20:35
coffee_cathmm, we could, but do we really want to?20:35
*** armaan has quit IRC20:36
*** armaan_ has quit IRC20:37
*** schmots has quit IRC20:39
mriedemnot if i can help it20:40
mriedemreviewing now20:40
coffee_catanyway, so if I need to choose between extending check_attach or rebasing this new attach patch a couple more times, I do choose the latter20:41
*** moshele has quit IRC20:52
*** moshele has joined #openstack-cinder20:54
openstackgerritMarc proposed openstack/cinder master: Solving permission errors due to directory ownership on NFS  https://review.openstack.org/52472820:59
*** moshele has quit IRC20:59
*** AlexeyAbashkin has joined #openstack-cinder21:11
*** moshele has joined #openstack-cinder21:11
*** AlexeyAbashkin has quit IRC21:15
*** linkmark has joined #openstack-cinder21:18
*** rcernin has joined #openstack-cinder21:22
*** ociuhandu has joined #openstack-cinder21:34
*** moshele has quit IRC21:34
openstackgerritSean McGinnis proposed openstack/cinder master: Add cg policies and clean up old policy handling  https://review.openstack.org/52473321:36
*** ociuhandu has quit IRC21:40
*** raunak has joined #openstack-cinder21:46
*** raunak has quit IRC22:00
*** abishop has quit IRC22:11
openstackgerritMarc proposed openstack/cinder master: Solving permission errors due to directory ownership on NFS  https://review.openstack.org/52472822:43
*** felipemonteiro_ has quit IRC22:48
*** itlinux has quit IRC22:59
*** markstur has joined #openstack-cinder23:03
*** dustins has quit IRC23:07
*** itlinux has joined #openstack-cinder23:08
*** dhinesh_ has joined #openstack-cinder23:08
*** dhinesh__ has joined #openstack-cinder23:09
*** dhinesh_ has quit IRC23:09
*** dhinesh has quit IRC23:11
*** hoonetorg has quit IRC23:19
openstackgerritRikimaru Honjo proposed openstack/os-brick master: Recover node.startup values after discovering  https://review.openstack.org/46614623:23
*** itlinux has quit IRC23:28
*** mriedem has quit IRC23:29
openstackgerritMerged openstack/cinder master: nimble: handle unicode strings in volume create  https://review.openstack.org/52118523:36
*** salv-orlando has quit IRC23:37
*** salv-orlando has joined #openstack-cinder23:38
*** ganso has quit IRC23:41
*** salv-orlando has quit IRC23:42

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