*** mriedem_afk is now known as mriedem | 00:03 | |
*** mriedem has quit IRC | 01:54 | |
*** tetsuro has joined #openstack-placement | 02:28 | |
*** tetsuro has quit IRC | 02:33 | |
openstackgerrit | Merged openstack/placement master: Mention OsProfiler in the testing doc https://review.opendev.org/667604 | 02:38 |
---|---|---|
*** tetsuro has joined #openstack-placement | 05:08 | |
*** tetsuro has quit IRC | 05:24 | |
*** tetsuro has joined #openstack-placement | 05:34 | |
*** tetsuro has quit IRC | 06:51 | |
*** artom has joined #openstack-placement | 06:54 | |
*** artom is now known as artom|gmtplus3 | 06:54 | |
*** tetsuro has joined #openstack-placement | 06:55 | |
*** ttsiouts has joined #openstack-placement | 07:26 | |
*** helenafm has joined #openstack-placement | 07:27 | |
*** itssurya has joined #openstack-placement | 07:34 | |
*** tetsuro has quit IRC | 07:43 | |
*** helenafm has quit IRC | 08:38 | |
*** helenafm has joined #openstack-placement | 08:39 | |
openstackgerrit | Merged openstack/placement master: Nested provider performance testing https://review.opendev.org/665695 | 09:05 |
openstackgerrit | Merged openstack/placement master: Move non-nested perfload shell commands to script https://review.opendev.org/666544 | 09:05 |
*** itssurya has quit IRC | 09:54 | |
*** helenafm has quit IRC | 10:18 | |
*** helenafm has joined #openstack-placement | 10:34 | |
*** ttsiouts has quit IRC | 10:56 | |
*** ttsiouts has joined #openstack-placement | 10:57 | |
*** helenafm has quit IRC | 11:17 | |
*** yikun has quit IRC | 11:33 | |
sean-k-mooney | o/ | 11:58 |
*** helenafm has joined #openstack-placement | 11:59 | |
sean-k-mooney | efried: edleafe would either of ye have time to look at a donwstream bug i am triaging. https://bugzilla.redhat.com/show_bug.cgi?id=1718127 while the trace back seams valid, "i ran rally, it worked but there as an error in the placment log" is not normal an acepatably standard fo bug reporting espcially when you dont say what test caused the issue. | 12:15 |
openstack | bugzilla.redhat.com bug 1718127 in openstack-nova "Nova placement IndexError: list index out of range" [Unspecified,New] - Assigned to smooney | 12:15 |
efried | sean-k-mooney: looking | 12:17 |
efried | sean-k-mooney: help me understand what level this is? | 12:18 |
efried | "osp13" means nothing to me | 12:18 |
sean-k-mooney | oh level you mean release | 12:19 |
sean-k-mooney | osp 13 is queens | 12:19 |
sean-k-mooney | so its an old version of intree placement | 12:19 |
sean-k-mooney | normally i would be tempted to close this for insuficent data as what you see is the only info i have on it too. but on the other hand the traceback looks real and im wondering if there was a racecondition that was fixe which we shoudl be backporting | 12:21 |
efried | sean-k-mooney: sorry, I'm being distracted | 12:25 |
efried | this does look familiar, but I would have to do some digging. Did you look through resolved placement-tagged nova bugs? | 12:26 |
efried | ...for this stack trace | 12:26 |
sean-k-mooney | well that was one of my open quetions. this bug would be filed in launchpad or story borad if it existited? | 12:27 |
sean-k-mooney | i have not yet | 12:27 |
sean-k-mooney | but i was about to | 12:27 |
sean-k-mooney | im also looking at the code to see if we modifyed it with a commit that semas related | 12:27 |
sean-k-mooney | this would be in launchpad for queens correct? | 12:28 |
sean-k-mooney | in the nova bug tracker | 12:28 |
efried | sean-k-mooney: https://bugs.launchpad.net/nova/+bug/1731668 blam | 12:29 |
openstack | Launchpad bug 1731668 in OpenStack Compute (nova) "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [High,Fix released] - Assigned to Jay Pipes (jaypipes) | 12:29 |
sean-k-mooney | efried: awsome and sorry for wasting your time | 12:30 |
efried | not a waste at all, glad I could help. | 12:30 |
sean-k-mooney | i had almost got to that in myy likst of but too :) | 12:30 |
efried | according to the bug, though, it was fixed in queens. | 12:30 |
sean-k-mooney | in beta2 | 12:32 |
sean-k-mooney | so we definetly should have it but ill check the souce code. | 12:32 |
efried | sean-k-mooney: the patch that fixed it renamed 'conn' to 'ctx', and you can see from the stack trace that that rename exists in whatever code they're running. | 12:33 |
efried | so | 12:33 |
efried | the fix... didn't. | 12:33 |
sean-k-mooney | i see | 12:35 |
efried | sean-k-mooney: However, assuming the problem is of the same ilk, you're probably going to need zzzeek or cdent to make any headway on it. sqla is still a swirling fog of magical mystery to me. | 12:37 |
sean-k-mooney | efried: well what i needed to do before the bug call was decide if this is valid | 12:37 |
sean-k-mooney | and i think th answer is yes | 12:38 |
efried | (or bring jaypipes back from the dead) | 12:38 |
sean-k-mooney | cdent was working on some performace mesurament code for placement right | 12:38 |
efried | yes... is that pertinent? | 12:39 |
sean-k-mooney | im wondering if that would be a simpler way to repoduce | 12:39 |
efried | we just merged https://review.opendev.org/#/c/665695/ | 12:39 |
sean-k-mooney | then a full osp 13 deployment + rally | 12:39 |
efried | mm, there may be nontrivial work necessary in the framework (wsgi setup, db opt parsing which I think is being done via env vars which I don't think worked yet in Q, etc.) if you wanted to make perfload happen. | 12:40 |
efried | I don't know what rally is. | 12:41 |
sean-k-mooney | if i understand correctly we just need to hammer the create allocation api endpoint | 12:41 |
sean-k-mooney | efried: rally is a perfmacne benchmarking suite for openstack the build on top of tempest | 12:41 |
sean-k-mooney | https://github.com/openstack/rally | 12:41 |
efried | ack | 12:43 |
sean-k-mooney | so basically the reporter said to repoduce this bug you need to deploy fa full cloud + tempest+ rally and then launch 1000s or vms with a concerancy of 36 and "should" have a message like this | 12:43 |
sean-k-mooney | im wondering if i can create a functional test to reporduce or something less then there suggestion | 12:44 |
sean-k-mooney | efried: ok ill do whats needed for the downstream bug and ill link it to the upstream one. | 12:46 |
efried | 2714 | 12:47 |
efried | or | 12:47 |
efried | ✔ | 12:47 |
sean-k-mooney | do you just rememebr that :) | 12:48 |
efried | yeah, use it frequently. | 12:48 |
*** mriedem has joined #openstack-placement | 13:24 | |
*** artom|gmtplus3 has quit IRC | 14:37 | |
*** ttsiouts has quit IRC | 14:53 | |
*** ttsiouts has joined #openstack-placement | 14:54 | |
*** ttsiouts has quit IRC | 14:54 | |
*** helenafm has quit IRC | 15:16 | |
*** efried is now known as fried_rolls | 16:15 | |
*** mriedem is now known as mriedem_away | 16:18 | |
*** e0ne has joined #openstack-placement | 17:26 | |
*** mriedem_away is now known as mriedem | 17:27 | |
*** e0ne has quit IRC | 18:12 | |
*** e0ne has joined #openstack-placement | 18:13 | |
*** e0ne has quit IRC | 18:16 | |
*** e0ne has joined #openstack-placement | 18:26 | |
*** e0ne has quit IRC | 18:28 | |
openstackgerrit | Matt Riedemann proposed openstack/placement master: Remove gate/post_test_hook.sh https://review.opendev.org/667996 | 18:38 |
*** dklyle has quit IRC | 19:36 | |
*** fried_rolls is now known as efried | 19:49 | |
*** dklyle has joined #openstack-placement | 20:18 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!