| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-08 | |||
| 13:24:42 | jaypipes | np | |
| 13:25:11 | sdague | asettle: I'm changing that index page a bit right now anyway based on the email I sent, so maybe hold a second | |
| 13:25:24 | bhagyashris | jaypipes: yeah I will do this. but I have one query about the patches, As bfv case is handled properly at placement side so Is there a need of these two patches ? By looking at the patches it looks like they are only updating compute_node table records which are not required. | |
| 13:25:26 | asettle | sdague: I am holding :) | |
| 13:25:26 | stephenfin | cdent: That's fair. Always worth asking | |
| 13:26:07 | sdague | once I get all these references building together | |
| 13:26:42 | stephenfin | asettle, sdague: Pushed up the latest version of the "add stuff we missed from the admin guide" patch to address mriedem's comments. That probably affects your work, sdague? https://review.openstack.org/490952 | |
| 13:27:49 | sdague | stephenfin: yeh, I think it will | |
| 13:35:31 | jaypipes | bhagyashris: I'm not sure... I need to revisit those patches. | |
| 13:36:49 | bhagyashris | jaypipes: ok. Thank you :) | |
| 13:38:10 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Remove host filter for _cleanup_running_deleted_instances periodic task https://review.openstack.org/491808 | |
| 13:39:20 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Remove host filter for _cleanup_running_deleted_instances periodic task https://review.openstack.org/491808 | |
| 13:40:35 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Remove host filter for _cleanup_running_deleted_instances() periodic task https://review.openstack.org/491808 | |
| 13:44:25 | sdague | stephenfin: while I'm chugging through toc references - https://review.openstack.org/#/c/491761 would be handy to get in to reduce 404s | |
| 13:48:49 | stephenfin | sdague: What happened to nova-cert? | |
| 13:48:56 | stephenfin | (just out of curiosity) | |
| 13:49:38 | dansmith | jaypipes: still crankin' on both patches yes? | |
| 13:52:59 | sdague | stephenfin: we deleted it | |
| 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/ | |