| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-26 | |||
| 16:43:07 | jaypipes | edleafe: we've not allowed adding code that adds the microversion in one patch and modifies code that touches that microversion in a later patch. | |
| 16:43:15 | dansmith | right, exactly that | |
| 16:43:21 | sean-k-mooney | dansmith: do you know why we never versioned the base LiveMigrationData object. im fixing that in https://review.openstack.org/#/c/515423/1/nova/objects/migrate_data.py but not sure why it was not done in the first place when changes to the base require version bumps to all the childeren anyway. | |
| 16:43:27 | jaypipes | sorry, I guess I misread edleafe's comment. | |
| 16:43:31 | edleafe | jaypipes: sure, but that's not what I asked | |
| 16:43:36 | edleafe | sorta jinx | |
| 16:43:43 | dansmith | sean-k-mooney: because it's required to be implemented per virt driver, so it's not possible to use the base object | |
| 16:43:47 | dansmith | sean-k-mooney: so please don't "fix" that :) | |
| 16:44:03 | jaypipes | edleafe: ok. so is it clear now then? | |
| 16:44:22 | sean-k-mooney | dansmith: but that breaks the versioning of all child objects if we dont... | |
| 16:44:32 | dansmith | sean-k-mooney: breaks how? | |
| 16:44:33 | edleafe | jaypipes: yes; it seemed earlier you were stating the opposite | |
| 16:44:54 | dansmith | sean-k-mooney: adding something to the base object requires a bump in all the subclasses.. that's intentional, not broken :) | |
| 16:44:59 | jaypipes | edleafe: ok, not sure why/how I was unclear earlier but as long as I'm clear now, ok. | |
| 16:45:22 | dansmith | sean-k-mooney: it's like an abc. just a template for the real implementations and a common superclass parent | |
| 16:45:29 | dansmith | jaypipes: I was confused about what you said initially too, so.. | |
| 16:45:33 | sean-k-mooney | dansmith: yes it does but not versioning the base mean we have to copy the make compatible function to all the childern | |
| 16:46:23 | dansmith | sean-k-mooney: I'm not sure what or why you say that | |
| 16:46:37 | dansmith | oh I see what you mean | |
| 16:46:57 | dansmith | sean-k-mooney: then yeah, that's what it means | |
| 16:47:15 | dansmith | the versions are all different, so you have to handle the version each one took that new change in | |
| 16:47:19 | dansmith | registering doesn't help AFAIK | |
| 16:48:12 | mriedem | heh, wtf, this dict is already a primitive :) https://github.com/openstack/nova/blob/d36dcd52c24c32418fd358d245688c86664025d5/nova/scheduler/utils.py#L83 | |
| 16:48:13 | sean-k-mooney | ill be off until wednesday. but let me know if you would perfer me to duplicate the removal of the vifs into all the childeren or if the approch in the patch is ok | |
| 16:48:48 | dansmith | sean-k-mooney: if that's what you have to do then yeah | |
| 16:49:10 | dansmith | sean-k-mooney: you can unify a handler for that in the base class, and then just trigger it at the appropriate version for each child object or something | |
| 16:49:36 | sean-k-mooney | that what i basically did in https://review.openstack.org/#/c/515423/1/nova/objects/migrate_data.py | |
| 16:50:18 | dansmith | sean-k-mooney: ack, will comment | |
| 16:51:24 | sean-k-mooney | dansmith: i don't really mind which way its done i just taught this refactor would be cleaner but im happy to do it another way too. | |
| 16:51:34 | dansmith | okay | |
| 17:16:39 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: rbd: flatten images when unshelving https://review.openstack.org/457886 | |
| 17:39:58 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | |
| 17:39:58 | openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | |
| 17:39:59 | openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/510159 | |
| 17:39:59 | openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | |
| 17:40:00 | openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | |
| 17:40:29 | edleafe | mriedem: jaypipes: ^^ implemented SchedulerLimits for Selection | |
| 17:43:10 | gibi | mriedem: I've left my report about the notification subteam on the today's meeting agenda as I cannot participate. | |
| 17:43:28 | mriedem | gibi: ok, thanks | |
| 17:47:46 | mriedem | edleafe: thanks, soft -1 on the new comment that got added | |
| 17:47:54 | mriedem | since i'm working on making that comment partially untrue | |
| 17:53:54 | jaypipes | edleafe: cheers. will review shortly. | |
| 17:59:02 | edleafe | mriedem: well, since Jay wanted it recorded, I thought I'd include it, but I can easily remove it | |
| 17:59:13 | mriedem | jaypipes: i assumed he meant recorded in gerrit | |
| 17:59:33 | edleafe | mriedem: and Ill remove the mixin | |
| 18:00:02 | edleafe | Hmmm... when I'm researching code behavior, Gerrit isn't where I look | |
| 18:01:42 | mriedem | so leave a generic comment about us having to include limits in this thing because they are per-alternative host which isn't the same thing as limits in reqspec and filter_properties | |
| 18:01:42 | mriedem | idk | |
| 18:06:36 | edleafe | I removed it. It's not like adding it there will suddenly unlock the secrets of Nova to future generations of coders :) | |
| 18:07:25 | mriedem | right - i will still have to re-learn limits everytime i need to look at any of this | |
| 18:10:42 | cdent | jaypipes: since you were in that stack recently, you have time to blip in the spec tune up? https://review.openstack.org/#/c/509136/ | |
| 18:32:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use the RequestSpec when getting scheduler_hints in compute https://review.openstack.org/515461 | |
| 18:32:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass RequestSpec to ConductorTaskAPI.build_instances https://review.openstack.org/515495 | |
| 18:39:41 | cfriesen | repeat of earlier question since nobody answered it before...In _post_live_migration() at https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6075 why are we calling self._update_scheduler_instance_info() rather than self._delete_scheduler_instance_info() ? Seems logical that we would want to tell the scheduler "hey, this instance isn't here anymore". | |
| 18:40:17 | jaypipes | cdent: done | |
| 18:40:22 | cdent | thanks | |
| 18:42:26 | mriedem | cfriesen: it's likely wrong | |
| 18:42:39 | mriedem | cfriesen: i don't see post_live_migration_at_destination calling _update_scheduler_instance_info either | |
| 18:44:40 | lbragstad | mriedem: i'm reading some of nova's policy documentation - is this a typo? https://github.com/openstack/nova/blob/master/nova/policies/security_groups.py#L35 | |
| 18:44:48 | lbragstad | s/not/now/ / | |
| 18:45:54 | mriedem | lbragstad: no, | |
| 18:46:10 | mriedem | because you can still add/remove a security group to a server | |
| 18:46:17 | mriedem | via the addSecurityGroup API action on the server resource | |
| 18:46:35 | mriedem | https://developer.openstack.org/api-ref/compute/#add-security-group-to-a-server-addsecuritygroup-action | |
| 18:46:41 | lbragstad | mriedem: ok - cool, just double checking | |
| 18:46:53 | mriedem | we could probably deprecate that... | |
| 18:47:04 | mriedem | we deprecated things like addFixedIP and addFloatingIP in pike | |
| 18:47:14 | mriedem | because you can do those in neutron via the port associated with the instance | |
| 18:47:45 | mriedem | and all nova is doing is getting the ports for the instance and the security group and updating the port to use the secgroup | |
| 18:47:49 | mriedem | that's stuff you could do in neutron | |
| 18:48:08 | mriedem | i can't remember if alex_xu proposed removing the secgroup actions as well | |
| 18:48:48 | lbragstad | yeah - i was just double checking because as i was reading it i wasn't sure if it was suppose to be deprecated or not and was a type | |
| 18:48:51 | lbragstad | typo* | |
| 18:54:54 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | |
| 18:54:55 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Limit number of attempts to delete allocations https://review.openstack.org/507224 | |
| 18:54:55 | openstackgerrit | Chris Dent proposed openstack/nova master: Include project_id and user_id in AllocationList.get_all_by_consumer_id https://review.openstack.org/512420 | |
| 18:54:56 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | |
| 18:54:56 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | |
| 18:55:52 | cdent | jaypipes: ^ that ought to have it squashed and address the issues (I think) | |
| 18:56:08 | jaypipes | cdent: awesomesauce. thanks man. will review shortly. | |
| 19:01:40 | openstackgerrit | Merged openstack/nova-specs master: Fix issues for post-allocations spec https://review.openstack.org/509136 | |
| 19:17:07 | cdent | that’s the end of my day | |
| 19:22:43 | edleafe | mriedem: so tests get messy without the ComparableVersionedObject. Is there a strong objection to keeping it in Selection? | |
| 19:27:21 | efried | cfriesen Could it be because we don't want to delete allocations from a shared RP, at least in theory? | |
| 19:32:54 | smatzek | I'm seeing some VM spawn issues in the Trove gate and am wondering if anyone has seen similar behavior. Trove spins up VMs in the nova-compute in devstack as part of its CI. 10-20% of the time now I'm seeing the VMs get stuck in spawn. It looks like the VM is started in libvirt and then Nova hangs out in this timer and never leaves. https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2824 | |
| 19:56:08 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Add _get_providers_with_any_resource method https://review.openstack.org/514197 | |
| 19:56:08 | openstackgerrit | Eric Fried proposed openstack/nova master: Enable _get_usages_by_provider_and_rc not filter on the resource class id https://review.openstack.org/514649 | |
| 19:56:09 | openstackgerrit | Eric Fried proposed openstack/nova master: Add _get_traits_by_rp_ids method https://review.openstack.org/511184 | |
| 19:56:09 | openstackgerrit | Eric Fried proposed openstack/nova master: Add ProviderSummaryList object https://review.openstack.org/514198 | |
| 19:56:10 | openstackgerrit | Eric Fried proposed openstack/nova master: Add more caches for the ProviderSummary and ProviderSummaryList https://review.openstack.org/514199 | |
| 19:56:10 | openstackgerrit | Eric Fried proposed openstack/nova master: Add AggregatedProviderSummary and AggregatedProviderSummaryList objects https://review.openstack.org/514200 | |
| 19:56:11 | openstackgerrit | Eric Fried proposed openstack/nova master: Add has_resources method to the AggregatedProviderSummary object https://review.openstack.org/514650 | |
| 19:56:11 | openstackgerrit | Eric Fried proposed openstack/nova master: Add as_allocation_request_obj method to AggregatedProvierSummary obj https://review.openstack.org/514651 | |
| 19:56:12 | openstackgerrit | Eric Fried proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters https://review.openstack.org/479766 | |
| 19:56:32 | efried | alex_xu ^ I uncovered a bug, so I WIP'd the bottom change - see notes in commit message. | |
| 19:56:34 | mikal | jaypipes: Mr Finucane and I have a question for you on https://review.openstack.org/#/c/515196 when you have a moment. | |
| 19:58:06 | cfriesen | efried: mriedem: thanks, just wanted to make sure it wasn't something I missed | |
| 19:59:03 | bfife | kashyap: ping, bug#1706083. In Newton after an upgrade, I'm not seeing the proper cache value assigned to ephemeral ceph-backed storage on newly created instances. I've traced it to this commit | |
| 20:04:07 | jaypipes | mikal: done | |
| 20:06:50 | mikal | jaypipes: thanks man | |