| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-03 | |||
| 18:30:00 | edmondsw | :) | |
| 18:30:15 | mriedem | #"os_compute_api:os-server-groups:create": "rule:os_compute_api:os-server-groups" | |
| 18:30:26 | mriedem | #"os_compute_api:os-server-groups": "rule:admin_or_owner" | |
| 18:30:26 | mriedem | points to | |
| 18:30:30 | edmondsw | yep | |
| 18:30:34 | mriedem | my god | |
| 18:30:52 | mriedem | lbragstad: nvm | |
| 18:31:12 | edmondsw | so when we remove os_compute_api:os-server-groups we have to also, at the same time, update the defaults for the other rules to go straight to admin_or_owner instead of via the removed rule | |
| 18:33:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 18:33:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | |
| 18:33:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | |
| 18:33:21 | mriedem | yup | |
| 18:33:25 | mriedem | and we can do that in queens i guess | |
| 18:34:37 | openstackgerrit | Dan Smith proposed openstack/nova master: Add a caveat section about cellsv2 upcalls https://review.openstack.org/490612 | |
| 18:35:01 | mriedem | edmondsw: thanks for holding my hand | |
| 18:35:06 | mriedem | policy is a scary neighborhood | |
| 18:35:25 | mriedem | like TMNT lane that dan lives on | |
| 18:35:49 | edmondsw | mriedem np... policy will drive you nuts | |
| 18:37:27 | lbragstad | fact | |
| 18:40:08 | dansmith | mriedem: heh | |
| 18:40:40 | openstackgerrit | Jay Pipes proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | |
| 18:40:41 | openstackgerrit | Jay Pipes proposed openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | |
| 18:41:06 | jaypipes | dansmith: ok, so I'm still stumped on that ocata to pike ironic failure. everything else is passing. | |
| 18:41:23 | jaypipes | dansmith: hoping you might put some fresh eyes on that, since I need to run soon. | |
| 18:41:53 | mriedem | melwitt: you want to look at this additional cells v2 multi-cell doc goody? https://review.openstack.org/#/c/490612/ | |
| 18:41:57 | jaypipes | dansmith: I suspect it might have something to do with the new resources extra specs processing thing, but honestly I'm not positive. | |
| 18:42:10 | dansmith | jaypipes: yeah. should we slam in mriedem's single node patch at the bttom before you go? | |
| 18:42:30 | jaypipes | dansmith: I hadn't seen that, but I trust you whatever you want sure | |
| 18:42:33 | dansmith | oh wait | |
| 18:42:41 | dansmith | er, cancel that wait | |
| 18:42:57 | dansmith | jaypipes: this guy: https://review.openstack.org/#/c/490085/7 | |
| 18:42:57 | jaypipes | heh | |
| 18:43:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate '--version' parameters https://review.openstack.org/453808 | |
| 18:43:30 | jaypipes | dansmith: ah, yeah, totes. I didn't touch that guy | |
| 18:43:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate '--version' parameters https://review.openstack.org/453808 | |
| 18:43:44 | dansmith | yeah | |
| 18:43:45 | jaypipes | dansmith: +2 from me. | |
| 18:43:52 | dansmith | mriedem: you okay with me +2ing that even though I did the test bit? | |
| 18:45:22 | mriedem | for my change? | |
| 18:45:23 | mriedem | sure | |
| 18:45:28 | dansmith | yeah | |
| 18:45:30 | mriedem | it wasn't much - just removing TODOs | |
| 18:46:00 | mriedem | not to make your contribution to the change seem small or otherwise unimportant, of course :) | |
| 18:47:57 | dansmith | jaypipes: just unit tests I need to look at right? functional all work? | |
| 18:48:24 | jaypipes | dansmith: nope, all units should work fine. it's one functional test in tests/functional/compute/test_resource_tracker.py. | |
| 18:48:32 | dansmith | oh okay | |
| 18:48:55 | jaypipes | dansmith: it's expecting instance_claim() to have written allocation records for an ironic instance but it's not writing those allocations for some reason. | |
| 18:49:06 | dansmith | okay | |
| 18:49:14 | jaypipes | dansmith: and if I had to guess, maybe something to do with how we're pulling resources now with extra specs? | |
| 18:49:36 | dansmith | yeah found it | |
| 18:49:40 | dansmith | the test I mean | |
| 18:50:36 | prashkre | mriedem: Hi. Have you created a LP bug for issue with patch in https://review.openstack.org/#/c/391113/? | |
| 18:52:12 | mriedem | prashkre: i did but invalidated it once i understood how the policy rules were being linked | |
| 18:54:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add track_instance_changes note in disable_group_policy_check_upcall https://review.openstack.org/490627 | |
| 18:56:31 | prashkre | mriedem: I got confused with your query, now I got it why we had made it like that. due to compatablitiy with people who already started using that rule, it was made like that.. | |
| 18:58:28 | mriedem | melwitt: L35 in here about cell caches https://etherpad.openstack.org/p/nova-pike-cells-v2-todos - i think we have that covered in the FAQs page | |
| 18:58:35 | mriedem | but let me know if you were thinking of something else | |
| 19:08:07 | openstackgerrit | Merged openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | |
| 19:10:30 | openstackgerrit | Ed Leafe proposed openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954 | |
| 19:15:53 | mriedem | dansmith: ok i traced this ServerGroupAntiAffinityFilter test which is doing min_count=2 so multi-create, and the filter scheduler correctly puts the instances on different hosts | |
| 19:16:19 | mriedem | because of this https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L229 | |
| 19:16:29 | dansmith | mriedem: because one scheduling request for N instances maintains the state, but two parallel requests do not, right? | |
| 19:16:33 | mriedem | the first host chosen for the first instance is put in the request spec hosts list | |
| 19:16:46 | mriedem | yeah this is single request, | |
| 19:16:51 | mriedem | parallel would require the upcall safety check | |
| 19:16:56 | dansmith | right | |
| 19:16:59 | mriedem | single request, 2 instances, seems ok | |
| 19:17:02 | dansmith | I thought that's what I was tryin gto say | |
| 19:17:26 | mriedem | ah ok | |
| 19:17:49 | mriedem | so if you care about affinity/anti-affinity, the user has to do multi-create or stagger their create requests | |
| 19:18:03 | mriedem | i'd think most are doing multi-create in this kind of use case anyway... | |
| 19:18:08 | dansmith | well, if you are about it, presumably you want it to actually work, not just work in some circumstances | |
| 19:18:15 | dansmith | I dunno, maybe | |
| 19:18:36 | dansmith | if you're booting two nodes to be mirrors, then maybe | |
| 19:18:39 | mriedem | yeah if you don't care that much, you create the server group using the soft affinity/anti-affinity policy | |
| 19:19:22 | mriedem | if i'm creating 2 db VMs and want them on different hosts for protection, then i want it to work | |
| 19:19:59 | dansmith | right, which might be a num_instances=2 thing | |
| 19:20:08 | dansmith | but if you're bursting, | |
| 19:20:09 | cfriesen | mriedem: what about heat? | |
| 19:20:25 | dansmith | two things might be adding more webserver processes in parallel in response to load | |
| 19:20:26 | cfriesen | mriedem: we've had a bunch of issues due to heat issuing requests in quick succession | |
| 19:20:30 | dansmith | and those wouldn't be the single request | |
| 19:20:38 | mriedem | dansmith: yeah | |
| 19:21:14 | mriedem | sucks for the end user since the way this is going to behave is not up to them, it's dependent on the configuration of the cloude, | |
| 19:21:15 | mriedem | *cloud, | |
| 19:21:20 | mriedem | but in the beforetimes, | |
| 19:21:23 | mriedem | this could fail due to load anyway | |
| 19:21:26 | mriedem | or dropped messages | |
| 19:22:14 | dansmith | well, the late check will fail and error the instance if it can't verify | |
| 19:22:39 | cfriesen | dansmith: weren't we dropping the late check with cellsv2? or did that get added back? | |
| 19:22:50 | dansmith | cfriesen: that's what we're discussing | |
| 19:23:02 | mriedem | sweet my test passed http://logs.openstack.org/54/489754/6/experimental/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/a4fbbb3/console.html#_2017-08-03_18_04_43_399430 | |
| 19:23:29 | cfriesen | dansmith: for what it's worth, we're missing an equivalent late check in the live-migration path. we found a race there. | |
| 19:23:59 | mriedem | we need an equivalent to dan's "HA and Pray" slogan for live migration | |
| 19:24:40 | dansmith | live migration is always prayer-powered | |
| 19:25:39 | dfisher | i shouldn't have laughed that hard | |
| 19:28:12 | cfriesen | seems like to close these sched races we need something like "make the choice, store the choice in the DB, verify the choice was valid and retry if not". | |
| 19:28:16 | dansmith | jaypipes: you're gone right? | |
| 19:28:27 | dansmith | cfriesen: that's placement | |
| 19:32:45 | dansmith | mriedem: I think I've fixed jaypipes' thing | |