| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-03 | |||
| 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 | mriedem | powervm != powervc | |
| 20:06:29 | dansmith | I don't think powervm via libvirt works from nova | |
| 20:06:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | |
| 20:06:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | |
| 20:06:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | |
| 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:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | |
| 20:06:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | |
| 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 | |
| 20:13:02 | mriedem | hypercall? | |
| 20:13:06 | mriedem | that sounds enterprise ready | |
| 20:13:29 | dansmith | hypercal is like a syscall but to the hypervisor | |
| 20:13:31 | dansmith | *hypercall | |
| 20:14:38 | efried | http://www-03.ibm.com/systems/power/software/virtualization/index.html is the PowerVM that we're enabling via nova.virt.powervm | |
| 20:40:15 | openstackgerrit | Sean Dague proposed openstack/nova master: Add Contributor Guide section page https://review.openstack.org/490644 | |
| 20:45:04 | mriedem | ooo team meeting in 15 minutes | |
| 20:47:40 | sdague | mriedem: https://review.openstack.org/#/c/477488 - follow on comment about why it's fine to have it where it is | |
| 20:48:13 | melwitt | mriedem, dansmith: question on the cellsv2 doc https://review.openstack.org/490612 | |
| 20:48:49 | sdague | mriedem: basically to make the TOC not complete sidebar vomit, we're going to strip a lot out of it an the front page and build topic landing pages with deep links like - https://review.openstack.org/#/c/490644/ | |
| 20:49:23 | sdague | that lets us tidy things up without another giant page shuffle that we'd be 404 chasing | |
| 20:49:39 | mriedem | melwitt: replied | |
| 20:50:09 | openstackgerrit | Merged openstack/nova master: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | |
| 20:52:10 | dansmith | mriedem: do you remember the name of the affinity reporting conf item? I was going to add that in there for her, but I can never remember the name | |
| 20:52:21 | melwitt | mriedem: okay, so I guess that means there's nothing wrong with compute node reporting, it's just that the lag between the two (report vs affinity check) could cause a problem with parallel requests | |
| 20:52:46 | dansmith | melwitt: no | |
| 20:52:56 | dansmith | melwitt: there is a compute->scheduler reporting mechanism that can't happen either | |
| 20:53:09 | dansmith | track_instance_changes | |
| 20:53:18 | mriedem | yeah that one | |
| 20:53:19 | dansmith | he had it in his previous comment | |
| 20:53:21 | mriedem | i mentioned that eralier | |
| 20:53:24 | mriedem | *earlier even | |
| 20:53:37 | melwitt | okay, maybe I'm hard of reading :( | |
| 20:53:38 | mriedem | there are about 10 pieces to any single affinity filter puzzle | |
| 20:54:07 | mriedem | like, (1) server group (2) scheduler hint (3) filters (4) various config options | |
| 20:54:14 | mriedem | and then, maybe, it might work | |
| 20:54:18 | dansmith | (5) luck | |
| 20:55:13 | melwitt | yeah, it's just when I read "instance info reporting doesn't work" seemed sort of alarming. the second paragraph explains what the end point is, it's just after reading something like "compute node reporting doesn't work" seems sort of alarming, at least to me | |
| 20:55:18 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 20:55:30 | cfriesen | dansmith: /s/luck/timing | |