| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-23 | |||
| 15:33:10 | ktibi | leakypipes, ho I can see two compute06 | |
| 15:34:46 | ktibi | leakypipes, http://paste.openstack.org/show/710169/ | |
| 15:35:09 | mlavalle | rybridges: in that log statement right above L225, could you log aggregate.uuid? The reason I am asking this is https://review.openstack.org/#/c/415031/, which fixes https://bugs.launchpad.net/nova/+bug/1652642 | |
| 15:35:10 | openstack | Launchpad bug 1652642 in OpenStack Compute (nova) "Nova API doesn't return aggregate's uuid, which is needed when using the placement API" [High,Fix released] - Assigned to Matt Riedemann (mriedem) | |
| 15:35:55 | leakypipes | ktibi: DELETE FROM compute_nodes WHERE id = 15; | |
| 15:36:07 | leakypipes | ktibi: that *should* fix things up... | |
| 15:36:22 | ktibi | leakypipes, ok I restart nova compute on 06 | |
| 15:36:44 | openstackgerrit | melanie witt proposed openstack/nova master: Move _make_instance_list call outside of DB transaction context https://review.openstack.org/555093 | |
| 15:37:44 | mlavalle | rybridges: this is the Nova API version you require for aggregates to have a uuid: https://review.openstack.org/#/c/415031/9/nova/api/openstack/api_version_request.py@100 | |
| 15:37:51 | ktibi | leakypipes, same error on compute06 : Another thread already created a resource provider with the UUID 91f9e750-351f-463e-aa00-c8843c37560e. Grabbing that record from the placement API. | |
| 15:37:59 | ktibi | maybe need to restart placement ? | |
| 15:38:29 | leakypipes | ktibi: well, that's not an error. :) the error is the "Cannot refresh inventory" thing. | |
| 15:38:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute https://review.openstack.org/541420 | |
| 15:38:38 | leakypipes | ktibi: no, no need to restart placement. | |
| 15:38:57 | leakypipes | ktibi: what we want to see is inventory records being created in placement. | |
| 15:39:04 | ktibi | leakypipes, ok I'll test to migrate instance to this host | |
| 15:42:45 | figleaf | cdent: yes, one microversion for that makes sense | |
| 15:43:07 | ktibi | leakypipes, ok fail :/ | |
| 15:43:44 | ktibi | leakypipes, exception thrown: No valid host was found. Unable to move instance f8e70257-244e-4fa7-b545-0a888b742b84 to host compute06. There is not enough capacity on the host for the instance. | |
| 15:44:04 | giblet | mriedem, leakypipes, mlavalle: is it OK for you If I publish a new version of the bandwidth spec soon, or you are just in the middle of reviewing the current PS? | |
| 15:44:22 | mlavalle | giblet: I'm good | |
| 15:45:07 | mriedem | giblet: i haven't gone back over the comments since i last reviewed it last week | |
| 15:45:29 | mriedem | i could do that quick i guess... | |
| 15:46:06 | ktibi | leakypipes, http://paste.openstack.org/show/710182/ | |
| 15:46:17 | giblet | mriedem: I still need about an hour to do the update locally, if that is enough for you. Or you can go back after I published the new rev. | |
| 15:46:25 | kashyap | mriedem: So just checked w/ the folks on -infra, the Gate already does have the "Bionic" (upcoming Ubuntu LTS) images. Do we have any non-voting Nova jobs for it? | |
| 15:46:32 | kashyap | (Near as see, none.) | |
| 15:46:36 | mriedem | kashyap: no | |
| 15:46:51 | mriedem | kashyap: i'm just trying to get us to using the queens UCA right now | |
| 15:47:05 | mriedem | https://review.openstack.org/#/c/554317/ | |
| 15:47:20 | giblet | mriedem: there will some new problems described in the update | |
| 15:47:39 | giblet | mriedem: so you might want to read the new rev anyhow | |
| 15:48:11 | mriedem | giblet: i've got about 20 minutes so i'll go over the replies quick | |
| 15:48:38 | giblet | mriedem: OK, cool, thanks | |
| 15:52:49 | mriedem | giblet: yeah i can already see issues with https://review.openstack.org/#/c/502306/17/specs/rocky/approved/bandwidth-resource-provider.rst@100 | |
| 15:52:52 | mriedem | so commenting | |
| 15:54:51 | mriedem | and gerrit just shit the bed | |
| 15:56:06 | giblet | gerrit: you are not helpful at all! | |
| 15:59:15 | giblet | mriedem: gerrit is back | |
| 15:59:50 | mriedem | huzzah | |
| 15:59:54 | bauwser | sean-k-mooney[m]: +Wd | |
| 16:00:05 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Snapshot https://review.openstack.org/543023 | |
| 16:01:43 | rybridges | mlavalle: you might be on to something. When I try to print aggregate.uuid, my log message gets swallowed and i dont see anything getting printed | |
| 16:01:49 | bauwser | hum | |
| 16:01:50 | bauwser | https://review.openstack.org/#/dashboard/?foreach=(project%253Aopenstack%252Fnova-specs)+status%253Aopen+NOT+owner%253Aself+NOT+label%253AWorkflow%3C%253D-1+label%253AVerified%3E%253D1%252Cjenkins+NOT+reviewedby%253Aself+is%253Amergeable&title=Nova+Specs+Review+Inbox+&Needs+final+%252B2=NOT+label%253ACode-Review%3C%253D-1%252Cnova-core+label%253ACode-Review%3E%253D2&Small+Patches=NOT+label%253ACode-Review%3C%253D-1%252Cnova-cor | |
| 16:01:51 | bauwser | delta%253A%3C%253D10&Needs+Feedback+(Changes+older+than+5+days+that+have+not+been+reviewed+by+anyone)=NOT+label%253ACode-Review%3C%253D2+age%253A5d&You+are+a+reviewer%252C+but+haven't+voted+in+the+current+revision=reviewer%253Aself&Passed+Jenkins%252C+No+Negative+Core+Feedback=NOT+label%253ACode-Review%3C%253D-1%252Cnova-core+NOT+message%253A%22Closes-Bug%253A+%22&Wayward+Changes+(Changes+with+no+code+review+in+the+last+5+days) | |
| 16:01:51 | bauwser | abel%253ACode-Review%3C%253D2+NOT+label%253ACode-Review%3C%253D-1%252Cnova-core+age%253A5d | |
| 16:02:07 | bauwser | my gerrit dashboard is just getting me only two specs... | |
| 16:02:25 | bauwser | WTF ? | |
| 16:02:37 | kashyap | Give it a couple of mins to breathe | |
| 16:06:53 | rybridges | mlavalle: I should have the right version though... Everything I am running is just stock stable/ocata built from source. dont even have any patches yet | |
| 16:07:02 | mriedem | giblet: replies in PS17 https://review.openstack.org/#/c/502306/17/ | |
| 16:07:40 | giblet | mriedem: looking | |
| 16:09:28 | giblet | mriedem_afk: good points. I will dig it | |
| 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 | |