| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-13 | |||
| 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 | mriedem | dansmith: the patch is up | |
| 21:15:12 | dansmith | which is probably not helping load | |
| 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. | |
| 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 | |