Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-13
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.
00:38:30 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the payload https://review.openstack.org/536243
01:08:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update admin/flavors document https://review.openstack.org/573063
01:11:39 openstackgerrit zhangdaolong proposed openstack/nova master: Fix error log output information parameters. https://review.openstack.org/575277
01:16:18 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
01:42:24 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542
02:00:54 openstackgerrit Corey Bryant proposed openstack/nova master: Fix execute mock for test_convert_image_with_errors https://review.openstack.org/575305
02:19:53 mnaser https://review.openstack.org/#/c/561284/
02:19:56 mnaser this would be nice to land :>
02:34:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Subject: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
02:34:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
02:41:08 mtreinish melwitt: sry I was stuck in the san jose convention center at ibm booth for o'reilly velocity today (and tomorrow too), what do I need to look at?
02:41:14 mtreinish I'll take a look after I grab some dinner
02:59:07 pvc_ hi
02:59:09 simondodsley can someone have a look at nova-cells-v1. It seems to be failing tempest.api.compute.servers.test_multiple_create.MultipleCreateTestJSON.test_multiple_create for every patch
02:59:25 pvc_ is there a way that i can see the progress of my server snapshot? or can i optimize it's speed because it took so long
03:03:48 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the payload https://review.openstack.org/536243
03:05:08 zcorneli simondodsley: They're already on it - https://review.openstack.org/#/c/575132
03:07:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add the guideline to write API reference https://review.openstack.org/569058
03:28:23 melwitt efried: cool, thanks
03:29:01 melwitt mtreinish: was mostly just kidding around, twas this and it's going thru the gate now https://review.openstack.org/575132
03:41:24 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092
04:32:57 openstackgerrit Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090
05:32:00 openstackgerrit Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090
05:36:54 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the payload https://review.openstack.org/536243
05:41:20 openstackgerrit yatin proposed openstack/nova stable/queens: [Stable Only] Initialise failed_builds in IronicNodeState https://review.openstack.org/574997
06:14:30 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542
06:17:28 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092
06:33:36 openstackgerrit Shilpa Devharakar proposed openstack/python-novaclient master: Modify novaclient to support basic attributes https://review.openstack.org/572285
06:36:10 openstackgerrit Shuo Liu proposed openstack/nova-specs master: fix spelling mistake https://review.openstack.org/573479
07:47:21 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the update payload https://review.openstack.org/536243
08:04:54 Kevin_Zheng gibi, Hi, Thanks alot for the help, I think https://review.openstack.org/#/c/564092/ is ready for review now.
08:12:34 ykarel hi can we get +2 +1W on https://review.openstack.org/#/c/574997/, this already had +2, +1W earlier
08:16:01 sq4ind Hey guys, I have a problem with a live migrations. Basically they are working, however when they are running too long they are failing. I've already enabled send_service_user_token and enabled post_copy... Here are the logs: https://pastebin.com/cY82PP34 Any ideas ?
08:16:45 sq4ind Oh and the expired tokens are enabled as well
08:32:57 gibi Kevin_Zheng: thanks for working on that patch. I will review it shortyl
08:33:02 gibi shortly
08:35:59 gibi Kevin_Zheng: what was the reason to change the traceback replacement with ANY to the new callback based replacement?
08:44:48 Kevin_Zheng gibi, sorry I was AFK
08:45:30 Kevin_Zheng gibi, the reason is that seems to make it work with the current replacement logic, I have to replace the whole 'fault' field
08:45:36 Kevin_Zheng not just traceback
08:46:25 gibi Kevin_Zheng: thanks for the info. Let me dig into that a bit today

Earlier   Later