Friday, 2013-10-04

*** bdperkin_gone has joined #openstack-qa00:24
*** bdperkin_gone is now known as bdperkin00:24
jog0clarkb: looks like the bot is failing to post to gerrit00:27
jog0if you click on https://review.openstack.org/49644 there is no comment there00:27
*** matsuhashi has joined #openstack-qa00:28
clarkbjog0: so it detects that a comment is needed, then that thread bombs preventing any future reports? (makes sense as we got unrecognized errors then finally when it recognizes something it dies)00:29
jog0clarkb: yeah00:29
jog0something like that00:29
clarkbjog0: I think I see the issue. patch incoming00:32
jog0stream.leaveCommnt00:32
jog0woot!00:32
*** salv-orlando has quit IRC00:32
*** salv-orlando has joined #openstack-qa00:33
jog0teamwork FTW00:34
openstackgerritClark Boylan proposed a change to openstack-infra/elastic-recheck: Leave comment when single bug is found.  https://review.openstack.org/4965600:39
clarkbjog0: ^ I actually haven't run tests for that yet as I wasn't sure what the test situation looks like. The actual fix is simple though00:40
jog0the tests are a little funny right now00:41
jog0so I am fine with it00:41
*** nosnos has joined #openstack-qa00:45
*** anteaya has quit IRC00:55
openstackgerritA change was merged to openstack-infra/elastic-recheck: Leave comment when single bug is found.  https://review.openstack.org/4965600:58
*** masayukig has left #openstack-qa00:58
jog0clarkb: e-r will now need hopefully its last kick01:02
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some negative tests for flavor  https://review.openstack.org/4958801:03
clarkbjog0 ok. I just witnessed a car accident so will be a bit later until I can do that woo01:07
*** kong has joined #openstack-qa01:15
*** mriedem has joined #openstack-qa01:15
*** yeylon_ has quit IRC01:16
*** rfolco has quit IRC01:37
*** jasondot_ has joined #openstack-qa01:40
jog0clarkb: ouch01:45
clarkbthankfully everyone was fine. Now I can see about kicking that bot01:52
*** openstackrecheck has quit IRC01:55
*** openstackrecheck has joined #openstack-qa01:55
clarkbjog0: bot restarted thanks to fungi01:55
fungiyup01:56
fungicar accident on the walk home? eek01:56
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Clean up existing instances when not using tenant isolation  https://review.openstack.org/4943101:56
clarkbfungi: yup fun times01:57
kongcould anyone help to review https://review.openstack.org/#/c/49588/? Thanks very much!01:58
*** jasondot_ has quit IRC02:00
openstackgerritJamie Lennox proposed a change to openstack-dev/grenade: Source lib/tls in upgrade-keystone  https://review.openstack.org/4966202:01
mriedemkong: done02:02
mriedemseems pretty straight-forward02:02
kongmriedem, thanks! I'm a beginner, writing some simple tests is good for me, :)02:03
mriedemkong: well in that case i'm sure someone will come along and no doubt find a reason to -1 it :)02:05
kongI hope so, and I found there is lack of tests in fact for projects including Nova, Cinder, Neutron.02:06
clarkbif you want me to nit pick ... :) I am a fan of verbose commit messages. Listing the tests isn't very informative without more info. But I don't typically review tempest changes so don't want to step on any accepted practices02:07
kongclarkb, good suggestion!02:08
openstackgerritJamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates  https://review.openstack.org/4707602:11
openstackrecheckopenstack/neutron change: https://review.openstack.org/49644 failed tempest because of: https://bugs.launchpad.net/bugs/122400102:18
*** mriedem has quit IRC02:18
clarkbjog0: comment was left on that change. fingers crossed that it works now02:18
*** bdperkin has quit IRC02:24
*** bdperkin has joined #openstack-qa02:24
openstackrecheckopenstack/nova change: https://review.openstack.org/49660 failed tempest with an unrecognized error02:29
jog0looks like it is02:30
*** giulivo has quit IRC02:44
*** jamielennox is now known as jamielennox|away02:45
*** sourabh has joined #openstack-qa03:02
*** prekarat has joined #openstack-qa03:36
*** prekarat has quit IRC03:41
*** dkranz has joined #openstack-qa03:43
openstackgerritA change was merged to openstack-dev/grenade: Install the new schema-image.json  https://review.openstack.org/4588403:54
*** prekarat has joined #openstack-qa03:54
raieshi04:40
raiesanybody who can help me on tempest04:40
*** DennyZhang has joined #openstack-qa04:53
*** Madan has joined #openstack-qa04:56
openstackgerritA change was merged to openstack/tempest: Do not check for id in the keystone output  https://review.openstack.org/4901704:58
openstackrecheckopenstack/nova change: https://review.openstack.org/49660 failed tempest with an unrecognized error05:05
*** afazekas has joined #openstack-qa05:06
*** turul_ has joined #openstack-qa05:06
*** jhenner has joined #openstack-qa05:23
openstackgerritMh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler  https://review.openstack.org/4863205:25
*** turul_ has quit IRC05:26
*** afazekas has quit IRC05:26
*** afazekas has joined #openstack-qa05:26
raiesafazekas: hi05:29
openstackgerritRyan Hsu proposed a change to openstack-dev/devstack: Fix typo in property passed to glance  https://review.openstack.org/4967405:29
*** shilpa_ has joined #openstack-qa05:30
raiesis there any way to make my code dependent of two different codes05:30
raies?05:30
*** DennyZhang has quit IRC05:32
afazekasraies: I am not sure, may be if you create merge commit with 2 dependent change, and re-base your change to the merge05:34
clarkbgerrit typically won't accept merge commits. instead you have to make the other two dependent on one or the other05:35
clarkb(assuming code will eventually be pushed to gerrit)05:35
raiesclarkb: I did not understand. Please elaborate a bit. if you mean to say that add code B dependent of A and then add code C dependent of B. ?05:39
raiesmy demand was to add code C which is dependent of A and B both but A and B are not dependent.05:40
*** SergeyLukjanov has joined #openstack-qa05:40
clarkbraies yes A <- B <- C05:42
*** Madan has quit IRC05:42
clarkbraies: I understand, afazekas' suggestion would do it but then you can't push C to gerrit05:42
raiesafazekas, clarkb: then is there any alternative way to fulfill my demand ?05:45
openstackrecheckopenstack/tempest change: https://review.openstack.org/48792 failed tempest because of: https://bugs.launchpad.net/bugs/122400105:48
*** SergeyLukjanov has quit IRC05:54
*** matsuhashi has quit IRC06:01
afazekasraies: which change is A and B now ?06:08
*** nosnos has quit IRC06:08
*** jlibosva has joined #openstack-qa06:09
*** HenryG has quit IRC06:12
*** HenryG has joined #openstack-qa06:12
*** matsuhashi has joined #openstack-qa06:24
raiesafazekas: A and B are https://review.openstack.org/#/c/48220/ and https://review.openstack.org/#/c/46050/ respectively06:25
raiesand C is https://review.openstack.org/#/c/48632/06:26
openstackrecheckopenstack/cinder change: https://review.openstack.org/49679 failed tempest with an unrecognized error06:26
raiessorry C is yet to commit06:27
raiesafazekas: ^^06:27
*** Dharmit has joined #openstack-qa06:28
raiesafazekas: https://review.openstack.org/#/c/48632/ is the code what I have incorporated with your review comment. You are also requested to check this.06:28
*** Madan has joined #openstack-qa06:32
openstackrecheckopenstack/nova change: https://review.openstack.org/49677 failed tempest with an unrecognized error06:32
raiesafazekas: are you there ?06:39
*** yeylon_ has joined #openstack-qa06:48
openstackrecheckopenstack/neutron change: https://review.openstack.org/49683 failed tempest because of: https://bugs.launchpad.net/bugs/122400106:48
openstackrecheckopenstack/neutron change: https://review.openstack.org/49682 failed tempest because of: https://bugs.launchpad.net/bugs/122400106:49
openstackrecheckopenstack/nova change: https://review.openstack.org/49660 failed tempest with an unrecognized error06:50
afazekasraies: I just said if the raise 'cls.skipException(msg)' clearing the isolated credentials is recommend, but looks like the isolated credentials was not used by the network tests06:51
afazekasraies: IMHO you do not need to add it by this change06:52
raiesafazekas: Yes no isolated credentials was used for networks test06:53
*** nosnos has joined #openstack-qa06:54
raiesthen should I use base.py file what I had used in earlier patch06:54
raies?06:54
afazekasraies: yes06:54
raiesafazekas: ok I will apply next patch06:55
raiesafazekas: please have a look on https://bugs.launchpad.net/tempest/+bug/123502906:56
openstackgerritafazekas proposed a change to openstack-dev/devstack: lib/swift variable changes and dd replaced by truncate  https://review.openstack.org/4666406:58
openstackgerritMh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler  https://review.openstack.org/4863207:06
afazekasraies: I guess the allow_overlapping_ips = True is missing from the neutron conf07:13
openstackrecheckopenstack/cinder change: https://review.openstack.org/49679 failed tempest with an unrecognized error07:14
openstackrecheckopenstack/neutron change: https://review.openstack.org/45725 failed tempest with an unrecognized error07:14
raiesafazekas: it my environment in the file "/etc/neutron/neutron.conf"07:16
raiesas per default setting ==>  allow_overlapping_ips = True07:17
afazekashttps://bugs.launchpad.net/tempest/+bug/123502907:17
afazekasFIXME but the bulk subnet creation  usues a network which does not used by any other test class07:18
raiesI didn't get what are you saying07:19
afazekasFIXME if the  allow_overlapping_ips = True (and you have namespaces) you can use the same cidr in two different net07:19
*** fbo_away is now known as fbo07:19
afazekasraies: I guess when the exception happened with #1235029 the  allow_overlapping_ips was False07:20
*** ken1ohmichi has joined #openstack-qa07:21
raiesthis thing I am getting, but I wanted to know what is FIXME ?07:21
raiesis it a keyword to convey message in comments during bug ?07:21
raiesafazekas: thus as you seggested, it is a environment bug right ?07:23
afazekasFIXME .. correct me if I am wrong07:23
afazekasraies: yes it can be07:23
raiesafazekas: thanks07:24
openstackrecheckopenstack/cinder change: https://review.openstack.org/49685 failed tempest with an unrecognized error07:28
raiesafazekas: you had asked a question "raies: which change is A and B now ?"07:32
afazekasraies: and you have answered it07:39
raiesyes :)07:39
openstackrecheckopenstack/cinder change: https://review.openstack.org/49686 failed tempest with an unrecognized error07:41
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for aggregate  https://review.openstack.org/4915707:52
openstackrecheckopenstack/cinder change: https://review.openstack.org/49688 failed tempest with an unrecognized error07:55
openstackrecheckopenstack/neutron change: https://review.openstack.org/49682 failed tempest because of: https://bugs.launchpad.net/bugs/122400107:55
*** giulivo has joined #openstack-qa08:09
*** yassine has joined #openstack-qa08:15
openstackrecheckopenstack/tempest change: https://review.openstack.org/49157 failed tempest with an unrecognized error08:24
openstackgerritMehdi Abaakouk proposed a change to openstack-dev/devstack: force run_process to really spawn a bash process  https://review.openstack.org/4939408:25
openstackrecheckopenstack/requirements change: https://review.openstack.org/49696 failed tempest with an unrecognized error08:33
openstackrecheckopenstack/neutron change: https://review.openstack.org/49424 failed tempest with an unrecognized error08:35
mkodererafazekas: about https://review.openstack.org/#/c/42920/08:42
mkodererafazekas: I would like to have at least some logging and maybe some failure handling08:42
mkodererafazekas: or what do you think?08:42
openstackgerritMehdi Abaakouk proposed a change to openstack/tempest: Adds initial ceilometerclient testing code  https://review.openstack.org/4970008:47
openstackrecheckopenstack/neutron change: https://review.openstack.org/49596 failed tempest because of: https://bugs.launchpad.net/bugs/122400108:48
openstackgerritMehdi Abaakouk proposed a change to openstack/tempest: Adds initial ceilometerclient testing code  https://review.openstack.org/4970008:49
*** comstud has quit IRC08:50
afazekasmkoderer: it has a long TODO list already, it can be added later08:50
mkodererafazekas: ok I'll put my nits and give a +108:51
afazekasAFAIK tkammer will be long-long vacation soon08:51
afazekasmkoderer: ok08:51
openstackgerritMate Lakat proposed a change to openstack-dev/devstack: xenapi: max out VCPU count  https://review.openstack.org/4970309:02
openstackgerritMehdi Abaakouk proposed a change to openstack/tempest: Adds initial ceilometerclient testing code  https://review.openstack.org/4970009:05
openstackrecheckopenstack/neutron change: https://review.openstack.org/48999 failed tempest because of: https://bugs.launchpad.net/bugs/122400109:06
openstackrecheckopenstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/122400109:12
*** mkollaro has joined #openstack-qa09:13
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for aggregate  https://review.openstack.org/4915709:16
openstackgerritafazekas proposed a change to openstack-infra/devstack-gate: unbroke stable gate (chown)  https://review.openstack.org/4970509:20
*** SergeyLukjanov has joined #openstack-qa09:20
*** Dharmit has quit IRC09:25
*** fbo is now known as fbo_away09:26
raiesafazekas: please have a look on https://review.openstack.org/#/c/48632/09:29
openstackrecheckopenstack/tempest change: https://review.openstack.org/49700 failed tempest with an unrecognized error09:36
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: port test_flavors into v3 part1  https://review.openstack.org/4970909:43
openstackrecheckopenstack/tempest change: https://review.openstack.org/49157 failed tempest with an unrecognized error09:54
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Fix to use proper random values  https://review.openstack.org/4688610:01
openstackrecheckopenstack/neutron change: https://review.openstack.org/48999 failed tempest because of: https://bugs.launchpad.net/bugs/122400110:05
*** fbo_away is now known as fbo10:10
openstackrecheckopenstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/122400110:12
*** SergeyLukjanov has quit IRC10:18
*** SergeyLukjanov has joined #openstack-qa10:19
openstackgerritCyril Roelandt proposed a change to openstack/tempest: Neutron metering agent: add CRUD tests  https://review.openstack.org/4971910:22
*** prekarat has quit IRC10:24
*** matsuhashi has quit IRC10:28
openstackgerritGiulio Fidente proposed a change to openstack/tempest: pass stop_on_error to _has_error_in_logs  https://review.openstack.org/4972110:32
*** ken1ohmichi has quit IRC10:36
openstackrecheckopenstack/tempest change: https://review.openstack.org/49157 failed tempest with an unrecognized error11:09
openstackrecheckopenstack/neutron change: https://review.openstack.org/49596 failed tempest because of: https://bugs.launchpad.net/bugs/122400111:19
giulivoping mkoderer11:19
giulivoping cyeoh11:20
giulivomkoderer, cyeoh I'm updating the name, thanks for reviewing11:24
*** Madan has quit IRC11:25
openstackgerritGiulio Fidente proposed a change to openstack/tempest: introduces skip_because decorator  https://review.openstack.org/4812111:26
*** nosnos has quit IRC11:32
openstackrecheckopenstack/nova change: https://review.openstack.org/44238 failed tempest with an unrecognized error11:44
openstackgerritJoe Mills proposed a change to openstack-dev/devstack: Change Midonet vif driver to generic  https://review.openstack.org/4973311:48
*** weshay has joined #openstack-qa12:01
*** afazekas has quit IRC12:01
openstackrecheckopenstack/neutron change: https://review.openstack.org/49596 failed tempest because of: https://bugs.launchpad.net/bugs/122400112:02
*** afazekas has joined #openstack-qa12:14
*** lynnkong has joined #openstack-qa12:21
*** kong has quit IRC12:24
*** rfolco has joined #openstack-qa12:25
openstackgerritA change was merged to openstack-dev/devstack: Add back rpc_notifier when ceilometer is enabled  https://review.openstack.org/4835112:37
openstackrecheckopenstack/neutron change: https://review.openstack.org/47557 failed tempest because of: https://bugs.launchpad.net/bugs/122400112:45
*** SergeyLukjanov has quit IRC12:48
openstackrecheckopenstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/122400112:49
openstackrecheckopenstack/python-glanceclient change: https://review.openstack.org/48850 failed tempest with an unrecognized error12:52
*** SergeyLukjanov has joined #openstack-qa13:10
*** adalbas has joined #openstack-qa13:14
*** mriedem has joined #openstack-qa13:16
*** shilpa_ has quit IRC13:16
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for aggregate  https://review.openstack.org/4915713:18
*** lynnkong has left #openstack-qa13:18
*** lynnkong has joined #openstack-qa13:18
*** lynnkong has quit IRC13:19
*** sthaha has quit IRC13:21
*** kong has joined #openstack-qa13:21
*** sthaha has joined #openstack-qa13:21
*** sthaha has joined #openstack-qa13:21
*** sthaha has quit IRC13:27
*** prekarat has joined #openstack-qa13:36
*** prekarat has quit IRC13:37
*** prekarat has joined #openstack-qa13:39
giulivoping cyeoh13:41
giulivowanted to ask about this https://review.openstack.org/#/c/4750813:41
giulivoI'm still unsure if we should ignore those, suggest to remove, to update or not to remove13:41
openstackrecheckopenstack/tempest change: https://review.openstack.org/47337 failed tempest with an unrecognized error13:43
openstackrecheckopenstack/neutron change: https://review.openstack.org/49748 failed tempest with an unrecognized error13:43
openstackgerritGiulio Fidente proposed a change to openstack/tempest: introduces skip_because decorator  https://review.openstack.org/4812113:48
Anjugiulivo:  what is restore_delete13:50
Anju?13:50
Anjuafazekas: ping13:52
kongAnju, resotore a soft-deleted server13:55
Anjukong: is any api for this?13:56
kongyes13:56
kongAnju, http://api.openstack.org/api-ref-compute.html13:57
kongAnju, please search 'restore'13:57
konghttps://review.openstack.org/#/c/49488/13:58
Anjuafter deletion can we store the vm ?14:01
*** prekarat has quit IRC14:05
konganju, unless 'reclaim_instance_interval' is set14:06
openstackrecheckopenstack/cinder change: https://review.openstack.org/49755 failed tempest with an unrecognized error14:07
*** russellb is now known as rustlebee14:11
*** anteaya has joined #openstack-qa14:13
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Fix to use proper random values  https://review.openstack.org/4688614:14
openstackrecheckopenstack/cinder change: https://review.openstack.org/49679 failed tempest with an unrecognized error14:20
openstackrecheckopenstack/neutron change: https://review.openstack.org/47557 failed tempest because of: https://bugs.launchpad.net/bugs/122400114:20
openstackrecheckopenstack/neutron change: https://review.openstack.org/46927 failed tempest because of: https://bugs.launchpad.net/bugs/122400114:20
openstackrecheckopenstack/neutron change: https://review.openstack.org/49748 failed tempest because of: https://bugs.launchpad.net/bugs/122400114:21
afazekassdague: https://review.openstack.org/#/c/49705/14:34
*** jlibosva has quit IRC14:37
giulivoping afazekas14:40
afazekasgiulivo: pong14:40
giulivoI spotted a few things here https://review.openstack.org/#/c/4863214:40
giulivolike cls.network_cfg defined twice in base14:40
giulivoalso I don't think there is any need for the setUpClass in the test class?14:41
giulivoand last but not least, the methods added in network_client are valuable but maybe should come with some tests?14:41
afazekasgiulivo: you are correct14:42
afazekasraies: ^14:42
giulivooh that's the nick, thanks14:43
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Remove unused CONF variable  https://review.openstack.org/4976314:43
*** afazekas is now known as afazekas_zz14:47
* afazekas_zz self check failed --> go rest14:47
openstackgerritLingxian Kong proposed a change to openstack/tempest: add a negative test for flavor_extra_specs  https://review.openstack.org/4957714:51
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Remove unused CONF variable and import statement  https://review.openstack.org/4976314:56
dims"2013-10-04 14:54:20.479 | chown: cannot access `/opt/stack/data/tempest': No such file or directory" <<<<==== is this a known issue?14:57
dims3 check jobs in https://review.openstack.org/#/c/49371/ hit it at the same time14:58
*** mlavalle has joined #openstack-qa15:06
openstackrecheckopenstack/nova change: https://review.openstack.org/49371 failed tempest with an unrecognized error15:08
openstackgerritLingxian Kong proposed a change to openstack/tempest: add two negative tests for flavor-access  https://review.openstack.org/4950115:10
openstackrecheckopenstack/nova change: https://review.openstack.org/44238 failed tempest with an unrecognized error15:13
openstackgerritJeremy Stanley proposed a change to openstack-infra/devstack-gate: Make sure tempest directories exist when modifying  https://review.openstack.org/4977215:18
openstackrecheckopenstack/tempest change: https://review.openstack.org/49577 failed tempest with an unrecognized error15:32
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for host  https://review.openstack.org/4977815:39
*** yassine has quit IRC15:41
*** krtaylor has quit IRC15:43
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for host  https://review.openstack.org/4977815:51
openstackgerritLingxian Kong proposed a change to openstack/tempest: add two negative tests for flavor-access  https://review.openstack.org/4950115:53
openstackgerritLingxian Kong proposed a change to openstack/tempest: add some tests for host  https://review.openstack.org/4977815:56
*** mkollaro has quit IRC15:57
openstackrecheckopenstack/cinder change: https://review.openstack.org/49688 failed tempest with an unrecognized error15:59
*** yeylon_ has quit IRC16:06
openstackrecheckopenstack/cinder change: https://review.openstack.org/49685 failed tempest with an unrecognized error16:12
*** dims has quit IRC16:15
*** SergeyLukjanov has quit IRC16:23
*** krtaylor has joined #openstack-qa16:23
*** mlavalle has quit IRC16:23
*** mlavalle has joined #openstack-qa16:23
*** kong has quit IRC16:24
openstackrecheckopenstack/cinder change: https://review.openstack.org/49686 failed tempest with an unrecognized error16:26
openstackrecheckopenstack/python-glanceclient change: https://review.openstack.org/48850 failed tempest with an unrecognized error16:26
*** dims has joined #openstack-qa16:28
*** fbo is now known as fbo_away16:28
*** raies_ has joined #openstack-qa16:31
raies_afazekas: giulivo has given a minor comment in https://review.openstack.org/#/c/48632/5/tempest/services/network/json/network_client.py16:33
raies_afazekas_zz ^^16:34
raies_I have added these just for the sake so that I can add futher tess cases16:34
raies_*further16:34
raies_so should I put all these here ?16:35
raies_I think yes16:35
raies_afazekas_zz: what you say on this16:35
openstackgerritMh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler  https://review.openstack.org/4863216:39
openstackrecheckopenstack/cinder change: https://review.openstack.org/49684 failed tempest with an unrecognized error16:39
openstackrecheckopenstack/neutron change: https://review.openstack.org/49644 failed tempest because of: https://bugs.launchpad.net/bugs/122400116:39
openstackrecheckopenstack/neutron change: https://review.openstack.org/49782 failed tempest because of: https://bugs.launchpad.net/bugs/122400116:40
clarkbjog0: the bot seems to be working well :)16:40
jog0clarkb: yeah! lots of failed tempest jobs to check up on though16:44
jog0clarkb: and http://status.openstack.org/elastic-recheck/ is producing great results too16:47
*** jog0 is now known as flashgordon16:48
*** prekarat has joined #openstack-qa16:57
*** prekarat has quit IRC16:57
openstackrecheckopenstack/nova change: https://review.openstack.org/48557 failed tempest with an unrecognized error16:59
openstackrecheckopenstack/nova change: https://review.openstack.org/48196 failed tempest with an unrecognized error16:59
*** SergeyLukjanov has joined #openstack-qa17:03
openstackrecheckopenstack/cinder change: https://review.openstack.org/48935 failed tempest with an unrecognized error17:03
openstackrecheckopenstack/nova change: https://review.openstack.org/49623 failed tempest with an unrecognized error17:05
openstackrecheckopenstack/tempest change: https://review.openstack.org/48632 failed tempest with an unrecognized error17:08
*** fbo_away is now known as fbo17:16
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Remove query for fixed bug 1226791  https://review.openstack.org/4979517:19
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add message to assertNotEquals in test_valid_bugs  https://review.openstack.org/4979617:19
*** mlavalle has quit IRC17:29
openstackrecheckopenstack/nova change: https://review.openstack.org/49790 failed tempest with an unrecognized error17:33
clarkbflashgordon: we have a lot of unrecognized errors now. I assume that means your push to fix classified bugs is working :)17:48
mtreinishclarkb: I was just looking at a few of them. Some seem to be bad patches in the check queue17:48
clarkbmtreinish: that would explain it17:49
flashgordonclarkb mtreinish I am looking too17:49
flashgordonFound this in a few http://logstash.openstack.org/#eyJzZWFyY2giOiJAbWVzc2FnZTpcImZhaWxlZCB0byByZWFjaCBBQ1RJVkUgc3RhdHVzIHdpdGhpbiB0aGUgcmVxdWlyZWQgdGltZSAoNDAwIHMpLiBDdXJyZW50IHN0YXR1czogQlVJTERcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiODY0MDAiLCJncmFwaG1vZGUiOiJjb3VudCIsInRpbWUiOnsidXNlcl9pbnRlcnZhbCI6MH0sInN0YW1wIjoxMzgwOTA4NzUwNDU0fQ==17:49
mtreinishflashgordon: yeah I was just looking at one with that17:59
mtreinishI'm trying to figure out where it's going wrong17:59
flashgordonmtreinish: I vote open up a new bug for this17:59
flashgordonand add that query to elastic-recheck17:59
mtreinishthe state transition in the tempest log was BUILD/scheduling -> BUILD/none17:59
flashgordonand take it from there18:00
*** VinodGupta has joined #openstack-qa18:00
mtreinishflashgordon: I don't think that: "failed to reach ACTIVE status within the required time (400 s). Current status: BUILD" is specific enough18:00
mtreinishit applies to any vm that fails to come up18:00
mtreinishopen it against nova?18:00
flashgordonmtreinish: nova should never get hung in build state18:00
flashgordonyeeah18:00
flashgordonmtreinish: also we need to dedup http://status.openstack.org/rechecks/ again18:01
flashgordonthere are over a 100 bugs listed there pretty sure most are invalid or dupes18:01
mtreinishflashgordon: that's always a lot of fun18:01
* clarkb just realized that we should incorporate pruning rechecks and elastic-recheck to bug days18:03
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Split the creation of $STACK_USER account out of stack.sh  https://review.openstack.org/4979818:04
mtreinishclarkb: I basically spent the last tempest bug day just deduping bugs and cleaning up the recheck list18:05
flashgordonclarkb: we have a test for elastic-recheck to prune18:08
flashgordontwo infact18:08
openstackrecheckopenstack/nova change: https://review.openstack.org/49117 failed tempest with an unrecognized error18:10
flashgordongah there goes another18:11
*** fbo is now known as fbo_away18:15
flashgordonmtreinish: https://bugs.launchpad.net/tempest/+bug/122894218:15
flashgordonthats teh bug we want I think18:16
flashgordonhmm maybe not18:16
mtreinishflashgordon: yeah that looks similar18:16
mtreinishalthough it's a different test then the other fails18:16
flashgordonmaking a new bug will just be easier I htink18:17
mtreinishI don't like how that guy always puts swift in his bugs18:17
mtreinishflashgordon: yeah it probably is, and we can always set that one as a dup if it ends up being that way18:19
*** jhenner has quit IRC18:19
flashgordonyeah18:19
flashgordonmarking that one as incomplete as it looks like a bunch of bugs in one18:20
*** fbo_away is now known as fbo18:25
openstackgerritA change was merged to openstack-dev/devstack: Fix handling of pip and virtualenv on openSUSE  https://review.openstack.org/4922018:37
openstackgerritA change was merged to openstack-dev/devstack: Revert "Revert "Swift: configure Ceilometer when it is enabled""  https://review.openstack.org/4867818:40
mriedemopen tempest question, there are some tests that produce nice instance names like this "ServerActionsTestJSON-instance1052674456"18:48
mriedemthose are created through the BaseComputeTest.create_server18:48
mriedemsome create instances with names like this: "server1344500157"18:48
mriedemwhich isn't so helpful18:49
mriedemthose go through the servers client18:49
mriedemanyone opposed to changing the tests to use the BaseComputeTest.create_server rather than servers_client?18:49
mtreinishmriedem: so for the most part thats a good thing. But there are some cases where create_server() can't be used, because it does some things automatically like cleanup.18:51
mtreinishthere are also tests that create servers that aren't part of the compute tests18:51
mtreinishif you can work out the edge cases go for it18:51
mtreinishbecause it will probably prevent server leaks18:51
mriedemmtreinish: yeah, this would only be compute tests18:51
mriedemmtreinish: and yeah, i'm looking at this b/c of our conversation the other day about test_list_servers_negative failing on leaked instances18:52
mtreinishmriedem: yeah negative tests are one of the cases where using the base test case create gets tricky18:52
mtreinishbecause we normally expect things to fail18:53
mtreinishand the base test class create doesn't like that18:53
mriedemmtreinish: ListServerFiltersTestJSON is the one i'm looking at as an example18:53
mriedemlooks like it keeps track of it's own 3 class level instances and then on teardown deletes them explicitly18:54
mriedemwhere as the base class would handle that for you18:54
mtreinishmriedem: yeah that would be a good case to move that to the common create method.18:54
mtreinishit probably wasn't done originally because it wasn't leaking :)18:55
mriedemmtreinish: good news is i guess i only see 2 rotten apples right now18:56
mriedemtest_list_server_filters and test_volumes_actions (cinder test)18:56
mriedemthe cinder one could probably be dealt with differently18:56
mriedemmtreinish: do i need a bug for this cleanup?18:56
mtreinishmriedem: I'd be fine without one, but if you want to make one go ahead18:57
mtreinishsomeone might knock you for it though18:57
mriedemand that someone would be sdague!18:58
openstackrecheckopenstack/neutron change: https://review.openstack.org/45677 failed tempest because of: https://bugs.launchpad.net/bugs/1230407 and https://bugs.launchpad.net/bugs/1224001 and19:00
mtreinish^^^ I really like the multiple fails. I wonder what the commiter does in these cases19:01
openstackrecheckopenstack/neutron change: https://review.openstack.org/45678 failed tempest with an unrecognized error19:01
openstackrecheckopenstack/neutron change: https://review.openstack.org/49748 failed tempest because of: https://bugs.launchpad.net/bugs/122400119:02
raies_mtreinish: please have a look on https://review.openstack.org/#/c/48632/19:02
openstackrecheckopenstack/neutron change: https://review.openstack.org/45676 failed tempest because of: https://bugs.launchpad.net/bugs/122400119:03
mriedemgood lord19:05
mriedemhttps://bugs.launchpad.net/tempest/+bug/123541119:05
jgriffithmriedem: :)19:07
mtreinishmriedem: heh, I'm not sure the irc pastebin is neccessary but ok :)19:07
mriedemmtreinish: it must be there for all time19:07
mriedemmtreinish: actually that's so i don't forget about what you said about not fudging up cleanups19:07
* jgriffith get's some stone tablets19:07
mtreinishmriedem: well this channel is logged despite my objections about that19:08
mtreinishwhich I can probably look up in the logs :)19:08
*** mlavalle has joined #openstack-qa19:08
mriedemah, well then. i'm used to nova not being logged19:08
mtreinishmriedem: http://eavesdrop.openstack.org/irclogs/%23openstack-qa/19:09
* mriedem stashes conversations with dansmith to use against him later19:09
mriedemyup19:09
openstackgerritDavid Kranz proposed a change to openstack/tempest: Initial log check script to be called by devstack-gate  https://review.openstack.org/4980819:11
*** anteaya is now known as i_am_now_steve19:16
openstackgerritA change was merged to openstack-dev/devstack: set errexit and xtrace in helper scripts  https://review.openstack.org/4921519:17
openstackgerritA change was merged to openstack-dev/devstack: Using no proxy option to skip wget and curl proxy settings in config  https://review.openstack.org/4644719:17
*** jhenner has joined #openstack-qa19:19
*** i_am_now_steve is now known as anteaya19:23
openstackrecheckopenstack/neutron change: https://review.openstack.org/49086 failed tempest with an unrecognized error19:34
*** SergeyLukjanov has quit IRC19:43
flashgordonmtreinish: ping19:50
mtreinishflashgordon: pong19:50
flashgordonmtreinish: you file a bug for "failed to reach ACTIVE  status within the required time (400 s). Current status:  BUILD"19:51
mtreinishflashgordon: not yet, I got sidetracked on some other things19:52
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Cleanup test_list_server_filters setup/teardown  https://review.openstack.org/4981919:52
mtreinishflashgordon: let me take care of it right now19:52
flashgordoncool19:52
flashgordonthen we can merge the query and start tracking it19:52
mtreinishflashgordon: do you have a log link I can put in the bug?19:56
flashgordonmtreinish:  http://logs.openstack.org/23/49623/3/check/check-tempest-devstack-vm-full/b0d348a/testr_results.html.gz19:57
flashgordonfound with @message:"failed to reach ACTIVE status within the required time (400 s). Current status: BUILD"19:57
mtreinishflashgordon: ok thanks19:57
flashgordonmtreinish: thank you, filing another bug as we speak19:58
mtreinishflashgordon: https://bugs.launchpad.net/nova/+bug/123543719:59
mtreinishsure np19:59
mtreinishI should have done it earlier sorry about that19:59
flashgordonmtreinish: no problem can you add the query to e-r20:00
flashgordonso it pops on on the e-r bot and stats.o.o/e-r20:00
flashgordonstaatus*20:00
mtreinishsure one sec20:00
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Use predictable instance/volume names in test_volumes_actions  https://review.openstack.org/4982120:01
*** VinodGupta has quit IRC20:02
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add query for bug 1235435  https://review.openstack.org/4982220:02
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Use predictable instance/volume names in test_volumes_actions  https://review.openstack.org/4982120:02
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Add a query for bug 1235437  https://review.openstack.org/4982320:04
mtreinishflashgordon: ^^^20:04
flashgordonmtreinish: can you review my query patches as well20:04
flashgordonhttps://review.openstack.org/#/q/status:open+project:openstack-infra/elastic-recheck+branch:master+topic:queries,n,z20:04
mtreinishflashgordon: sure20:05
flashgordonhmm we both appended we may be forced to rebase but we can deal with it when it happens20:05
* mtreinish senses a merge conflict with the force20:05
mtreinishflashgordon: you can specify 2 messages in a query?20:07
flashgordonmtreinish: yeah heh20:08
flashgordonfigured that out with trial and error20:08
mtreinishflashgordon: heh, cool20:09
flashgordonmtreinish: there is also this one https://review.openstack.org/#/c/49795/20:14
flashgordonour first removal20:14
mtreinishflashgordon: oh cool a major milestone!20:15
flashgordonmtreinish: yeah I noticed from http://status.openstack.org/elastic-recheck/20:16
mtreinishflashgordon: those graphs are super useful20:17
mtreinishjeblair: ^^^ thanks20:17
openstackrecheckopenstack/neutron change: https://review.openstack.org/48468 failed tempest because of: https://bugs.launchpad.net/bugs/122400120:17
mtreinishflashgordon: we can probably pull out Bug 1226337 sometime next week20:19
mtreinishit doesn't look like it's happening anymore20:19
flashgordonmtreinish: yeah20:20
flashgordonhttps://review.openstack.org/#/c/49796/20:20
flashgordonI think we generally remove things only when they don't happen and are closed20:21
flashgordonbut not one or the other20:21
mtreinishwell I think it's closed already, but that's why I said next week to show a week trend of not occuring20:22
mtreinishflashgordon: yeah I guess a equals exception isn't exactly clear there20:22
flashgordonit wasn't at first glance20:23
openstackrecheckopenstack/tempest change: https://review.openstack.org/49819 failed tempest with an unrecognized error20:27
*** krtaylor has quit IRC20:31
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Cleanup test_list_server_filters setup/teardown  https://review.openstack.org/4981920:34
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Use predictable instance/volume names in test_volumes_actions  https://review.openstack.org/4982120:34
*** raies_ has quit IRC20:34
openstackrecheckopenstack/neutron change: https://review.openstack.org/49782 failed tempest because of: https://bugs.launchpad.net/bugs/122400120:42
openstackrecheckopenstack/cinder change: https://review.openstack.org/49824 failed tempest with an unrecognized error20:56
*** rfolco has quit IRC20:58
openstackrecheckopenstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/122400121:02
*** mkollaro has joined #openstack-qa21:12
*** fbo is now known as fbo_away21:14
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Remove general assumption in get_packages()  https://review.openstack.org/4983821:21
*** weshay has quit IRC21:24
*** mkollaro has quit IRC21:29
openstackgerritDavid Kranz proposed a change to openstack-infra/devstack-gate: Fail successful tempest runs if ERRORs in logs  https://review.openstack.org/4984021:29
dkranzsdague: ping21:31
mtreinishdkranz: he's out of the office today21:32
dkranzmtreinish: Ok. Do you know how devstack gate works with stable branches?21:33
dkranzmtreinish: Like in https://review.openstack.org/#/c/49840/1/devstack-vm-gate.sh,unified21:33
dkranzmtreinish: which I just pushed, is it a problem that the tools/check_logs.py does not exist on stable branches?21:33
mtreinishdkranz: it sets a flag to make devstack config changes for the stable branches: https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L4921:34
mtreinishI believe the code repos are prepopulated because of network reliability issues21:35
mtreinishso you might just want to check for the stable flags and skip what you're adding there21:35
dkranzmtreinish: ok, thanks.21:36
*** mriedem has quit IRC21:36
mtreinishdkranz: I would just make that a separate if block after that large job one and do the check 'if not stable and not stress' run the tool21:36
mtreinishit should be safe to run with coverage (it's just a full tempest run with some extra scripts running around it21:37
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Split the creation of $STACK_USER account out of stack.sh  https://review.openstack.org/4979821:37
dkranzmtreinish: I thought about that. Have to check that the tempest job succeeded first21:37
dkranzmtreinish: and I'm shell-challenged but will figure it out.21:37
dkranzmtreinish: It would be nice if we had a version number to compare rather than having to list all old stable branches.21:39
openstackgerritA change was merged to openstack-infra/elastic-recheck: Remove query for fixed bug 1226791  https://review.openstack.org/4979521:41
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add message to assertNotEquals in test_valid_bugs  https://review.openstack.org/4979621:41
mtreinishdkranz: well there are only 2 in there at any given time so it's not that bad21:42
mtreinishand it's actually a branch name that is being passed in from the git repo21:42
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for bug 1235435  https://review.openstack.org/4982221:42
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Add a query for bug 1235437  https://review.openstack.org/4982321:45
openstackrecheckopenstack/neutron change: https://review.openstack.org/49424 failed tempest with an unrecognized error21:47
giulivomtreinish, apparently I screwed up here https://review.openstack.org/#/c/4893322:03
*** krtaylor has joined #openstack-qa22:10
openstackrecheckopenstack-infra/devstack-gate change: https://review.openstack.org/49840 failed tempest with an unrecognized error22:11
giulivomtreinish, pretty badly actually22:13
*** jhenner has quit IRC22:14
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add query for bug 1235486  https://review.openstack.org/4984622:19
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: Avoid implying DevStack is POSIX-compatible  https://review.openstack.org/4984722:19
flashgordonsalv-orlando: ^^22:19
openstackgerritAdam Spiers proposed a change to openstack-dev/devstack: Fix "instal_prereqs.sh" typo  https://review.openstack.org/4984822:20
salv-orlandothanks flashgordon22:25
openstackrecheckopenstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/122400123:22
*** hemna_ is now known as hemnafk23:29
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add a query for bug 1235437  https://review.openstack.org/4982323:40

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