| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-08 | |||
| 14:19:12 | gibi | mriedem, bauzas, dansmith: here is the bug for the cut https://bugs.launchpad.net/nova/+bug/1709328 | |
| 14:19:13 | openstack | Launchpad bug 1709328 in OpenStack Compute (nova) "the default of the scheduler.enabled_filters config is inconsistent" [Undecided,New] | |
| 14:19:47 | robcresswell | sdague: Yeah, that got ported over as part of the docs -> repos move in the past month. | |
| 14:19:55 | sdague | robcresswell: gotcha | |
| 14:20:34 | sdague | stephenfin: before I fix the gramar bits in the top patch, you want to go through - https://review.openstack.org/#/c/491785 as well? | |
| 14:20:40 | sdague | so I can get them in one go | |
| 14:21:06 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Add format_dom for PCI device addresses https://review.openstack.org/491822 | |
| 14:21:16 | bauzas | mriedem: dansmith: like I said to other folks there, I won't be there tomorrow only | |
| 14:21:39 | bauzas | mriedem: dansmith: plus being on vacations for 2.5 weeks starting on Monday till Wed (incl.) two weeks after | |
| 14:22:05 | stephenfin | sdague: Yup, I'm on that atm | |
| 14:23:01 | gibi | mriedem: FYI, I'm planning to skip the notification meeting today | |
| 14:23:28 | sdague | stephenfin: thank you | |
| 14:24:02 | dansmith | bauzas: like I said, probably easier if you just tell us when you _will_ be here | |
| 14:24:23 | dansmith | thu/fri of this week sounds like? | |
| 14:24:28 | bauzas | yup | |
| 14:24:43 | dansmith | okay, I'm out those days (first PTO all year) | |
| 14:24:46 | mriedem | gibi: ok | |
| 14:25:09 | bauzas | June was terrible to me, I don't want August to be the same | |
| 14:27:29 | stephenfin | sdague: done | |
| 14:29:43 | stephenfin | For reference, I'm off the rest of the week and most of next week. Not aware of anything hinging on me though, thankfully | |
| 14:29:52 | stephenfin | dansmith, sdague, bauzas, mriedem, alex_xu, melwitt, jaypipes, (anyone I might have missed): ^ | |
| 14:30:20 | jaypipes | stephenfin: cool. I'll assign a bunch of stuff to you on Friday. | |
| 14:31:06 | mriedem | nope. i'm out next week as well. | |
| 14:31:40 | stephenfin | jaypipes: Yay! It'll be just like being back in Intel | |
| 14:32:28 | jaypipes | hehe | |
| 14:33:34 | mriedem | jaypipes: where are we at with this? https://review.openstack.org/#/c/488510/ | |
| 14:33:55 | jaypipes | mriedem: just wrapping up local testing. should be pushed in <30 mins | |
| 14:33:57 | sdague | mriedem: cool, so am I also out the next 2 weeks as we do a family road trip into the eclipse zone | |
| 14:35:38 | stephenfin | sdague: Looks like this one's good to go, if it doesn't interfere with your stuff too much https://review.openstack.org/#/c/490952/ | |
| 14:41:23 | sdague | stephenfin: yeh, let me look | |
| 14:41:56 | sdague | stephenfin: +A | |
| 14:43:03 | openstackgerrit | Merged openstack/nova master: placement: filtering the resource provider id when delete trait association https://review.openstack.org/489205 | |
| 14:43:18 | mriedem | edleafe: re https://review.openstack.org/#/c/487925/ someone pointed out https://review.openstack.org/#/c/449587/ is basically the same change | |
| 14:43:29 | mriedem | i was -1 on that at the time but looks like for the wrong reason | |
| 14:44:09 | edleafe | mriedem: yeah, I mentioned that in one of my comments, and listed Lucas a co-author | |
| 14:45:06 | mriedem | ah ok | |
| 14:48:08 | stephenfin | asettle, sdague: I assume we can abandon this now as we've gone another direction? https://review.openstack.org/#/c/478485/ | |
| 14:48:12 | stephenfin | *I can | |
| 14:48:25 | cfriesen | if nobody else is working on gibi's bug above, I can do it. | |
| 14:48:29 | asettle | stephenfin: by all means | |
| 14:51:06 | gibi | cfriesen: I have no intention to start fixing it now. | |
| 14:53:10 | mriedem | stephenfin: did you see my question about migrating the hypervisor-specific config references, e.g. https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/config-reference/source/compute/hypervisor-hyper-v.rst ? | |
| 14:53:29 | mriedem | stephenfin: that was tipped off to me by jungleboyj in cinder b/c apparently some cinder docs for windows volumes references that | |
| 14:53:41 | mriedem | but i know the vmware config reference is quite detailed also | |
| 14:53:58 | mriedem | maybe we just report a bug that it's a gap? | |
| 14:54:06 | stephenfin | mriedem: Yeah, I haven't migrated any of the config-reference stuff at the moment | |
| 14:54:17 | mriedem | ok, i'll open a bug to track it | |
| 14:55:09 | stephenfin | Yeah, none of the info on the options themselves needs to be migrated because oslo.config does this for us automagically now | |
| 14:55:56 | mriedem | https://bugs.launchpad.net/nova/+bug/1709335 | |
| 14:55:57 | openstack | Launchpad bug 1709335 in OpenStack Compute (nova) "docs migration is missing hypervisor-specific configuration references" [Undecided,New] | |
| 14:56:12 | stephenfin | ...which is miles better than the openstack-doc-tools insanity that was required beforehand (as asettle can testify to) | |
| 14:56:15 | mriedem | yeah that's fine - these are more about actually setting up the compute host for a given virt backend | |
| 14:56:23 | asettle | stephenfin: yeah fuck mate don't bring up that shit | |
| 14:56:24 | stephenfin | mriedem: Gotcha. I'll do that in the next few hours | |
| 14:56:45 | mriedem | asettle: that sailor talk will have to be reserved for fleet week | |
| 14:56:57 | asettle | Soorrrryyy | |
| 14:57:02 | smcginnis | :D | |
| 14:59:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Extend nfv feature matrix with pinning/NUMA https://review.openstack.org/327126 | |
| 14:59:25 | mriedem | melwitt: are you planning on putting any notes about the location of the console auth service in a multi-cell layout in the docs? | |
| 15:05:32 | openstackgerrit | Sean Dague proposed openstack/nova master: Create For End Users index section https://review.openstack.org/491785 | |
| 15:05:32 | openstackgerrit | Sean Dague proposed openstack/nova master: Add For Operators section to front page https://review.openstack.org/491815 | |
| 15:05:33 | openstackgerrit | Sean Dague proposed openstack/nova master: rework index intro to describe nova https://review.openstack.org/491834 | |
| 15:06:02 | sdague | stephenfin: I think I got them all sorted, plus a final change to actually make the index intro fit it's new role | |
| 15:07:48 | stephenfin | sdague: Cool. I'll finish importing the config guide then take a gawk | |
| 15:08:11 | sdague | great | |
| 15:11:15 | openstackgerrit | Thomas Bechtold proposed openstack/nova master: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308 | |
| 15:12:08 | toabctl | mriedem, hey. could you have another look at https://review.openstack.org/#/c/398308/ ? you added your +2 already month ago. I rebased the patch and addressed 2 more comments... | |
| 15:13:06 | mriedem | toabctl: didn't jichenjc have a conflicting change for this? | |
| 15:13:37 | mriedem | yeah https://review.openstack.org/#/c/363585/ | |
| 15:15:14 | toabctl | mriedem, hm. need to check if theses are the the issues | |
| 15:16:31 | toabctl | mriedem, https://review.openstack.org/#/c/398308/ catches the exception "globally" | |
| 15:16:39 | toabctl | mriedem, see your own comment :) | |
| 15:18:23 | mriedem | i don't trust anything i said over 6 months ago | |
| 15:18:38 | dansmith | s/months/days for me | |
| 15:18:46 | toabctl | heh | |
| 15:18:52 | dansmith | and if it's about placement, then .. hours | |
| 15:36:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: no instance info cache update if instance deleted https://review.openstack.org/363585 | |
| 15:41:41 | cfriesen | anyone seen this error message (in the context of block live migration of boot-from-volume instance with a config drive): error : qemuMigrationPrecreateStorage:1714 : Operation not supported: pre-creation of storage targets for incremental storage migration is not supported | |
| 15:45:34 | sdague | ok, it's about 3 hour wait time for check nodes right now, so I'm going to walk away for a bit. Any review of those doc bits would be nice | |
| 15:50:41 | mriedem | dansmith: melwitt: do we want to try and get this into ocata? https://review.openstack.org/#/c/428481/ | |
| 15:50:45 | mriedem | *pike i mean | |
| 15:51:18 | stephenfin | mriedem, asettle: Is it just me, or does this not belong in the install guide? https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/config-reference/source/compute/hypervisor-hyper-v.rst | |
| 15:51:27 | dansmith | mriedem: didn't jaypipes' fix for the allocation thing I wrote effectively do this? | |
| 15:51:32 | stephenfin | (and all hypervisor-xxx docs) | |
| 15:51:38 | asettle | stephenfin: that belongs in config | |
| 15:51:41 | asettle | Why is it in the install?! | |
| 15:52:07 | mriedem | dansmith: no idea | |
| 15:52:39 | stephenfin | asettle: It seems like an install step to me? (Configuring compute nodes) | |
| 15:52:48 | stephenfin | It's currently in '/admin' | |
| 15:53:07 | stephenfin | Or, rather, I have it moved into '/admin' at the moment | |
| 15:53:09 | sdague | stephenfin: I think this is the issue of the docs which were feature oriented (the existing placement doc is the same thing) where concept, install, config were all in one doc | |
| 15:53:28 | dansmith | mriedem: https://review.openstack.org/#/c/490514/7/nova/scheduler/utils.py | |
| 15:53:46 | sdague | honestly, I would just say pick one for now, and all those are going to need to get broken up later to be more usage oriented | |
| 15:54:00 | mriedem | stephenfin: leave in config (/admin) for now | |
| 15:54:03 | dansmith | mriedem: the scheduler equivalent of that | |
| 15:54:20 | asettle | sdague does have a point. I would move to admin for now, and then we can figure it out | |
| 15:54:34 | stephenfin | mriedem, sdague, asettle: "He stays" (in admin) | |
| 15:54:36 | sdague | I also honestly think where things land on disk is not super important | |
| 15:54:38 | mriedem | dansmith: so do we run melwitt's functional test in https://review.openstack.org/#/c/428505/11/nova/tests/functional/test_boot_from_volume.py and see if the problem is gone? | |
| 15:54:52 | sdague | our consumers for the most part aren't browsing a local filesystem | |