Monday, 2013-10-21

openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add an es_query tool  https://review.openstack.org/5284700:11
*** matsuhashi has joined #openstack-qa00:25
*** jamielennox|away is now known as jamielennox00:32
*** adalbas has joined #openstack-qa00:48
*** nosnos has joined #openstack-qa00:56
*** ken1ohmichi has joined #openstack-qa00:56
*** nosnos has quit IRC01:10
*** nosnos has joined #openstack-qa01:11
*** dkranz has quit IRC01:17
*** SergeyLukjanov has joined #openstack-qa01:53
openstackrecheckopenstack/cinder change: https://review.openstack.org/52830 failed tempest because of: https://bugs.launchpad.net/bugs/123985602:07
openstackgerritDong Liu proposed a change to openstack/tempest: add some negative tests for security group:  https://review.openstack.org/5024902:18
*** adalbas has quit IRC02:19
openstackgerritzhangyanzi proposed a change to openstack/tempest: add some negative tests for attach volumes  https://review.openstack.org/5285302:26
*** dims has quit IRC02:34
openstackgerrithuangtianhua proposed a change to openstack/tempest: add role negative tests  https://review.openstack.org/5110702:36
openstackgerritzhangyanzi proposed a change to openstack/tempest: add some negative tests for attach volumes  https://review.openstack.org/5285302:43
*** SergeyLukjanov has quit IRC02:45
*** ivanzhu has joined #openstack-qa02:47
openstackgerritDong Liu proposed a change to openstack/tempest: add some negative tests for security group:  https://review.openstack.org/5024902:58
*** dkranz has joined #openstack-qa03:09
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Convert test_classifier to be a unit test  https://review.openstack.org/5073303:11
*** dkranz has quit IRC03:20
*** yfried has joined #openstack-qa03:24
ivanzhuafazekas: could you please to review this one : https://review.openstack.org/#/c/52562/03:26
*** ivanzhu1 has joined #openstack-qa03:27
*** ivanzhu has quit IRC03:30
openstackrecheckopenstack/tempest change: https://review.openstack.org/51107 failed tempest with an unrecognized error03:37
openstackgerrithuangtianhua proposed a change to openstack/tempest: add role negative tests  https://review.openstack.org/5110703:37
openstackrecheckopenstack/tempest change: https://review.openstack.org/50249 failed tempest with an unrecognized error03:39
*** ivanzhu1 has quit IRC03:40
openstackgerritA change was merged to openstack/tempest: add token test for token-api  https://review.openstack.org/5081103:48
*** Madan has joined #openstack-qa03:54
*** ivanzhu has joined #openstack-qa03:55
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add filenames to skip_tracker.py output  https://review.openstack.org/5223704:00
openstackgerritA change was merged to openstack-infra/elastic-recheck: don't explode if a query no longer has failures  https://review.openstack.org/5250904:04
*** openstackrecheck has quit IRC04:08
*** openstackrecheck has joined #openstack-qa04:08
*** yfried has quit IRC04:11
openstackrecheckopenstack/tempest change: https://review.openstack.org/50592 failed tempest with an unrecognized error04:17
openstackgerritA change was merged to openstack/tempest: add test for updating server's disk_config test  https://review.openstack.org/5204604:29
openstackgerritRyan Hsu proposed a change to openstack/tempest: Add support for testing VMware  https://review.openstack.org/5285904:44
openstackgerritRyan Hsu proposed a change to openstack/tempest: Add support for testing VMware  https://review.openstack.org/5285904:48
*** yfried|mob has joined #openstack-qa04:54
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds "list hosts" test case - Cinder  https://review.openstack.org/4893304:57
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds test to test_servers_negative  https://review.openstack.org/5286104:58
*** Dharmit has joined #openstack-qa05:09
*** marun has joined #openstack-qa05:18
*** matsuhashi has quit IRC05:20
*** matsuhashi has joined #openstack-qa05:21
*** raies has quit IRC05:28
*** raies has joined #openstack-qa05:29
*** ivanzhu has quit IRC05:32
openstackrecheckopenstack/tempest change: https://review.openstack.org/49892 failed tempest because of: https://bugs.launchpad.net/bugs/122400105:33
*** prekarat has joined #openstack-qa05:48
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add shelve/unshelve test of nova APIs  https://review.openstack.org/4989205:54
*** yfried|mob has quit IRC05:54
*** ivanzhu has joined #openstack-qa05:55
*** yfried has joined #openstack-qa05:57
openstackrecheckopenstack/tempest change: https://review.openstack.org/48933 failed tempest with an unrecognized error05:58
*** SergeyLukjanov has joined #openstack-qa06:00
*** nosnos has quit IRC06:00
*** nosnos_ has joined #openstack-qa06:00
*** matsuhas_ has joined #openstack-qa06:01
*** matsuhashi has quit IRC06:01
openstackgerritRyan Hsu proposed a change to openstack/tempest: Add support for testing VMware  https://review.openstack.org/5285906:03
*** thomasbiege has joined #openstack-qa06:08
*** thomasbiege2 has joined #openstack-qa06:09
*** thomasbiege has quit IRC06:12
*** yfried_mob has joined #openstack-qa06:25
mkodereryfried: ping06:27
yfriedmkoderer: pong06:27
mkodereryfried: regarding https://review.openstack.org/#/c/51459/906:27
yfriedmkoderer: yeah?06:28
mkodereryfried: why are those tests all flagged as smoke?06:28
mkodereris there a special reason for that?06:29
yfriedmkoderer: some of them were flagged before I moved them to negative test, so I continued. I'm still not sure how you decide if a test should be "smoke" or not06:29
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds test to test_servers_negative  https://review.openstack.org/5286506:30
mkodereryfried: ok I think we have to define that06:30
yfriedmkoderer: what should I do with those test for now?06:31
mkodereryfried: let me ask around a bit to see other opinions06:31
mkodererafazekas: ^ what do you think about negative test flagged as smoke tests?06:32
mkodererafazekas: IMHO we should only flag the real important ones06:32
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for getting server administrative password  https://review.openstack.org/5122306:34
yfriedmkoderer: this might be a stupid question, but why aren't negative tests important?06:39
*** jlibosva has joined #openstack-qa06:40
mkodereryfried: they are. but for me a smoke test means if this certain test doesn't work the whole systems is not usable.06:40
yfriedmkoderer: could you please merge this patch and discuss all changes on this (later) patch https://review.openstack.org/#/c/51590/, since any change will require manual rebase in the later patch?06:40
mkodereryfried: so if it's just a test that a wrong value must return a "BadRequest" I don't see that it's a smoke test06:40
yfriedmkoderer: so you think negative tests should be marked 'gate'?06:41
mkodereryfried: yes for most of them06:42
yfriedmkoderer: maybe, since they are already tagged "negative" we could treat negative+smoke different than regular "smoke"?06:43
mkodereryfried: or we simply find them and correct them in one patch afterwards06:44
*** matsuhas_ has quit IRC06:44
yfriedmkoderer: didn't understand you last msg06:44
*** matsuhashi has joined #openstack-qa06:45
mkodereryfried: I mean we just search for all negative tests that are flagged as smoke and replace smoke with gate06:45
yfriedmkoderer: np. could I do that on the later patch?06:46
mkodereryfried: yes06:46
*** nosnos has joined #openstack-qa06:47
yfriedmkoderer: in that case i will wait for this one to be merged and for https://review.openstack.org/#/c/51590/ to get -106:47
*** nosnos_ has quit IRC06:48
*** matsuhashi has quit IRC06:49
*** ivanzhu has quit IRC06:50
openstackgerritYair Fried proposed a change to openstack/tempest: api/network/security_groups_negative add testcases  https://review.openstack.org/5145906:54
openstackrecheckopenstack/tempest change: https://review.openstack.org/49892 failed tempest with an unrecognized error06:54
mkodereryfried: ok done06:55
yfriedmkoderer: it failed to merge. I did a manual rebase.06:56
openstackgerritYair Fried proposed a change to openstack/tempest: api/network/security_groups_negative add testcases  https://review.openstack.org/5145906:56
mkodereryfried: you mean this one https://review.openstack.org/#/c/51459/ ?06:56
yfriedmkoderer: yes. I have no idea what just happend06:58
openstackrecheckopenstack/nova change: https://review.openstack.org/52864 failed tempest with an unrecognized error07:00
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds test to test_servers_negative  https://review.openstack.org/5286907:00
*** matsuhashi has joined #openstack-qa07:02
*** ivanzhu has joined #openstack-qa07:04
openstackrecheckopenstack/tempest change: https://review.openstack.org/51223 failed tempest with an unrecognized error07:05
*** afazekas_ has joined #openstack-qa07:07
mkodereryfried: ah do you get an error while doing a "git review" ?07:08
*** thomasbiege2 has quit IRC07:20
*** yfried_mob has quit IRC07:22
*** yfried_mob has joined #openstack-qa07:22
yfriedmkoderer: yes07:25
openstackgerritYair Fried proposed a change to openstack/tempest: edit inheritence tree api/network/security_groups  https://review.openstack.org/5159007:26
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds test to test_servers_negative  https://review.openstack.org/5286107:30
openstackrecheckopenstack/neutron change: https://review.openstack.org/52810 failed tempest with an unrecognized error07:31
openstackrecheckopenstack/tempest change: https://review.openstack.org/52865 failed tempest with an unrecognized error07:33
yfriedmkoderer: could you please look at this? https://review.openstack.org/#/c/51459/07:36
yfriedI don't know if I have to do something or if all is ok07:36
*** SergeyLukjanov is now known as _SergeyLukjanov07:37
*** _SergeyLukjanov has quit IRC07:38
openstackgerritafazekas proposed a change to openstack-dev/devstack: Additional ssh tests with tempest  https://review.openstack.org/5287007:40
openstackgerritafazekas proposed a change to openstack-dev/devstack: Additional ssh tests with tempest  https://review.openstack.org/5287007:41
*** gszasz has joined #openstack-qa07:41
*** SergeyLukjanov has joined #openstack-qa07:44
*** SergeyLukjanov has quit IRC07:49
*** thomasbiege has joined #openstack-qa07:54
*** ivanzhu has quit IRC08:00
*** ivanzhu1 has joined #openstack-qa08:01
*** yfried_mob has quit IRC08:20
*** yfried_mob has joined #openstack-qa08:20
*** yassine has joined #openstack-qa08:20
*** thomasbiege2 has joined #openstack-qa08:25
*** safchain has joined #openstack-qa08:27
*** matsuhashi has quit IRC08:27
*** thomasbiege has quit IRC08:28
*** matsuhashi has joined #openstack-qa08:28
*** fbo_away is now known as fbo08:33
*** prekarat has quit IRC08:34
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds test to test_servers_negative  https://review.openstack.org/5286508:35
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API  https://review.openstack.org/4857808:46
*** prekarat has joined #openstack-qa08:55
raiesafazekas: during devstack installation with swift enabled "ERROR: Invalid OpenStack Nova credentials." is detected08:55
openstackgerritYang Yu proposed a change to openstack/tempest: Adding negative tests for glance API  https://review.openstack.org/4857808:57
*** HenryG_ has joined #openstack-qa08:59
*** nosnos has quit IRC09:01
*** nosnos has joined #openstack-qa09:01
*** HenryG has quit IRC09:01
*** ivanzhu1 has quit IRC09:03
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> tests  https://review.openstack.org/4798309:04
openstackgerritHoisaleshwara Madan V S proposed a change to openstack/tempest: Adds test to test_servers_negative  https://review.openstack.org/5286509:09
openstackgerritYang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> active tests  https://review.openstack.org/4798309:12
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for getting server administrative password  https://review.openstack.org/5122309:15
*** yfried_mob has quit IRC09:17
*** yfried_mob has joined #openstack-qa09:17
*** prekarat has quit IRC09:20
*** dkorn has joined #openstack-qa09:20
openstackgerritChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension  https://review.openstack.org/5054209:22
*** prekarat has joined #openstack-qa09:31
openstackgerritzhangyanzi proposed a change to openstack/tempest: Added some tests fo os-update_readonly_flag  https://review.openstack.org/5288509:32
*** yfried|mob has joined #openstack-qa09:37
*** yfried_mob has quit IRC09:37
*** yfried_mob has joined #openstack-qa09:37
openstackgerritChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension  https://review.openstack.org/5054209:38
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds "list hosts" test case - Cinder  https://review.openstack.org/4893309:39
*** yfried|mob has quit IRC09:41
openstackgerritAbhijeet Malawade proposed a change to openstack/tempest: Add testcases for security groups and images  https://review.openstack.org/4707909:45
*** thomasbiege2 has quit IRC09:48
*** ken1ohmichi has quit IRC09:50
*** thomasbiege1 has joined #openstack-qa09:55
*** thomasbiege1 has quit IRC10:08
*** yfried|mob has joined #openstack-qa10:08
*** gszasz has quit IRC10:11
*** yfried|mob has quit IRC10:13
*** jlibosva has quit IRC10:19
*** jlibosva has joined #openstack-qa10:19
*** raies has quit IRC10:20
*** raies has joined #openstack-qa10:21
*** prekarat has quit IRC10:22
openstackgerritwanghao proposed a change to openstack/tempest: add positive tests for volume  https://review.openstack.org/5081310:27
*** thomasbiege1 has joined #openstack-qa10:28
*** dkorn has quit IRC10:29
openstackgerritwanghao proposed a change to openstack/tempest: add negative test cases for create snapshots:  https://review.openstack.org/5021710:29
*** dkorn has joined #openstack-qa10:30
*** matsuhashi has quit IRC10:32
*** matsuhashi has joined #openstack-qa10:32
*** thomasbiege1 has quit IRC10:36
*** matsuhashi has quit IRC10:37
*** HenryG_ has quit IRC10:45
*** HenryG has joined #openstack-qa10:46
*** thomasbiege1 has joined #openstack-qa11:01
*** yfried|mob has joined #openstack-qa11:07
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add docstrings to ResultSet  https://review.openstack.org/5289811:14
*** dims has joined #openstack-qa11:22
*** mkollaro has joined #openstack-qa11:25
openstackrecheckopenstack/nova change: https://review.openstack.org/51129 failed tempest because of: https://bugs.launchpad.net/bugs/1225664 and https://bugs.launchpad.net/bugs/1240256 and11:26
*** giulivo has joined #openstack-qa11:32
openstackrecheckopenstack/neutron change: https://review.openstack.org/52697 failed tempest with an unrecognized error11:40
*** adalbas has joined #openstack-qa11:41
*** thomasbiege2 has joined #openstack-qa11:54
*** dkranz has joined #openstack-qa11:56
*** thomasbiege1 has quit IRC11:57
*** thomasbiege2 has quit IRC11:59
*** dkorn has quit IRC12:00
*** weshay has joined #openstack-qa12:03
*** yfried|mob has quit IRC12:04
*** yfried|mob has joined #openstack-qa12:04
*** dims has quit IRC12:09
*** gszasz has joined #openstack-qa12:22
*** dims has joined #openstack-qa12:22
openstackrecheckopenstack/neutron change: https://review.openstack.org/52697 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 and https://bugs.launchpad.net/bugs/1235486 and12:28
openstackrecheckopenstack/neutron change: https://review.openstack.org/28137 failed tempest because of: https://bugs.launchpad.net/bugs/123963712:29
*** anteaya has joined #openstack-qa12:33
openstackgerritYair Fried proposed a change to openstack/tempest: added support for security_groups under neutron  https://review.openstack.org/5089212:37
*** yfried has quit IRC12:47
*** Madan has quit IRC12:52
*** dkranz has quit IRC12:54
*** yfried|mob has quit IRC13:01
*** yfried|mob has joined #openstack-qa13:02
*** yfried has joined #openstack-qa13:04
*** thomasbiege1 has joined #openstack-qa13:06
*** yfried has quit IRC13:06
*** nosnos has quit IRC13:09
*** nosnos has joined #openstack-qa13:10
*** Dharmit has quit IRC13:10
*** yfried has joined #openstack-qa13:12
yfriedmkoderer: ping13:13
*** nosnos has quit IRC13:14
openstackrecheckopenstack/tempest change: https://review.openstack.org/50892 failed tempest with an unrecognized error13:14
*** yfried|mob has quit IRC13:14
sdaguemtreinish: once you get in, you able to look at my patch series - https://review.openstack.org/#/q/status:open+project:openstack-infra/elastic-recheck+branch:master+topic:result_set,n,z13:19
openstackrecheckopenstack/nova change: https://review.openstack.org/44714 failed tempest with an unrecognized error13:21
openstackrecheckopenstack/neutron change: https://review.openstack.org/52914 failed tempest because of: https://bugs.launchpad.net/bugs/1235437 and https://bugs.launchpad.net/bugs/1240256 and13:33
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for getting server administrative password  https://review.openstack.org/5122313:35
*** yfried has quit IRC13:38
mkoderersdague: mtreinish: what's your opinion on negative test flagged as 'smoke'?13:40
sdaguemkoderer: it's probably not a smoke test unless there is a really good reason for it13:42
sdagueman, we should probably have a tags clean up discussion13:43
mkoderersdague: exaclty13:43
*** thomasbiege1 has quit IRC13:43
mkoderersdague: +1 I'll put it on the QA meeting agenda13:43
sdagueactually, can you submit a summit session on it, as there is actually another piece in this around the scenario tests that I want to make sure we cover, as I'd like to delete the smoke tag entirely in Icehouse13:44
sdaguefwiw, I tentatively have your stress test session at 2:50 on Wed of the summit13:45
mkoderersdague: ok I register one, and I would really like to speak about the negative tests...13:46
sdagueoh, right, that's a good topic as well13:46
sdagueit's a good thing I left us a few slots open :)13:46
mkodererdon't know if David wanted to cover this topic13:46
sdagueyeh, maybe once he gets around today we can sort it13:47
sdagueafazekas: you know if dkranz is going to be on today?13:47
mkoderersdague:  I will register it and I don't have any problem if somebody else will take it over13:47
sdaguecool, thank!13:48
sdagueit's actually easier for me if something's in the system, then I can play with the scheduler to make sure things look fine13:48
seiflotfyhey guys13:48
seiflotfy+1 for mkoderer13:48
*** prekarat has joined #openstack-qa13:48
seiflotfy:P13:48
seiflotfyvote mkoderer for president (or prime minister of germany)13:48
seiflotfy:P13:48
seiflotfysdague: +1?13:49
sdaguedtroyer: also, when you get around to it today, can I get you to submit a grenade update and next steps session for QA track13:49
sdague:)13:49
sdaguedtroyer: dansmith actually had some interesting thoughts about breaking out the upgrade bits so that we could have a shared set up upgrade scripts between grenade and other package based models, so that's something I'd like to ensure we take time to chat about13:50
*** thomasbiege1 has joined #openstack-qa13:52
*** dkranz has joined #openstack-qa13:54
sdagueah, speak of the devil :)13:54
sdaguedkranz: do you think the stack trace work deserves a summit session?13:55
dkranzsdague: Yes, but only if all projects can participate.13:55
dkranzsdague: I thought about this but was doubtful which is why I didn't propose one13:56
mkodererdkranz: did you plan the register a talk about the negative tests?13:56
*** DennyZhang has joined #openstack-qa13:56
mkodererdkranz: or should I do it?13:56
dkranzmkoderer: I thought there was one in the etherpad. If not, sure go ahead.13:56
mkodererdkranz: thats right13:57
*** thomasbiege1 has quit IRC13:57
mkodererdkranz: but sdague told me to register some13:57
mkodererdkranz: let me have a look13:57
dkranzsdague: I think the plan for errors is pretty clear. Are there some issues you thought should be discussed?13:57
dkranzsdague: I got disconected and don't have the irc for the last hour. Should I look at the log?13:58
sdagueI don't think there was much13:58
mkoderersdague: ok so dkranz will talk about the negative tests, I will add a point about the test tags14:00
dkranzsdague: ok. It might be better to just socialize the error thing since we are running out of slots for things that are less well developed14:00
mkoderersdague: should I add it on etherpad or directly on summit.opentstack? or both?14:00
sdaguedkranz: sure14:00
sdaguemkoderer: so at this point we should probably just start working over in summit.openstack.org14:01
*** dkorn has joined #openstack-qa14:01
dkranzsdague, mkoderer : I have to back to sleep, I mean to a meeting, but am clear for the week starting at 3 today.14:01
sdaguedkranz: ok, so I think I didn't realize that you had an item in the etherpad that wasn't yet submit14:01
sdagueheh14:01
sdagueok, that's fine14:01
dkranzsdague: Sorry, I didn't realize we were supposed to move them to  the summit site.14:01
sdagueafazekas: can you propose in the fedora one as well14:01
dkranzmkoderer: Can you do that for me with the negative test thing? Else I will do it this afternoon.14:02
sdaguedkranz: yeh, my bad, things were busy last week. I was just going through everything this morning to try to nail some things down14:02
mkodererdkranz: ok I will14:02
dkranzsdague: np14:02
sdaguemkoderer: so I actually think the test tags thing is something I'll pull back into my lore session (which I think I really should radically rename)14:03
dkranzmkoderer: Just saw your comment about fuzz testing14:03
dkranzmkoderer: I thought the goal was to replace negative tests14:04
mkoderersdague: ok I am fine with it14:04
dkranzmkoderer: And was just saying we are far from that14:04
dkranzmkoderer: I don't actually have an objectoin to what you were proposing14:04
mkodererdkranz: yes your are right. I wasn't clear at the end with that.14:05
mkodererdkranz: I agree that we are far from replacing the negative test with it14:05
mkodererdkranz: but as a little fuzzy tool we could use it14:06
dkranzmkoderer: Sure. I'm just not sure it solves a "big" problem14:06
mkodererdkranz: yes, I think I need to work on it a bit more, maybe it will :)14:07
mkodererdkranz: just wanted to give a code exmaple for the discussions that we have on the summit14:07
dkranzmkoderer: Sure, thanks!14:07
mtreinishdkranz: hey sorry about that e-r sample config mistake, I never actually used the option I just let paramiko use the default key. It looks like someone else had the same issue too: https://review.openstack.org/#/c/52734/14:07
dkranzmtreinish: I'm just happy I still have some hair left :)14:08
*** ianw has quit IRC14:08
*** jamielennox has quit IRC14:08
*** jamielennox has joined #openstack-qa14:09
dkranzmtreinish: I approved that change.14:09
dkranzmtreinish: Still dont understand why it worked last week. Probably never will.14:09
*** jaypipes has joined #openstack-qa14:09
*** ChanServ sets mode: +o jaypipes14:09
mtreinishdkranz: were you running daemonized? It could have just been using the default key location.14:11
*** ianw has joined #openstack-qa14:11
openstackgerritA change was merged to openstack-infra/elastic-recheck: The key's value is misleading in the config sample.  https://review.openstack.org/5273414:11
*** jamielennox is now known as jamielennox|away14:13
mkoderersdague: negative tests: http://summit.openstack.org/cfp/details/32514:14
*** dkranz has quit IRC14:14
sdaguemkoderer: great14:14
*** openstackrecheck has quit IRC14:16
*** openstackrecheck has joined #openstack-qa14:16
openstackgerritQing Xin Meng proposed a change to openstack/tempest: Add test for getting server administrative password  https://review.openstack.org/5122314:17
*** jasondotstar|afk is now known as jasondotstar14:19
openstackgerritIvan-Zhu proposed a change to openstack/tempest: add tests for show and update of Flavor Extra Spec API extension  https://review.openstack.org/5293214:24
*** bill_az_ has joined #openstack-qa14:24
openstackrecheckopenstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/122566414:25
openstackrecheckopenstack/cinder change: https://review.openstack.org/52872 failed tempest with an unrecognized error14:26
*** fungi has quit IRC14:41
*** liudong has joined #openstack-qa14:42
openstackrecheckopenstack/tempest change: https://review.openstack.org/47079 failed tempest with an unrecognized error14:43
openstackgerritDenis Egorenko proposed a change to openstack-dev/devstack: Added Savanna Project  https://review.openstack.org/5060114:52
openstackgerritDenis Egorenko proposed a change to openstack-dev/devstack: Added Savanna Project  https://review.openstack.org/5060114:55
*** liudong has quit IRC14:59
*** fungi has joined #openstack-qa15:02
*** dkorn has quit IRC15:03
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/41869 failed tempest with an unrecognized error15:04
openstackrecheckopenstack/swift change: https://review.openstack.org/51876 failed tempest because of: https://bugs.launchpad.net/bugs/1225664 and https://bugs.launchpad.net/bugs/1240256 and15:10
mtreinishsdague: hey can you push this through again: https://review.openstack.org/#/c/50733/ it just needed a rebase15:12
*** marun has quit IRC15:14
*** marun has joined #openstack-qa15:15
*** yfried has joined #openstack-qa15:18
*** marun has quit IRC15:19
*** thomasbiege1 has joined #openstack-qa15:21
sdaguedone15:23
openstackgerritafazekas proposed a change to openstack/tempest: Simplify xml/json client selection  https://review.openstack.org/3693715:24
mtreinishsdague: cool thanks15:24
*** jlibosva has quit IRC15:25
openstackgerritA change was merged to openstack-infra/elastic-recheck: Convert test_classifier to be a unit test  https://review.openstack.org/5073315:25
*** mlavalle has joined #openstack-qa15:25
sdaguemtreinish: so is - https://review.openstack.org/#/c/52731/ just a -1 because of the whitespace?15:27
*** openstackrecheck has quit IRC15:27
mtreinishsdague: yes :) Although after reviewing everything I think you should rebase things a bit15:27
*** openstackrecheck has joined #openstack-qa15:27
mtreinishmake the first patch just the ResultSet object in it's finished state15:27
mtreinishand go from there15:27
*** dkranz has joined #openstack-qa15:28
sdagueyeh, so 1, 2, and 6 all merge?15:28
*** marun has joined #openstack-qa15:28
sdagueactually, how about I squash 1, 2, & 6 into one, 3 & 4 into one patch (4 is really unit tests for 3), and 5 hangs out on it's own15:29
mtreinishyes except for on 2 I liked that the elasticRecheck.py (and other file conversions) conversions were separate15:29
mtreinishsdague: sure whatever makes it easier for me to follow15:29
sdagueok, you want the elasticRecheck port in a seperate patch still?15:30
mtreinishit doesn't really matter15:30
sdagueok, let me spin a rebase set and see how that looks15:30
jog0have a fun bug for us to watch: http://logs.openstack.org/72/52872/2/check/check-tempest-devstack-vm-postgres-full/e6b90f0/testr_results.html.gz  http://logs.openstack.org/69/41869/22/check/check-tempest-devstack-vm-full/e445d64/testr_results.html.gz  @message:"MismatchError: {u'service':" AND  @message:"not in [{" AND @fields.filename:"console.html"15:34
*** yfried_mob has joined #openstack-qa15:34
mtreinishjog0: great another issue with hosts/az/aggregates15:36
*** dkranz has quit IRC15:36
jog0mtreinish: :( can you find a bug and file the elastic-recheck query for it15:37
*** sthaha has quit IRC15:37
*** yfried has quit IRC15:37
mtreinishjog0: sure15:37
jog0I would but my batter is precious (forgot adapter at hotel)  and at linuxcon EU15:37
jog0battery*15:37
*** liudong has joined #openstack-qa15:38
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add SearchResultSet and Hit objects  https://review.openstack.org/5269815:39
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: refactor templates into query_builder  https://review.openstack.org/5281815:40
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add an es_query tool  https://review.openstack.org/5284715:40
mtreinishI really like the people who just match test names when using the recheck tool. Even if the error is completely different15:40
sdaguemtreinish: ok, that's the squashed results15:41
openstackrecheckopenstack/heat change: https://review.openstack.org/52943 failed tempest with an unrecognized error15:42
sdaguees_query still needs the other comments you put into it, but I'll deal with that later today15:42
mtreinishsdague: ok, I was just about to -1 it because of that, I'll hold off then15:43
sdaguemtreinish: that's on top of the queue though, the ones below can get a real look15:43
mtreinishsdague: hahaha, you broke the unit tests I just asked you to push through15:44
sdagueheh, nice15:44
sdagueok, let me see what's up there15:45
sdagueyay, unit tests15:45
openstackgerritDong Liu proposed a change to openstack/tempest: add some negative tests for security group:  https://review.openstack.org/5024915:47
sdagueok, I forgot you'd actually mocked out this part. Ok, so I'll unwind that later today, as it requires some tweaking to get your faking integrated with my faking15:49
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Add query for bug 1225024  https://review.openstack.org/5296015:50
mtreinishjog0: ^^^15:50
mtreinishif you've still got the battery15:50
mtreinishsdague: ok, yeah I think I left a comment about that somewhere on one of the patches15:51
sdaguesomething else I realized yesterday, is we can get es to automatically do histogram faceting, which would be useful for the graphing work15:51
clarkbsdague: yup kibana3 does that. I noticed late last week15:52
sdagueclarkb: right, but more importantly I know how to structure es queries to get the same thing :)15:52
sdagueso we'll get it in the es results15:53
*** yfried has joined #openstack-qa15:53
clarkbcool15:53
sdagueok, out for a bit, back later15:53
openstackgerritA change was merged to openstack-infra/elastic-recheck: Change test_queries from logical AND to OR  https://review.openstack.org/5016015:55
*** thomasbiege1 has quit IRC15:56
*** openstackrecheck has quit IRC15:57
*** openstackrecheck has joined #openstack-qa15:57
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Remove bug 1229475 from queries.yaml  https://review.openstack.org/5296316:00
openstackrecheckopenstack/heat change: https://review.openstack.org/52946 failed tempest because of: https://bugs.launchpad.net/bugs/122400116:00
openstackgerritMatthew Treinish proposed a change to openstack-infra/elastic-recheck: Add query for bug 1225024  https://review.openstack.org/5296016:03
*** yfried has quit IRC16:06
*** yfried_mob has quit IRC16:09
*** yfried|mob has joined #openstack-qa16:09
openstackrecheckopenstack/neutron change: https://review.openstack.org/52949 failed tempest with an unrecognized error16:17
*** fbo is now known as fbo_away16:18
*** DennyZhang has quit IRC16:18
openstackrecheckopenstack/nova change: https://review.openstack.org/36417 failed tempest because of: https://bugs.launchpad.net/bugs/123985616:19
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Fix typos and thinkos in docs  https://review.openstack.org/5251416:21
*** yfried|mob has quit IRC16:32
*** yfried|mob has joined #openstack-qa16:32
*** prekarat has quit IRC16:33
*** dkranz has joined #openstack-qa16:34
*** yassine has quit IRC16:38
*** safchain has quit IRC16:41
*** yfried has joined #openstack-qa16:50
openstackrecheckopenstack/neutron change: https://review.openstack.org/52934 failed tempest because of: https://bugs.launchpad.net/bugs/122400116:55
*** rhsu has joined #openstack-qa17:05
*** dkranz has quit IRC17:12
openstackrecheckopenstack/neutron change: https://review.openstack.org/49415 failed tempest with an unrecognized error17:14
*** SergeyLukjanov has joined #openstack-qa17:15
*** liudong has quit IRC17:19
*** mlavalle has quit IRC17:21
*** yfried|mob has quit IRC17:29
adalbasmtreinish, do you know if anyone is looking at the tempest smoke tests related to this : https://bugs.launchpad.net/tempest/+bug/1023131 ?17:29
*** yfried|mob has joined #openstack-qa17:29
mtreinishadalbas: yeah maurosr said he'd handle it soon. It's actually that there should be scenario tests to cover the old exercises not a smoke tag.17:31
openstackrecheckopenstack/swift change: https://review.openstack.org/51876 failed tempest with an unrecognized error17:32
sdaguemtreinish: ok, so I'm trying to decypher your magic query bit17:32
mtreinishsdague: good luck, I barely understand why it works :)17:32
sdagueheh, yeh, well I'm trying to figure out what you are attempt to do there, so I can make it work with the resultset changes17:33
*** mriedem has joined #openstack-qa17:33
adalbasmaurosr : are you going to look at that?17:33
*** SergeyLukjanov is now known as _SergeyLukjanov17:35
*** _SergeyLukjanov has quit IRC17:35
adalbasmtreinish, does this need to wait to get more discussions in the summit, or could it be addressed sooner. I think I saw one summit proposal about that.17:37
mtreinishsdague: basically the fakes work in 2 ways, depending on what it's testing.17:37
mtreinishif it's test classifier it will only pass the matches found check if it's the magic query17:37
mtreinishfor is_ready it just passes urls_matches for all the fake queries by making a fake response with the right entries17:39
sdagueoh, I think I figured out the voodoo you did17:40
sdaguethe urls_match is a short circuit of something that the data structure doesn't actually do17:41
sdagueman, this is hack tastic :)17:43
mtreinishheh, yeah it kind of is. To be fair you +A'd it twice17:45
sdagueyeh, well, we're still moving quick here. I think I actually got it figured out17:46
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add SearchResultSet and Hit objects  https://review.openstack.org/5269817:47
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: refactor templates into query_builder  https://review.openstack.org/5281817:47
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add an es_query tool  https://review.openstack.org/5284717:47
sdagueok, that should pass unit tests17:47
sdaguethe thing that tripped me up was the hits: 217:48
sdaguethat you were actually checking for in the urls_match stub17:48
*** openstackrecheck has quit IRC17:49
*** SergeyLukjanov has joined #openstack-qa17:52
*** SergeyLukjanov has quit IRC17:53
*** SergeyLukjanov has joined #openstack-qa17:54
sdagueheh, well I did collapse down 3 commits to one... I guess I could have thrown out some of the messages17:57
mtreinishI think that's the first time I've said the commit message had too much in it17:58
sdague:)17:58
maurosrmtreinish: adalbas: sorry, I'm back.. and yes it's one of things that I'm working18:11
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add config feature verification script  https://review.openstack.org/5298818:18
*** marun has quit IRC18:26
*** yfried|mob has quit IRC18:27
*** yfried|mob has joined #openstack-qa18:27
*** SergeyLukjanov is now known as _SergeyLukjanov18:27
*** _SergeyLukjanov has quit IRC18:28
*** SergeyLukjanov has joined #openstack-qa18:32
*** afazekas_ has quit IRC19:07
*** mlavalle has joined #openstack-qa19:10
mriedemsdague: ping19:17
*** yfried|mob has quit IRC19:25
*** yfried|mob has joined #openstack-qa19:25
*** yfried|mob has quit IRC19:26
*** gszasz has quit IRC19:39
*** wramthun has quit IRC19:58
*** dkranz has joined #openstack-qa20:02
*** wramthun has joined #openstack-qa20:02
openstackgerritDan Smith proposed a change to openstack-dev/devstack: Allow starting nova-compute manually  https://review.openstack.org/5300920:10
dansmithsdague: not sure if that's what you were intending, but I hope so :)20:12
sdaguedansmith: cool, I'll check it out in a bit20:16
*** jamielennox|away is now known as jamielennox20:23
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add volume extensions tests  https://review.openstack.org/5301120:28
*** atiwari has joined #openstack-qa20:28
*** thomasbiege1 has joined #openstack-qa20:29
*** wramthun has quit IRC20:33
*** wramthun has joined #openstack-qa20:33
*** giulivo has quit IRC20:39
*** yfried has quit IRC20:42
*** thomasbiege1 has quit IRC20:49
*** giulivo has joined #openstack-qa20:53
*** giulivo has quit IRC21:02
*** anteaya_ has joined #openstack-qa21:03
*** weshay has quit IRC21:04
*** anteaya has quit IRC21:06
*** mlavalle has quit IRC21:07
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Use isolation credentials for neutron api tests  https://review.openstack.org/5301621:08
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Use isolation credentials for neutron api tests  https://review.openstack.org/5301621:09
*** mlavalle has joined #openstack-qa21:09
*** giulivo has joined #openstack-qa21:16
*** openstackrecheck has joined #openstack-qa21:19
openstackgerritSreeram Yerrapragada proposed a change to openstack/tempest: Fix issue with ImagesOneServerJSON test.  https://review.openstack.org/5301721:19
openstackgerritDan Smith proposed a change to openstack-dev/devstack: Allow starting nova-compute manually  https://review.openstack.org/5300921:24
*** marun has joined #openstack-qa21:29
*** dkranz has quit IRC21:35
*** mriedem has quit IRC21:46
*** giulivo has quit IRC21:58
*** anteaya_ has quit IRC22:12
openstackrecheckopenstack/requirements change: https://review.openstack.org/52137 failed tempest because of: https://bugs.launchpad.net/bugs/122566422:12
clarkbsdague: mtreinish ^ woot I didn't break it :)22:12
clarkbI checked the graphs too and those looked like they were being updated22:13
mtreinishclarkb: \o/22:13
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add lib/stackforge to let us test wsme / pecan from git  https://review.openstack.org/5302622:13
sdagueclarkb: nice22:13
dansmithsdague: so, my change to devstack fails because it changes the calling convention that grenade uses for start_nova22:15
*** SergeyLukjanov is now known as _SergeyLukjanov22:15
*** _SergeyLukjanov is now known as SergeyLukjanov22:15
sdagueso it failed because start_nova no longer starts nova-compute, and a running nova-compute is required. So part of this is going to be changing the stable branch to have stop_nova not stop the compute, right?22:19
sdagueor have the upgrade script for nova start up old nova-compute22:20
*** atiwari has quit IRC22:21
openstackgerritSreeram Yerrapragada proposed a change to openstack/tempest: Fix issue with ImagesOneServerJSON test.  https://review.openstack.org/5301722:22
*** jhenner has quit IRC22:23
*** SergeyLukjanov is now known as _SergeyLukjanov22:28
*** _SergeyLukjanov is now known as SergeyLukjanov22:28
dansmithsdague: um. doesn't grenade run the patch I'm adding?22:32
sdagueyes22:33
dansmithsdague: so, my patch modifies devstack to require you to call nova_start_compute before nova_start, which grenade isn't doing22:35
dansmithhow about I make nova_start call nova_start_compute and nova_start_all (which is the old nova_start)22:36
dansmiththen I can modify grenade22:36
dansmiththen I can go back and clean that up in devstack?22:36
dansmiththis was why I used an arg in the first version of this, to keep the calling convention the same...22:37
sdagueso I thought what you wanted was to not stop old nova-compute?22:37
sdaguebut yeh, " nova_start call nova_start_compute and nova_start_all (which is the old nova_start)" sounds fine22:38
*** dims has quit IRC22:38
sdaguethat will let us stepwise there22:38
sdaguereally, we don't yet have stable/havana => master grenade running (it's still stable/grizzly => master). maurosr is working on that22:39
sdagues/really/realize/22:39
clarkbyeah we haven't uplifted everything yet22:40
clarkbthe stable jobs need adding too22:40
dansmithI'm just trying to get the requirements in place that we need in order to do this when we're ready22:40
sdaguedansmith: yeh, the split, and the top level function to call them both I think is goodness22:40
sdagueprobably call it nova_start_rest instead of nova_start_all22:41
dansmithexactly what I just typed22:41
sdague:)22:42
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add lib/stackforge to let us test wsme / pecan from git  https://review.openstack.org/5302622:45
openstackgerritDan Smith proposed a change to openstack-dev/devstack: Allow starting nova-compute manually  https://review.openstack.org/5300922:45
*** SergeyLukjanov has quit IRC22:45
*** SergeyLukjanov has joined #openstack-qa22:46
*** SergeyLukjanov is now known as _SergeyLukjanov22:46
*** _SergeyLukjanov has quit IRC22:48
openstackrecheckopenstack/horizon change: https://review.openstack.org/52837 failed tempest with an unrecognized error22:49
*** mkollaro has quit IRC22:52
openstackgerritSean Dague proposed a change to openstack-dev/devstack: add lib/stackforge to let us test wsme / pecan from git  https://review.openstack.org/5302622:53
*** dims has joined #openstack-qa22:55
sdaguedansmith: so one last thing on that, do you want to split the stop as well?22:59
dansmithsdague: hmm, because you want to make me start up a full old stack and then roll everything forward except for nova-compute, is that right?23:01
sdagueyep23:01
dansmithand that's better ... why again? :)23:01
sdaguemostly because we have the infrastructure to do it that way already :)23:01
dansmithhrm.. that's a sucky reason for the test to take longer and do more work, but... I guess23:02
sdaguewell, maybe there is another way to adapt grenade here23:02
dansmithwell, I'm okay with going that way to start, but I think it'd be nice to avoid having to do that23:03
sdagueanyway, it's dinner time here, so I'm done for the night. Maybe catch up on it tomorrow.23:03
dansmiththe other thing I was thinking of,23:03
dansmithif we just got another copy of the tree somewhere, we might be able to just run normal devstack without n-cpu,23:03
dansmithand then start up the other n-cpu in a venv from the other tree23:03
dansmithI thought you might like that better than my switcheroo23:04
dansmithokay23:04
sdagueyeh, so it's actually the dueling trees thing in the gate that's kind of challenging to get right23:04
openstackgerritSreeram Yerrapragada proposed a change to openstack/tempest: Fix issue with ImagesOneServerJSON test  https://review.openstack.org/5301723:04
dansmitheven if we local clone the  other one?23:04
dansmithanyway, we can continue tomorrow23:04
sdaguesure23:05
sdaguewe should also involve some infra folks, because some of the concerns are about the devstack node setups23:05
dansmithokay23:05
*** rockyg has joined #openstack-qa23:06
*** SergeyLukjanov has joined #openstack-qa23:10
*** rockyg has quit IRC23:10
*** mlavalle has left #openstack-qa23:14
openstackgerritSam Alba proposed a change to openstack-dev/devstack: Fixed Docker package install + upgraded Docker to the latest 0.6.4  https://review.openstack.org/5303123:19
*** sthaha has joined #openstack-qa23:33
*** SergeyLukjanov is now known as _SergeyLukjanov23:33
*** _SergeyLukjanov is now known as SergeyLukjanov23:33
*** SergeyLukjanov is now known as _SergeyLukjanov23:34
*** _SergeyLukjanov is now known as SergeyLukjanov23:34
*** hemna is now known as hemnafk23:49

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