marun | but i digress | 00:00 |
---|---|---|
*** dimtruck is now known as zz_dimtruck | 00:00 | |
lifeless | marun: jogo: so read my link | 00:01 |
marun | lifeless: wil do | 00:01 |
lifeless | marun: jogo: and think about things like quota consistency issues | 00:01 |
jogo | lifeless: just added it to pocket, will read it | 00:01 |
lifeless | and how much easier they would be to address in such a world vs the maybe-you-get-themessage-maybe-you-don't world we program in today | 00:02 |
*** ianw has quit IRC | 00:04 | |
lifeless | and the scheduler race | 00:04 |
lifeless | if nodes claimed into a log | 00:04 |
lifeless | and the schedulers - all of them read from the same log | 00:05 |
lifeless | allcoations by schedulers could go into the same log | 00:05 |
lifeless | and it would all eventually line up | 00:05 |
*** ianw has joined #openstack-qa | 00:05 | |
jogo | while that whole class of things are pretty broken, I don't think fixing those is near the top of the priorities list yet | 00:06 |
jogo | well from the OpenStack view not just the 'nova' view | 00:06 |
jogo | mtreinish: so the servers_client delete commands stink | 00:09 |
*** proverma has quit IRC | 00:09 | |
jogo | delete is async but they don't check if things get deleted at all | 00:09 |
*** zz_dimtruck is now known as dimtruck | 00:10 | |
jogo | mtreinish: also the xml clients don't validate the response | 00:10 |
jogo | cyeoh: ^ | 00:10 |
jogo | or at least make sure the return code is good | 00:11 |
jogo | ohh they do have error checking for return codes at least | 00:13 |
*** melwitt has quit IRC | 00:16 | |
*** gustyhopeful has quit IRC | 00:20 | |
jogo | mtreinish: https://bugs.launchpad.net/tempest/+bug/1372696 | 00:26 |
*** yamahata has quit IRC | 00:27 | |
jogo | the fix should be easy but not sure what the prefered way of doing it is | 00:27 |
*** mlavalle has quit IRC | 00:28 | |
*** yjiang5 has quit IRC | 00:30 | |
*** aimon has joined #openstack-qa | 00:31 | |
jogo | lifeless: I would rather more work on the cross-project issues | 00:31 |
jogo | functional testing, better APIs, better fault isolation/tolerance etc | 00:31 |
*** oomichi has joined #openstack-qa | 00:34 | |
lifeless | jogo: ECONTEXT ? | 00:34 |
lifeless | jogo: oh, page of prose, reading ;) | 00:34 |
jogo | lifeless: half of ^ is to mtreinish about tempest | 00:35 |
jogo | notmyname: so just spotted this at the top of the gate queue https://jenkins01.openstack.org/job/gate-swift-python27/4032/console | 00:35 |
lifeless | jogo: so the reason I am arguing for this is because our cross project story is painful because we don't have a good unifying thing other than 'poll cross service' | 00:35 |
lifeless | jogo: so I'm talking root cause fix for a raft (haha) of issues | 00:35 |
clarkb | that pun | 00:36 |
jogo | lifeless: ahh makes sense | 00:36 |
*** HenryG has quit IRC | 00:36 | |
jogo | lifeless: well we do have a 'RPC' bus | 00:36 |
jogo | but that hasn't been treated as a public API | 00:36 |
*** rodrigods_ has joined #openstack-qa | 00:37 | |
cyeoh | jogo: if we always check server delete succeeded that might push up the test runtime though. Not sure by how much | 00:38 |
cyeoh | jogo: I'd guess thats why we don't always do it by default. | 00:38 |
jogo | cyeoh: hmm good point, although I don't think the current model is the best either | 00:38 |
jogo | cyeoh: as its too easy for bugs to sneak in to delete server | 00:39 |
lifeless | jogo: and its not really great as a public API | 00:39 |
lifeless | jogo: messages may be dropped | 00:39 |
lifeless | jogo: they aren't guaranteed, even with guarantees (e.g. they can be dropped in the socket buffer) | 00:39 |
lifeless | jogo: I could go on | 00:39 |
jogo | lifeless: yup! | 00:39 |
jogo | cyeoh: and we have at least one nova delete bug | 00:39 |
cyeoh | jogo: yep, though perhaps we just need a test which stresses delete more. | 00:40 |
lifeless | jogo: so here's an example for delete | 00:40 |
jogo | that doesn't always cause failures | 00:40 |
lifeless | jogo: imagine if delete a) verified ACLs etc, b) journalled an intent to the log, c) journalled success when it completeed. | 00:40 |
jogo | cyeoh: hmm that may work. but I think running concurrent tempest is a good stress tester itself | 00:40 |
lifeless | jogo: we could tell exactly what deletes are outstanding | 00:40 |
jogo | lifeless: I will read your link in a bit :) | 00:41 |
lifeless | jogo: or even have a task whose only job is to fix things up | 00:41 |
cyeoh | jogo: that's true. I guess we could try always checking deletes succeed and just see how much it costs us in terms of time. | 00:41 |
lifeless | jogo: :) | 00:41 |
cyeoh | jogo: btw what is the delete bug you are thinking of? | 00:41 |
jogo | cyeoh: yeah, so not sure where that bit of code should live in tempest | 00:41 |
jogo | cyeoh: https://bugs.launchpad.net/nova/+bug/1372670 | 00:42 |
cyeoh | jogo: thx, haven't seen that one before | 00:43 |
jogo | cyeoh: yeah I just noticed it earlier today | 00:43 |
jogo | which led me down the rabbit hole | 00:43 |
jogo | of why it wasn't always causing gate jobs to fail | 00:44 |
jogo | I am afraid to see what happens if we cause tempest to fail on a failed delete server | 00:45 |
jogo | anyway time to relocate and read some articles o/ | 00:46 |
cyeoh | heh, yea probably wouldn't be pretty.... | 00:46 |
*** salv-orlando has quit IRC | 00:46 | |
cyeoh | cya! | 00:46 |
*** suneelb has quit IRC | 00:55 | |
*** jecarey has joined #openstack-qa | 01:02 | |
*** Longgeek has joined #openstack-qa | 01:04 | |
*** jyuso has joined #openstack-qa | 01:05 | |
*** jecarey has quit IRC | 01:06 | |
*** jecarey has joined #openstack-qa | 01:07 | |
*** Longgeek has quit IRC | 01:11 | |
*** Longgeek has joined #openstack-qa | 01:11 | |
*** oomichi has quit IRC | 01:16 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Disable cinder with ironic https://review.openstack.org/115803 | 01:17 |
openstackgerrit | A change was merged to openstack/tempest: Backward compatibility to credentials in conf https://review.openstack.org/119060 | 01:17 |
*** dimtruck is now known as zz_dimtruck | 01:30 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 02:01 |
*** rodrigods_ has quit IRC | 02:01 | |
openstackgerrit | A change was merged to openstack/tempest: Migrate computev3 API tests to resource_* fixtures https://review.openstack.org/121539 | 02:02 |
*** zz_dimtruck is now known as dimtruck | 02:06 | |
openstackgerrit | A change was merged to openstack/tempest: Migrate identity API tests to resource_* fixtures https://review.openstack.org/121540 | 02:12 |
*** dims_ has quit IRC | 02:15 | |
openstackgerrit | A change was merged to openstack/tempest: Migrate network API tests to resource_* fixtures https://review.openstack.org/121541 | 02:15 |
*** dims has joined #openstack-qa | 02:16 | |
*** dims has quit IRC | 02:20 | |
*** andreaf has quit IRC | 02:23 | |
*** andreaf has joined #openstack-qa | 02:24 | |
openstackgerrit | Victoria Martínez de la Cruz proposed a change to openstack/tempest: Updates Tempest to match Zaqar's API name change https://review.openstack.org/123252 | 02:28 |
*** HenryG has joined #openstack-qa | 02:48 | |
*** dimtruck is now known as zz_dimtruck | 02:51 | |
marun | jogo, lifeless: late on this, but I wanted to say that I went to a python meetup here in SF last winter where the topic was scalability and 5/5 presentations from startups that were working at scale were using kafka | 02:53 |
marun | I remembered thinking - man, we should use that in openstack! | 02:54 |
marun | And then, apparently, developed amnesia of the whole affair. | 02:54 |
*** markmcclain has quit IRC | 02:54 | |
*** marun is now known as marun_afk | 02:55 | |
*** zz_dimtruck is now known as dimtruck | 02:56 | |
*** jyuso has quit IRC | 02:59 | |
openstackgerrit | Victoria Martínez de la Cruz proposed a change to openstack/tempest: Updates Tempest to match Zaqar's API name change https://review.openstack.org/123252 | 03:05 |
lifeless | marun_afk: :) | 03:12 |
*** mmedvede has quit IRC | 03:14 | |
*** marun_afk has quit IRC | 03:16 | |
*** pcrews has quit IRC | 03:22 | |
*** pcrews has joined #openstack-qa | 03:25 | |
*** chandankumar has joined #openstack-qa | 03:31 | |
*** jyuso has joined #openstack-qa | 03:32 | |
*** pcrews has quit IRC | 03:41 | |
*** chandankumar has quit IRC | 03:42 | |
*** pcrews has joined #openstack-qa | 03:59 | |
*** ekhugen has quit IRC | 04:04 | |
*** ekhugen has joined #openstack-qa | 04:05 | |
*** yamahata has joined #openstack-qa | 04:06 | |
*** markmcclain has joined #openstack-qa | 04:07 | |
*** Poornima has joined #openstack-qa | 04:11 | |
*** markmcclain has quit IRC | 04:17 | |
*** boden has joined #openstack-qa | 04:17 | |
*** boden has quit IRC | 04:22 | |
*** gmann has quit IRC | 04:27 | |
*** gmann has joined #openstack-qa | 04:27 | |
*** yfried|afk has quit IRC | 04:32 | |
*** swamireddy has joined #openstack-qa | 04:32 | |
*** pcrews has quit IRC | 04:34 | |
*** Madan has joined #openstack-qa | 04:35 | |
*** cmyster_away is now known as cmyster | 04:38 | |
*** chandankumar has joined #openstack-qa | 04:38 | |
*** Ch00k has joined #openstack-qa | 04:58 | |
*** yamahata has quit IRC | 04:59 | |
*** yamahata has joined #openstack-qa | 05:00 | |
*** Longgeek has quit IRC | 05:02 | |
*** Ch00k has quit IRC | 05:16 | |
*** Longgeek has joined #openstack-qa | 05:17 | |
*** boden has joined #openstack-qa | 05:18 | |
openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Only build a custom image if no image file https://review.openstack.org/116164 | 05:18 |
openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Allow heat standalone to work devstack stack https://review.openstack.org/123330 | 05:18 |
*** boden has quit IRC | 05:22 | |
*** Ch00k has joined #openstack-qa | 05:23 | |
*** harlowja is now known as harlowja_away | 05:31 | |
*** bkopilov has quit IRC | 05:38 | |
*** yfried|afk has joined #openstack-qa | 05:43 | |
*** chandankumar has quit IRC | 05:45 | |
*** Madan has quit IRC | 05:46 | |
swamireddy | cyeoh: sorry I was on wrong window... | 05:47 |
cyeoh | swamireddy: I just responded in the other channel :-) | 05:48 |
*** chandankumar has joined #openstack-qa | 05:50 | |
swamireddy | cyeoh: yes | 05:52 |
*** tnurlygayanov has quit IRC | 05:52 | |
*** tnurlygayanov has joined #openstack-qa | 05:52 | |
*** SridharG has joined #openstack-qa | 05:52 | |
*** Madan has joined #openstack-qa | 05:52 | |
*** ilyashakhat has quit IRC | 05:53 | |
*** afazekas has joined #openstack-qa | 05:53 | |
swamireddy | cyeoh: Thats correct... we cann´t merge them...so I will rollback to my previuos patch | 05:54 |
*** rdekel has joined #openstack-qa | 05:54 | |
swamireddy | cyeoh: and remove the for loops too | 05:54 |
cyeoh | swamireddy: ok sorry about that - sounds good. | 05:54 |
*** ilyashakhat has joined #openstack-qa | 05:55 | |
cyeoh | swamireddy, hey maybe add a comment saying that its intentional with the old test to delete the instance before it comes up properly? | 05:55 |
*** tnurlygayanov has quit IRC | 05:56 | |
*** tnurlygayanov has joined #openstack-qa | 05:56 | |
*** ilyashakhat has quit IRC | 06:00 | |
*** ilyashakhat has joined #openstack-qa | 06:01 | |
*** flaper87|afk is now known as flaper87 | 06:05 | |
*** ilyashakhat has quit IRC | 06:05 | |
swamireddy | cyeoh: Its fine...Sure, will add a commnet to avoid the confusion | 06:06 |
*** ilyashakhat has joined #openstack-qa | 06:06 | |
*** chandankumar has quit IRC | 06:08 | |
*** tnurlygayanov has quit IRC | 06:08 | |
*** tnurlygayanov has joined #openstack-qa | 06:12 | |
*** dimtruck is now known as zz_dimtruck | 06:13 | |
*** boden has joined #openstack-qa | 06:18 | |
*** nmagnezi has joined #openstack-qa | 06:19 | |
*** chandankumar has joined #openstack-qa | 06:20 | |
*** boden has quit IRC | 06:23 | |
*** k4n0 has joined #openstack-qa | 06:23 | |
*** Ch00k has quit IRC | 06:30 | |
*** andreaf has quit IRC | 06:32 | |
*** andreaf has joined #openstack-qa | 06:32 | |
*** Ch00k has joined #openstack-qa | 06:38 | |
openstackgerrit | varun kumar yadav proposed a change to openstack/tempest: Test to update port with CIDR value https://review.openstack.org/113861 | 06:41 |
*** neeti has joined #openstack-qa | 06:49 | |
*** chandankumar has quit IRC | 06:52 | |
*** chandankumar has joined #openstack-qa | 06:52 | |
*** Ch00k has quit IRC | 06:52 | |
*** om has joined #openstack-qa | 06:57 | |
*** mpaolino has joined #openstack-qa | 07:01 | |
*** andreaf has quit IRC | 07:11 | |
*** markus_z has joined #openstack-qa | 07:17 | |
*** gszasz has joined #openstack-qa | 07:17 | |
*** bkopilov has joined #openstack-qa | 07:27 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in network_basic_ops https://review.openstack.org/102700 | 07:31 |
*** salv-orlando has joined #openstack-qa | 07:31 | |
*** jlanoux has joined #openstack-qa | 07:39 | |
*** tkammer has joined #openstack-qa | 07:43 | |
*** Albert_13 has joined #openstack-qa | 07:44 | |
*** evgenyf has joined #openstack-qa | 07:45 | |
*** e0ne has joined #openstack-qa | 07:50 | |
*** jordanP has joined #openstack-qa | 07:55 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 07:57 | |
*** e0ne has quit IRC | 07:59 | |
*** boden has joined #openstack-qa | 08:00 | |
*** wuhg has joined #openstack-qa | 08:00 | |
*** e0ne has joined #openstack-qa | 08:00 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Switch to static content for glance upload test https://review.openstack.org/122727 | 08:01 |
*** safchain has joined #openstack-qa | 08:04 | |
*** ylobankov has joined #openstack-qa | 08:06 | |
*** Longgeek has quit IRC | 08:07 | |
*** SridharG has left #openstack-qa | 08:08 | |
*** liusheng has joined #openstack-qa | 08:13 | |
*** mpaolino has quit IRC | 08:14 | |
*** Longgeek has joined #openstack-qa | 08:29 | |
*** SridharG has joined #openstack-qa | 08:51 | |
*** e0ne has quit IRC | 08:58 | |
*** jlibosva has joined #openstack-qa | 08:59 | |
*** e0ne has joined #openstack-qa | 09:09 | |
*** qba73 has joined #openstack-qa | 09:10 | |
*** gfidente has joined #openstack-qa | 09:16 | |
*** jhenner has quit IRC | 09:16 | |
*** swamireddy1 has joined #openstack-qa | 09:28 | |
swamireddy1 | afazekas: HI | 09:30 |
swamireddy1 | afazekas: For the https://review.openstack.org/#/c/115944/ - you suggested to remove the for loop and assert + reservation[0]... | 09:30 |
swamireddy1 | afazekas: is this asserts + reservation[0] used in some other test case ...I wanted to see as ref | 09:31 |
*** swamireddy has quit IRC | 09:32 | |
*** swamireddy has joined #openstack-qa | 09:34 | |
*** cnesa14 has joined #openstack-qa | 09:36 | |
*** andreaf_ is now known as andreaf | 09:36 | |
*** swamireddy1 has quit IRC | 09:37 | |
*** tkammer has quit IRC | 09:46 | |
*** yfried|afk has quit IRC | 09:50 | |
*** yfried has joined #openstack-qa | 09:50 | |
*** yfried is now known as yfried|afk | 09:50 | |
openstackgerrit | Sergey Shnaidman proposed a change to openstack/tempest: DHCPv6 network tests https://review.openstack.org/117458 | 09:51 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 09:52 |
*** yfried has joined #openstack-qa | 09:53 | |
*** yfried|afk has quit IRC | 09:55 | |
andreaf | afazekas, mkoderer_ : fix for the non-isolated jobs: https://review.openstack.org/#/c/121149/ | 09:55 |
*** aix has joined #openstack-qa | 09:57 | |
*** yfried has quit IRC | 09:58 | |
*** chandankumar has quit IRC | 09:59 | |
*** Ch00k has joined #openstack-qa | 10:02 | |
*** hyperair has joined #openstack-qa | 10:02 | |
*** hyperair has left #openstack-qa | 10:02 | |
*** openstack038 has joined #openstack-qa | 10:12 | |
*** jpich has quit IRC | 10:12 | |
*** ashishg has joined #openstack-qa | 10:12 | |
*** tkammer has joined #openstack-qa | 10:19 | |
openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: XenAPI: stackrc now depends on nproc https://review.openstack.org/123409 | 10:23 |
*** e0ne has quit IRC | 10:23 | |
openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: XenAPI: stackrc now depends on nproc https://review.openstack.org/123409 | 10:24 |
openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added neutron cli test case for vpnaas https://review.openstack.org/123411 | 10:26 |
*** Poornima has quit IRC | 10:28 | |
*** jlanoux_ has joined #openstack-qa | 10:28 | |
*** jlanoux has quit IRC | 10:31 | |
*** mkollaro has joined #openstack-qa | 10:38 | |
*** tosky has joined #openstack-qa | 10:43 | |
*** jlanoux has joined #openstack-qa | 10:45 | |
*** Poornima has joined #openstack-qa | 10:45 | |
*** e0ne has joined #openstack-qa | 10:46 | |
*** jlanoux_ has quit IRC | 10:46 | |
*** ashishg has quit IRC | 10:48 | |
*** Longgeek has quit IRC | 10:50 | |
*** yfried has joined #openstack-qa | 10:52 | |
*** yfried is now known as yfried|afk | 10:52 | |
*** jhenner has joined #openstack-qa | 10:54 | |
andreaf | mkoderer_, thanks for your review | 10:55 |
mkoderer_ | andreaf: np ;) | 10:55 |
*** mkoderer_ is now known as mkoderer | 10:55 | |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 11:02 |
*** Longgeek has joined #openstack-qa | 11:03 | |
*** jyuso has quit IRC | 11:04 | |
*** edwa6643 has quit IRC | 11:04 | |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Drop autoscaling scenario test https://review.openstack.org/123260 | 11:09 |
*** om has quit IRC | 11:12 | |
*** Poornima has quit IRC | 11:20 | |
*** Ch00k has quit IRC | 11:23 | |
*** chandankumar has joined #openstack-qa | 11:24 | |
*** dims has joined #openstack-qa | 11:25 | |
*** [1]evgenyf has joined #openstack-qa | 11:29 | |
*** Ch00k has joined #openstack-qa | 11:29 | |
*** Ch00k has quit IRC | 11:30 | |
*** evgenyf has quit IRC | 11:31 | |
*** [1]evgenyf is now known as evgenyf | 11:31 | |
*** Madan has quit IRC | 11:34 | |
*** eglynn-officeafk is now known as eglynn-office | 11:41 | |
*** chandan_kumar has joined #openstack-qa | 11:46 | |
*** chandankumar has quit IRC | 11:47 | |
*** Longgeek has quit IRC | 11:52 | |
*** edwa6643 has joined #openstack-qa | 11:52 | |
*** chandan_kumar has quit IRC | 11:54 | |
*** edwa66431 has joined #openstack-qa | 11:55 | |
*** weshay has joined #openstack-qa | 11:56 | |
*** edwa6643 has quit IRC | 11:57 | |
*** mwagner_lap has quit IRC | 11:59 | |
*** eglynn-office is now known as eglynn-lunch | 11:59 | |
*** Longgeek has joined #openstack-qa | 12:00 | |
psedlak | hi, i was just wondering, to which program python-openstackclient belongs to? | 12:00 |
*** Longgeek has quit IRC | 12:00 | |
*** Longgeek has joined #openstack-qa | 12:00 | |
*** tkammer has quit IRC | 12:02 | |
*** swamireddy has quit IRC | 12:03 | |
*** chandankumar has joined #openstack-qa | 12:08 | |
*** k4n0 has quit IRC | 12:09 | |
*** yfried|afk is now known as yfried | 12:11 | |
*** dims has quit IRC | 12:13 | |
*** chandankumar has quit IRC | 12:14 | |
*** dims has joined #openstack-qa | 12:14 | |
*** chandankumar has joined #openstack-qa | 12:15 | |
*** tkammer has joined #openstack-qa | 12:18 | |
*** yfried is now known as yfried|afk | 12:21 | |
andreaf | psedlak: no idea, it's not in http://git.openstack.org/cgit/openstack/governance/tree/reference/programs.yaml | 12:23 |
*** yfried|afk is now known as yfried | 12:24 | |
*** chandankumar has quit IRC | 12:24 | |
psedlak | andreaf: yeah, that's where i went (almost) first, but at the end i didn't found anything, just it was rejected from Supporting projects (with that it should go to stackforge and incubate as library first) ... so i got quite confused, which info is outdated and what's missing where | 12:24 |
*** neeti has quit IRC | 12:24 | |
*** dkranz has joined #openstack-qa | 12:25 | |
*** acfleury has joined #openstack-qa | 12:25 | |
*** mmedvede has joined #openstack-qa | 12:26 | |
*** jaypipes has joined #openstack-qa | 12:27 | |
*** ChanServ sets mode: +o jaypipes | 12:27 | |
openstackgerrit | Sergey Shnaidman proposed a change to openstack/tempest: DHCPv6 network tests https://review.openstack.org/117458 | 12:29 |
*** dmellado has quit IRC | 12:31 | |
*** dmellado has joined #openstack-qa | 12:32 | |
openstackgerrit | Sergey Shnaidman proposed a change to openstack/tempest: DHCPv6 network tests https://review.openstack.org/117458 | 12:37 |
*** yfried is now known as yfried|afk | 12:38 | |
*** vponomaryov has quit IRC | 12:38 | |
*** aysyanne has joined #openstack-qa | 12:39 | |
*** vponomaryov has joined #openstack-qa | 12:41 | |
*** yfried|afk is now known as yfried | 12:42 | |
openstackgerrit | Mikhail S Medvedev proposed a change to openstack/tempest: Skip orchestration tests if Neutron is not available https://review.openstack.org/118454 | 12:45 |
*** qba73 has quit IRC | 12:46 | |
*** qba73 has joined #openstack-qa | 12:47 | |
*** sabeen1 has quit IRC | 12:50 | |
*** weshay has quit IRC | 12:51 | |
*** gszasz has quit IRC | 12:52 | |
*** chandankumar has joined #openstack-qa | 12:52 | |
*** gszasz has joined #openstack-qa | 12:52 | |
*** tosky has quit IRC | 12:56 | |
*** masayukig has quit IRC | 12:58 | |
*** gszasz has quit IRC | 12:58 | |
*** masayukig has joined #openstack-qa | 13:03 | |
*** chandankumar has quit IRC | 13:04 | |
*** weshay has joined #openstack-qa | 13:05 | |
mtreinish | jogo: the thing I think you might have missed is that not every test uses addCleanup to wait on delete, it's also baked into the base nova api test class: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n214 and http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n143 | 13:07 |
mtreinish | that gets called for teardown class | 13:07 |
*** toure has joined #openstack-qa | 13:17 | |
*** chandankumar has joined #openstack-qa | 13:19 | |
*** mriedem has joined #openstack-qa | 13:19 | |
*** k4n0 has joined #openstack-qa | 13:20 | |
*** dustins has joined #openstack-qa | 13:23 | |
*** eglynn-lunch is now known as eglynn | 13:26 | |
andreaf | mtreinish: this fixes the non-isolated job https://review.openstack.org/#/c/121149/ (running through gate) | 13:26 |
mtreinish | andreaf: ah, ok cool | 13:28 |
*** bknudson has joined #openstack-qa | 13:28 | |
andreaf | once it merges I'll retest https://review.openstack.org/#/c/107685/ | 13:29 |
mtreinish | andreaf: well it should still work with the non-neutron isolated job right? or am I missing something? | 13:30 |
andreaf | mtreinish: yes it should, it running through the tests now - only I expect the neutron one to fail | 13:31 |
*** swamireddy has joined #openstack-qa | 13:31 | |
mtreinish | andreaf: heh, it failed on devstack: https://jenkins04.openstack.org/job/check-tempest-dsvm-full-non-isolated/1/console | 13:32 |
mtreinish | so you'll have to run it again anyway | 13:32 |
andreaf | mtreinish: ok - probably glance api not starting? do you know what's causing that? | 13:33 |
mtreinish | andreaf: no it was a rax apt mirror failure: http://logs.openstack.org/85/107685/37/experimental/check-tempest-dsvm-full-non-isolated/98184fb/logs/devstacklog.txt.gz#_2014-09-23_12_40_08_566 | 13:34 |
mtreinish | the glance api oslo.config error on icehouse I thought was fixed | 13:34 |
andreaf | mtreinish: probably I just saw it yesterday night | 13:34 |
andreaf | mtreinish: on a different topic, we made some progress on the resource cleanup bp, 3 or 4 changes merged through the night - if you have time could you have a look at the next few ones? https://review.openstack.org/#/q/topic:+bp/resource-cleanup,n,z | 13:36 |
andreaf | mtreinish: hopefully we can merge them before too many rebase are required | 13:36 |
*** andreaf is now known as andreaf_ | 13:38 | |
*** sabeen1 has joined #openstack-qa | 13:39 | |
*** andreaf has joined #openstack-qa | 13:40 | |
*** zz_jgrimm is now known as jgrimm | 13:40 | |
*** swamireddy1 has joined #openstack-qa | 13:42 | |
*** nmagnezi has quit IRC | 13:42 | |
*** Ch00k has joined #openstack-qa | 13:42 | |
*** swamireddy1 has quit IRC | 13:43 | |
*** swamireddy has quit IRC | 13:43 | |
*** dustins has quit IRC | 13:45 | |
*** dustins has joined #openstack-qa | 13:46 | |
*** gszasz has joined #openstack-qa | 13:50 | |
*** gszasz has quit IRC | 13:50 | |
*** markmcclain has joined #openstack-qa | 13:50 | |
*** mwagner_lap has joined #openstack-qa | 13:50 | |
*** gszasz has joined #openstack-qa | 13:50 | |
*** gszasz has quit IRC | 13:50 | |
*** markmcclain1 has joined #openstack-qa | 13:52 | |
*** gszasz has joined #openstack-qa | 13:53 | |
*** markmcclain1 has quit IRC | 13:53 | |
*** gszasz has quit IRC | 13:54 | |
*** markmcclain has quit IRC | 13:55 | |
*** gszasz has joined #openstack-qa | 13:55 | |
*** markmcclain has joined #openstack-qa | 13:56 | |
*** jecarey has quit IRC | 13:57 | |
*** dustins has quit IRC | 13:58 | |
*** gszasz has quit IRC | 13:58 | |
*** marun has joined #openstack-qa | 13:58 | |
*** gszasz has joined #openstack-qa | 13:58 | |
*** tkammer has quit IRC | 13:59 | |
*** mlavalle has joined #openstack-qa | 13:59 | |
*** weshay has quit IRC | 14:00 | |
*** Ch00k has quit IRC | 14:02 | |
*** dustins has joined #openstack-qa | 14:03 | |
*** yfried is now known as yfried|afk | 14:03 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add a test documentation section to the docs https://review.openstack.org/113674 | 14:05 |
*** dkranz has quit IRC | 14:06 | |
*** sweston has quit IRC | 14:06 | |
*** jecarey has joined #openstack-qa | 14:07 | |
*** sweston has joined #openstack-qa | 14:08 | |
*** rbak has joined #openstack-qa | 14:10 | |
*** tkammer has joined #openstack-qa | 14:11 | |
*** weshay has joined #openstack-qa | 14:13 | |
*** cnesa15 has joined #openstack-qa | 14:14 | |
*** cnesa14 has quit IRC | 14:14 | |
*** bkopilov has quit IRC | 14:18 | |
*** blackchip has joined #openstack-qa | 14:20 | |
*** gordc has joined #openstack-qa | 14:21 | |
*** andreaf has quit IRC | 14:24 | |
*** zz_dimtruck is now known as dimtruck | 14:25 | |
*** dimtruck is now known as zz_dimtruck | 14:26 | |
*** zz_dimtruck is now known as dimtruck | 14:26 | |
openstackgerrit | Joseph Lanoux proposed a change to openstack/tempest: Verify network interfaces are in requested order https://review.openstack.org/122434 | 14:30 |
*** bkopilov has joined #openstack-qa | 14:33 | |
*** blackchip has quit IRC | 14:33 | |
*** SridharG has left #openstack-qa | 14:36 | |
*** yfried|afk is now known as yfried | 14:37 | |
*** e0ne has quit IRC | 14:46 | |
*** andreaf has joined #openstack-qa | 14:51 | |
*** e0ne has joined #openstack-qa | 14:52 | |
*** andreaf has quit IRC | 14:58 | |
*** andreaf has joined #openstack-qa | 14:58 | |
*** mlavalle has quit IRC | 15:00 | |
*** jlibosva has quit IRC | 15:01 | |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 15:01 |
*** flaper87 is now known as flaper87|afk | 15:01 | |
*** wuhg has quit IRC | 15:03 | |
*** dkranz has joined #openstack-qa | 15:04 | |
*** bkopilov has quit IRC | 15:08 | |
*** mwagner_lap has quit IRC | 15:11 | |
*** jpmontez has joined #openstack-qa | 15:15 | |
*** markmcclain has quit IRC | 15:18 | |
*** Longgeek has quit IRC | 15:23 | |
openstackgerrit | Roey Dekel proposed a change to openstack/tempest: temp https://review.openstack.org/123475 | 15:26 |
openstackgerrit | Roey Dekel proposed a change to openstack/tempest: temp https://review.openstack.org/123476 | 15:26 |
openstackgerrit | Roey Dekel proposed a change to openstack/tempest: Connectivity check from host via multiply methods https://review.openstack.org/123477 | 15:26 |
*** gszasz has quit IRC | 15:29 | |
*** jlibosva has joined #openstack-qa | 15:29 | |
*** dimtruck is now known as zz_dimtruck | 15:33 | |
*** bobj has joined #openstack-qa | 15:33 | |
*** zz_dimtruck is now known as dimtruck | 15:33 | |
openstackgerrit | Roey Dekel proposed a change to openstack/tempest: Connectivity check from host via multiply methods https://review.openstack.org/123477 | 15:36 |
bobj | Hello, I ran the "run-tempest-stress" and in log file the error 503 isn't recognized as a failure. Why? | 15:37 |
*** e0ne has quit IRC | 15:37 | |
*** chandankumar has quit IRC | 15:39 | |
*** lcurtis has joined #openstack-qa | 15:43 | |
*** tkammer has quit IRC | 15:43 | |
*** Ch00k has joined #openstack-qa | 15:45 | |
*** mwagner_lap has joined #openstack-qa | 15:46 | |
*** toni__ has joined #openstack-qa | 15:51 | |
toni__ | I can not install devstack with icehouse I got the error: 'amqplib' is not in global-requirements.txt | 15:51 |
*** aimon has quit IRC | 15:52 | |
*** jgrimm is now known as zz_jgrimm | 15:53 | |
*** rodrigods_ has joined #openstack-qa | 15:59 | |
toni__ | Nם' I עםא: 'docutils' is not in global-requirements.txt | 16:00 |
toni__ | Now I got: 'docutils' is not in global-requirements.txt | 16:00 |
*** jlanoux has quit IRC | 16:03 | |
*** SridharG has joined #openstack-qa | 16:04 | |
openstackgerrit | Roey Dekel proposed a change to openstack/tempest: Connectivity check from host via multiply methods https://review.openstack.org/123477 | 16:04 |
*** toni__ has quit IRC | 16:04 | |
mtreinish | bobj: are you running with -S/--stop? Otherwise the tests will keep running until the duration expires and print all the failures at the end. | 16:06 |
mtreinish | bobj: but, to know exactly what was happening I'd have to see the log in question | 16:07 |
*** rushiagr has joined #openstack-qa | 16:07 | |
*** liusheng has quit IRC | 16:09 | |
openstackgerrit | A change was merged to openstack/tempest: Fix scenarios not passing down specific network https://review.openstack.org/121149 | 16:09 |
*** liusheng has joined #openstack-qa | 16:10 | |
bobj | mtreinish: I'm running run-tempest-stress -t tempest/stress/etc/stress_test.json -d 10 | 16:11 |
*** SridharG has left #openstack-qa | 16:16 | |
*** rdekel has quit IRC | 16:17 | |
mtreinish | bobj: ok then it should print all the test results after 10secs including any tests that failed | 16:18 |
*** tosky has joined #openstack-qa | 16:18 | |
*** Ch00k has quit IRC | 16:19 | |
*** markmcclain has joined #openstack-qa | 16:20 | |
*** markmcclain has quit IRC | 16:21 | |
*** markmcclain has joined #openstack-qa | 16:22 | |
*** Ch00k has joined #openstack-qa | 16:22 | |
*** Ch00k has quit IRC | 16:25 | |
*** aimon has joined #openstack-qa | 16:26 | |
*** mkollaro has quit IRC | 16:30 | |
bobj | mtreinish: Yes, the log file shows all the test results, but when a 503 code is returned it's not recognized as a failure. It's a known behavior? | 16:31 |
*** rushiagr is now known as rushiagr_away | 16:31 | |
jogo | mtreinish: I did miss that. the part that is throwing me off is I am seeing failed deletes that aren't causing tempest to fail | 16:32 |
*** rushiagr_away is now known as rushiagr | 16:32 | |
*** rodrigods_ has quit IRC | 16:32 | |
openstackgerrit | Rob Crittenden proposed a change to openstack-dev/devstack: Configure endpoints to use SSL natively or via proxy https://review.openstack.org/98854 | 16:34 |
*** Albert_13 has quit IRC | 16:35 | |
*** rushiagr is now known as rushiagr_away | 16:36 | |
openstackgerrit | Aaron Rosen proposed a change to openstack/tempest: Replace wait with communicate to avoid potential deadlock https://review.openstack.org/123280 | 16:37 |
*** proverma has joined #openstack-qa | 16:39 | |
*** afazekas has quit IRC | 16:40 | |
jogo | mtreinish: so I am still a bit lost, failed deletes? | 16:40 |
jogo | mtreinish: how come I still saw failed deletes that didn't cause tempest to fail | 16:41 |
*** zz_jgrimm is now known as jgrimm | 16:43 | |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Lock test_resize_server_using_overlimit_* tests on 'compute_quotas' https://review.openstack.org/123510 | 16:48 |
*** Ch00k has joined #openstack-qa | 16:48 | |
*** toure has left #openstack-qa | 16:51 | |
*** markus_z has quit IRC | 16:51 | |
*** evgenyf has quit IRC | 16:52 | |
*** yfried is now known as yfried|afk | 16:52 | |
openstackgerrit | pritesh proposed a change to openstack-dev/devstack: Remove the Cisco Nexus monolithic plugin support https://review.openstack.org/121623 | 16:53 |
*** neeti has joined #openstack-qa | 16:53 | |
*** e0ne has joined #openstack-qa | 16:53 | |
*** chandankumar has joined #openstack-qa | 16:53 | |
*** safchain has quit IRC | 16:54 | |
*** suneelb has joined #openstack-qa | 16:55 | |
*** toure has joined #openstack-qa | 16:55 | |
*** bkopilov has joined #openstack-qa | 16:56 | |
*** proverma has quit IRC | 17:00 | |
*** proverma has joined #openstack-qa | 17:01 | |
openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added neutron cli test case for vpnaas https://review.openstack.org/123411 | 17:05 |
*** qba73 has quit IRC | 17:05 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 17:08 |
*** rushiagr_away is now known as rushiagr | 17:08 | |
*** e0ne is now known as e0ne_ | 17:13 | |
*** jordanP has quit IRC | 17:16 | |
*** rwsu has quit IRC | 17:17 | |
openstackgerrit | Ramy Asselin proposed a change to openstack-infra/devstack-gate: Add a pre-clean hook. https://review.openstack.org/122896 | 17:19 |
*** harlowja_away is now known as harlowja | 17:19 | |
openstackgerrit | Aaron Rosen proposed a change to openstack/tempest: Remove duplicate _ping_ip_address() methods https://review.openstack.org/123517 | 17:20 |
*** Poornima has joined #openstack-qa | 17:21 | |
*** torandu has joined #openstack-qa | 17:22 | |
*** jhenner has quit IRC | 17:25 | |
*** e0ne_ has quit IRC | 17:25 | |
*** neeti has quit IRC | 17:26 | |
*** markmcclain has quit IRC | 17:28 | |
*** chandankumar has quit IRC | 17:31 | |
arosen | mtreinish: is there a way to see the debug output from the tests run running them? | 17:34 |
arosen | ive been running them like: ./run_tempest.sh tempest.scenario.test_network_advanced_server_ops.TestNetworkAdvancedServerOps.test_server_connectivity_resize | 17:34 |
*** chandankumar has joined #openstack-qa | 17:35 | |
*** e0ne has joined #openstack-qa | 17:35 | |
*** rwsu has joined #openstack-qa | 17:35 | |
*** yfried|afk is now known as yfried | 17:37 | |
*** jecarey has quit IRC | 17:38 | |
*** JayJ has joined #openstack-qa | 17:40 | |
mtreinish | arosen: debug output? Like debug log messages? everything should go into the tempest log file | 17:42 |
*** atiwari has joined #openstack-qa | 17:42 | |
arosen | ah okay sorry i see. | 17:43 |
mtreinish | arosen: were the log messages what you're looking for? Or like stdout or something? | 17:43 |
arosen | mtreinish: i think i got it now thanks. | 17:44 |
mtreinish | jogo: there might be a bug where we don't ensure deletion properly somewhere (it's more than likely) but I don't think it's a widespread issue more of a specific place it's falling apart | 17:44 |
*** jecarey has joined #openstack-qa | 17:45 | |
*** rushiagr is now known as rushiagr_away | 17:46 | |
*** mriedem is now known as mriedem_afk | 17:46 | |
*** pcrews has joined #openstack-qa | 17:54 | |
*** Poornima has quit IRC | 17:55 | |
*** proverma has quit IRC | 17:56 | |
openstackgerrit | Aaron Rosen proposed a change to openstack/tempest: Log output from ping command https://review.openstack.org/123526 | 17:56 |
*** yjiang5 has joined #openstack-qa | 17:57 | |
*** proverma has joined #openstack-qa | 17:57 | |
openstackgerrit | Aaron Rosen proposed a change to openstack/tempest: Remove duplicate _ping_ip_address() methods https://review.openstack.org/123517 | 17:57 |
openstackgerrit | Aaron Rosen proposed a change to openstack/tempest: Log output from ping command https://review.openstack.org/123526 | 17:57 |
*** rushiagr_away is now known as rushiagr | 17:57 | |
mtreinish | arosen: ^^^ heh, you didn't really have to fix the whitespace, I gave it a +2 :) | 17:58 |
arosen | mtreinish: no worries. :P | 17:59 |
arosen | mtreinish: one thing i'm wondering about is that tempest uses subprocess (not the eventlet version) for ping and otherstuff which is blocking. | 18:00 |
arosen | I wonder if this could be causing some weird issues if things are running in parallel as this will block. I'm not sure though | 18:00 |
*** yfried is now known as yfried|afk | 18:00 | |
mtreinish | arosen: well the tests themselves are single threaded. Testr just divides all the tests into subgroups for a set of worker processes | 18:02 |
jogo | mtreinish: well here is the bug that I filed https://bugs.launchpad.net/tempest/+bug/1372696 | 18:02 |
arosen | mtreinish: ah so each test running is it's own process. I guess then it doesn't matter. | 18:02 |
jogo | mtreinish: I included a test that seems to have a delete cause a stacktrace | 18:03 |
*** sfineberg has joined #openstack-qa | 18:04 | |
mtreinish | arosen: yeah, it shouldn't matter. I actually think not blocking for the call out would potentially cause more issues | 18:05 |
mtreinish | jogo: yeah so looking at that test it should do the wait as part of tearDownClass | 18:05 |
mtreinish | it's doing the delete as part of an addCleanup there, so it'll end up making 2 delete calls (which might be a bug) | 18:06 |
*** jlibosva has quit IRC | 18:06 | |
*** amotoki has quit IRC | 18:07 | |
arosen | mtreinish: do you know off hand which keypair tempest uses for it's tests? | 18:08 |
*** sfineberg has left #openstack-qa | 18:09 | |
mtreinish | arosen: I think it will generate one in the tests, otherwise there should be a config option to specify it | 18:09 |
*** tosky has quit IRC | 18:09 | |
*** cnesa15 has quit IRC | 18:14 | |
*** sfineberg has joined #openstack-qa | 18:16 | |
*** arnaud has joined #openstack-qa | 18:18 | |
jogo | mtreinish: if you update the bug report with that I can try fixing it | 18:19 |
mtreinish | jogo: ok, one sec I'll leave a comment on the bug | 18:19 |
*** catohornet has joined #openstack-qa | 18:20 | |
arosen | mtreinish: another question for you. It looks like devstack is configured to use configdrive. do you know how that works in the cirros image? I see it's not getting it's ssh key from the cloud-init script. | 18:22 |
arosen | Does config drive just copy the ssh key into the guest image directly or is there a script in the guest to do this? I haven't found anything in the guest that does this or logs there. | 18:23 |
*** aix has quit IRC | 18:24 | |
*** yfried|afk is now known as yfried | 18:25 | |
*** k4n0_ has joined #openstack-qa | 18:25 | |
mtreinish | arosen: don't know of the top of my head. salv-orlando ^^^ was looking at this very recently for a ssh bug we were hitting | 18:26 |
mtreinish | so he might know the answer | 18:28 |
mtreinish | jogo: ok, comment left with more details about that delete path | 18:28 |
*** k4n0 has quit IRC | 18:29 | |
*** rushiagr is now known as rushiagr_away | 18:30 | |
*** proverma has quit IRC | 18:31 | |
*** arnaud has quit IRC | 18:32 | |
*** arnaud has joined #openstack-qa | 18:33 | |
*** proverma has joined #openstack-qa | 18:34 | |
*** mlavalle has joined #openstack-qa | 18:35 | |
*** suneelb has quit IRC | 18:35 | |
salv-orlando | arosen: config drive is mounted somewhere in the image being launched but the actual ssk is done by the cirros init process - see cirros-per | 18:36 |
*** suneelb has joined #openstack-qa | 18:36 | |
arosen | salv-orlando: The guest is doing the mounting of the drive and copying the files right? | 18:36 |
*** markmcclain has joined #openstack-qa | 18:37 | |
*** markmcclain has quit IRC | 18:37 | |
salv-orlando | arosen: I guess so. Nova sets up the drive, the guest mounts it, read stuff from it and configures the appropriate things in the root drive | 18:37 |
arosen | salv-orlando: that was my guess too. Unfortunately, there doesn't seem to be any logging of this in the guest that this is going on . | 18:38 |
openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Allow BASE_RUN_SMOKE to be overridden by env https://review.openstack.org/123538 | 18:38 |
salv-orlando | the cirros-per process just leaves a few bits in the console log. | 18:39 |
arosen | salv-orlando: i'm surprised that nova reboot <instance>; takes so long for the instance to come up and allow me to ssh into it (even locally) ~a min. Where are nova boot takes ~20 seconds for me to be able to initally access it. | 18:39 |
salv-orlando | arosen: could this be because of the recent changes to give a chance to instance to shutdown gracefully on reboot/shutoff/resize? | 18:40 |
salv-orlando | not even that recent now | 18:40 |
salv-orlando | they merged over a month ago | 18:40 |
mtreinish | salv-orlando, arosen: https://review.openstack.org/#/c/122156/ this isn't related to what you're looking at is it? | 18:41 |
arosen | salv-orlando: in all of these tempest failures it seems like the guest is pingable but not ssh'able. I think it's perhaps maybe just the guest being slow. It seems to not start it's sshd process until it reads from configdrive i think. | 18:42 |
mtreinish | arosen: yeah, taking >196 sec is definitely slow for a cirros image... | 18:43 |
*** markmcclain has joined #openstack-qa | 18:44 | |
arosen | that patch was never merged it seems. | 18:44 |
salv-orlando | arosen: ah so you’re looking at the “no route to host” failure. I thoguth you were talking about another bug. I need to go for dinner bbl | 18:44 |
arosen | mtreinish: we still use SOFT here https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_advanced_server_ops.py#L108 | 18:44 |
arosen | salv-orlando: i think they are probably related as far as things taking to long. | 18:45 |
mtreinish | arosen: oh, mriedem_afk ^^^ go fix it | 18:45 |
*** dimtruck is now known as zz_dimtruck | 18:45 | |
*** mriedem_afk is now known as mriedem | 18:45 | |
mtreinish | arosen: that's a good catch -1 mriedem_afk's patch. I'll fast approve it after he fixes that test too | 18:45 |
mriedem | gah https://review.openstack.org/#/c/122156/ | 18:45 |
mriedem | good thing it's taken over a week to get it through the gate | 18:45 |
jogo | mtreinish: thanks | 18:46 |
mtreinish | arosen: yeah so the reboot path can take forever if it's attempting a soft reboot. Because it'll fallback to hard after it attempts and fails a soft | 18:46 |
*** zz_dimtruck is now known as dimtruck | 18:49 | |
*** tkammer has joined #openstack-qa | 18:51 | |
*** mlavalle has quit IRC | 18:56 | |
mriedem | mtreinish: is there a bug for https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_advanced_server_ops.py#L108 ? | 18:56 |
mriedem | i'll fix that on top of 12215 | 18:56 |
mriedem | *122156 | 18:56 |
mtreinish | mriedem: heh, stat padding season already :) | 18:57 |
mriedem | mtreinish: no, i just already have approval on the other | 18:57 |
mtreinish | mriedem: dunno, arosen is looking at a bug, so probably just related-bug that one | 18:57 |
mriedem | https://bugs.launchpad.net/tempest/+bug/1372696 ? | 18:58 |
mriedem | f it, i'll fix it on the one i have | 18:58 |
mriedem | and harass you to approve | 18:58 |
arosen | mriedem: i'd probably just squash that with your patch | 18:58 |
mtreinish | mriedem: heh, I'm just giving you a hard time. But, I'll fast approve it either way | 18:59 |
mtreinish | mriedem: that bug is a different one that jogo was looking at | 18:59 |
arosen | mriedem: i'd squash that other soft case with your patch. Hopefully that will give it a better change of merging from a random test failure. | 19:00 |
mriedem | i'm doing it now | 19:00 |
mriedem | just got back from my quarterly haircut | 19:00 |
*** johnthetubaguy is now known as zz_johnthetubagu | 19:02 | |
*** mlavalle has joined #openstack-qa | 19:03 | |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Change scenario test usage of server reboot from SOFT to HARD https://review.openstack.org/122156 | 19:03 |
mriedem | there | 19:03 |
*** sabeen1 has quit IRC | 19:04 | |
jogo | mriedem: want to fix by bug too: https://bugs.launchpad.net/tempest/+bug/1372696 | 19:05 |
mriedem | s/by/my/ ? | 19:05 |
mtreinish | mriedem: ok re-approved | 19:05 |
openstackgerrit | A change was merged to openstack/tempest: Removes bogus negative create image compute https://review.openstack.org/122353 | 19:06 |
*** rockyg has joined #openstack-qa | 19:07 | |
*** marun has quit IRC | 19:07 | |
*** aimon has left #openstack-qa | 19:07 | |
openstackgerrit | A change was merged to openstack/tempest: Migrate baremetal API tests to resource_* fixtures https://review.openstack.org/121542 | 19:08 |
*** e0ne has quit IRC | 19:09 | |
*** aimon has joined #openstack-qa | 19:10 | |
*** marun has joined #openstack-qa | 19:12 | |
*** flaper87|afk is now known as flaper87 | 19:14 | |
*** Annamarie_Nitzsc has joined #openstack-qa | 19:18 | |
jogo | mtreinish: I don't think http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n143 is the correct behavior | 19:18 |
mtreinish | jogo: what specifically is wrong with it? I agree we should be stricter in the exception handling. But, I don't think exploding there is correct either | 19:20 |
jogo | mtreinish: if a delete fails that is a bug we need to catch | 19:22 |
jogo | mtreinish: one could argue that something like this belongs in a nova functional test | 19:22 |
jogo | mtreinish: create and delete servers at random and make sure no failures happen in create or deletes | 19:23 |
mtreinish | jogo: yeah, but failing in a tearDownClass because of that will just be confusing and hard to debug. I'm fine with logging the failure, but if the delete really needs to be tested it should be in the test | 19:23 |
mtreinish | jogo: yeah it might be a good functional test | 19:24 |
jogo | mtreinish: that is what my original thinking was, before I realized you try doing a delete on every tearDownClass | 19:24 |
jogo | mtreinish: the default behavior in at least compute tests should be to do a wait for delete | 19:24 |
*** Annamarie_Nitzsc has quit IRC | 19:25 | |
jogo | mtreinish: so my original thinking was we add a function in tempest.api.compute.base that does delete and wait for deleted together | 19:25 |
jogo | and use that everywhere a test explicitly does a delete | 19:25 |
*** HenryG is now known as HenryG_afk | 19:27 | |
*** suneelb has quit IRC | 19:27 | |
*** sabeen has joined #openstack-qa | 19:27 | |
*** suneelb has joined #openstack-qa | 19:27 | |
jogo | mtreinish: I have a hunch our current testing model (poor testing) on deletes is one of the reason why they don't work well in production | 19:27 |
mtreinish | jogo: I was thinking of something like this: http://paste.openstack.org/show/114670/ | 19:27 |
jogo | mtreinish: yeah that is better then what we have | 19:28 |
jogo | and I agree failing in cleanupClass would be hard to debug | 19:28 |
mtreinish | jogo: honestly if you want to do that just replace the cls.servers.extend() in create_test_server() with 2 add cleanups one for wait and the other for delete | 19:29 |
mtreinish | we do that in the scenario tests a bunch | 19:29 |
*** jecarey has quit IRC | 19:29 | |
mtreinish | jogo: yeah failing in cleanup is just confusing, the cinder volume delete stuff is a perfect example of what happens when it goes bad | 19:29 |
mtreinish | jogo: do you want to push out the extra logging patch as a start? | 19:31 |
jogo | mtreinish: yup, by that line of thought adding cleanups in create_test_server wouldn't work since that is a class method right | 19:31 |
jogo | mtreinish: yeah should i push that up or do you want to? | 19:32 |
jogo | mtreinish: looks like you already wrote it | 19:32 |
mtreinish | jogo: oh is it? yeah then it won't work there | 19:32 |
*** e0ne has joined #openstack-qa | 19:32 | |
jogo | mtreinish: I'll push up a patch with what I am thinking shortly | 19:33 |
mtreinish | jogo: you can, I've got too many in progress patches, and I can +2 it if it's from you... | 19:33 |
jogo | mtreinish: sounds good | 19:33 |
*** proverma has quit IRC | 19:34 | |
*** jecarey has joined #openstack-qa | 19:37 | |
*** Ch00k has quit IRC | 19:37 | |
*** SridharG has joined #openstack-qa | 19:38 | |
openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Log deletion errors in clean servers https://review.openstack.org/123555 | 19:38 |
jogo | mtreinish: patch one | 19:39 |
*** marun has quit IRC | 19:40 | |
andreaf | jogo: I think you don't have to catch NotFound on the wait, it's already handled there - see my comment | 19:43 |
mtreinish | andreaf: oh, yeah that's a good point. I forgot about that, although I don't think it would matter, it would never get raised up | 19:44 |
andreaf | mtreinish: yes it wouldn't harm but it may be confusing for who's looking at the code then | 19:45 |
*** chandankumar has quit IRC | 19:47 | |
andreaf | dkranz: would you like to have some more fun with https://review.openstack.org/#/q/topic:+bp/resource-cleanup,n,z ? | 19:48 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 19:48 |
*** e0ne has quit IRC | 19:48 | |
openstackgerrit | Akihiro Motoki proposed a change to openstack-dev/devstack: Update Horizon Neutron-related settings https://review.openstack.org/123561 | 19:50 |
openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Wait for successful server termination via addCleanup https://review.openstack.org/123562 | 19:51 |
andreaf | mtreinish: do you want me to change the commit message here: https://review.openstack.org/#/c/121551/8 | 19:51 |
*** SridharG has left #openstack-qa | 19:52 | |
openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Log deletion errors in clean servers https://review.openstack.org/123555 | 19:55 |
jogo | mtreinish: updated | 19:55 |
jogo | andreaf: thanks for the review | 19:55 |
*** proverma has joined #openstack-qa | 19:57 | |
andreaf | jogo, mtreinish: so the 2nd patch actually changes the behaviour - first it won't start next test until clean-up is done - that may increase gate time | 19:59 |
andreaf | jogo, mtreinish: and second it may fail on BuildErrorException or Timeout which wasn't the case before - so it means that a failure in cleanup becomes a test failure | 20:00 |
jogo | andreaf: yup, I think better testing of deletes is worth it. But I am not sure how much slower this makes things and if that is an acceptable amount | 20:00 |
*** edwa66431 has quit IRC | 20:00 | |
jogo | andreaf: see the bug, that is the point of the patch. | 20:01 |
andreaf | jogo, ok I see - another way could be to log cleanup issues, and to have a test that checks for any cleanup issue in the logs... but it's a bit convoluted - I agree anyways that errors on cleanup are errors and we should fail some or or another | 20:02 |
andreaf | jogo: ok let's see if there is any difference in check run time, perhaps we try a recheck or two, if it goes well I'm ok with it | 20:02 |
jogo | andreaf: sounds like a plan. I am in no rush to land that patch | 20:03 |
*** sabeen has quit IRC | 20:11 | |
*** sabeen has joined #openstack-qa | 20:13 | |
andreaf | mtreinish, stevebaker: what do you think about dropping the autoscale test then? https://review.openstack.org/#/c/123260/ | 20:13 |
stevebaker | andreaf, +1 from me | 20:14 |
andreaf | stevebaker: eh eh I though so :) | 20:14 |
stevebaker | andreaf, eventually I'd like a test which just confirms metrics and alarms are flowing to/from heat/ceilometer | 20:15 |
andreaf | stevebaker: won't that have the same issues that autoscale has? | 20:17 |
mtreinish | andreaf: whatever, it's fine either way. I just figured I should stop there with all the api tests +2d. Let those work there way through and when things catch up I'll go through the last set | 20:17 |
*** JayJ has quit IRC | 20:18 | |
andreaf | mtreinish: ok that was about the commit message right? | 20:18 |
*** JayJ has joined #openstack-qa | 20:18 | |
andreaf | mtreinish: what do you think about https://review.openstack.org/#/c/123260/? (dropping autoscale) | 20:18 |
stevebaker | andreaf, nope, it will be a much simpler test which only boots one server. Rather than booting and killing 3 servers over 10 minutes | 20:18 |
mtreinish | andreaf: yeah the commit message one | 20:19 |
andreaf | stevebaker, ok I see - so the current autoscale is trying to test too many things at the same time | 20:19 |
mtreinish | andreaf: I guess I'm fine with it. If the test has never run that's fine, we can always add it back if we deem it necessary | 20:19 |
stevebaker | andreaf, it should really be an in-tree heat functional test with API mocking. In the short term though it will become an in-tree heat integration test | 20:20 |
andreaf | good | 20:20 |
andreaf | I'm trying to get a closure on the scenario migration tests, and this is one of the last bits | 20:21 |
stevebaker | andreaf, https://review.openstack.org/#/c/112189/ | 20:21 |
andreaf | stevebaker, I see you actually setup a full devstack and run the test | 20:23 |
*** proverma has quit IRC | 20:23 | |
stevebaker | andreaf, yes. The hope is that tests developed in that job will migrate to a tempest scenario, or an in-tree functional test | 20:25 |
andreaf | stevebaker, for in-tree functional test you'd have to mock all the required apis, right? | 20:26 |
stevebaker | andreaf, Yes, it doesn't exist yet. Maybe we'll use something like mimic | 20:27 |
*** arnaud has quit IRC | 20:27 | |
andreaf | masayukig, last reviews to complete scenario migration (the test part at least): https://review.openstack.org/#/c/123260, https://review.openstack.org/122196 and https://review.openstack.org/#/c/118232/ | 20:28 |
andreaf | stevebaker: if successful it may be something to be shared with other projects in a way or another | 20:29 |
*** proverma has joined #openstack-qa | 20:30 | |
stevebaker | andreaf, yes, we could functional test all manner of internal heat things if we only mock keystone | 20:32 |
openstackgerrit | Ramy Asselin proposed a change to openstack-infra/devstack-gate: Add a pre-clean and post-clean hooks. https://review.openstack.org/122896 | 20:33 |
openstackgerrit | Ramy Asselin proposed a change to openstack-infra/devstack-gate: Refactor the hook logic into a reusable function https://review.openstack.org/123578 | 20:33 |
*** boden has quit IRC | 20:38 | |
*** boden has joined #openstack-qa | 20:38 | |
*** proverma has quit IRC | 20:40 | |
*** boden has quit IRC | 20:43 | |
*** toure has quit IRC | 20:45 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add docs for the uncategorized page https://review.openstack.org/122293 | 20:50 |
*** jhenner has joined #openstack-qa | 20:50 | |
*** proverma has joined #openstack-qa | 20:51 | |
*** skata has quit IRC | 20:52 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Properly preserve trace on error during wait https://review.openstack.org/122861 | 20:58 |
mtreinish | andreaf, dkranz: ^^^ if you've got a sec | 20:58 |
*** suneelb has quit IRC | 21:02 | |
*** suneelb has joined #openstack-qa | 21:02 | |
mtreinish | ok how does this unit test pass?: https://review.openstack.org/#/c/122538/1 I'm at a loss | 21:03 |
mtreinish | jaypipes: ^^^ any ideas? | 21:04 |
*** rockyg has quit IRC | 21:05 | |
jaypipes | mtreinish: wow. | 21:05 |
mtreinish | jaypipes: yeah I know, I'm not sure what's going on | 21:06 |
mtreinish | oh, the testr results say it got skipped? | 21:07 |
*** aysyanne has quit IRC | 21:08 | |
mtreinish | well something to dig into at least | 21:08 |
jaypipes | pt1.367: tempest.tests.test_decorators.TestServicesDecorator.test_services_list | 21:08 |
jaypipes | mtreinish: was skipped | 21:08 |
*** rockyg has joined #openstack-qa | 21:10 | |
mtreinish | jaypipes: hahaha, it's testing a decorator that raises a skip exception. doh | 21:10 |
jaypipes | lol | 21:10 |
andreaf | mtreinish, +2 on https://review.openstack.org/122861 | 21:10 |
mtreinish | andreaf: cool, thanks | 21:11 |
*** dustins has quit IRC | 21:17 | |
*** gordc has quit IRC | 21:20 | |
openstackgerrit | Ramy Asselin proposed a change to openstack-infra/devstack-gate: Add a pre-clean and post-clean hooks. https://review.openstack.org/122896 | 21:24 |
*** melwitt has joined #openstack-qa | 21:25 | |
*** jhenner has quit IRC | 21:28 | |
*** mmedvede has quit IRC | 21:33 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix use of code name in services decorator https://review.openstack.org/122538 | 21:37 |
*** andreaf has quit IRC | 21:41 | |
*** mriedem has quit IRC | 21:41 | |
*** andreaf has joined #openstack-qa | 21:42 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Start using subunit-trace for unit tests too https://review.openstack.org/123589 | 21:45 |
*** rockyg has quit IRC | 21:51 | |
*** harlowja is now known as harlowja_away | 22:01 | |
*** harlowja_away is now known as harlowja | 22:03 | |
*** flaper87 is now known as flaper87|afk | 22:07 | |
*** bknudson has quit IRC | 22:07 | |
*** proverma has quit IRC | 22:07 | |
*** JayJ has quit IRC | 22:08 | |
*** cnesa15 has joined #openstack-qa | 22:23 | |
*** atiwari has quit IRC | 22:28 | |
*** lcurtis has quit IRC | 22:29 | |
*** jecarey has quit IRC | 22:31 | |
*** openstackgerrit has quit IRC | 22:31 | |
*** openstackgerrit has joined #openstack-qa | 22:32 | |
openstackgerrit | Baodong (Robert) Li proposed a change to openstack-dev/devstack: Configure pci_passthrough_whitelist and pci_alias https://review.openstack.org/123599 | 22:43 |
*** jgrimm is now known as zz_jgrimm | 22:45 | |
*** gfidente has quit IRC | 22:45 | |
openstackgerrit | Baodong (Robert) Li proposed a change to openstack-dev/devstack: Configure pci_passthrough_whitelist and pci_alias https://review.openstack.org/123599 | 22:47 |
openstackgerrit | Baodong (Robert) Li proposed a change to openstack-dev/devstack: Configure pci_passthrough_whitelist and pci_alias https://review.openstack.org/123599 | 22:48 |
*** weshay has quit IRC | 22:48 | |
*** skata_ has joined #openstack-qa | 22:50 | |
*** dkranz has quit IRC | 22:51 | |
*** dims has quit IRC | 22:53 | |
openstackgerrit | Miguel Lavalle proposed a change to openstack/tempest: Migrate test_load_balancer_basic to tempest client https://review.openstack.org/118232 | 22:53 |
*** dims has joined #openstack-qa | 22:54 | |
*** jpmontez has quit IRC | 22:56 | |
*** mlavalle has quit IRC | 22:58 | |
*** dims has quit IRC | 22:59 | |
*** rbak has quit IRC | 23:02 | |
*** aimon has quit IRC | 23:02 | |
*** dansmith has quit IRC | 23:06 | |
openstackgerrit | Adam Gandelman proposed a change to openstack-infra/devstack-gate: Enable DIB service for Ironic https://review.openstack.org/122539 | 23:07 |
*** boden has joined #openstack-qa | 23:09 | |
*** dansmith has joined #openstack-qa | 23:10 | |
*** salv-orlando has quit IRC | 23:12 | |
*** beekneemech has joined #openstack-qa | 23:13 | |
*** bnemec has quit IRC | 23:13 | |
*** andreaf has quit IRC | 23:13 | |
*** andreaf has joined #openstack-qa | 23:14 | |
*** andreaf has quit IRC | 23:15 | |
*** boden has quit IRC | 23:15 | |
*** dims has joined #openstack-qa | 23:19 | |
*** dhellmann has quit IRC | 23:21 | |
*** salv-orlando has joined #openstack-qa | 23:21 | |
*** dhellmann has joined #openstack-qa | 23:21 | |
*** jpich has joined #openstack-qa | 23:30 | |
*** mmedvede has joined #openstack-qa | 23:31 | |
*** Madie_Batz has joined #openstack-qa | 23:37 | |
*** tkammer has quit IRC | 23:38 | |
*** edwa6643 has joined #openstack-qa | 23:43 | |
*** Madie_Batz has quit IRC | 23:45 | |
*** Abner_Zulauf83 has joined #openstack-qa | 23:47 | |
*** amotoki has joined #openstack-qa | 23:53 | |
*** danfinch has quit IRC | 23:54 | |
*** yamahata has quit IRC | 23:56 | |
*** jaypipes has quit IRC | 23:57 | |
*** amotoki has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!