| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-03 | |||
| 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 | |
| 19:32:50 | openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | |
| 19:33:53 | cfriesen | dansmith: so are we leaving races in there (due to parallel schedulers for example)? As I understand it we originally needed the re-verification because we didn't store the scheduler decision to the DB until after the request hit the compute node. | |
| 19:34:05 | mriedem | sdague: these old images actually seem pretty useful https://review.openstack.org/#/c/478470/ | |
| 19:34:10 | mriedem | i wonder why they were no longer used | |
| 19:34:42 | dansmith | cfriesen: when we track affinity in placement we won't need any of this because we'll be making the writing to the db as part of making the decison | |
| 19:34:43 | mriedem | although the might be inaccurate now | |
| 19:35:22 | cfriesen | dansmith: ah, okay. | |
| 19:36:16 | dansmith | gdi there's some gabbit failure now | |
| 19:36:20 | dansmith | which I suck at | |
| 19:38:40 | sdague | mriedem yeh, sfinucan and I went back and forth on that patch. I pushed for wanting to keep them, but they aren't used today, and it is more important that we get things merged | |
| 19:39:32 | mriedem | sdague: we also have https://docs.openstack.org/nova/latest/reference/vm-states.html | |
| 19:39:38 | mriedem | which is probably due for an audit also | |
| 19:39:50 | sdague | mriedem: many things are due for an audit | |
| 19:39:51 | mriedem | i'm trying to find the ironic doc that has all of their state machinery | |
| 19:40:14 | sdague | while I was fixing up the all misrendered block quotes I got a survery through what's there | |
| 19:40:33 | mriedem | why don't we have a search toolbar with the new docs theme? | |
| 19:40:34 | sdague | we need a pretty concrete discussion on information architecture at the PTG | |
| 19:40:41 | sdague | mriedem: I don't know | |
| 19:41:03 | mriedem | oh i found it | |
| 19:41:04 | mriedem | way at the bottom | |
| 19:42:37 | sdague | that's not search in our docs though, what is that? | |
| 19:43:52 | openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | |
| 19:43:53 | openstackgerrit | Dan Smith proposed openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | |
| 19:55:51 | mriedem | efried: what am i supposed to call powervm again? so as to not confuse it with powervm support via libvirt (phyp?) | |
| 19:56:33 | efried | mriedem PowerVM. That other thing doesn't exist. thorst_afk ? | |
| 19:57:09 | mriedem | PowerVMv2: the re-powering | |
| 19:58:35 | efried | mriedem Is there some context in which that other thing is actually present an causing confusion? | |
| 19:58:40 | efried | and* | |
| 20:00:06 | thorst_afk | It was my understanding that the libvirt code for powervm is not fully functional (I'll have to investigate a bit). I know there was some work on it several years ago, but I don't know that anyone has been maintaining or updating it for years. | |
| 20:00:34 | mriedem | no i was just going to add it to the list here https://review.openstack.org/#/c/480074/7 | |
| 20:00:40 | mriedem | and wasn't sure if there was a more specific name | |
| 20:00:44 | mriedem | that wasn't NEO or something | |
| 20:00:59 | mriedem | but we use PowerVM in the feature support matrix, so that makes sense to use here also | |
| 20:01:15 | thorst_afk | PowerVM is what we want to go with | |
| 20:01:42 | efried | In the words of Michael Bolton, "Why should I change? He's the one who sucks." | |
| 20:05:55 | mriedem | great, now i'm thinking about all of michael bolton's classic smokey throated wonder hits | |
| 20:06:09 | dansmith | mriedem: powervm != powerkvm if that's what you mean | |
| 20:06:23 | mriedem | dansmith: heh, i know | |
| 20:06:29 | dansmith | I don't think powervm via libvirt works from nova | |
| 20:06:29 | mriedem | powervm != powervc | |
| 20:06:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | |
| 20:06:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | |
| 20:06:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | |
| 20:06:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 20:06:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | |
| 20:06:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | |
| 20:06:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | |
| 20:06:42 | mriedem | the 3rd party ci says it's working | |
| 20:06:52 | dansmith | link? | |
| 20:10:01 | mriedem | http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/65/398265/34/check/tempest-dsvm-full-xenial/8144094/ | |
| 20:10:05 | mriedem | they even have a grenade job | |
| 20:10:11 | mriedem | http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/65/398265/34/check/grenade-dsvm-xenial/27db95e/ | |
| 20:10:26 | dansmith | that's pkvm == powerkvm right? | |
| 20:10:34 | mriedem | yeah | |
| 20:10:43 | mriedem | oh you said powervm via libvirt | |
| 20:10:50 | dansmith | yeah | |
| 20:10:55 | mriedem | yeah nvm then | |
| 20:10:58 | mmedvede | so, it is KVM on Power, not to be confused with PowerKVM OS | |
| 20:11:08 | mmedvede | naming is unfortunate | |
| 20:11:29 | mriedem | https://libvirt.org/drvphyp.html | |
| 20:11:47 | mriedem | is http://www-03.ibm.com/systems/power/software/virtualization/index.html the old powervm driver? | |
| 20:11:59 | mriedem | that ran ssh commands through the vios shell or whatever? | |
| 20:12:04 | mriedem | it's been awhile | |
| 20:12:10 | dansmith | mriedem: phyp via libvirt is the thing I doubt works with nova | |
| 20:12:14 | dansmith | mriedem: which is different than the powervm driver that was in nova at first | |
| 20:12:28 | dansmith | I think the phyp driver uses local hypercall access, IIRC | |