| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-08 | |||
| 13:04:31 | bhagyashris | jaypipes, melwitt, dansmitt: Hi, | |
| 13:05:02 | jaypipes | bhagyashris: we're still working on fixing bugs around shared resources. it doesn't look like Pike will include proper support for shared disk :( | |
| 13:05:03 | alex_xu | jaypipes: a easy one https://review.openstack.org/#/c/489205/ for a bug of traits api | |
| 13:05:51 | bhagyashris | jaypipes: ok. I just wanted to ask related to the bfv | |
| 13:06:29 | bhagyashris | I believe bfv case is handle at the placement side properly so Is it necessity of [1][2] | |
| 13:06:29 | bhagyashris | jaypipes, melwitt, dansmith: Why we are spending less attention to these patches[1][2]? [1]: https://review.openstack.org/#/c/428481/ [2]: https://review.openstack.org/#/c/428505 | |
| 13:06:35 | stephenfin | asettle: To which file do you refer "As I said earlier, only major qualm is changing it to Administration Guide rather than Administrators." https://review.openstack.org/#/c/490952/ | |
| 13:06:39 | stephenfin | ? | |
| 13:06:54 | asettle | stephenfin: no file there. I was talking about the build. I was going to pull it down, and edit the index file. | |
| 13:06:55 | stephenfin | I need to rework that to include some more changes, so I might as well change it here | |
| 13:06:58 | stephenfin | Ahgg | |
| 13:06:59 | stephenfin | *ahh | |
| 13:06:59 | asettle | But then I got confused, and edited sdague 's patch | |
| 13:07:09 | asettle | So, I fixed it, but in sdague 's patch, and not yours :) | |
| 13:07:13 | asettle | So, do nothing there :P | |
| 13:07:17 | asettle | I hope not | |
| 13:07:24 | stephenfin | ok then :) | |
| 13:08:29 | jaypipes | bhagyashris: we've been trying to increase functional test case coverage of resource accounting over the last couple weeks, focusing on move operations (resize, migrate, evacuate, etc). | |
| 13:08:50 | jaypipes | bhagyashris: I'd be keen to look at those patches for Pike, but they need a rebase. | |
| 13:09:04 | jaypipes | bhagyashris: are you interested in rebasing them? | |
| 13:12:43 | jaypipes | alex_xu: done | |
| 13:17:54 | alex_xu | jaypipes: thx | |
| 13:19:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add additional content to admin guide https://review.openstack.org/490952 | |
| 13:19:29 | stephenfin | cdent, sdague: Remind me - what's changing wrt ports in OpenStack services? https://review.openstack.org/#/c/491796/ | |
| 13:21:33 | vdrok | good morning everyone! | |
| 13:22:34 | cdent | stephenfin: the goals that has been expressed in devstack is that services will be on prefixes rather than ports, so that everything is either on port 80 or 443, depending the use of ssl, and you get service urls like http://exampe.com/compute http://example.com/image http://example.com/placeent | |
| 13:22:41 | vdrok | have a small question, is it ok to have <address type="pci" ...> for ceph volume in the instance xml, not <address type="drive"...>? in the <disk> section of course | |
| 13:22:47 | cdent | where example.com is likely a reverse proxy of some kind | |
| 13:23:14 | cdent | in my experience, packagers aren’t bought into this idea | |
| 13:23:19 | stephenfin | cdent: That sounds familiar alright. Do we need to include any references to that in that document? | |
| 13:23:52 | stephenfin | And, more importantly, I guess that document is still correct as it stands? | |
| 13:23:53 | jaypipes | mdbooth: perhaps you might know the answer to vdrok's question above? | |
| 13:24:21 | cdent | stephenfin: I don’t really feel like I’m in a position to answer that: every time I’ve tried to point out ports and such in the docs, it has ended up conflicting with what’s in debs and rpms and we’ve followed what’s in them | |
| 13:24:30 | vdrok | jaypipes: thx :) | |
| 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? | |