| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-08 | |||
| 14:13:56 | robcresswell | Hopefully that means less headaches from now on :) | |
| 14:13:57 | stephenfin | sdague: Done. Because this is mostly new docs, I've started pulling on grammar stuff. Figure it's worth doing now rather than kicking the can | |
| 14:14:00 | sdague | robcresswell: ++ | |
| 14:14:06 | sdague | stephenfin: sure thing | |
| 14:14:13 | gibi | bauzas, jaypipes: anyhow I'm filing a bug that the default is not consistent. Either we need all three Core,Ram,DiskFilter or non of them | |
| 14:14:21 | mriedem | we should remove them | |
| 14:14:24 | mriedem | caching scheduler isn't the default | |
| 14:14:27 | mriedem | so that's not the reason | |
| 14:14:37 | mriedem | we just didn't remove ram/disk yet because nobody did it | |
| 14:14:56 | sdague | robcresswell: other question, when I was linking to horizon docs this morning, I was surprised that they didn't include screen shots | |
| 14:15:00 | mriedem | we also didn't do https://review.openstack.org/#/c/427200/ yet because the logic is complicated | |
| 14:15:09 | sdague | is that on your plan? | |
| 14:15:31 | bauzas | mriedem: agreed, we could cut that in Pike | |
| 14:15:39 | dansmith | mriedem: bauzas: agreed | |
| 14:15:43 | bauzas | mriedem: we didn't done that for Ocata because of the upgrade | |
| 14:15:53 | bauzas | want me to do so ? | |
| 14:16:05 | bauzas | with a devstack related patch | |
| 14:17:13 | sdague | mriedem: more 404 cleanups - https://review.openstack.org/#/c/491761 | |
| 14:18:08 | robcresswell | sdague: It's not something we've done before, since it was mostly dev / contributor docs. It's a sensible idea for the admin / user / operator content though. | |
| 14:18:51 | sdague | robcresswell: hmm ok, this didn't seem like dev docs - https://docs.openstack.org/horizon/latest/user/launch-instances.html | |
| 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/ | |