Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-11
15:56:07 ildikov jgriffith: it seems we're off the hook this time :) ^^
15:56:23 mriedem there is another one like that in here
15:56:27 mriedem might as well fix that one too
16:03:58 openstackgerrit Jay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030
16:03:58 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029
16:03:59 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032
16:03:59 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031
16:04:00 openstackgerrit Jay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035
16:04:00 openstackgerrit Jay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033
16:04:01 openstackgerrit Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921
16:04:01 openstackgerrit Jay Pipes proposed openstack/nova master: placement: set/check if inventory change in tree https://review.openstack.org/470575
16:04:02 openstackgerrit Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
16:04:02 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138
16:04:05 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
16:04:05 openstackgerrit Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
16:07:59 ildikov mriedem: will do, thanks
16:08:22 ildikov mriedem: I was too eager to get a clean run on the new attach patch :)
16:11:57 openstackgerrit sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580
16:12:50 dansmith mdbooth: if you want that bdm uuid patch for the serial number thing, I think we should slam it in soonly
16:13:09 dansmith I think everyone knows it's a good idea and uncontentious, it's just never been in the critical path for anyone for long enough
16:13:19 dansmith so I say get it cleaned up and we should be able to get it in, IMHO
16:13:46 mdbooth dansmith: I'm going to write a spec for this first thing in the morning. Can I combine it with the serial number thing in 1 spec?
16:14:08 dansmith mdbooth: the spec is the serial number thing, uuid on bdm is just a work item in that spec
16:14:09 openstackgerrit Eric Fried proposed openstack/nova master: DNM: CI check https://review.openstack.org/511006
16:14:21 mdbooth Incidentally, having a DriverBlockDevice object for the root disk is also something which is useful for a bunch of other stuff.
16:15:10 mdbooth Ok. Look for me adding you to it tomorrow :)
16:18:21 mriedem melwitt: dansmith: see comment #16, i thought we had this fixed already? https://bugs.launchpad.net/nova/+bug/1682423
16:18:22 openstack Launchpad bug 1682423 in OpenStack Compute (nova) "create instance with dashbaord / 404 returned" [Undecided,Confirmed]
16:21:10 dansmith mriedem: afaik yeah
16:21:35 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_image_certificates https://review.openstack.org/500396
16:21:47 dansmith mriedem: "latest available ocata packages" doesn't mean much to me
16:22:04 mriedem i meant 15.0.7
16:22:09 mriedem not whatever ubuntu has currently
16:22:19 dansmith I mean the person who said it's still happening for them
16:22:36 mriedem oh
16:23:39 mriedem i seem to remember some tempest bugs we were trying to fix around the time of this change https://review.openstack.org/#/c/427782/
16:25:43 mdbooth mriedem: Any chance you could look at https://review.openstack.org/#/c/510106/ ? It's a reapproval. You asked for some changes during Newton, which I've added to this version of the spec.
16:26:08 mriedem not right now
16:26:12 mdbooth lyarwood's stable virt
16:26:28 mdbooth mriedem: Ok. Is it likely to make next week?
16:26:38 mriedem idk
16:26:44 mriedem lots of people can review specs
16:26:53 mriedem i've got my own to write and update as well
16:27:11 mriedem kind of concerned with a bunch of people saying they are busted on ocata atm
16:27:12 mdbooth mriedem: np, just pinging you specifically as you've been involved with it previously.
16:32:07 mriedem dansmith: wonder if these people are running with uwsgi? :)
16:32:13 mriedem remember that defeated the service version check
16:32:15 mriedem for tripleo
16:32:17 dansmith mriedem: ocata so no right?
16:32:25 dansmith oh
16:32:26 mriedem but they were hitting something similar,
16:32:29 mriedem unti lwe told them not to do that
16:32:48 dansmith hmm
16:33:11 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot https://review.openstack.org/511209
16:33:51 dansmith mriedem: see comment 9
16:33:57 dansmith mriedem: makes reference to having everything upgraded
16:35:45 mriedem yeah i think there are two different groups of people with different issues
16:35:55 mriedem once is talking about having old computes, which shouldn't have anything to do with looking up the instnace
16:36:09 mriedem i think the others are talking about a race to lookup the instance while it's building
16:36:21 mriedem which is why they added a sleep in there i guess and said it's working
16:36:37 dansmith well, one unupdated api will wreck it right?
16:36:50 mriedem b/c of the api version check?
16:36:53 dansmith yeah
16:37:02 dansmith it'll go straight to the cell db and not look at the BR
16:37:35 mriedem yeah
16:37:43 mriedem but, someone said they have a full new install
16:37:52 mriedem and that was the person that put the time.sleep(1) workaround in
16:38:29 dansmith all they need is one api service record that reported zero for the version
16:38:49 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306
16:39:37 dansmith and they did it before that check
16:39:43 dansmith the sleep I mean
16:41:30 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306
16:48:05 mriedem assuming their api services are all updated, i don't see how they could hit this really
16:48:34 mriedem we delete the build request after the instance mapping is updated,
16:48:40 mriedem we update the instance mapping after the instance is created in a cell
16:49:02 openstackgerrit melanie witt proposed openstack/nova stable/pike: Target context for build notification in conductor https://review.openstack.org/511292
16:49:04 mriedem if the instance mapping doesn't have the cell mapping in it, we get the instance from the build request - and if that's gone by the time we look, we check the instance mapping again
16:49:27 dansmith mriedem: aye
16:49:43 mriedem gdi this is like 2 days i've spent all day chasing weird bugs
17:21:15 mriedem melwitt: does https://review.openstack.org/#/c/511292/ also need to go to ocata?
17:22:43 melwitt hm, lemme check. I had thought not but maybe this code was from ocata
17:22:44 mriedem looks like yes https://github.com/openstack/nova/blob/stable/ocata/nova/conductor/manager.py#L921
17:23:11 mriedem it also looks like ocata is not targeting the instance action in the cell, so that's getting created in cell0
17:23:22 melwitt yeah, was about to say that too
17:23:54 openstackgerrit Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
17:23:55 openstackgerrit Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
17:23:55 openstackgerrit Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419
17:23:56 openstackgerrit Dan Smith proposed openstack/nova master: Split get_allocations_for_instance() into useful bits https://review.openstack.org/511306
17:23:56 openstackgerrit Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638
17:24:10 mriedem https://github.com/openstack/nova/commit/ca51db7e2db72cb774c94fb039509f2b4e6c94ca
17:24:19 dansmith mriedem: ^ updated to determine the source compute from allocations instead of trying to look it up
17:24:37 mriedem melwitt: https://github.com/openstack/nova/commit/ca51db7e2db72cb774c94fb039509f2b4e6c94ca fixed the instance action thing in pike, we'll want to backport that as well...
17:24:41 mriedem didn't have a bug o nit
17:24:51 melwitt ah, crap
17:25:17 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Make conductor create InstanceAction in the proper cell https://review.openstack.org/511307
17:25:20 mriedem thar she blar
17:25:27 melwitt k, cool
17:25:50 melwitt I should stack the other backport on that I think
17:26:01 mriedem yup
17:26:15 mriedem dansmith: ok

Earlier   Later