Earlier  
Posted Nick Remark
#openstack-nova - 2018-06-13
20:59:09 mriedem no, i mean,
20:59:15 dansmith artom: waiting and also looking fairly impatient
20:59:21 dansmith artom: like a bus is late
20:59:22 mriedem numa networks just won't work with live migration, because numa doesn't work with live migration
20:59:26 mriedem s/waiting/punting/
20:59:35 artom mriedem, great, more pressure
20:59:50 artom No wonder no one wanted to touch this for years
20:59:53 artom ;)
21:00:03 mriedem https://www.youtube.com/watch?v=SmHeP9Sve48
21:00:10 dansmith I did hear him say that if only we had heartbeating of RPC calls, then other stuff would be a lot better and he could get back to work
21:00:16 dansmith dunno if that's true, but sounded really good
21:00:37 mriedem i did my part
21:00:55 artom ... is it worrying that I have no idea what "heartbeating of RPC calls" means?
21:01:11 mriedem EVERYONE has been talking about it
21:01:19 mriedem it's like the new containers
21:01:28 dansmith it slices
21:01:29 dansmith it dices
21:01:35 mriedem it...pings?
21:01:38 dansmith it does the kids' homework
21:01:41 dansmith heh
21:02:00 artom This is high school all over again :(
21:02:01 mriedem artom: https://review.openstack.org/#/c/566696/
21:02:13 dansmith artom: mildly live-migration-related :P
21:02:21 jmlowe so, about that url http(s) problem, seems to be fixed with a trailing /
21:02:54 artom Ah, I see
21:07:17 jmlowe Is that worth filing a documentation bug? If you are using ssl then you need a trailing slash on your nova endpoint url
21:08:38 mgagne you do?!
21:14:54 dansmith mriedem: did that tempest test get reverted?
21:15:06 dansmith just noticing a lot of nova things in check have failing tests
21:15:12 dansmith which is probably not helping load
21:15:12 mriedem dansmith: the patch is up
21:15:21 dansmith right, but unmerged?
21:15:24 mriedem http://status.openstack.org/elastic-recheck/#1776684
21:15:26 melwitt fwiw, I found this old thread about trailing slash, not sure if it's the same thing http://lists.openstack.org/pipermail/openstack-dev/2016-March/090143.html
21:15:27 dansmith it's now 2h15m to get running
21:15:30 mriedem correct https://review.openstack.org/#/c/575132
21:15:53 dansmith wonder if we could poke someone to slam that in?
21:16:18 mriedem i pinged mtreinish and gmann when i posted it in -qa
21:16:23 mriedem but gmann won't be around for awhile
21:16:27 mriedem ken'ichi is qa core
21:16:33 dansmith okay
21:16:38 mriedem but is asking for a config option, which i don't really think should hold up the revert
21:16:47 dansmith maybe melwitt can take that torch, I'm gonna disappear soon
21:20:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Ignore UserWarning for scope checks during test runs https://review.openstack.org/575237
21:20:05 mriedem this is for cdent ^
21:20:56 mriedem dansmith: you could also comment on or +1 the revert
21:21:04 dansmith I thought I did
21:21:26 mriedem call sdague out of the bullpen
21:23:43 melwitt which torch? this tempest revert?
21:23:51 dansmith melwitt: yeah
21:23:56 melwitt ack
22:01:15 tonyb Should we wait on the
22:01:59 tonyb Revert "Add server name check for multiple creations" to merge or just balcklist the test in the short term to unblock the queens gate?
22:19:07 openstackgerrit Matt Riedemann proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405
22:19:08 openstackgerrit Matt Riedemann proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
22:19:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
22:19:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324
22:20:36 tonyb dansmith, mriedem I'll nag gmann in a bit when he's online
22:21:50 mriedem thanks
22:22:52 melwitt tonyb, mriedem: that's an idea ... we do have a special test rc file for the cells v1 job IIRC. and it's affecting master too right, not just queens?
22:23:30 tonyb melwitt: Yeah it's probably all branches you're right
22:23:52 melwitt https://github.com/openstack/nova/blob/master/devstack/tempest-dsvm-cells-rc
22:24:12 tonyb melwitt: so that's less appealing as we'd need to merge and revert is on (potentailly) 4 branches :(
22:24:31 melwitt yeah
22:24:48 tonyb rats
22:24:51 melwitt good to point out though, I had forgotten about the cells rc file
22:25:07 mriedem i mentioned it this morning when we identified the bug
22:25:11 mriedem and from the tempest revert,
22:25:17 mriedem you'll see i abandoned it at one point to go with the rc,
22:25:19 melwitt okay, sorry, I didn't see that
22:25:20 tonyb in other news that was probably the closest I've come to writing real code in a month ;P
22:25:21 mriedem however, it's not just cells that this could break
22:25:47 melwitt mtreinish where are you, we need you! :P
22:25:51 mriedem it's config-driven api behavior that was available < rocky
22:26:48 mriedem * < queens
22:27:07 mriedem but since tempest is branchless, people could be running with this configured differently in ocata or pike and then tempest would fail for them
22:27:42 mriedem which is why i'm saying on the tempest change, you can't really assert this until pike goes into extended maintenance mode
22:28:34 mriedem oops, i'm wrong, it was rocky https://review.openstack.org/#/c/499612/
22:28:35 melwitt yeah, I think I understand now
22:28:53 melwitt ah okay
22:29:36 tonyb Yeah the tempest revert is clearly the rigth thing to do, but I guess I'm just impatient ;P
22:31:15 mriedem fixed the commit message
22:31:28 tonyb mriedem: danke
22:39:35 melwitt thanks gmann
22:40:15 gmann melwitt: mriedem seems like new PS.
22:41:37 gmann melwitt: done
22:41:54 melwitt gmann: cool, thanks. not sure if that might've failed pep8
22:42:48 openstackgerrit Zack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876
22:42:59 gmann melwitt: yea it should not anyways i ll keep eyes on this till merge.
22:43:18 melwitt awesome thanks ++
22:47:58 gmann melwitt: when cell v1 going away? i am thinking to run nova cell v1 job on tempest too to avoid such failure.
22:48:36 gmann thinking if that will be worth to do or not
22:49:39 melwitt gmann: it's currently already deprecated, the plan as discussed at the ptg is to remove nova-network first, then cells v1 https://etherpad.openstack.org/p/nova-ptg-rocky L301,
22:50:09 melwitt we're making some progress on nova-network removal this cycle but are keeping the core intact as a safety net for the CERN folks as they're in the middle of a nova-network => neutron migration,
22:50:24 melwitt but we have _not_ started any cells v1 removal. the earliest we will be able to remove is stein
22:55:08 gmann yea.
22:56:24 gmann thanks , ll check if we can add cell v1 in tempest also and remove together with nova.
22:56:56 gmann dansmith: melwitt sorry for that tempest patch and blocking nova.
22:59:00 melwitt gmann: no worries, thanks for the help with solving it
#openstack-nova - 2018-06-14
00:17:24 efried melwitt: I think we're good on the blueprints I know about.

Earlier   Later