| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-10 | |||
| 18:40:23 | mriedem | yeah, i finally know about the EMC system enabler https://review.openstack.org/#/c/492105/2/doc/source/figures/emc-enabler.png,unified | |
| 18:42:35 | mtreinish | cdent: looks like you're right the file is coming from mod_proxy | |
| 18:43:08 | mtreinish | I'll write up a patch to create an isolated virtual host for the glance apache endpoint and hopefully that should isolate the sendcl param | |
| 18:43:20 | openstackgerrit | Sean Dague proposed openstack/nova master: clean ups from I6815958b2533d462a2e5d27e7be57440d9f4f40a https://review.openstack.org/492645 | |
| 18:43:33 | sdague | mriedem / melwitt ^^^ ok, go to town ripping on that | |
| 18:43:43 | sdague | I'll iterate until we get it good enough | |
| 18:45:07 | melwitt | cool, thanks | |
| 18:55:22 | sdague | melwitt: on https://review.openstack.org/#/c/491853 realize that's just the import plus reference corrections to make it build | |
| 18:56:06 | melwitt | sdague: ah, I see that now. I thought the Cells v1 vs v2 warning was new but I see now that it's not | |
| 18:56:14 | sdague | yeh | |
| 18:56:46 | sdague | let us say... the existing docs are a mixed bag. It's probably better to have them in, but I think we're going to need a doc sprint early in the cycle to clean up a bunch of this | |
| 18:57:15 | melwitt | sdague: I think the only other thing not imported is the database config section that's not mentioned as skipped in the commit message. is that one supposed to be skipped? | |
| 18:57:39 | sdague | yeh, it turned into a really goofy thing | |
| 18:57:44 | melwitt | k | |
| 18:57:49 | sdague | let me pull up a couple of links | |
| 18:58:34 | sdague | the database doc in question was this - https://github.com/openstack/openstack-manuals/blob/6f9fc171800e8a435011f38cd4558e900884ce86/doc/config-reference/source/compute/database-connections.rst#L2 | |
| 18:59:04 | sdague | which besides being kind of inaccurate with the "any sqla" bit, it was also pretty vapid in what it said | |
| 18:59:37 | sdague | and was mostly a link to https://github.com/openstack/openstack-manuals/blob/6f9fc171800e8a435011f38cd4558e900884ce86/doc/config-reference/source/tables/common-database.rst#L11 | |
| 18:59:49 | melwitt | cool. just making sure it wasn't missed. I'm kind of just skimming this | |
| 18:59:56 | sdague | yeh | |
| 19:00:05 | sdague | the common tables things was kind of a bear | |
| 19:01:22 | sdague | steven spent a day or two fixing references, and I spent another day after him doing that. There was a lot more common normalization in the openstack-manuals repo that had to be denormalized for all of this. | |
| 19:03:03 | sdague | ok, afk for a bit | |
| 19:03:12 | mtreinish | efried_afk: https://review.openstack.org/#/c/492651/ | |
| 19:03:20 | mtreinish | sdague: ^^^ that should fix the glance thing | |
| 19:07:00 | mriedem | sdague: melwitt: ok several comments in the config reference one https://review.openstack.org/#/c/491853/6 | |
| 19:07:04 | mriedem | looks like some are duplicates of mel's | |
| 19:07:45 | melwitt | mriedem: yeah, turns out this is a straight forklift and I didn't realize that at first | |
| 19:07:54 | melwitt | so there's lots of latent issues in it | |
| 19:08:10 | mriedem | "The nova-conductor service is the only service that writes to the database." | |
| 19:08:10 | mriedem | heh, | |
| 19:08:20 | mriedem | "If nova-conductor is not used, entries to the database are mostly written by the nova-scheduler service, although all services must be able to update entries in the database." | |
| 19:08:27 | mriedem | ffs | |
| 19:08:30 | melwitt | noice | |
| 19:08:51 | mriedem | only conductor can write to the database, except everything else can and must be able to also, including your dog | |
| 19:08:55 | melwitt | o local conductor, I barely remember you | |
| 19:08:56 | mriedem | mr ruffles | |
| 19:09:21 | melwitt | heh | |
| 19:09:22 | mriedem | some of the other stuff i pointed out is just plain wrong, like "kvm is the default compute driver" | |
| 19:09:29 | mriedem | "to change compute drivers, configure the [libvirt] section" | |
| 19:09:41 | mriedem | thanks | |
| 19:09:41 | mriedem | hello libvirt, please to be giving me vmware | |
| 19:10:09 | melwitt | as you do | |
| 19:11:09 | mriedem | sdague: i think we should rename that cells.rst to cellsv1.rst at some point | |
| 19:11:14 | mriedem | since it's totally cellsv1 content | |
| 19:11:34 | mriedem | mikal: this is for you https://review.openstack.org/#/c/491853/6/doc/source/admin/configuration/samples/rootwrap.conf.rst@5 | |
| 19:19:23 | openstackgerrit | Ed Leafe proposed openstack/nova master: Deprecate bare metal filters https://review.openstack.org/492563 | |
| 19:20:26 | mnaser | jaypipes whenever you have a minute re https://review.openstack.org/#/c/491561/ -- i was thinking about this and realized that all_weighers will now include this which might mean that both ramweigher and rampercentageweigher would both be used (and maybe that might cause some weird scheduling and a change in default behaviour) .. any thoughts (more than welcome anyone else input too) | |
| 19:21:14 | jaypipes | mnaser: hmm... | |
| 19:21:26 | mriedem | ok the other rc1 things are approved, that just leaves https://review.openstack.org/#/c/487954/ | |
| 19:21:52 | jaypipes | mnaser: yeah, that's a good point. keep it in the separate file then... | |
| 19:22:08 | jaypipes | mnaser: mb add a note to the top saying "this is a separate file because..." | |
| 19:22:23 | mnaser | jaypipes i think that even if its a separate file it'll automatically get included with all_weighers :X | |
| 19:22:32 | mnaser | let me find the code for that again | |
| 19:22:43 | mnaser | if i remember right, its not hard coded, used some loadable stuff to autoload all of the stuff in the folder | |
| 19:23:37 | mnaser | https://github.com/openstack/nova/blob/master/nova/scheduler/weights/__init__.py#L46-L48 => https://github.com/openstack/nova/blob/master/nova/loadables.py#L78-L96 | |
| 19:24:30 | jaypipes | mnaser: ugh... | |
| 19:24:31 | mnaser | the reason this test didnt catch it because it doesnt check for an exact match but "if it contains" https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/weights/test_weights_hosts.py | |
| 19:24:37 | mnaser | yeah.. its not pretty | |
| 19:25:25 | jaypipes | mnaser: well, even if it's included in the list of filters, that doesn't necessarily mean it's enabled, right? | |
| 19:25:41 | mnaser | jaypipes this is weigher and afaik there is no enabled or disabled | |
| 19:26:42 | mnaser | jaypipes ouch.. "but wait, there's more" - https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L389-L394 | |
| 19:26:48 | mnaser | we couldnt even drop it from all_weighers lol | |
| 19:27:43 | mriedem | weighers are all enabled by default | |
| 19:27:57 | efried | mtreinish Sweet, trying... | |
| 19:28:11 | mriedem | mnaser: you can't set weight_classes ? | |
| 19:28:17 | mriedem | default=["nova.scheduler.weights.all_weighers"], | |
| 19:28:39 | mriedem | mnaser: you should be able to specify the specific list of weighers if you want to exclude some | |
| 19:28:46 | jaypipes | mnaser: yeah, you'd essentially have to list all the other weight classes, minus RamWeigher, and add in RamPercentageWeigher. ugh. | |
| 19:28:51 | mnaser | mriedem - i could, but the idea is when introducing this change -- https://review.openstack.org/#/c/491561/ -- that weigher will become included with the list of default all_weighers | |
| 19:29:05 | mnaser | i have them manually listed and it works fine for me right now | |
| 19:29:08 | mriedem | so you want an excluded_weighers option | |
| 19:29:27 | mriedem | how about, included_by_conditional_on_virt_backend_weighers? :) | |
| 19:29:33 | mriedem | s/by/but./ | |
| 19:29:36 | mnaser | i dont need it, i can manually list it, but if that change merges, the behaviour of all_weighers will change .. which is probably not okay? | |
| 19:29:53 | jaypipes | mriedem: no, he needs to change the behaviour of the existing RamWeigher if, say, a [scheduler]ram_weight_use_percentage configuration option is set... | |
| 19:30:16 | mriedem | ok, so you need a config option to change the behavior of the existing ram weigher, | |
| 19:30:19 | jaypipes | mnaser: right, it's not ok, since both Ram weighers will produce different weights. | |
| 19:30:20 | mriedem | and default it to behave the old way | |
| 19:30:25 | mnaser | thats a good idea | |
| 19:30:29 | jaypipes | yeah | |
| 19:30:30 | mnaser | i can do that | |
| 19:30:35 | mriedem | $5 please | |
| 19:30:37 | jaypipes | mnaser: thanks man. :) | |
| 19:30:56 | mnaser | jaypipes np | |
| 19:31:02 | mnaser | mriedem i'll start a tab | |
| 19:31:24 | mriedem | speaking of, | |
| 19:31:40 | mriedem | craiglist person is coming over in an hour to buy a $10 kids wagon, emailed me asking if i have change for a $20 | |
| 19:31:45 | mriedem | this will be fun | |
| 19:31:59 | mriedem | "i do, if you're ok with singles and $1 coins" | |
| 19:32:06 | mriedem | "here is a bag of nickels" | |
| 19:32:18 | melwitt | mriedem: anything else need review for rc1 that I can look at? | |
| 19:32:29 | mriedem | melwitt: hell yes you can https://review.openstack.org/#/c/487954/ | |
| 19:32:44 | melwitt | sweet | |
| 19:41:06 | sdague | mriedem: I agree. I honestly think we should probably rename all the cells v1 stuff to legacy cells actually | |
| 19:41:31 | sdague | especially when dealing with bugs, people don't really understand the v1 v2 thing | |
| 19:41:46 | sdague | legacy cells vs. cells would probably make things clearer | |
| 19:43:09 | sdague | mriedem: can we get ironic folks voting on https://review.openstack.org/#/c/487954 | |
| 19:44:25 | bauzas | do I need to chime in somewhere? | |
| 19:45:10 | sdague | bauzas: I think 487954 is the last patch required for the branch | |