Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-23
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
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

Earlier   Later