| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-08 | |||
| 13:53:39 | sdague | https://review.openstack.org/#/c/461762/ | |
| 13:55:20 | jsheeren | hi all, how are the filenames generated for the image_cache under /var/lib/nova/instances/_base ? is it a sha1 sum or md5sum of the glance image id or something? | |
| 13:55:54 | openstackgerrit | Sean Dague proposed openstack/nova master: Add For Operators section to front page https://review.openstack.org/491815 | |
| 13:56:20 | sdague | stephenfin: ok, see what you think of that bit | |
| 13:56:59 | stephenfin | also, I'd no idea nova-cert was gone. Good to know | |
| 13:57:23 | stephenfin | asettle: https://review.openstack.org/491815 | |
| 13:57:34 | stephenfin | another one for you too ;) | |
| 13:58:02 | asettle | Hooray | |
| 14:01:37 | cdent | jaypipes: if you’re in a cruising reviews kind of way, I’m pretty sure that we don’t want to do this so you might be able to quickly dismiss it: https://review.openstack.org/#/c/483506/ | |
| 14:02:22 | cdent | jaypipes: but we might want to do this: https://review.openstack.org/#/c/483460/ (they used to be the same patches by the contributor I suggested they split them) | |
| 14:02:36 | jaypipes | dansmith: yup. | |
| 14:03:12 | jaypipes | cdent: k, will look at them during next test run. | |
| 14:03:16 | dansmith | jaypipes: okay, is is the bottom patch that is giving you trouble or the top? If the top, then maybe you could push what you have for the bottom one so we can get it reviewed and maybe on the way? | |
| 14:04:15 | jaypipes | dansmith: bottom one, unfortunately. I added a patch before the confirm/resize patch that cleans up the _remove_deleted_instance_allocations() method of the RT and adds a bunch of debug logging. | |
| 14:04:27 | dansmith | okay | |
| 14:04:36 | jaypipes | dansmith: only two failures left. should be pushing shortly. | |
| 14:04:48 | dansmith | alright | |
| 14:08:28 | jsheeren | nevemrind i found it | |
| 14:09:01 | jsheeren | https://github.com/openstack/nova/blob/stable/newton/nova/virt/libvirt/imagecache.py#L53 | |
| 14:09:04 | jsheeren | quit | |
| 14:09:39 | jaypipes | lol | |
| 14:10:38 | gibi | jaypipes, bauzas, cdent: Why do we still have RamFilter and DiskFilter in the enabled_filters? https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L251-L252 | |
| 14:10:54 | jaypipes | gibi: I don't know. | |
| 14:11:26 | gibi | jaypipes: interestingly CoreFilter is not there | |
| 14:11:40 | jaypipes | gibi: I presume for caching scheduler. | |
| 14:11:50 | jaypipes | gibi: since it doesn't use placement | |
| 14:11:59 | gibi | does CoreFilter needed also for the caching? | |
| 14:12:19 | bauzas | jaypipes: gibi: because CachingScheduler indeed | |
| 14:12:44 | gibi | bauzas: then the missing CoreFilter is a bug I guess | |
| 14:12:48 | jaypipes | gibi: I would assume so, yes. | |
| 14:12:55 | bauzas | and also because when we used placement in Ocata, we agreed to not modify the nova.conf | |
| 14:13:03 | robcresswell | Re: Horizons magic edit flavor action, its now disabled-by-default, and the documentation for the setting has a big fat warning sign all over it. https://review.openstack.org/#/c/491442/ | |
| 14:13:03 | bauzas | now we're in Pike, we could just cut that | |
| 14:13:36 | bauzas | gibi: I know, but devstack runs it | |
| 14:13:39 | bauzas | gibi: crazy, I know | |
| 14:13:39 | cdent | robcresswell++ | |
| 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 | |