| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-16 | |||
| 13:34:48 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Change a log level for overwriting allocation https://review.openstack.org/652994 | |
| 13:35:04 | sean-k-mooney | mnaser: i think i asked NewBruce to check that against each of the neutron api enpoints directly in the past bypassing the loadblancer | |
| 13:35:52 | sean-k-mooney | it would cause this issue espically if there was a network partion or some other factor cause different nodes to prefer diffeent api nodes | |
| 13:36:14 | sean-k-mooney | but if that was the case we would expect the same resules for osa to osa too right? | |
| 13:38:23 | mnaser | that is true as well | |
| 13:38:36 | mnaser | sean-k-mooney: but this is one of those issues where you're just desperately trying to do whatever works lol | |
| 13:38:59 | efried | dansmith: When you get a chance, would you please do the channel topic thing with https://etherpad.openstack.org/p/nova-spec-review-day ? | |
| 13:39:11 | openstackgerrit | Merged openstack/nova-specs master: Add host and hypervisor_hostname flag to create server https://review.openstack.org/645458 | |
| 13:39:39 | sean-k-mooney | ya i know from what we can tell it should be working if it wasnt for the evidence e.g. its not that would be my respocne if some one asked shoudl this work | |
| 13:43:28 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Add missing libvirt exception during device detach https://review.openstack.org/651639 | |
| 13:43:38 | openstackgerrit | sean mooney proposed openstack/os-traits master: add libvirt image metadata traits https://review.openstack.org/652996 | |
| 13:52:04 | bauzas | mriedem: sorry, finished earlier yesterday | |
| 13:52:19 | bauzas | mriedem: could you please ping me again which stable changes I could review ? | |
| 13:53:09 | mriedem | bauzas: at this point, probably queens https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/queens | |
| 13:53:28 | bauzas | ack | |
| 13:53:38 | mriedem | bauzas: and https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike+topic:bug/1669054 for pike | |
| 13:57:34 | dansmith | efried: as a placement and request groups expert, I hope you can look at this one and provide a short path to being able to make request groups expressive enough that we don't have to keep adding legacy stuff like this: https://review.openstack.org/#/c/650963/2 | |
| 13:59:01 | efried | dansmith: on my list. I had suggested https://review.openstack.org/#/c/650476/ which is generating lots of discussion both on the spec and on the ML http://lists.openstack.org/pipermail/openstack-discuss/2019-April/thread.html#4782 | |
| 13:59:37 | dansmith | efried: ack, glad to see that | |
| 13:59:42 | efried | unfortunately it's not an easy problem. | |
| 13:59:54 | dansmith | stephenfin: bauzas: I trust you will be all over that ^ spec to make sure it will solve your problems | |
| 14:00:02 | efried | which is why we've been talking about it for a couple years | |
| 14:02:52 | mriedem | melwitt: are consoles/console pools in the db model a xen only thing? | |
| 14:04:09 | mriedem | ah yes, "The nova-console service is deprecated as it is Xen specific" | |
| 14:05:17 | melwitt | cool. I don't recall the term "console pools" off the top of my head | |
| 14:07:28 | NewBruce | mnaser have run it in a loop and no changes | |
| 14:08:50 | mriedem | melwitt: ok, reason being https://review.openstack.org/#/c/570202/7/nova/db/sqlalchemy/api.py@1794 | |
| 14:09:49 | mnaser | hmm | |
| 14:09:50 | mriedem | looks like we likely leak consoles records when a server is deleted, | |
| 14:09:57 | mriedem | but that's super latent if so, and xen specific, | |
| 14:10:00 | mriedem | so i care little | |
| 14:10:20 | mriedem | Theo could open a bug for it but i wouldn't hold this up for it | |
| 14:10:23 | melwitt | haha, nice | |
| 14:10:25 | mnaser | is migrate_data stored in a db anywhere or is it thrown into the rpc request | |
| 14:10:34 | mriedem | mnaser: it's only rpc, not persisted | |
| 14:12:55 | melwitt | we deprecated the os-console service kind of recently https://review.openstack.org/610075 | |
| 14:12:57 | mriedem | stephenfin: the bottom of your cells v1 removal series needs to be fixed | |
| 14:13:07 | stephenfin | mriedem: I just saw :( | |
| 14:13:11 | melwitt | oh yeah, you already said that | |
| 14:13:13 | stephenfin | Looking at it now | |
| 14:13:15 | mriedem | melwitt: yeah i was looking at that - that deprecates the service, | |
| 14:13:16 | mriedem | but not the apis | |
| 14:13:23 | melwitt | ah | |
| 14:13:39 | mriedem | so if we were to remove the nova-console service the os-consoles api would be broken, | |
| 14:13:54 | mriedem | so we can't really deprecate the api with a microversion and expect it to still work on 2.1 w/o the service itself, | |
| 14:14:16 | mriedem | which would mean we either (1) leave the nova-console service forever or (2) obsolete the api and drop the service, like we're doing with nova-cells and nova-network | |
| 14:14:43 | mriedem | maybe ptg topic fodder - but it'd be nice to have xen people there to actually tell us if they need the thing anymore, BobBall seemed to suggest in the ML that we didn't | |
| 14:15:01 | mriedem | i don't know who works on or uses xenapi in nova anymore though | |
| 14:15:29 | NewBruce | hey mriedem seen the latest in our lovely migration issues ?! | |
| 14:15:34 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Standardize CPU resource tracking https://review.openstack.org/555081 | |
| 14:15:47 | mriedem | NewBruce: not really - can't get the neutron api extension to go away? | |
| 14:16:11 | mriedem | melwitt: i'll just add an item in the ptg etherpad | |
| 14:16:44 | NewBruce | we upgraded the entire site to rocky, and now RDO - RDO fails in addition to RDO - OSA! | |
| 14:16:54 | NewBruce | (updated the launchpad) | |
| 14:18:54 | mriedem | :/ | |
| 14:19:13 | mriedem | we definitely have multinode jobs running live migration in rocky that uses the new flow | |
| 14:19:56 | melwitt | yeah, I had thought BobBall gave the go ahead to remove it but agreed, I don't know of any people currently using xenapi. actually, I thought I've seen some patches proposed over there fairly recently. /me looks | |
| 14:21:13 | melwitt | oh, nvm, it was coreycb fixing an openssl handling thing https://review.openstack.org/635533 | |
| 14:22:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove '/os-cells' REST APIs https://review.openstack.org/651291 | |
| 14:22:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling cells v1 in '/os-hypervisors' API https://review.openstack.org/651292 | |
| 14:22:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling cells v1 in '/os-servers' API https://review.openstack.org/651293 | |
| 14:22:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'nova-manage cell' commands https://review.openstack.org/651294 | |
| 14:22:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling cells v1 for console authentication https://review.openstack.org/651295 | |
| 14:22:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove old-style cell v1 instance listing https://review.openstack.org/651296 | |
| 14:23:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'bdm_(update_or_create|destroy)_at_top' https://review.openstack.org/651297 | |
| 14:23:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'instance_fault_create_at_top' https://review.openstack.org/651298 | |
| 14:23:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'instance_info_cache_update_at_top' https://review.openstack.org/651299 | |
| 14:23:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'get_keypair_at_top' https://review.openstack.org/651300 | |
| 14:23:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'instance_update_at_top', 'instance_destroy_at_top' https://review.openstack.org/651301 | |
| 14:23:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'instance_update_from_api' https://review.openstack.org/651302 | |
| 14:23:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling 'update_cells' on 'BandwidthUsage.create' https://review.openstack.org/651303 | |
| 14:23:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling cells v1 for instance naming https://review.openstack.org/651304 | |
| 14:23:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove cells code https://review.openstack.org/651306 | |
| 14:23:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling 'InstanceUnknownCell' exception https://review.openstack.org/651307 | |
| 14:23:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove unnecessary wrapper https://review.openstack.org/651308 | |
| 14:23:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove cell APIs https://review.openstack.org/651309 | |
| 14:25:21 | mriedem | NewBruce: and you've checked to make sure there aren't any lingering nova-compute services table records in the rdo site database with an older version causing issues? | |
| 14:25:25 | stephenfin | melwitt: This is something we could do later this cycle, right? https://review.openstack.org/#/q/topic:bp/remove-consoleauth | |
| 14:25:40 | mriedem | NewBruce: and unpinned rpc versions for [upgrade_levels]/compute ? | |
| 14:25:52 | NewBruce | mriedem posted the contents of the services table to the launchpad; we have 3 entries which are not 35 but all have deleted status > 0 | |
| 14:25:53 | stephenfin | melwitt: Also, this has the feel of a bugfix to me. Thoughts? https://review.openstack.org/#/c/652967/ | |
| 14:27:15 | mriedem | stephenfin: the consoleauth thing has been really confusing for people, in rocky at least, | |
| 14:27:38 | mriedem | if there is some migration that people need to do and we can check that in some automated way, to say "you shouldn't upgrade to train" that would be good before ripping the service out | |
| 14:28:08 | stephenfin | mriedem: You mean more than the nova-status check we already have? | |
| 14:28:18 | mriedem | stephenfin: also note that there is a rest api that relies on consoleauth https://developer.openstack.org/api-ref/compute/?expanded=create-remote-console-detail#create-remote-console | |
| 14:29:14 | stephenfin | Oh, I thought that was able to use the DB tokens too? I must have misread it | |
| 14:29:31 | mriedem | maybe i'm thinking of https://developer.openstack.org/api-ref/compute/?expanded=create-remote-console-detail#show-console-connection-information | |
| 14:30:01 | mriedem | ok remote-consoles is something else, nvm | |
| 14:30:03 | stephenfin | mriedem: I had a look at that and thought it was also able to use the DB tokens | |
| 14:30:18 | mriedem | i think you are https://review.openstack.org/#/c/652969/1/nova/api/openstack/compute/console_auth_tokens.py | |
| 14:31:12 | bauzas | err, I was about to update my thoughts on the cpu-resources spec :) | |
| 14:31:21 | bauzas | but then I saw a new PS :) | |
| 14:31:27 | bauzas | dammit | |
| 14:31:29 | stephenfin | bauzas: Update them away :) I'm apply them retrospectively | |
| 14:31:50 | stephenfin | It hasn't changed significantly outside of changing how we use Flavour.vcpus | |
| 14:31:52 | bauzas | stephenfin: I'm trying to wrap my head around the upgrade impact | |
| 14:32:06 | melwitt | stephenfin: hm, yeah, I guess that xvp thing was missed.. so it feels more like a bug fix | |
| 14:32:49 | stephenfin | melwitt: Cool. I can drag that out so. I imagine no one has spotted it because no one is using it (It's Xen-specific and BobBall said we could kill it) | |
| 14:33:17 | melwitt | yeah, that's what I'm thinking too as far as it not being noticed | |
| 14:33:40 | stephenfin | Alas, that was only deprecated last cycle so I guess we can't kill that too this cycle | |