Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-08
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: Add For Operators section to front page https://review.openstack.org/491815
15:05:32 openstackgerrit Sean Dague proposed openstack/nova master: Create For End Users index section https://review.openstack.org/491785
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
15:54:56 openstackgerrit Merged openstack/nova master: Fix all >= 2 hit 404s https://review.openstack.org/491761
15:55:09 sdague they are coming in from the web, so it's just about how things end up on the glass with navigation patterns
15:55:18 stephenfin sdague: It kind of does because the filesystem manifests itself in the URL structure
15:55:43 sdague stephenfin: which is ignored by 99% of consumers
15:55:46 dansmith mriedem: I dunno, I think that test looks at a lot of RT stuff doesn't it?
15:55:56 stephenfin and I was pretty sure the plan was to still have central "Admin guide" and "Install guide" pages that deep link into each folder
15:56:02 stephenfin sdague: True
15:56:23 bauzas jaypipes: cdent: gibi: I clarified my opinion in https://review.openstack.org/#/c/491529/5/nova/tests/functional/integrated_helpers.py@75 about using all the filters
15:56:31 stephenfin Documentation is hard work.
15:56:35 sdague stephenfin: for sure
15:56:48 bauzas jaypipes: cdent: gibi: I'm okay with using all of them by default, just not in the same change, that's it
15:57:06 sdague stephenfin: it's possible we get there, there is so much near duplication of content from these different sources though, it's going to take a lot of massaging to figure it out
15:57:09 asettle stephenfin sdague - I'm going to print screen that you two are both agreeing that docs are hard work and cherish that forever
15:57:22 asettle Do you know how many times I've heard that docs aren't necessary and my job is worthless :D

Earlier   Later