openstackgerrit | Vincent Hou proposed openstack/cinder: LVM: add the exception handling to volume copy https://review.openstack.org/218750 | 00:02 |
---|---|---|
*** takedakn1 has joined #openstack-cinder | 00:03 | |
*** akerr has joined #openstack-cinder | 00:07 | |
*** takedakn has quit IRC | 00:07 | |
*** takedakn1 is now known as takedakn | 00:07 | |
*** akerr_ has joined #openstack-cinder | 00:08 | |
*** takedakn1 has joined #openstack-cinder | 00:10 | |
*** akerr has quit IRC | 00:11 | |
*** takedakn has quit IRC | 00:11 | |
*** takedakn1 is now known as takedakn | 00:11 | |
*** rushil_ has quit IRC | 00:13 | |
*** rushil has joined #openstack-cinder | 00:14 | |
*** tobasco_ has quit IRC | 00:15 | |
*** tobasco has joined #openstack-cinder | 00:16 | |
*** garthb__ has quit IRC | 00:18 | |
openstackgerrit | Matt Riedemann proposed openstack/python-cinderclient: Update path to subunit2html in post_test_hook https://review.openstack.org/219929 | 00:19 |
mriedem | you're welcome! ^ | 00:19 |
*** goodygum has quit IRC | 00:19 | |
*** mtanino_ has quit IRC | 00:21 | |
*** goodygum has joined #openstack-cinder | 00:21 | |
*** leeantho has quit IRC | 00:33 | |
*** bill_az_ has quit IRC | 00:36 | |
*** hodos has quit IRC | 00:36 | |
*** takedakn1 has joined #openstack-cinder | 00:40 | |
*** p0rtal_ has quit IRC | 00:43 | |
*** takedakn has quit IRC | 00:44 | |
*** p0rtal has joined #openstack-cinder | 00:44 | |
*** ganso_ has quit IRC | 00:45 | |
*** IanGovett has quit IRC | 00:45 | |
openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 00:45 |
*** takedakn1 has quit IRC | 00:46 | |
*** p0rtal has quit IRC | 00:48 | |
*** rushil has quit IRC | 01:02 | |
*** julim has joined #openstack-cinder | 01:05 | |
*** zul has quit IRC | 01:14 | |
jgriffith | mriedem: ? | 01:15 |
*** julim has quit IRC | 01:15 | |
jgriffith | mriedem: of course thank you for the bug and the shiny patch to go with it | 01:15 |
mriedem | apparently novaclient did a thing and other clients copied it | 01:16 |
mriedem | and we didn't see the ML thread about it going to break | 01:16 |
openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Improve image operations https://review.openstack.org/202954 | 01:16 |
*** zul has joined #openstack-cinder | 01:17 | |
jgriffith | mriedem: yeah, your reference is the first I saw the thread as well :( | 01:17 |
jgriffith | mriedem: thanks for being on top of it and fixing it up | 01:17 |
*** julim has joined #openstack-cinder | 01:20 | |
*** BharatK has joined #openstack-cinder | 01:20 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 01:21 |
*** rbradbury has quit IRC | 01:26 | |
*** ganso_ has joined #openstack-cinder | 01:27 | |
*** Lee1092 has joined #openstack-cinder | 01:30 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 01:42 |
*** julim has quit IRC | 01:47 | |
*** boris-42 has quit IRC | 01:50 | |
*** sghanekar has quit IRC | 01:53 | |
*** akerr_ has quit IRC | 02:23 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Remove duplicate code in functional tests https://review.openstack.org/219772 | 02:24 |
*** julim has joined #openstack-cinder | 02:26 | |
*** akerr has joined #openstack-cinder | 02:27 | |
*** akerr is now known as akerr_away | 02:27 | |
*** akerr_away has quit IRC | 02:29 | |
*** haomaiwang has quit IRC | 02:36 | |
*** haomaiwang has joined #openstack-cinder | 02:38 | |
openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Improve image operations https://review.openstack.org/202954 | 02:44 |
*** hakimo_ has quit IRC | 02:54 | |
openstackgerrit | John Griffith proposed openstack/cinder: Add placholder for migration backports in Liberty https://review.openstack.org/219965 | 02:55 |
*** mudassirlatif has quit IRC | 02:56 | |
*** haomaiwang has quit IRC | 02:57 | |
*** hakimo has joined #openstack-cinder | 02:57 | |
*** haomaiwa_ has joined #openstack-cinder | 03:00 | |
*** mriedem has quit IRC | 03:00 | |
*** xiaohui has quit IRC | 03:02 | |
*** xiaohui has joined #openstack-cinder | 03:02 | |
*** bkopilov has quit IRC | 03:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 03:05 |
*** rushil has joined #openstack-cinder | 03:07 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/210918 | 03:10 |
*** rushil has quit IRC | 03:17 | |
*** guest1 has joined #openstack-cinder | 03:19 | |
openstackgerrit | Gerald McBrearty proposed openstack/cinder: Switch SVC driver to use lsportfc to determine FC target WWPNS https://review.openstack.org/219973 | 03:27 |
*** erhudy1 has quit IRC | 03:29 | |
*** guest1 has quit IRC | 03:30 | |
*** guest1 has joined #openstack-cinder | 03:31 | |
*** dims has joined #openstack-cinder | 03:35 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use oslo-config-generator https://review.openstack.org/219977 | 03:38 |
*** Zhongjun_ has joined #openstack-cinder | 03:48 | |
*** julim has quit IRC | 03:54 | |
*** gouthamr has quit IRC | 03:59 | |
*** boris-42 has joined #openstack-cinder | 03:59 | |
*** dims has quit IRC | 04:00 | |
*** links has joined #openstack-cinder | 04:02 | |
*** BharatK has quit IRC | 04:04 | |
*** baojg has joined #openstack-cinder | 04:11 | |
*** svilgelm has joined #openstack-cinder | 04:14 | |
*** guest1 has quit IRC | 04:30 | |
*** baojg has quit IRC | 04:31 | |
*** p0rtal has joined #openstack-cinder | 04:41 | |
*** p0rtal has quit IRC | 04:41 | |
*** p0rtal has joined #openstack-cinder | 04:42 | |
*** ganso_ has quit IRC | 04:45 | |
*** shyama has joined #openstack-cinder | 04:47 | |
*** yrabl has quit IRC | 04:49 | |
*** svilgelm has quit IRC | 04:51 | |
*** dannywilson has joined #openstack-cinder | 04:52 | |
*** bkopilov has joined #openstack-cinder | 04:57 | |
*** dims has joined #openstack-cinder | 04:59 | |
*** BharatK has joined #openstack-cinder | 04:59 | |
*** afazekas has joined #openstack-cinder | 05:04 | |
*** BharatK has quit IRC | 05:06 | |
*** dims has quit IRC | 05:06 | |
openstackgerrit | Tina Tang proposed openstack/cinder: Clone cg support in VNX driver https://review.openstack.org/215530 | 05:12 |
*** BharatK has joined #openstack-cinder | 05:13 | |
*** shausy has joined #openstack-cinder | 05:16 | |
*** deepakcs has joined #openstack-cinder | 05:18 | |
*** p0rtal has quit IRC | 05:21 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 05:22 |
*** p0rtal has joined #openstack-cinder | 05:25 | |
*** p0rtal has quit IRC | 05:25 | |
*** p0rtal has joined #openstack-cinder | 05:26 | |
*** shausy has quit IRC | 05:27 | |
*** lpetrut has joined #openstack-cinder | 05:29 | |
*** p0rtal has quit IRC | 05:35 | |
*** dannywilson has quit IRC | 05:41 | |
openstackgerrit | Tina Tang proposed openstack/cinder: Clone cg support in VNX driver https://review.openstack.org/215530 | 05:43 |
*** ishant has joined #openstack-cinder | 05:43 | |
*** yrabl has joined #openstack-cinder | 05:46 | |
*** nkrinner has joined #openstack-cinder | 06:02 | |
*** yrabl has quit IRC | 06:08 | |
*** yrabl has joined #openstack-cinder | 06:10 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/218741 | 06:18 |
*** shyama has quit IRC | 06:21 | |
*** agarciam has joined #openstack-cinder | 06:23 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: LVM: add the exception handling to volume copy https://review.openstack.org/218750 | 06:26 |
*** sgotliv has joined #openstack-cinder | 06:30 | |
*** kaisers1 has quit IRC | 06:34 | |
*** lpetrut has quit IRC | 06:37 | |
*** stefan_amann has joined #openstack-cinder | 06:57 | |
*** shyama has joined #openstack-cinder | 06:58 | |
*** svilgelm has joined #openstack-cinder | 07:13 | |
*** amoturi has quit IRC | 07:19 | |
*** amoturi has joined #openstack-cinder | 07:20 | |
*** guest1 has joined #openstack-cinder | 07:21 | |
*** kvidvans has quit IRC | 07:22 | |
*** guest1 has quit IRC | 07:22 | |
*** ronis has joined #openstack-cinder | 07:26 | |
*** anshul has joined #openstack-cinder | 07:31 | |
*** anshul has quit IRC | 07:33 | |
*** anshul has joined #openstack-cinder | 07:33 | |
*** afazekas__ has joined #openstack-cinder | 07:33 | |
*** ociuhandu has quit IRC | 07:41 | |
*** Zhongjun_ has quit IRC | 07:47 | |
*** lpetrut has joined #openstack-cinder | 07:47 | |
*** ociuhandu has joined #openstack-cinder | 07:52 | |
*** vbala has joined #openstack-cinder | 07:52 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the destination volume check in delete_volume https://review.openstack.org/219108 | 08:01 |
*** links has quit IRC | 08:02 | |
*** zhipeng has joined #openstack-cinder | 08:03 | |
*** ndipanov has joined #openstack-cinder | 08:06 | |
*** vbala has left #openstack-cinder | 08:12 | |
*** zhipeng has quit IRC | 08:14 | |
*** zhipeng has joined #openstack-cinder | 08:15 | |
*** alexschm has joined #openstack-cinder | 08:17 | |
*** zhipeng_ has joined #openstack-cinder | 08:23 | |
*** hodos has joined #openstack-cinder | 08:25 | |
*** zhipeng has quit IRC | 08:25 | |
*** zhipeng_ has quit IRC | 08:27 | |
*** zhipeng has joined #openstack-cinder | 08:29 | |
*** jistr has joined #openstack-cinder | 08:37 | |
*** zhipeng has quit IRC | 08:39 | |
*** zhipeng has joined #openstack-cinder | 08:41 | |
*** guest1 has joined #openstack-cinder | 08:45 | |
*** guest1 has quit IRC | 08:46 | |
*** vgridnev has joined #openstack-cinder | 08:58 | |
*** d0ugal has left #openstack-cinder | 09:02 | |
*** vgridnev has quit IRC | 09:02 | |
*** zhipeng has quit IRC | 09:02 | |
*** vgridnev has joined #openstack-cinder | 09:03 | |
*** vgridnev has quit IRC | 09:03 | |
*** vgridnev has joined #openstack-cinder | 09:03 | |
*** bluex-pl has joined #openstack-cinder | 09:16 | |
*** bluex-pl has quit IRC | 09:16 | |
*** bluex-pl has joined #openstack-cinder | 09:16 | |
openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 09:23 |
*** belmoreira has joined #openstack-cinder | 09:26 | |
*** e0ne has joined #openstack-cinder | 09:32 | |
dulek | e0ne: Hi! Can you please take a look at my comment on https://review.openstack.org/#/c/217857/ to confirm I'm not talking nonsense? | 09:55 |
*** IanGovett has joined #openstack-cinder | 10:02 | |
*** e0ne has quit IRC | 10:04 | |
*** vbala_ has joined #openstack-cinder | 10:05 | |
*** vbala_ has quit IRC | 10:06 | |
*** e0ne has joined #openstack-cinder | 10:08 | |
*** kbyrne has quit IRC | 10:12 | |
DuncanT | dulek: This python stuff is all nonsense. | 10:13 |
*** kbyrne has joined #openstack-cinder | 10:15 | |
*** vbala has joined #openstack-cinder | 10:16 | |
DuncanT | dulek: On a more serious note, I think it is going to be a worthwhile summit session to define exactly what AZ means to different people, and what AZ usecases we want to support. My understanding of an AZ means that the idea of a fallback AZ makes little sense, but clearly other people have a different idea of how AZs should work | 10:16 |
e0ne | DuncanT: +1. we need more feedback from operators | 10:18 |
DuncanT | e0ne: I'm just emailing the submitter of that bug now. At the very least I can write up how we want AZs to work, and hopefully get details of how they want AZs to work too, then try to ensure cinder can do both | 10:19 |
DuncanT | e0ne: Currently it's a bit organic, which I don't like - there's no good advice on how to actually build a resilient system | 10:19 |
DuncanT | e0ne: If you have an idea of how AZs should work, please drop me an email (and anybody else who thinks they have an understanding of what they want from AZs - regardless of what we currently have) | 10:20 |
e0ne | DuncanT: personal mail for you or to openstack-dev? | 10:22 |
DuncanT | e0ne: To me directly to start with please, I'll try to summarise before we take it to the list | 10:23 |
e0ne | dulek: i'm agree with DuncanT's comment | 10:23 |
e0ne | dulek: tbh, i don't like this patch but won't block it if operators want such strange feature | 10:24 |
dulek | DuncanT: Haven't all 3 threads on openstack-dev on the topic explained what operators expect? | 10:24 |
e0ne | DuncanT: ok. I will mail you and try to get feedback from our ops | 10:24 |
e0ne | dulek: we still don't know WHY they want it | 10:25 |
DuncanT | dulek: Not clearly, no. I will go through those as well though, and try to get diagrams of the (at least) two different models that are being deployed. We can then look at the failure cases for each model and see if it makes sense | 10:25 |
e0ne | dulek: it was an issue in Nova. and know they say: we don't want that issue to be fixed. it's good enough for us. | 10:26 |
DuncanT | dulek: Having a good clear statement of what each group is trying to achieve would help me a lot for certain, and I doubt I'm alone. For my mental model of AZs, what is being asked for in that patch makes no sense, and providing a volume in anything other than the requested AZ is a bug | 10:27 |
dulek | DuncanT, e0ne: An explanation why an operator want it: http://lists.openstack.org/pipermail/openstack-dev/2015-August/073141.html | 10:27 |
dulek | DuncanT: And I agree that diagrams and analysis of possible failures would be beneficial. | 10:28 |
e0ne | dulek, DuncanT: I saw it. that's why I proposed an option AZ per backend | 10:28 |
dulek | e0ne, DuncanT: Sorry, got to go to a meeting, I'll answer you in some time. | 10:28 |
DuncanT | dulek: But that explaination makes little sense unless they're willing to accept that cinder node failures will take down multiple AZs, and they are allowing nova to mount cross-AZ - if that is the case then fine, but I want to make that clear | 10:29 |
e0ne | dulek: but DuncanT's concern about 'fake AZ' looks right for some cases | 10:29 |
e0ne | DuncanT: +1 | 10:29 |
DuncanT | dulek: I can then document how to configure that setup, and document what the failure modes are, and the advantages and disadvantages of doing so | 10:29 |
DuncanT | dulek: I want all of the supported configurations to be documented | 10:30 |
*** links has joined #openstack-cinder | 10:31 | |
*** yrabl has quit IRC | 10:33 | |
*** aix has quit IRC | 10:46 | |
*** deepakcs has quit IRC | 10:54 | |
*** ganso_ has joined #openstack-cinder | 10:55 | |
*** aix has joined #openstack-cinder | 11:15 | |
*** ishant has quit IRC | 11:19 | |
*** pschaef has joined #openstack-cinder | 11:19 | |
*** eharney has joined #openstack-cinder | 11:21 | |
*** BharatK has quit IRC | 11:22 | |
*** takedakn has joined #openstack-cinder | 11:25 | |
*** sayali has quit IRC | 11:26 | |
*** timcl has joined #openstack-cinder | 11:27 | |
nikeshm | DotHill CI is passed on the bug-fix by hemna https://review.openstack.org/#/c/219534/, its need one more +2 | 11:36 |
*** ociuhandu has quit IRC | 11:36 | |
*** takedakn has quit IRC | 11:36 | |
*** BharatK has joined #openstack-cinder | 11:36 | |
*** takedakn has joined #openstack-cinder | 11:36 | |
*** julim has joined #openstack-cinder | 11:37 | |
*** Yogi1 has joined #openstack-cinder | 11:37 | |
*** svilgelm has quit IRC | 11:39 | |
*** takedakn1 has joined #openstack-cinder | 11:41 | |
*** Yogi1 has quit IRC | 11:42 | |
*** takedakn has quit IRC | 11:44 | |
*** takedakn1 is now known as takedakn | 11:44 | |
*** bkopilov has quit IRC | 11:51 | |
*** breitz has quit IRC | 11:51 | |
*** breitz has joined #openstack-cinder | 11:51 | |
*** takedakn has quit IRC | 11:52 | |
*** ociuhandu has joined #openstack-cinder | 11:55 | |
*** diogogmt has joined #openstack-cinder | 11:56 | |
*** alexpilotti_ has joined #openstack-cinder | 11:58 | |
*** guest1 has joined #openstack-cinder | 12:11 | |
*** pschaef has quit IRC | 12:11 | |
DuncanT | nikeshm: Approved. Any idea why the CI failed? Something you fixed or an intermittent issue? | 12:12 |
nikeshm | DuncanT: Vedams CI is different from recheck-HP Storage CI | 12:14 |
nikeshm | DuncanT: vedams CI was not failed on this patch | 12:14 |
DuncanT | nikeshm: Oh, sorry, I misread the history. Thanks. | 12:15 |
nikeshm | DuncanT: looks like recheck-HP Storage CI was failed | 12:15 |
nikeshm | DuncanT: thanks for +2, :) , so what weeked plan (friday and saturday) | 12:16 |
DuncanT | nikeshm: Beer. Lots of beer. Some dancing. Maybe a trip down to the desert. | 12:17 |
*** guest1 has quit IRC | 12:17 | |
*** raildo-afk is now known as raildo | 12:18 | |
DuncanT | @nikeshm I should probably put up my casual review friday list before the end of the day, too | 12:19 |
nikeshm | DuncanT: happy weekend after long tiring week | 12:20 |
DuncanT | nikeshm: Cheers :-) | 12:21 |
*** JoseMello has joined #openstack-cinder | 12:22 | |
*** martyturner has joined #openstack-cinder | 12:23 | |
*** svilgelm has joined #openstack-cinder | 12:24 | |
*** xyang1 has joined #openstack-cinder | 12:24 | |
*** shyama has quit IRC | 12:25 | |
*** akerr has joined #openstack-cinder | 12:27 | |
*** diogogmt has quit IRC | 12:27 | |
*** akerr has quit IRC | 12:28 | |
*** svilgelm has quit IRC | 12:28 | |
*** akerr has joined #openstack-cinder | 12:29 | |
*** Yogi1 has joined #openstack-cinder | 12:31 | |
*** diogogmt has joined #openstack-cinder | 12:31 | |
*** svilgelm has joined #openstack-cinder | 12:34 | |
*** stefan_amann has quit IRC | 12:34 | |
*** shyama has joined #openstack-cinder | 12:34 | |
*** stefan_amann has joined #openstack-cinder | 12:35 | |
*** porrua has joined #openstack-cinder | 12:36 | |
openstackgerrit | Gerald McBrearty proposed openstack/cinder: Switch SVC driver to use lsportfc to determine FC target WWPNS https://review.openstack.org/219973 | 12:39 |
*** diogogmt has quit IRC | 12:40 | |
*** dims has joined #openstack-cinder | 12:41 | |
*** Yogi1 has quit IRC | 12:41 | |
geguileo | dulek: ping - rolling upgrades with VOs | 12:42 |
*** sayali has joined #openstack-cinder | 12:42 | |
dulek | geguileo: Hello! | 12:44 |
geguileo | dulek: Hi | 12:44 |
dulek | geguileo: What's up? :) | 12:44 |
geguileo | dulek: New worries, that's what's up :-( | 12:45 |
openstackgerrit | abhiram moturi proposed openstack/cinder: ZFSSA driver to return project 'available' space https://review.openstack.org/211299 | 12:45 |
geguileo | dulek: I'm worried about rolling migrations with VOs | 12:45 |
geguileo | dulek: Because I believe we don't have the structure to support it | 12:45 |
dulek | geguileo: Okay, can you elaborate? | 12:45 |
geguileo | dulek: For upgrades we may have DB migrations | 12:46 |
* geguileo hoping he's wrong and dulek can remove his worries | 12:46 | |
*** diogogmt has joined #openstack-cinder | 12:46 | |
DuncanT | VOs? What is a VO? | 12:47 |
geguileo | DuncanT: Versioned Objects | 12:47 |
geguileo | dulek: So how do we manage DB migrations with the rolling upgrades? | 12:47 |
DuncanT | geguileo: We don't, yet | 12:48 |
DuncanT | geguileo: Some of the infrastructure is there, but far from all | 12:48 |
dulek | geguileo: Give me a minute to rethink if I'm not missing something. | 12:48 |
*** anshul has quit IRC | 12:49 | |
geguileo | DuncanT: My worry is that our current architecture does not allow us to do rolling upgrades | 12:49 |
DuncanT | geguileo: Well, given it isn't finished, no it doesn't. It will do though - thang had a blueprint for how it will fit together | 12:49 |
geguileo | dulek: Thanks, though I'm really hopping that it's me the one that is missing something | 12:49 |
geguileo | DuncanT: I'll have a look, but I think it's missing the part that worries me | 12:50 |
DuncanT | geguileo: Basically we tell all services to pin the max object version they send over the wire (downgrading before send as necessary) until every service is upgraded, then we remove the pin | 12:50 |
geguileo | DuncanT: Yeah, that's the easy stuff | 12:51 |
geguileo | DuncanT: What about schema migrations? | 12:51 |
dulek | geguileo: Okay, so Nova wants to address it with two-phase DB migrations - first phase adds new columns and second removes them when all services are upgraded | 12:51 |
geguileo | dulek: Nova has a Conductor | 12:51 |
DuncanT | geguileo: I'm sure we had an answer, but I need to go back and re-read plans | 12:51 |
dulek | geguileo: I don't see how conductor changes two-phase migrations idea. | 12:52 |
*** sayali has quit IRC | 12:52 | |
geguileo | DuncanT: I'll have a look at the spec | 12:52 |
DuncanT | geguileo: I think the basic answer is objects need to be able to talk to both schema versions | 12:52 |
DuncanT | geguileo: I'm not sure it got as far as a concrete design, now you come to mention it :-( | 12:52 |
geguileo | dulek: So all new added colums will forcefully have default values or allow NULL, right? | 12:53 |
dulek | geguileo: http://specs.openstack.org/openstack/neutron-specs/specs/liberty/online-schema-migrations.html (Neuron spec is a little better defined than Nova, and Neutron has no conductor) | 12:53 |
geguileo | dulek: Wrong | 12:53 |
geguileo | dulek: I spoke with a Neutron guy yesterday | 12:53 |
dulek | geguileo: Actually you cannot add new column to the DB without def value or allowing NULL. | 12:54 |
geguileo | dulek: And they have a kind of conductor | 12:54 |
geguileo | dulek: No, I mean, that new entries must have default value (not existing ones) | 12:54 |
geguileo | dulek: So neutron passes all DB access through their kind of conductor and then manage messages to agents | 12:55 |
*** chutwig has joined #openstack-cinder | 12:55 | |
*** sayali has joined #openstack-cinder | 12:56 | |
xek | geguileo, the code that actually talks with the db is sqlalchemy | 12:56 |
geguileo | XD XD XD | 12:56 |
geguileo | xek: That's not what we are talking about | 12:57 |
geguileo | xek: That's a library for the abstraction layer | 12:57 |
xek | geguileo, so in a case of an upgrade, there may be two different sqlalchemy models, and that's why two-phase migrations are a good idea | 12:58 |
dulek | geguileo, DuncanT: xek is my team member who's a little more into oslo.VO and generally upgrades in various projects. | 12:58 |
geguileo | xek: Yep, we would have different models on different nodes | 12:58 |
dulek | geguileo: About conductor stuff. I'm not that familiar with Neutron, so let's get back to Nova. It isn't true that all DB accesses in Nova's services go through conductor. | 12:59 |
*** BharatK has quit IRC | 13:00 | |
geguileo | dulek: Well, they are "supposed to" XD | 13:00 |
*** tpsilva has joined #openstack-cinder | 13:01 | |
dulek | geguileo: I was always confused by what was the real goal of conductor, but you may be right. ;) | 13:01 |
xek | dulek, geguileo it depends on the configuration | 13:01 |
xek | what is always done by the conductor is backporting of objects for the computes which run with an older version | 13:02 |
openstackgerrit | Oleksii Butenko proposed openstack/python-cinderclient: Add functional tests for python-cinderclient https://review.openstack.org/218895 | 13:02 |
geguileo | xek: So compute nodes send requests directly to DB if DB version is the same as their VOs and through conductor if they are not? | 13:03 |
* geguileo must be understanding it wrong, because that sounds weird | 13:03 | |
xek | geguileo, it depends on the CONF.conductor.use_local setting | 13:05 |
* dulek is going to talk through a common version of what's happening with xek. ;) | 13:05 | |
geguileo | dulek: What do you mean by that ^? | 13:05 |
openstackgerrit | Kendall Nelson proposed openstack/cinder: WIP: Dynamically create cinder.conf.sample https://review.openstack.org/219700 | 13:08 |
*** timcl has quit IRC | 13:08 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fix exception messages https://review.openstack.org/220132 | 13:10 |
dulek | geguileo: I've needed to make sure that we understand each other correctly, and that's easier to do in person and in my native language. ;) | 13:11 |
*** diogogmt_ has joined #openstack-cinder | 13:12 | |
dulek | geguileo, DuncanT: Now I think that to handle DB migrations we also need online DB schema migrations like Nova or Neutron. | 13:12 |
dulek | geguileo, DuncanT: Given that we're actually removing or renaming columns through migrations, which I believe isn't happening. | 13:12 |
*** diogogmt has quit IRC | 13:13 | |
*** diogogmt_ is now known as diogogmt | 13:13 | |
* dulek is glancing through Liberty migration scripts. | 13:13 | |
*** edmondsw has joined #openstack-cinder | 13:13 | |
geguileo | dulek: But we may want to in the future | 13:13 |
*** dustins has joined #openstack-cinder | 13:15 | |
*** sayali has quit IRC | 13:16 | |
*** BharatK has joined #openstack-cinder | 13:16 | |
dulek | geguileo: I agree, we should have that possibility. | 13:16 |
dulek | geguileo: From what I see we don't have any incompatible DB schema changes in Liberty, but that doesn't mean there won't be any in the future. | 13:17 |
*** Yogi11 has joined #openstack-cinder | 13:18 | |
*** gouthamr has joined #openstack-cinder | 13:18 | |
dulek | geguileo, DuncanT: Is two-phase DB schema migrations a solution for that in your opinion? | 13:18 |
dulek | s/Is/Are ;) | 13:18 |
geguileo | dulek: I'm going to give it some thought, as it isn't a trivial matter | 13:19 |
geguileo | dulek: But the first thing that comes to mind is that it could work | 13:19 |
DuncanT | dulek: I don't know. I think we need a dummy migration as a demo... a migration that adds a column to one table and removes one from another, and shows the objects doing a version adaption | 13:19 |
DuncanT | dulek: I guess adding and removing a column from the same table is the third test case | 13:20 |
dulek | DuncanT: I don't think that's how it is supposed to work. | 13:20 |
DuncanT | dulek: oh? | 13:20 |
dulek | DuncanT: It's more like - add columns before upgrades - everything works fine, older versions just ignore the column (given it has default in the DB or is nullable). | 13:21 |
dulek | DuncanT: After everything is upgraded - run second phase that will remove the unused columns. | 13:21 |
*** sayali has joined #openstack-cinder | 13:22 | |
dulek | DuncanT: A harder case would be how to handle conversions of column values. I'm not sure on that, need to take a look into Nova's and Neutron's specs. | 13:22 |
*** sayali has quit IRC | 13:24 | |
dulek | DuncanT: But Nova needs also to handle such cases - there can be a lot of conductors and these may not be upgraded at once. | 13:24 |
*** BharatK has quit IRC | 13:26 | |
*** sayali has joined #openstack-cinder | 13:27 | |
e0ne | volume attachment API is broken in nova:(. only non-voting job in cinder found this issue:( | 13:35 |
e0ne | *non-voting rally job | 13:35 |
*** jgregor has joined #openstack-cinder | 13:36 | |
geguileo | e0ne: With the new tests you added? | 13:37 |
e0ne | geguileo: no. volume attach scenario was merged a while ago | 13:38 |
e0ne | nobody cares that python-somethingclient works | 13:38 |
e0ne | nobody cares about non-voting job failed | 13:38 |
geguileo | e0ne: Which client is broken? | 13:39 |
e0ne | geguileo: actually, it's not nova-client. it's nova api | 13:39 |
e0ne | geguileo: but simple test for client could find such issue | 13:39 |
geguileo | e0ne: And their CI didn't catch it? | 13:39 |
e0ne | geguileo: no. | 13:39 |
e0ne | geguileo: https://bugs.launchpad.net/cinder/+bug/1491842 | 13:40 |
openstack | Launchpad bug 1491842 in Cinder "Can't attach volume without specifying device name" [High,Confirmed] | 13:40 |
*** merooney has joined #openstack-cinder | 13:40 | |
* geguileo has finally lost his faith in OpenStack's CIs | 13:40 | |
*** links has quit IRC | 13:40 | |
jgriffith | geguileo: I'm impressed that you ever had any | 13:40 |
jgriffith | geguileo: more != better | 13:41 |
geguileo | jgriffith: Well, it only lasted 6 months so... ;-) | 13:41 |
jgriffith | geguileo: HA | 13:41 |
e0ne | jgriffith: :) | 13:42 |
e0ne | my first lesson with openstack was: do not read the documentation. read the code! | 13:43 |
jgriffith | e0ne: so I'm curious however on that bug | 13:43 |
geguileo | e0ne: Yep, I had that one early on as well | 13:44 |
e0ne | jgriffith: imo, root cause is: we don't test our clients with API well. tempest tests only API | 13:46 |
jgriffith | e0ne: so that IS one of the reasons we were supposed to start moving functional in to Cinder remember | 13:47 |
jgriffith | although this case I don't understand yet and it wouldn't have found it | 13:47 |
e0ne | jgriffith: absolutly agree | 13:47 |
jgriffith | e0ne: I'm trying to understand the bug, and what exactly rally does that exposes it? | 13:48 |
e0ne | jgriffith: what do you mean? | 13:48 |
jgriffith | I'm doing nova volume-attach all day long here | 13:48 |
e0ne | jgriffith: rally calls something like "nova volume-attach <instance_id> <volume_id>" | 13:48 |
*** eharney has quit IRC | 13:49 | |
e0ne | jgriffith: but for now, nova requires something like "nova volume-attach <instance_id> <volume_id> <device_name>" | 13:49 |
*** asselin_ has joined #openstack-cinder | 13:50 | |
jgriffith | e0ne: yeah, so I'm trying to understand what's the "cinder" bug | 13:50 |
jgriffith | sounds like a Rally bug | 13:50 |
* jgriffith is confused | 13:50 | |
e0ne | jgriffith: it's a nova bug | 13:50 |
e0ne | jgriffith: my concern was: only Cinder CI found it | 13:50 |
jgriffith | e0ne: although dberrange (I think) had proposed about dropping dev altogether | 13:50 |
jgriffith | e0ne: oh... I see | 13:51 |
*** anshul has joined #openstack-cinder | 13:51 | |
jgriffith | e0ne: well, kinda :) | 13:51 |
johnthetubaguy | looking to tag a new python-novaclient soon, thats very related | 13:51 |
jgriffith | kinda not | 13:51 |
jgriffith | johnthetubaguy: :) | 13:51 |
jgriffith | johnthetubaguy: I knew there was some discussion I caught around this, just couldn't remember details :) | 13:52 |
johnthetubaguy | the device_name is just not properly honoured, so we are trying to remove that (in the eventual sense) | 13:52 |
johnthetubaguy | but thats a different discussion I guess | 13:52 |
jgriffith | johnthetubaguy: yeah, I know Vish fought it for years, sounds like it's time to just punt on it and be done :) | 13:52 |
jgriffith | johnthetubaguy: but I'm confused, because apparantly e0ne is expecting it to work today without the "dev" arg | 13:53 |
*** alexpilotti_ is now known as alexpilotti | 13:53 | |
jgriffith | which I never thought was something that was done | 13:53 |
jgriffith | or supported yet, just talked about | 13:53 |
e0ne | jgriffith, johnthetubaguy: "device" argument should be optional | 13:53 |
johnthetubaguy | I kinda thought thats been optional for a while, but I would have to go dig to say when, I could be wrong | 13:54 |
jgriffith | e0ne: I must be wrong, I'll look; I thought the "option" was "dev=auto" | 13:54 |
johnthetubaguy | optional is a bit racey, as it just picks the "next" one | 13:54 |
jgriffith | but honestly haven't looked in quite a while so I should likely be ignored :) | 13:54 |
johnthetubaguy | jgriffith: oh, possibly... | 13:54 |
johnthetubaguy | yeah, not dug recently either | 13:55 |
e0ne | johnthetubaguy: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3057 | 13:56 |
e0ne | johnthetubaguy: the same for the python-novaclient | 13:56 |
*** sayali has quit IRC | 13:57 | |
jgriffith | hmm... well then | 13:57 |
e0ne | and https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/volumes.py#L78 | 13:58 |
*** timcl has joined #openstack-cinder | 13:58 | |
e0ne | looks like optional params are required now | 13:58 |
*** mriedem has joined #openstack-cinder | 13:59 | |
*** sayali has joined #openstack-cinder | 13:59 | |
*** Yogi11 has quit IRC | 14:02 | |
*** bnemec has quit IRC | 14:02 | |
*** zhongjun has quit IRC | 14:02 | |
*** bnemec has joined #openstack-cinder | 14:02 | |
*** rushil has joined #openstack-cinder | 14:04 | |
openstackgerrit | Marian Horban proposed openstack/cinder: Remove wsgi functionality https://review.openstack.org/219671 | 14:04 |
*** svilgelm has quit IRC | 14:07 | |
*** thangp has joined #openstack-cinder | 14:08 | |
*** stefan_amann has quit IRC | 14:08 | |
*** rushil has quit IRC | 14:09 | |
openstackgerrit | Peter Penchev proposed openstack/cinder: Reintroduce the StorPool driver. https://review.openstack.org/220155 | 14:09 |
*** stefan_amann has joined #openstack-cinder | 14:12 | |
openstackgerrit | Peter Penchev proposed openstack/os-brick: Add the StorPool brick connector. https://review.openstack.org/192639 | 14:15 |
*** eharney has joined #openstack-cinder | 14:19 | |
dulek | jgriffith: https://review.openstack.org/#/c/217857/ - is my understanding presented in the comments there valid? | 14:28 |
*** rushil_ has joined #openstack-cinder | 14:28 | |
*** vbala has quit IRC | 14:30 | |
*** amoturi has quit IRC | 14:30 | |
*** bkopilov has joined #openstack-cinder | 14:32 | |
*** Yogi1 has joined #openstack-cinder | 14:32 | |
*** devlaps has quit IRC | 14:33 | |
*** eharney has quit IRC | 14:34 | |
*** mtanino has joined #openstack-cinder | 14:35 | |
*** chutwig has quit IRC | 14:36 | |
jgriffith | dulek: ahh... indeed | 14:37 |
*** markstur has quit IRC | 14:38 | |
*** theanalyst has quit IRC | 14:39 | |
*** amoturi has joined #openstack-cinder | 14:39 | |
*** sayali has quit IRC | 14:40 | |
jgriffith | dulek: Edmund made some valid points as well. But the gist of it is that "yes" we want to log the resource we're creating. We don't have any handle to that in this case though | 14:40 |
*** asselin__ has joined #openstack-cinder | 14:40 | |
jgriffith | dulek: My opinion would be to merge this and file a bug, rather than drag Edmund through another change on it | 14:40 |
jgriffith | dulek: mostly because he added it at my request ;) | 14:40 |
*** garthb has joined #openstack-cinder | 14:40 | |
*** theanalyst has joined #openstack-cinder | 14:41 | |
dulek | jgriffith: Okay, I'm getting my -1 back. | 14:42 |
*** asselin_ has quit IRC | 14:42 | |
*** markstur has joined #openstack-cinder | 14:43 | |
jgriffith | dulek: thanks for pointing that out, I updated my vote and comment | 14:44 |
nikeshm | which nova service to restart after changing nova.volume.cinder.py in devstack | 14:44 |
jgriffith | nikeshm: api usually I tihnk | 14:45 |
dulek | jgriffith: Ah, we're going this way. Okay, it's fine for me. Thanks for help! | 14:45 |
jgriffith | dulek: ? | 14:45 |
openstackgerrit | Tom Barron proposed openstack/cinder: NetApp E-Series over-subscription support https://review.openstack.org/215801 | 14:45 |
jgriffith | dulek: No, I'm saying your -1 was correct/valid... I'm agreeing with you :) | 14:45 |
dulek | jgriffith: "My opinion would be to merge this and file a bug" and then you get your +2 back. | 14:46 |
dulek | jgriffith: That got me confused, but I agree getting that in RC is proper option. | 14:46 |
*** diogogmt_ has joined #openstack-cinder | 14:46 | |
jgriffith | dulek: yeah, I don't communicate well :) | 14:46 |
*** diogogmt has quit IRC | 14:47 | |
*** diogogmt_ is now known as diogogmt | 14:47 | |
dulek | jgriffith: It's fine. ;) | 14:47 |
*** shyama has quit IRC | 14:48 | |
*** eharney has joined #openstack-cinder | 14:48 | |
*** asselin_ has joined #openstack-cinder | 14:49 | |
*** amoturi has quit IRC | 14:49 | |
*** asselin__ has quit IRC | 14:52 | |
*** mtanino has quit IRC | 14:53 | |
*** anshul has quit IRC | 14:53 | |
*** jungleboyj has joined #openstack-cinder | 14:56 | |
*** erhudy has joined #openstack-cinder | 14:59 | |
*** Zhongjun_ has joined #openstack-cinder | 15:00 | |
*** edtubill has joined #openstack-cinder | 15:00 | |
*** jistr is now known as jistr|call | 15:02 | |
*** ronis has quit IRC | 15:02 | |
*** rushil_ has quit IRC | 15:04 | |
*** sayali has joined #openstack-cinder | 15:05 | |
*** markvoelker has joined #openstack-cinder | 15:08 | |
*** esker has joined #openstack-cinder | 15:10 | |
*** sayali has quit IRC | 15:10 | |
*** vgridnev has quit IRC | 15:12 | |
*** chutwig has joined #openstack-cinder | 15:14 | |
*** esker has quit IRC | 15:14 | |
*** alexpilotti_ has joined #openstack-cinder | 15:16 | |
*** alexpilotti has quit IRC | 15:17 | |
*** alexpilotti_ is now known as alexpilotti | 15:17 | |
*** shyama has joined #openstack-cinder | 15:17 | |
openstackgerrit | Edmund Rhudy proposed openstack/cinder: Adds allow_availability_zone_fallback option to Cinder https://review.openstack.org/217857 | 15:20 |
*** mriedem has quit IRC | 15:21 | |
*** annegentle has joined #openstack-cinder | 15:21 | |
*** sayali has joined #openstack-cinder | 15:22 | |
*** p0rtal has joined #openstack-cinder | 15:26 | |
*** dims has quit IRC | 15:26 | |
*** p0rtal has quit IRC | 15:26 | |
*** p0rtal has joined #openstack-cinder | 15:26 | |
*** hemnafk is now known as hemna | 15:28 | |
*** thangp has quit IRC | 15:29 | |
*** harlowja_at_home has joined #openstack-cinder | 15:29 | |
*** jdurgin1 has joined #openstack-cinder | 15:31 | |
*** dannywilson has joined #openstack-cinder | 15:31 | |
*** sayali has quit IRC | 15:32 | |
*** nkrinner has quit IRC | 15:34 | |
*** stefan_amann has quit IRC | 15:35 | |
*** dims has joined #openstack-cinder | 15:36 | |
*** sayali has joined #openstack-cinder | 15:36 | |
*** mtanino has joined #openstack-cinder | 15:36 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use oslo-config-generator https://review.openstack.org/219977 | 15:41 |
*** markvoelker_ has joined #openstack-cinder | 15:42 | |
*** thangp has joined #openstack-cinder | 15:42 | |
*** jistr|call is now known as jistr | 15:44 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use oslo-config-generator https://review.openstack.org/219977 | 15:45 |
*** markvoelker has quit IRC | 15:45 | |
*** harlowja_at_home has quit IRC | 15:48 | |
*** breitz has quit IRC | 15:49 | |
*** jdurgin1 has quit IRC | 15:50 | |
*** rushil has joined #openstack-cinder | 15:56 | |
*** bkopilov has quit IRC | 15:58 | |
*** afazekas__ has quit IRC | 15:59 | |
*** p0rtal_ has joined #openstack-cinder | 16:00 | |
*** p0rtal has quit IRC | 16:03 | |
*** zhongjun has joined #openstack-cinder | 16:03 | |
*** p0rtal_ has quit IRC | 16:04 | |
*** jistr has quit IRC | 16:06 | |
*** jaypipes has joined #openstack-cinder | 16:07 | |
*** bkopilov has joined #openstack-cinder | 16:10 | |
*** p0rtal has joined #openstack-cinder | 16:11 | |
*** p0rtal has quit IRC | 16:11 | |
*** leeantho has joined #openstack-cinder | 16:11 | |
*** aix has quit IRC | 16:12 | |
*** p0rtal has joined #openstack-cinder | 16:12 | |
*** crose has joined #openstack-cinder | 16:14 | |
*** ndipanov has quit IRC | 16:21 | |
*** shyama has quit IRC | 16:25 | |
*** chutwig has quit IRC | 16:25 | |
*** sayali has quit IRC | 16:26 | |
*** pots1 has joined #openstack-cinder | 16:27 | |
*** chutwig has joined #openstack-cinder | 16:29 | |
*** pots has quit IRC | 16:29 | |
*** mudassirlatif has joined #openstack-cinder | 16:31 | |
*** vgridnev has joined #openstack-cinder | 16:32 | |
*** shyama has joined #openstack-cinder | 16:32 | |
*** ronis has joined #openstack-cinder | 16:34 | |
*** e0ne has quit IRC | 16:34 | |
*** sayali has joined #openstack-cinder | 16:35 | |
*** annegentle has quit IRC | 16:36 | |
*** dflorea has joined #openstack-cinder | 16:37 | |
*** annegentle has joined #openstack-cinder | 16:37 | |
*** esker has joined #openstack-cinder | 16:38 | |
*** afazekas__ has joined #openstack-cinder | 16:40 | |
*** dflorea has quit IRC | 16:41 | |
*** mriedem has joined #openstack-cinder | 16:42 | |
*** p0rtal_ has joined #openstack-cinder | 16:43 | |
*** sghanekar has joined #openstack-cinder | 16:45 | |
*** dflorea has joined #openstack-cinder | 16:45 | |
*** p0rtal has quit IRC | 16:46 | |
*** p0rtal_ has quit IRC | 16:47 | |
*** lpetrut has quit IRC | 16:49 | |
*** yuriy_n17 has quit IRC | 16:53 | |
*** martyturner has quit IRC | 16:57 | |
*** haomaiw__ has joined #openstack-cinder | 17:02 | |
*** timcl has quit IRC | 17:02 | |
*** Lee1092 has quit IRC | 17:03 | |
*** haomaiwa_ has quit IRC | 17:03 | |
*** bluex-pl has quit IRC | 17:04 | |
*** p0rtal has joined #openstack-cinder | 17:04 | |
*** p0rtal has quit IRC | 17:05 | |
*** martyturner has joined #openstack-cinder | 17:05 | |
*** belmoreira has quit IRC | 17:07 | |
*** afazekas__ has quit IRC | 17:08 | |
*** afazekas__ has joined #openstack-cinder | 17:12 | |
openstackgerrit | Derrick Wippler proposed openstack/cinder: Hacking log format arg check https://review.openstack.org/220246 | 17:13 |
*** timcl has joined #openstack-cinder | 17:14 | |
*** markvoelker_ has quit IRC | 17:16 | |
*** erhudy has quit IRC | 17:19 | |
*** julim has quit IRC | 17:21 | |
*** afazekas__ has quit IRC | 17:21 | |
*** kvidvans has joined #openstack-cinder | 17:23 | |
*** esker has quit IRC | 17:23 | |
*** zul has quit IRC | 17:26 | |
*** zul has joined #openstack-cinder | 17:27 | |
*** zul has quit IRC | 17:27 | |
*** zul has joined #openstack-cinder | 17:28 | |
*** martyturner has quit IRC | 17:30 | |
jgriffith | diablo_rojo: hey | 17:31 |
*** annegentle has quit IRC | 17:33 | |
*** hogepodge has joined #openstack-cinder | 17:33 | |
hogepodge | hi o/ | 17:33 |
hogepodge | is there a good place to see the current list of drivers that are being tested in cinder CI? | 17:33 |
hogepodge | I've looked at the wiki, but that seems to be updated manually. | 17:33 |
jgriffith | hogepodge: yah, that's a fun game :) | 17:34 |
jgriffith | hogepodge: so two things I can think of: | 17:34 |
asselin | hogepodge, this is a new one that I like: http://ci-watch.tintri.com/project?project=cinder&time=24+hours | 17:34 |
jgriffith | asselin: :) | 17:34 |
jgriffith | that's the first one :) | 17:34 |
hogepodge | asselin: oh, that's exactly what I need in the format I need it in. | 17:35 |
asselin | makes IBM GPFS CI look bad though | 17:35 |
asselin | jungleboyj, ^^ | 17:35 |
*** timcl has quit IRC | 17:36 | |
jgriffith | asselin: theyre flash one too | 17:36 |
jgriffith | funny how "nothing" ever fails | 17:36 |
jgriffith | guess that's one way to do it :) | 17:36 |
asselin | glad to hear that random failures are good to have :) | 17:37 |
*** timcl has joined #openstack-cinder | 17:37 | |
*** alexschm has quit IRC | 17:37 | |
Swanson | While that page makes one of the dell drivers look really bad in our defense someone stole the SAN. | 17:38 |
nikeshm | hemna: hi, thanks | 17:38 |
*** timcl1 has joined #openstack-cinder | 17:39 | |
nikeshm | hemna: is this also need to change https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/san/hp/hp_3par_common.py#L1574 | 17:39 |
asselin | Swanson, actually that's what I like about it. I can quickly see that I should ignore those failure results | 17:39 |
nikeshm | or already some patch is there to change | 17:39 |
hemna | nikeshm, https://review.openstack.org/#/c/219498/ | 17:40 |
nikeshm | hemna: thanks | 17:41 |
*** timcl has quit IRC | 17:41 | |
*** annegentle has joined #openstack-cinder | 17:42 | |
*** lpetrut has joined #openstack-cinder | 17:43 | |
jgriffith | Swanson: LMAO | 17:44 |
*** e0ne has joined #openstack-cinder | 17:46 | |
*** p0rtal has joined #openstack-cinder | 17:47 | |
*** sayali has quit IRC | 17:50 | |
*** sayali has joined #openstack-cinder | 17:53 | |
*** Zhongjun_ has quit IRC | 17:57 | |
*** dflorea has quit IRC | 17:58 | |
*** julim has joined #openstack-cinder | 17:58 | |
*** lpetrut has quit IRC | 18:06 | |
*** sayali has quit IRC | 18:06 | |
*** martyturner has joined #openstack-cinder | 18:10 | |
*** esker has joined #openstack-cinder | 18:17 | |
*** esker has quit IRC | 18:19 | |
*** garthb has quit IRC | 18:19 | |
*** esker has joined #openstack-cinder | 18:21 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fix exception messages https://review.openstack.org/220132 | 18:25 |
*** agarciam has quit IRC | 18:26 | |
*** agarciam has joined #openstack-cinder | 18:26 | |
*** rushil has quit IRC | 18:29 | |
* hemna says the panic about volume migration sets in..... | 18:31 | |
*** harlowja has quit IRC | 18:31 | |
*** annegentle has quit IRC | 18:31 | |
Swanson | hemna: panic? | 18:31 |
patrickeast | panic?! | 18:32 |
* patrickeast panics | 18:32 | |
*** harlowja has joined #openstack-cinder | 18:32 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fix exception messages https://review.openstack.org/220132 | 18:32 |
hemna | yah. | 18:33 |
hemna | bleh | 18:33 |
hemna | re: https://review.openstack.org/#/c/219498/4/cinder/volume/drivers/san/hp/hp_3par_common.py | 18:33 |
e0ne | what did i miss? | 18:33 |
patrickeast | ew yea i saw that | 18:34 |
hemna | well, for whatever reason I didn't know about update_migrated_volume and the mess that not implementing that creates. | 18:34 |
hemna | :( | 18:34 |
*** martyturner has quit IRC | 18:34 | |
hemna | what a freaking disaster | 18:34 |
patrickeast | we should definitely have to documented somewhere if it isn't already | 18:35 |
patrickeast | s/to/that/ | 18:35 |
Swanson | hemna: I had to implement update_migrated_volume to keep from losing track of my volumes. Was horrible. | 18:35 |
hemna | I'm pretty sure our drivers are broken if someone migrates | 18:36 |
patrickeast | if i understand correctly i'm safe by using 'name' with mine... but now i really need to go test it more thoroughly... again | 18:36 |
hemna | we use id | 18:36 |
hemna | wah wah wahhhh | 18:37 |
patrickeast | haha now i'm glad you blocked my change to try and switch from name to id | 18:37 |
hemna | lo | 18:37 |
hemna | l | 18:37 |
Swanson | I was told to use id. I still use id. I just have update_migrated_volume to fix things up. | 18:37 |
* hemna fires up a new vagrant to test migrating from lvm -> 3PAR/LeftHand | 18:37 | |
*** esker has quit IRC | 18:38 | |
hemna | I can taste the exception stack trace now..... | 18:38 |
patrickeast | lol | 18:38 |
Swanson | Speaking of migration did live migration work in kilo? | 18:38 |
hemna | this also seems like it should be a test case handled in 3rd party CI | 18:39 |
hemna | nova live migration? | 18:39 |
*** esker has joined #openstack-cinder | 18:39 | |
hemna | heh, Swanson I'd say no* | 18:39 |
*** esker has quit IRC | 18:39 | |
hemna | for some drivers it might have. | 18:39 |
*** esker has joined #openstack-cinder | 18:40 | |
Swanson | I have a change in that fixes multiple calls to initialize_connection but I'd heard it was broken beyond that. But I might be thinking of something else. | 18:40 |
Swanson | Need my full test environment working again.... | 18:40 |
hemna | FC was probably borked with multipath enabled | 18:40 |
hemna | there is a bunch of code in nova that tries to preserve the multipath id | 18:41 |
hemna | it's a mess | 18:41 |
hemna | I have plans to clean that out in M, now that nova uses brick and brick doesn't even use multipath_id anymore. | 18:41 |
*** timcl1 has quit IRC | 18:41 | |
Swanson | I think the issue is with iscsi. Timezones are not helping me. | 18:42 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from delete methods https://review.openstack.org/205835 | 18:44 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 18:44 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Remove more API races https://review.openstack.org/216378 | 18:44 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move get_by_id to CinderObject https://review.openstack.org/218012 | 18:44 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve metadata update operations https://review.openstack.org/216376 | 18:44 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods https://review.openstack.org/216377 | 18:44 |
openstackgerrit | Patrick East proposed openstack/cinder: Use consolidated update for failover_replication https://review.openstack.org/219875 | 18:48 |
*** kvidvans has quit IRC | 18:52 | |
*** eharney has quit IRC | 18:52 | |
*** lpetrut has joined #openstack-cinder | 18:53 | |
*** jwcroppe_ is now known as jwcroppe | 18:58 | |
*** martyturner has joined #openstack-cinder | 18:59 | |
*** annegentle has joined #openstack-cinder | 19:01 | |
openstackgerrit | Gerald McBrearty proposed openstack/cinder: Switch SVC driver to use lsportfc to determine FC target WWPNS https://review.openstack.org/219973 | 19:06 |
*** eharney has joined #openstack-cinder | 19:07 | |
*** timcl has joined #openstack-cinder | 19:08 | |
*** agarciam has quit IRC | 19:12 | |
openstackgerrit | Derrick Wippler proposed openstack/cinder: Hacking log format arg check https://review.openstack.org/220246 | 19:13 |
diablo_rojo | jgriffith: Hey. Sorry I couldn't reply right away; I was in meetings. | 19:19 |
*** vgridnev has quit IRC | 19:21 | |
*** alexpilotti has quit IRC | 19:26 | |
*** lpetrut has quit IRC | 19:27 | |
*** _cjones_ has joined #openstack-cinder | 19:29 | |
*** lpetrut has joined #openstack-cinder | 19:30 | |
*** changbl has quit IRC | 19:30 | |
*** akerr has quit IRC | 19:31 | |
*** _cjones_ has quit IRC | 19:31 | |
diablo_rojo | jgriffith: I was confused by your pushing up a patch based heavily on Sergey's given your and the community's distaste with that approach. I pushed up my WIP patch in the hopes that we could work together on it and was surprised to see that you had pushed up your own today returning to the non-dynamic approach. I would love to collaborate with you particularly since you seem to have a good understanding of the oslo-config-generat | 19:31 |
*** _cjones_ has joined #openstack-cinder | 19:32 | |
jgriffith | diablo_rojo: couple things: | 19:32 |
jgriffith | My *distaste* doesn't really matter :) and my distaste was in reference to something slightly different. The reality is this is how it's being implemented currently. | 19:32 |
jgriffith | diablo_rojo: also, back in July I thought we'd have something done by Sept :) | 19:33 |
jgriffith | diablo_rojo: finally... it turns out that even the "auto generated" version seems to have a good bit of hard coding in it as well | 19:33 |
diablo_rojo | Right, but a dynamic way of implementing it is what the community wanted I thought | 19:33 |
jgriffith | diablo_rojo: Yeah... and I want a pony :) | 19:33 |
diablo_rojo | And no one had attempted it previously becuase it has been incredibly difficult | 19:34 |
jgriffith | diablo_rojo: my point is we need to fix it, if that's with your version, great | 19:34 |
diablo_rojo | Gotta give me a little credit with how new I am :) | 19:34 |
jgriffith | if it's with Sergey and my version (like every other openstack project) that's cool too | 19:34 |
jgriffith | or if it's step 1, then step 2 that's even beter | 19:34 |
jgriffith | diablo_rojo: OHHH | 19:34 |
jgriffith | don't take it the wrong way | 19:34 |
jgriffith | Not a knock on you or your work at all!!! | 19:35 |
jgriffith | That's not the point | 19:35 |
jgriffith | diablo_rojo: sorry if it came across that way, not my intent | 19:35 |
jgriffith | bottom line, it's a bug, it needs fixed... that's all | 19:35 |
diablo_rojo | If you and he are doing all the oslo-config-stuff right thats awesome. We can put my script to generate the cinder opts with what you have | 19:35 |
jgriffith | diablo_rojo: well... about that ;) | 19:35 |
jgriffith | diablo_rojo: a few things about your script we should look at | 19:36 |
*** lpetrut has quit IRC | 19:36 | |
jgriffith | diablo_rojo: so there's no real reason to do the "import xxxx as some-name", we can nuke that step out of there | 19:36 |
*** _cjones_ has quit IRC | 19:36 | |
jgriffith | diablo_rojo: I'm not sure I follow why a bunch of it is hard-coded? | 19:36 |
diablo_rojo | I was doing that so that later when we reference the opts there is a name to get them from. | 19:37 |
diablo_rojo | Which hard coded parts are you referring to? | 19:37 |
diablo_rojo | initial setup? | 19:37 |
jgriffith | diablo_rojo: https://review.openstack.org/#/c/219700/3/cinder/config/generate_cinder_opts.py | 19:37 |
jgriffith | kymgr, BRCD,CISCO BRCD.... | 19:38 |
jgriffith | fc-zone-manager | 19:38 |
jgriffith | etc | 19:38 |
diablo_rojo | Thats the different sections | 19:38 |
diablo_rojo | thats not specific opt lists being hard coded | 19:38 |
diablo_rojo | in Nova and other projects they have the lists of opts that have been registered broken up into sections | 19:38 |
jgriffith | yeah... ? | 19:39 |
jgriffith | and ? | 19:39 |
diablo_rojo | so like CONF.register_opts(lost_o_opts, group = keymgr) | 19:39 |
diablo_rojo | lost_o_opts shouldn't go into the default, it needs to get sorted into the proper section | 19:39 |
diablo_rojo | which is what that is | 19:39 |
jgriffith | diablo_rojo: yes, I'm familiar, not following though | 19:39 |
*** rmetcalf has joined #openstack-cinder | 19:40 | |
jgriffith | diablo_rojo: my point is "it's not an auto-generated" opts file if you hvae to hard code the sections in it :) | 19:40 |
diablo_rojo | You had said that I was hardcoding things? | 19:40 |
jgriffith | see what I mean? | 19:40 |
diablo_rojo | Oh | 19:40 |
jgriffith | :) | 19:40 |
jgriffith | diablo_rojo: so look... like I said... not knocking what you're doing | 19:40 |
jgriffith | diablo_rojo: more than happy to just leave it up to you | 19:40 |
jgriffith | diablo_rojo: if you want me to start reviewing what you have and making suggestions happy to do it | 19:41 |
jgriffith | diablo_rojo: just say the word | 19:41 |
jgriffith | but we do need this and db migration fixed... :) | 19:41 |
diablo_rojo | I get it. I can work on coming up with a way of doing that and push it up in another patch later on. I think the important thing right now is to get what you have and what I have together and merged. | 19:41 |
rmetcalf | I need some Cinder code reading advice. With REST APIs how do I determine the code path for a GET and PUT when the URL is the same (/v2/tenant/type, list or create volume types)? Thanks in advance. | 19:41 |
jgriffith | diablo_rojo: sure, whichever works for you. I'll leave it up to you :) | 19:42 |
diablo_rojo | jgriffith: How should we put those halves together? Or shall I continue on the route I am currently on? | 19:42 |
jgriffith | diablo_rojo: also... you might consider something like oswalk all the py files, and read in what you need and process it as you go | 19:42 |
jgriffith | diablo_rojo: we don't have to put them together right now | 19:43 |
diablo_rojo | Thats kinda what happens now. | 19:43 |
jgriffith | diablo_rojo: "kinda" being the key word there :) | 19:43 |
diablo_rojo | The generate_sample.sh greps for all the CONF.register_opts(xxxxx) | 19:43 |
diablo_rojo | and pipes that into my script | 19:43 |
jgriffith | diablo_rojo: right, what I'm suggesting is that instead you have just call "generate-opts" that just build all of that mess, don't mess with piping back and forth. You'll get way better performance | 19:44 |
jgriffith | diablo_rojo: and you'll get more reuse out of your tools that you're building | 19:44 |
jgriffith | diablo_rojo: but anyway... completely up to you | 19:45 |
*** jecarey has joined #openstack-cinder | 19:45 | |
jgriffith | I'll buzz off and leave you to what you're doing | 19:45 |
diablo_rojo | I had thought about that originally, it just made more sense to me to process all of that in a separate script | 19:45 |
jgriffith | diablo_rojo: cool! Then go with it :) | 19:45 |
diablo_rojo | jgriffith: Okay, works for me :) | 19:45 |
jgriffith | diablo_rojo: let me know when you get an update, and thanks! | 19:46 |
diablo_rojo | if you wanna make comments on what I have out there go ahead, I do appreciate the constuctive criticism. | 19:46 |
diablo_rojo | :) | 19:46 |
jgriffith | diablo_rojo: it is one of the trickier things to be trying to work on here I think | 19:46 |
diablo_rojo | Will do! | 19:46 |
diablo_rojo | Tell me about it ;) | 19:46 |
jgriffith | :) | 19:46 |
diablo_rojo | I didn't know the beast I was taking on when I started this. | 19:46 |
jgriffith | diablo_rojo: that's how we usually trick people into doing things around here ;) | 19:47 |
diablo_rojo | Clever, very very clever. | 19:47 |
*** sghanekar has quit IRC | 19:50 | |
*** eharney has quit IRC | 19:55 | |
*** eharney has joined #openstack-cinder | 20:00 | |
chutwig | DuncanT: here is my first draft of the cinder AZ stuff: https://gist.github.com/erhudy/71892f08bfcb823da4e1 | 20:04 |
*** chutwig is now known as erhud1 | 20:04 | |
*** erhud1 is now known as erhudy1 | 20:04 | |
erhudy1 | let me know if it's what you were looking for | 20:05 |
*** sghanekar has joined #openstack-cinder | 20:09 | |
*** pots has joined #openstack-cinder | 20:10 | |
*** dflorea has joined #openstack-cinder | 20:10 | |
*** pots has left #openstack-cinder | 20:11 | |
*** e0ne has quit IRC | 20:12 | |
*** vgridnev has joined #openstack-cinder | 20:14 | |
*** pots has joined #openstack-cinder | 20:15 | |
*** Yogi1 has quit IRC | 20:16 | |
*** julim has quit IRC | 20:18 | |
*** lpetrut has joined #openstack-cinder | 20:21 | |
*** akerr has joined #openstack-cinder | 20:21 | |
*** julim has joined #openstack-cinder | 20:21 | |
*** kvidvans has joined #openstack-cinder | 20:22 | |
*** ronis has quit IRC | 20:24 | |
*** e0ne has joined #openstack-cinder | 20:25 | |
*** jungleboyj has quit IRC | 20:25 | |
*** akerr has quit IRC | 20:26 | |
*** e0ne has quit IRC | 20:26 | |
*** eharney has quit IRC | 20:27 | |
*** tpsilva has quit IRC | 20:27 | |
*** IanGovett has quit IRC | 20:29 | |
*** julim has quit IRC | 20:33 | |
*** thangp has quit IRC | 20:33 | |
*** akerr has joined #openstack-cinder | 20:33 | |
*** lpetrut has quit IRC | 20:38 | |
*** eharney has joined #openstack-cinder | 20:42 | |
*** julim has joined #openstack-cinder | 20:42 | |
*** julim has quit IRC | 20:54 | |
*** akerr is now known as akerr_away | 20:59 | |
*** julim has joined #openstack-cinder | 21:02 | |
*** jwcroppe_ has joined #openstack-cinder | 21:03 | |
*** vgridnev has quit IRC | 21:05 | |
*** dustins has quit IRC | 21:05 | |
*** jwcroppe has quit IRC | 21:06 | |
*** raildo is now known as raildo-afk | 21:08 | |
*** mudassirlatif has quit IRC | 21:08 | |
*** mudassirlatif has joined #openstack-cinder | 21:10 | |
*** greghaynes has quit IRC | 21:10 | |
*** akerr_away has quit IRC | 21:16 | |
*** crose has quit IRC | 21:18 | |
*** gouthamr has quit IRC | 21:18 | |
*** jwcroppe_ has quit IRC | 21:21 | |
*** jwcroppe has joined #openstack-cinder | 21:22 | |
*** Yogi1 has joined #openstack-cinder | 21:25 | |
*** eharney has quit IRC | 21:26 | |
*** diablo_rojo has left #openstack-cinder | 21:27 | |
*** porrua has quit IRC | 21:27 | |
*** jwcroppe_ has joined #openstack-cinder | 21:29 | |
*** jwcroppe has quit IRC | 21:31 | |
*** IlyaG has joined #openstack-cinder | 21:36 | |
*** annegentle has quit IRC | 21:38 | |
*** merooney has quit IRC | 21:40 | |
*** niknakpaddywak has quit IRC | 21:42 | |
*** dflorea has quit IRC | 21:42 | |
*** jwcroppe has joined #openstack-cinder | 21:43 | |
*** rmetcalf has quit IRC | 21:45 | |
*** jwcroppe_ has quit IRC | 21:45 | |
*** jgregor has quit IRC | 21:45 | |
*** jwcroppe_ has joined #openstack-cinder | 21:48 | |
*** mriedem has quit IRC | 21:49 | |
*** jwcroppe has quit IRC | 21:50 | |
*** cbader has quit IRC | 21:50 | |
*** haomaiwa_ has joined #openstack-cinder | 21:54 | |
*** haomaiw__ has quit IRC | 21:56 | |
*** dflorea has joined #openstack-cinder | 21:56 | |
*** annegentle has joined #openstack-cinder | 21:57 | |
*** annegentle has quit IRC | 22:03 | |
*** mudassirlatif has quit IRC | 22:04 | |
*** sghanekar_ has joined #openstack-cinder | 22:04 | |
openstackgerrit | John Griffith proposed openstack/cinder: Add bug number to skip in VMware test https://review.openstack.org/220326 | 22:07 |
*** sghanekar has quit IRC | 22:07 | |
*** JoseMello has quit IRC | 22:09 | |
openstackgerrit | John Griffith proposed openstack/cinder: Add mechanism to update snapshot provider_id https://review.openstack.org/219864 | 22:10 |
*** sghanekar has joined #openstack-cinder | 22:11 | |
*** sghanekar_ has quit IRC | 22:13 | |
*** martyturner has quit IRC | 22:15 | |
*** edtubill has quit IRC | 22:17 | |
*** jwcroppe has joined #openstack-cinder | 22:20 | |
*** jwcroppe_ has quit IRC | 22:22 | |
*** Yogi1 has quit IRC | 22:23 | |
*** jwcroppe_ has joined #openstack-cinder | 22:24 | |
*** jwcroppe has quit IRC | 22:26 | |
*** jwcroppe has joined #openstack-cinder | 22:27 | |
*** jwcroppe_ has quit IRC | 22:29 | |
*** IlyaG has quit IRC | 22:30 | |
*** greghaynes has joined #openstack-cinder | 22:33 | |
DuncanT | erhudy1: So you if I'm reading that correctly, you want a single cinder AZ with one or more nova AZs? | 22:34 |
erhudy1 | yes, that's how we have things set up now | 22:35 |
*** sghanekar has quit IRC | 22:35 | |
erhudy1 | we don't want cinder to care because dealing with data availability is ceph's responsibility | 22:35 |
DuncanT | erhudy1: Thanks for the writeup. That config makes perfect sense and is definitely one I think we should support. | 22:36 |
erhudy1 | no problem | 22:37 |
jgriffith | DuncanT: FWIW, that used to be our default :) | 22:44 |
jgriffith | DuncanT: but some "do gooder" saw it as a bug and fixed it :( | 22:44 |
DuncanT | jgriffith: We can un-fix it again. Fortunately easier to do with code than dogs | 22:45 |
*** sghanekar has joined #openstack-cinder | 22:45 | |
jgriffith | erhudy1: DuncanT one place to consider this falling apart is people that want to actually do locality scheduling based on AZ's (they're out there). BTW, not disagreeing with the change | 22:45 |
jgriffith | just some background | 22:45 |
jgriffith | and the fact that it's configurable makes "most" people happy | 22:46 |
erhudy1 | i recognize that down the road as our clusters continue to expand we'll probably reach a point where a single crush map spanning the entire cluster no longer works | 22:47 |
*** adrianofr has quit IRC | 22:47 | |
erhudy1 | but i'd have to think about the implications of that and how we would try to deal with ceph in that scenario | 22:51 |
erhudy1 | and probably a lot of other assumptions we've made would break down before ceph did | 22:53 |
jgriffith | erhudy1: well... the other thing to consider is that Ceph isn't the only supported backend for Cinder ;) | 22:53 |
erhudy1 | i'm just ruminating | 22:53 |
*** edmondsw has quit IRC | 22:53 | |
erhudy1 | like a cow | 22:53 |
jgriffith | erhudy1: I do the same.. drives people nuts :) | 22:53 |
erhudy1 | or maybe a COW | 22:53 |
jgriffith | ha | 22:53 |
openstackgerrit | John Griffith proposed openstack/cinder: Add placholder for migration backports in Liberty https://review.openstack.org/219965 | 22:58 |
*** mriedem has joined #openstack-cinder | 22:58 | |
*** jecarey has quit IRC | 22:59 | |
*** mdenny has quit IRC | 23:05 | |
*** MIDENN_ has quit IRC | 23:05 | |
*** dflorea has quit IRC | 23:12 | |
*** markvoelker has joined #openstack-cinder | 23:13 | |
*** markvoelker has quit IRC | 23:17 | |
*** akerr has joined #openstack-cinder | 23:17 | |
*** tbarron is now known as tbarron_afk | 23:19 | |
*** dannywilson has quit IRC | 23:23 | |
*** markvoelker has joined #openstack-cinder | 23:25 | |
*** akerr is now known as akerr_away | 23:27 | |
*** annegentle has joined #openstack-cinder | 23:29 | |
*** dflorea_ has joined #openstack-cinder | 23:35 | |
*** akerr_away has quit IRC | 23:39 | |
*** dflorea_ has quit IRC | 23:41 | |
*** dims has quit IRC | 23:44 | |
*** hodos has quit IRC | 23:44 | |
*** dflorea has joined #openstack-cinder | 23:45 | |
*** dflorea has quit IRC | 23:48 | |
*** changbl has joined #openstack-cinder | 23:48 | |
*** pots1 has quit IRC | 23:52 | |
*** dims has joined #openstack-cinder | 23:53 | |
*** dflorea has joined #openstack-cinder | 23:57 | |
*** aix has joined #openstack-cinder | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!