Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-29
16:15:58 cdent efried: these issues are why I pressed rado to start talking about and experimenting with this stuff. I think the sort of gravitational mechanics of nested providers is going to have lots of weird
16:16:23 cdent and I'd much prefer to see us working from concrete situations then not
16:16:48 efried cdent Agree. At the moment I'm actually leaning towards the thing I originally said was the worst idea: always using a single numbered request group.
16:17:59 Spazmotic Always good when the best idea is your worst idea
16:21:07 cdent efried: I guess we have time to experiement...
16:25:35 artom mriedem, https://bugs.launchpad.net/nova/+bug/1746032
16:25:36 openstack Launchpad bug 1746032 in OpenStack Compute (nova) "By rebuilding twice with the same "forbidden" image one can circumvent scheduler rebuild restrictions" [Undecided,New]
16:27:34 mriedem artom: L145 https://etherpad.openstack.org/p/nova-ptg-rocky
16:28:14 artom mriedem, oh hey and I'll actually be there this time :)
16:28:32 mriedem we can finally meet, face to face, and settle all the scores
16:28:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.openstack.org/512552
16:28:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.openstack.org/512553
16:28:39 openstack bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Undecided,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer)
16:28:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.openstack.org/512623
16:28:39 mriedem there can be only one
16:28:49 artom ... one of *what*?
16:29:00 mriedem idk, i'm just thinking about the quickening
16:29:04 mriedem even though that was scotland
16:29:29 artom Man, your references are out of control!
16:31:40 edleafe cdent: I've been following along, but don't have a clear picture of how things are arranged. Is there a diagram or something that would illustrate this?
16:32:22 cdent edleafe: a) which "this"?, b) I wish I never showed up today because I simply don't have the capacity to actual engage well with this topic
16:32:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961
16:32:32 mriedem artom: ok here we go, single patch ^
16:32:55 edleafe cdent: the arrangement of entities in vmware
16:33:09 edleafe and which is the "root" that placement would be selecting
16:34:20 mriedem stable/newton is still open....but not sure we want to hold up EOL for this fix
16:35:03 Spazmotic Korea in.. 6 days
16:35:05 Spazmotic Whew
16:36:06 dansmith man, newton is like the undead release
16:36:15 efried edleafe I was considering drawing that up, along with an executive summary of this conversation, and putting it all in a response to the ML thread.
16:36:32 artom Can we call U 'undead'? As an hommage?
16:36:53 cdent edleafe: not that I'm aware of at this time, but I'll check with rado. if efried is able to do something (even if it has errors), that provides at least a starting point
16:37:10 cdent the "root" I was thinking of was "compute-node"
16:37:11 mriedem i'll say it again, the path to LTS upstream is just never EOL'ing anything
16:37:21 edleafe cdent: ok, thanks. I assumed that there must be some internal documentation that would be helpful
16:38:12 cdent edleafe: could very well be, but if there are pretty pictures I haven't seen them. That doesn't mean it doesn't exist. I interact _very_ little on that side of things.
16:38:55 Spazmotic Anyone know any of the cores familiar with XenAPI drivers? Besides Garbutt that is, I got his attention on but was curious if any others were familiar that I could poke :D
16:39:14 openstackgerrit Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.openstack.org/538224
16:39:14 openstackgerrit Stephen Finucane proposed openstack/nova master: mypy: Add 'types' module https://review.openstack.org/538225
16:39:17 edleafe cdent: ok, cool. If you come across something, post it so we can all better understand what is needed
16:41:07 efried Spazmotic Maybe bauzas ?
16:41:36 artom Wasn't Bob Ball xenapi? Is he still around?
16:41:37 Spazmotic I know the driver isn't looked at too frequently by folks, so just didn't want to bark up the wrong trees wwhen I had questions or was looking for some eyeballs.
16:41:42 Spazmotic Yeah bob's still around but he's no longer core
16:42:09 Spazmotic He always gives me good ideas though so.. love me some Bob
16:47:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961
16:47:28 mriedem artom: jaypipes: added a variable for the original image ref and a comment about CastAsCall
16:47:51 mriedem Spazmotic: johnthetubaguy knows the xenapi stuff
16:48:01 mriedem and is still core, but not working as much upstream these days
16:48:09 artom mriedem, and defiantly ignored my grammar nit ;)
16:48:19 Spazmotic Yeah he's been busy with family stuff.. I got in touch with him the other day though, but figured i'd ask since we need 2 cores
16:48:35 johnthetubaguy yeah, I am around for some questions, much of the time
16:48:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961
16:48:48 mriedem artom: there you go
16:48:49 Spazmotic Now look what you did
16:48:52 johnthetubaguy (still trying to juggle back more reviewing time)
16:49:04 mriedem it's is always something i'll think of since school as a possessive
16:49:23 artom mriedem, cheers :)
16:49:40 Spazmotic hehe spend time with your family john, more important than reviews :D
16:49:43 jaypipes mriedem: danke
16:51:35 mriedem johnthetubaguy: if you are around, i'll badger you about https://review.openstack.org/#/c/527541/
16:51:59 johnthetubaguy mriedem: sure thing
16:53:59 johnthetubaguy mriedem: is the TODO about jay's join magic important?
16:54:08 mriedem no
16:54:29 jaypipes quoi?
16:54:45 johnthetubaguy https://review.openstack.org/#/c/527541/4/nova/cmd/status.py@413
16:54:47 mriedem https://review.openstack.org/#/c/527541/4/nova/cmd/status.py@413
16:54:48 mriedem yeah
16:55:01 Spazmotic someone owes someone a coke dependant upon latency and such
16:55:03 mriedem jaypipes: it's a nova-status check for ironic / instance flavor migration
16:55:17 mriedem something we should have done in pike, so i plan on backporting
16:56:56 jaypipes ah, k
16:59:07 mriedem gibi: speaking of things i don't think we want to backport if we can help it https://review.openstack.org/#/c/522112/
16:59:21 mriedem ^ adds the ability to specify service/admin user creds for cinder operations,
16:59:22 mriedem like neutron
16:59:34 mriedem so things like periodic tasks can actually detach/delete volumes once the user token is gone
17:21:22 stephenfin jaypipes: Any chance you could take a look at this today? https://review.openstack.org/#/c/537363/
17:24:43 cfriesen mriedem: is there a reason not to use a service token here? (other than that they're experimental)
17:25:43 melwitt mriedem: I wondered about that too (the eventlet os.open thing). I didn't realize the timing coincided with that update
17:26:05 melwitt *the timing of the constant fails
17:31:20 Spazmotic aigoo.. that's enough computers for today
17:31:24 Spazmotic Have a good night everyone.
17:40:33 mriedem cfriesen: are you conflating the service user thing here?
17:40:39 mriedem cfriesen: different issues
17:40:56 mriedem the service user thing is for re-auth if a user token expires
17:41:16 mriedem this is different, it's just creds to be able to have nova do stuff when there is no user token
17:41:37 mriedem the service user token stuff for re-auth should probably no longer be called experimental
17:41:58 mriedem i don't know of anyone that's done performance testing with it at scale, but haven't heard anyone complain about it either, and it's optional
17:42:34 mriedem cfriesen: speaking of performance, did you ever get any updates on the perf regression you reported last week/
17:42:35 mriedem ?
17:43:23 jaypipes stephenfin: on it.
17:47:33 melwitt mriedem: this looks good, but are we supposed to not approve things yet because of zuul? https://review.openstack.org/#/c/538961
17:48:17 cfriesen mriedem: we've got some additional data, but no smoking guns. the cpu usage just seems higher overall, with the cpu usage of most services looking more spread-out and less spiky.
17:48:25 Spazmotic zuul is all better ap parently
17:48:40 Spazmotic Was restarted and caught up
17:48:59 Spazmotic Was it's zuul so all better is relative.. but yeah anyway
17:49:03 melwitt Spazmotic: ah, thanks. I see the status alert now in the backscroll
17:51:12 Spazmotic np melwitt. johnthetubaguy if you can get a cahhance to review my notes on that commit today i'd appreciate it, but don't stress over it if not. If you decide to +2 it i'll see if some of the other UK guys can take a look tomorrow to finish it off :)
17:51:15 Spazmotic Night folks, have a good rest of your day
17:58:30 cfriesen mriedem: when authenticating a token, is it possible for services to explicitly check whether a valid service token was attached? (This is related to the "glance won't allow update of ceph image location during nova snapshot" issue.)

Earlier   Later