| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-23 | |||
| 16:10:00 | openstackgerrit | Merged openstack/nova-specs master: Reintroduced nic feature based scheduling for rocky https://review.openstack.org/545951 | |
| 16:16:22 | bauwser | leakypipes: just a comment on https://review.openstack.org/#/c/545057/5 preventing a +2, just add your thoughts on error handling and I'm +2 | |
| 16:17:01 | rybridges | mlavalle: One other thing to note perhaps... When I do openstack aggregate show, I see the ID, but not the uuid in the output. If I want to see the uuid I need to go to the db and look at the aggregates table. | |
| 16:17:30 | mlavalle | rybridges: give me a few minutes and I'll check in my dev system | |
| 16:17:43 | mlavalle | I was going to propose you check with the CLI | |
| 16:17:50 | mlavalle | so you got ahead of me ;-) | |
| 16:19:07 | rybridges | cool thanks! Yea i am taking a look at the CLI now. Even when I run aggregate list with the --debug option i still do not see the uuid from the db anywhere in the response. That is likely related to the problem | |
| 16:20:18 | rybridges | The response body looks like this: {"aggregates": [{"name": "Neutron segment id b9d173e2-fff2-4a33-ac50-312e6f3be914", "availability_zone": null, "deleted": false, "created_at": "2018-03-23T15:59:51.000000", "updated_at": null, "hosts": [], "deleted_at": null, "id": 19, "metadata": {}}]} | |
| 16:33:35 | cfriesen | sahid: for what it's worth, I think your short-term proposal to run isolated emulator threads on 'CONF.cpu_shared_set' is reasonable, though I have one question--what would we do if that set is empty? Allocate a new host CPU like we do now and emit a log? | |
| 16:34:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730 | |
| 16:34:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729 | |
| 16:34:51 | openstackgerrit | Dan Smith proposed openstack/nova master: Make get_allocation_candidates() honor aggregate restrictions https://review.openstack.org/547990 | |
| 16:34:51 | openstackgerrit | Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728 | |
| 16:34:52 | openstackgerrit | Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002 | |
| 16:34:52 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282 | |
| 16:34:53 | openstackgerrit | Dan Smith proposed openstack/nova master: Add an index on aggregate_metadata.value https://review.openstack.org/555851 | |
| 16:34:56 | mlavalle | rybridges: this is what I see in my dev environment, which was built from trunk about a week ago: http://paste.openstack.org/show/710244/ | |
| 16:35:57 | openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/555590 | |
| 16:37:12 | leakypipes | giblet: totally cool with me if you publish a new version of the bw spec | |
| 16:37:39 | leakypipes | ktibi: ok, so inventory isn't being created properly on the compute06 for some reason... | |
| 16:37:49 | leakypipes | ktibi: still trying to figure out why that is. | |
| 16:38:12 | giblet | mriedem_afk: I left my view in the reply https://review.openstack.org/#/c/502306/17/specs/rocky/approved/bandwidth-resource-provider.rst@100 | |
| 16:38:18 | ktibi | leakypipes, yes need to clean all compute06 in all DB for works. | |
| 16:38:37 | ktibi | thx for your time :p | |
| 16:40:41 | leakypipes | np | |
| 16:42:48 | rybridges | mlavalle: I did not build from trunk. I am running stable/ocata. My output looks like this: http://paste.openstack.org/show/710254/ | |
| 16:43:44 | openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova master: Add `hide_hypervisor_id` flavor extra_spec https://review.openstack.org/555861 | |
| 16:43:46 | mlavalle | rybridges: run the command with --debug, so we can see the response. the client might be out of synch with the API | |
| 16:46:29 | cdent | leakypipes: I suspect my forbidden changes in allocation candidates are going to collide a fair bit with the stuff you are working on for nested, especially since I think both sets of work are going to reveal some necessary changes with how trait handling is desired to work. I don't think it will be dire or anything, just a heads up | |
| 16:51:43 | mlavalle | rybridges: I'll be away from my keyboard for a while. I have to pick up my wife. I'll ping you when I come back | |
| 16:52:30 | leakypipes | cdent: ack, you are correct. | |
| 16:53:08 | leakypipes | cdent: I should have the next revision of that code (https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/nested-resource-providers-allocation-candidates) pushed within the next hour. | |
| 16:53:29 | cdent | cool, my stuff is very spikey at the moment but almost working | |
| 16:53:36 | leakypipes | cdent: just trying to get fried_rolls's update provider tree series reviewed right now | |
| 16:53:41 | leakypipes | cdent: coolio. | |
| 16:55:33 | rybridges | mlavalle: Here is debug output http://paste.openstack.org/show/710270/ | |
| 17:05:27 | cfriesen | bauwser: question for you..if an operator doesn't want to expost a host as NUMA-aware, why not just reboot with "numa=off" on the kernel boot args? | |
| 17:09:37 | openstackgerrit | Elod Illes proposed openstack/nova stable/ocata: Functional test for regression bug #1713783 https://review.openstack.org/505160 | |
| 17:09:38 | openstack | bug 1713783 in OpenStack Compute (nova) ocata "After failed evacuation the recovered source compute tries to delete the instance" [High,In progress] https://launchpad.net/bugs/1713783 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 17:15:37 | openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: cold migrate to compute down https://review.openstack.org/496280 | |
| 17:29:51 | openstackgerrit | Tyler Blakeslee proposed openstack/nova master: Add __repr__ for NovaException https://review.openstack.org/555812 | |
| 17:30:08 | openstackgerrit | Jay Pipes proposed openstack/nova-specs master: mirror nova host aggregates to placement API https://review.openstack.org/545057 | |
| 17:32:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | |
| 17:35:12 | giblet | mriedem, leakypipes, mlavalle, rubasov: I've published a new rev from the bandwidth spec covering two new issues 1) simulating neutron backend selection with a new scheduler weigher 2) handling the mapping between physical resource consumption and the resource claim https://review.openstack.org/#/c/502306 | |
| 17:35:41 | sahid | cfriesen: is the set is empty so we use the default behavior of hw:emulator_threads_policy="share" | |
| 17:35:50 | sahid | i will update the spec for monday | |
| 17:41:47 | leakypipes | giblet: reviewing it as we speak... | |
| 17:42:13 | mriedem | rybridges: aggregate uuids were in a later microversion, openstack CLI defaults to 2.1, so if you want the uuid in the openstack aggregate show output, you need to opt into the microversion | |
| 17:42:51 | mriedem | openstack aggregate list --openstack-compute-api-version 2.41 | |
| 17:42:54 | mriedem | something like that | |
| 17:43:14 | mriedem | --os-compute-api-version i mean | |
| 17:45:00 | rybridges | ok thanks mriedem. I can try that. Is there a way to tell nova to use that microversion on the backend as well? That may work for me as a user on the CLI, but the backend also needs to be able to use that microversion as well | |
| 17:48:26 | jgwentworth | maybe some progress has been made but yesterday rybridges said there weren't any hosts in the nova_api.aggregate_hosts table, proving that the request to add the host to the aggregate never made it there | |
| 17:48:49 | mriedem | rybridges: i don't understand your question | |
| 17:48:54 | mriedem | what is "the backend"? | |
| 17:48:54 | jgwentworth | so not a microversion showing the uuids issue | |
| 17:49:06 | openstackgerrit | Merged openstack/nova master: Always deallocate networking before reschedule if using Neutron https://review.openstack.org/520248 | |
| 17:49:27 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Always deallocate networking before reschedule if using Neutron https://review.openstack.org/555418 | |
| 17:50:37 | mriedem | rybridges: specifying --os-compute-api-version 2.x on the CLI passes that in a header to the compute REST API, which then processes that single request with that microversion | |
| 17:52:19 | rybridges | mriedem: What I mean is.. I need neutron to also use the correct microversion when it creates aggregates. Like in this call: https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/plugin.py#L224 | |
| 17:52:55 | rybridges | I realize i can pass the microversion manually when I am running on the CLI, but how does neutron know to use the right microversion when it needs to create aggregates on my behalf? | |
| 17:53:12 | rybridges | I am guessing there is some configuration somewhere in neutron.conf or nova.conf where I can specify the microversion? | |
| 17:53:23 | mriedem | neutron doesn't have microversions | |
| 17:53:29 | mriedem | and nova aggregates don't have anything to do with neutron | |
| 17:53:38 | mriedem | nova doesn't pass a microversion through it's compute API to neutron | |
| 17:53:48 | jgwentworth | you can pass microversions with the novaclient python bindings, so the neutron plugin could do that, I think | |
| 17:53:49 | mriedem | microversions in config files isn't a thing | |
| 17:54:11 | mriedem | neutron is a client, | |
| 17:54:13 | jgwentworth | he's talking about some kinda neutron plugin that creates nova aggregates and adds hosts to aggregates when using the routed provider network segments stuff | |
| 17:54:19 | mriedem | of the placement API (and compute API?), | |
| 17:54:28 | mriedem | so neutron, as a client, will need to specify a microversion when it makes its own requests | |
| 17:54:33 | mriedem | if they require some minimum microversion | |
| 17:55:00 | mriedem | this is the same thing as cinder calling the compute os-server-external-events API with a microversion to pass volume-extended events | |
| 17:55:32 | mriedem | https://github.com/openstack/cinder/blob/master/cinder/compute/nova.py#L207 | |
| 17:56:35 | rybridges | ya jgwentworth gets what I am doing. So in my case, the openstack network subnet create command is creating the aggregate for me. I never ask nova explicitly myself to do it, it happens in the backend as part of the subnet create command. So I need to pass the microversion into the subnet create command maybe.. I can try that | |
| 17:57:46 | jgwentworth | rybridges: here is what I was talking about, when the client object is instantiated, a microversion can be specified, but I think there's also a way to specify per call https://docs.openstack.org/python-novaclient/pike/reference/api/index.html#usage | |
| 17:58:21 | jgwentworth | not seeing that in this doc tho | |
| 18:01:50 | jgwentworth | actually, not seeing a way to specify a microversion per-call in the novaclient python API | |
| 18:02:45 | mriedem | fried_rolls: hopefully i've word-smithed this well enough https://review.openstack.org/#/c/555481/ | |
| 18:03:07 | mriedem | rybridges: you should not have to pass through a compute api version to neutron | |
| 18:03:18 | mriedem | if neutron is going to create aggregates in nova, and needs the uuid, then it must use at least microversion 2.41 | |
| 18:03:23 | mriedem | else it's a bug | |
| 18:04:01 | mriedem | there is not a way to specify a microversion per call in novaclient's python binding API, as it's stored on the client object when it's constructed, | |
| 18:04:16 | mriedem | you could probably mutate the internals of the client object, but we don't have a supported hook for doing that | |
| 18:04:38 | rybridges | okay thanks mriedem thtat is helpful to know | |
| 18:05:02 | rybridges | The uuid is not being returned by the python novaclient when an aggregate is created from neutron, that i know for sure | |
| 18:05:23 | leakypipes | giblet: small request in review. | |
| 18:06:04 | mriedem | where in the neutron code does neutron create a nova host aggregate? | |
| 18:06:06 | rybridges | so maybe this code in neutron that instantiates the client is not correct and/or is not instantiating the python novaclient with the correct microversion | |
| 18:06:09 | mriedem | https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/placement_client.py#L163 is NOT nova | |
| 18:06:25 | mriedem | placement aggregates != nova host aggregates | |
| 18:06:28 | mriedem | completely different REST APIs | |
| 18:06:49 | rybridges | right here: https://github.com/openstack/neutron/blob/master/neutron/services/segments/plugin.py#L221 | |
| 18:06:56 | rybridges | this is going thru nova client, not placement client | |
| 18:06:58 | rybridges | to create the aggregate | |
| 18:07:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | |
| 18:07:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: resource requests for nested providers https://review.openstack.org/554529 | |
| 18:07:21 | leakypipes | fried_rolls: ok, ready for your re-re-re-review ^^ | |
| 18:07:28 | leakypipes | figleaf: you too :) | |
| 18:07:45 | mriedem | rybridges: https://github.com/openstack/neutron/blob/master/neutron/services/segments/plugin.py#L48 | |