openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir https://review.openstack.org/59305 | 00:04 |
---|---|---|
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase https://review.openstack.org/59272 | 00:04 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: method extraction for readability https://review.openstack.org/59263 | 00:04 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns https://review.openstack.org/59262 | 00:04 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols https://review.openstack.org/59261 | 00:04 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: Revert "Add test_cases for cinder cli in v2 version" https://review.openstack.org/59306 | 00:11 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: Revert "Add test_cases for cinder cli in v2 version" https://review.openstack.org/59306 | 00:12 |
*** matsuhashi has joined #openstack-qa | 00:19 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/54202 failed tempest with an unrecognized error | 00:26 |
sdague | lifeless: so that fail ^^^ is actually another instance of wanting testr / testtools to randomize test order | 00:28 |
sdague | turns out we landed some tests which worked 100% of the time, until some other timing change landed | 00:28 |
sdague | then new tests are overlapping each other a lot | 00:29 |
sdague | and racing | 00:29 |
sdague | my revert above it is just turning off the tests, because this is actually a test problem, not a server race | 00:30 |
*** matsuhashi has quit IRC | 00:30 | |
*** matsuhashi has joined #openstack-qa | 00:30 | |
*** matsuhashi has quit IRC | 00:35 | |
openstackrecheck | openstack/heat change: https://review.openstack.org/59302 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 00:35 |
lifeless | sdague: hmm | 00:38 |
lifeless | sdague: if they are interacting then perhaps there should be a timing lock that such tests take out? | 00:39 |
lifeless | sdague: I'm not sure random will help all that much with the underlying problem (but like I said I'm very open to patches) | 00:39 |
sdague | lifeless: so random would have given us a chance to see that interaction earlier, hopefully before we landed it | 00:42 |
sdague | basically I want chaos monkey for test order | 00:42 |
lifeless | sdague: sure | 00:42 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Change unstable test which gets console output https://review.openstack.org/59116 | 00:42 |
sdague | because the stability of test ordering can actually hide interactions until post landing | 00:43 |
lifeless | sdague: which, as I acked, is two layered, trivial patch needed for testr, more complex for testtools but also doable. | 00:43 |
sdague | ok, I hadn't quite figured out where we got with that conv, as I had to run away to drink beer with jog0 and jeblair that afternoon | 00:43 |
lifeless | sdague: I'm not pushing back on the desire; I get that the issue is that the interactions don't fail-hard, they fail-sometimes and that what is killer | 00:43 |
lifeless | sdague: it's in the 'I will review it if someone else writes it' bucket. | 00:43 |
lifeless | sdague: e.g. blueprint approved but noone assigned | 00:44 |
sdague | ok. | 00:44 |
lifeless | I would say 'if I get a chance I'll do it' but I'm overcommitted right now and realistically for a while | 00:44 |
lifeless | so it would be false hope | 00:44 |
sdague | yeh, fair. Honestly I don't think I can take the context switch for it at the moment either | 00:45 |
*** zul has joined #openstack-qa | 00:46 | |
sdague | it is amazing how quick it can make things an issue though. That cinder cli race went from 0 to nearly 100% | 00:46 |
sdague | it blocked the metadata fix patch twice | 00:46 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/59306 failed tempest with an unrecognized error | 00:48 |
*** dmorita has joined #openstack-qa | 00:58 | |
openstackgerrit | A change was merged to openstack/tempest: Revert "Add test_cases for cinder cli in v2 version" https://review.openstack.org/59306 | 00:59 |
*** ken1ohmichi has joined #openstack-qa | 01:02 | |
*** matsuhashi has joined #openstack-qa | 01:05 | |
*** adalbas has quit IRC | 01:09 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Fix neutron log format with colorization https://review.openstack.org/58350 | 01:12 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase https://review.openstack.org/59272 | 01:25 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir https://review.openstack.org/59252 | 01:25 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: method extraction for readability https://review.openstack.org/59263 | 01:25 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns https://review.openstack.org/59262 | 01:25 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols https://review.openstack.org/59261 | 01:25 |
*** marun has joined #openstack-qa | 01:27 | |
*** nosnos has joined #openstack-qa | 01:33 | |
*** sthaha has joined #openstack-qa | 01:34 | |
*** sthaha has joined #openstack-qa | 01:34 | |
*** dims has joined #openstack-qa | 01:41 | |
*** macjack has joined #openstack-qa | 01:44 | |
*** dims has quit IRC | 01:54 | |
*** macjack has quit IRC | 01:56 | |
openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Replace assertGreaterEqual - is not in py26 testtools https://review.openstack.org/59127 | 02:01 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 02:09 |
openstackgerrit | Ma Wen Cheng proposed a change to openstack/tempest: Add static IPv6 injection scenario testcase https://review.openstack.org/58721 | 02:40 |
openstackrecheck | openstack/nova change: https://review.openstack.org/55499 failed tempest with an unrecognized error | 02:53 |
*** marun has quit IRC | 02:56 | |
*** matsuhashi has quit IRC | 02:59 | |
*** matsuhashi has joined #openstack-qa | 03:02 | |
*** jamielennox|away is now known as jamielennox | 03:10 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/58721 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 03:28 |
cyeoh | are there any instructions around which explain how to add a failure to elastic recheck? | 03:57 |
*** matsuhashi has quit IRC | 04:02 | |
cyeoh | oh I think I just found them | 04:02 |
*** matsuhas_ has joined #openstack-qa | 04:05 | |
*** prekarat has quit IRC | 04:09 | |
*** markmcclain1 has joined #openstack-qa | 04:12 | |
*** markmcclain has quit IRC | 04:12 | |
*** SergeyLukjanov has joined #openstack-qa | 04:14 | |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Add tests for testing swift bulk middleware https://review.openstack.org/56183 | 04:16 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms https://review.openstack.org/39237 | 04:36 |
*** DennyZhang has joined #openstack-qa | 04:55 | |
*** jorisroovers has joined #openstack-qa | 05:02 | |
*** jroovers has joined #openstack-qa | 05:06 | |
*** jorisroovers has quit IRC | 05:09 | |
*** psharma has joined #openstack-qa | 05:14 | |
*** yfried has quit IRC | 05:15 | |
*** SergeyLukjanov is now known as _SergeyLukjanov | 05:19 | |
*** _SergeyLukjanov has quit IRC | 05:19 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/54202 failed tempest with an unrecognized error | 05:28 |
openstackgerrit | Feng Ju proposed a change to openstack/tempest: Refactors network test in scenario to support advanced network test https://review.openstack.org/58718 | 05:29 |
*** nosnos_ has joined #openstack-qa | 05:35 | |
*** nosnos has quit IRC | 05:37 | |
*** nosnos_ has quit IRC | 05:37 | |
*** nosnos has joined #openstack-qa | 05:38 | |
*** prekarat has joined #openstack-qa | 05:39 | |
*** yfried has joined #openstack-qa | 05:43 | |
*** prekarat has quit IRC | 05:44 | |
*** prekarat has joined #openstack-qa | 05:54 | |
jroovers | hi ken1ohmichi | 05:54 |
ken1ohmichi | hi jroovers | 05:55 |
jroovers | thanks for having a look at https://review.openstack.org/#/c/58419/ | 05:55 |
*** matsuhas_ has quit IRC | 05:55 | |
jroovers | wanted to ask you a couple of questions if that is ok | 05:55 |
jroovers | this is actually my very first commit | 05:55 |
ken1ohmichi | jroovers: I see | 05:56 |
*** matsuhashi has joined #openstack-qa | 05:56 | |
jroovers | so, I am working with minsel on this | 05:56 |
jroovers | and this is some prep work as part of a bigger effort for adding more tests within Neutron | 05:56 |
jroovers | well, within Tempest, for Neutron | 05:56 |
jroovers | so, in that context | 05:57 |
ken1ohmichi | jroovers: ok, that would be nice | 05:57 |
jroovers | :-) | 05:57 |
jroovers | now, being very new to this, I really don't have an answer to the XML client methods | 05:57 |
jroovers | I thought about that too | 05:58 |
jroovers | but I'm guessing this will be tackled as part of a bigger effor | 05:58 |
jroovers | *effort | 05:58 |
ken1ohmichi | jroovers: do you mean you don't have a plan to use the method? | 05:59 |
jroovers | well, we do | 05:59 |
jroovers | I was just talking about the XML client methods (you mentioned this in the comments of the review) | 05:59 |
ken1ohmichi | jroovers: OK, sorry. | 05:59 |
jroovers | I think that you make a very good point there | 06:00 |
jroovers | but I'm not sure how to tackle that | 06:00 |
*** rdekel has joined #openstack-qa | 06:00 | |
ken1ohmichi | jroovers: I guess XML client is necessary if adding some tests for using the methods. | 06:01 |
jroovers | no it is not | 06:01 |
jroovers | so, as far as I understand the code | 06:01 |
jroovers | there are currently not XML tests | 06:01 |
ken1ohmichi | jroovers: What file you will add some tests into? | 06:02 |
jroovers | oh sorry | 06:02 |
jroovers | I take that back | 06:02 |
jroovers | I'm looking at tempest/api/network | 06:02 |
jroovers | I believe I will need to add an extra file | 06:02 |
jroovers | test_agents | 06:02 |
yfried | sdague: ping | 06:03 |
ken1ohmichi | jroovers: There are test files under tempest/api/network contain each TestXML class(e.g. ExtensionsTestXML), that means necessary XML client. | 06:04 |
jroovers | ken1ohmichi: ok, so there is an XML network_client.py as well | 06:04 |
jroovers | ken1ohmichi, yeah, sorry, apologies for the noobness. | 06:04 |
jroovers | I missed that XML network_client file | 06:04 |
jroovers | in /tempest/services/network/xml/network_client.py | 06:04 |
jroovers | I will add tests as well | 06:05 |
ken1ohmichi | jroovers: thanks in advance. | 06:05 |
jroovers | so, for the tests | 06:05 |
jroovers | would that be in a new file | 06:05 |
jroovers | ? | 06:05 |
ken1ohmichi | jroovers: please wait for a minute, will check the other service tests. | 06:06 |
jroovers | ok, thanks | 06:06 |
ken1ohmichi | jroovers: how about tempest/api/network/admin/test_l3_agent_scheduler.py? similar method list_agents() is used in the test file. | 06:09 |
jroovers | ken1ohmichi: looking into it, sec | 06:11 |
jroovers | ken1ohmichi: I still think that it should be a separate test file | 06:14 |
jroovers | ken1ohmichi: http://docs.openstack.org/api/openstack-network/2.0/content/agent_ext.html | 06:14 |
*** Madan has joined #openstack-qa | 06:14 | |
jroovers | it are different parts of the API | 06:14 |
jroovers | ken1ohmichi: vs. http://docs.openstack.org/api/openstack-network/2.0/content/Scheduler_L3_Agent.html | 06:14 |
ken1ohmichi | jroovers: interesting. I agree with you. | 06:17 |
ken1ohmichi | jroovers: it seems better to separate test file from test_l3_agent_scheduler.py. | 06:18 |
jroovers | ok, so then I'll create a test_agents.py file in tempest/api/network/admin | 06:18 |
ken1ohmichi | jroovers: nice :-) | 06:19 |
jroovers | ken1ohmichi: thanks. Are you around for another 30 mins or so? Then I'll try to get that in right away | 06:19 |
*** SergeyLukjanov has joined #openstack-qa | 06:20 | |
ken1ohmichi | jroovers: yes, I will stay. | 06:20 |
jroovers | ken1ohmichi: thanks, very much appreciated! | 06:20 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/58718 failed tempest with an unrecognized error | 06:20 |
openstackrecheck | openstack/nova change: https://review.openstack.org/59285 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 06:32 |
*** wingwj has joined #openstack-qa | 06:42 | |
*** YorikSar has joined #openstack-qa | 06:43 | |
*** SushilKM has joined #openstack-qa | 06:45 | |
jroovers | ken1ohmichi: mm, I'm implementing the tests, but just noticed that there is no create method for the agent management API | 06:45 |
jroovers | ken1ohmichi: http://docs.openstack.org/api/openstack-network/2.0/content/agent_ext.html | 06:45 |
jroovers | ken1ohmichi: which of course makes sense as the agents are started manually on the relevant hosts | 06:46 |
ken1ohmichi | jroovers: how about adding list/get tests as the first step without create? | 06:48 |
openstackrecheck | openstack/keystone change: https://review.openstack.org/59121 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 06:48 |
jroovers | ken1ohmichi: well that is what I was trying to do | 06:48 |
jroovers | ken1ohmichi: but of course, how do you assert anything if you can't add an agent first | 06:49 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/53028 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 06:49 |
jroovers | ken1ohmichi: there are a number of default agents that are started, such as neutron-dhcp-agent | 06:50 |
jroovers | ken1ohmichi: but these are really deployment dependent | 06:51 |
ken1ohmichi | jroovers: We can check the response code of these APIs. | 06:51 |
jroovers | ken1ohmichi: yes, I can definitly do that | 06:51 |
jroovers | ken1ohmichi: do you think that is useful enough for now? | 06:51 |
ken1ohmichi | jroovers: can we check the number of agents? | 06:51 |
jroovers | I don't think so | 06:52 |
jroovers | as depending on the deployment there might be a different number | 06:52 |
ken1ohmichi | jroovers: if neutron is ruuning, the number of agents is bigger than 1? | 06:52 |
jroovers | yes | 06:52 |
ken1ohmichi | jroovers: in tempes, there are some kind of checking | 06:53 |
jroovers | ken1ohmichi: there are a bunch of them, like the metadata-agent, an dhcp agent | 06:53 |
jroovers | and many more | 06:53 |
ken1ohmichi | jroovers: nice, we can check the number of agents in additional test. | 06:53 |
jroovers | ken1ohmichi: well I don't think so | 06:54 |
jroovers | ken1ohmichi: as that might be different based on the deployment | 06:54 |
jroovers | ken1ohmichi: if we run the tests with a different config/setup, than the number will be different as well I think | 06:54 |
jroovers | ken1ohmichi: so what I think I can do, is fetch the list and then do the other tests based on that | 06:55 |
ken1ohmichi | jroovers: I am trying some examples for this test. | 06:55 |
jroovers | ken1ohmichi: ok, thanks | 06:55 |
*** matsuhashi has quit IRC | 06:56 | |
*** matsuhashi has joined #openstack-qa | 06:56 | |
*** matsuhashi has quit IRC | 06:57 | |
*** matsuhashi has joined #openstack-qa | 06:57 | |
*** nosnos_ has joined #openstack-qa | 06:57 | |
*** nosnos has quit IRC | 06:57 | |
*** marun has joined #openstack-qa | 06:59 | |
ken1ohmichi | jroovers: how about checking the length of list_agent API like https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_server_addresses.py#L56 ? | 06:59 |
jroovers | ken1ohmichi: yes, that is something we could do | 07:00 |
jroovers | also, checking that attributes are present like in the lines below | 07:00 |
ken1ohmichi | jroovers: and picking the first agent from the list, testing the get_agent API by the agent. | 07:01 |
jroovers | ken1ohmichi: exactly my thought | 07:01 |
jroovers | ken1ohmichi: well, we can do it for all of them too right | 07:01 |
ken1ohmichi | jroovers: good idea. | 07:01 |
jroovers | ken1ohmichi: ok, thanks :-) Gonna dig in :-) | 07:01 |
ken1ohmichi | jroovers: I'm looking forward to the test :-) | 07:03 |
*** jlibosva has joined #openstack-qa | 07:18 | |
*** SushilKM has quit IRC | 07:21 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/58681 failed tempest with an unrecognized error | 07:22 |
*** DennyZha` has joined #openstack-qa | 07:25 | |
*** DennyZhang has quit IRC | 07:25 | |
*** zigo has quit IRC | 07:27 | |
*** zigo has joined #openstack-qa | 07:27 | |
*** DennyZha` has quit IRC | 07:33 | |
jroovers | ken1ohmichi: another question | 07:34 |
*** tkammer has joined #openstack-qa | 07:35 | |
jroovers | ken1ohmichi: I was trying to check all the attributes returned for a given agent by list_agents and check whether those attributes had the same value for the show_agent method | 07:35 |
jroovers | ken1ohmichi: that seems to be true for most of the values, but not for all of them | 07:35 |
jroovers | ken1ohmichi: what is the best way to proceed in this case? Not check all the attributes, check the ones that we are sure about, do a loop over the attributes and exclude the ones that we know can be different (this last solution is not so future proof)? | 07:37 |
ken1ohmichi | jroovers: If the difference would be a bug, we should fix it. is that like a bug? | 07:40 |
jroovers | ken1ohmichi: not entirely sure | 07:40 |
jroovers | ken1ohmichi: so there are 2 differences that I have noticed | 07:41 |
jroovers | ken1ohmichi: one is a heartbeat attribute that is part of the list_agents | 07:41 |
jroovers | I don't think that is a bug | 07:41 |
ken1ohmichi | jroovers: OK, seems you are right. | 07:41 |
jroovers | ken1ohmichi: however, there is this other attribute | 07:41 |
ken1ohmichi | jroovers: what attribute? | 07:42 |
jroovers | 'description' that doesn't seem to be part of the show_agent | 07:42 |
ken1ohmichi | jroovers: interesting, could you post the latest patch for sharing the difference? | 07:43 |
jroovers | ken1ohmichi: so in agent_show that attributes seems to be 'None' | 07:44 |
jroovers | ok, gimme sec | 07:44 |
jroovers | ken1ohmichi: never appended a patch before | 07:44 |
jroovers | ken1ohmichi: but I think I know how to do it | 07:44 |
ken1ohmichi | jroovers: sorry, I'm not sure I understand you said. could you "git review" again? | 07:46 |
jroovers | ken1ohmichi: yes, but I need to do 'git commit -a -amend' right? | 07:46 |
ken1ohmichi | jroovers: you are right. If you add a new file, "git add <file>" and "git commit --amend" then "git review" | 07:47 |
openstackgerrit | Joris Roovers proposed a change to openstack/tempest: Adds full agent mgmt API support in network_client https://review.openstack.org/58419 | 07:48 |
jroovers | ok, done | 07:48 |
ken1ohmichi | jroovers: so fast:-) OK, will wait for Jenkins job. | 07:48 |
jroovers | well, this doesn't pass | 07:49 |
jroovers | ken1ohmichi: just some quick code that showcases the problem | 07:49 |
ken1ohmichi | jroovers: no problem. If this test finds a real bug, that test would be realy nice. | 07:50 |
jroovers | ken1ohmichi: sorry, I didn't actually include the assert, just a print statement in the test. I thought you wanted to have a quick look at that | 07:51 |
jroovers | ken1ohmichi: so I think this test DOES actually pass | 07:51 |
jroovers | ken1ohmichi: do you want me to add the assert in and push again? | 07:52 |
ken1ohmichi | jroovers: I'm glad if you doing it :-) | 07:52 |
openstackgerrit | Joris Roovers proposed a change to openstack/tempest: Adds full agent mgmt API support in network_client https://review.openstack.org/58419 | 07:54 |
jroovers | ok, done | 07:54 |
jroovers | ken1ohmichi: ok done | 07:55 |
*** zigo has quit IRC | 07:55 | |
*** zigo has joined #openstack-qa | 07:55 | |
*** prekarat1 has joined #openstack-qa | 08:04 | |
*** prekarat1 has quit IRC | 08:04 | |
*** prekarat1 has joined #openstack-qa | 08:04 | |
*** prekarat has quit IRC | 08:04 | |
*** SergeyLukjanov has quit IRC | 08:08 | |
*** matsuhashi has quit IRC | 08:10 | |
*** SergeyLukjanov has joined #openstack-qa | 08:15 | |
*** matsuhashi has joined #openstack-qa | 08:16 | |
*** nosnos_ has quit IRC | 08:27 | |
*** ken1ohmichi has quit IRC | 08:27 | |
*** nosnos has joined #openstack-qa | 08:28 | |
openstackgerrit | Sergey Kraynev proposed a change to openstack/tempest: Use correct types for thresholds https://review.openstack.org/59331 | 08:46 |
*** SergeyLukjanov is now known as _SergeyLukjanov | 08:48 | |
*** _SergeyLukjanov has quit IRC | 08:49 | |
raies | can someone check it for approval as it is pending for too long time https://review.openstack.org/#/q/owner:+raiesmh08,n,z | 08:51 |
*** SergeyLukjanov has joined #openstack-qa | 08:53 | |
*** che-arne has quit IRC | 08:55 | |
*** jlibosva has quit IRC | 08:58 | |
*** yassine has joined #openstack-qa | 09:00 | |
*** jhenner has quit IRC | 09:01 | |
*** prekarat has joined #openstack-qa | 09:01 | |
*** jlibosva has joined #openstack-qa | 09:02 | |
*** prekarat1 has quit IRC | 09:02 | |
*** raies has quit IRC | 09:04 | |
*** zigo has quit IRC | 09:06 | |
*** zigo has joined #openstack-qa | 09:06 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/55032 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 09:07 |
openstackrecheck | openstack/horizon change: https://review.openstack.org/57088 failed tempest with an unrecognized error | 09:08 |
*** jpich has joined #openstack-qa | 09:09 | |
openstackrecheck | openstack/glance change: https://review.openstack.org/54281 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 09:11 |
*** matsuhashi has quit IRC | 09:14 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/58721 failed tempest with an unrecognized error | 09:17 |
*** SergeyLukjanov has quit IRC | 09:20 | |
*** fbo_away is now known as fbo | 09:20 | |
*** afazekas has joined #openstack-qa | 09:22 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/57083 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 09:22 |
*** SergeyLukjanov has joined #openstack-qa | 09:23 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/57107 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 09:25 |
*** SergeyLukjanov has quit IRC | 09:29 | |
*** matsuhashi has joined #openstack-qa | 09:31 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/59117 failed tempest with an unrecognized error | 09:33 |
*** jhenner has joined #openstack-qa | 09:42 | |
openstackgerrit | Joris Roovers proposed a change to openstack/tempest: Tests for Neutron Agent Management API https://review.openstack.org/58419 | 09:44 |
*** jlibosva has quit IRC | 09:48 | |
*** jlibosva has joined #openstack-qa | 09:48 | |
*** rdekel1 has joined #openstack-qa | 09:50 | |
*** prekarat has quit IRC | 09:51 | |
*** rdekel has quit IRC | 09:52 | |
*** prekarat has joined #openstack-qa | 09:52 | |
kashyap | afazekas, Just saw the scroll. Yes - I'm using nested guests | 09:53 |
kashyap | andreaf, I remember the rule: -A POSTROUTING -p udp -m udp --dport 68 -j C | 09:53 |
kashyap | HECKSUM --checksum-fill rule in the L1 guests. | 09:53 |
kashyap | Buried in a couple of other issues, will let you know if I can see a DHCP OFFER in this setup | 09:54 |
*** SergeyLukjanov has joined #openstack-qa | 09:55 | |
*** jroovers has quit IRC | 10:05 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/57858 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 10:08 |
openstackgerrit | Sergey Kraynev proposed a change to openstack/tempest: Delete stack after test https://review.openstack.org/59341 | 10:08 |
dmorita | Hi, guys. When is next IRC mtg? Monday at 21 UTC or Thursday at 17 UTC? | 10:21 |
*** tkammer has quit IRC | 10:26 | |
*** jorisroovers has joined #openstack-qa | 10:30 | |
*** markmcclain1 has quit IRC | 10:33 | |
*** andreaf has quit IRC | 10:35 | |
*** andreaf has joined #openstack-qa | 10:35 | |
*** jorisroovers has quit IRC | 10:36 | |
*** markmcclain has joined #openstack-qa | 10:43 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/57858 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 10:49 |
*** prekarat has quit IRC | 10:57 | |
*** adalbas has joined #openstack-qa | 10:57 | |
*** che-arne has joined #openstack-qa | 10:58 | |
*** salv-orlando has joined #openstack-qa | 10:59 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/58419 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 10:59 |
openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/59310 failed tempest with an unrecognized error | 10:59 |
*** jorisroovers has joined #openstack-qa | 11:00 | |
*** Madan has quit IRC | 11:02 | |
*** rfolco has joined #openstack-qa | 11:07 | |
*** jorisroovers has quit IRC | 11:11 | |
*** safchain has joined #openstack-qa | 11:16 | |
*** matsuhashi has quit IRC | 11:22 | |
*** dmorita has quit IRC | 11:25 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/46372 failed tempest with an unrecognized error | 11:25 |
openstackrecheck | openstack/keystone change: https://review.openstack.org/56243 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 11:29 |
yfried | sdague: ping | 11:33 |
sdague | hey, what's up? | 11:33 |
*** HenryG has joined #openstack-qa | 11:35 | |
openstackgerrit | Mauro S. M. Rodrigues proposed a change to openstack-dev/grenade: Refactor new options into a new config file. https://review.openstack.org/57744 | 11:36 |
openstackgerrit | Mauro S. M. Rodrigues proposed a change to openstack-dev/grenade: Add clean configuration files to upgrade from H to I https://review.openstack.org/53874 | 11:36 |
*** mkollaro has joined #openstack-qa | 11:36 | |
*** mkollaro has quit IRC | 11:37 | |
*** mkollaro has joined #openstack-qa | 11:38 | |
*** rossella_s has joined #openstack-qa | 11:39 | |
*** tkammer has joined #openstack-qa | 11:46 | |
*** gszasz has joined #openstack-qa | 11:46 | |
openstackgerrit | Joe Hakim Rahme proposed a change to openstack/tempest: Adds improvements to the Swift TempURL test https://review.openstack.org/57287 | 11:51 |
*** jorisroovers has joined #openstack-qa | 11:53 | |
*** yassine has quit IRC | 11:55 | |
*** dtroyer_zz has joined #openstack-qa | 11:56 | |
*** zigo_ has joined #openstack-qa | 11:56 | |
*** tkammer_ has joined #openstack-qa | 11:56 | |
*** seiflotfy has quit IRC | 11:57 | |
*** cyeoh has quit IRC | 11:57 | |
*** dtroyer has quit IRC | 11:57 | |
*** tkammer has quit IRC | 11:57 | |
*** zigo has quit IRC | 11:57 | |
*** cyeoh has joined #openstack-qa | 11:57 | |
*** seiflotfy_ has joined #openstack-qa | 11:57 | |
*** seiflotfy_ is now known as seiflotfy | 11:59 | |
openstackrecheck | openstack/heat change: https://review.openstack.org/58299 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 12:18 |
*** nosnos has quit IRC | 12:22 | |
*** SergeyLukjanov is now known as _SergeyLukjanov | 12:26 | |
*** giulivo has joined #openstack-qa | 12:38 | |
openstackgerrit | Nejc Saje proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms https://review.openstack.org/39237 | 12:43 |
*** che-arne has quit IRC | 12:52 | |
yfried | sdague: ping | 12:54 |
sdague | yfried: instread of saying ping, please just type your question :) as we seem to keep missing each other | 12:56 |
yfried | sdague: for next time are you on US timezone? | 12:56 |
sdague | yes, EST | 12:56 |
*** matsuhashi has joined #openstack-qa | 12:57 | |
yfried | sdague: regarding your review - https://review.openstack.org/#/c/55810/17/tempest/api/network/common.py, I answered your comment but haven't didn't get a response. Do you accept my explanation or stick to your -1? | 12:58 |
*** matsuhashi has quit IRC | 13:06 | |
*** matsuhashi has joined #openstack-qa | 13:07 | |
*** dkranz has quit IRC | 13:07 | |
sdague | for the hash id, yes | 13:07 |
sdague | that was more of a question | 13:07 |
sdague | however, please fix the removal of unrelated whitespace | 13:07 |
maurosr | sdague: can you help me? just hit this, never happened locally, any idea? http://logs.openstack.org/74/53874/10/check/check-grenade-devstack-vm/773a2ff/logs/new/screen-n-cpu.txt.gz | 13:08 |
sdague | maurosr: can you link to a specific line? | 13:09 |
sdague | what am I looking for? | 13:09 |
maurosr | sdague: last line | 13:09 |
maurosr | *lines | 13:10 |
sdague | interesting, that's new to me | 13:10 |
sdague | sounds like a bad config | 13:10 |
*** sthaha has quit IRC | 13:11 | |
sdague | maybe someone landed something in stable that broke it? | 13:11 |
maurosr | weird, it went fine local... | 13:11 |
*** sthaha has joined #openstack-qa | 13:11 | |
*** sthaha has quit IRC | 13:11 | |
*** sthaha has joined #openstack-qa | 13:11 | |
*** matsuhashi has quit IRC | 13:11 | |
maurosr | sdague: don't think so, no new config in grenade latelly | 13:11 |
sdague | maurosr: right, but if a change in stable nova went in | 13:11 |
sdague | or devstack | 13:11 |
maurosr | ok, will investigate, thought it would requires something in grenade to land.. but in stable would be a problem | 13:12 |
maurosr | sdague: tks | 13:12 |
*** matsuhashi has joined #openstack-qa | 13:13 | |
sdague | yeh, we still need to solve that forward problem, I respun that config patch over the holiday | 13:13 |
sdague | hopefully that gets us there | 13:13 |
*** yfried has quit IRC | 13:14 | |
*** matsuhashi has quit IRC | 13:17 | |
*** che-arne has joined #openstack-qa | 13:17 | |
*** yfried has joined #openstack-qa | 13:18 | |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds "list hosts" test case - Cinder https://review.openstack.org/56869 | 13:25 |
*** SergeyLukjanov has joined #openstack-qa | 13:26 | |
openstackrecheck | openstack/python-novaclient change: https://review.openstack.org/59360 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 13:32 |
*** yfried has quit IRC | 13:37 | |
*** yfried has joined #openstack-qa | 13:51 | |
*** sthaha has quit IRC | 13:53 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/59142 failed tempest because of: https://bugs.launchpad.net/bugs/1210483 | 13:54 |
*** dkranz has joined #openstack-qa | 13:54 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: scenario/network_basic_ops: detach floating-ip https://review.openstack.org/55810 | 13:58 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: scenario/network_basic_ops: reassociate floating-ip https://review.openstack.org/56889 | 14:00 |
*** jasondotstar has joined #openstack-qa | 14:03 | |
*** jhenner1 has joined #openstack-qa | 14:05 | |
*** jhenner has quit IRC | 14:06 | |
*** mkollaro has quit IRC | 14:08 | |
*** marun has quit IRC | 14:08 | |
*** marun has joined #openstack-qa | 14:09 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/56869 failed tempest because of: https://bugs.launchpad.net/bugs/1254872 | 14:12 |
dkranz | sdague, mtreinish : I'm ready to turn on build failing on errors. | 14:12 |
*** mriedem has joined #openstack-qa | 14:12 | |
dkranz | There was no response to my message about it last week. Do you think another message is in order before doing this? | 14:13 |
*** prekarat has joined #openstack-qa | 14:13 | |
mriedem | gate failures for errors in the logs? | 14:14 |
yfried | dkranz: ping | 14:17 |
sdague | dkranz: I think it's fine to flag it on at this point, just realize you'll probably need to keep an eye on it this week | 14:17 |
sdague | and react quickly to stuff coming in | 14:17 |
*** reaper has quit IRC | 14:18 | |
jorisroovers | Hi, was wondering whether someone has some time to give me some guidance on some test cases I've been working on | 14:26 |
jorisroovers | https://review.openstack.org/#/c/58419/ | 14:26 |
*** jhenner1 has quit IRC | 14:28 | |
*** weshay has joined #openstack-qa | 14:31 | |
*** psharma has quit IRC | 14:31 | |
*** marun has quit IRC | 14:32 | |
*** marun has joined #openstack-qa | 14:32 | |
openstackrecheck | openstack/glance change: https://review.openstack.org/58559 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 14:32 |
mtreinish | dkranz: nice | 14:39 |
*** jhenner has joined #openstack-qa | 14:40 | |
*** jhenner has quit IRC | 14:49 | |
*** marun has quit IRC | 14:51 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/devstack-gate: Up the default concurrency on tempest runs https://review.openstack.org/58605 | 14:57 |
mriedem | is 'recheck no bug' going away or was it 'reverify no bug'? | 14:58 |
mriedem | just in the last day i'm surprised and saddened at the number of 'recheck no bugs' i'm seeing on patches i'm on | 14:58 |
mtreinish | mriedem: I think reverify no bug is the one going away | 14:58 |
mriedem | that's what i thought | 14:59 |
mriedem | any thoughts on how to curb 'recheck no bug' besides coaching? | 14:59 |
mtreinish | we could change the keyword and not tell anyone | 14:59 |
mriedem | seems that most of the coaching on that happens in the mailing list, but if you're a serial 'recheck no bug'ger, you probably don't read the ML | 14:59 |
sdague | reverify | 15:01 |
sdague | mriedem: I usually yell at people in reviews about it | 15:02 |
mriedem | sdague: yeah, doing that now | 15:03 |
mriedem | i just feel like a broken record | 15:03 |
mriedem | i need to save a good one and just copy/paste that | 15:03 |
mtreinish | mriedem: write a bot :) | 15:04 |
sdague | yeh, I also had a script somewhere that built a list of people that did it a lot | 15:04 |
openstackrecheck | openstack/swift change: https://review.openstack.org/58500 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 15:05 |
*** yfried has quit IRC | 15:05 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/55419 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 15:08 |
*** rwsu has joined #openstack-qa | 15:14 | |
*** rdekel1 has left #openstack-qa | 15:19 | |
*** mkollaro has joined #openstack-qa | 15:20 | |
anteaya | hey sdague | 15:21 |
openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Neutron metering agent: add CRUD tests https://review.openstack.org/49719 | 15:22 |
anteaya | any change you could attend the Neutron meeting today 2100 utc? https://wiki.openstack.org/wiki/Network/Meetings#Neutron_Tempest_.28mlavalle.29 | 15:22 |
*** rdekel has joined #openstack-qa | 15:22 | |
anteaya | EmilienM is working on a patch: https://review.openstack.org/#/c/55846/ | 15:22 |
anteaya | and there seems to be an impasse as to whether qa wants negative tempest tests from neutron | 15:23 |
*** reaper has joined #openstack-qa | 15:23 | |
anteaya | a word from you could help | 15:23 |
openstackgerrit | Rossella Sblendido proposed a change to openstack-dev/devstack: Test for neutron full stack https://review.openstack.org/59430 | 15:25 |
jog0 | dkranz: https://review.openstack.org/#/c/58848/ !! | 15:25 |
jog0 | thiss hould be fun | 15:25 |
jog0 | sdague: any reason tempest doesn't use hacking 0.8 yet? | 15:31 |
jog0 | I thought you submitted a patch to bump up to it | 15:31 |
*** giulivo has quit IRC | 15:32 | |
*** adalbas has quit IRC | 15:36 | |
*** devlaps has joined #openstack-qa | 15:38 | |
openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/58605 failed tempest with an unrecognized error | 15:41 |
*** dims has joined #openstack-qa | 15:43 | |
openstackgerrit | David Kranz proposed a change to openstack/tempest: Remove ceilometer entries from whitelist https://review.openstack.org/59443 | 15:44 |
*** giulivo has joined #openstack-qa | 15:45 | |
sdague | I did | 15:46 |
sdague | hmmmm | 15:46 |
sdague | https://review.openstack.org/#/c/56308/ | 15:48 |
sdague | it aged out | 15:48 |
*** adalbas has joined #openstack-qa | 15:48 | |
sdague | let me refresh it | 15:49 |
*** jroovers has joined #openstack-qa | 15:52 | |
openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Bump hacking to 0.8 https://review.openstack.org/59455 | 15:53 |
*** mlavalle has joined #openstack-qa | 15:53 | |
sdague | jog0: you need more than that :) | 15:53 |
sdague | I'm retesting my patch locally | 15:54 |
*** jorisroovers has quit IRC | 15:55 | |
jog0 | sdague: that patch worked locally, jenkins will tell us for sure though | 15:55 |
*** jorisroovers has joined #openstack-qa | 15:55 | |
jog0 | sdague: ahh the tools change was made separately dependant patch) | 15:55 |
sdague | ok, I had tripped over tools | 15:56 |
sdague | jog0: instead you could review my er patches.... :) | 15:56 |
sdague | I do have this patch about ready to go | 15:56 |
jog0 | sdague: getting close to em | 15:57 |
*** jroovers has quit IRC | 15:57 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/57420 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 15:57 |
*** SergeyLukjanov has quit IRC | 15:58 | |
*** dkranz has quit IRC | 15:58 | |
openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Create client, tests and config for Ceilometer API https://review.openstack.org/55276 | 16:01 |
*** SergeyLukjanov has joined #openstack-qa | 16:01 | |
*** jorisroovers has quit IRC | 16:03 | |
*** SergeyLukjanov has quit IRC | 16:05 | |
*** jlibosva has quit IRC | 16:06 | |
openstackgerrit | A change was merged to openstack/tempest: Start failing logs with errors except neutron https://review.openstack.org/58848 | 16:09 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: update to hacking 0.8 https://review.openstack.org/56308 | 16:10 |
*** liudong_ has joined #openstack-qa | 16:13 | |
*** tkammer_ has quit IRC | 16:13 | |
*** dkranz has joined #openstack-qa | 16:13 | |
*** liudong_ has quit IRC | 16:13 | |
*** SushilKM has joined #openstack-qa | 16:14 | |
sdague | mtreinish: I'm going to harass you for some er reviews as well - https://review.openstack.org/#/q/status:open+project:openstack-infra/elastic-recheck+branch:master+topic:cleanups,n,z | 16:16 |
afazekas | sdague,cyeoh: https://review.openstack.org/#/c/40638/ looks like v3 bug sneaked in | 16:20 |
sdague | afazekas: yep | 16:21 |
openstackgerrit | Rafael Folco proposed a change to openstack-dev/devstack: This patch fixes wrong assumptions for bus types of disk and cdrom on ppc64. Qemu driver assumes IDE bus type for cdrom device, which is not supported on ppc arch. Adds capability to add --property key-value to the glance image-create command. https://review.openstack.org/59473 | 16:24 |
afazekas | This neutron vpnaas change started since long https://review.openstack.org/#/c/47348/ it should get some attention. | 16:27 |
*** rdekel has quit IRC | 16:29 | |
openstackgerrit | Rafael Folco proposed a change to openstack-dev/devstack: Generate glance image with SCSI bus type for ppc arch https://review.openstack.org/59473 | 16:30 |
openstackgerrit | afazekas proposed a change to openstack/tempest: Enable run_ssh and add floatingip + keypair auth https://review.openstack.org/50337 | 16:32 |
*** yfried has joined #openstack-qa | 16:32 | |
*** adalbas has quit IRC | 16:33 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/59452 failed tempest with an unrecognized error | 16:36 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/58720 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 16:41 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase https://review.openstack.org/59272 | 16:44 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir https://review.openstack.org/59252 | 16:44 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: method extraction for readability https://review.openstack.org/59263 | 16:44 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns https://review.openstack.org/59262 | 16:44 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols https://review.openstack.org/59261 | 16:44 |
*** SushilKM has quit IRC | 16:46 | |
*** adalbas has joined #openstack-qa | 16:47 | |
openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/57517 failed tempest because of: https://bugs.launchpad.net/bugs/1240256 | 16:50 |
openstackrecheck | openstack/nova change: https://review.openstack.org/55419 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 16:52 |
sdague | mtreinish: you have a pov on https://review.openstack.org/#/c/59252/ ? | 16:53 |
mtreinish | sdague: looks fine to me | 16:54 |
sdague | cool | 16:55 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: move queries.yaml into a queries subdir https://review.openstack.org/59252 | 16:57 |
*** openstackrecheck has quit IRC | 16:57 | |
*** openstackrecheck has joined #openstack-qa | 16:58 | |
openstackgerrit | Rossella Sblendido proposed a change to openstack-infra/devstack-gate: Test neutron full stack https://review.openstack.org/59484 | 16:58 |
*** dkranz has quit IRC | 16:59 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: wrapping the README.rst file to 80 cols https://review.openstack.org/59261 | 16:59 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: time to grow up and fit in 80 columns https://review.openstack.org/59262 | 16:59 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: method extraction for readability https://review.openstack.org/59263 | 16:59 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Fix E122,E126,E128 items in codebase https://review.openstack.org/59272 | 16:59 |
*** gszasz has quit IRC | 17:03 | |
*** safchain has quit IRC | 17:03 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/55276 failed tempest with an unrecognized error | 17:05 |
openstackrecheck | openstack/glance change: https://review.openstack.org/59471 failed tempest with an unrecognized error | 17:06 |
*** openstackrecheck has quit IRC | 17:08 | |
*** openstackrecheck has joined #openstack-qa | 17:08 | |
*** dkranz has joined #openstack-qa | 17:13 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/59467 failed tempest with an unrecognized error | 17:15 |
openstackrecheck | openstack/glance change: https://review.openstack.org/58056 failed tempest with an unrecognized error | 17:21 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/50337 failed tempest with an unrecognized error | 17:21 |
*** SergeyLukjanov has joined #openstack-qa | 17:21 | |
*** yfried has quit IRC | 17:26 | |
*** SushilKM has joined #openstack-qa | 17:27 | |
*** afazekas has quit IRC | 17:29 | |
*** fbo is now known as fbo_away | 17:34 | |
*** jpich has quit IRC | 17:37 | |
openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/59484 failed tempest with an unrecognized error | 17:41 |
*** adalbas has quit IRC | 17:44 | |
*** yfried has joined #openstack-qa | 17:47 | |
openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/57517 failed tempest with an unrecognized error | 17:51 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/57335 failed tempest with an unrecognized error | 17:53 |
openstackrecheck | openstack/requirements change: https://review.openstack.org/59196 failed tempest with an unrecognized error | 17:55 |
*** adalbas has joined #openstack-qa | 17:58 | |
mtreinish | dkranz: I was thinking it may make sense to add a special e-r irc message (and maybe gerrit) if there is an error in the logs failure (without a bug associated with it) | 18:03 |
mtreinish | now that we're enforcing it | 18:03 |
dkranz | mtreinish: Not sure we need it in gerrit. The build will fail and the error show at the end, right? | 18:04 |
dkranz | mtreinish: I don't know hot to add an irc message. | 18:05 |
dkranz | mtreinish: I guess a gerrit message could not hurt. | 18:05 |
mtreinish | true, but I think just in IRC it would helpful. Just for first pass filtering when looking for new bugs | 18:05 |
mtreinish | dkranz: it's not hard I can push out a commit to do it | 18:05 |
dkranz | mtreinish: Sure. That would be great. Have you seen any failures? | 18:06 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/56326 failed tempest with an unrecognized error | 18:06 |
mtreinish | dkranz: no I haven't really been looking though | 18:06 |
dkranz | mtreinish: I guess we'll find out :) | 18:06 |
mtreinish | I just wonder how many of the unrecognized errors are ERROR log messages vs real test failures | 18:06 |
dkranz | mtreinish: I'll check a few | 18:07 |
openstackrecheck | openstack/glance change: https://review.openstack.org/54376 failed tempest with an unrecognized error | 18:07 |
openstackrecheck | openstack/python-cinderclient change: https://review.openstack.org/59071 failed tempest with an unrecognized error | 18:09 |
dkranz | mtreinish: I looked at about 10 and none were log failures | 18:10 |
*** mestery_ has joined #openstack-qa | 18:11 | |
*** mestery has quit IRC | 18:11 | |
*** jhenner has joined #openstack-qa | 18:13 | |
*** openstackgerrit has quit IRC | 18:18 | |
*** SushilKM__ has joined #openstack-qa | 18:18 | |
*** openstackgerrit has joined #openstack-qa | 18:18 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/59443 failed tempest with an unrecognized error | 18:19 |
*** SushilKM has quit IRC | 18:19 | |
*** jlibosva has joined #openstack-qa | 18:20 | |
*** mestery_ is now known as mestery | 18:23 | |
*** rhsu has joined #openstack-qa | 18:32 | |
*** bill_az has joined #openstack-qa | 18:46 | |
openstackgerrit | Brent Eagles proposed a change to openstack/tempest: Add a control point for floating IP assignment https://review.openstack.org/59517 | 18:54 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/56076 failed tempest with an unrecognized error | 18:57 |
*** rhsu1 has joined #openstack-qa | 18:58 | |
*** rhsu has quit IRC | 19:00 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Switch the base URL to git.openstack.org https://review.openstack.org/56749 | 19:04 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/56738 failed tempest with an unrecognized error | 19:04 |
*** che-arne has quit IRC | 19:07 | |
openstackrecheck | openstack/cinder change: https://review.openstack.org/58736 failed tempest with an unrecognized error | 19:07 |
*** SushilKM__ has quit IRC | 19:13 | |
openstackgerrit | Arnaud Legendre proposed a change to openstack-dev/devstack: Attempt to retrieve the vmdk descriptor data-pair https://review.openstack.org/58356 | 19:14 |
*** fungi has quit IRC | 19:15 | |
openstackgerrit | Arnaud Legendre proposed a change to openstack-dev/devstack: Attempt to retrieve the vmdk descriptor data-pair https://review.openstack.org/58356 | 19:15 |
*** SushilKM__ has joined #openstack-qa | 19:17 | |
*** openstackrecheck has quit IRC | 19:19 | |
*** openstackrecheck has joined #openstack-qa | 19:19 | |
*** fungi has joined #openstack-qa | 19:20 | |
*** SushilKM has joined #openstack-qa | 19:21 | |
*** SushilKM__ has quit IRC | 19:21 | |
openstackgerrit | Mauro S. M. Rodrigues proposed a change to openstack-dev/grenade: Add clean configuration files to upgrade from H to I https://review.openstack.org/53874 | 19:24 |
*** annegentle has quit IRC | 19:28 | |
*** SushilKM has quit IRC | 19:28 | |
openstackrecheck | openstack/heat change: https://review.openstack.org/58202 failed tempest with an unrecognized error | 19:28 |
mtreinish | dkranz: did you have an e-r query that you were using for errors in logs? | 19:32 |
mtreinish | err elastic search query | 19:32 |
dkranz | mtreinish: No and I'm not sure we can. I was just looking at this. | 19:33 |
dkranz | mtreinish: It seems you cannot have regexp in logstash queries | 19:33 |
dkranz | mtreinish: There is some simple wildcarding but only for an individual word. | 19:33 |
dkranz | mtreinish: I could be not understanding something though. | 19:34 |
openstackrecheck | openstack/glance change: https://review.openstack.org/59514 failed tempest with an unrecognized error | 19:34 |
*** markmcclain has quit IRC | 19:35 | |
mtreinish | dkranz: actually all I was looking for was: http://logs.openstack.org/43/59443/1/check/check-tempest-devstack-vm-full/a46bb63/console.html#_2013-12-02_18_11_15_801 | 19:37 |
mtreinish | that's enough for me to group the failure | 19:38 |
sdague | dkranz: it's a search engine, which means no regex | 19:39 |
*** yfried has quit IRC | 19:39 | |
sdague | but if the issue is just that you have a beginning and ending of a thing, you can just specify message twice | 19:39 |
dkranz | sdague: I have to see if that is good enough to take a whitelist entry and determine if there are any matches. | 19:40 |
sdague | yep | 19:40 |
sdague | the syntax might be a little different | 19:40 |
dkranz | sdague: It is, with different escaped characters as well\ | 19:41 |
sdague | but I'm pretty sure you can get equivalence | 19:41 |
dkranz | sdague: I was looking at that but stopped to do some reviews | 19:41 |
dkranz | mtreinish: I'm not sure what you are looking for. | 19:41 |
dkranz | mtreinish: BTW, that failure is not real. It was my test to see if ceilometer had any real errors left. It does. | 19:41 |
dkranz | sdague: What is our policy on patches that are missing addCleanup because the code was copied from another method with that same problem? | 19:43 |
sdague | dkranz: we shouldn't land new buggy code | 19:43 |
mtreinish | dkranz: all I wanted was a way to check if there are errors in the logs. The string: "Logs have errors" does that for me | 19:43 |
dkranz | sdague: Should we request that the thing from which it was copied be fixed too? | 19:43 |
dkranz | mtreinish: Great! | 19:43 |
sdague | dkranz: sure, but I won't make that a condition of landing the first one | 19:44 |
dkranz | sdague: ok. We still have these kind of errors all over the place. We really need a more gc-like method for doing this. I'll think about it some more. | 19:44 |
*** RecheckWatchBotB has joined #openstack-qa | 19:54 | |
sdague | hmmm... so the default env doesn't have a logging config? | 19:55 |
*** RecheckWatchBotB has quit IRC | 19:55 | |
*** rwsu has quit IRC | 19:55 | |
*** rwsu has joined #openstack-qa | 19:56 | |
*** RecheckWatchBotB has joined #openstack-qa | 19:58 | |
*** rossella_s has quit IRC | 19:59 | |
*** annegentle has joined #openstack-qa | 20:00 | |
*** RecheckWatchBotB has quit IRC | 20:01 | |
*** RecheckWatchBotB has joined #openstack-qa | 20:04 | |
*** RecheckWatchBotB has quit IRC | 20:05 | |
*** RecheckWatchBotB has joined #openstack-qa | 20:06 | |
openstackrecheck | openstack/glance change: https://review.openstack.org/58056 failed tempest because of: https://bugs.launchpad.net/bugs/1254872 | 20:07 |
RecheckWatchBotB | openstack/glance change: https://review.openstack.org/58056 failed tempest because of: https://bugs.launchpad.net/bugs/1254872 | 20:07 |
sdague | ok, so it is working :) | 20:07 |
sdague | yay | 20:07 |
*** RecheckWatchBotB has quit IRC | 20:07 | |
*** hemna_ is now known as hemna | 20:09 | |
*** notmyname has quit IRC | 20:10 | |
*** fbo_away is now known as fbo | 20:12 | |
*** notmyname has joined #openstack-qa | 20:14 | |
openstackgerrit | Kaitlin Farr proposed a change to openstack/tempest: Adds scenario tests for volume encryption https://review.openstack.org/58018 | 20:28 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Mention errors in service logs for irc messages https://review.openstack.org/59534 | 20:29 |
mtreinish | dkranz: ^^^ I think that'll do what we were talking about earlier | 20:29 |
openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/58605 failed tempest with an unrecognized error | 20:30 |
dkranz | mtreinish: I'l a little confused. The code that sticks in "Logs have errors" only runs on successful builds. | 20:36 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Use isolation credentials for neutron api tests https://review.openstack.org/53459 | 20:37 |
mtreinish | dkranz: but it returns a 1 error code right which would fail the job. | 20:38 |
mtreinish | I believe that is what is set in the logstash metadata | 20:39 |
openstackrecheck | openstack/nova change: https://review.openstack.org/55062 failed tempest with an unrecognized error | 20:40 |
dkranz | mtreinish: Perhaps so but the code in devstack-gate only runs the log checker if the tempest run succeeded. | 20:41 |
*** rhsu1 has quit IRC | 20:41 | |
dkranz | mtreinish: So the status does not tell if the failure was tempest or log check | 20:41 |
dkranz | mtreinish: But it can't be both | 20:41 |
openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/57517 failed tempest with an unrecognized error | 20:54 |
*** marun has joined #openstack-qa | 20:58 | |
*** rhsu has joined #openstack-qa | 20:58 | |
*** rhsu has quit IRC | 20:58 | |
*** rossella_s has joined #openstack-qa | 20:58 | |
*** markmcclain has joined #openstack-qa | 20:59 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/51202 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 21:01 |
*** rhsu has joined #openstack-qa | 21:02 | |
mtreinish | dkranz: oh, ok cool that means I can make the message more specific then | 21:06 |
dkranz | mtreinish: That was my thought. | 21:06 |
mtreinish | sdague: about 59534 I think it's perfectly in scope for e-r. It already says that a run failed which is really useful for characterizing new failures. Why not make that slightly more specific and say whether it's a log failure or a test failure. (it's not doing this in gerrit just IRC) | 21:08 |
mtreinish | I think the value of the unrecognized failure messages from e-r are overlooked | 21:08 |
*** lifeless has quit IRC | 21:08 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Mention errors in service logs for irc messages https://review.openstack.org/59534 | 21:10 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/59531 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 21:10 |
*** mlavalle_ has joined #openstack-qa | 21:12 | |
*** SergeyLukjanov is now known as _SergeyLukjanov | 21:13 | |
sdague | mtreinish: honestly, until we actually cover more of the types of test runs (like grenade) I'd rather not expand scope there. Especially if it needs code vs. config. | 21:14 |
mtreinish | sdague: but this really isn't an expansion of scope. It's just checking if tempest failed or if there run failed because of errors in logs (which are mutually exclusive states) | 21:17 |
mtreinish | it's about delineating test failures (which are often race conditions) vs errors in the logs to prioritize which failures we look at in the irc messages | 21:18 |
mtreinish | if e-r is about race conditions this just helps isolate them from the log failures | 21:20 |
*** jhenner has quit IRC | 21:20 | |
*** dkranz has quit IRC | 21:20 | |
*** mlavalle has quit IRC | 21:20 | |
*** giulivo has quit IRC | 21:20 | |
*** mlavalle_ is now known as mlavalle | 21:20 | |
*** giulivo has joined #openstack-qa | 21:27 | |
sdague | well, at minimum it needs tests, and if this is just about the bot, it should be limited there. | 21:27 |
*** dkranz has joined #openstack-qa | 21:27 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/58018 failed tempest with an unrecognized error | 21:28 |
*** jhenner has joined #openstack-qa | 21:29 | |
*** wramthun has quit IRC | 21:29 | |
*** wramthun has joined #openstack-qa | 21:29 | |
*** che-arne has joined #openstack-qa | 21:36 | |
mtreinish | sdague: I was just trying to leverage the fancy modular classes you added and avoid doing the hard coded stuff we were doing before | 21:37 |
mtreinish | but I'll rework it to make it self contained in bot | 21:37 |
sdague | mtreinish: you can still use those class, but just pull them into the bot. It's like the fact we don't put the bug strings in code. | 21:40 |
*** weshay has quit IRC | 21:47 | |
*** jasondotstar has quit IRC | 21:50 | |
*** adalbas has quit IRC | 21:51 | |
*** lifeless has joined #openstack-qa | 21:53 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix rebuild_server() function https://review.openstack.org/59561 | 21:58 |
*** jamielennox is now known as jamielennox|away | 21:59 | |
dkranz | marun: You had some comments on https://review.openstack.org/#/c/57402/ that were not 100% agreed. Could you please +1/-1 ? | 22:01 |
marun | dkranz: sure | 22:02 |
marun | dkranz: do you have any opinion on the change in question? | 22:03 |
marun | dkranz: I still like seeing explicit asserts. The way the change masks them in object creation doesn't seem like an improvement to me. | 22:03 |
*** jlibosva has quit IRC | 22:04 | |
marun | dkranz: It reduces the self-documenting nature of the assertions. | 22:04 |
*** markmcclain has quit IRC | 22:04 | |
dkranz | marun: Oops. I am late and have to run. I'll respond later. | 22:05 |
marun | ok | 22:05 |
*** dkranz has quit IRC | 22:10 | |
*** rossella_s has quit IRC | 22:12 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/54647 failed tempest with an unrecognized error | 22:16 |
*** jamielennox|away is now known as jamielennox | 22:17 | |
*** zul has quit IRC | 22:18 | |
*** zul has joined #openstack-qa | 22:20 | |
openstackrecheck | openstack-dev/pbr change: https://review.openstack.org/59532 failed tempest with an unrecognized error | 22:20 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/53459 failed tempest with an unrecognized error | 22:21 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Use isolation credentials for neutron api tests https://review.openstack.org/53459 | 22:24 |
*** rwsu has quit IRC | 22:27 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/59486 failed tempest with an unrecognized error | 22:29 |
openstackrecheck | openstack/swift change: https://review.openstack.org/59533 failed tempest with an unrecognized error | 22:29 |
openstackgerrit | Edgar Magana proposed a change to openstack-dev/devstack: Adds entries for missing parameters in PLUMgrid plugin https://review.openstack.org/59565 | 22:29 |
openstackrecheck | openstack/keystone change: https://review.openstack.org/59547 failed tempest with an unrecognized error | 22:34 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/59548 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 22:35 |
*** mriedem has quit IRC | 22:39 | |
*** rwsu has joined #openstack-qa | 22:43 | |
openstackgerrit | Brant Knudson proposed a change to openstack/tempest: Allow user to not exist when assign role https://review.openstack.org/56106 | 22:46 |
openstackrecheck | openstack/keystone change: https://review.openstack.org/49460 failed tempest with an unrecognized error | 22:47 |
*** dims has quit IRC | 22:51 | |
*** fbo is now known as fbo_away | 22:53 | |
*** giulivo has quit IRC | 22:53 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/59560 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 23:00 |
*** obondarev has quit IRC | 23:03 | |
*** obondarev has joined #openstack-qa | 23:03 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: add support for installing the web dashboard https://review.openstack.org/59289 | 23:31 |
*** hemna has quit IRC | 23:31 | |
*** yfried has joined #openstack-qa | 23:32 | |
*** hemna has joined #openstack-qa | 23:32 | |
*** hemna has quit IRC | 23:33 | |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/59565 failed tempest with an unrecognized error | 23:33 |
*** hemna has joined #openstack-qa | 23:38 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/57497 failed tempest with an unrecognized error | 23:39 |
*** marun has quit IRC | 23:40 | |
*** mlavalle has left #openstack-qa | 23:47 | |
*** rhsu has quit IRC | 23:53 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/elastic-recheck: Update job names https://review.openstack.org/59573 | 23:58 |
*** rhsu has joined #openstack-qa | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!