*** martyt has quit IRC | 00:00 | |
*** salv-orlando has joined #openstack-cinder | 00:00 | |
*** haomaiwa_ has quit IRC | 00:01 | |
*** haomaiwa_ has joined #openstack-cinder | 00:01 | |
*** sasukeh has quit IRC | 00:02 | |
openstackgerrit | John Griffith proposed openstack/cinder: Handle TFlow short comings in SF driver https://review.openstack.org/300078 | 00:06 |
---|---|---|
*** salv-orlando has quit IRC | 00:06 | |
*** garthb_ has quit IRC | 00:09 | |
*** diogogmt has quit IRC | 00:14 | |
*** yangyapeng has quit IRC | 00:14 | |
*** mylu has joined #openstack-cinder | 00:14 | |
*** mylu has quit IRC | 00:15 | |
*** jaypipes has quit IRC | 00:21 | |
*** mtanino has quit IRC | 00:21 | |
*** sasukeh has joined #openstack-cinder | 00:23 | |
*** smoriya_ has joined #openstack-cinder | 00:25 | |
*** karthikp has joined #openstack-cinder | 00:32 | |
*** leeantho_ has quit IRC | 00:34 | |
*** dflorea has quit IRC | 00:38 | |
*** mylu has joined #openstack-cinder | 00:39 | |
*** mylu has quit IRC | 00:41 | |
*** mylu has joined #openstack-cinder | 00:42 | |
*** mylu has quit IRC | 00:44 | |
*** mylu has joined #openstack-cinder | 00:44 | |
*** laughterwym has joined #openstack-cinder | 00:46 | |
*** laughterwym has quit IRC | 00:47 | |
*** laughterwym has joined #openstack-cinder | 00:47 | |
*** diogogmt has joined #openstack-cinder | 00:49 | |
*** mylu has quit IRC | 00:50 | |
*** mylu has joined #openstack-cinder | 00:52 | |
*** markvoelker_ has quit IRC | 00:53 | |
*** bjornar has quit IRC | 00:54 | |
*** edmondsw has joined #openstack-cinder | 00:54 | |
*** martyt has joined #openstack-cinder | 00:57 | |
*** yangyapeng has joined #openstack-cinder | 00:59 | |
*** cheneydc has joined #openstack-cinder | 01:00 | |
*** haomaiwa_ has quit IRC | 01:01 | |
*** haomaiwa_ has joined #openstack-cinder | 01:01 | |
*** martyt has quit IRC | 01:02 | |
*** mylu has quit IRC | 01:03 | |
*** lixiaoy1 has joined #openstack-cinder | 01:04 | |
*** mylu has joined #openstack-cinder | 01:04 | |
*** mylu has quit IRC | 01:09 | |
*** yangyapeng has quit IRC | 01:10 | |
*** mylu has joined #openstack-cinder | 01:15 | |
*** EinstCrazy has joined #openstack-cinder | 01:16 | |
*** sasukeh has quit IRC | 01:16 | |
*** EinstCra_ has joined #openstack-cinder | 01:19 | |
*** mylu has quit IRC | 01:22 | |
*** sasukeh has joined #openstack-cinder | 01:22 | |
*** EinstCrazy has quit IRC | 01:22 | |
*** sasukeh_ has joined #openstack-cinder | 01:24 | |
*** mylu has joined #openstack-cinder | 01:24 | |
*** knikolla has quit IRC | 01:25 | |
*** sasukeh has quit IRC | 01:27 | |
*** mtanino has joined #openstack-cinder | 01:28 | |
*** sasukeh_ has quit IRC | 01:28 | |
*** edmondsw has quit IRC | 01:30 | |
*** karthikp has quit IRC | 01:32 | |
*** krtaylor has joined #openstack-cinder | 01:35 | |
*** apoorvad has quit IRC | 01:44 | |
*** mriedem1 has joined #openstack-cinder | 01:45 | |
*** mriedem has quit IRC | 01:47 | |
*** baojg has joined #openstack-cinder | 01:48 | |
*** mylu has quit IRC | 01:50 | |
*** martyt has joined #openstack-cinder | 01:58 | |
*** haomaiwa_ has quit IRC | 02:01 | |
*** haomaiwang has joined #openstack-cinder | 02:01 | |
*** martyt has quit IRC | 02:03 | |
*** bardia_ has quit IRC | 02:03 | |
*** haomaiwang has quit IRC | 02:15 | |
*** mylu has joined #openstack-cinder | 02:19 | |
*** akshai_ has quit IRC | 02:23 | |
*** dencaval has quit IRC | 02:35 | |
*** dencaval has joined #openstack-cinder | 02:38 | |
*** mriedem1 has quit IRC | 02:38 | |
*** diogogmt has quit IRC | 02:49 | |
*** diogogmt has joined #openstack-cinder | 02:50 | |
*** gcb has joined #openstack-cinder | 02:52 | |
*** 32NAAPLEF has joined #openstack-cinder | 02:54 | |
*** 32NAAPLEF has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** Lee1092 has joined #openstack-cinder | 03:07 | |
*** esker has joined #openstack-cinder | 03:07 | |
*** mylu has quit IRC | 03:08 | |
*** mylu has joined #openstack-cinder | 03:09 | |
*** mylu has quit IRC | 03:11 | |
*** mylu has joined #openstack-cinder | 03:11 | |
*** krotscheck is now known as krotscheck_vaca | 03:20 | |
*** diogogmt has quit IRC | 03:25 | |
gcb | thingee, can you look at https://review.openstack.org/#/c/285712/ again ? | 03:28 |
*** Poornima has joined #openstack-cinder | 03:36 | |
*** links has joined #openstack-cinder | 03:41 | |
*** baojg has quit IRC | 03:43 | |
*** markvoelker has joined #openstack-cinder | 03:47 | |
*** mylu has quit IRC | 03:54 | |
*** dflorea has joined #openstack-cinder | 03:55 | |
*** markvoelker_ has joined #openstack-cinder | 03:56 | |
*** markvoelker has quit IRC | 03:56 | |
*** gouthamr has quit IRC | 03:58 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-cinder | 04:01 | |
*** karthikp has joined #openstack-cinder | 04:03 | |
*** karthikp has quit IRC | 04:08 | |
*** mylu has joined #openstack-cinder | 04:10 | |
*** mylu has quit IRC | 04:10 | |
*** mylu has joined #openstack-cinder | 04:13 | |
*** baojg has joined #openstack-cinder | 04:16 | |
*** asselin__ has joined #openstack-cinder | 04:17 | |
*** asselin has quit IRC | 04:18 | |
*** laughter_ has joined #openstack-cinder | 04:22 | |
*** laughterwym has quit IRC | 04:26 | |
*** ndipanov has quit IRC | 04:29 | |
*** markvoelker_ has quit IRC | 04:33 | |
*** markvoelker has joined #openstack-cinder | 04:33 | |
*** shausy has joined #openstack-cinder | 04:37 | |
*** markvoelker has quit IRC | 04:38 | |
*** sgotliv has joined #openstack-cinder | 04:40 | |
*** mylu has quit IRC | 04:41 | |
*** mtreinish has quit IRC | 04:44 | |
*** mylu has joined #openstack-cinder | 04:46 | |
*** pradipta has joined #openstack-cinder | 04:49 | |
*** pradipta has quit IRC | 04:50 | |
*** dflorea has quit IRC | 04:51 | |
*** sgotliv has quit IRC | 04:52 | |
*** mylu has quit IRC | 04:57 | |
*** rcernin has joined #openstack-cinder | 04:57 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwa_ has joined #openstack-cinder | 05:01 | |
*** stevemar_ has quit IRC | 05:02 | |
*** stevemar has joined #openstack-cinder | 05:02 | |
*** mylu has joined #openstack-cinder | 05:05 | |
*** sgotliv has joined #openstack-cinder | 05:07 | |
*** cdelatte has quit IRC | 05:10 | |
*** daneyon has quit IRC | 05:10 | |
*** cfriesen has joined #openstack-cinder | 05:21 | |
*** baojg has quit IRC | 05:21 | |
*** tpeoples is now known as tpeoples_vacatio | 05:23 | |
*** cfriesen__ has quit IRC | 05:23 | |
*** tpeoples_vacatio is now known as tpeoplesvacation | 05:24 | |
*** markvoelker has joined #openstack-cinder | 05:29 | |
*** laughter_ has quit IRC | 05:34 | |
*** laughterwym has joined #openstack-cinder | 05:34 | |
*** baojg has joined #openstack-cinder | 05:35 | |
*** shausy has quit IRC | 05:35 | |
*** shausy has joined #openstack-cinder | 05:36 | |
*** markvoelker has quit IRC | 05:37 | |
*** mylu has quit IRC | 05:43 | |
*** ChubYann has quit IRC | 05:43 | |
*** mylu has joined #openstack-cinder | 05:48 | |
*** mylu has quit IRC | 05:48 | |
*** sgotliv has quit IRC | 05:49 | |
*** mtreinish has joined #openstack-cinder | 05:52 | |
*** nkrinner has joined #openstack-cinder | 05:53 | |
*** ankit_ag has joined #openstack-cinder | 05:53 | |
*** laughterwym has quit IRC | 05:54 | |
*** Poornima has quit IRC | 05:55 | |
*** lprice1 has quit IRC | 05:58 | |
*** Poornima has joined #openstack-cinder | 05:59 | |
*** laughterwym has joined #openstack-cinder | 05:59 | |
*** haomaiwa_ has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** laughterwym has quit IRC | 06:02 | |
*** laughterwym has joined #openstack-cinder | 06:03 | |
*** lpetrut has joined #openstack-cinder | 06:22 | |
*** laughterwym has quit IRC | 06:44 | |
*** laughterwym has joined #openstack-cinder | 06:48 | |
*** mjura has joined #openstack-cinder | 06:49 | |
*** anshul has joined #openstack-cinder | 06:57 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-cinder | 07:01 | |
*** salv-orlando has joined #openstack-cinder | 07:05 | |
*** cfriesen has quit IRC | 07:06 | |
*** e0ne has joined #openstack-cinder | 07:07 | |
*** baojg has quit IRC | 07:11 | |
*** baojg has joined #openstack-cinder | 07:12 | |
*** jamielennox is now known as jamielennox|away | 07:17 | |
*** markvoelker has joined #openstack-cinder | 07:19 | |
*** baojg has quit IRC | 07:21 | |
*** salv-orl_ has joined #openstack-cinder | 07:23 | |
*** arecknag has joined #openstack-cinder | 07:24 | |
*** markvoelker has quit IRC | 07:25 | |
*** salv-orlando has quit IRC | 07:26 | |
*** e0ne has quit IRC | 07:35 | |
*** asselin__ has quit IRC | 07:35 | |
*** e0ne has joined #openstack-cinder | 07:35 | |
*** lpetrut has quit IRC | 07:37 | |
*** e0ne has quit IRC | 07:40 | |
*** e0ne has joined #openstack-cinder | 07:45 | |
*** lixiaoy1 has quit IRC | 07:47 | |
*** lixiaoy1 has joined #openstack-cinder | 07:49 | |
*** ociuhandu has quit IRC | 07:51 | |
*** e0ne has quit IRC | 07:54 | |
*** tesseract has joined #openstack-cinder | 07:56 | |
*** tesseract is now known as Guest90877 | 07:56 | |
*** e0ne has joined #openstack-cinder | 07:57 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-cinder | 08:01 | |
*** martyt has joined #openstack-cinder | 08:03 | |
*** arecknag has quit IRC | 08:06 | |
*** ndipanov has joined #openstack-cinder | 08:06 | |
*** jistr has joined #openstack-cinder | 08:07 | |
*** martyt has quit IRC | 08:07 | |
*** e0ne has quit IRC | 08:10 | |
*** e0ne has joined #openstack-cinder | 08:13 | |
*** Poornima has quit IRC | 08:15 | |
*** e0ne has quit IRC | 08:16 | |
*** e0ne has joined #openstack-cinder | 08:20 | |
*** e0ne has quit IRC | 08:26 | |
*** Poornima has joined #openstack-cinder | 08:26 | |
*** iceyao has joined #openstack-cinder | 08:26 | |
*** e0ne has joined #openstack-cinder | 08:29 | |
*** sgotliv has joined #openstack-cinder | 08:30 | |
*** e0ne_ has joined #openstack-cinder | 08:31 | |
*** e0ne has quit IRC | 08:32 | |
*** e0ne_ has quit IRC | 08:33 | |
*** arecknag has joined #openstack-cinder | 08:34 | |
*** nikeshm has quit IRC | 08:36 | |
*** Poornima has quit IRC | 08:37 | |
*** laughterwym has quit IRC | 08:43 | |
*** laughterwym has joined #openstack-cinder | 08:45 | |
*** e0ne has joined #openstack-cinder | 08:48 | |
*** amit213 has quit IRC | 08:49 | |
*** laughterwym has quit IRC | 08:51 | |
*** e0ne has quit IRC | 08:53 | |
*** e0ne has joined #openstack-cinder | 08:58 | |
*** Poornima has joined #openstack-cinder | 08:59 | |
*** haomaiwang has quit IRC | 09:01 | |
*** haomaiwang has joined #openstack-cinder | 09:01 | |
*** p01nt3r75 has joined #openstack-cinder | 09:02 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: Cleanup DB schema after Mitaka https://review.openstack.org/300033 | 09:03 |
*** kbyrne has quit IRC | 09:06 | |
*** EinstCrazy has joined #openstack-cinder | 09:06 | |
*** EinstCra_ has quit IRC | 09:07 | |
*** Poornima has quit IRC | 09:07 | |
*** kbyrne has joined #openstack-cinder | 09:11 | |
*** markvoelker has joined #openstack-cinder | 09:11 | |
*** markvoelker has quit IRC | 09:16 | |
*** smcginnis_ has joined #openstack-cinder | 09: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 |
e0ne | smcginnis_: we've already got RC2 3 days ago | 09:27 |
e0ne | smcginnis_: http://lists.openstack.org/pipermail/openstack-dev/2016-March/090709.html | 09: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 |
e0ne | smcginnis_: ok, I misanderstood you comment above | 09:29 |
smcginnis_ | e0ne: No worries. :) | 09:29 |
*** Poornima has joined #openstack-cinder | 09: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 IRC | 09:36 | |
*** sgotliv has quit IRC | 09:37 | |
*** zhonghua has quit IRC | 09:37 | |
*** zhonghua-lee has joined #openstack-cinder | 09:39 | |
*** yhayashi has quit IRC | 09:39 | |
*** lpetrut has joined #openstack-cinder | 09:45 | |
*** mvk has joined #openstack-cinder | 09:46 | |
*** e0ne has joined #openstack-cinder | 09:47 | |
geguileo | dulek: ping - rolling upgrades | 09:48 |
openstackgerrit | Merged openstack/cinder: python34 tests don't run without python3-dev https://review.openstack.org/298969 | 09:49 |
sheel | smcginnis_: nothing interesting after you cut of | 09:50 |
sheel | s/of/off | 09:50 |
smcginnis_ | sheel: OK, good. Thanks! | 09:50 |
dulek | geguileo: Hi! | 09:50 |
sheel | smcginnis_: welcome | 09:50 |
geguileo | dulek: Hey, have you tested lately the rolling upgrades? | 09:51 |
geguileo | dulek: Because I think we have a problem | 09:51 |
geguileo | dulek: When we change for example the Volume version | 09:51 |
dulek | geguileo: Last time I've tested it between Liberty and RC2. | 09:51 |
geguileo | dulek: But we don't have Versioned Object version changes there, right? | 09:52 |
dulek | geguileo: You mean we're not implementing obj_make_compat? | 09:52 |
geguileo | dulek: Because I think if you change the Volume VO version number it will just break | 09:52 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove XML API https://review.openstack.org/231663 | 09:52 |
geguileo | dulek: Yes, we are not implementing it and default oslo behavior (as I've seen in the code) will break | 09:53 |
geguileo | dulek: So I think we won't be able to do rolling upgrades without a change in our base or a change in oslo | 09:53 |
sheel | e0ne: hi | 09:53 |
geguileo | dulek: Because it raises an exception when trying to make compatible | 09:53 |
geguileo | dulek: On the relationships | 09:53 |
e0ne | sheel: hi | 09:54 |
dulek | geguileo: I certainly tested a case when an object was backported - even if besides version change the backport was a no-op. | 09:54 |
sheel | e0ne: its about https://review.openstack.org/#/c/294146/1 | 09:54 |
sheel | e0ne: are we planning to take this ahead | 09:54 |
sheel | e0ne: OSC guys were ok with it | 09:54 |
geguileo | dulek: Then maybe it was something wrong in my env... r:-?? | 09:54 |
sheel | e0ne: as discussed in yesterday's meeting with them | 09:54 |
dulek | geguileo: relationships may be a corner caseā¦ | 09:54 |
e0ne | sheel: it's good news | 09:54 |
sheel | e0ne: just need to add tests | 09:54 |
dulek | geguileo: You've hit that on your job distribution patches? | 09:54 |
sheel | e0ne: yep :) | 09:54 |
geguileo | dulek: Well, I changed the version in a Volume without adding any new fields | 09:55 |
e0ne | sheel: I have to read meeting notes | 09:55 |
geguileo | dulek: And it broke because volume_type relationship could not be made compatible | 09:55 |
sheel | e0ne: ok, sure | 09:55 |
e0ne | sheel: actually, that patch requires a bit more, than adding tests | 09:55 |
geguileo | dulek: No, I saw that on the clanup patches | 09:55 |
sheel | e0ne: just change in doc....3 lines changes | 09:55 |
geguileo | s/clanup/cleanup | 09:55 |
e0ne | sheel: did you see my ML thread in openstack-dev? | 09:55 |
sheel | e0ne: but for volume and all, we have to change more | 09:55 |
sheel | e0ne: yes i am sync | 09:56 |
* sheel http://lists.openstack.org/pipermail/openstack-dev/2016-March/090375.html | 09:56 | |
sheel | e0ne: this one, right? | 09:56 |
geguileo | dulek: When I accidentally had a node in an older version | 09:56 |
e0ne | sheel: ok, so I'll fix backups and snapshots now | 09:56 |
e0ne | sheel: yes | 09:56 |
geguileo | dulek: And creating/deleting a volume crashed | 09:56 |
sheel | e0ne: ok | 09:56 |
e0ne | sheel: volumes will be next step | 09:56 |
sheel | e0ne: yes, seems easy fix but big inconsistency will be removed | 09:57 |
sheel | e0ne: but we could discuss for this patch only | 09:57 |
dulek | geguileo: Maybe you've hit a situation when it tried to "forwardport" an object? | 09:57 |
geguileo | dulek: But maybe there was something weird in my env... | 09:57 |
sheel | e0ne: so for volume and all , may be we need one small discussion further with OSC guys | 09:57 |
dulek | geguileo: Normally you should upgrade c-api first. | 09:57 |
geguileo | dulek: No, it was backporting to an earlier version | 09:57 |
geguileo | dulek: That I saw | 09:57 |
geguileo | dulek: Yeah, c-api was upgraded | 09:57 |
geguileo | dulek: And it went from api to sch ok (there was no volume_type there) | 09:58 |
geguileo | dulek: And when the scheduler added the volume_type with a VO and tried to pass it to c-vol | 09:58 |
dulek | Okay, can you explain the setup a little? c-api in RC2 version, c-sch in RC2 and c-vol in older, right? | 09:58 |
geguileo | dulek: It broke because it couldn't make compatible | 09:58 |
geguileo | dulek: I think it was something like that | 09:59 |
*** cheneydc has quit IRC | 10:00 | |
geguileo | dulek: iirc the problem is that oslo VO for Lists catches ObjectActionError here: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L793 | 10:00 |
*** haomaiwang has quit IRC | 10:01 | |
*** shausy has quit IRC | 10:01 | |
dulek | geguileo: Okay, let me try to reproduce that. I haven't hit it while testing, maybe I've missed that. | 10:01 |
geguileo | dulek: But not on the base https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L459 | 10:01 |
*** haomaiwang has joined #openstack-cinder | 10:01 | |
geguileo | dulek: That's as far as I got since I was in the middle of my own tests... | 10:01 |
dulek | geguileo: Sure. I'll get back to you with results. | 10:02 |
geguileo | dulek: Thank you very much!!!! | 10:02 |
geguileo | dulek: 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-cinder | 10:05 | |
*** iceyao has quit IRC | 10:05 | |
*** smcginnis_ has quit IRC | 10:05 | |
*** markvoelker has joined #openstack-cinder | 10:06 | |
*** e0ne has quit IRC | 10:08 | |
*** martyt has quit IRC | 10:09 | |
dulek | geguileo: 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 |
geguileo | dulek: And have you changed the version of Volume? | 10:10 |
dulek | geguileo: Ah, I should do so on stable/mitaka, right? | 10:10 |
geguileo | dulek: I think so | 10:11 |
geguileo | dulek: With a changed VO in API and SCH | 10:11 |
*** EinstCrazy has quit IRC | 10:11 | |
dulek | geguileo: At least liberty->mitaka upgrade seems unaffected, hopefully we'll just need to get it right in Newtonā¦ | 10:12 |
*** markvoelker has quit IRC | 10:12 | |
*** e0ne has joined #openstack-cinder | 10:12 | |
geguileo | dulek: Good | 10:12 |
*** sdague has joined #openstack-cinder | 10:14 | |
*** bjornar has joined #openstack-cinder | 10:15 | |
dulek | geguileo: 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 |
geguileo | dulek: Ok, then maybe I forgot to add them to the history and that's why it broke... | 10:16 |
geguileo | dulek: I don't remember all the code I had at that time, sorry for wasting your time :-( | 10:17 |
dulek | geguileo: I've tried to do that without history version. Still success. | 10:17 |
dulek | geguileo: Oh, no, no, until we have a proper CI testing that, every such doubt is certainly not a waste of time. :) | 10:18 |
geguileo | dulek: Once I finish these patches I'll try to reproduce it myself and see if it was something weird at that moment... | 10:18 |
dulek | geguileo: It may be that you haven't restarted newer services once an older one was introduced. | 10:18 |
dulek | geguileo: Newer services detect version pins only at startup or on sighup. | 10:19 |
geguileo | dulek: But if they didn't detect the new version why would they try to make compatible with an older version? | 10:19 |
dulek | geguileo: Although the error was in c-sch, so that doesn't fit. | 10:19 |
geguileo | dulek: Don't worry, I'll look into it next week (hopefully) | 10:20 |
*** laughterwym has joined #openstack-cinder | 10:20 | |
*** laughterwym has quit IRC | 10:25 | |
*** sgotliv has joined #openstack-cinder | 10:28 | |
*** ociuhandu has joined #openstack-cinder | 10:31 | |
*** gcb has quit IRC | 10:37 | |
*** gcb has joined #openstack-cinder | 10:38 | |
*** alaski is now known as lascii | 10:38 | |
*** baojg has joined #openstack-cinder | 10:39 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove XML API https://review.openstack.org/231663 | 10:40 |
*** iceyao has joined #openstack-cinder | 10:41 | |
openstackgerrit | Merged openstack/cinder: http header value must be a string https://review.openstack.org/286821 | 10:44 |
*** e0ne has quit IRC | 10:47 | |
*** smoriya_ has quit IRC | 10:51 | |
*** e0ne has joined #openstack-cinder | 10:52 | |
*** haomaiwang has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-cinder | 11:01 | |
*** markvoelker has joined #openstack-cinder | 11:02 | |
*** sgotliv has quit IRC | 11:03 | |
*** e0ne has quit IRC | 11:04 | |
*** markvoelker has quit IRC | 11:06 | |
*** iceyao has quit IRC | 11:06 | |
*** e0ne has joined #openstack-cinder | 11:08 | |
*** haomaiwang has quit IRC | 11:10 | |
*** e0ne has quit IRC | 11:11 | |
*** iceyao has joined #openstack-cinder | 11:17 | |
*** tpsilva has joined #openstack-cinder | 11:17 | |
openstackgerrit | aditi sharma proposed openstack/python-cinderclient: Add missing user message https://review.openstack.org/269548 | 11:21 |
*** nikeshm has joined #openstack-cinder | 11:25 | |
nikeshm | hi | 11:26 |
*** e0ne has joined #openstack-cinder | 11:33 | |
swamireddy | Hi | 11:35 |
*** dalgaaf has joined #openstack-cinder | 11:37 | |
dalgaaf | Hi, could a second core reviewer take a look at https://review.openstack.org/#/c/293050/ ? Thanks! | 11:37 |
swamireddy | I 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 exisit | 11:39 |
swamireddy | is it know bug? if yes, do we have the bug id for this @ Launchpad? | 11:39 |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: HPE LeftHand: extend volume if larger than snapshot https://review.openstack.org/296371 | 11:41 |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: HPE LeftHand: extend volume if larger than snapshot https://review.openstack.org/296371 | 11:42 |
*** e0ne has quit IRC | 11:50 | |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: NexentaEdge: extend volume if larger than snapshot https://review.openstack.org/296387 | 11:51 |
*** e0ne has joined #openstack-cinder | 11:54 | |
openstackgerrit | Saverio Proto proposed openstack/cinder: RBD: Make snapshot_delete more robust https://review.openstack.org/300433 | 11:56 |
*** e0ne has quit IRC | 11:57 | |
nikeshm | hi, like i asked yesterday, when using cinder client with keystone v3, it is working fine, but when using python-cinderclient library directly, its giving error | 11:57 |
nikeshm | from cinderclient.v2 import client | 11:58 |
nikeshm | nt = client.Client('admin', 'vedams', 'admin', 'http://192.168.2.159:35357/v3') | 11:58 |
nikeshm | nt.volumes.list() | 11:59 |
nikeshm | cinderclient.exceptions.NotFound: The resource could not be found. (HTTP 404) | 11:59 |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: Datera: extend volume if larger than snapshot https://review.openstack.org/296446 | 11:59 |
nikeshm | when using CLI directly it is working and values are also same which i gave above | 12:00 |
nikeshm | i cheked by printing them | 12:00 |
nikeshm | is it some kind of bug | 12:00 |
*** mvk_ has joined #openstack-cinder | 12:03 | |
*** martyt has joined #openstack-cinder | 12:06 | |
*** mvk has quit IRC | 12:07 | |
openstackgerrit | Merged openstack/cinder: Dell SC: Checking volume size in create_volume_from_snapshot https://review.openstack.org/296040 | 12:08 |
openstackgerrit | Merged openstack/cinder: Alter wrong comment about param filters https://review.openstack.org/292311 | 12:08 |
*** e0ne has joined #openstack-cinder | 12:09 | |
*** martyt has quit IRC | 12:10 | |
*** gcb has quit IRC | 12:11 | |
*** haomaiwang has joined #openstack-cinder | 12:16 | |
*** mvk_ has quit IRC | 12:17 | |
*** laughterwym has joined #openstack-cinder | 12:20 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 12:21 |
*** timcl has joined #openstack-cinder | 12:22 | |
*** vincent_vdk has joined #openstack-cinder | 12:26 | |
vincent_vdk | Hi, I'm trying to configure multiple instances of the cinder-api service | 12:26 |
*** Poornima has quit IRC | 12:26 | |
vincent_vdk | each with their own backend and default AZ | 12:26 |
vincent_vdk | is it possible to add these services in Keystone? | 12:27 |
vincent_vdk | i added them to keystone, and the docs seem to be limited about this topic | 12:28 |
*** cdelatte has joined #openstack-cinder | 12:29 | |
swamireddy | I 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 exisit | 12:30 |
*** timcl1 has joined #openstack-cinder | 12:32 | |
*** erlon has joined #openstack-cinder | 12:32 | |
*** dave-mccowan has joined #openstack-cinder | 12:32 | |
*** timcl has quit IRC | 12:35 | |
*** zhonghua has joined #openstack-cinder | 12:37 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: WIP: Remove support for 1.x volume RPC API https://review.openstack.org/297701 | 12:37 |
*** zhonghua-lee has quit IRC | 12:38 | |
*** cbader has quit IRC | 12:41 | |
*** dims_ has quit IRC | 12:49 | |
*** dims has joined #openstack-cinder | 12:49 | |
*** mvk_ has joined #openstack-cinder | 12:50 | |
*** cbader has joined #openstack-cinder | 12:50 | |
*** swamireddy has quit IRC | 12:51 | |
*** rlrossit has joined #openstack-cinder | 12:53 | |
*** edmondsw has joined #openstack-cinder | 12:53 | |
*** e0ne has quit IRC | 12:57 | |
*** akshai has joined #openstack-cinder | 13:00 | |
*** mtanino has quit IRC | 13:01 | |
*** haomaiwang has quit IRC | 13:01 | |
*** haomaiwang has joined #openstack-cinder | 13:01 | |
*** timcl1 has quit IRC | 13:02 | |
*** akshai_ has joined #openstack-cinder | 13:02 | |
*** eharney has joined #openstack-cinder | 13:03 | |
*** swamireddy has joined #openstack-cinder | 13:05 | |
*** akshai has quit IRC | 13:05 | |
*** martyt has joined #openstack-cinder | 13:07 | |
*** cdelatte has quit IRC | 13:08 | |
*** timcl has joined #openstack-cinder | 13:08 | |
*** EinstCrazy has joined #openstack-cinder | 13:09 | |
*** martyt has quit IRC | 13:11 | |
*** ankit_ag has quit IRC | 13:12 | |
*** arecknag has quit IRC | 13:12 | |
*** links has quit IRC | 13:12 | |
*** e0ne has joined #openstack-cinder | 13:15 | |
*** cdelatte has joined #openstack-cinder | 13:17 | |
*** mjura has quit IRC | 13:18 | |
*** mjura has joined #openstack-cinder | 13:20 | |
*** julim_ has quit IRC | 13:20 | |
*** julim has joined #openstack-cinder | 13:22 | |
*** akerr has joined #openstack-cinder | 13:22 | |
*** salv-orlando has joined #openstack-cinder | 13:23 | |
*** dansmith is now known as bizarroman | 13:25 | |
*** bizarroman is now known as bizarrodan | 13:25 | |
*** salv-orl_ has quit IRC | 13:26 | |
*** crose has joined #openstack-cinder | 13:27 | |
*** dims_ has joined #openstack-cinder | 13:27 | |
*** porrua has joined #openstack-cinder | 13:27 | |
*** timcl has quit IRC | 13:28 | |
*** e0ne has quit IRC | 13:29 | |
*** timcl has joined #openstack-cinder | 13:29 | |
*** EinstCrazy has quit IRC | 13:30 | |
*** asselin__ has joined #openstack-cinder | 13:32 | |
*** xyang1 has joined #openstack-cinder | 13:32 | |
*** jistr has quit IRC | 13:32 | |
*** iceyao has quit IRC | 13:32 | |
*** openstackgerrit has quit IRC | 13:33 | |
*** openstackgerrit has joined #openstack-cinder | 13:34 | |
*** krtaylor has quit IRC | 13:35 | |
*** salv-orlando has quit IRC | 13:35 | |
*** asselin__ has quit IRC | 13:36 | |
*** crose has quit IRC | 13:38 | |
*** mjura has quit IRC | 13:39 | |
*** diogogmt has joined #openstack-cinder | 13:40 | |
*** gouthamr has joined #openstack-cinder | 13:40 | |
*** timcl has quit IRC | 13:43 | |
*** jgregor has joined #openstack-cinder | 13:45 | |
*** diogogmt has quit IRC | 13:46 | |
*** markvoelker has joined #openstack-cinder | 13:47 | |
*** mriedem has joined #openstack-cinder | 13:47 | |
*** baojg has quit IRC | 13:48 | |
*** EinstCrazy has joined #openstack-cinder | 13:50 | |
*** e0ne has joined #openstack-cinder | 13:51 | |
*** markvoelker has quit IRC | 13:51 | |
*** mtanino has joined #openstack-cinder | 13:51 | |
*** lascii is now known as lutf-8 | 13:52 | |
yuriy_n17 | eharney: Very sorry for disturbing. Please review https://review.openstack.org/#/c/300443/ whenever you have free time. | 13:58 |
*** openstack has quit IRC | 13:58 | |
*** openstack has joined #openstack-cinder | 14:00 | |
*** haomaiwang has quit IRC | 14:01 | |
*** haomaiwang has joined #openstack-cinder | 14:01 | |
*** erlon has quit IRC | 14:04 | |
*** erlon has joined #openstack-cinder | 14:05 | |
erlon | message | 14:07 |
erlon | message | 14:07 |
erlon | message | 14:07 |
erlon | message | 14:07 |
*** e0ne has quit IRC | 14:07 | |
*** iceyao has joined #openstack-cinder | 14:09 | |
*** timcl has joined #openstack-cinder | 14:10 | |
*** jungleboyj has joined #openstack-cinder | 14:13 | |
*** iceyao has quit IRC | 14:18 | |
*** haypo has joined #openstack-cinder | 14:21 | |
haypo | DuncanT: 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.29 | 14:22 |
haypo | DuncanT: 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 job | 14:23 |
*** lcurtis has joined #openstack-cinder | 14:24 | |
*** lcurtis has quit IRC | 14:24 | |
*** lcurtis has joined #openstack-cinder | 14:24 | |
*** lcurtis has quit IRC | 14:25 | |
*** lcurtis has joined #openstack-cinder | 14:25 | |
*** ivase has joined #openstack-cinder | 14:27 | |
*** mdenny has joined #openstack-cinder | 14:27 | |
*** knikolla has joined #openstack-cinder | 14:31 | |
*** timcl has quit IRC | 14:33 | |
*** markvoelker has joined #openstack-cinder | 14:41 | |
*** karthikp has joined #openstack-cinder | 14:41 | |
*** ametts has joined #openstack-cinder | 14:42 | |
*** karthikp has quit IRC | 14:43 | |
*** karthikp has joined #openstack-cinder | 14:43 | |
scottda | haypo: This was in response to a request from someone who had attempted to deploy Cinder using py3.... | 14:44 |
scottda | They had thought the table indicated that Cinder+py3 was fully supported. | 14:44 |
scottda | I guess it wasn't clear that the table indicated that only unite tests pass and the tests vote. | 14:45 |
*** jistr has joined #openstack-cinder | 14:46 | |
haypo | scottda: hum, i see. i should clarify the wiki page | 14:46 |
*** pots1 has joined #openstack-cinder | 14:46 | |
haypo | scottda: IMHO no openstack fully work on py3 yet | 14:46 |
*** markvoelker has quit IRC | 14:47 | |
*** nkrinner has quit IRC | 14:48 | |
scottda | haypo: Yes, that's OK. | 14:48 |
scottda | haypo: 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 Python34 | 14:49 |
*** bjornar has quit IRC | 14:49 | |
*** iceyao has joined #openstack-cinder | 14:51 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings https://review.openstack.org/300189 | 14:52 |
*** cfriesen has joined #openstack-cinder | 14:53 | |
*** mylu has joined #openstack-cinder | 14:53 | |
*** garthb_ has joined #openstack-cinder | 14:54 | |
*** e0ne has joined #openstack-cinder | 14:57 | |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Add /v3 endpoint support for cinderclient https://review.openstack.org/300028 | 14:57 |
*** delattec has joined #openstack-cinder | 14:58 | |
ameade | DuncanT: this is the impl for the messages spec https://review.openstack.org/#/c/298052/ | 14:58 |
ameade | looks like some things are still being hashed out on the spec | 14:59 |
openstackgerrit | Eric Harney proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 15:00 |
openstackgerrit | Eric Harney proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 15:00 |
*** haomaiwang has quit IRC | 15:01 | |
*** haomaiwa_ has joined #openstack-cinder | 15:01 | |
*** cdelatte has quit IRC | 15:02 | |
sheel | ameade: hey | 15:03 |
*** EinstCrazy has quit IRC | 15:05 | |
*** markvoelker has joined #openstack-cinder | 15:05 | |
e0ne | yuriy_n17: we can't merge code before spec is landed. please, addess comments in specs first | 15:06 |
*** lprice has joined #openstack-cinder | 15:06 | |
ameade | sheel: hey, in meeting for 30 mins but whats up? | 15:06 |
*** links has joined #openstack-cinder | 15:07 | |
sheel | ameade: hey buddy | 15:07 |
sheel | ameade: I did not find this good what is happening for patch part... | 15:08 |
sheel | ameade: we may discuss once you get some free time for this... | 15:08 |
*** martyt has joined #openstack-cinder | 15:08 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings https://review.openstack.org/300189 | 15: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-cinder | 15:10 | |
*** adrianofr_ has quit IRC | 15:10 | |
akerr | star trek | 15:11 |
ameade | sheel: we can chat now ill just be slow to respond | 15:11 |
guitarzan | jbernard_: http://knowyourmeme.com/memes/hes-dead-jimits-dead-jim | 15:11 |
sheel | scottda: hi, | 15:11 |
sheel | scottda: I have answered your comment on https://review.openstack.org/#/c/273938/ | 15:11 |
sheel | scottda: could you see once if you find it ok as per your time availibility? | 15:11 |
sheel | scottda: thanks | 15:11 |
jbernard_ | guitarzan: oh... i should know that! im clearly slipping | 15:12 |
guitarzan | :) | 15:12 |
guitarzan | it's some kind of nerd test :) | 15:12 |
jbernard_ | and i failed | 15:12 |
sheel | ameade: its about ownership of patch as you know | 15:12 |
sheel | ameade: I am surprized how can someone override other just by adding some code on existing patch? | 15:12 |
*** martyt has quit IRC | 15:13 | |
sheel | ameade: I am not able to justify this.. | 15:13 |
eharney | sheel: 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 difficult | 15:14 |
sheel | eharney: may be this is going in wrong direction | 15:14 |
sheel | eharney: I did no jump in | 15:14 |
sheel | eharney: I raise this topic in cinder in meeting | 15:14 |
sheel | eharney: i opened BP | 15:14 |
sheel | eharney: then I write spec which was discussed in midcycle | 15:15 |
sheel | eharney: if you still think i jumped in, I am ok to leave this to owner you | 15:15 |
sheel | eharney: please decide, I am ok to follow | 15:16 |
*** sgotliv has quit IRC | 15:17 | |
*** arch-nemesis has joined #openstack-cinder | 15:17 | |
ameade | i think i justified it on the patch | 15:17 |
ameade | just putting the patch up initially does not dibb it or something | 15:17 |
eharney | this thing was discussed at a summit in 2014... and is still the same thing we discussed there | 15:18 |
*** Yogi1 has joined #openstack-cinder | 15:18 | |
ameade | I havent been able to make it a priority until now which sucks but its my top priority now | 15:19 |
*** yuriy_n17 has quit IRC | 15:19 | |
tbarron | and ameade pushed this at cinder and manila mid-cycles this year | 15:19 |
sheel | ameade: but why only when i started this?? | 15:20 |
tbarron | sheel: that's what we're saying, this baby has been in production for a long time before | 15:20 |
tbarron | sheel: you weren't around and didn't see that I guess, but it's true | 15:21 |
ameade | sheel: you lit a fire and made it so my company and i made it a priority | 15:21 |
sheel | ameade: its priority for me too | 15:22 |
ameade | nothing is really wrong atm, i want the minimum feature merged soon | 15:22 |
sheel | eharney: tbarron ameade : I am not not opposing you guys and wont even in future but i think things should be disucced | 15:22 |
sheel | eharney: tbarron ameade there was no discussion on this with me... | 15:23 |
sheel | eharney: tbarron ameade it seems i was waisting my time to think about this feature | 15:23 |
eharney | i don't think anyone is opposed to discussing feature design | 15:23 |
ameade | what wasn't discussed? | 15:23 |
eharney | and i'm pretty sure we have been discussing this feature | 15:23 |
sheel | ameade: about ownership of patch | 15:23 |
sheel | ameade: we had discussino that you will be writing API on my code | 15:23 |
sheel | ameade: now you are taking it as whole | 15:24 |
tbarron | sheel: is this bout stackalytics, or about the direction/design of patch? | 15:24 |
*** mylu has quit IRC | 15:24 | |
sheel | ameade: stackalytics??? | 15:24 |
sheel | sorry | 15:24 |
sheel | tbarron: stackalytics? | 15:24 |
sheel | tbarron: ok you mean that statistics tool | 15:24 |
guitarzan | sheel: is there some kind of "credit" you're looking for? | 15:25 |
guitarzan | (same question applies to ameade) | 15:25 |
sheel | guitarzan: hehehe.. no no... its about if we are doing efforts that should be worth it | 15:25 |
guitarzan | "worth"? | 15:25 |
ameade | we oroginally discussed me writing the first iteration and you building on that | 15:26 |
sheel | ameade: really ? no.. | 15:26 |
ameade | i just went ahead and impl the whole patch cuz it was all in my head | 15:26 |
sheel | (please dont take anything wrong here... we are discussing amicably) | 15:26 |
sheel | guitarzan: 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 |
guitarzan | hopefully none of the rest of us care who writes the patch | 15:28 |
guitarzan | I just want the feature | 15:28 |
guitarzan | I'll let you two duke it out | 15:28 |
sheel | guitarzan: I am a bit new to community... so may be I am bit possesive for now.... | 15:28 |
*** mylu has joined #openstack-cinder | 15:28 | |
sheel | guitarzan: things used to improve with time... | 15:28 |
sheel | guitarzan: will happen with me as well after some time working with you guys | 15:29 |
ameade | guitarzan: 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 work | 15:29 |
sheel | ameade: guitarzan may be I should follow your advice | 15:29 |
sheel | ameade: lets devide the work once spec is passed | 15:30 |
sheel | ameade: are you ok with it? | 15:30 |
eharney | sheel: 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 |
sheel | eharney: my interest was to present good feature including horizon and cli | 15:30 |
sheel | eharney: ok, lets ameade work on it | 15:31 |
ameade | sheel sounds fine to me | 15:31 |
*** e0ne has quit IRC | 15:31 | |
sheel | ameade: ok(though i am bit said, but i will manage) | 15:31 |
ameade | sheel: why sad? | 15:33 |
sheel | ameade: i think we can leave it, I am happy to work with good guys | 15:34 |
sheel | ameade: lets discuss more about feature | 15:34 |
sheel | ameade: :) | 15:34 |
sheel | ameade: I will discuss work devision with you on monday or so | 15:35 |
*** haplo37 has joined #openstack-cinder | 15:35 | |
sheel | ameade: les complete this with full entho... | 15:35 |
*** bardia has joined #openstack-cinder | 15:36 | |
*** mylu has quit IRC | 15:38 | |
*** ociuhandu has quit IRC | 15:39 | |
*** daneyon has joined #openstack-cinder | 15:40 | |
*** mylu has joined #openstack-cinder | 15:40 | |
ameade | sheel: yes definitely | 15:40 |
*** jdurgin1 has joined #openstack-cinder | 15:41 | |
*** anshul has quit IRC | 15:42 | |
*** bjornar has joined #openstack-cinder | 15:43 | |
*** iceyao has quit IRC | 15:43 | |
*** krtaylor has joined #openstack-cinder | 15:44 | |
*** Guest90877 has quit IRC | 15:44 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: do the correct update after failover https://review.openstack.org/292570 | 15:49 |
openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings https://review.openstack.org/300189 | 15:54 |
*** links has quit IRC | 15:55 | |
openstackgerrit | Danny Al-Gaaf proposed openstack/os-brick: RBDClient: pass clustername to Rados https://review.openstack.org/300560 | 15:55 |
*** timcl has joined #openstack-cinder | 15:56 | |
*** IlyaG has joined #openstack-cinder | 15:56 | |
*** haplo37 has quit IRC | 15:58 | |
*** haplo37 has joined #openstack-cinder | 15:59 | |
*** haomaiwa_ has quit IRC | 16:01 | |
*** haomaiwang has joined #openstack-cinder | 16:01 | |
*** Yogi1 has quit IRC | 16:05 | |
*** apoorvad has joined #openstack-cinder | 16:06 | |
*** timcl has quit IRC | 16:10 | |
*** martyt has joined #openstack-cinder | 16:10 | |
*** garthb_ has quit IRC | 16:10 | |
*** bjornar has quit IRC | 16:12 | |
*** mriedem is now known as mriedem_lunch | 16:12 | |
*** zzzeek has quit IRC | 16:13 | |
openstackgerrit | Eric Harney proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 16:13 |
*** zzzeek has joined #openstack-cinder | 16:14 | |
*** leeantho has joined #openstack-cinder | 16:15 | |
*** martyt has quit IRC | 16:15 | |
*** mylu has quit IRC | 16:15 | |
openstackgerrit | Tom Barron proposed openstack/cinder: fix invalid uuid warnings from test_quota https://review.openstack.org/297854 | 16:20 |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix fake uuids in NetApp ESeries unit tests https://review.openstack.org/297363 | 16:20 |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix invalid uuids in EMC scaleio unit tests https://review.openstack.org/297375 | 16:20 |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix uuid warnings from db unit tests https://review.openstack.org/297255 | 16:20 |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix invalid uuids in sheepdog unit tests https://review.openstack.org/297453 | 16:20 |
*** e0ne has joined #openstack-cinder | 16:24 | |
*** salv-orlando has joined #openstack-cinder | 16:25 | |
scottda | mfisch: Hi | 16:28 |
*** e0ne has quit IRC | 16:29 | |
*** esker has quit IRC | 16:29 | |
*** xiaohui has quit IRC | 16:29 | |
*** e0ne has joined #openstack-cinder | 16:29 | |
*** xiaohui has joined #openstack-cinder | 16:30 | |
*** openstackgerrit has quit IRC | 16:31 | |
*** guitarzan has quit IRC | 16:31 | |
*** rcernin has quit IRC | 16:32 | |
*** guitarzan has joined #openstack-cinder | 16:32 | |
nikeshm | hi | 16:33 |
*** e0ne has quit IRC | 16:39 | |
*** dflorea has joined #openstack-cinder | 16:40 | |
*** dflorea has quit IRC | 16:41 | |
*** timcl has joined #openstack-cinder | 16:45 | |
*** openstackgerrit has joined #openstack-cinder | 16:46 | |
*** dflorea has joined #openstack-cinder | 16:48 | |
*** edmondsw has quit IRC | 16:50 | |
*** dflorea has quit IRC | 16:50 | |
*** dflorea has joined #openstack-cinder | 16:51 | |
*** dencaval has quit IRC | 16:51 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: WIP: Deprecating XML config file https://review.openstack.org/294076 | 16:55 |
*** e0ne has joined #openstack-cinder | 16:55 | |
*** haypo has left #openstack-cinder | 16:56 | |
openstackgerrit | Eric Harney proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 16:57 |
*** p01nt3r75 has quit IRC | 16:59 | |
*** delattec has quit IRC | 17:00 | |
*** haomaiwang has quit IRC | 17:01 | |
*** haomaiwang has joined #openstack-cinder | 17:01 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Tests: Fix invalid UUIDs in EMC VNX tests https://review.openstack.org/293119 | 17:02 |
*** eharney has quit IRC | 17:02 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings https://review.openstack.org/300189 | 17:04 |
*** genunix has quit IRC | 17:07 | |
*** dencaval has joined #openstack-cinder | 17:07 | |
openstackgerrit | Sheel Rana proposed openstack/cinder-specs: User facing error Summary Messages https://review.openstack.org/273938 | 17:10 |
*** martyt has joined #openstack-cinder | 17:10 | |
*** ndipanov has quit IRC | 17:14 | |
*** e0ne has quit IRC | 17:15 | |
*** martyt has quit IRC | 17:15 | |
*** lpetrut has quit IRC | 17:16 | |
*** laughterwym has quit IRC | 17:17 | |
*** bardia has quit IRC | 17:19 | |
*** xiaohui has quit IRC | 17:22 | |
ameade | scottda: https://review.openstack.org/#/c/300585/ | 17:22 |
*** xiaohui has joined #openstack-cinder | 17:22 | |
*** eharney has joined #openstack-cinder | 17:22 | |
*** mriedem_lunch is now known as mriedem | 17:24 | |
ildikov | scottda: mriedem: hi | 17:24 |
*** laughterwym has joined #openstack-cinder | 17:24 | |
ildikov | scottda: mriedem: just a quick question about the meeting | 17:24 |
ildikov | scottda: mriedem: we mentioned that we might want to have it as a series | 17:25 |
mriedem | ildikov: ok | 17:25 |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Updated to utilize provider_id https://review.openstack.org/300118 | 17:25 |
ildikov | scottda: mriedem do we want to start it now or after the summit? | 17:26 |
mriedem | ildikov: i was talking pre-summit | 17:26 |
mriedem | ildikov: there was a lot of fog in the meeting this week for multiattach | 17:26 |
mriedem | i think we need to have a much more clear picture on the plan for multiattach before the summit | 17:26 |
*** anshul has joined #openstack-cinder | 17:27 | |
ildikov | scottda: mriedem: ok, works for me, then I will register it as a series | 17:27 |
ildikov | scottda: mriedem I will also keep the slot if you're fine with it as we had quite many people around this week | 17:27 |
mriedem | ildikov: works for me | 17:28 |
hemna | mriedem, hey sorry I didn't attend that meeting. I wasn't at work | 17:28 |
hemna | mriedem, I'm reading the logs today from the meeting to get back up to speed on it. | 17:28 |
*** laughterwym has quit IRC | 17:28 | |
mriedem | brb | 17:29 |
ildikov | mriedem: coolio, thanks | 17:29 |
ildikov | thingee: ping | 17:30 |
*** mriedem has quit IRC | 17:33 | |
*** mriedem has joined #openstack-cinder | 17:34 | |
hemna | so, 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 |
hemna | initialize_connection needs the connector for it to find the correct attachment | 17:35 |
hemna | and/or the attachment_id | 17:35 |
*** delattec has joined #openstack-cinder | 17:35 | |
mriedem | hemna: ildikov: also, i'm told powervc has had multi-attach working since kilo and gfm has the patches (gerald mcbrearty) | 17:35 |
mriedem | i never see him irc, but i'm trying to get him/them involved | 17:36 |
*** salv-orlando has quit IRC | 17:36 | |
hemna | ok | 17:36 |
hemna | well it may work for some specific backends that do a new target for every attach | 17:37 |
hemna | 3PAR does the same thing | 17:37 |
mriedem | i'm assuming powervc is using storwize | 17:37 |
hemna | if the backend does that, it just worksā¢ | 17:37 |
ildikov | mriedem: wow, cool, I didn't know | 17:38 |
hemna | I'd have to test live migration though | 17:38 |
*** delatte has joined #openstack-cinder | 17:38 | |
hemna | that's the part that throws a wrench into everything. | 17:38 |
hemna | as the attachment switches hosts | 17:38 |
*** delattec has quit IRC | 17:40 | |
*** haomaiwang has quit IRC | 17:40 | |
*** haomaiwang has joined #openstack-cinder | 17:41 | |
hemna | scottda, ping | 17:42 |
ildikov | hemna: you mean live migration with those drivers that exports new target each time? | 17:42 |
hemna | just in general | 17:42 |
hemna | the host in the attachment needs to be updated | 17:43 |
hemna | re: if we are going to store instance id and host | 17:43 |
hemna | which we currently aren't, but we should | 17:43 |
hemna | this is part of the update attachment piece that doesn't exist today | 17:43 |
ildikov | we also discussed storing connector info during the meeting briefly | 17:44 |
hemna | reading through it... | 17:44 |
hemna | just getting to that point. | 17:44 |
ildikov | but the overall point is that we will need to update some info anyway | 17:44 |
*** dflorea has quit IRC | 17:44 | |
hemna | I think we need to start storing the host in the attachment | 17:44 |
ildikov | by the microversion support we should be in a much better situation regarding this now | 17:45 |
hemna | as well as keep it up to date | 17:45 |
*** salv-orlando has joined #openstack-cinder | 17:45 | |
hemna | and store the connector for each attachment. | 17:45 |
hemna | in cinder | 17:45 |
hemna | that solves a couple issues | 17:45 |
hemna | live migration, shelve, evacuate | 17:45 |
ildikov | yeah, that's where we ended up as well IIRC | 17:45 |
hemna | for both single attachments and multi-attach | 17:45 |
hemna | the other glaring issues is what nova should do at detach time. | 17:46 |
hemna | to call os-brick disconnect_volume or not. | 17:46 |
ildikov | it would be important to see further than multiattach as sometimes single attachments can be problematic as well | 17:46 |
hemna | maybe if cinder includes a shared=True flag in the connection_info back from initialize_connection, nova would know | 17:47 |
hemna | that means changing cinder drivers to include that flag | 17:47 |
hemna | that avoids the extra round trip by nova at detach time | 17:47 |
hemna | since I think nova is already saving the connection_info in the bdm at attach time. | 17:47 |
hemna | hrmm | 17:47 |
hemna | but | 17:47 |
ildikov | if we could add it to the attachment info somehow that would be good as we call Cinder before detach already | 17:48 |
hemna | that doesn't say if there is any other instance still using that volume right now. | 17:48 |
hemna | nova will have to see if other instances are using that same volume as well, even if the flag is True | 17:48 |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix uuid warnings from api.contrib unit tests https://review.openstack.org/297337 | 17:49 |
hemna | nova 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 detach | 17:49 |
ildikov | in 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 time | 17:49 |
mriedem | so 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 |
hemna | and there aren't other instances still using it. | 17:49 |
hemna | mriedem, yah | 17:49 |
ildikov | hemna: is this true for all back ends? | 17:50 |
mriedem | i thought it wasn't, and that's what the conn_info.shared flag is for | 17:50 |
ildikov | or in case of having separate targets than we need to call disconnect_volume every case still? | 17:50 |
hemna | the shared flag tells nova if the same target is exported by the cinder backend for the same volume. | 17:50 |
mriedem | separate targets would mean conn_info.shared=false, right? | 17:50 |
hemna | mriedem, correct | 17:51 |
hemna | mriedem, that's my assumption of the meaning of the flag. | 17:51 |
mriedem | the flag that exists in hemna's head today :) | 17:51 |
hemna | :) | 17:51 |
*** timcl has quit IRC | 17:51 | |
ildikov | ah cool, sorry I misread it | 17:51 |
ildikov | mriedem: yeah, i got that part :) | 17:52 |
hemna | crap, am I confusing everyone ? | 17:52 |
mriedem | i tihnk i'm following | 17:52 |
ildikov | I got there too | 17:52 |
*** markvoelker has quit IRC | 17:52 | |
ildikov | I'm just slower Friday evening than usual... :) | 17:52 |
hemna | so | 17:53 |
*** timcl has joined #openstack-cinder | 17:53 | |
hemna | if we store the host and the instance id in cinder | 17:53 |
hemna | that helps | 17:53 |
mriedem | store it in the vol attachment table? | 17:53 |
*** jdurgin1 has quit IRC | 17:53 | |
hemna | yup | 17:53 |
hemna | like the patch I had that I backed out | 17:53 |
hemna | in Cinder | 17:53 |
ildikov | yeah, that's what we tried to do but then we couldn't due to backward compat issues :( | 17:54 |
hemna | I think we need to microversion that guy in | 17:54 |
hemna | and then figure out how to cope with taht | 17:54 |
hemna | that | 17:54 |
hemna | since microversions are a thing now in Cinder. they solve everything....... | 17:54 |
hemna | </snark> | 17:54 |
mriedem | they really do | 17:55 |
ildikov | yeah, scottda mentioned that we can microversion now | 17:55 |
mriedem | i guess i'm not familiar with what the backward compat issue before was | 17:55 |
mriedem | returning something new in the vol attachment response, or adding a new parameter to the request? | 17:55 |
hemna | mriedem, so currently, cinder pukes if you pass them both in | 17:55 |
scottda | hemna: hey. I'm AFK. Back in a few minutes. | 17:55 |
mriedem | to the rest api | 17:55 |
ildikov | Cinder today fails when you add host name and instance_id too to the attach call | 17:56 |
mriedem | b/c it expects 1 and gets 2 | 17:56 |
mriedem | yeah, so you microversion that | 17:56 |
mriedem | to add the 2nd param | 17:56 |
*** markvoelker has joined #openstack-cinder | 17:56 | |
mriedem | let's say that's v2.2 | 17:56 |
ildikov | and hemna fixed it on the API, but in case of older Cinder deploys that does not help | 17:56 |
mriedem | nova asks cinder for it's versions and if cinder.server.version < 2.2, we can't do this | 17:56 |
ildikov | and send it only when we have v2.2 | 17:56 |
ildikov | or higher | 17:57 |
mriedem | yeah, that's why microversions solve everything :) the client can make an informed decision | 17:57 |
*** jistr has quit IRC | 17:58 | |
mriedem | in the case of older cinder deploys, nova can't do multiattach things with them | 17:58 |
mriedem | and we'll raise an error | 17:58 |
ildikov | yeah, my brain likes it, I just wasn't that emotionally ready for it until now :) | 17:58 |
mriedem | you'll find out you like it when you can move your API forward | 17:58 |
ildikov | if we/Cinder will use the host info for other cases as well we could do this separately from multiattach maybe | 17:58 |
*** dflorea has joined #openstack-cinder | 17:59 | |
ildikov | of course we will enable multiattach for that Cinder version that properly supports it, I meant the host name sending effort | 18:00 |
*** haomaiwang has quit IRC | 18:01 | |
hemna | https://github.com/openstack/cinder/commit/3632173486f89d541c121be8e041a1da636edf32 | 18:01 |
*** haomaiwang has joined #openstack-cinder | 18:01 | |
hemna | um, I thought we had backed that out ? | 18:01 |
mriedem | hemna: so that wasn't reverted? | 18:02 |
mriedem | https://review.openstack.org/#/c/275316/ | 18:02 |
mriedem | i think ildikov did the revert | 18:02 |
mriedem | during the midcycle | 18:02 |
*** dflorea has quit IRC | 18:02 | |
*** asselin__ has joined #openstack-cinder | 18:02 | |
hemna | guess not | 18:02 |
ildikov | I reverted it in Nova | 18:02 |
*** harlowja has quit IRC | 18:02 | |
hemna | I'm looking at the history on that file | 18:03 |
hemna | and there is no revert | 18:03 |
hemna | it's still in afaik | 18:03 |
mriedem | yeah | 18:03 |
*** rlrossit has quit IRC | 18:03 | |
ildikov | was it reverted in Cinder as well? | 18:03 |
mriedem | ildikov: do you have the change? | 18:03 |
hemna | so...there you have it. | 18:03 |
mriedem | ildikov: it wasn't reverted from cinder | 18:03 |
*** dflorea has joined #openstack-cinder | 18:04 | |
hemna | damn | 18:04 |
mriedem | but it doesn't work, right? | 18:04 |
ildikov | the Nova one: https://review.openstack.org/#/c/273097/ | 18:04 |
mriedem | ah right | 18:05 |
mriedem | yeah, jgriffith had a revert up too at the same time | 18:05 |
hemna | well I thought we had to revert it because it was an API change | 18:05 |
ildikov | I haven't touched Cinder as it does not do any harm until Nova does not try to start it... | 18:05 |
mriedem | hemna: you probably still do | 18:05 |
mriedem | it will have to be microversioned for nova to use it | 18:05 |
mriedem | there, i added cinder to the bug and added mitaka to it https://bugs.launchpad.net/cinder/+bug/1538620 | 18:06 |
openstack | Launchpad bug 1538620 in Cinder "Attach with host and instance_uuid not backwards compatible" [Undecided,New] | 18:06 |
mriedem | need to revert under that | 18:06 |
hemna | ok I'll rebase the change and push it up | 18:07 |
ildikov | hemna: cool, tnx | 18:07 |
mriedem | push it real good? | 18:07 |
hemna | :) | 18:07 |
ildikov | lol :) | 18:08 |
hemna | ok /me waits for local tox happiness | 18:09 |
*** bardia has joined #openstack-cinder | 18:10 | |
*** asselin__ has quit IRC | 18:10 | |
*** diablo_rojo has joined #openstack-cinder | 18:15 | |
scottda | hemna: Hi | 18:17 |
*** dflorea has quit IRC | 18:18 | |
*** asselin has joined #openstack-cinder | 18:19 | |
*** mvk_ has quit IRC | 18:19 | |
*** dflorea has joined #openstack-cinder | 18:20 | |
scottda | ameade: Cool on the devstack patch for Cinder v3...I'll test here in a little while. | 18:21 |
*** IlyaG has quit IRC | 18:22 | |
*** garthb has joined #openstack-cinder | 18:22 | |
*** dflorea has quit IRC | 18:23 | |
*** rlrossit has joined #openstack-cinder | 18:25 | |
*** p01nt3r75 has joined #openstack-cinder | 18:26 | |
*** asselin__ has joined #openstack-cinder | 18:27 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Revert "Allow host and instance_uuid at attach" https://review.openstack.org/275316 | 18:28 |
*** asselin has quit IRC | 18:29 | |
*** markvoelker has quit IRC | 18:31 | |
jgriffith | mriedem: hemna I thought I reverted that... it didn't work anyway IIRC | 18:32 |
*** edmondsw has joined #openstack-cinder | 18:32 | |
jgriffith | mriedem: hemna oh well, sounds like you're on it now | 18:32 |
*** lprice has quit IRC | 18:32 | |
hemna | I had thought it had gotten in :( | 18:32 |
jgriffith | It broke a bunch of things.. so I'm pretty sure we reverted it | 18:33 |
hemna | until I looked at the git history on the file. | 18:33 |
ildikov | jgriffith: this chang worked, but when we added the complementing part to Nova it broke older Cinder deploys | 18:33 |
jgriffith | ildikov: right you are | 18:33 |
ildikov | jgriffith: I reverted the Nova change when the issue turned out | 18:33 |
ildikov | jgriffith: but as the Cinder change did not do any harm by itself that was kept finally | 18:34 |
openstackgerrit | Tom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings https://review.openstack.org/300189 | 18:34 |
jgriffith | hmm... | 18:34 |
jgriffith | ildikov: well, I don't necessarily agree with that but regardless :) Water under the bridge now | 18:34 |
ildikov | hemna: I assume this needs to be backported as well to Mitaka | 18:34 |
* jgriffith goes back to shaving his Yak | 18:34 | |
hemna | ildikov, +1 | 18:35 |
ildikov | jgriffith: well, I'm not saying it was good to leave it there, but it kinda happened, anyway hemna is fixing it now :) | 18:35 |
jgriffith | ildikov: yeah, I get ya.. no worries | 18:35 |
*** karthikp has quit IRC | 18:35 | |
ildikov | jgriffith: end of week now, so I definitely will not miss any sleep because of it :) | 18:37 |
jgriffith | ildikov: haha.. I hear that!! | 18:37 |
hemna | :) | 18:37 |
jgriffith | ildikov: as I've become older I've also realized that there are few things worth missing sleep over anyway :) | 18:37 |
guitarzan | <3 sleep | 18:38 |
ildikov | jgriffith: yeah, I started to realise that myself as well | 18:38 |
ildikov | jgriffith: just recently though ;) | 18:38 |
ildikov | guitarzan: +1 :) | 18:38 |
*** dalgaaf has quit IRC | 18:38 | |
scottda | hemna: You want me to put up a patch that microversions "Allow host and instance_uuid at attach" ? | 18:39 |
hemna | scottda, or teach me how to microversion ? | 18:40 |
scottda | hemna: Either way is fine with me. Is it easier to show how to microversion by me putting up the patch? | 18:41 |
hemna | if it's easy for you, then go ahead | 18:41 |
hemna | I do want to learn/figure out how to do it | 18:41 |
scottda | ok. Did you have something else to discuss earlier when you ping'ed me? | 18:42 |
scottda | Was it about a spec for new api to update connector and/or host info? | 18:42 |
scottda | hemna: Cause I wanted to discuss some details of that with you, at some point. | 18:42 |
*** dflorea has joined #openstack-cinder | 18:43 | |
*** dflorea has quit IRC | 18:43 | |
hemna | yah | 18:43 |
hemna | I think so | 18:43 |
*** ChubYann has joined #openstack-cinder | 18:43 | |
*** dflorea has joined #openstack-cinder | 18:43 | |
scottda | Got time to talk now? PM me if /when ready | 18:44 |
thingee | ildikov: 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 today | 18:49 |
ildikov | thingee: thanks, but no worries, I pinged you for another reason this time | 18:50 |
ildikov | thingee: just a very quick question | 18:50 |
*** bjornar has joined #openstack-cinder | 18:50 | |
ildikov | thingee: is that ok if I schedule a Cinder-Nova API interactions weekly meeting on #openstack-meeting-cp channel? | 18:51 |
*** Lee1092 has quit IRC | 18:52 | |
ildikov | thingee: the question is mainly whether it is ok to use that channel for this purpose? | 18:52 |
*** david-lyle has quit IRC | 18:53 | |
*** markvoelker has joined #openstack-cinder | 18:53 | |
*** markvoelker has quit IRC | 18:53 | |
*** markvoelker has joined #openstack-cinder | 18:54 | |
*** markvoelker has quit IRC | 18:54 | |
*** markvoelker has joined #openstack-cinder | 18:54 | |
*** dflorea has quit IRC | 18:57 | |
*** david-lyle has joined #openstack-cinder | 18:59 | |
*** dflorea has joined #openstack-cinder | 19:01 | |
*** haomaiwang has quit IRC | 19:01 | |
*** haomaiwang has joined #openstack-cinder | 19:01 | |
thingee | ildikov: 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 |
thingee | ildikov: let me double check with ttx though. | 19:03 |
ildikov | thingee: I think when we manage to sort out the interaction problems we will stop with the series as well | 19:03 |
ildikov | thingee: but for now it would be good to have a periodical follow up on items | 19:03 |
ildikov | thingee: and as it is cross-project activity that channel is tempting, but I'm ok to find another one if it still does not fit | 19:04 |
ildikov | thingee: tnx for checking! | 19:04 |
*** salv-orlando has quit IRC | 19:07 | |
*** lprice has joined #openstack-cinder | 19:09 | |
*** ebalduf has joined #openstack-cinder | 19:11 | |
*** martyt has joined #openstack-cinder | 19:12 | |
*** martyt has quit IRC | 19:17 | |
*** karthikp has joined #openstack-cinder | 19:27 | |
*** timcl1 has joined #openstack-cinder | 19:31 | |
*** lprice1 has joined #openstack-cinder | 19:31 | |
*** timcl has quit IRC | 19:32 | |
*** lprice has quit IRC | 19:34 | |
*** harlowja has joined #openstack-cinder | 19:37 | |
*** sgotliv has joined #openstack-cinder | 19:39 | |
*** dflorea has quit IRC | 19:39 | |
*** dflorea has joined #openstack-cinder | 19:41 | |
*** dflorea has quit IRC | 19:42 | |
*** mvk_ has joined #openstack-cinder | 19:43 | |
*** salv-orlando has joined #openstack-cinder | 19:48 | |
*** dflorea_ has joined #openstack-cinder | 19:49 | |
mfisch | scottda: hey, sorry I missed you earlier | 19:52 |
scottda | mfisch: No problem. I updated the whiteboard in the bug. | 19:52 |
mfisch | ok | 19:53 |
scottda | mfisch: I've put this in my queue and get something up next week. | 19:53 |
mfisch | I kinda of rediscovered it today when an Icinga check didn't fail that should have | 19:53 |
mfisch | awesome thats great | 19:53 |
scottda | mfisch: Yeah, sorry, I forgot all about this. | 19:53 |
mfisch | np | 19:54 |
*** timcl1 has quit IRC | 19:54 | |
*** garthb_ has joined #openstack-cinder | 19:57 | |
*** ebalduf has quit IRC | 19:58 | |
*** garthb has quit IRC | 20:00 | |
*** haomaiwang has quit IRC | 20:01 | |
Swanson | failture | 20:01 |
*** haomaiwang has joined #openstack-cinder | 20:01 | |
jungleboyj | Swanson: I sent them a note yesterday. | 20:01 |
*** pots1 has quit IRC | 20:02 | |
*** pots1 has joined #openstack-cinder | 20:02 | |
Swanson | :) | 20:03 |
*** apoorvad has quit IRC | 20:04 | |
*** dflorea_ has quit IRC | 20:04 | |
*** dflorea has joined #openstack-cinder | 20:05 | |
*** pots1 has quit IRC | 20:06 | |
*** porrua has quit IRC | 20:08 | |
*** akerr has quit IRC | 20:11 | |
*** dflorea has quit IRC | 20:14 | |
*** jungleboyj has quit IRC | 20:14 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Updated to utilize provider_id https://review.openstack.org/300118 | 20:17 |
*** rlrossit has quit IRC | 20:18 | |
*** salv-orl_ has joined #openstack-cinder | 20:23 | |
*** salv-orlando has quit IRC | 20:26 | |
openstackgerrit | Gerald McBrearty proposed openstack/cinder: Storwize: fix unmapped multi-attached volume failure https://review.openstack.org/299673 | 20:31 |
*** IlyaG has joined #openstack-cinder | 20:37 | |
*** markvoelker has quit IRC | 20:37 | |
*** apoorvad has joined #openstack-cinder | 20:38 | |
karthikp | I see that change made as part of https://review.openstack.org/#/c/198341/14/cinder/volume/api.py been reverted? | 20:39 |
karthikp | It kind of seems like it because master shows old code: | 20:39 |
karthikp | https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L252 | 20:39 |
karthikp | If it wasnt reverted it would be strange thing to see! | 20:39 |
*** raildo is now known as raildo-afk | 20:42 | |
*** mylu has joined #openstack-cinder | 20:45 | |
*** dflorea has joined #openstack-cinder | 20:50 | |
DuncanT | karthikp: 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 explaination | 20:50 |
*** jgregor has quit IRC | 20:52 | |
*** dflorea has quit IRC | 20:55 | |
*** delatte has quit IRC | 20:56 | |
*** mriedem1 has joined #openstack-cinder | 20:56 | |
*** eharney has quit IRC | 20:58 | |
*** eharney has joined #openstack-cinder | 20:59 | |
*** mriedem has quit IRC | 20:59 | |
*** haomaiwang has quit IRC | 21:01 | |
*** haomaiwang has joined #openstack-cinder | 21:01 | |
*** haplo37 has quit IRC | 21:01 | |
*** diablo_rojo has quit IRC | 21:05 | |
*** gouthamr has quit IRC | 21:10 | |
karthikp | DuncanT: I tried using the git blame -L252 cinder/volume/api.py and it shows an old change by John | 21:10 |
DuncanT | karthikp: Curious.... | 21:16 |
*** dflorea has joined #openstack-cinder | 21:22 | |
karthikp | DuncanT, sorry it is clarified..was checking a different line | 21:25 |
*** dflorea has quit IRC | 21:28 | |
*** mriedem1 is now known as mriedem | 21:29 | |
*** akshai_ has quit IRC | 21:30 | |
*** dflorea has joined #openstack-cinder | 21:31 | |
*** haplo37 has joined #openstack-cinder | 21:31 | |
*** sgotliv has quit IRC | 21:35 | |
*** bjornar has quit IRC | 21:36 | |
*** mriedem has quit IRC | 21:38 | |
*** bjornar has joined #openstack-cinder | 21:38 | |
*** jungleboyj has joined #openstack-cinder | 21:44 | |
*** xyang1 has quit IRC | 21:49 | |
*** ivase has quit IRC | 21:55 | |
*** sdague has quit IRC | 21:56 | |
*** jungleboyj has quit IRC | 21:57 | |
*** haomaiwang has quit IRC | 22:01 | |
*** markvoelker_ has joined #openstack-cinder | 22:01 | |
*** haomaiwang has joined #openstack-cinder | 22:01 | |
*** dflorea has quit IRC | 22:04 | |
*** gouthamr has joined #openstack-cinder | 22:06 | |
*** anshul has quit IRC | 22:09 | |
*** dflorea has joined #openstack-cinder | 22:11 | |
*** lutf-8 is now known as alaski | 22:12 | |
*** dflorea has quit IRC | 22:13 | |
*** edmondsw has quit IRC | 22:15 | |
*** dflorea has joined #openstack-cinder | 22:15 | |
*** pots1 has joined #openstack-cinder | 22:15 | |
*** pots1 has quit IRC | 22:19 | |
*** pots1 has joined #openstack-cinder | 22:19 | |
*** lcurtis has quit IRC | 22:21 | |
*** pots1 has quit IRC | 22:24 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: Allow host and instance_uuid at attach https://review.openstack.org/300684 | 22:25 |
scottda | hemna: ^^^ | 22:25 |
*** sheel has quit IRC | 22:27 | |
*** ametts has quit IRC | 22:28 | |
*** mylu has quit IRC | 22:32 | |
hemna | sweet | 22:33 |
*** mylu has joined #openstack-cinder | 22:33 | |
*** bardia has quit IRC | 22:33 | |
*** markvoelker_ has quit IRC | 22:39 | |
scottda | We can talk about the details Monday or so if you'd like. | 22:43 |
*** krtaylor has quit IRC | 22:44 | |
hemna | scottda, ok sounds good | 22:48 |
*** knikolla has quit IRC | 22:53 | |
*** IlyaG has quit IRC | 22:54 | |
*** IlyaG has joined #openstack-cinder | 22:54 | |
*** IlyaG has quit IRC | 22:55 | |
*** IlyaG has joined #openstack-cinder | 22:55 | |
*** karthikp has quit IRC | 22:55 | |
*** IlyaG has quit IRC | 22:55 | |
*** krtaylor has joined #openstack-cinder | 22:56 | |
*** haomaiwang has quit IRC | 23:01 | |
*** haomaiwang has joined #openstack-cinder | 23:01 | |
*** alyson_ has quit IRC | 23:02 | |
*** mylu has quit IRC | 23:03 | |
*** dflorea has quit IRC | 23:11 | |
*** dflorea has joined #openstack-cinder | 23:12 | |
*** mtanino has quit IRC | 23:13 | |
*** dflorea has quit IRC | 23:17 | |
*** iceyao has joined #openstack-cinder | 23:19 | |
*** knikolla has joined #openstack-cinder | 23:22 | |
*** haplo37 has quit IRC | 23:22 | |
*** erlon has quit IRC | 23:27 | |
*** harlowja has quit IRC | 23:36 | |
*** marcusvrn_ has quit IRC | 23:37 | |
*** dflorea has joined #openstack-cinder | 23:37 | |
*** dflorea has quit IRC | 23:37 | |
*** dflorea has joined #openstack-cinder | 23:38 | |
*** dflorea has quit IRC | 23:40 | |
*** esker has joined #openstack-cinder | 23:43 | |
*** esker has quit IRC | 23:47 | |
*** dflorea has joined #openstack-cinder | 23:52 | |
*** arch-nemesis has quit IRC | 23:52 | |
*** dflorea has quit IRC | 23:53 | |
*** dflorea has joined #openstack-cinder | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!