| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-13 | |||
| 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 | |
| 08:48:23 | Kevin_Zheng | and it makes no sense to write all those, and I also considered making a function that takes in params, and generates a dict, but there are still many fields that varies, like exception name, message etc, I also check how ANY works, it is basically just delete it, so I guess the best way is to just delete it with a callback function, and it could also be used in other future tests if needed | |
| 08:48:30 | Kevin_Zheng | sure, thanks for the review. | |
| 09:01:52 | BlackDex | Hello there. I'm trying to get vmware to work with nova, but i'm not getting the networking part. And the docs i see on openstack.org arn't the helpfull | |
| 09:02:03 | BlackDex | someone here knows where i can look for more info? | |