| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-08 | |||
| 12:40:19 | cdent | bauzas: well clearly we do have a problem because stuff is broken: we cannot write the allocations we want to write. we can either not do the doubling or we can break the way max_unit works, or we can just use a bigger fake driver. neither of the two first options are worth doing at this time | |
| 12:40:19 | jaypipes | bauzas: noted | |
| 12:40:21 | asettle | Also stephenfin I have another file you need to move with the admin guide patch | |
| 12:40:25 | bauzas | and then, I'll have my summer vacations starting next week for 2.5 weeks | |
| 12:40:34 | asettle | Oh wait, nvm, you've already done it! | |
| 12:40:46 | openstackgerrit | Sean Dague proposed openstack/nova master: Create For End Users index section https://review.openstack.org/491785 | |
| 12:41:01 | bauzas | jaypipes: dansmith: cdent: gibi: Monday 14th Aug. to 30th Aug. incl. | |
| 12:41:17 | bauzas | I'm telling you because it was a problem for June | |
| 12:41:23 | bauzas | knowing when I was around | |
| 12:42:48 | jaypipes | bauzas: k. thx for the heads up! | |
| 12:48:17 | stephenfin | jaypipes: Any chance I could get eyes on this today (before I head off for another week)? https://review.openstack.org/#/c/453808 | |
| 12:48:56 | stephenfin | Ignore the bp topic - that's just to link it. It's actually a prerequisite for that bp which I hope to work on next cycle | |
| 12:52:14 | jaypipes | stephenfin: done | |
| 12:52:36 | stephenfin | jaypipes: Shound | |
| 12:53:02 | jaypipes | stephenfin: is that a type of dog that hangs out in libraries? | |
| 12:53:49 | jaypipes | so has gibi come up with a bug of the day yet in placement? :) | |
| 12:54:03 | stephenfin | jaypipes: :D No, it's the rural Irish equivalent of "thank you" | |
| 12:54:17 | jaypipes | stephenfin: :) good to know! | |
| 12:54:49 | jaypipes | stephenfin: how is that pronounced? | |
| 12:55:37 | stephenfin | "Want to sound like a true Irishman? Use 'Sh' instead of 'S' for literally every word you encounter. Tashty shtuff" | |
| 12:56:54 | jaypipes | heh | |
| 13:00:31 | gibi | jaypipes: not yet. I'm making myself busy with the removal of change scheduler from the func test and the fix for the resize to too big flavor bug | |
| 13:00:45 | jaypipes | gibi: almost done with review on that | |
| 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. | |