Friday, 2019-06-28

*** mriedem_afk is now known as mriedem00:03
*** mriedem has quit IRC01:54
*** tetsuro has joined #openstack-placement02:28
*** tetsuro has quit IRC02:33
openstackgerritMerged openstack/placement master: Mention OsProfiler in the testing doc  https://review.opendev.org/66760402:38
*** tetsuro has joined #openstack-placement05:08
*** tetsuro has quit IRC05:24
*** tetsuro has joined #openstack-placement05:34
*** tetsuro has quit IRC06:51
*** artom has joined #openstack-placement06:54
*** artom is now known as artom|gmtplus306:54
*** tetsuro has joined #openstack-placement06:55
*** ttsiouts has joined #openstack-placement07:26
*** helenafm has joined #openstack-placement07:27
*** itssurya has joined #openstack-placement07:34
*** tetsuro has quit IRC07:43
*** helenafm has quit IRC08:38
*** helenafm has joined #openstack-placement08:39
openstackgerritMerged openstack/placement master: Nested provider performance testing  https://review.opendev.org/66569509:05
openstackgerritMerged openstack/placement master: Move non-nested perfload shell commands to script  https://review.opendev.org/66654409:05
*** itssurya has quit IRC09:54
*** helenafm has quit IRC10:18
*** helenafm has joined #openstack-placement10:34
*** ttsiouts has quit IRC10:56
*** ttsiouts has joined #openstack-placement10:57
*** helenafm has quit IRC11:17
*** yikun has quit IRC11:33
sean-k-mooneyo/11:58
*** helenafm has joined #openstack-placement11:59
sean-k-mooneyefried: 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
openstackbugzilla.redhat.com bug 1718127 in openstack-nova "Nova placement IndexError: list index out of range" [Unspecified,New] - Assigned to smooney12:15
efriedsean-k-mooney: looking12:17
efriedsean-k-mooney: help me understand what level this is?12:18
efried"osp13" means nothing to me12:18
sean-k-mooneyoh level you mean release12:19
sean-k-mooneyosp 13 is queens12:19
sean-k-mooneyso its an old version of intree placement12:19
sean-k-mooneynormally 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 backporting12:21
efriedsean-k-mooney: sorry, I'm being distracted12:25
efriedthis 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 trace12:26
sean-k-mooneywell that was one of my open quetions. this bug would be filed in launchpad or story borad if it existited?12:27
sean-k-mooneyi have not yet12:27
sean-k-mooneybut i was about to12:27
sean-k-mooneyim also looking at the code to see if we modifyed it with a commit that semas related12:27
sean-k-mooneythis would be in launchpad for queens correct?12:28
sean-k-mooneyin the nova bug tracker12:28
efriedsean-k-mooney: https://bugs.launchpad.net/nova/+bug/1731668 blam12:29
openstackLaunchpad 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-mooneyefried: awsome and sorry for wasting your time12:30
efriednot a waste at all, glad I could help.12:30
sean-k-mooneyi had almost got to that in myy likst of but too :)12:30
efriedaccording to the bug, though, it was fixed in queens.12:30
sean-k-mooneyin beta212:32
sean-k-mooneyso we definetly should have it but ill check the souce code.12:32
efriedsean-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
efriedso12:33
efriedthe fix... didn't.12:33
sean-k-mooneyi see12:35
efriedsean-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-mooneyefried: well what i needed to do before the bug call was decide if this is valid12:37
sean-k-mooneyand i think th answer is yes12:38
efried(or bring jaypipes back from the dead)12:38
sean-k-mooneycdent was working on some performace mesurament code for placement right12:38
efriedyes... is that pertinent?12:39
sean-k-mooneyim wondering if that would be a simpler way to repoduce12:39
efriedwe just merged https://review.opendev.org/#/c/665695/12:39
sean-k-mooneythen a full osp 13 deployment + rally12:39
efriedmm, 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
efriedI don't know what rally is.12:41
sean-k-mooneyif i understand correctly we just need to hammer the create allocation api endpoint12:41
sean-k-mooneyefried: rally is  a perfmacne benchmarking suite for openstack the build on top of tempest12:41
sean-k-mooneyhttps://github.com/openstack/rally12:41
efriedack12:43
sean-k-mooneyso 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 this12:43
sean-k-mooneyim wondering if i can create a functional test to reporduce or something less then there suggestion12:44
sean-k-mooneyefried: ok ill do whats needed for the downstream bug and ill link it to the upstream one.12:46
efried271412:47
efriedor12:47
efried12:47
sean-k-mooneydo you just rememebr that :)12:48
efriedyeah, use it frequently.12:48
*** mriedem has joined #openstack-placement13:24
*** artom|gmtplus3 has quit IRC14:37
*** ttsiouts has quit IRC14:53
*** ttsiouts has joined #openstack-placement14:54
*** ttsiouts has quit IRC14:54
*** helenafm has quit IRC15:16
*** efried is now known as fried_rolls16:15
*** mriedem is now known as mriedem_away16:18
*** e0ne has joined #openstack-placement17:26
*** mriedem_away is now known as mriedem17:27
*** e0ne has quit IRC18:12
*** e0ne has joined #openstack-placement18:13
*** e0ne has quit IRC18:16
*** e0ne has joined #openstack-placement18:26
*** e0ne has quit IRC18:28
openstackgerritMatt Riedemann proposed openstack/placement master: Remove gate/post_test_hook.sh  https://review.opendev.org/66799618:38
*** dklyle has quit IRC19:36
*** fried_rolls is now known as efried19:49
*** dklyle has joined #openstack-placement20:18

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!