Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-25
07:34:40 openstackgerrit Lucian Petrut proposed openstack/nova master: Hyper-V + OVS: plug vifs before starting VMs https://review.openstack.org/585661
07:34:42 openstack Meeting ended Wed Jul 25 07:34:39 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
07:34:43 openstack Minutes: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-25-06.02.html
07:34:44 openstack Minutes (text): http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-25-06.02.txt
07:34:45 openstack Log: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-07-25-06.02.log.html
07:34:50 Kevin_Zheng gmann a test related question
07:34:59 gmann Kevin_Zheng: yup
07:35:12 Kevin_Zheng https://review.openstack.org/#/c/583927/ I'm working on this
07:35:39 Kevin_Zheng and Matt asked me to use a real mc backend instead of mock
07:36:01 Kevin_Zheng Seems like it need to have some set_up besides just override the configs
07:36:20 gmann checking
07:36:22 Kevin_Zheng I have no clue how do it, do you have this kind of experience/
07:36:24 Kevin_Zheng ?
07:36:34 Kevin_Zheng Thanks
07:39:17 gmann i have not done yet but trying to look
07:41:20 gmann Kevin_Zheng: it did not work with dogpile.cache.memory also ?
07:41:50 gibi mriedem: No worries, I didn't planned to have a meeting this week (either) http://lists.openstack.org/pipermail/openstack-dev/2018-July/132410.html
07:42:13 Kevin_Zheng yeah, seems the backend is not working, the client can be correctly initialized, but no data was saved
07:42:25 Kevin_Zheng so cannot retrive any data
07:50:37 openstackgerrit Lucian Petrut proposed openstack/nova master: Hyper-V + OVS: plug vifs before starting VMs https://review.openstack.org/585661
07:53:01 gmann Kevin_Zheng: i was thinking expiration_time is missing but is defaulted here - https://github.com/openstack/nova/blob/ddc807f85aad4e3d2ed6434809325320c1c21281/nova/cache_utils.py#L69
07:53:36 Kevin_Zheng hmm, yeah
07:54:08 Kevin_Zheng so you don't think we should do any setup action?
07:55:06 gmann Kevin_Zheng: means?
07:55:24 Kevin_Zheng I mean, do we need to setup the backend?
07:55:52 gmann Kevin_Zheng: i do not think so
07:56:01 Kevin_Zheng hmm ok
07:56:17 Kevin_Zheng Then I should check more on the params
07:56:22 gmann let me dig into that. need to go for firstcontact SIG now.
07:56:32 Kevin_Zheng NP, thanks anyway
07:56:36 gmann ll check later and let you know
07:56:53 Kevin_Zheng thanks
08:09:28 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: WIP: Enable nested allocation candidates in scheduler https://review.openstack.org/585672
08:15:14 tetsuro ^gibi, jaypipes, efried and everyone: I'd like to help you on consuming nrp-aloc-cands work in nova side, but I think I need brief feedback if this is a right way and where I should go.
08:17:21 tetsuro This is at least working on getting candidates with child rp + custom resource class inventory and have allocations on them with compute host resources.
08:17:37 tetsuro ...with a new instance
08:28:36 cdent tetsuro: I looked at that, looks like a good start and the right thing for there. The complexity is going to happen if/when an instance with nested wants to do a migration. There's quite a bit of a TODO in the report client about that kind of thing.
08:29:27 gmann tetsuro: we were discussing about your BP in api office hour - https://review.openstack.org/#/q/topic:bp/handling-down-cell+(status:open+OR+status:merged)
08:30:09 gmann tetsuro: what's the plan for this? is it still target for Rocky or stein. i saw the comment of doing service list also along with server list
08:30:31 gibi tetsuro: left some feedback in that patch. I think simply reading the consumer generation from placement during the claim_resources is defeate the purpose of having consumer generations
08:30:57 gibi tetsuro: I'm trying to add support for consumer generations here https://review.openstack.org/#/c/583667/
08:31:19 cdent tetsuro, gibi: for the case where we have a new instance the consumer is new, so generation is meaningless?
08:31:38 gibi cdent: unfortunately that is not the only case when claim_resources are called
08:32:00 cdent yeah, I know, I'm trying to work out if we can streamline the easy path
08:32:07 cdent because it is the common path too
08:32:41 cdent in the uncommon cases there will be existing allocations, yes?
08:32:58 cdent thus what the code is doing is at least close, if not fully correct
08:33:18 gibi cdent: plus when we assume that it is a new consumer we can still need to prepare that we get consumer generation conflict and handle that (most probably fail the build)
08:33:45 cdent clearly there needs to be some conflict detection, but bar that the logic is mostly right unless there's a detail I'm not aware of (which is very likely)
08:34:22 cdent yes, a generation conflict on a new build would mean that UUIDs are broken :)
08:34:49 gibi cdent: or that our assumption in the claim_resources about the fact that it is a new build is invalid
08:36:11 gibi cdent: even if we are coming from the scheduer select_destination it is possible that the consumer exists as evacuate does not use migration.uuid to hold the allocation of the source host
08:36:47 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.openstack.org/571410
08:37:17 gibi cdent: I agree that assuming consumer_generation = None in claim_resource a good first step
08:38:13 gibi cdent: I just fustrated about the hairy cases that I have to takle in my patch and I don't think we can merge support for 1.29 before solving 1.28 for the edge cases too
08:38:41 lyarwood mdbooth: ^ would you mind taking a look at that again?
08:38:48 cdent I'm not disputing that we need to merge (and understand) your 1.28 code first
08:39:18 cdent just, as I said, trying to make sure that we don't overcomplicate things when we claim, if possible
08:39:28 mdbooth lyarwood: yup
08:39:51 cdent if new: gen is none, if gen conflict bail, something went badly wrong
08:40:44 gibi cdent: when you say 'if new' do you mean checking if there is no other allocation in placement for the consumer?
08:41:02 cdent if existing allocations for this consumer uuid, do something else
08:41:27 cdent that would be one way, but I assume the caller of claim also knows?
08:42:32 cdent the "something else" looks to me to be very close to what is already being done in the code: merge the allocations, set the generation to what the server told us
08:42:36 gibi cdent: what if the caller thinks that it is not a new consumer but the claim_resources checks in placement and sees it is a new consumer and silently uses consumer_generation None instead of raise?
08:43:36 cdent then we should fix the caller because it sounds like the caller is broken? I'm not sure why we should overcomplicate the report client to deal with the fact that other code is flakey. What situation would cause what you're describing?
08:43:36 gibi cdent: I think in this case the caller needs to provider his view of the consumer generation and not let claim_resource decide
08:45:16 gibi cdent: sure it needs fix on the caller side but to know that the caller is broken we need to see the conflict and not simply use the consumer gen that will succeed in placement by asking it from placement
08:45:47 cdent I don't mean a fix at runtime, I mean a fix in the code
08:46:04 cdent Thus my question of "what situation.."
08:47:04 gibi cdent: I don't know exactly the case. I guess some parallel consumer update can happen. This is why we introduced consumer generation in the first place, isn't it?
08:47:10 cdent If we take this back to very basics there are two main situations: the consumer has existing allocations and this consumer does not have existing allocations. Are there others?
08:47:46 gibi there is the case when caller thing consumer gen is 1 but in the meantime somebody else modified the consumer
08:47:54 gibi s/thing/thinks/
08:48:31 cdent for "gen is 1 but somebody else modified" that is a specifc case of "the consumer has existing allocations"
08:48:57 gibi cdent: OK, then that is the two basic case
08:49:55 cdent From the point of the scheduler's management of allocations, consumer generations are not very important (we have to manage conflict detection, but we still just want to claim) because the act of claiming for an instance is a single task. We want to claim if at all possible. Nobody else is trying to find a destination for this instance.
08:50:36 gibi of course we can say that consumers handled by nova is owned by nova and no parallel update is done from nova on those and then we don't need to take consumers generations too seriously
08:50:58 cdent I'm saying that's true for in the scheduler. It's not true in the compute node, or in neutron agents, or whatever
08:51:11 cdent And we should keep it true in the scheudler if at all possible
08:52:05 gibi but scheduler coexists with all the others so I don't know how to ensure that the consumer the scheduler uses are not modified by other sources
08:52:40 gibi if scheduler would only ever create a new consumer during claim then it would be easier
08:52:51 cdent indeed
08:53:01 gibi due to evacuate it is not the case today
08:53:23 cdent but in the (presumably rare) cases where it doesn't the only real option we have is to get the existing allocations and do some kind of merge or choose to clobber
08:53:24 gibi there is an item in the PTG etherpad to 'fix' evacuate
08:54:01 cdent _move_operation_alloc_request is the code that is supposed to handle that sort of thing, but may not be mature enough of for nested and shared
08:55:45 cdent I think (or perhaps just hope) that we have a chance here to keep the happy path relatively smooth. I hope for that because we keep talking about how we want the claim stage to be as fast and unencumbered as possible.
08:55:46 gibi cdent: I agree that in case of non-forced evac when the scheduler claims on the target host we cannot do anything else just read the generation from placement. I will try to be really explicit in my patch about the different cases.
08:56:59 cdent tetsuro: you have anything to add to all that ^
08:57:42 gibi cdent: I share your hope. My bad feeling is about our assumption that I can identify the happy path unambigously
08:58:01 openstackgerrit Radoslav Gerganov proposed openstack/nova master: doc: add missing permission for the vCenter service account https://review.openstack.org/585683
08:58:40 cdent that we can't should be considered a bug ;)
09:00:15 gibi cdent: I guess I need to lower my expectations about my ability to cover every cases at once in my patch and let a bug born if it needs to be born
09:00:56 cdent yeah, I think that's going to be the case for a lot of this stuff because it's so hard to see clearly how things work
09:01:11 cdent And bugs are _good_. They give people a clear way to contribute.
09:02:03 gibi cdent: I thikn _these_ possible bugs will give a clear message for new contributors not to touch nova ;)
09:02:41 cdent I don't know, I think we misunderstand the mechanics of why people choose or choose not to contribute
09:02:59 cdent So often with nova if you want to contribute you have to spend 4 years understanding things
09:03:09 cdent there are few easy entries

Earlier   Later