*** gjayavelu has joined #openstack-placement | 00:34 | |
*** tetsuro has joined #openstack-placement | 00:41 | |
*** gjayavelu has quit IRC | 01:05 | |
*** yikun has joined #openstack-placement | 01:06 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299 | 02:05 |
---|---|---|
openstackgerrit | Merged openstack/nova master: Revert "Re-using the code of os brick cinder" https://review.openstack.org/576136 | 02:08 |
*** tssurya has joined #openstack-placement | 02:18 | |
*** tssurya has quit IRC | 02:22 | |
openstackgerrit | Merged openstack/nova master: Remove unnecessary parameters from create volume API https://review.openstack.org/574598 | 02:23 |
openstackgerrit | Merged openstack/nova master: Set scope for remaining placement policy rules https://review.openstack.org/571201 | 02:24 |
*** mriedem has quit IRC | 03:02 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Use parameter default_flavor. https://review.openstack.org/576318 | 03:05 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove useless parameter default_flavor. https://review.openstack.org/576318 | 03:09 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-extended-availability-zone https://review.openstack.org/576331 | 03:29 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-extended-status https://review.openstack.org/576340 | 03:55 |
*** tssurya has joined #openstack-placement | 03:59 | |
*** tssurya has quit IRC | 04:04 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344 | 04:05 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092 | 04:55 |
openstackgerrit | Dinesh Bhor proposed openstack/nova master: NUMACell, InstanceNUMACell: Adopt 'PCPU' changes https://review.openstack.org/576021 | 05:16 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-extended-availability-zone https://review.openstack.org/576331 | 05:39 |
*** tssurya has joined #openstack-placement | 05:43 | |
*** tssurya has quit IRC | 05:47 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-extended-status https://review.openstack.org/576340 | 05:55 |
*** gjayavelu has joined #openstack-placement | 05:56 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove useless parameter default_flavor https://review.openstack.org/576318 | 06:00 |
*** avolkov has joined #openstack-placement | 06:19 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove Chancefilter https://review.openstack.org/576384 | 06:38 |
*** tssurya has joined #openstack-placement | 06:45 | |
*** rubasov has joined #openstack-placement | 06:47 | |
openstackgerrit | garyk proposed openstack/nova master: Resource tracker: improve resource tracker periodic task https://review.openstack.org/576052 | 06:48 |
*** tssurya has quit IRC | 06:49 | |
openstackgerrit | garyk proposed openstack/nova master: Resource tracker: remove costly copy https://review.openstack.org/576099 | 06:50 |
*** tssurya has joined #openstack-placement | 07:00 | |
*** ttsiouts has joined #openstack-placement | 07:14 | |
*** ttsiouts has quit IRC | 07:20 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Optimize member_of check for nested providers https://review.openstack.org/573203 | 07:27 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 07:32 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-extended-status https://review.openstack.org/576340 | 07:36 |
*** tetsuro has quit IRC | 07:36 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-extended-status https://review.openstack.org/576340 | 07:45 |
*** tssurya_ has joined #openstack-placement | 07:47 | |
*** tssurya_ has quit IRC | 07:51 | |
*** ttsiouts has joined #openstack-placement | 08:02 | |
*** ttsiouts has quit IRC | 08:08 | |
*** ttsiouts has joined #openstack-placement | 08:09 | |
*** nicolasbock has joined #openstack-placement | 08:17 | |
gibi | efried: I left some question inline in https://review.openstack.org/#/c/572583 | 08:18 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092 | 08:20 |
*** nicolasbock has quit IRC | 08:24 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Make host_aggregate_map dictionary case-insensitive https://review.openstack.org/498334 | 08:29 |
*** gjayavelu has quit IRC | 08:29 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-extended-availability-zone https://review.openstack.org/576331 | 08:33 |
*** e0ne has joined #openstack-placement | 08:34 | |
*** cdent has joined #openstack-placement | 08:37 | |
*** ttsiouts has quit IRC | 08:39 | |
*** ttsiouts has joined #openstack-placement | 08:40 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case https://review.openstack.org/574240 | 08:42 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092 | 08:47 |
*** tssurya_ has joined #openstack-placement | 08:49 | |
*** tssurya_ has quit IRC | 08:53 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-config-drive https://review.openstack.org/576435 | 08:57 |
*** tetsuro has joined #openstack-placement | 09:01 | |
openstackgerrit | jichenjc proposed openstack/nova master: Mention server status in api-ref when rebuild https://review.openstack.org/576438 | 09:02 |
*** ttsiouts_ has joined #openstack-placement | 09:02 | |
*** ttsiouts has quit IRC | 09:02 | |
*** tetsuro_ has joined #openstack-placement | 09:06 | |
*** ttsiouts_ has quit IRC | 09:09 | |
*** ttsiouts has joined #openstack-placement | 09:12 | |
*** tetsur___ has joined #openstack-placement | 09:12 | |
*** tetsuro has quit IRC | 09:15 | |
*** ttsiouts has quit IRC | 09:15 | |
*** tetsuro_ has quit IRC | 09:16 | |
*** ttsiouts has joined #openstack-placement | 09:22 | |
*** tetsur___ is now known as tetsuro | 09:29 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832 | 09:42 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object https://review.openstack.org/573628 | 09:42 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375 | 09:42 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401 | 09:42 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166 | 09:42 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model https://review.openstack.org/571465 | 09:42 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534 | 09:42 |
openstackgerrit | jichenjc proposed openstack/nova master: Adjust log style and remove ocata support https://review.openstack.org/576462 | 09:43 |
*** tetsuro_ has joined #openstack-placement | 09:47 | |
*** tetsuro has quit IRC | 09:48 | |
*** ttsiouts has quit IRC | 09:48 | |
openstackgerrit | Merged openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583 | 09:54 |
*** tetsuro_ has quit IRC | 09:57 | |
openstackgerrit | Merged openstack/nova master: Only run placement request filters when Placement will be called https://review.openstack.org/569969 | 10:30 |
openstackgerrit | Merged openstack/nova master: libvirt: remove unused get_ovs_interfaceid() https://review.openstack.org/572699 | 10:53 |
*** takashin has left #openstack-placement | 11:03 | |
efried | ō/ | 11:39 |
*** edmondsw has joined #openstack-placement | 11:50 | |
openstackgerrit | Merged openstack/nova master: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/561284 | 12:01 |
openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 12:22 |
openstackgerrit | Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 12:22 |
*** mriedem has joined #openstack-placement | 12:53 | |
efried | edmondsw: I added comments on the vTPM spec https://review.openstack.org/#/c/571111/ going into a little more detail on how placement integration might work. Thanks for bringing that to my attention. | 13:01 |
edmondsw | efried tx | 13:01 |
cdent | efried: did you ever read my infinite resource classes blog post? | 13:08 |
efried | cdent: Not sure. Link? | 13:12 |
cdent | https://anticdent.org/simple-resource-provision.html | 13:12 |
edleafe | cdent: bad ideas never die, do they? | 13:16 |
edleafe | :-P | 13:16 |
* cdent shakes fist | 13:17 | |
cdent | that you are unable to conceive of the fundamental brilliance of that solution is some kind of sign | 13:17 |
* cdent is the only human here | 13:17 | |
alex_xu | efried: I feel strange for this test https://review.openstack.org/#/c/560317/9/nova/virt/fake.py, I'm thinking you are looking for test the real update_provider_tree method, right? | 13:20 |
efried | cdent: By "universally", does that include affordance in a database table for the concept of float('inf')? | 13:20 |
efried | alex_xu: stand by... | 13:20 |
cdent | efried: yeah, I checked that at the time | 13:20 |
efried | alex_xu: Yes, you're correct. The point is to make sure that the actual implementation of update_provider_tree in this patch does the right thing with respect to traits it does and does not own. | 13:22 |
alex_xu | efried: another strange thing is, the trait merge logic is in each virt driver update_provider_tree method, and each feature is responsible for merging traits. So that test won't be a generic merge trait test, it is cpu trait merge test. | 13:22 |
cdent | efried: the last paragraph in my response to jay in the comments is what it boils down to. Obviously I'm not suggesting we ever do this (now that we have what we have), but it is interesting with regards to things like TPMs | 13:23 |
efried | alex_xu: Correct | 13:23 |
efried | cdent: Yeah, agree. | 13:23 |
alex_xu | efried: ok, cool, thanks, I got what you need now :) | 13:23 |
efried | cdent: We could use the capability bit in the extra specs as the signal to the driver to make one of those things and attach it. But that would be a little weird. | 13:24 |
jaypipes | cdent: what spec are we discussing? | 13:24 |
cdent | jaypipes: https://review.openstack.org/#/c/571111/ | 13:24 |
*** avolkov has quit IRC | 13:24 | |
*** ttsiouts has joined #openstack-placement | 13:27 | |
jaypipes | cdent: sorry, I'm not following... you had said above "the last paragraph in my response to jay in the comments". am I the Jay you were referring to, and is https://review.openstack.org/#/c/571111/ the spec you were referring to? b/c I don't see any comments from you on that? | 13:32 |
* jaypipes wonders if he is indeed losing his mind... | 13:33 | |
cdent | jaypipes: we moved from that spec to me pointing out my old infinite resource providers blog post to efried, since he commented on the spec that we don't have a good way to represent unlimited resource classes | 13:34 |
jaypipes | ahhhh | 13:34 |
cdent | I was just trying to provide him with a bit of historical color | 13:34 |
jaypipes | gotcha, ok thanks :) | 13:34 |
jaypipes | sorry for thick-headedness. | 13:34 |
cdent | no need to apologize, it was not at all linear | 13:35 |
efried | In general I'm fine with representing qualitative attributes as traits rather than as infinite resources - but in this case we do have an effectively infinite resource that's an actual resource rather than a capability. | 13:35 |
jaypipes | cdent: infinitely curved? ;) | 13:35 |
cdent | irc on a moebius strip | 13:36 |
jaypipes | heh | 13:37 |
openstackgerrit | Dan Smith proposed openstack/nova stable/queens: [Stable Only] Add amd-ssbd and amd-no-ssb CPU flags https://review.openstack.org/576270 | 13:47 |
efried | jaypipes: SQL efficiency question for ya here https://review.openstack.org/#/c/573203/3/nova/api/openstack/placement/objects/resource_provider.py@782 | 13:55 |
efried | TL;DR: is it more efficient to duplicate the same `rp_id IN ($rp_ids)` within multiple INNER JOIN clauses, or do it once as a WHERE on the outside? | 13:56 |
openstackgerrit | Dan Smith proposed openstack/nova stable/queens: [Stable Only] Add amd-ssb and amd-no-ssb CPU flags https://review.openstack.org/576270 | 13:56 |
efried | I'm sure it depends on the number of JOIN clauses, the number of $rp_ids, the size of the database, etc etc. (Or maybe it doesn't - maybe the engine is smart enough to optimize either way.) | 13:57 |
*** ttsiouts has quit IRC | 14:00 | |
*** ttsiouts has joined #openstack-placement | 14:01 | |
openstackgerrit | Dan Smith proposed openstack/nova stable/queens: [Stable Only] Add amd-ssbd and amd-no-ssb CPU flags https://review.openstack.org/576270 | 14:01 |
openstackgerrit | Merged openstack/os-traits master: Adds HW_CPU_HYPERTHREADING standard trait https://review.openstack.org/576030 | 14:13 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1764883 https://review.openstack.org/562072 | 14:24 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Ensure pre-migrating instances are destroyed during init_host https://review.openstack.org/562284 | 14:24 |
openstack | bug 1764883 in OpenStack Compute (nova) "Evacuation fails if the source host returns while the migration is still in progress" [Undecided,In progress] https://launchpad.net/bugs/1764883 - Assigned to Lee Yarwood (lyarwood) | 14:24 |
*** e0ne has quit IRC | 14:27 | |
*** e0ne has joined #openstack-placement | 14:27 | |
jaypipes | efried: k, reviewed. | 14:29 |
efried | jaypipes: cool, thx | 14:29 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Ensure pre-migrating instances are destroyed during init_host https://review.openstack.org/562284 | 14:30 |
efried | jaypipes: Okay, so it's not about runtime efficiency, just about making the code prettier. | 14:31 |
jaypipes | efried: less complicated, yes. | 14:32 |
jaypipes | efried: runtime would be identical. | 14:33 |
efried | got it | 14:33 |
efried | I thought, if there was a runtime efficiency reason to put it in each join, that would be okay. | 14:33 |
jaypipes | efried: if the join was against a derived table that had a group by clause, it would be prudent to place the condition inside the derived table as well as the outer selection. but not in this case. | 14:34 |
efried | cool | 14:35 |
*** ttsiouts has quit IRC | 14:45 | |
*** ttsiouts has joined #openstack-placement | 14:46 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876 | 14:50 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Libvirt file backed memory https://review.openstack.org/575766 | 14:50 |
*** nicolasbock has joined #openstack-placement | 14:56 | |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove Chancefilter https://review.openstack.org/576384 | 14:58 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove Chancefilter https://review.openstack.org/576384 | 15:04 |
*** ttsiouts has quit IRC | 15:20 | |
*** ttsiouts has joined #openstack-placement | 15:20 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 15:31 |
openstackgerrit | huanhongda proposed openstack/nova stable/ocata: [Stable Only] Remove soft-deleted instances from quota_usages https://review.openstack.org/570653 | 15:35 |
*** e0ne has quit IRC | 15:44 | |
*** e0ne has joined #openstack-placement | 15:44 | |
*** belmoreira has quit IRC | 15:53 | |
*** ttsiouts has quit IRC | 16:02 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Configure multiple vGPU types https://review.openstack.org/564214 | 16:07 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: mdevs returning parent and vendor PCI info https://review.openstack.org/562304 | 16:07 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Get inventories per vGPU type https://review.openstack.org/564215 | 16:07 |
mriedem | efried: so i need to finally brush this off https://review.openstack.org/#/c/538498/ but i'm a bit scared from all of the upt changes since i haven't dug into those; is it generally just a matter of injecting the compute capabilities into the traits for the root provider in the tree? | 16:28 |
efried | mriedem: More or less. The 'more' part, though... | 16:28 |
efried | mriedem: Remember we have to know which traits we "own" and which we don't. | 16:29 |
mriedem | not sure what that means regarding this - i just need to merge the lists right? | 16:30 |
efried | mriedem: So you have to start by knowing the comprehensive set of all possible traits for this virt driver / compute service. Explicitly remove any that are unset, and explicitly set the ones that are set. | 16:30 |
mriedem | get the traits reported by the driver, and merge in the compute capabilities | 16:30 |
efried | mriedem: because you have to be able to clear one if it was set e.g. on the CLI. But you have to not touch it if it's one you don't own. | 16:31 |
efried | You have dansmith to thank for this one. | 16:31 |
mriedem | you mean clear one set on the CLI that the virt driver doesn't support now | 16:32 |
mriedem | and that one is 'owned' by the virt driver | 16:32 |
mriedem | so we can overwrite it | 16:32 |
efried | yeah | 16:32 |
mriedem | but not CUSTOM_TRAIT_FOO set by the user | 16:32 |
efried | right. | 16:32 |
efried | means it's not as simple as existing_traits | detected_compute_capability_traits | 16:33 |
mriedem | maybe i should find something else to do... | 16:34 |
openstackgerrit | Dan Smith proposed openstack/nova master: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576580 | 16:37 |
efried | mriedem: Want me to have a crack at it? | 16:40 |
mriedem | i'm rebasing and looking at it | 16:42 |
efried | mriedem: Assuming you don't want to use the new os-traits (https://review.openstack.org/546713), you already have the information you need because you're looping over capability, supported. | 16:43 |
mriedem | i plan on using the standard compute capabilities from os-traits | 16:44 |
efried | mriedem: Okay, then you're going to have to make a mapping from legacy names, or something? | 16:44 |
mriedem | yup | 16:44 |
efried | mriedem: Anyway, you're going to have to add an `else` to your `if supported`. The upshot is: `if supported: trait needs to be added to the whole; else: trait needs to be removed from the whole` | 16:45 |
mriedem | so, it looks like the libvirt driver's upt method doesn't set traits? | 16:47 |
mriedem | only inventory | 16:47 |
efried | Right. And libvirt doesn't have a get_traits method. | 16:48 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384 | 16:48 |
mriedem | ugh | 16:48 |
efried | Sokay, that's a good thing for you. | 16:48 |
mriedem | i was hoping to avoid redundant calls to get the virt driver defined traits | 16:48 |
mriedem | so if the libvirt driver implemented get_traits, i'd expect those to also just be in the update_provider_tree method | 16:49 |
mriedem | i expect the libvirt driver wouldn't implement get_traits now because it would just shove the traits into the tree in update_provider_tree | 16:49 |
mriedem | but if it does, i won't know which traits are defined by the virt driver and which are externally defined, if they are already in the tree... | 16:50 |
efried | mriedem: We haggled over that when upt was slotted into the rt. Originally I had it so upt and get_traits were mutually exclusive. | 16:50 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384 | 16:50 |
efried | At this point, only ironic implements get_traits. We should shut that down and remove it completely to simplify things. | 16:51 |
mriedem | i wonder if this would be easier if i just did the compute capabilities traits from the parent ComputeDriver.get_traits method | 16:51 |
mriedem | the reason i was doing this in the RT was to avoid having to copy this logic into all virt drivers' get_traits methods | 16:51 |
efried | mriedem: There's only one virt driver get_traits method: ironic. We should just cut ironic over to upt and get rid of get_traits entirely from the ComputeDriver interface. | 16:52 |
mriedem | if we do that, i still won't know which traits in the tree are defined by the driver and which are already set, maybe externally, on the provider right? | 16:53 |
efried | No, but that doesn't matter for your purposes | 16:53 |
efried | because you know the full set of compute capabilities traits that are possible. You explicitly rip out any that are unset, and explicitly set any that are set. Everything else you leave alone. | 16:54 |
efried | And btw, I'm saying you should do this after update_provider_tree, before update_from_provider_tree. | 16:54 |
efried | otherwise the virt driver may blow away your capability bits. | 16:54 |
mriedem | yeah that's where i put my todo | 16:54 |
mriedem | in rt | 16:54 |
efried | I'm going to play with cutting ironic over to update_provider_tree... | 16:55 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Convert ironic virt driver to update_provider_tree https://review.openstack.org/576588 | 17:04 |
efried | mriedem: ^ | 17:04 |
efried | jaypipes: ^ | 17:04 |
openstackgerrit | zhangyangyang proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384 | 17:05 |
openstackgerrit | Merged openstack/nova master: Fix MigrateData object tests for compat routines https://review.openstack.org/576256 | 17:06 |
*** tssurya has quit IRC | 17:06 | |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Convert ironic virt driver to update_provider_tree https://review.openstack.org/576588 | 17:08 |
efried | d'oh | 17:08 |
efried | Seems like git review used to not let you push when you had local changes still. | 17:08 |
*** gjayavelu has joined #openstack-placement | 17:15 | |
*** e0ne has quit IRC | 17:19 | |
openstackgerrit | Eric Fried proposed openstack/nova master: RT stats deepcopy 1x/node instead of 1x/instance https://review.openstack.org/576591 | 17:25 |
*** e0ne has joined #openstack-placement | 17:29 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova stable/queens: Trying to reproduce bug 1777540 https://review.openstack.org/576593 | 17:33 |
openstack | bug 1777540 in OpenStack Compute (nova) ""IndexError: list index out of range" in ExceptionPayload.from_exception during resize failure" [Undecided,New] https://launchpad.net/bugs/1777540 | 17:33 |
*** e0ne has quit IRC | 17:34 | |
openstackgerrit | Eric Fried proposed openstack/nova master: RT stats deepcopy 1x/node instead of 1x/instance https://review.openstack.org/576591 | 17:43 |
openstackgerrit | Matt Riedemann proposed openstack/os-traits master: Add COMPUTE_TRUSTED_CERTS trait https://review.openstack.org/576595 | 17:44 |
efried | jaypipes: You got https://review.openstack.org/#/c/559480/ today? | 17:49 |
jaypipes | efried: looking now. | 17:53 |
jaypipes | (sorry, still on PTO technically...) | 17:54 |
mriedem | efried: that series could go into the runways queue | 17:59 |
efried | mriedem: It could, I suppose, wouldn't hurt to have others look, but it's really me & jaypipes that need to review it. | 17:59 |
efried | save runways for stuff that's not getting attention otherwise. | 18:00 |
efried | mriedem: like zvm | 18:00 |
jaypipes | efried: done. | 18:02 |
efried | jaypipes: cool mon, thanks. I'm looking at the next one (the microversion) but I still don't think it's quite right. When are you "back"? | 18:05 |
jaypipes | efried: Monday, officially. | 18:08 |
efried | k | 18:08 |
*** e0ne has joined #openstack-placement | 18:31 | |
*** tssurya has joined #openstack-placement | 18:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Convert driver supported capabilities to compute node provider traits https://review.openstack.org/538498 | 18:52 |
mriedem | efried: WIP, but should be closer ^ | 18:52 |
openstackgerrit | Eric Fried proposed openstack/nova master: Squash: exclude nested alloc cands in older microversions https://review.openstack.org/576609 | 18:52 |
efried | mriedem: ack, will look. | 18:52 |
openstackgerrit | Eric Fried proposed openstack/nova master: Squash: exclude nested alloc cands in older microversions https://review.openstack.org/576609 | 18:55 |
efried | mriedem: Reviewed, looks right. I'm lazily waiting for zuul to tell me which tests I need to fix for the ironic thing, then I'll spruce that up. | 19:10 |
mriedem | ack thanks | 19:14 |
*** e0ne has quit IRC | 19:16 | |
*** e0ne has joined #openstack-placement | 19:17 | |
*** e0ne has quit IRC | 19:24 | |
cdent | efried, jaypipes, mriedem : I think we talked about environ and oslo_config for placement database connection. I've done some further experimentation and wrote up http://lists.openstack.org/pipermail/openstack-dev/2018-June/131631.html | 19:32 |
cdent | option 3 could be a goer with some pending work that the oslo team is working on | 19:33 |
efried | yuh, reading now. | 19:33 |
efried | cdent: Have you written a patch for 3 yet? | 19:36 |
efried | cdent: ...and/or what "pending work that the oslo team is working on"? | 19:36 |
cdent | follow the link to the irc discussion, to when dhellman joins in | 19:37 |
cdent | where he explains the concept of drivers | 19:37 |
cdent | and links to the spec thereof | 19:37 |
cdent | so, no, I haven't written a patch yet | 19:37 |
*** e0ne has joined #openstack-placement | 19:39 | |
openstackgerrit | Merged openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 19:45 |
*** cdent has quit IRC | 19:48 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: DNM: Use claim context during live migration https://review.openstack.org/576222 | 19:54 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix regression when listing build_requests with marker and ip filter https://review.openstack.org/576161 | 19:56 |
openstackgerrit | Merged openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092 | 20:18 |
*** e0ne has quit IRC | 20:46 | |
*** edmondsw has quit IRC | 20:47 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Convert ironic virt driver to update_provider_tree https://review.openstack.org/576588 | 21:05 |
efried | mriedem, jaypipes, jroll: ^ | 21:06 |
*** tssurya has quit IRC | 21:14 | |
mriedem | comments inline | 21:20 |
*** nicolasbock has quit IRC | 21:35 | |
mriedem | i could probably take a crack at implementing upt for the fake driver | 21:43 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't heal allocations for deleted servers https://review.openstack.org/575222 | 21:43 |
mriedem | if you want | 21:43 |
efried | mriedem: No, I'm responding, it's covered thoroughly in the preceding test class. | 21:46 |
efried | mriedem: ...with several different styles of implementation of upt, which you wouldn't (easily) get if you just implemented it in the fake driver. | 21:47 |
efried | mriedem: Responded. | 21:51 |
mriedem | +2 | 22:00 |
efried | thanks dood | 22:06 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 22:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged before starting live migration https://review.openstack.org/558001 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 22:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: Validate transport_url in nova-manage cell_v2 commands https://review.openstack.org/569858 | 22:29 |
efried | mriedem: This bp is approved now, yes? https://review.openstack.org/#/c/569649/ | 22:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle strings as booleans in nova.network.model.VIF utils https://review.openstack.org/576664 | 23:36 |
*** takashin has joined #openstack-placement | 23:48 | |
openstackgerrit | Merged openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 23:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!