Friday, 2013-09-20

mtreinishit'll poll launchpad to make sure it's a valid bug00:00
mtreinishand do a sanity check on the logstash query (not sure how I'll do this yet)00:00
*** matsuhashi has joined #openstack-qa00:00
jog0mtreinish: oh no  Iam working on that too00:03
jog0ohh good00:03
jog0I am doing the sanity check part00:03
mtreinishjog0: haha ok I'll just handle the lp part (that's really simple)00:03
jog0perfect00:04
jog0check for if the  bug is open and if its closed we want to leave it around for a week or so to be safe Ithink00:04
mtreinishok00:04
jog0actually I think we should just make sure the bug is a valid openstack bug00:07
jog0and not the state00:07
jog0because the logstash query part will detect closed bugs (if no  hits in logstash for something that worked  before, it can be closed00:07
mtreinishok, is there a list of openstack lp projects somewhere?00:08
jog0https://launchpad.net/openstack00:08
mtreinishyep that'll do it00:08
jog0you can look up bugs on that I think00:08
mtreinishyeah there is a bug list there00:09
mtreinishof course that assumes launchpadlib lets you query that00:11
jog0I played with the launchpadlib as part of recheckwatch00:12
jog0its pretty nice00:12
RecheckWatchBotopenstack/nova change: https://review.openstack.org/47504 failed tempest with an unrecognized error00:17
jog0mtreinish: pushed a patch00:18
mtreinishjog0: ok I'll pull it00:19
mtreinishwe've got similar naming tastes00:19
* jog0 is looking forward to using gerrit instead of git00:19
jog0mtreinish: I figured00:19
jog0you can just roll your code into that file00:19
mtreinishyeah that's what I was planning to do00:19
mtreinishit's funny how used to the gerrit workflow you get00:19
jog0great minds think alike00:20
jog0yeah00:20
jog0so the queries test cought some bugs I had to fix00:20
jog0so it works00:20
*** yfried has joined #openstack-qa00:22
mtreinishheh, nice00:22
mtreinishdid you want to leave queries.json hardcoded? I had it read it in from config00:23
*** nosnos has joined #openstack-qa00:23
jog0I did not00:28
jog0just lazy00:29
jog0can you add a TODO fix me for that00:29
mtreinishjog0: it's a 2 line fix I'll just include it in the lp check patch00:29
jog0anyway heading out thanks again00:29
jog0mtreinish: works for me00:29
jog0until we are in gerrit I am playing loose with the patches00:30
*** sthaha has joined #openstack-qa00:33
*** sthaha has joined #openstack-qa00:33
openstackgerritZhi Kun Liu proposed a change to openstack/tempest: add volume list tests  https://review.openstack.org/4750801:04
*** Anju has quit IRC01:09
RecheckWatchBotopenstack/nova change: https://review.openstack.org/36078 failed tempest with an unrecognized error01:18
RecheckWatchBotopenstack/heat change: https://review.openstack.org/45462 failed tempest because of: https://bugs.launchpad.net/bugs/119196001:22
RecheckWatchBotopenstack/nova change: https://review.openstack.org/36078 failed tempest with an unrecognized error01:26
*** dkranz has joined #openstack-qa01:30
*** mriedem has joined #openstack-qa01:33
*** Anju has joined #openstack-qa02:58
*** Anju has quit IRC03:29
RecheckWatchBotopenstack/ceilometer change: https://review.openstack.org/46100 failed tempest because of: https://bugs.launchpad.net/bugs/122346903:39
*** ken1ohmichi has quit IRC03:40
*** jerryz has quit IRC03:43
*** Anju has joined #openstack-qa03:43
*** prekarat has joined #openstack-qa04:14
*** prekarat has quit IRC04:15
*** prekarat has joined #openstack-qa04:15
*** madan has joined #openstack-qa04:21
*** mriedem has quit IRC04:28
*** raies has joined #openstack-qa04:36
raiesanyone from neutron ?04:37
*** madan has quit IRC04:44
*** madan has joined #openstack-qa04:46
raiesafazekas: hi04:50
*** krtaylor has joined #openstack-qa04:55
raiesmkoderer: hi04:55
*** prekarat has quit IRC05:00
*** prekarat has joined #openstack-qa05:02
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas vpnservice test cases  https://review.openstack.org/4733905:12
*** yfried has quit IRC05:22
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests  https://review.openstack.org/4734805:24
openstackgerritMh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests  https://review.openstack.org/4734805:30
raiessdague: hi05:31
raieswill yu please check https://review.openstack.org/#/c/47335/05:32
RecheckWatchBotopenstack/tempest change: https://review.openstack.org/47339 failed tempest with an unrecognized error05:42
raiesRecheckWatchBot: As for I think it is failing due to not being implemented https://review.openstack.org/#/c/47335/05:46
raieswill someone from tempest, please check and approve https://blueprints.launchpad.net/tempest/+spec/vpnaas-api-test-cases05:51
RecheckWatchBotopenstack/tempest change: https://review.openstack.org/47348 failed tempest with an unrecognized error05:59
*** prekarat has quit IRC06:07
*** prekarat has joined #openstack-qa06:07
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: First cert attempt  https://review.openstack.org/4707606:20
*** ken1ohmichi has joined #openstack-qa06:21
*** jlibosva has joined #openstack-qa06:23
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates  https://review.openstack.org/4707606:27
*** matsuhashi has quit IRC06:27
*** matsuhashi has joined #openstack-qa06:28
RecheckWatchBotopenstack/cinder change: https://review.openstack.org/46694 failed tempest with an unrecognized error06:42
*** Madan1 has joined #openstack-qa06:51
*** thomasbiege has joined #openstack-qa07:08
*** thomasbiege has quit IRC07:08
*** CaptainMorgan is now known as morganfainberg07:19
*** giulivo has joined #openstack-qa07:37
*** yfried has joined #openstack-qa07:44
*** jamielennox is now known as jamielennox|away07:55
*** yfried has quit IRC07:55
*** yassine has joined #openstack-qa07:56
*** matsuhas_ has joined #openstack-qa08:03
*** matsuhashi has quit IRC08:03
*** arrsim` has quit IRC08:15
*** fbo_away is now known as fbo08:18
*** bashok has joined #openstack-qa08:26
*** matsuhas_ has quit IRC08:39
*** matsuhashi has joined #openstack-qa08:47
*** matsuhashi has quit IRC08:49
*** matsuhashi has joined #openstack-qa08:50
*** matsuhas_ has joined #openstack-qa08:51
*** matsuhashi has quit IRC08:55
*** mkollaro has joined #openstack-qa09:07
*** mkollaro has quit IRC09:12
*** mkollaro has joined #openstack-qa09:13
*** mkollaro has quit IRC09:18
*** mkollaro has joined #openstack-qa09:20
*** mkollaro has quit IRC09:28
*** mkollaro has joined #openstack-qa09:31
*** leizhang has joined #openstack-qa09:32
*** jhenner has joined #openstack-qa09:37
*** mkollaro1 has joined #openstack-qa09:37
*** mkollaro has quit IRC09:37
*** mkollaro1 has quit IRC09:42
*** matsuhas_ has quit IRC09:43
*** mkollaro1 has joined #openstack-qa09:43
*** mkollaro1 has quit IRC09:43
*** mkollaro has joined #openstack-qa09:45
*** mkollaro has quit IRC09:45
*** mkollaro has joined #openstack-qa09:47
*** leizhang has quit IRC09:56
*** leizhang has joined #openstack-qa09:56
*** masayukig has joined #openstack-qa10:14
*** nosnos has quit IRC10:17
*** fbo is now known as fbo_away10:24
*** fbo_away is now known as fbo10:24
openstackgerritCyril Roelandt proposed a change to openstack/tempest: Do not review this  https://review.openstack.org/4719410:25
*** ken1ohmichi has quit IRC10:32
openstackgerritAmala Basha proposed a change to openstack-dev/devstack: edit-glance-manage-command-for-recreate-db  https://review.openstack.org/4755210:59
*** masayukig has left #openstack-qa11:15
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: Add a create_router utility function  https://review.openstack.org/4755311:22
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: API tests for neutron router gateway  https://review.openstack.org/4755411:22
*** Madan1 has quit IRC11:25
*** madan has quit IRC11:25
openstackgerritA change was merged to openstack/tempest: Skip test tokens  https://review.openstack.org/4634112:14
*** weshay has joined #openstack-qa12:27
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: Enable tenant connectivity check over unreachable nets  https://review.openstack.org/4690012:27
openstackgerritA change was merged to openstack/tempest: Remove wait_for_image_resp_code  https://review.openstack.org/3899512:30
openstackgerritA change was merged to openstack/tempest: Enable the quota test cases  https://review.openstack.org/3315212:30
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Added Negative tests for image member  https://review.openstack.org/4716712:33
*** sthaha has quit IRC12:48
*** adalbas has joined #openstack-qa12:53
*** hootjr has joined #openstack-qa12:55
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Added Negative tests for image member  https://review.openstack.org/4716713:06
*** mriedem has joined #openstack-qa13:13
*** jlibosva has quit IRC13:16
*** fbo has quit IRC13:18
*** fbo has joined #openstack-qa13:18
*** jasondotstar has joined #openstack-qa13:28
openstackgerritA change was merged to openstack/tempest: Fix misused assertTrue in unit tests  https://review.openstack.org/4685313:41
*** russellb is now known as rustlebee13:48
*** prekarat has quit IRC14:11
openstackgerritA change was merged to openstack/tempest: Adds tests covering Swift's container quotas middleware  https://review.openstack.org/4623214:12
*** thomasbiege1 has joined #openstack-qa14:23
*** thomasbiege1 has quit IRC14:24
*** malini has joined #openstack-qa14:29
*** giulivo has quit IRC14:30
*** giulivo has joined #openstack-qa14:31
*** gfidente has joined #openstack-qa14:36
*** giulivo has quit IRC14:38
openstackgerritA change was merged to openstack/tempest: Remove claim that scenario tests need 2 services  https://review.openstack.org/4706714:55
*** RecheckWatchBot has quit IRC15:06
*** wramthun has quit IRC15:17
*** wramthun has joined #openstack-qa15:22
*** RecheckWatchBot has joined #openstack-qa15:26
*** mkollaro has quit IRC15:30
RecheckWatchBotopenstack/horizon change: https://review.openstack.org/47584 failed tempest with an unrecognized error15:31
*** openstackgerrit has quit IRC15:33
*** openstackgerrit has joined #openstack-qa15:33
*** yassine has quit IRC15:35
RecheckWatchBotopenstack/keystone change: https://review.openstack.org/46613 failed tempest because of: https://bugs.launchpad.net/bugs/121191515:42
*** mkollaro has joined #openstack-qa15:49
*** mkollaro has quit IRC15:49
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add locking to test_list_hosts_with_zone  https://review.openstack.org/4759915:51
mtreinishjog0: ^^ a new bug caught by elasticRecheck :)15:52
*** jeblair is now known as corvus15:53
*** corvus is now known as jeblair16:00
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Sync install_venv_common.py from oslo-incubator  https://review.openstack.org/4760216:01
RecheckWatchBotopenstack/nova change: https://review.openstack.org/47114 failed tempest because of: https://bugs.launchpad.net/bugs/119196016:08
*** dims has joined #openstack-qa16:09
dimsmtreinish, Hi, Got a few mins for a tempest review? Please let me know. Taking a stab at something that is triggering a lot of rechecks - https://review.openstack.org/#/c/47494/16:10
*** thomasbiege1 has joined #openstack-qa16:13
RecheckWatchBotopenstack/horizon change: https://review.openstack.org/47584 failed tempest with an unrecognized error16:13
dimsannegentle, i have a review for the FontTriplet/Jenkins issue too (https://review.openstack.org/#/c/47509/) - not sure who i have to ask for review and on which channel16:13
*** thomasbiege1 has quit IRC16:16
*** bashok has quit IRC16:18
*** comstud is now known as bearhands16:24
openstackgerritAnju Tiwari proposed a change to openstack/tempest: Added Negative tests for image member  https://review.openstack.org/4716716:57
*** hemnafk is now known as hemna16:57
*** bashok has joined #openstack-qa16:57
*** fbo is now known as fbo_away17:04
mtreinishdims: sure give me a min though I just got back from lunch17:12
dimsmtreinish, thx!17:13
mtreinishdims: reviewed17:38
dimsmtreinish, thx will fix the commit message17:41
mtreinishdims: ok, but I'm still not comfortable pushing it through until we better characterize that bug.17:42
*** jasondotstar has quit IRC17:44
annegentledims: huh, do we ignore fop errors at our own peril though?17:44
dimsannegentle, we are just making sure the cache is empty, that's all17:45
dimsthe Fop 1.0 is 3-4 years old17:45
annegentledims: oh ok, and I see the pool for multiprocessing too17:45
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/tempest: Ensure server is in ACTIVE state, Fix for transient failures  https://review.openstack.org/4749417:46
mtreinishdims: ^^ my objection on the title wasn't the '-possible' but the whole fix for transient failures part :)17:47
mtreinishEnsure server is in ACTIVE state is enough for the commit subject17:48
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/tempest: Ensure server is in ACTIVE state  https://review.openstack.org/4749417:49
dimsthx again17:49
*** dkranz has quit IRC18:00
mtreinishjog0: one thing I've been noticing with elasticRecheck is that it spends a lot of time waiting for ES and I'm pretty sure that it drops other failures while it's waiting18:16
mtreinishI'm thinking we might want to break out each classify() call into it's own thread or process18:17
mtreinish(a thread probably makes more sense)18:18
*** thomasbiege has joined #openstack-qa18:20
mtreinishjog0: something like this: http://paste.openstack.org/show/47336/ (and an equivelent change for bot.py)18:33
*** thomasbiege has quit IRC18:36
*** jasondotstar has joined #openstack-qa18:40
*** leizhang has quit IRC18:55
clarkbmtreinish: or a queue at least19:03
mtreinishclarkb: yeah that would work too, but I still think there is a bug in the waiting code, because it's still waiting on: https://review.openstack.org/#/c/47387/219:12
mtreinishwhich failed quite some time ago19:12
clarkbgotcha19:13
*** RecheckWatchBot has quit IRC19:13
*** RecheckWatchBot has joined #openstack-qa19:14
*** weshay has quit IRC19:32
*** malini is now known as malini_afk19:33
openstackgerritClint Byrum proposed a change to openstack/tempest: Test that Heat max_template_size is applied  https://review.openstack.org/4573219:34
*** weshay has joined #openstack-qa19:48
morganfainbergclarkb, once a change has gone into tempest (merged) is there a delay (or are devstack gates take the latest tempest each run?) before it takes effect?20:03
morganfainbergclarkb, if you know.20:03
clarkbmorganfainberg: no delay20:04
clarkbmorganfainberg: in fact the change behind it in the queue is tested with the change (no need to wait for merging)20:04
morganfainbergclarkb, awesome, thanks.20:04
morganfainbergclarkb, helps me move towards getting RC1 bugs worked out :)20:05
RecheckWatchBotopenstack/keystone change: https://review.openstack.org/46207 failed tempest with an unrecognized error20:08
RecheckWatchBotopenstack/keystone change: https://review.openstack.org/47514 failed tempest with an unrecognized error20:11
jog0mtreinish: looking20:12
jog0mtreinish: I was under the assumption that the gerritlib queued changes20:13
jog0I think using a queue is better then adding threading in because its less complex20:14
mtreinishjog0: ok that's what I figured (and I agree too) This was just kind of a cheap fix to get around when we get stuck waiting.20:16
jog0mtreinish: and I do think there is something wrong with teh wait logic20:17
jog0not sure what yet though20:17
jog0looking now20:19
*** dims has quit IRC20:21
*** dims has joined #openstack-qa20:25
jog0mtreinish: about to board a flight so will have to pick this up in a few hours20:27
mtreinishjog0: sure no problem I'll take a quick look too before I head home in <1hr20:28
jog0oh right east coast, o/20:29
RecheckWatchBotopenstack/neutron change: https://review.openstack.org/47355 failed tempest because of: https://bugs.launchpad.net/bugs/121191520:31
*** bashok is now known as ashok298820:31
mikaljog0: so... I've now cycled back to that tempest failure from yesterday20:36
mikaljog0: what's the standard way forward where tempest is revealing a genuine bug in nova? Disable the tempest test and file a bug?20:36
RecheckWatchBotopenstack/swift change: https://review.openstack.org/45134 failed tempest with an unrecognized error20:41
*** hootjr has quit IRC20:45
*** ashok2988 has quit IRC20:47
*** bdperkin is now known as bdperkin_gone20:53
*** weshay has quit IRC21:00
*** malini_afk is now known as malini21:04
*** rfolco has quit IRC21:16
*** EmilienM has quit IRC21:22
openstackgerritEoghan Glynn proposed a change to openstack-dev/devstack: Modified ceilometer alarm evaluator console script  https://review.openstack.org/4763721:23
*** EmilienM has joined #openstack-qa21:23
*** mriedem has quit IRC21:33
sdaguemikal: is this an always fail, or a sometimes fail?21:34
mikalsdague: well, its a little complicated21:34
mikalI just fixed a bug which was ignoring failures to inject mandatory data into images21:35
mikalWhich uncovered a persistant failure in tempest, which previously wasn't fatal to the test but should have been21:35
mikal(i.e. an instance is trying to inject data into a non-existant image because its a boot from volume instance)21:35
sdaguemikal: the problem with disabling tests in tempest... is they tend to stay that way for 6 months21:35
mikalSo, my patch didn't break this test21:35
mikalBut it did reveal a previously masked bug21:36
sdagueso it would be much better if we could get the main issue fixed instead21:36
mikalWhich is sort-of-related21:36
mikalI'm not keen on blocking my current bug fix because it revealed another bug though21:36
mikalI agree the other bug should be fixed, but I'd like to see that done in parallel21:36
mikalPartially because I think we need a mailing list thread here?21:37
mikalShould file inject work on boot from volume? I'm undecided.21:37
sdaguesure, why don't we take it there, and agree on path forward, including the tempest side21:37
mikalWell, if we filed a bug for the newly revealed failure, and added a skip for this tempest test21:37
mikalThen I would assign that bug to myself immediate and start the tread21:37
mikals/immediate/immediately/21:38
mikals/tread/thread/21:38
mikalsigh21:38
mikalI think its a pretty big deal that file injection currently silently fails21:38
sdagueagreed21:38
sdaguepush all the reviews, get a thread going. I'll jump on it monday ... also why you working sat morning :)21:39
mikalBecause I'm a loser21:39
mikalWell, the review for the inject fix is out there now, but fails tempest so will never land21:40
mikalhttps://review.openstack.org/#/c/46867/21:40
sdaguehmmm... so if I look at that fail it seems to indication that boot from volume no longer works after that patch. is that correct?21:43
mikalBoot from volume where file injection is configured will no longer work.21:43
mikalBut it wasn't working already21:43
mikali.e. an instance without its injected files is kind of useless21:43
mikal(no root password, no network config, etc etc)21:44
sdaguenot if it is cloud initing21:44
mikalI'd be comfortable with a world in which we said to use config drive or metadata server for this instance type21:44
mikalThat's true, but then why is injection turned on?21:44
mikalOk, I have a draft email I will send now21:47
mikalYou're cc'ed21:47
sdagueok21:48
*** adalbas has quit IRC21:54
*** notmyname has joined #openstack-qa21:57
notmynameI have a tempest question21:57
notmynameor maybe just a request for more eyes21:57
notmynameanyone here who can assist with looking at logs on a failing test run?21:58
sdaguenotmyname: fire away, though it's friday evening, so I'm going to depart soon for a drink21:58
notmynamesdague: don't worry, this will make you want to drink more ;-)21:58
sdagueheh21:58
notmynamesdague: http://logs.openstack.org/34/45134/8/check/gate-tempest-devstack-vm-postgres-full/4006575/console.html  <-- failed test21:59
notmynamelooks like a swift bug, right?21:59
notmynameok, so look into the swift proxy logs: http://logs.openstack.org/34/45134/8/check/gate-tempest-devstack-vm-postgres-full/4006575/logs/screen-s-proxy.txt.gz (search for TestContainer189244130)21:59
notmynameand you'll see that swift is returning an error21:59
notmynameie trying to create a container, but getting a 40421:59
notmynamethis happens when the account doesn't exist22:00
sdagueok22:00
notmynamebut if you look at the line(s) right above, you'll see that the account was created22:00
notmynameand for further wierdness, http://logs.openstack.org/34/45134/8/check/gate-tempest-devstack-vm-neutron/f3e1ad1/console.html passed the same test on the same patch22:00
notmyname(search for http://logs.openstack.org/34/45134/8/check/gate-tempest-devstack-vm-neutron/f3e1ad1/console.html)22:00
notmynameerr...22:00
notmynamesearch for tempest.api.object_storage.test_object_services.PublicObjectTest.test_access_public_container_object_without_using_creds22:01
notmynameso it passes on one, fails on the other (for the same swift patch)22:01
notmynameand the swift logs say that it's doing the right thing (other than the final 404)22:01
sdagueso those tests may very well be executing at exactly the same time22:02
notmynamelooking at the transaction id (tx3ff8dd5a2b1b4f4dbfc70-00523cbd62) for the failing test in the other logs (ie account logs), you'll see that it all looks fine22:02
sdaguecould there be an issolation issue22:02
notmynameperhaps (and perhaps likely), let me look at one more thing22:03
sdagueespecially if cleanup wasn't issolated correctly22:04
sdaguewhich would be my wild guess22:04
notmynamethere is no delete for the account logged, so I don't think it's that22:04
sdagueok22:04
notmynameFWIW, AUTH_f334ca5f79a94192966fe3833c947d85 is the account string to search for22:04
sdaguenotmyname: ok, in which logs22:05
notmynameany of the swift ones22:05
sdaguecontainer or object22:05
sdagueok22:05
notmynameok, the account server PUT request for that account (ie when it's created) has the same transaction id as the failing test (tx3ff8dd5a2b1b4f4dbfc70-00523cbd62)22:06
notmynamesdague: anything else you can tell me about how these tests are run? any diferences between the one that passes and the one that fails?22:10
sdagueactually, they are in the same class, so they aren't running in parallel22:11
*** gfidente has quit IRC22:11
notmynameah, cool good to know22:12
sdaguewe are running parallel by class in tempest, on 4 cores. But testr distributes them somewhat arbitrarily22:13
morganfainbergany tempest folks around that have a few to step me through a couple things?  I'm struggling to understand some of the identity setup stuff (e.g. create_user) and chase down possibly the mishmash of tenantId, project_id, default_project_id that seems to be referencing items on the user that may be incorrect.22:13
sdagueso there may be other object tests running at the same time22:13
sdagueso all tests in a class will be in on testr process, but different test classes could be in different processes22:14
morganfainbergspecifically, is http://git.openstack.org/cgit/openstack/tempest/tree/tempest/services/identity/v3/json/identity_client.py#n50 supposed to be the default project?22:15
*** dims has quit IRC22:16
sdagueanyway, time for me to run off to dinner, and I think most folks here are east coast or .eu, so they are probably at the bar or asleep atm22:20
notmynamesdague: thanks22:25
notmynamesdague: we're still looking over in -swift22:25
*** dims has joined #openstack-qa22:30
*** openstack has quit IRC22:35
*** openstack has joined #openstack-qa22:35
*** malini is now known as malini_afk22:36
*** adalbas has joined #openstack-qa23:15
*** hemna is now known as hemnafk23:27
openstackgerritSalvatore Orlando proposed a change to openstack-dev/devstack: Nicira plugin: do not die if NVP gateway IP is missing  https://review.openstack.org/4741423:31
*** dims has quit IRC23:59

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