| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-08 | |||
| 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 | stephenfin | cdent: That's fair. Always worth asking | |
| 13:25:26 | asettle | sdague: I am holding :) | |
| 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 | bauzas | now we're in Pike, we could just cut that | |
| 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:36 | bauzas | gibi: I know, but devstack runs it | |
| 14:13:39 | cdent | robcresswell++ | |
| 14:13:39 | bauzas | gibi: crazy, I know | |
| 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 | |