Friday, 2016-04-01

*** martyt has quit IRC00:00
*** salv-orlando has joined #openstack-cinder00:00
*** haomaiwa_ has quit IRC00:01
*** haomaiwa_ has joined #openstack-cinder00:01
*** sasukeh has quit IRC00:02
openstackgerritJohn Griffith proposed openstack/cinder: Handle TFlow short comings in SF driver  https://review.openstack.org/30007800:06
*** salv-orlando has quit IRC00:06
*** garthb_ has quit IRC00:09
*** diogogmt has quit IRC00:14
*** yangyapeng has quit IRC00:14
*** mylu has joined #openstack-cinder00:14
*** mylu has quit IRC00:15
*** jaypipes has quit IRC00:21
*** mtanino has quit IRC00:21
*** sasukeh has joined #openstack-cinder00:23
*** smoriya_ has joined #openstack-cinder00:25
*** karthikp has joined #openstack-cinder00:32
*** leeantho_ has quit IRC00:34
*** dflorea has quit IRC00:38
*** mylu has joined #openstack-cinder00:39
*** mylu has quit IRC00:41
*** mylu has joined #openstack-cinder00:42
*** mylu has quit IRC00:44
*** mylu has joined #openstack-cinder00:44
*** laughterwym has joined #openstack-cinder00:46
*** laughterwym has quit IRC00:47
*** laughterwym has joined #openstack-cinder00:47
*** diogogmt has joined #openstack-cinder00:49
*** mylu has quit IRC00:50
*** mylu has joined #openstack-cinder00:52
*** markvoelker_ has quit IRC00:53
*** bjornar has quit IRC00:54
*** edmondsw has joined #openstack-cinder00:54
*** martyt has joined #openstack-cinder00:57
*** yangyapeng has joined #openstack-cinder00:59
*** cheneydc has joined #openstack-cinder01:00
*** haomaiwa_ has quit IRC01:01
*** haomaiwa_ has joined #openstack-cinder01:01
*** martyt has quit IRC01:02
*** mylu has quit IRC01:03
*** lixiaoy1 has joined #openstack-cinder01:04
*** mylu has joined #openstack-cinder01:04
*** mylu has quit IRC01:09
*** yangyapeng has quit IRC01:10
*** mylu has joined #openstack-cinder01:15
*** EinstCrazy has joined #openstack-cinder01:16
*** sasukeh has quit IRC01:16
*** EinstCra_ has joined #openstack-cinder01:19
*** mylu has quit IRC01:22
*** sasukeh has joined #openstack-cinder01:22
*** EinstCrazy has quit IRC01:22
*** sasukeh_ has joined #openstack-cinder01:24
*** mylu has joined #openstack-cinder01:24
*** knikolla has quit IRC01:25
*** sasukeh has quit IRC01:27
*** mtanino has joined #openstack-cinder01:28
*** sasukeh_ has quit IRC01:28
*** edmondsw has quit IRC01:30
*** karthikp has quit IRC01:32
*** krtaylor has joined #openstack-cinder01:35
*** apoorvad has quit IRC01:44
*** mriedem1 has joined #openstack-cinder01:45
*** mriedem has quit IRC01:47
*** baojg has joined #openstack-cinder01:48
*** mylu has quit IRC01:50
*** martyt has joined #openstack-cinder01:58
*** haomaiwa_ has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:01
*** martyt has quit IRC02:03
*** bardia_ has quit IRC02:03
*** haomaiwang has quit IRC02:15
*** mylu has joined #openstack-cinder02:19
*** akshai_ has quit IRC02:23
*** dencaval has quit IRC02:35
*** dencaval has joined #openstack-cinder02:38
*** mriedem1 has quit IRC02:38
*** diogogmt has quit IRC02:49
*** diogogmt has joined #openstack-cinder02:50
*** gcb has joined #openstack-cinder02:52
*** 32NAAPLEF has joined #openstack-cinder02:54
*** 32NAAPLEF has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** Lee1092 has joined #openstack-cinder03:07
*** esker has joined #openstack-cinder03:07
*** mylu has quit IRC03:08
*** mylu has joined #openstack-cinder03:09
*** mylu has quit IRC03:11
*** mylu has joined #openstack-cinder03:11
*** krotscheck is now known as krotscheck_vaca03:20
*** diogogmt has quit IRC03:25
gcbthingee,  can you look at https://review.openstack.org/#/c/285712/ again ?03:28
*** Poornima has joined #openstack-cinder03:36
*** links has joined #openstack-cinder03:41
*** baojg has quit IRC03:43
*** markvoelker has joined #openstack-cinder03:47
*** mylu has quit IRC03:54
*** dflorea has joined #openstack-cinder03:55
*** markvoelker_ has joined #openstack-cinder03:56
*** markvoelker has quit IRC03:56
*** gouthamr has quit IRC03:58
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-cinder04:01
*** karthikp has joined #openstack-cinder04:03
*** karthikp has quit IRC04:08
*** mylu has joined #openstack-cinder04:10
*** mylu has quit IRC04:10
*** mylu has joined #openstack-cinder04:13
*** baojg has joined #openstack-cinder04:16
*** asselin__ has joined #openstack-cinder04:17
*** asselin has quit IRC04:18
*** laughter_ has joined #openstack-cinder04:22
*** laughterwym has quit IRC04:26
*** ndipanov has quit IRC04:29
*** markvoelker_ has quit IRC04:33
*** markvoelker has joined #openstack-cinder04:33
*** shausy has joined #openstack-cinder04:37
*** markvoelker has quit IRC04:38
*** sgotliv has joined #openstack-cinder04:40
*** mylu has quit IRC04:41
*** mtreinish has quit IRC04:44
*** mylu has joined #openstack-cinder04:46
*** pradipta has joined #openstack-cinder04:49
*** pradipta has quit IRC04:50
*** dflorea has quit IRC04:51
*** sgotliv has quit IRC04:52
*** mylu has quit IRC04:57
*** rcernin has joined #openstack-cinder04:57
*** haomaiwang has quit IRC05:01
*** haomaiwa_ has joined #openstack-cinder05:01
*** stevemar_ has quit IRC05:02
*** stevemar has joined #openstack-cinder05:02
*** mylu has joined #openstack-cinder05:05
*** sgotliv has joined #openstack-cinder05:07
*** cdelatte has quit IRC05:10
*** daneyon has quit IRC05:10
*** cfriesen has joined #openstack-cinder05:21
*** baojg has quit IRC05:21
*** tpeoples is now known as tpeoples_vacatio05:23
*** cfriesen__ has quit IRC05:23
*** tpeoples_vacatio is now known as tpeoplesvacation05:24
*** markvoelker has joined #openstack-cinder05:29
*** laughter_ has quit IRC05:34
*** laughterwym has joined #openstack-cinder05:34
*** baojg has joined #openstack-cinder05:35
*** shausy has quit IRC05:35
*** shausy has joined #openstack-cinder05:36
*** markvoelker has quit IRC05:37
*** mylu has quit IRC05:43
*** ChubYann has quit IRC05:43
*** mylu has joined #openstack-cinder05:48
*** mylu has quit IRC05:48
*** sgotliv has quit IRC05:49
*** mtreinish has joined #openstack-cinder05:52
*** nkrinner has joined #openstack-cinder05:53
*** ankit_ag has joined #openstack-cinder05:53
*** laughterwym has quit IRC05:54
*** Poornima has quit IRC05:55
*** lprice1 has quit IRC05:58
*** Poornima has joined #openstack-cinder05:59
*** laughterwym has joined #openstack-cinder05:59
*** haomaiwa_ has quit IRC06:01
*** haomaiwang has joined #openstack-cinder06:01
*** laughterwym has quit IRC06:02
*** laughterwym has joined #openstack-cinder06:03
*** lpetrut has joined #openstack-cinder06:22
*** laughterwym has quit IRC06:44
*** laughterwym has joined #openstack-cinder06:48
*** mjura has joined #openstack-cinder06:49
*** anshul has joined #openstack-cinder06:57
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
*** salv-orlando has joined #openstack-cinder07:05
*** cfriesen has quit IRC07:06
*** e0ne has joined #openstack-cinder07:07
*** baojg has quit IRC07:11
*** baojg has joined #openstack-cinder07:12
*** jamielennox is now known as jamielennox|away07:17
*** markvoelker has joined #openstack-cinder07:19
*** baojg has quit IRC07:21
*** salv-orl_ has joined #openstack-cinder07:23
*** arecknag has joined #openstack-cinder07:24
*** markvoelker has quit IRC07:25
*** salv-orlando has quit IRC07:26
*** e0ne has quit IRC07:35
*** asselin__ has quit IRC07:35
*** e0ne has joined #openstack-cinder07:35
*** lpetrut has quit IRC07:37
*** e0ne has quit IRC07:40
*** e0ne has joined #openstack-cinder07:45
*** lixiaoy1 has quit IRC07:47
*** lixiaoy1 has joined #openstack-cinder07:49
*** ociuhandu has quit IRC07:51
*** e0ne has quit IRC07:54
*** tesseract has joined #openstack-cinder07:56
*** tesseract is now known as Guest9087707:56
*** e0ne has joined #openstack-cinder07:57
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** martyt has joined #openstack-cinder08:03
*** arecknag has quit IRC08:06
*** ndipanov has joined #openstack-cinder08:06
*** jistr has joined #openstack-cinder08:07
*** martyt has quit IRC08:07
*** e0ne has quit IRC08:10
*** e0ne has joined #openstack-cinder08:13
*** Poornima has quit IRC08:15
*** e0ne has quit IRC08:16
*** e0ne has joined #openstack-cinder08:20
*** e0ne has quit IRC08:26
*** Poornima has joined #openstack-cinder08:26
*** iceyao has joined #openstack-cinder08:26
*** e0ne has joined #openstack-cinder08:29
*** sgotliv has joined #openstack-cinder08:30
*** e0ne_ has joined #openstack-cinder08:31
*** e0ne has quit IRC08:32
*** e0ne_ has quit IRC08:33
*** arecknag has joined #openstack-cinder08:34
*** nikeshm has quit IRC08:36
*** Poornima has quit IRC08:37
*** laughterwym has quit IRC08:43
*** laughterwym has joined #openstack-cinder08:45
*** e0ne has joined #openstack-cinder08:48
*** amit213 has quit IRC08:49
*** laughterwym has quit IRC08:51
*** e0ne has quit IRC08:53
*** e0ne has joined #openstack-cinder08:58
*** Poornima has joined #openstack-cinder08:59
*** haomaiwang has quit IRC09:01
*** haomaiwang has joined #openstack-cinder09:01
*** p01nt3r75 has joined #openstack-cinder09:02
openstackgerritMichal Dulko proposed openstack/cinder: Cleanup DB schema after Mitaka  https://review.openstack.org/30003309:03
*** kbyrne has quit IRC09:06
*** EinstCrazy has joined #openstack-cinder09:06
*** EinstCra_ has quit IRC09:07
*** Poornima has quit IRC09:07
*** kbyrne has joined #openstack-cinder09:11
*** markvoelker has joined #openstack-cinder09:11
*** markvoelker has quit IRC09:16
*** smcginnis_ has joined #openstack-cinder09:23
smcginnis_dhellmann: We cut Cinder RC2 the other day. No need for an RC3. We should be all set.09:23
smcginnis_dhellmann: Thanks for checking!09:24
e0nesmcginnis_: we've already got RC2 3 days ago09:27
e0nesmcginnis_: http://lists.openstack.org/pipermail/openstack-dev/2016-March/090709.html09:28
smcginnis_e0ne: Yep. dhellmann was asking if we were cutting another one. I think other than a small translation update, there hasn't been anything else in to stable/mitaka.09:28
e0nesmcginnis_: ok, I misanderstood you comment above09:29
smcginnis_e0ne: No worries. :)09:29
*** Poornima has joined #openstack-cinder09:33
smcginnis_e0ne: Looks like the logging of the meeting got cut off. Anything interesting after I had to drop off?09:33
smcginnis_e0ne: Thanks for running it by the way. I had asked DuncanT to cover it, but I guess he ran late.09:34
*** e0ne has quit IRC09:36
*** sgotliv has quit IRC09:37
*** zhonghua has quit IRC09:37
*** zhonghua-lee has joined #openstack-cinder09:39
*** yhayashi has quit IRC09:39
*** lpetrut has joined #openstack-cinder09:45
*** mvk has joined #openstack-cinder09:46
*** e0ne has joined #openstack-cinder09:47
geguileodulek: ping - rolling upgrades09:48
openstackgerritMerged openstack/cinder: python34 tests don't run without python3-dev  https://review.openstack.org/29896909:49
sheelsmcginnis_: nothing interesting after you cut of09:50
sheels/of/off09:50
smcginnis_sheel: OK, good. Thanks!09:50
dulekgeguileo: Hi!09:50
sheelsmcginnis_: welcome09:50
geguileodulek: Hey, have you tested lately the rolling upgrades?09:51
geguileodulek: Because I think we have a problem09:51
geguileodulek: When we change for example the Volume version09:51
dulekgeguileo: Last time I've tested it between Liberty and RC2.09:51
geguileodulek: But we don't have Versioned Object version changes there, right?09:52
dulekgeguileo: You mean we're not implementing obj_make_compat?09:52
geguileodulek: Because I think if you change the Volume VO version number it will just break09:52
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Remove XML API  https://review.openstack.org/23166309:52
geguileodulek: Yes, we are not implementing it and default oslo behavior (as I've seen in the code) will break09:53
geguileodulek: So I think we won't be able to do rolling upgrades without a change in our base or a change in oslo09:53
sheele0ne: hi09:53
geguileodulek: Because it raises an exception when trying to make compatible09:53
geguileodulek: On the relationships09:53
e0nesheel: hi09:54
dulekgeguileo: I certainly tested a case when an object was backported - even if besides version change the backport was a no-op.09:54
sheele0ne:  its about https://review.openstack.org/#/c/294146/109:54
sheele0ne: are we planning to take this ahead09:54
sheele0ne:  OSC guys were ok with it09:54
geguileodulek: Then maybe it was something wrong in my env... r:-??09:54
sheele0ne: as discussed in yesterday's meeting with them09:54
dulekgeguileo: relationships may be a corner caseā€¦09:54
e0nesheel: it's good news09:54
sheele0ne: just need to add tests09:54
dulekgeguileo: You've hit that on your job distribution patches?09:54
sheele0ne: yep :)09:54
geguileodulek: Well, I changed the version in a Volume without adding any new fields09:55
e0nesheel: I have to read meeting notes09:55
geguileodulek: And it broke because volume_type relationship could not be made compatible09:55
sheele0ne: ok, sure09:55
e0nesheel: actually, that patch requires a bit more, than adding tests09:55
geguileodulek: No, I saw that on the clanup patches09:55
sheele0ne: just change in doc....3 lines changes09:55
geguileos/clanup/cleanup09:55
e0nesheel: did you see my ML thread in openstack-dev?09:55
sheele0ne: but for volume and all, we have to change more09:55
sheele0ne: yes i am sync09:56
* sheel http://lists.openstack.org/pipermail/openstack-dev/2016-March/090375.html09:56
sheele0ne: this one, right?09:56
geguileodulek: When I accidentally had a node in an older version09:56
e0nesheel: ok, so I'll fix backups and snapshots now09:56
e0nesheel: yes09:56
geguileodulek: And creating/deleting a volume crashed09:56
sheele0ne:  ok09:56
e0nesheel: volumes will be  next step09:56
sheele0ne: yes, seems easy fix but big inconsistency will be removed09:57
sheele0ne: but we could discuss for this patch only09:57
dulekgeguileo: Maybe you've hit a situation when it tried to "forwardport" an object?09:57
geguileodulek: But maybe there was something weird in my env...09:57
sheele0ne: so for volume and all , may be we need one small discussion further with OSC guys09:57
dulekgeguileo: Normally you should upgrade c-api first.09:57
geguileodulek: No, it was backporting to an earlier version09:57
geguileodulek: That I saw09:57
geguileodulek: Yeah, c-api was upgraded09:57
geguileodulek: And it went from api to sch ok (there was no volume_type there)09:58
geguileodulek: And when the scheduler added the volume_type with a VO and tried to pass it to c-vol09:58
dulekOkay, can you explain the setup a little? c-api in RC2 version, c-sch in RC2 and c-vol in older, right?09:58
geguileodulek: It broke because it couldn't make compatible09:58
geguileodulek: I think it was something like that09:59
*** cheneydc has quit IRC10:00
geguileodulek: iirc the problem is that oslo VO for Lists catches ObjectActionError here: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L79310:00
*** haomaiwang has quit IRC10:01
*** shausy has quit IRC10:01
dulekgeguileo: Okay, let me try to reproduce that. I haven't hit it while testing, maybe I've missed that.10:01
geguileodulek: But not on the base https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L45910:01
*** haomaiwang has joined #openstack-cinder10:01
geguileodulek: That's as far as I got since I was in the middle of my own tests...10:01
dulekgeguileo: Sure. I'll get back to you with results.10:02
geguileodulek: Thank you very much!!!!10:02
geguileodulek: I really wanted to get into it myself and solve it, but then the cleanup stuff would just keep getting pushed for later... :-(10:02
*** martyt has joined #openstack-cinder10:05
*** iceyao has quit IRC10:05
*** smcginnis_ has quit IRC10:05
*** markvoelker has joined #openstack-cinder10:06
*** e0ne has quit IRC10:08
*** martyt has quit IRC10:09
dulekgeguileo: Were you creating volumes with some specific options? I've tried on stable/mitaka c-api and c-sch and stable/liberty c-vol - that seems to be working fine.10:09
geguileodulek: And have you changed the version of Volume?10:10
dulekgeguileo: Ah, I should do so on stable/mitaka, right?10:10
geguileodulek: I think so10:11
geguileodulek: With a changed VO in API and SCH10:11
*** EinstCrazy has quit IRC10:11
dulekgeguileo: At least liberty->mitaka upgrade seems unaffected, hopefully we'll just need to get it right in Newtonā€¦10:12
*** markvoelker has quit IRC10:12
*** e0ne has joined #openstack-cinder10:12
geguileodulek: Good10:12
*** sdague has joined #openstack-cinder10:14
*** bjornar has joined #openstack-cinder10:15
dulekgeguileo: I cannot reproduce it. I've bumped the objects.volume to '1.4', added history version, so it will get backported and executed 'cinder create 1'. It passed.10:16
geguileodulek: Ok, then maybe I forgot to add them to the history and that's why it broke...10:16
geguileodulek: I don't remember all the code I had at that time, sorry for wasting your time  :-(10:17
dulekgeguileo: I've tried to do that without history version. Still success.10:17
dulekgeguileo: Oh, no, no, until we have a proper CI testing that, every such doubt is certainly not a waste of time. :)10:18
geguileodulek: Once I finish these patches I'll try to reproduce it myself and see if it was something weird at that moment...10:18
dulekgeguileo: It may be that you haven't restarted newer services once an older one was introduced.10:18
dulekgeguileo: Newer services detect version pins only at startup or on sighup.10:19
geguileodulek: But if they didn't detect the new version why would they try to make compatible with an older version?10:19
dulekgeguileo: Although the error was in c-sch, so that doesn't fit.10:19
geguileodulek: Don't worry, I'll look into it next week (hopefully)10:20
*** laughterwym has joined #openstack-cinder10:20
*** laughterwym has quit IRC10:25
*** sgotliv has joined #openstack-cinder10:28
*** ociuhandu has joined #openstack-cinder10:31
*** gcb has quit IRC10:37
*** gcb has joined #openstack-cinder10:38
*** alaski is now known as lascii10:38
*** baojg has joined #openstack-cinder10:39
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Remove XML API  https://review.openstack.org/23166310:40
*** iceyao has joined #openstack-cinder10:41
openstackgerritMerged openstack/cinder: http header value must be a string  https://review.openstack.org/28682110:44
*** e0ne has quit IRC10:47
*** smoriya_ has quit IRC10:51
*** e0ne has joined #openstack-cinder10:52
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** markvoelker has joined #openstack-cinder11:02
*** sgotliv has quit IRC11:03
*** e0ne has quit IRC11:04
*** markvoelker has quit IRC11:06
*** iceyao has quit IRC11:06
*** e0ne has joined #openstack-cinder11:08
*** haomaiwang has quit IRC11:10
*** e0ne has quit IRC11:11
*** iceyao has joined #openstack-cinder11:17
*** tpsilva has joined #openstack-cinder11:17
openstackgerritaditi sharma proposed openstack/python-cinderclient: Add missing user message  https://review.openstack.org/26954811:21
*** nikeshm has joined #openstack-cinder11:25
nikeshmhi11:26
*** e0ne has joined #openstack-cinder11:33
swamireddyHi11:35
*** dalgaaf has joined #openstack-cinder11:37
dalgaafHi, could a second core reviewer take a look at https://review.openstack.org/#/c/293050/ ? Thanks!11:37
swamireddyI have volume v1 -> created a snapshot : v1_snap -> and created a volume from this snapshot : v1_snap_v2.  Now I have deleted the v1_snap using the "cinder snapshot- delete  v1_snap".  here cinder is not througing an y error, but the snapshot still exisit11:39
swamireddyis it know bug? if yes, do we have the bug id for this @ Launchpad?11:39
openstackgerritDanny Al-Gaaf proposed openstack/cinder: HPE LeftHand: extend volume if larger than snapshot  https://review.openstack.org/29637111:41
openstackgerritDanny Al-Gaaf proposed openstack/cinder: HPE LeftHand: extend volume if larger than snapshot  https://review.openstack.org/29637111:42
*** e0ne has quit IRC11:50
openstackgerritDanny Al-Gaaf proposed openstack/cinder: NexentaEdge: extend volume if larger than snapshot  https://review.openstack.org/29638711:51
*** e0ne has joined #openstack-cinder11:54
openstackgerritSaverio Proto proposed openstack/cinder: RBD: Make snapshot_delete more robust  https://review.openstack.org/30043311:56
*** e0ne has quit IRC11:57
nikeshmhi, like i asked yesterday, when using cinder client with keystone v3, it is working fine, but when using python-cinderclient library directly, its giving error11:57
nikeshmfrom cinderclient.v2 import client11:58
nikeshmnt = client.Client('admin', 'vedams', 'admin', 'http://192.168.2.159:35357/v3')11:58
nikeshmnt.volumes.list()11:59
nikeshmcinderclient.exceptions.NotFound: The resource could not be found. (HTTP 404)11:59
openstackgerritDanny Al-Gaaf proposed openstack/cinder: Datera: extend volume if larger than snapshot  https://review.openstack.org/29644611:59
nikeshmwhen using CLI directly it is working and values are also same which i gave above12:00
nikeshmi cheked by printing them12:00
nikeshmis it some kind of bug12:00
*** mvk_ has joined #openstack-cinder12:03
*** martyt has joined #openstack-cinder12:06
*** mvk has quit IRC12:07
openstackgerritMerged openstack/cinder: Dell SC: Checking volume size in create_volume_from_snapshot  https://review.openstack.org/29604012:08
openstackgerritMerged openstack/cinder: Alter wrong comment about param filters  https://review.openstack.org/29231112:08
*** e0ne has joined #openstack-cinder12:09
*** martyt has quit IRC12:10
*** gcb has quit IRC12:11
*** haomaiwang has joined #openstack-cinder12:16
*** mvk_ has quit IRC12:17
*** laughterwym has joined #openstack-cinder12:20
openstackgerritYuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request  https://review.openstack.org/30044312:21
*** timcl has joined #openstack-cinder12:22
*** vincent_vdk has joined #openstack-cinder12:26
vincent_vdkHi, I'm trying to configure multiple instances of the cinder-api service12:26
*** Poornima has quit IRC12:26
vincent_vdkeach with their own backend and default AZ12:26
vincent_vdkis it possible to add these services in Keystone?12:27
vincent_vdki added them to keystone, and the docs seem to be limited about this topic12:28
*** cdelatte has joined #openstack-cinder12:29
swamireddyI have volume v1 -> created a snapshot : v1_snap -> and created a volume from this snapshot : v1_snap_v2.  Now I have deleted the v1_snap using the "cinder snapshot- delete  v1_snap".  here cinder is not througing an y error, but the snapshot still exisit12:30
*** timcl1 has joined #openstack-cinder12:32
*** erlon has joined #openstack-cinder12:32
*** dave-mccowan has joined #openstack-cinder12:32
*** timcl has quit IRC12:35
*** zhonghua has joined #openstack-cinder12:37
openstackgerritMichal Dulko proposed openstack/cinder: WIP: Remove support for 1.x volume RPC API  https://review.openstack.org/29770112:37
*** zhonghua-lee has quit IRC12:38
*** cbader has quit IRC12:41
*** dims_ has quit IRC12:49
*** dims has joined #openstack-cinder12:49
*** mvk_ has joined #openstack-cinder12:50
*** cbader has joined #openstack-cinder12:50
*** swamireddy has quit IRC12:51
*** rlrossit has joined #openstack-cinder12:53
*** edmondsw has joined #openstack-cinder12:53
*** e0ne has quit IRC12:57
*** akshai has joined #openstack-cinder13:00
*** mtanino has quit IRC13:01
*** haomaiwang has quit IRC13:01
*** haomaiwang has joined #openstack-cinder13:01
*** timcl1 has quit IRC13:02
*** akshai_ has joined #openstack-cinder13:02
*** eharney has joined #openstack-cinder13:03
*** swamireddy has joined #openstack-cinder13:05
*** akshai has quit IRC13:05
*** martyt has joined #openstack-cinder13:07
*** cdelatte has quit IRC13:08
*** timcl has joined #openstack-cinder13:08
*** EinstCrazy has joined #openstack-cinder13:09
*** martyt has quit IRC13:11
*** ankit_ag has quit IRC13:12
*** arecknag has quit IRC13:12
*** links has quit IRC13:12
*** e0ne has joined #openstack-cinder13:15
*** cdelatte has joined #openstack-cinder13:17
*** mjura has quit IRC13:18
*** mjura has joined #openstack-cinder13:20
*** julim_ has quit IRC13:20
*** julim has joined #openstack-cinder13:22
*** akerr has joined #openstack-cinder13:22
*** salv-orlando has joined #openstack-cinder13:23
*** dansmith is now known as bizarroman13:25
*** bizarroman is now known as bizarrodan13:25
*** salv-orl_ has quit IRC13:26
*** crose has joined #openstack-cinder13:27
*** dims_ has joined #openstack-cinder13:27
*** porrua has joined #openstack-cinder13:27
*** timcl has quit IRC13:28
*** e0ne has quit IRC13:29
*** timcl has joined #openstack-cinder13:29
*** EinstCrazy has quit IRC13:30
*** asselin__ has joined #openstack-cinder13:32
*** xyang1 has joined #openstack-cinder13:32
*** jistr has quit IRC13:32
*** iceyao has quit IRC13:32
*** openstackgerrit has quit IRC13:33
*** openstackgerrit has joined #openstack-cinder13:34
*** krtaylor has quit IRC13:35
*** salv-orlando has quit IRC13:35
*** asselin__ has quit IRC13:36
*** crose has quit IRC13:38
*** mjura has quit IRC13:39
*** diogogmt has joined #openstack-cinder13:40
*** gouthamr has joined #openstack-cinder13:40
*** timcl has quit IRC13:43
*** jgregor has joined #openstack-cinder13:45
*** diogogmt has quit IRC13:46
*** markvoelker has joined #openstack-cinder13:47
*** mriedem has joined #openstack-cinder13:47
*** baojg has quit IRC13:48
*** EinstCrazy has joined #openstack-cinder13:50
*** e0ne has joined #openstack-cinder13:51
*** markvoelker has quit IRC13:51
*** mtanino has joined #openstack-cinder13:51
*** lascii is now known as lutf-813:52
yuriy_n17eharney: Very sorry for disturbing. Please review https://review.openstack.org/#/c/300443/ whenever you have free time.13:58
*** openstack has quit IRC13:58
*** openstack has joined #openstack-cinder14:00
*** haomaiwang has quit IRC14:01
*** haomaiwang has joined #openstack-cinder14:01
*** erlon has quit IRC14:04
*** erlon has joined #openstack-cinder14:05
erlonmessage14:07
erlonmessage14:07
erlonmessage14:07
erlonmessage14:07
*** e0ne has quit IRC14:07
*** iceyao has joined #openstack-cinder14:09
*** timcl has joined #openstack-cinder14:10
*** jungleboyj has joined #openstack-cinder14:13
*** iceyao has quit IRC14:18
*** haypo has joined #openstack-cinder14:21
haypoDuncanT: hi. you changed the status of the cinder port to py3 from "done" to "in progress". why? https://wiki.openstack.org/w/index.php?title=Python3&diff=next&oldid=122983#OpenStack_applications_.28tc-approved.2914:22
haypoDuncanT: it's not written, but these tables are restricted to unit tests. last time i checked, cinder ran *all* unit tests on py3 with its voting py3 job14:23
*** lcurtis has joined #openstack-cinder14:24
*** lcurtis has quit IRC14:24
*** lcurtis has joined #openstack-cinder14:24
*** lcurtis has quit IRC14:25
*** lcurtis has joined #openstack-cinder14:25
*** ivase has joined #openstack-cinder14:27
*** mdenny has joined #openstack-cinder14:27
*** knikolla has joined #openstack-cinder14:31
*** timcl has quit IRC14:33
*** markvoelker has joined #openstack-cinder14:41
*** karthikp has joined #openstack-cinder14:41
*** ametts has joined #openstack-cinder14:42
*** karthikp has quit IRC14:43
*** karthikp has joined #openstack-cinder14:43
scottdahaypo: This was in response to a request from someone who had attempted to deploy Cinder using py3....14:44
scottdaThey had thought the table indicated that Cinder+py3 was fully supported.14:44
scottdaI guess it wasn't clear that the table indicated that only unite tests pass and the tests vote.14:45
*** jistr has joined #openstack-cinder14:46
hayposcottda: hum, i see. i should clarify the wiki page14:46
*** pots1 has joined #openstack-cinder14:46
hayposcottda: IMHO no openstack fully work on py3 yet14:46
*** markvoelker has quit IRC14:47
*** nkrinner has quit IRC14:48
scottdahaypo: Yes, that's OK.14:48
scottdahaypo: It started when someone filed a bug saying "xxx breaks python34". It took me some time to figure out if Cinder was even supposed to work with Python3414:49
*** bjornar has quit IRC14:49
*** iceyao has joined #openstack-cinder14:51
openstackgerritEric Harney proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings  https://review.openstack.org/30018914:52
*** cfriesen has joined #openstack-cinder14:53
*** mylu has joined #openstack-cinder14:53
*** garthb_ has joined #openstack-cinder14:54
*** e0ne has joined #openstack-cinder14:57
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add /v3 endpoint support for cinderclient  https://review.openstack.org/30002814:57
*** delattec has joined #openstack-cinder14:58
ameadeDuncanT: this is the impl for the messages spec https://review.openstack.org/#/c/298052/14:58
ameadelooks like some things are still being hashed out on the spec14:59
openstackgerritEric Harney proposed openstack/cinder: NFS snapshots  https://review.openstack.org/14718615:00
openstackgerritEric Harney proposed openstack/cinder: NFS snapshots  https://review.openstack.org/14718615:00
*** haomaiwang has quit IRC15:01
*** haomaiwa_ has joined #openstack-cinder15:01
*** cdelatte has quit IRC15:02
sheelameade: hey15:03
*** EinstCrazy has quit IRC15:05
*** markvoelker has joined #openstack-cinder15:05
e0neyuriy_n17: we can't merge code before spec is landed. please, addess comments in specs first15:06
*** lprice has joined #openstack-cinder15:06
ameadesheel: hey, in meeting for 30 mins but whats up?15:06
*** links has joined #openstack-cinder15:07
sheelameade: hey buddy15:07
sheelameade: I did not find this good  what is happening for patch part...15:08
sheelameade:  we may discuss once you get some free time for this...15:08
*** martyt has joined #openstack-cinder15:08
openstackgerritTom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings  https://review.openstack.org/30018915:09
jbernard_jgriffith: "It's dead Jim!", where does that come from?15:09
jbernard_jgriffith: i mean, the spec is great, it's the pop culture reference that im failing to follow :)15:10
*** sgotliv has joined #openstack-cinder15:10
*** adrianofr_ has quit IRC15:10
akerrstar trek15:11
ameadesheel: we can chat now ill just be slow to respond15:11
guitarzanjbernard_: http://knowyourmeme.com/memes/hes-dead-jimits-dead-jim15:11
sheelscottda:  hi,15:11
sheelscottda: I have answered your comment on https://review.openstack.org/#/c/273938/15:11
sheelscottda: could you see once if you find it ok  as per your time availibility?15:11
sheelscottda:  thanks15:11
jbernard_guitarzan: oh... i should know that! im clearly slipping15:12
guitarzan:)15:12
guitarzanit's some kind of nerd test :)15:12
jbernard_and i failed15:12
sheelameade: its about ownership of patch as you know15:12
sheelameade:  I am surprized how can someone override other just by adding some code on existing patch?15:12
*** martyt has quit IRC15:13
sheelameade: I am not able to justify this..15:13
eharneysheel: ameade has been working on this effort for a long time, jumping in and trying to just write the whole thing is kinda making it difficult15:14
sheeleharney:  may be this is going in wrong direction15:14
sheeleharney:  I did no jump in15:14
sheeleharney:  I raise this topic in cinder in meeting15:14
sheeleharney: i opened BP15:14
sheeleharney: then I write spec which was discussed in midcycle15:15
sheeleharney:  if you still think i jumped in, I am ok to leave this to owner you15:15
sheeleharney: please decide, I am ok to follow15:16
*** sgotliv has quit IRC15:17
*** arch-nemesis has joined #openstack-cinder15:17
ameadei think i justified it on the patch15:17
ameadejust putting the patch up initially does not dibb it or something15:17
eharneythis thing was discussed at a summit in 2014... and is still the same thing we discussed there15:18
*** Yogi1 has joined #openstack-cinder15:18
ameadeI havent been able to make it a priority until now which sucks but its my top priority now15:19
*** yuriy_n17 has quit IRC15:19
tbarronand ameade pushed this at cinder and manila mid-cycles this year15:19
sheelameade:  but why only when i started this??15:20
tbarronsheel: that's what we're saying, this baby has been in production for a long time before15:20
tbarronsheel: you weren't around and didn't see that I guess, but it's true15:21
ameadesheel: you lit a fire and made it so my company and i made it a priority15:21
sheelameade:  its priority for me too15:22
ameadenothing is really wrong atm, i want the minimum feature merged soon15:22
sheeleharney: tbarron ameade  : I am not not opposing you guys and wont even in future but i think things should be disucced15:22
sheeleharney:  tbarron  ameade  there was no discussion on this with me...15:23
sheeleharney:  tbarron  ameade  it seems i was waisting my time to think about this feature15:23
eharneyi don't think anyone is opposed to discussing feature design15:23
ameadewhat wasn't discussed?15:23
eharneyand i'm pretty sure we have been discussing this feature15:23
sheelameade:  about ownership of patch15:23
sheelameade:  we had discussino that you will be writing API on my code15:23
sheelameade:  now you are taking it as whole15:24
tbarronsheel: is this bout stackalytics, or about the direction/design of patch?15:24
*** mylu has quit IRC15:24
sheelameade:  stackalytics???15:24
sheelsorry15:24
sheeltbarron:  stackalytics?15:24
sheeltbarron:  ok you mean that statistics tool15:24
guitarzansheel: is there some kind of "credit" you're looking for?15:25
guitarzan(same question applies to ameade)15:25
sheelguitarzan:  hehehe.. no no... its about if we are doing efforts that should be worth it15:25
guitarzan"worth"?15:25
ameadewe oroginally discussed me writing the first iteration and you building on that15:26
sheelameade:  really ? no..15:26
ameadei just went ahead and impl the whole patch cuz it was all in my head15:26
sheel(please dont take anything wrong here... we are discussing amicably)15:26
sheelguitarzan:  I dont have specifically what does worth mean here... but in rough words it is like your efforts should not go in vain..15:27
guitarzanhopefully none of the rest of us care who writes the patch15:28
guitarzanI just want the feature15:28
guitarzanI'll let you two duke it out15:28
sheelguitarzan:  I am a bit new to community... so may be I am bit possesive for now....15:28
*** mylu has joined #openstack-cinder15:28
sheelguitarzan:  things used to improve with time...15:28
sheelguitarzan:  will happen with me as well after some time working with you guys15:29
ameadeguitarzan: on the surface yes, I dont want to keep working on the feature and not ever get an authorship when my company, etc, is putting efforts to core work15:29
sheelameade: guitarzan  may be I should follow your advice15:29
sheelameade:  lets devide the work once spec is passed15:30
sheelameade:  are you ok with it?15:30
eharneysheel: as i recall, much of your interest was on using this capability and integrating it with horizon etc, whereas ameade has a pretty good run already at building the base level infrastructure for it (which we really need to get into newton as early as possible, and then build on that...)15:30
sheeleharney:  my interest was to present good feature including horizon and cli15:30
sheeleharney:  ok, lets ameade  work on it15:31
ameadesheel sounds fine to me15:31
*** e0ne has quit IRC15:31
sheelameade:  ok(though i am bit said, but i will manage)15:31
ameadesheel: why sad?15:33
sheelameade:  i think we can leave it,  I am happy to work with good guys15:34
sheelameade:  lets discuss more about feature15:34
sheelameade: :)15:34
sheelameade:  I will discuss work devision with you on monday or so15:35
*** haplo37 has joined #openstack-cinder15:35
sheelameade:  les complete this with full entho...15:35
*** bardia has joined #openstack-cinder15:36
*** mylu has quit IRC15:38
*** ociuhandu has quit IRC15:39
*** daneyon has joined #openstack-cinder15:40
*** mylu has joined #openstack-cinder15:40
ameadesheel: yes definitely15:40
*** jdurgin1 has joined #openstack-cinder15:41
*** anshul has quit IRC15:42
*** bjornar has joined #openstack-cinder15:43
*** iceyao has quit IRC15:43
*** krtaylor has joined #openstack-cinder15:44
*** Guest90877 has quit IRC15:44
openstackgerritVincent Hou proposed openstack/cinder: Storwize: do the correct update after failover  https://review.openstack.org/29257015:49
openstackgerritEric Harney proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings  https://review.openstack.org/30018915:54
*** links has quit IRC15:55
openstackgerritDanny Al-Gaaf proposed openstack/os-brick: RBDClient: pass clustername to Rados  https://review.openstack.org/30056015:55
*** timcl has joined #openstack-cinder15:56
*** IlyaG has joined #openstack-cinder15:56
*** haplo37 has quit IRC15:58
*** haplo37 has joined #openstack-cinder15:59
*** haomaiwa_ has quit IRC16:01
*** haomaiwang has joined #openstack-cinder16:01
*** Yogi1 has quit IRC16:05
*** apoorvad has joined #openstack-cinder16:06
*** timcl has quit IRC16:10
*** martyt has joined #openstack-cinder16:10
*** garthb_ has quit IRC16:10
*** bjornar has quit IRC16:12
*** mriedem is now known as mriedem_lunch16:12
*** zzzeek has quit IRC16:13
openstackgerritEric Harney proposed openstack/cinder: NFS snapshots  https://review.openstack.org/14718616:13
*** zzzeek has joined #openstack-cinder16:14
*** leeantho has joined #openstack-cinder16:15
*** martyt has quit IRC16:15
*** mylu has quit IRC16:15
openstackgerritTom Barron proposed openstack/cinder: fix invalid uuid warnings from test_quota  https://review.openstack.org/29785416:20
openstackgerritTom Barron proposed openstack/cinder: Fix fake uuids in NetApp ESeries unit tests  https://review.openstack.org/29736316:20
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuids in EMC scaleio unit tests  https://review.openstack.org/29737516:20
openstackgerritTom Barron proposed openstack/cinder: Fix uuid warnings from db unit tests  https://review.openstack.org/29725516:20
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuids in sheepdog unit tests  https://review.openstack.org/29745316:20
*** e0ne has joined #openstack-cinder16:24
*** salv-orlando has joined #openstack-cinder16:25
scottdamfisch: Hi16:28
*** e0ne has quit IRC16:29
*** esker has quit IRC16:29
*** xiaohui has quit IRC16:29
*** e0ne has joined #openstack-cinder16:29
*** xiaohui has joined #openstack-cinder16:30
*** openstackgerrit has quit IRC16:31
*** guitarzan has quit IRC16:31
*** rcernin has quit IRC16:32
*** guitarzan has joined #openstack-cinder16:32
nikeshmhi16:33
*** e0ne has quit IRC16:39
*** dflorea has joined #openstack-cinder16:40
*** dflorea has quit IRC16:41
*** timcl has joined #openstack-cinder16:45
*** openstackgerrit has joined #openstack-cinder16:46
*** dflorea has joined #openstack-cinder16:48
*** edmondsw has quit IRC16:50
*** dflorea has quit IRC16:50
*** dflorea has joined #openstack-cinder16:51
*** dencaval has quit IRC16:51
openstackgerritAdriano Freires Rosso proposed openstack/cinder: WIP: Deprecating XML config file  https://review.openstack.org/29407616:55
*** e0ne has joined #openstack-cinder16:55
*** haypo has left #openstack-cinder16:56
openstackgerritEric Harney proposed openstack/cinder: NFS snapshots  https://review.openstack.org/14718616:57
*** p01nt3r75 has quit IRC16:59
*** delattec has quit IRC17:00
*** haomaiwang has quit IRC17:01
*** haomaiwang has joined #openstack-cinder17:01
openstackgerritTom Barron proposed openstack/cinder: Tests: Fix invalid UUIDs in EMC VNX tests  https://review.openstack.org/29311917:02
*** eharney has quit IRC17:02
openstackgerritTom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings  https://review.openstack.org/30018917:04
*** genunix has quit IRC17:07
*** dencaval has joined #openstack-cinder17:07
openstackgerritSheel Rana proposed openstack/cinder-specs: User facing error Summary Messages  https://review.openstack.org/27393817:10
*** martyt has joined #openstack-cinder17:10
*** ndipanov has quit IRC17:14
*** e0ne has quit IRC17:15
*** martyt has quit IRC17:15
*** lpetrut has quit IRC17:16
*** laughterwym has quit IRC17:17
*** bardia has quit IRC17:19
*** xiaohui has quit IRC17:22
ameadescottda: https://review.openstack.org/#/c/300585/17:22
*** xiaohui has joined #openstack-cinder17:22
*** eharney has joined #openstack-cinder17:22
*** mriedem_lunch is now known as mriedem17:24
ildikovscottda: mriedem: hi17:24
*** laughterwym has joined #openstack-cinder17:24
ildikovscottda: mriedem: just a quick question about the meeting17:24
ildikovscottda: mriedem: we mentioned that we might want to have it as a series17:25
mriedemildikov: ok17:25
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Updated to utilize provider_id  https://review.openstack.org/30011817:25
ildikovscottda: mriedem do we want to start it now or after the summit?17:26
mriedemildikov: i was talking pre-summit17:26
mriedemildikov: there was a lot of fog in the meeting this week for multiattach17:26
mriedemi think we need to have a much more clear picture on the plan for multiattach before the summit17:26
*** anshul has joined #openstack-cinder17:27
ildikovscottda: mriedem: ok, works for me, then I will register it as a series17:27
ildikovscottda: mriedem I will also keep the slot if you're fine with it as we had quite many people around this week17:27
mriedemildikov: works for me17:28
hemnamriedem, hey sorry I didn't attend that meeting.  I wasn't at work17:28
hemnamriedem, I'm reading the logs today from the meeting to get back up to speed on it.17:28
*** laughterwym has quit IRC17:28
mriedembrb17:29
ildikovmriedem: coolio, thanks17:29
ildikovthingee: ping17:30
*** mriedem has quit IRC17:33
*** mriedem has joined #openstack-cinder17:34
hemnaso, the patch that john put together won't work in multi-attach, which is one of the reasons I never perused that approach.17:34
hemnainitialize_connection needs the connector for it to find the correct attachment17:35
hemnaand/or the attachment_id17:35
*** delattec has joined #openstack-cinder17:35
mriedemhemna: ildikov: also, i'm told powervc has had multi-attach working since kilo and gfm has the patches (gerald mcbrearty)17:35
mriedemi never see him irc, but i'm trying to get him/them involved17:36
*** salv-orlando has quit IRC17:36
hemnaok17:36
hemnawell it may work for some specific backends that do a new target for every attach17:37
hemna3PAR does the same thing17:37
mriedemi'm assuming powervc is using storwize17:37
hemnaif the backend does that, it just worksā„¢17:37
ildikovmriedem: wow, cool, I didn't know17:38
hemnaI'd have to test live migration though17:38
*** delatte has joined #openstack-cinder17:38
hemnathat's the part that throws a wrench into everything.17:38
hemnaas the attachment switches hosts17:38
*** delattec has quit IRC17:40
*** haomaiwang has quit IRC17:40
*** haomaiwang has joined #openstack-cinder17:41
hemnascottda, ping17:42
ildikovhemna: you mean live migration with those drivers that exports new target each time?17:42
hemnajust in general17:42
hemnathe host in the attachment needs to be updated17:43
hemnare: if we are going to store instance id and host17:43
hemnawhich we currently aren't, but we should17:43
hemnathis is part of the update attachment piece that doesn't exist today17:43
ildikovwe also discussed storing connector info during the meeting briefly17:44
hemnareading through it...17:44
hemnajust getting to that point.17:44
ildikovbut the overall point is that we will need to update some info anyway17:44
*** dflorea has quit IRC17:44
hemnaI think we need to start storing the host in the attachment17:44
ildikovby the microversion support we should be in a much better situation regarding this now17:45
hemnaas well as keep it up to date17:45
*** salv-orlando has joined #openstack-cinder17:45
hemnaand store the connector for each attachment.17:45
hemnain cinder17:45
hemnathat solves a couple issues17:45
hemnalive migration, shelve, evacuate17:45
ildikovyeah, that's where we ended up as well IIRC17:45
hemnafor both single attachments and multi-attach17:45
hemnathe other glaring issues is what nova should do at detach time.17:46
hemnato call os-brick disconnect_volume or not.17:46
ildikovit would be important to see further than multiattach as sometimes single attachments can be problematic as well17:46
hemnamaybe if cinder includes a shared=True flag in the connection_info back from initialize_connection, nova would know17:47
hemnathat means changing cinder drivers to include that flag17:47
hemnathat avoids the extra round trip by nova at detach time17:47
hemnasince I think nova is already saving the connection_info in the bdm at attach time.17:47
hemnahrmm17:47
hemnabut17:47
ildikovif we could add it to the attachment info somehow that would be good as we call Cinder before detach already17:48
hemnathat doesn't say if there is any other instance still using that volume right now.17:48
hemnanova will have to see if other instances are using that same volume as well, even if the flag is True17:48
openstackgerritTom Barron proposed openstack/cinder: Fix uuid warnings from api.contrib unit tests  https://review.openstack.org/29733717:49
hemnanova would have to find out if there are other instances on the same host that still have the volume.  it can only call disconnect_volume if it's the last instance requesting detach17:49
ildikovin case of Nova we need to skip disconnect_volume only if multiple attachments are on the same host and the driver does not export new targets every time17:49
mriedemso if vol.mulitattach and conn_info.shared and len(attachments) > 1 and attachments.instance.host == <the same> - then don't disconnect the volume from the host?17:49
hemnaand there aren't other instances still using it.17:49
hemnamriedem, yah17:49
ildikovhemna: is this true for all back ends?17:50
mriedemi thought it wasn't, and that's what the conn_info.shared flag is for17:50
ildikovor in case of having separate targets than we need to call disconnect_volume every case still?17:50
hemnathe shared flag tells nova if the same target is exported by the cinder backend for the same volume.17:50
mriedemseparate targets would mean conn_info.shared=false, right?17:50
hemnamriedem, correct17:51
hemnamriedem, that's my assumption of the meaning of the flag.17:51
mriedemthe flag that exists in hemna's head today :)17:51
hemna:)17:51
*** timcl has quit IRC17:51
ildikovah cool, sorry I misread it17:51
ildikovmriedem: yeah, i got that part :)17:52
hemnacrap, am I confusing everyone ?17:52
mriedemi tihnk i'm following17:52
ildikovI got there too17:52
*** markvoelker has quit IRC17:52
ildikovI'm just slower Friday evening than usual... :)17:52
hemnaso17:53
*** timcl has joined #openstack-cinder17:53
hemnaif we store the host and the instance id in cinder17:53
hemnathat helps17:53
mriedemstore it in the vol attachment table?17:53
*** jdurgin1 has quit IRC17:53
hemnayup17:53
hemnalike the patch I had that I backed out17:53
hemnain Cinder17:53
ildikovyeah, that's what we tried to do but then we couldn't due to backward compat issues :(17:54
hemnaI think we need to microversion that guy in17:54
hemnaand then figure out how to cope with taht17:54
hemnathat17:54
hemnasince microversions are a thing now in Cinder.  they solve everything.......17:54
hemna</snark>17:54
mriedemthey really do17:55
ildikovyeah, scottda mentioned that we can microversion now17:55
mriedemi guess i'm not familiar with what the backward compat issue before was17:55
mriedemreturning something new in the vol attachment response, or adding a new parameter to the request?17:55
hemnamriedem, so currently, cinder pukes if you pass them both in17:55
scottdahemna: hey. I'm AFK. Back in a few minutes.17:55
mriedemto the rest api17:55
ildikovCinder today fails when you add host name and instance_id too to the attach call17:56
mriedemb/c it expects 1 and gets 217:56
mriedemyeah, so you microversion that17:56
mriedemto add the 2nd param17:56
*** markvoelker has joined #openstack-cinder17:56
mriedemlet's say that's v2.217:56
ildikovand hemna fixed it on the API, but in case of older Cinder deploys that does not help17:56
mriedemnova asks cinder for it's versions and if cinder.server.version < 2.2, we can't do this17:56
ildikovand send it only when we have v2.217:56
ildikovor higher17:57
mriedemyeah, that's why microversions solve everything :) the client can make an informed decision17:57
*** jistr has quit IRC17:58
mriedemin the case of older cinder deploys, nova can't do multiattach things with them17:58
mriedemand we'll raise an error17:58
ildikovyeah, my brain likes it, I just wasn't that emotionally ready for it until now :)17:58
mriedemyou'll find out you like it when you can move your API forward17:58
ildikovif we/Cinder will use the host info for other cases as well we could do this separately from multiattach maybe17:58
*** dflorea has joined #openstack-cinder17:59
ildikovof course we will enable multiattach for that Cinder version that properly supports it, I meant the host name sending effort18:00
*** haomaiwang has quit IRC18:01
hemnahttps://github.com/openstack/cinder/commit/3632173486f89d541c121be8e041a1da636edf3218:01
*** haomaiwang has joined #openstack-cinder18:01
hemnaum, I thought we had backed that out ?18:01
mriedemhemna: so that wasn't reverted?18:02
mriedemhttps://review.openstack.org/#/c/275316/18:02
mriedemi think ildikov did the revert18:02
mriedemduring the midcycle18:02
*** dflorea has quit IRC18:02
*** asselin__ has joined #openstack-cinder18:02
hemnaguess not18:02
ildikovI reverted it in Nova18:02
*** harlowja has quit IRC18:02
hemnaI'm looking at the history on that file18:03
hemnaand there is no revert18:03
hemnait's still in afaik18:03
mriedemyeah18:03
*** rlrossit has quit IRC18:03
ildikovwas it reverted in Cinder as well?18:03
mriedemildikov: do you have the change?18:03
hemnaso...there you have it.18:03
mriedemildikov: it wasn't reverted from cinder18:03
*** dflorea has joined #openstack-cinder18:04
hemnadamn18:04
mriedembut it doesn't work, right?18:04
ildikovthe Nova one: https://review.openstack.org/#/c/273097/18:04
mriedemah right18:05
mriedemyeah, jgriffith had a revert up too at the same time18:05
hemnawell I thought we had to revert it because it was an API change18:05
ildikovI haven't touched Cinder as it does not do any harm until Nova does not try to start it...18:05
mriedemhemna: you probably still do18:05
mriedemit will have to be microversioned for nova to use it18:05
mriedemthere, i added cinder to the bug and added mitaka to it https://bugs.launchpad.net/cinder/+bug/153862018:06
openstackLaunchpad bug 1538620 in Cinder "Attach with host and instance_uuid not backwards compatible" [Undecided,New]18:06
mriedemneed to revert under that18:06
hemnaok I'll rebase the change and push it up18:07
ildikovhemna: cool, tnx18:07
mriedempush it real good?18:07
hemna:)18:07
ildikovlol :)18:08
hemnaok /me waits for local tox happiness18:09
*** bardia has joined #openstack-cinder18:10
*** asselin__ has quit IRC18:10
*** diablo_rojo has joined #openstack-cinder18:15
scottdahemna: Hi18:17
*** dflorea has quit IRC18:18
*** asselin has joined #openstack-cinder18:19
*** mvk_ has quit IRC18:19
*** dflorea has joined #openstack-cinder18:20
scottdaameade: Cool on the devstack patch for Cinder v3...I'll test here in a little while.18:21
*** IlyaG has quit IRC18:22
*** garthb has joined #openstack-cinder18:22
*** dflorea has quit IRC18:23
*** rlrossit has joined #openstack-cinder18:25
*** p01nt3r75 has joined #openstack-cinder18:26
*** asselin__ has joined #openstack-cinder18:27
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Revert "Allow host and instance_uuid at attach"  https://review.openstack.org/27531618:28
*** asselin has quit IRC18:29
*** markvoelker has quit IRC18:31
jgriffithmriedem: hemna I thought I reverted that...  it didn't work anyway IIRC18:32
*** edmondsw has joined #openstack-cinder18:32
jgriffithmriedem: hemna oh well, sounds like you're on it now18:32
*** lprice has quit IRC18:32
hemnaI had thought it had gotten in :(18:32
jgriffithIt broke a bunch of things.. so I'm pretty sure we reverted it18:33
hemnauntil I looked at the git history on the file.18:33
ildikovjgriffith: this chang worked, but when we added the complementing part to Nova it broke older Cinder deploys18:33
jgriffithildikov: right you are18:33
ildikovjgriffith: I reverted the Nova change when the issue turned out18:33
ildikovjgriffith: but as the Cinder change did not do any harm by itself that was kept finally18:34
openstackgerritTom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings  https://review.openstack.org/30018918:34
jgriffithhmm...18:34
jgriffithildikov: well, I don't necessarily agree with that but regardless :)  Water under the bridge now18:34
ildikovhemna: I assume this needs to be backported as well to Mitaka18:34
* jgriffith goes back to shaving his Yak18:34
hemnaildikov, +118:35
ildikovjgriffith: well, I'm not saying it was good to leave it there, but it kinda happened, anyway hemna is fixing it now :)18:35
jgriffithildikov: yeah, I get ya.. no worries18:35
*** karthikp has quit IRC18:35
ildikovjgriffith: end of week now, so I definitely will not miss any sleep because of it :)18:37
jgriffithildikov: haha.. I hear that!!18:37
hemna:)18:37
jgriffithildikov: as I've become older I've also realized that there are few things worth missing sleep over anyway :)18:37
guitarzan<3 sleep18:38
ildikovjgriffith: yeah, I started to realise that myself as well18:38
ildikovjgriffith: just recently though ;)18:38
ildikovguitarzan: +1 :)18:38
*** dalgaaf has quit IRC18:38
scottdahemna: You want me to put up a patch that microversions "Allow host and instance_uuid at attach" ?18:39
hemnascottda, or teach me how to microversion ?18:40
scottdahemna: Either way is fine with me. Is it easier to show how to microversion by me putting up the patch?18:41
hemnaif it's easy for you, then go ahead18:41
hemnaI do want to learn/figure out how to do it18:41
scottdaok. Did you have something else to discuss earlier when you ping'ed me?18:42
scottdaWas it about a spec for new api to update connector and/or host info?18:42
scottdahemna: Cause I wanted to discuss some details of that with you, at some point.18:42
*** dflorea has joined #openstack-cinder18:43
*** dflorea has quit IRC18:43
hemnayah18:43
hemnaI think so18:43
*** ChubYann has joined #openstack-cinder18:43
*** dflorea has joined #openstack-cinder18:43
scottdaGot time to talk now? PM me if /when ready18:44
thingeeildikov: still have your cross-project thing on my list. been a bit busy with summit preparation and getting stuff ready for the release from the foundation's side. Expect an email from me today18:49
ildikovthingee: thanks, but no worries, I pinged you for another reason this time18:50
ildikovthingee: just a very quick question18:50
*** bjornar has joined #openstack-cinder18:50
ildikovthingee: is that ok if I schedule a Cinder-Nova API interactions weekly meeting on #openstack-meeting-cp channel?18:51
*** Lee1092 has quit IRC18:52
ildikovthingee: the question is mainly whether it is ok to use that channel for this purpose?18:52
*** david-lyle has quit IRC18:53
*** markvoelker has joined #openstack-cinder18:53
*** markvoelker has quit IRC18:53
*** markvoelker has joined #openstack-cinder18:54
*** markvoelker has quit IRC18:54
*** markvoelker has joined #openstack-cinder18:54
*** dflorea has quit IRC18:57
*** david-lyle has joined #openstack-cinder18:59
*** dflorea has joined #openstack-cinder19:01
*** haomaiwang has quit IRC19:01
*** haomaiwang has joined #openstack-cinder19:01
thingeeildikov: that's a good question. I think most of our scheduled stuff there are things that are not indefinite in lasting. Like the service catalog work, quotas, etc.19:02
thingeeildikov: let me double check with ttx though.19:03
ildikovthingee: I think when we manage to sort out the interaction problems we will stop with the series as well19:03
ildikovthingee: but for now it would be good to have a periodical follow up on items19:03
ildikovthingee: and as it is cross-project activity that channel is tempting, but I'm ok to find another one if it still does not fit19:04
ildikovthingee: tnx for checking!19:04
*** salv-orlando has quit IRC19:07
*** lprice has joined #openstack-cinder19:09
*** ebalduf has joined #openstack-cinder19:11
*** martyt has joined #openstack-cinder19:12
*** martyt has quit IRC19:17
*** karthikp has joined #openstack-cinder19:27
*** timcl1 has joined #openstack-cinder19:31
*** lprice1 has joined #openstack-cinder19:31
*** timcl has quit IRC19:32
*** lprice has quit IRC19:34
*** harlowja has joined #openstack-cinder19:37
*** sgotliv has joined #openstack-cinder19:39
*** dflorea has quit IRC19:39
*** dflorea has joined #openstack-cinder19:41
*** dflorea has quit IRC19:42
*** mvk_ has joined #openstack-cinder19:43
*** salv-orlando has joined #openstack-cinder19:48
*** dflorea_ has joined #openstack-cinder19:49
mfischscottda: hey, sorry I missed you earlier19:52
scottdamfisch: No problem. I updated the whiteboard in the bug.19:52
mfischok19:53
scottdamfisch: I've put this in my queue and get something up next week.19:53
mfischI kinda of rediscovered it today when an Icinga check didn't fail that should have19:53
mfischawesome thats great19:53
scottdamfisch: Yeah, sorry, I forgot all about this.19:53
mfischnp19:54
*** timcl1 has quit IRC19:54
*** garthb_ has joined #openstack-cinder19:57
*** ebalduf has quit IRC19:58
*** garthb has quit IRC20:00
*** haomaiwang has quit IRC20:01
Swansonfailture20:01
*** haomaiwang has joined #openstack-cinder20:01
jungleboyjSwanson: I sent them a note yesterday.20:01
*** pots1 has quit IRC20:02
*** pots1 has joined #openstack-cinder20:02
Swanson:)20:03
*** apoorvad has quit IRC20:04
*** dflorea_ has quit IRC20:04
*** dflorea has joined #openstack-cinder20:05
*** pots1 has quit IRC20:06
*** porrua has quit IRC20:08
*** akerr has quit IRC20:11
*** dflorea has quit IRC20:14
*** jungleboyj has quit IRC20:14
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Updated to utilize provider_id  https://review.openstack.org/30011820:17
*** rlrossit has quit IRC20:18
*** salv-orl_ has joined #openstack-cinder20:23
*** salv-orlando has quit IRC20:26
openstackgerritGerald McBrearty proposed openstack/cinder: Storwize: fix unmapped multi-attached volume failure  https://review.openstack.org/29967320:31
*** IlyaG has joined #openstack-cinder20:37
*** markvoelker has quit IRC20:37
*** apoorvad has joined #openstack-cinder20:38
karthikpI see that change made as part of https://review.openstack.org/#/c/198341/14/cinder/volume/api.py  been reverted?20:39
karthikpIt kind of seems like it because master shows old code:20:39
karthikphttps://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L25220:39
karthikpIf it wasnt reverted it would be strange thing to see!20:39
*** raildo is now known as raildo-afk20:42
*** mylu has joined #openstack-cinder20:45
*** dflorea has joined #openstack-cinder20:50
DuncanTkarthikp: Maybe re verted, maybe just redone as part of a later patch. 'git blame' should show you the details, all reverts appear in the logs, usually with a good explaination20:50
*** jgregor has quit IRC20:52
*** dflorea has quit IRC20:55
*** delatte has quit IRC20:56
*** mriedem1 has joined #openstack-cinder20:56
*** eharney has quit IRC20:58
*** eharney has joined #openstack-cinder20:59
*** mriedem has quit IRC20:59
*** haomaiwang has quit IRC21:01
*** haomaiwang has joined #openstack-cinder21:01
*** haplo37 has quit IRC21:01
*** diablo_rojo has quit IRC21:05
*** gouthamr has quit IRC21:10
karthikpDuncanT: I tried using the git blame -L252 cinder/volume/api.py and it shows an old change by John21:10
DuncanTkarthikp: Curious....21:16
*** dflorea has joined #openstack-cinder21:22
karthikpDuncanT, sorry it is clarified..was checking a different line21:25
*** dflorea has quit IRC21:28
*** mriedem1 is now known as mriedem21:29
*** akshai_ has quit IRC21:30
*** dflorea has joined #openstack-cinder21:31
*** haplo37 has joined #openstack-cinder21:31
*** sgotliv has quit IRC21:35
*** bjornar has quit IRC21:36
*** mriedem has quit IRC21:38
*** bjornar has joined #openstack-cinder21:38
*** jungleboyj has joined #openstack-cinder21:44
*** xyang1 has quit IRC21:49
*** ivase has quit IRC21:55
*** sdague has quit IRC21:56
*** jungleboyj has quit IRC21:57
*** haomaiwang has quit IRC22:01
*** markvoelker_ has joined #openstack-cinder22:01
*** haomaiwang has joined #openstack-cinder22:01
*** dflorea has quit IRC22:04
*** gouthamr has joined #openstack-cinder22:06
*** anshul has quit IRC22:09
*** dflorea has joined #openstack-cinder22:11
*** lutf-8 is now known as alaski22:12
*** dflorea has quit IRC22:13
*** edmondsw has quit IRC22:15
*** dflorea has joined #openstack-cinder22:15
*** pots1 has joined #openstack-cinder22:15
*** pots1 has quit IRC22:19
*** pots1 has joined #openstack-cinder22:19
*** lcurtis has quit IRC22:21
*** pots1 has quit IRC22:24
openstackgerritScott DAngelo proposed openstack/cinder: Allow host and instance_uuid at attach  https://review.openstack.org/30068422:25
scottdahemna: ^^^22:25
*** sheel has quit IRC22:27
*** ametts has quit IRC22:28
*** mylu has quit IRC22:32
hemnasweet22:33
*** mylu has joined #openstack-cinder22:33
*** bardia has quit IRC22:33
*** markvoelker_ has quit IRC22:39
scottdaWe can talk about the details Monday or so if you'd like.22:43
*** krtaylor has quit IRC22:44
hemnascottda, ok sounds good22:48
*** knikolla has quit IRC22:53
*** IlyaG has quit IRC22:54
*** IlyaG has joined #openstack-cinder22:54
*** IlyaG has quit IRC22:55
*** IlyaG has joined #openstack-cinder22:55
*** karthikp has quit IRC22:55
*** IlyaG has quit IRC22:55
*** krtaylor has joined #openstack-cinder22:56
*** haomaiwang has quit IRC23:01
*** haomaiwang has joined #openstack-cinder23:01
*** alyson_ has quit IRC23:02
*** mylu has quit IRC23:03
*** dflorea has quit IRC23:11
*** dflorea has joined #openstack-cinder23:12
*** mtanino has quit IRC23:13
*** dflorea has quit IRC23:17
*** iceyao has joined #openstack-cinder23:19
*** knikolla has joined #openstack-cinder23:22
*** haplo37 has quit IRC23:22
*** erlon has quit IRC23:27
*** harlowja has quit IRC23:36
*** marcusvrn_ has quit IRC23:37
*** dflorea has joined #openstack-cinder23:37
*** dflorea has quit IRC23:37
*** dflorea has joined #openstack-cinder23:38
*** dflorea has quit IRC23:40
*** esker has joined #openstack-cinder23:43
*** esker has quit IRC23:47
*** dflorea has joined #openstack-cinder23:52
*** arch-nemesis has quit IRC23:52
*** dflorea has quit IRC23:53
*** dflorea has joined #openstack-cinder23:53

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