| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-11 | |||
| 16:04:02 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | |
| 16:04:02 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 16:04:05 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 16:04:05 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 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: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | |
| 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:56 | openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | |
| 17:23:56 | openstackgerrit | Dan Smith proposed openstack/nova master: Split get_allocations_for_instance() into useful bits https://review.openstack.org/511306 | |
| 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 | |
| 17:26:18 | melwitt | to increase the chance of clean backport. k | |
| 17:26:22 | mriedem | dansmith: +2 on mel's pike backport https://review.openstack.org/#/c/511292/ | |
| 17:26:52 | mriedem | i need food in my belly | |
| 17:32:12 | jaypipes | dansmith: I think zuul might be lying about the merge failure messages. | |
| 17:32:29 | dansmith | jaypipes: I don't think they've actually started the cutover yet | |
| 17:32:31 | dansmith | which means, yes | |
| 17:34:12 | melwitt | it's expected to emit false merge conflicts prior to cutover? I guess I am missing something obvious | |
| 17:34:29 | dansmith | it's been doing that all week | |
| 17:34:38 | melwitt | I know | |
| 17:35:48 | dansmith | I'm just saying, it's been doing it all week, and the changes they have planned today haven't started yet AFAIK, so I think we'd expect the same behavior | |
| 17:36:20 | dansmith | I would assume that whatever has been giving zuul merge conflict heartburn over the last week is factored into the switchover | |