| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-26 | |||
| 12:21:59 | alex_xu | currently, what I'm doing is rewrite the comments...and add more unittests | |
| 12:22:50 | artom_ | kaisers1, ping? | |
| 12:22:52 | alex_xu | efried: how about I write some todo note in the commit message, then you can continue those todo, or fix anything you found when review the code? | |
| 12:23:03 | efried | alex_xu Perfect. | |
| 12:23:04 | kaisers_ | artom_: pong Hi | |
| 12:24:11 | alex_xu | efried: cool, let us try how was that | |
| 12:25:08 | efried | alex_xu 很好 :) | |
| 12:25:35 | artom_ | kaisers_, hey, do you still run quobyte ci with dynamic_ownership = 0 in libvirt conf? | |
| 12:29:27 | efried | jaypipes These failures are weird http://logs.openstack.org/93/385693/58/check/openstack-tox-py27/786b250/testr_results.html.gz -- you on top of it or want me to debug? | |
| 12:29:37 | jaypipes | efried: on it already. | |
| 12:29:41 | efried | jaypipes coo | |
| 12:30:16 | kaisers_ | artom_: i think the setup did not change, yes. why? | |
| 12:31:03 | efried | jaypipes Oh, the headers. I was misreading those as being part of the endpoint filter. Phew, not crazy. | |
| 12:31:11 | artom_ | kaisers_, we've run into an interesting side effect of the workaround we put in place for that | |
| 12:31:48 | artom_ | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3033 specifically | |
| 12:32:14 | artom_ | If the compute node dies ungracefully, instances can't be started again because of permission denied on console.log | |
| 12:32:22 | artom_ | We can probably hack something around that | |
| 12:32:40 | artom_ | But I was kinda hoping the original reason for the original workaround had gone away | |
| 12:34:39 | kaisers_ | artom_: hmm, yeah, would be great to get rid of that whole thing. I think i've an open idea on how to improve on that but currently i'm not coming 'round to implement that, at least not in the next 5-8 weeks | |
| 12:35:11 | artom_ | kaisers_, does the idea depend on knowledge of quobyte? Otherwise I'm game :) | |
| 12:36:48 | kaisers_ | artom_: only marginally i think. I've to re-check my notes. I think it was centered around the idea to give a specific tag to libvirt which should ensure Quobyte is understood as a shared file system and libvirt should refrain from touching the files | |
| 12:36:59 | kaisers_ | artom_: but that's from clouded memory | |
| 12:37:03 | kaisers_ | pun not intended | |
| 12:37:23 | artom_ | kaisers_, ah, Dan Berrange's comment | |
| 12:37:26 | artom_ | I've read that | |
| 12:37:38 | artom_ | That wouldn't be in nova code though, correct? | |
| 12:37:49 | kaisers_ | artom_: yep, that's what i thought | |
| 12:37:57 | kaisers_ | artom_: i think it should be in nova | |
| 12:38:10 | kaisers_ | The libvirt xml would have to contain the tag iirc | |
| 12:38:27 | artom_ | kaisers_, ah? I'll try to read up on that | |
| 12:39:56 | kaisers_ | artom_: i think i started on that some time ago but had a hard time determining how exactly the tag has to look like | |
| 12:41:00 | artom_ | kaisers_, alright, I'll try and figure out what exact shape the fix should take | |
| 12:41:13 | artom_ | Are you usually around at this time? I might poke you for info :) | |
| 12:44:09 | kaisers_ | yep, usually i'm online around 07h - 15h UTC | |
| 12:46:18 | artom_ | kaisers_, gotcha, thanks! Right this instant I need to tend to daycare logistics, but I'll try to work on this some more this morning (I'm on NA east coast) | |
| 12:49:23 | kaisers_ | artom: Go for it! :) | |
| 12:54:40 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: properly decode error message from qemu guest agent https://review.openstack.org/511459 | |
| 12:55:59 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 12:57:05 | jaypipes | efried, gibi: ^ fixed unit test failrues | |
| 12:58:06 | efried | jaypipes +1 thanks | |
| 13:09:38 | gibi | jaypipes: thanks, I but back my vote | |
| 13:13:16 | openstackgerrit | Elod Illes proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514 | |
| 13:13:22 | mriedem | claudiub|2: lpetrut: looks like hyperv ci might have issues on stable/pike https://review.openstack.org/#/c/515330/ | |
| 13:13:58 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix AttributeError in BlockDeviceMapping.obj_load_attr https://review.openstack.org/515404 | |
| 13:14:25 | mriedem | sdague: bauzas: simple fix backport for a regression in pike https://review.openstack.org/#/c/515404/ | |
| 13:22:55 | openstackgerrit | Alex Xu proposed openstack/nova master: Enable _get_usages_by_provider_and_rc not filter on the resource class id https://review.openstack.org/514649 | |
| 13:22:55 | openstackgerrit | Alex Xu proposed openstack/nova master: Add _get_providers_with_any_resource method https://review.openstack.org/514197 | |
| 13:22:56 | openstackgerrit | Alex Xu proposed openstack/nova master: Add ProviderSummaryList object https://review.openstack.org/514198 | |
| 13:22:56 | openstackgerrit | Alex Xu proposed openstack/nova master: Add _get_traits_by_rp_ids method https://review.openstack.org/511184 | |
| 13:22:57 | openstackgerrit | Alex Xu proposed openstack/nova master: Add AggregatedProviderSummary and AggregatedProviderSummaryList objects https://review.openstack.org/514200 | |
| 13:22:57 | openstackgerrit | Alex Xu proposed openstack/nova master: Add more caches for the ProviderSummary and ProviderSummaryList https://review.openstack.org/514199 | |
| 13:22:58 | openstackgerrit | Alex Xu proposed openstack/nova master: Add as_allocation_request_obj method to AggregatedProvierSummary obj https://review.openstack.org/514651 | |
| 13:22:58 | openstackgerrit | Alex Xu proposed openstack/nova master: Add has_resources method to the AggregatedProviderSummary object https://review.openstack.org/514650 | |
| 13:22:59 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters https://review.openstack.org/479766 | |
| 13:23:53 | alex_xu | efried: jaypipes, all the functional tests passed ^ in the last patch, but it still is WIP, more unittests, and better commit message, and so on... | |
| 13:24:31 | efried | alex_xu Ack. Anything in particular in there that I could/should work on today? | |
| 13:25:11 | alex_xu | efried: I added todo note in the commit message https://review.openstack.org/#/c/514197/ | |
| 13:25:19 | alex_xu | I want to separate that single unittest to multiple one | |
| 13:25:29 | cdent | I like this todo in commit message handoff idea | |
| 13:25:48 | efried | alex_xu Okay, I'll see if I can tackle that today. | |
| 13:26:01 | jaypipes | alex_xu: yup, I'm getting to those reviews this morning. thanks for your patience. | |
| 13:26:35 | alex_xu | efried: and for other patch, few of them just missing the unittest, I didn't write the todo | |
| 13:26:43 | alex_xu | but you can find them easily | |
| 13:26:45 | alex_xu | jaypipes: thanks | |
| 13:27:09 | efried | alex_xu I think I mentioned this in a comment, but eventually I'd like to extend _get_providers_with_any_resource to be able to do either 'any' or 'all'. | |
| 13:27:21 | efried | alex_xu Because numbered groups are going to want *all*. | |
| 13:27:29 | efried | which is a much easier thing to write SQL for. | |
| 13:27:50 | efried | In fact, perhaps I can just write a separate method that does all resources *and* traits at the same time. | |
| 13:28:27 | alex_xu | efried: that what we doing currently, finally that will be huge sql, that is hard to maintain I think | |
| 13:29:08 | efried | alex_xu I think the "any" thing is what makes it really tricky. But I'll play with it and see if it winds up looking too heinous to do in one shot. | |
| 13:30:28 | alex_xu | efried: yea, it will be good you can take a look at the overall first, let us see which direction better | |
| 13:31:03 | efried | alex_xu Sure thing. I'll go over the updates today in any case. | |
| 13:35:23 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix the ocata config-reference URLs https://review.openstack.org/515410 | |
| 13:35:41 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Import the config drive docs from openstack-manuals https://review.openstack.org/515053 | |
| 13:35:51 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Import user-data page from openstack-manuals https://review.openstack.org/515054 | |
| 13:36:00 | lbragstad | gmann_afk: as far as i know, the policy deprecation work in oslo policy is just waiting on reviews | |
| 13:42:48 | mriedem | stvnoyes: did you ever get a chance to test the new cinder attach patches and try to attach the same volume to the same instance multiple times even if the volume wasn't multiattach=true? | |
| 13:43:53 | stvnoyes | no, I've been off on something else for the past week. I'll give it a try today | |
| 13:43:57 | edleafe | alex_xu: I think adding a TODO in a commit message for a WIP patch is very smart | |
| 13:44:27 | mriedem | stvnoyes: ok, alternatively i can just add something to a tempest test, push that up and make it depend on the nova api change | |
| 13:44:45 | stvnoyes | having it in tempest would be good to have | |
| 13:44:57 | mriedem | ok i'll put something up this morning | |
| 13:46:05 | openstackgerrit | Eric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151 | |
| 13:46:06 | cdent | edleafe, efried, alex_xu: are you familiar with the “park on a slope” concept, when experiencing task interruptions? | |
| 13:46:06 | openstackgerrit | Eric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223 | |
| 13:46:07 | efried | cdent I went ahead and made those couple of changes since I was editing the base patch anyway ^ | |
| 13:46:26 | efried | cdent Not by that name. Googling... | |
| 13:46:44 | edleafe | cdent: sorry, you just interrupted my task | |
| 13:46:45 | cdent | efried: cool, given other commitments today it is unlikely I would have got back to it and/or there’s not enough coffee in the world to make my brain work | |
| 13:46:52 | edleafe | cdent: no, not familiar | |
| 13:47:31 | efried | Turns out that's kind of a tough thing to google. | |
| 13:48:28 | cdent | it’s aligned with this idea of leaving a TODO in the wip commit. park your car in a way that if the battery is flat when you get back to it, you can still rolling start it | |
| 13:48:44 | cdent | or in general: make it easy, by leaving info, to rolling start | |
| 13:48:48 | cdent | stigmergy | |
| 13:48:50 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Implement query param schema for flavor index https://review.openstack.org/515413 | |
| 13:49:14 | cdent | efried: google stigmergy if you want something detailed to google | |
| 13:50:23 | efried | cdent Neat - the ants-following-scent-trails thing was a great way to crystallize the concept in my head. | |
| 13:50:53 | edleafe | efried: that's what we are, after all :) | |
| 13:51:15 | efried | cdent edleafe Not sure I would go so far as to say we "lack any memory, intelligence or even individual awareness of each other" though. | |
| 13:51:44 | edleafe | efried: so effin' literal! | |