*** bdperkin_gone has joined #openstack-qa | 00:24 | |
*** bdperkin_gone is now known as bdperkin | 00:24 | |
jog0 | clarkb: looks like the bot is failing to post to gerrit | 00:27 |
---|---|---|
jog0 | if you click on https://review.openstack.org/49644 there is no comment there | 00:27 |
*** matsuhashi has joined #openstack-qa | 00:28 | |
clarkb | jog0: 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 |
jog0 | clarkb: yeah | 00:29 |
jog0 | something like that | 00:29 |
clarkb | jog0: I think I see the issue. patch incoming | 00:32 |
jog0 | stream.leaveCommnt | 00:32 |
jog0 | woot! | 00:32 |
*** salv-orlando has quit IRC | 00:32 | |
*** salv-orlando has joined #openstack-qa | 00:33 | |
jog0 | teamwork FTW | 00:34 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/elastic-recheck: Leave comment when single bug is found. https://review.openstack.org/49656 | 00:39 |
clarkb | jog0: ^ 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 though | 00:40 |
jog0 | the tests are a little funny right now | 00:41 |
jog0 | so I am fine with it | 00:41 |
*** nosnos has joined #openstack-qa | 00:45 | |
*** anteaya has quit IRC | 00:55 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Leave comment when single bug is found. https://review.openstack.org/49656 | 00:58 |
*** masayukig has left #openstack-qa | 00:58 | |
jog0 | clarkb: e-r will now need hopefully its last kick | 01:02 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some negative tests for flavor https://review.openstack.org/49588 | 01:03 |
clarkb | jog0 ok. I just witnessed a car accident so will be a bit later until I can do that woo | 01:07 |
*** kong has joined #openstack-qa | 01:15 | |
*** mriedem has joined #openstack-qa | 01:15 | |
*** yeylon_ has quit IRC | 01:16 | |
*** rfolco has quit IRC | 01:37 | |
*** jasondot_ has joined #openstack-qa | 01:40 | |
jog0 | clarkb: ouch | 01:45 |
clarkb | thankfully everyone was fine. Now I can see about kicking that bot | 01:52 |
*** openstackrecheck has quit IRC | 01:55 | |
*** openstackrecheck has joined #openstack-qa | 01:55 | |
clarkb | jog0: bot restarted thanks to fungi | 01:55 |
fungi | yup | 01:56 |
fungi | car accident on the walk home? eek | 01:56 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Clean up existing instances when not using tenant isolation https://review.openstack.org/49431 | 01:56 |
clarkb | fungi: yup fun times | 01:57 |
kong | could anyone help to review https://review.openstack.org/#/c/49588/? Thanks very much! | 01:58 |
*** jasondot_ has quit IRC | 02:00 | |
openstackgerrit | Jamie Lennox proposed a change to openstack-dev/grenade: Source lib/tls in upgrade-keystone https://review.openstack.org/49662 | 02:01 |
mriedem | kong: done | 02:02 |
mriedem | seems pretty straight-forward | 02:02 |
kong | mriedem, thanks! I'm a beginner, writing some simple tests is good for me, :) | 02:03 |
mriedem | kong: well in that case i'm sure someone will come along and no doubt find a reason to -1 it :) | 02:05 |
kong | I hope so, and I found there is lack of tests in fact for projects including Nova, Cinder, Neutron. | 02:06 |
clarkb | if 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 practices | 02:07 |
kong | clarkb, good suggestion! | 02:08 |
openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates https://review.openstack.org/47076 | 02:11 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49644 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 02:18 |
*** mriedem has quit IRC | 02:18 | |
clarkb | jog0: comment was left on that change. fingers crossed that it works now | 02:18 |
*** bdperkin has quit IRC | 02:24 | |
*** bdperkin has joined #openstack-qa | 02:24 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/49660 failed tempest with an unrecognized error | 02:29 |
jog0 | looks like it is | 02:30 |
*** giulivo has quit IRC | 02:44 | |
*** jamielennox is now known as jamielennox|away | 02:45 | |
*** sourabh has joined #openstack-qa | 03:02 | |
*** prekarat has joined #openstack-qa | 03:36 | |
*** prekarat has quit IRC | 03:41 | |
*** dkranz has joined #openstack-qa | 03:43 | |
openstackgerrit | A change was merged to openstack-dev/grenade: Install the new schema-image.json https://review.openstack.org/45884 | 03:54 |
*** prekarat has joined #openstack-qa | 03:54 | |
raies | hi | 04:40 |
raies | anybody who can help me on tempest | 04:40 |
*** DennyZhang has joined #openstack-qa | 04:53 | |
*** Madan has joined #openstack-qa | 04:56 | |
openstackgerrit | A change was merged to openstack/tempest: Do not check for id in the keystone output https://review.openstack.org/49017 | 04:58 |
openstackrecheck | openstack/nova change: https://review.openstack.org/49660 failed tempest with an unrecognized error | 05:05 |
*** afazekas has joined #openstack-qa | 05:06 | |
*** turul_ has joined #openstack-qa | 05:06 | |
*** jhenner has joined #openstack-qa | 05:23 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler https://review.openstack.org/48632 | 05:25 |
*** turul_ has quit IRC | 05:26 | |
*** afazekas has quit IRC | 05:26 | |
*** afazekas has joined #openstack-qa | 05:26 | |
raies | afazekas: hi | 05:29 |
openstackgerrit | Ryan Hsu proposed a change to openstack-dev/devstack: Fix typo in property passed to glance https://review.openstack.org/49674 | 05:29 |
*** shilpa_ has joined #openstack-qa | 05:30 | |
raies | is there any way to make my code dependent of two different codes | 05:30 |
raies | ? | 05:30 |
*** DennyZhang has quit IRC | 05:32 | |
afazekas | raies: I am not sure, may be if you create merge commit with 2 dependent change, and re-base your change to the merge | 05:34 |
clarkb | gerrit typically won't accept merge commits. instead you have to make the other two dependent on one or the other | 05:35 |
clarkb | (assuming code will eventually be pushed to gerrit) | 05:35 |
raies | clarkb: 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 |
raies | my 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-qa | 05:40 | |
clarkb | raies yes A <- B <- C | 05:42 |
*** Madan has quit IRC | 05:42 | |
clarkb | raies: I understand, afazekas' suggestion would do it but then you can't push C to gerrit | 05:42 |
raies | afazekas, clarkb: then is there any alternative way to fulfill my demand ? | 05:45 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/48792 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 05:48 |
*** SergeyLukjanov has quit IRC | 05:54 | |
*** matsuhashi has quit IRC | 06:01 | |
afazekas | raies: which change is A and B now ? | 06:08 |
*** nosnos has quit IRC | 06:08 | |
*** jlibosva has joined #openstack-qa | 06:09 | |
*** HenryG has quit IRC | 06:12 | |
*** HenryG has joined #openstack-qa | 06:12 | |
*** matsuhashi has joined #openstack-qa | 06:24 | |
raies | afazekas: A and B are https://review.openstack.org/#/c/48220/ and https://review.openstack.org/#/c/46050/ respectively | 06:25 |
raies | and C is https://review.openstack.org/#/c/48632/ | 06:26 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49679 failed tempest with an unrecognized error | 06:26 |
raies | sorry C is yet to commit | 06:27 |
raies | afazekas: ^^ | 06:27 |
*** Dharmit has joined #openstack-qa | 06:28 | |
raies | afazekas: 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-qa | 06:32 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/49677 failed tempest with an unrecognized error | 06:32 |
raies | afazekas: are you there ? | 06:39 |
*** yeylon_ has joined #openstack-qa | 06:48 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49683 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 06:48 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49682 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 06:49 |
openstackrecheck | openstack/nova change: https://review.openstack.org/49660 failed tempest with an unrecognized error | 06:50 |
afazekas | raies: 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 tests | 06:51 |
afazekas | raies: IMHO you do not need to add it by this change | 06:52 |
raies | afazekas: Yes no isolated credentials was used for networks test | 06:53 |
*** nosnos has joined #openstack-qa | 06:54 | |
raies | then should I use base.py file what I had used in earlier patch | 06:54 |
raies | ? | 06:54 |
afazekas | raies: yes | 06:54 |
raies | afazekas: ok I will apply next patch | 06:55 |
raies | afazekas: please have a look on https://bugs.launchpad.net/tempest/+bug/1235029 | 06:56 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: lib/swift variable changes and dd replaced by truncate https://review.openstack.org/46664 | 06:58 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler https://review.openstack.org/48632 | 07:06 |
afazekas | raies: I guess the allow_overlapping_ips = True is missing from the neutron conf | 07:13 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49679 failed tempest with an unrecognized error | 07:14 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/45725 failed tempest with an unrecognized error | 07:14 |
raies | afazekas: it my environment in the file "/etc/neutron/neutron.conf" | 07:16 |
raies | as per default setting ==> allow_overlapping_ips = True | 07:17 |
afazekas | https://bugs.launchpad.net/tempest/+bug/1235029 | 07:17 |
afazekas | FIXME but the bulk subnet creation usues a network which does not used by any other test class | 07:18 |
raies | I didn't get what are you saying | 07:19 |
afazekas | FIXME if the allow_overlapping_ips = True (and you have namespaces) you can use the same cidr in two different net | 07:19 |
*** fbo_away is now known as fbo | 07:19 | |
afazekas | raies: I guess when the exception happened with #1235029 the allow_overlapping_ips was False | 07:20 |
*** ken1ohmichi has joined #openstack-qa | 07:21 | |
raies | this thing I am getting, but I wanted to know what is FIXME ? | 07:21 |
raies | is it a keyword to convey message in comments during bug ? | 07:21 |
raies | afazekas: thus as you seggested, it is a environment bug right ? | 07:23 |
afazekas | FIXME .. correct me if I am wrong | 07:23 |
afazekas | raies: yes it can be | 07:23 |
raies | afazekas: thanks | 07:24 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49685 failed tempest with an unrecognized error | 07:28 |
raies | afazekas: you had asked a question "raies: which change is A and B now ?" | 07:32 |
afazekas | raies: and you have answered it | 07:39 |
raies | yes :) | 07:39 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49686 failed tempest with an unrecognized error | 07:41 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for aggregate https://review.openstack.org/49157 | 07:52 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49688 failed tempest with an unrecognized error | 07:55 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49682 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 07:55 |
*** giulivo has joined #openstack-qa | 08:09 | |
*** yassine has joined #openstack-qa | 08:15 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49157 failed tempest with an unrecognized error | 08:24 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack-dev/devstack: force run_process to really spawn a bash process https://review.openstack.org/49394 | 08:25 |
openstackrecheck | openstack/requirements change: https://review.openstack.org/49696 failed tempest with an unrecognized error | 08:33 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49424 failed tempest with an unrecognized error | 08:35 |
mkoderer | afazekas: about https://review.openstack.org/#/c/42920/ | 08:42 |
mkoderer | afazekas: I would like to have at least some logging and maybe some failure handling | 08:42 |
mkoderer | afazekas: or what do you think? | 08:42 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/tempest: Adds initial ceilometerclient testing code https://review.openstack.org/49700 | 08:47 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49596 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 08:48 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/tempest: Adds initial ceilometerclient testing code https://review.openstack.org/49700 | 08:49 |
*** comstud has quit IRC | 08:50 | |
afazekas | mkoderer: it has a long TODO list already, it can be added later | 08:50 |
mkoderer | afazekas: ok I'll put my nits and give a +1 | 08:51 |
afazekas | AFAIK tkammer will be long-long vacation soon | 08:51 |
afazekas | mkoderer: ok | 08:51 |
openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: max out VCPU count https://review.openstack.org/49703 | 09:02 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/tempest: Adds initial ceilometerclient testing code https://review.openstack.org/49700 | 09:05 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/48999 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 09:06 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 09:12 |
*** mkollaro has joined #openstack-qa | 09:13 | |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for aggregate https://review.openstack.org/49157 | 09:16 |
openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: unbroke stable gate (chown) https://review.openstack.org/49705 | 09:20 |
*** SergeyLukjanov has joined #openstack-qa | 09:20 | |
*** Dharmit has quit IRC | 09:25 | |
*** fbo is now known as fbo_away | 09:26 | |
raies | afazekas: please have a look on https://review.openstack.org/#/c/48632/ | 09:29 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49700 failed tempest with an unrecognized error | 09:36 |
openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: port test_flavors into v3 part1 https://review.openstack.org/49709 | 09:43 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49157 failed tempest with an unrecognized error | 09:54 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Fix to use proper random values https://review.openstack.org/46886 | 10:01 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/48999 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 10:05 |
*** fbo_away is now known as fbo | 10:10 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 10:12 |
*** SergeyLukjanov has quit IRC | 10:18 | |
*** SergeyLukjanov has joined #openstack-qa | 10:19 | |
openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Neutron metering agent: add CRUD tests https://review.openstack.org/49719 | 10:22 |
*** prekarat has quit IRC | 10:24 | |
*** matsuhashi has quit IRC | 10:28 | |
openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: pass stop_on_error to _has_error_in_logs https://review.openstack.org/49721 | 10:32 |
*** ken1ohmichi has quit IRC | 10:36 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49157 failed tempest with an unrecognized error | 11:09 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49596 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 11:19 |
giulivo | ping mkoderer | 11:19 |
giulivo | ping cyeoh | 11:20 |
giulivo | mkoderer, cyeoh I'm updating the name, thanks for reviewing | 11:24 |
*** Madan has quit IRC | 11:25 | |
openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: introduces skip_because decorator https://review.openstack.org/48121 | 11:26 |
*** nosnos has quit IRC | 11:32 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/44238 failed tempest with an unrecognized error | 11:44 |
openstackgerrit | Joe Mills proposed a change to openstack-dev/devstack: Change Midonet vif driver to generic https://review.openstack.org/49733 | 11:48 |
*** weshay has joined #openstack-qa | 12:01 | |
*** afazekas has quit IRC | 12:01 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49596 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 12:02 |
*** afazekas has joined #openstack-qa | 12:14 | |
*** lynnkong has joined #openstack-qa | 12:21 | |
*** kong has quit IRC | 12:24 | |
*** rfolco has joined #openstack-qa | 12:25 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Add back rpc_notifier when ceilometer is enabled https://review.openstack.org/48351 | 12:37 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/47557 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 12:45 |
*** SergeyLukjanov has quit IRC | 12:48 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 12:49 |
openstackrecheck | openstack/python-glanceclient change: https://review.openstack.org/48850 failed tempest with an unrecognized error | 12:52 |
*** SergeyLukjanov has joined #openstack-qa | 13:10 | |
*** adalbas has joined #openstack-qa | 13:14 | |
*** mriedem has joined #openstack-qa | 13:16 | |
*** shilpa_ has quit IRC | 13:16 | |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for aggregate https://review.openstack.org/49157 | 13:18 |
*** lynnkong has left #openstack-qa | 13:18 | |
*** lynnkong has joined #openstack-qa | 13:18 | |
*** lynnkong has quit IRC | 13:19 | |
*** sthaha has quit IRC | 13:21 | |
*** kong has joined #openstack-qa | 13:21 | |
*** sthaha has joined #openstack-qa | 13:21 | |
*** sthaha has joined #openstack-qa | 13:21 | |
*** sthaha has quit IRC | 13:27 | |
*** prekarat has joined #openstack-qa | 13:36 | |
*** prekarat has quit IRC | 13:37 | |
*** prekarat has joined #openstack-qa | 13:39 | |
giulivo | ping cyeoh | 13:41 |
giulivo | wanted to ask about this https://review.openstack.org/#/c/47508 | 13:41 |
giulivo | I'm still unsure if we should ignore those, suggest to remove, to update or not to remove | 13:41 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/47337 failed tempest with an unrecognized error | 13:43 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49748 failed tempest with an unrecognized error | 13:43 |
openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: introduces skip_because decorator https://review.openstack.org/48121 | 13:48 |
Anju | giulivo: what is restore_delete | 13:50 |
Anju | ? | 13:50 |
Anju | afazekas: ping | 13:52 |
kong | Anju, resotore a soft-deleted server | 13:55 |
Anju | kong: is any api for this? | 13:56 |
kong | yes | 13:56 |
kong | Anju, http://api.openstack.org/api-ref-compute.html | 13:57 |
kong | Anju, please search 'restore' | 13:57 |
kong | https://review.openstack.org/#/c/49488/ | 13:58 |
Anju | after deletion can we store the vm ? | 14:01 |
*** prekarat has quit IRC | 14:05 | |
kong | anju, unless 'reclaim_instance_interval' is set | 14:06 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49755 failed tempest with an unrecognized error | 14:07 |
*** russellb is now known as rustlebee | 14:11 | |
*** anteaya has joined #openstack-qa | 14:13 | |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Fix to use proper random values https://review.openstack.org/46886 | 14:14 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49679 failed tempest with an unrecognized error | 14:20 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/47557 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 14:20 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/46927 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 14:20 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49748 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 14:21 |
afazekas | sdague: https://review.openstack.org/#/c/49705/ | 14:34 |
*** jlibosva has quit IRC | 14:37 | |
giulivo | ping afazekas | 14:40 |
afazekas | giulivo: pong | 14:40 |
giulivo | I spotted a few things here https://review.openstack.org/#/c/48632 | 14:40 |
giulivo | like cls.network_cfg defined twice in base | 14:40 |
giulivo | also I don't think there is any need for the setUpClass in the test class? | 14:41 |
giulivo | and last but not least, the methods added in network_client are valuable but maybe should come with some tests? | 14:41 |
afazekas | giulivo: you are correct | 14:42 |
afazekas | raies: ^ | 14:42 |
giulivo | oh that's the nick, thanks | 14:43 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Remove unused CONF variable https://review.openstack.org/49763 | 14:43 |
*** afazekas is now known as afazekas_zz | 14:47 | |
* afazekas_zz self check failed --> go rest | 14:47 | |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add a negative test for flavor_extra_specs https://review.openstack.org/49577 | 14:51 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Remove unused CONF variable and import statement https://review.openstack.org/49763 | 14: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 |
dims | 3 check jobs in https://review.openstack.org/#/c/49371/ hit it at the same time | 14:58 |
*** mlavalle has joined #openstack-qa | 15:06 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/49371 failed tempest with an unrecognized error | 15:08 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add two negative tests for flavor-access https://review.openstack.org/49501 | 15:10 |
openstackrecheck | openstack/nova change: https://review.openstack.org/44238 failed tempest with an unrecognized error | 15:13 |
openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/devstack-gate: Make sure tempest directories exist when modifying https://review.openstack.org/49772 | 15:18 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49577 failed tempest with an unrecognized error | 15:32 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for host https://review.openstack.org/49778 | 15:39 |
*** yassine has quit IRC | 15:41 | |
*** krtaylor has quit IRC | 15:43 | |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for host https://review.openstack.org/49778 | 15:51 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add two negative tests for flavor-access https://review.openstack.org/49501 | 15:53 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for host https://review.openstack.org/49778 | 15:56 |
*** mkollaro has quit IRC | 15:57 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49688 failed tempest with an unrecognized error | 15:59 |
*** yeylon_ has quit IRC | 16:06 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49685 failed tempest with an unrecognized error | 16:12 |
*** dims has quit IRC | 16:15 | |
*** SergeyLukjanov has quit IRC | 16:23 | |
*** krtaylor has joined #openstack-qa | 16:23 | |
*** mlavalle has quit IRC | 16:23 | |
*** mlavalle has joined #openstack-qa | 16:23 | |
*** kong has quit IRC | 16:24 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49686 failed tempest with an unrecognized error | 16:26 |
openstackrecheck | openstack/python-glanceclient change: https://review.openstack.org/48850 failed tempest with an unrecognized error | 16:26 |
*** dims has joined #openstack-qa | 16:28 | |
*** fbo is now known as fbo_away | 16:28 | |
*** raies_ has joined #openstack-qa | 16:31 | |
raies_ | afazekas: giulivo has given a minor comment in https://review.openstack.org/#/c/48632/5/tempest/services/network/json/network_client.py | 16:33 |
raies_ | afazekas_zz ^^ | 16:34 |
raies_ | I have added these just for the sake so that I can add futher tess cases | 16:34 |
raies_ | *further | 16:34 |
raies_ | so should I put all these here ? | 16:35 |
raies_ | I think yes | 16:35 |
raies_ | afazekas_zz: what you say on this | 16:35 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler https://review.openstack.org/48632 | 16:39 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49684 failed tempest with an unrecognized error | 16:39 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49644 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 16:39 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49782 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 16:40 |
clarkb | jog0: the bot seems to be working well :) | 16:40 |
jog0 | clarkb: yeah! lots of failed tempest jobs to check up on though | 16:44 |
jog0 | clarkb: and http://status.openstack.org/elastic-recheck/ is producing great results too | 16:47 |
*** jog0 is now known as flashgordon | 16:48 | |
*** prekarat has joined #openstack-qa | 16:57 | |
*** prekarat has quit IRC | 16:57 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/48557 failed tempest with an unrecognized error | 16:59 |
openstackrecheck | openstack/nova change: https://review.openstack.org/48196 failed tempest with an unrecognized error | 16:59 |
*** SergeyLukjanov has joined #openstack-qa | 17:03 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/48935 failed tempest with an unrecognized error | 17:03 |
openstackrecheck | openstack/nova change: https://review.openstack.org/49623 failed tempest with an unrecognized error | 17:05 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/48632 failed tempest with an unrecognized error | 17:08 |
*** fbo_away is now known as fbo | 17:16 | |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove query for fixed bug 1226791 https://review.openstack.org/49795 | 17:19 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add message to assertNotEquals in test_valid_bugs https://review.openstack.org/49796 | 17:19 |
*** mlavalle has quit IRC | 17:29 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/49790 failed tempest with an unrecognized error | 17:33 |
clarkb | flashgordon: we have a lot of unrecognized errors now. I assume that means your push to fix classified bugs is working :) | 17:48 |
mtreinish | clarkb: I was just looking at a few of them. Some seem to be bad patches in the check queue | 17:48 |
clarkb | mtreinish: that would explain it | 17:49 |
flashgordon | clarkb mtreinish I am looking too | 17:49 |
flashgordon | Found this in a few http://logstash.openstack.org/#eyJzZWFyY2giOiJAbWVzc2FnZTpcImZhaWxlZCB0byByZWFjaCBBQ1RJVkUgc3RhdHVzIHdpdGhpbiB0aGUgcmVxdWlyZWQgdGltZSAoNDAwIHMpLiBDdXJyZW50IHN0YXR1czogQlVJTERcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiODY0MDAiLCJncmFwaG1vZGUiOiJjb3VudCIsInRpbWUiOnsidXNlcl9pbnRlcnZhbCI6MH0sInN0YW1wIjoxMzgwOTA4NzUwNDU0fQ== | 17:49 |
mtreinish | flashgordon: yeah I was just looking at one with that | 17:59 |
mtreinish | I'm trying to figure out where it's going wrong | 17:59 |
flashgordon | mtreinish: I vote open up a new bug for this | 17:59 |
flashgordon | and add that query to elastic-recheck | 17:59 |
mtreinish | the state transition in the tempest log was BUILD/scheduling -> BUILD/none | 17:59 |
flashgordon | and take it from there | 18:00 |
*** VinodGupta has joined #openstack-qa | 18:00 | |
mtreinish | flashgordon: I don't think that: "failed to reach ACTIVE status within the required time (400 s). Current status: BUILD" is specific enough | 18:00 |
mtreinish | it applies to any vm that fails to come up | 18:00 |
mtreinish | open it against nova? | 18:00 |
flashgordon | mtreinish: nova should never get hung in build state | 18:00 |
flashgordon | yeeah | 18:00 |
flashgordon | mtreinish: also we need to dedup http://status.openstack.org/rechecks/ again | 18:01 |
flashgordon | there are over a 100 bugs listed there pretty sure most are invalid or dupes | 18:01 |
mtreinish | flashgordon: that's always a lot of fun | 18:01 |
* clarkb just realized that we should incorporate pruning rechecks and elastic-recheck to bug days | 18:03 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Split the creation of $STACK_USER account out of stack.sh https://review.openstack.org/49798 | 18:04 |
mtreinish | clarkb: I basically spent the last tempest bug day just deduping bugs and cleaning up the recheck list | 18:05 |
flashgordon | clarkb: we have a test for elastic-recheck to prune | 18:08 |
flashgordon | two infact | 18:08 |
openstackrecheck | openstack/nova change: https://review.openstack.org/49117 failed tempest with an unrecognized error | 18:10 |
flashgordon | gah there goes another | 18:11 |
*** fbo is now known as fbo_away | 18:15 | |
flashgordon | mtreinish: https://bugs.launchpad.net/tempest/+bug/1228942 | 18:15 |
flashgordon | thats teh bug we want I think | 18:16 |
flashgordon | hmm maybe not | 18:16 |
mtreinish | flashgordon: yeah that looks similar | 18:16 |
mtreinish | although it's a different test then the other fails | 18:16 |
flashgordon | making a new bug will just be easier I htink | 18:17 |
mtreinish | I don't like how that guy always puts swift in his bugs | 18:17 |
mtreinish | flashgordon: yeah it probably is, and we can always set that one as a dup if it ends up being that way | 18:19 |
*** jhenner has quit IRC | 18:19 | |
flashgordon | yeah | 18:19 |
flashgordon | marking that one as incomplete as it looks like a bunch of bugs in one | 18:20 |
*** fbo_away is now known as fbo | 18:25 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Fix handling of pip and virtualenv on openSUSE https://review.openstack.org/49220 | 18:37 |
openstackgerrit | A change was merged to openstack-dev/devstack: Revert "Revert "Swift: configure Ceilometer when it is enabled"" https://review.openstack.org/48678 | 18:40 |
mriedem | open tempest question, there are some tests that produce nice instance names like this "ServerActionsTestJSON-instance1052674456" | 18:48 |
mriedem | those are created through the BaseComputeTest.create_server | 18:48 |
mriedem | some create instances with names like this: "server1344500157" | 18:48 |
mriedem | which isn't so helpful | 18:49 |
mriedem | those go through the servers client | 18:49 |
mriedem | anyone opposed to changing the tests to use the BaseComputeTest.create_server rather than servers_client? | 18:49 |
mtreinish | mriedem: 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 |
mtreinish | there are also tests that create servers that aren't part of the compute tests | 18:51 |
mtreinish | if you can work out the edge cases go for it | 18:51 |
mtreinish | because it will probably prevent server leaks | 18:51 |
mriedem | mtreinish: yeah, this would only be compute tests | 18:51 |
mriedem | mtreinish: and yeah, i'm looking at this b/c of our conversation the other day about test_list_servers_negative failing on leaked instances | 18:52 |
mtreinish | mriedem: yeah negative tests are one of the cases where using the base test case create gets tricky | 18:52 |
mtreinish | because we normally expect things to fail | 18:53 |
mtreinish | and the base test class create doesn't like that | 18:53 |
mriedem | mtreinish: ListServerFiltersTestJSON is the one i'm looking at as an example | 18:53 |
mriedem | looks like it keeps track of it's own 3 class level instances and then on teardown deletes them explicitly | 18:54 |
mriedem | where as the base class would handle that for you | 18:54 |
mtreinish | mriedem: yeah that would be a good case to move that to the common create method. | 18:54 |
mtreinish | it probably wasn't done originally because it wasn't leaking :) | 18:55 |
mriedem | mtreinish: good news is i guess i only see 2 rotten apples right now | 18:56 |
mriedem | test_list_server_filters and test_volumes_actions (cinder test) | 18:56 |
mriedem | the cinder one could probably be dealt with differently | 18:56 |
mriedem | mtreinish: do i need a bug for this cleanup? | 18:56 |
mtreinish | mriedem: I'd be fine without one, but if you want to make one go ahead | 18:57 |
mtreinish | someone might knock you for it though | 18:57 |
mriedem | and that someone would be sdague! | 18:58 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/45677 failed tempest because of: https://bugs.launchpad.net/bugs/1230407 and https://bugs.launchpad.net/bugs/1224001 and | 19:00 |
mtreinish | ^^^ I really like the multiple fails. I wonder what the commiter does in these cases | 19:01 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/45678 failed tempest with an unrecognized error | 19:01 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49748 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 19:02 |
raies_ | mtreinish: please have a look on https://review.openstack.org/#/c/48632/ | 19:02 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/45676 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 19:03 |
mriedem | good lord | 19:05 |
mriedem | https://bugs.launchpad.net/tempest/+bug/1235411 | 19:05 |
jgriffith | mriedem: :) | 19:07 |
mtreinish | mriedem: heh, I'm not sure the irc pastebin is neccessary but ok :) | 19:07 |
mriedem | mtreinish: it must be there for all time | 19:07 |
mriedem | mtreinish: actually that's so i don't forget about what you said about not fudging up cleanups | 19:07 |
* jgriffith get's some stone tablets | 19:07 | |
mtreinish | mriedem: well this channel is logged despite my objections about that | 19:08 |
mtreinish | which I can probably look up in the logs :) | 19:08 |
*** mlavalle has joined #openstack-qa | 19:08 | |
mriedem | ah, well then. i'm used to nova not being logged | 19:08 |
mtreinish | mriedem: http://eavesdrop.openstack.org/irclogs/%23openstack-qa/ | 19:09 |
* mriedem stashes conversations with dansmith to use against him later | 19:09 | |
mriedem | yup | 19:09 |
openstackgerrit | David Kranz proposed a change to openstack/tempest: Initial log check script to be called by devstack-gate https://review.openstack.org/49808 | 19:11 |
*** anteaya is now known as i_am_now_steve | 19:16 | |
openstackgerrit | A change was merged to openstack-dev/devstack: set errexit and xtrace in helper scripts https://review.openstack.org/49215 | 19:17 |
openstackgerrit | A change was merged to openstack-dev/devstack: Using no proxy option to skip wget and curl proxy settings in config https://review.openstack.org/46447 | 19:17 |
*** jhenner has joined #openstack-qa | 19:19 | |
*** i_am_now_steve is now known as anteaya | 19:23 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49086 failed tempest with an unrecognized error | 19:34 |
*** SergeyLukjanov has quit IRC | 19:43 | |
flashgordon | mtreinish: ping | 19:50 |
mtreinish | flashgordon: pong | 19:50 |
flashgordon | mtreinish: you file a bug for "failed to reach ACTIVE status within the required time (400 s). Current status: BUILD" | 19:51 |
mtreinish | flashgordon: not yet, I got sidetracked on some other things | 19:52 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Cleanup test_list_server_filters setup/teardown https://review.openstack.org/49819 | 19:52 |
mtreinish | flashgordon: let me take care of it right now | 19:52 |
flashgordon | cool | 19:52 |
flashgordon | then we can merge the query and start tracking it | 19:52 |
mtreinish | flashgordon: do you have a log link I can put in the bug? | 19:56 |
flashgordon | mtreinish: http://logs.openstack.org/23/49623/3/check/check-tempest-devstack-vm-full/b0d348a/testr_results.html.gz | 19:57 |
flashgordon | found with @message:"failed to reach ACTIVE status within the required time (400 s). Current status: BUILD" | 19:57 |
mtreinish | flashgordon: ok thanks | 19:57 |
flashgordon | mtreinish: thank you, filing another bug as we speak | 19:58 |
mtreinish | flashgordon: https://bugs.launchpad.net/nova/+bug/1235437 | 19:59 |
mtreinish | sure np | 19:59 |
mtreinish | I should have done it earlier sorry about that | 19:59 |
flashgordon | mtreinish: no problem can you add the query to e-r | 20:00 |
flashgordon | so it pops on on the e-r bot and stats.o.o/e-r | 20:00 |
flashgordon | staatus* | 20:00 |
mtreinish | sure one sec | 20:00 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Use predictable instance/volume names in test_volumes_actions https://review.openstack.org/49821 | 20:01 |
*** VinodGupta has quit IRC | 20:02 | |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add query for bug 1235435 https://review.openstack.org/49822 | 20:02 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Use predictable instance/volume names in test_volumes_actions https://review.openstack.org/49821 | 20:02 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Add a query for bug 1235437 https://review.openstack.org/49823 | 20:04 |
mtreinish | flashgordon: ^^^ | 20:04 |
flashgordon | mtreinish: can you review my query patches as well | 20:04 |
flashgordon | https://review.openstack.org/#/q/status:open+project:openstack-infra/elastic-recheck+branch:master+topic:queries,n,z | 20:04 |
mtreinish | flashgordon: sure | 20:05 |
flashgordon | hmm we both appended we may be forced to rebase but we can deal with it when it happens | 20:05 |
* mtreinish senses a merge conflict with the force | 20:05 | |
mtreinish | flashgordon: you can specify 2 messages in a query? | 20:07 |
flashgordon | mtreinish: yeah heh | 20:08 |
flashgordon | figured that out with trial and error | 20:08 |
mtreinish | flashgordon: heh, cool | 20:09 |
flashgordon | mtreinish: there is also this one https://review.openstack.org/#/c/49795/ | 20:14 |
flashgordon | our first removal | 20:14 |
mtreinish | flashgordon: oh cool a major milestone! | 20:15 |
flashgordon | mtreinish: yeah I noticed from http://status.openstack.org/elastic-recheck/ | 20:16 |
mtreinish | flashgordon: those graphs are super useful | 20:17 |
mtreinish | jeblair: ^^^ thanks | 20:17 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/48468 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 20:17 |
mtreinish | flashgordon: we can probably pull out Bug 1226337 sometime next week | 20:19 |
mtreinish | it doesn't look like it's happening anymore | 20:19 |
flashgordon | mtreinish: yeah | 20:20 |
flashgordon | https://review.openstack.org/#/c/49796/ | 20:20 |
flashgordon | I think we generally remove things only when they don't happen and are closed | 20:21 |
flashgordon | but not one or the other | 20:21 |
mtreinish | well I think it's closed already, but that's why I said next week to show a week trend of not occuring | 20:22 |
mtreinish | flashgordon: yeah I guess a equals exception isn't exactly clear there | 20:22 |
flashgordon | it wasn't at first glance | 20:23 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49819 failed tempest with an unrecognized error | 20:27 |
*** krtaylor has quit IRC | 20:31 | |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Cleanup test_list_server_filters setup/teardown https://review.openstack.org/49819 | 20:34 |
openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Use predictable instance/volume names in test_volumes_actions https://review.openstack.org/49821 | 20:34 |
*** raies_ has quit IRC | 20:34 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49782 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 20:42 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/49824 failed tempest with an unrecognized error | 20:56 |
*** rfolco has quit IRC | 20:58 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 21:02 |
*** mkollaro has joined #openstack-qa | 21:12 | |
*** fbo is now known as fbo_away | 21:14 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Remove general assumption in get_packages() https://review.openstack.org/49838 | 21:21 |
*** weshay has quit IRC | 21:24 | |
*** mkollaro has quit IRC | 21:29 | |
openstackgerrit | David Kranz proposed a change to openstack-infra/devstack-gate: Fail successful tempest runs if ERRORs in logs https://review.openstack.org/49840 | 21:29 |
dkranz | sdague: ping | 21:31 |
mtreinish | dkranz: he's out of the office today | 21:32 |
dkranz | mtreinish: Ok. Do you know how devstack gate works with stable branches? | 21:33 |
dkranz | mtreinish: Like in https://review.openstack.org/#/c/49840/1/devstack-vm-gate.sh,unified | 21:33 |
dkranz | mtreinish: which I just pushed, is it a problem that the tools/check_logs.py does not exist on stable branches? | 21:33 |
mtreinish | dkranz: 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#L49 | 21:34 |
mtreinish | I believe the code repos are prepopulated because of network reliability issues | 21:35 |
mtreinish | so you might just want to check for the stable flags and skip what you're adding there | 21:35 |
dkranz | mtreinish: ok, thanks. | 21:36 |
*** mriedem has quit IRC | 21:36 | |
mtreinish | dkranz: 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 tool | 21:36 |
mtreinish | it should be safe to run with coverage (it's just a full tempest run with some extra scripts running around it | 21:37 |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Split the creation of $STACK_USER account out of stack.sh https://review.openstack.org/49798 | 21:37 |
dkranz | mtreinish: I thought about that. Have to check that the tempest job succeeded first | 21:37 |
dkranz | mtreinish: and I'm shell-challenged but will figure it out. | 21:37 |
dkranz | mtreinish: It would be nice if we had a version number to compare rather than having to list all old stable branches. | 21:39 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Remove query for fixed bug 1226791 https://review.openstack.org/49795 | 21:41 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add message to assertNotEquals in test_valid_bugs https://review.openstack.org/49796 | 21:41 |
mtreinish | dkranz: well there are only 2 in there at any given time so it's not that bad | 21:42 |
mtreinish | and it's actually a branch name that is being passed in from the git repo | 21:42 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for bug 1235435 https://review.openstack.org/49822 | 21:42 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Add a query for bug 1235437 https://review.openstack.org/49823 | 21:45 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49424 failed tempest with an unrecognized error | 21:47 |
giulivo | mtreinish, apparently I screwed up here https://review.openstack.org/#/c/48933 | 22:03 |
*** krtaylor has joined #openstack-qa | 22:10 | |
openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/49840 failed tempest with an unrecognized error | 22:11 |
giulivo | mtreinish, pretty badly actually | 22:13 |
*** jhenner has quit IRC | 22:14 | |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add query for bug 1235486 https://review.openstack.org/49846 | 22:19 |
openstackgerrit | Adam Spiers proposed a change to openstack-dev/devstack: Avoid implying DevStack is POSIX-compatible https://review.openstack.org/49847 | 22:19 |
flashgordon | salv-orlando: ^^ | 22:19 |
openstackgerrit | Adam Spiers proposed a change to openstack-dev/devstack: Fix "instal_prereqs.sh" typo https://review.openstack.org/49848 | 22:20 |
salv-orlando | thanks flashgordon | 22:25 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/49424 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 23:22 |
*** hemna_ is now known as hemnafk | 23:29 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add a query for bug 1235437 https://review.openstack.org/49823 | 23:40 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!