| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-13 | |||
| 23:56:16 | sean-k-mooney | melwitt: how would you feel about adding cpu pinning or hugepage traits? | |
| 23:56:39 | melwitt | I don't know :( I know very little about those | |
| 23:57:06 | sean-k-mooney | they dont exisats. yet... actully i have a better way to do what i want to do if i have thos | |
| 23:57:12 | sean-k-mooney | *those | |
| 23:57:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676290 | |
| 23:58:16 | sean-k-mooney | dpdk requried you to have hugepages to work with openstack | |
| 23:58:56 | sean-k-mooney | if we have a hugepage traits we could make it so your dpdk host would not allow vms without hugepages to be scheuled | |
| 23:59:22 | sean-k-mooney | for pinning we already have the dedicated cpus spec | |
| #openstack-nova - 2019-08-14 | |||
| 00:00:19 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676291 | |
| 00:00:24 | sean-k-mooney | i would proably do this slightly differently however. e.g. not requrie the operator to confgiure aggreates and do it like my image metadtata tratis changes | |
| 00:02:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/pike: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676292 | |
| 00:02:49 | sean-k-mooney | melwitt: i havent reviewd the code for this but i hope they are caching the aggreate metadata | |
| 00:04:21 | sean-k-mooney | the only why the prefilter could work would eb to retrive all of the nova host aggreates and there metadata then calulate the aggrates that wont suppor the instance and generate the &member_of=!in:<agg1>,<agg2>,<agg3> query paramater | |
| 00:05:02 | melwitt | sean-k-mooney: here's the code, it's close to merging so get in on it while you can https://review.opendev.org/#/q/topic:bp/placement-req-filter-forbidden-aggregates+(status:open+OR+status:merged) | |
| 00:07:33 | sean-k-mooney | ya so no caching... | |
| 00:07:52 | sean-k-mooney | the proablem with caching is keping the cache valid | |
| 00:11:05 | sean-k-mooney | melwitt: the schduler does not directly talk to the db right? i.e. does the schduler acess the db via the conductor like teh compute service? | |
| 00:11:35 | melwitt | sean-k-mooney: I think it talks directly | |
| 00:11:39 | melwitt | only the compute service does not | |
| 00:12:00 | sean-k-mooney | ok so all the "control plane" service talk directly but not comptues | |
| 00:13:25 | melwitt | I think so | |
| 00:13:58 | sean-k-mooney | ok i was not sure if it was only the api,metadata service and conductor that went direct. | |
| 00:14:29 | sean-k-mooney | i guess it makes sense for the schduler to be able to take a fast path to the db | |
| 00:41:46 | melwitt | sean-k-mooney: you can see which services go through conductor by looking for 'indirection_api' under nova/cmd | |
| 00:42:23 | sean-k-mooney | indirection_api is new to me | |
| 00:43:32 | melwitt | yeah, that's the magic for objects going through conductor | |
| 00:44:15 | sean-k-mooney | ah ok. | |
| 00:46:47 | sean-k-mooney | the way the prefilter is workign bugs me | |
| 00:47:15 | sean-k-mooney | i need to re read the spec but i dont think it workign the way i expect it too | |
| 00:47:21 | sean-k-mooney | it may be equivalent | |
| 00:48:44 | sean-k-mooney | if the image has no trait request i think it would allow all aggreates | |
| 00:49:10 | sean-k-mooney | and i think you would have to list all traits in the imgae/flavor on the aggreate for it to match. | |
| 00:49:38 | sean-k-mooney | i proably should be less tired when reviewing this but this seams reversed to me | |
| 00:52:44 | melwitt | yeah, I think it will get all the aggregates, add any without the trait to the "no" bin, and then pass the "no" bin to placement and say "these are the forbidden aggregates, don't return hosts in them" | |
| 00:53:02 | sean-k-mooney | i think the logic is inverted. | |
| 00:53:13 | melwitt | which does seem backward but I assume it's the only way to do it | |
| 00:53:26 | sean-k-mooney | its treating the image and flaovr as the attoritive source nto the metadata | |
| 00:53:40 | sean-k-mooney | this is broken in the same way the existing filter is | |
| 00:53:58 | sean-k-mooney | and it the different betten the out of tree one and the in tree one | |
| 00:55:01 | sean-k-mooney | for this prefilter to work every trait that is in the image+flavor would have to be on the host_aggreate metadata | |
| 00:55:02 | melwitt | it does specifically say (in the proposed doc change) that image/flavor need not be set and will still not land on hosts in aggregates with =required. I just don't know how that's done | |
| 00:55:34 | sean-k-mooney | its done by invertin the relationship | |
| 00:56:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676295 | |
| 00:56:14 | sean-k-mooney | so what you need to do is get the metadata for all host_aggreates. then for each aggreate you check all the keys are present itn the flvore/image requst | |
| 00:56:32 | sean-k-mooney | if a key is set on the aggreate but not in the flaor/imgae then you add that to the forbindin list | |
| 00:56:34 | melwitt | oh ok | |
| 00:56:38 | melwitt | yeah | |
| 00:56:56 | sean-k-mooney | again exactly what the out of tree filter does :) | |
| 00:57:09 | sean-k-mooney | ok ill comment on the review | |
| 00:58:03 | sean-k-mooney | as implement this would work exactly the same as the aggregate_instance_extra_specs filter | |
| 00:58:24 | sean-k-mooney | well it sort of | |
| 00:58:41 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676295 | |
| 00:58:41 | sean-k-mooney | it use traits instead of any proerty and also looks at the image | |
| 00:58:47 | sean-k-mooney | but conceptuly its the same | |
| 01:01:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Fix misuse of nova.objects.base.obj_equal_prims https://review.opendev.org/676295 | |
| 01:04:05 | sean-k-mooney | melwitt: actully it might be correct https://review.opendev.org/#/c/671074/6/nova/objects/aggregate.py@476 | |
| 01:04:58 | sean-k-mooney | in need to figure out what sql that generates | |
| 01:10:15 | melwitt | sean-k-mooney: note the ~ (not) squiggle | |
| 01:11:47 | sean-k-mooney | ya looking at the doc sting i think its correct after all | |
| 01:12:16 | melwitt | phew good | |
| 01:12:49 | sean-k-mooney | so it selecting the aggrreate if the aggreate metadta key is not in the set passed in | |
| 01:16:34 | openstackgerrit | ZHOU YAO proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/621646 | |
| 01:32:32 | sean-k-mooney | ok im goign to log off for the night o/ | |
| 01:43:44 | sean-k-mooney | oh infra change how logs are renderd in the gate. | |
| 01:44:12 | sean-k-mooney | i dont think i like it just becasue its different the it has been for 6+ years | |
| 01:47:51 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix collapse of 'host_status' description https://review.opendev.org/676301 | |
| 02:23:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Refactor some methods for reuse by Cyborg-related code. https://review.opendev.org/673734 | |
| 02:23:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242 | |
| 02:23:54 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 02:23:54 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | |
| 02:23:55 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: WIP: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 02:23:55 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245 | |
| 02:44:00 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 03:06:03 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963 | |
| 03:06:52 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969 | |
| 03:08:59 | boxiang | hi gibi , I have finished the regression functional test for this https://review.opendev.org/649963 | |
| 04:05:31 | openstackgerrit | Merged openstack/nova master: Execute TargetDBSetupTask https://review.opendev.org/633853 | |
| 05:49:21 | openstackgerrit | ZHOU YAO proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/621646 | |
| 06:22:31 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 07:27:11 | openstackgerrit | Merged openstack/nova master: api-ref: Fix collapse of 'host_status' description https://review.opendev.org/676301 | |
| 09:55:39 | openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626 | |
| 10:05:57 | ut2k3 | Hi guys, I have unfortunately two Volumes stock in "detaching" - Is there any chance I can force detach these from the instances? `nova volume-detach...` as well reseting once their state with `cinder reset-state...` did not help | |
| 10:56:11 | openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | |
| 10:58:32 | stephenfin | efried: I'm kind of stuck on the reshape for VCPU -> PCPU. I think what we're doing at the moment is wrong, and the tests are hiding that fact https://review.opendev.org/#/c/674895/7/nova/virt/libvirt/driver.py@6891 | |
| 10:59:39 | openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | |
| 11:00:33 | stephenfin | efried: What I think we need to do is a multi-step process, (a) figure out if any instance on the host is using pinned instances by looking at the (host) NUMATopology.pinned_cpus attribute, if so (b) figure out if there are any PCPU allocations against the host, if not (c) figure out how many VCPUs to migrate to PCPUs and do that | |
| 11:01:52 | stephenfin | efried: But I can't do (b) easily since the pattern we have for reshaping is to only provide allocations to 'update_provider_tree' if there's a 'ReshapeNeeded' exception raised, so I'm stuck | |
| 11:04:37 | openstackgerrit | YAMAMOTO Takashi proposed openstack/nova master: WIP: midonet doesn't have plug-time vif events https://review.opendev.org/676388 | |
| 11:23:17 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Testing tls with ipv6 also https://review.opendev.org/676391 | |
| 11:48:20 | openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275 | |
| 11:58:34 | stephenfin | alex_xu, gibi: More than happy to let you handle the update :) | |
| 11:58:58 | stephenfin | I shall sit at the back of the room and judge mercilessly :P | |
| 11:59:56 | gibi | stephenfin: I will do the placement project update with tetsuro so you can have the nova project update :) | |
| 12:01:10 | alex_xu | the world depends on three of us~ | |
| 12:04:28 | gibi | will be fun :) | |
| 12:04:41 | alex_xu | hah | |
| 12:07:33 | sean-k-mooney | stephenfin: your not allowed to anounce that we are killing nova and truning it into a sig :P | |
| 12:09:03 | alex_xu | haha | |
| 12:09:17 | alex_xu | I'm thinking how a sig works | |
| 12:10:02 | sean-k-mooney | alex_xu: they are allowed to have repos but have no ptl to heard the cats in a common direction | |