| jogo | nova volume-list thinks it does | 00:00 |
|---|---|---|
| mtreinish | jogo: I think it does, but I'm not sure | 00:01 |
| mtreinish | jogo: we have tempest tests that pass filter params to volume list on cinder v1 and v2 | 00:01 |
| jogo | mtreinish: hmm I didn't find it | 00:04 |
| jogo | mtreinish: I'll look again | 00:04 |
| *** yjiang5 has quit IRC | 00:04 | |
| mtreinish | jogo: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/volume/test_volumes_list.py#n144 | 00:05 |
| *** Sukhdev has quit IRC | 00:05 | |
| *** mlavalle has quit IRC | 00:05 | |
| mtreinish | the default is to run with v2, but there are subclasses for v1 and v1/v2 xml | 00:05 |
| jogo | there is no regex parsing based on names etc in there | 00:06 |
| mtreinish | there is a name filter test, but no regex test | 00:07 |
| mtreinish | jogo: the filter gets passed right to sqlalchemy: http://git.openstack.org/cgit/openstack/cinder/tree/cinder/db/sqlalchemy/api.py#n1318 | 00:11 |
| mtreinish | although off the top of my head I don't know if that means a regex will work or not | 00:12 |
| jogo | mtreinish: looks like it isn't a regex | 00:13 |
| *** mitz has quit IRC | 00:15 | |
| *** mitz_ has joined #openstack-qa | 00:18 | |
| *** sabeen has quit IRC | 00:26 | |
| *** zzzeek has quit IRC | 00:29 | |
| *** dims has quit IRC | 00:32 | |
| *** yamahata has joined #openstack-qa | 00:36 | |
| *** dims has joined #openstack-qa | 00:37 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix another_role variable for swift https://review.openstack.org/118105 | 00:42 |
| *** suneelb has quit IRC | 00:45 | |
| *** Sukhdev has joined #openstack-qa | 00:48 | |
| *** weshay has quit IRC | 00:50 | |
| *** zz_dimtruck is now known as dimtruck | 00:55 | |
| *** dims_ has joined #openstack-qa | 01:03 | |
| *** dims has quit IRC | 01:05 | |
| *** dims_ has quit IRC | 01:06 | |
| *** dims has joined #openstack-qa | 01:06 | |
| hockeynut | hi mtrenish - looking at a CR about adding barbican tests - https://review.openstack.org/#/c/117032/ - just want to clarify things. We have tempest tests that run in the devstack gate and they live in the barbican repo. Our impression was that would be where the tests lived, and how they would be run. | 01:06 |
| hockeynut | mtreinish ^^ sorry for the typoh in your name :-) | 01:07 |
| *** jyuso has joined #openstack-qa | 01:13 | |
| *** dims has quit IRC | 01:16 | |
| *** dims has joined #openstack-qa | 01:19 | |
| openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Add test list_servers_filtered_by_name_regex https://review.openstack.org/119081 | 01:22 |
| *** wuhg has joined #openstack-qa | 01:23 | |
| *** dims has quit IRC | 01:25 | |
| *** dims has joined #openstack-qa | 01:26 | |
| *** dims_ has joined #openstack-qa | 01:26 | |
| *** dims has quit IRC | 01:30 | |
| *** dimtruck is now known as zz_dimtruck | 01:37 | |
| *** dims_ has quit IRC | 01:42 | |
| *** Sukhdev has quit IRC | 01:44 | |
| *** dims has joined #openstack-qa | 01:47 | |
| *** dims has quit IRC | 01:50 | |
| *** dims has joined #openstack-qa | 01:50 | |
| *** nosnos has joined #openstack-qa | 01:51 | |
| *** dims_ has joined #openstack-qa | 01:53 | |
| *** dims has quit IRC | 01:54 | |
| *** mestery has quit IRC | 01:57 | |
| *** mestery_ has joined #openstack-qa | 01:57 | |
| *** dims_ has quit IRC | 02:04 | |
| *** dims has joined #openstack-qa | 02:04 | |
| *** pcrews has quit IRC | 02:06 | |
| *** dims has quit IRC | 02:09 | |
| *** pcrews has joined #openstack-qa | 02:23 | |
| *** openstackgerrit has quit IRC | 02:33 | |
| *** sabeen has joined #openstack-qa | 02:45 | |
| *** adalbas has quit IRC | 02:51 | |
| *** arnaud has joined #openstack-qa | 02:53 | |
| *** rbak has joined #openstack-qa | 02:55 | |
| *** sabeen has quit IRC | 02:57 | |
| *** sabeen has joined #openstack-qa | 02:58 | |
| *** yamahata has quit IRC | 02:58 | |
| *** chandankumar has joined #openstack-qa | 03:01 | |
| *** arnaud__ has joined #openstack-qa | 03:02 | |
| *** Dafna has quit IRC | 03:04 | |
| *** rbak has quit IRC | 03:11 | |
| *** zz_jgrimm has quit IRC | 03:13 | |
| *** zz_jgrimm has joined #openstack-qa | 03:14 | |
| *** openstack has joined #openstack-qa | 03:42 | |
| *** fungi has joined #openstack-qa | 03:42 | |
| *** ZZelle has joined #openstack-qa | 03:44 | |
| *** dosaboy has joined #openstack-qa | 03:46 | |
| *** bdperkin has joined #openstack-qa | 03:46 | |
| *** andreaf_ has joined #openstack-qa | 03:46 | |
| *** yamahata_ has joined #openstack-qa | 03:46 | |
| *** torandu has joined #openstack-qa | 03:46 | |
| *** masayukig has joined #openstack-qa | 03:46 | |
| *** mriedem1 has joined #openstack-qa | 03:46 | |
| *** luqas has joined #openstack-qa | 03:46 | |
| *** pcrews has joined #openstack-qa | 03:46 | |
| *** dmellado has joined #openstack-qa | 03:46 | |
| *** dutsmoc has joined #openstack-qa | 03:46 | |
| *** hockeynut_ has joined #openstack-qa | 03:46 | |
| *** dtroyer has joined #openstack-qa | 03:46 | |
| *** mrodden has joined #openstack-qa | 03:46 | |
| *** stevebak` has joined #openstack-qa | 03:46 | |
| *** melwitt1 has joined #openstack-qa | 03:46 | |
| *** flaper87 has joined #openstack-qa | 03:46 | |
| *** StevenK has joined #openstack-qa | 03:46 | |
| *** davidlenwell_ has joined #openstack-qa | 03:46 | |
| *** enikanorov__ has joined #openstack-qa | 03:46 | |
| *** zz_jgrimm has joined #openstack-qa | 03:46 | |
| *** arnaud__ has joined #openstack-qa | 03:46 | |
| *** chandankumar has joined #openstack-qa | 03:46 | |
| *** sabeen has joined #openstack-qa | 03:46 | |
| *** arnaud has joined #openstack-qa | 03:46 | |
| *** mestery_ has joined #openstack-qa | 03:46 | |
| *** nosnos has joined #openstack-qa | 03:46 | |
| *** wuhg has joined #openstack-qa | 03:46 | |
| *** jyuso has joined #openstack-qa | 03:46 | |
| *** mitz_ has joined #openstack-qa | 03:46 | |
| *** aimon has joined #openstack-qa | 03:46 | |
| *** mwagner_lap has joined #openstack-qa | 03:46 | |
| *** jpich has joined #openstack-qa | 03:46 | |
| *** sbadia has joined #openstack-qa | 03:46 | |
| *** Shrews has joined #openstack-qa | 03:46 | |
| *** mdenny has joined #openstack-qa | 03:46 | |
| *** ilyashakhat has joined #openstack-qa | 03:46 | |
| *** amotoki has joined #openstack-qa | 03:46 | |
| *** cmyster has joined #openstack-qa | 03:46 | |
| *** annegentle has joined #openstack-qa | 03:46 | |
| *** toabctl has joined #openstack-qa | 03:46 | |
| *** boris-42 has joined #openstack-qa | 03:46 | |
| *** liusheng has joined #openstack-qa | 03:46 | |
| *** enikanorov has joined #openstack-qa | 03:46 | |
| *** krtaylor has joined #openstack-qa | 03:46 | |
| *** shardy has joined #openstack-qa | 03:46 | |
| *** jamielennox has joined #openstack-qa | 03:46 | |
| *** psedlak has joined #openstack-qa | 03:46 | |
| *** bnemec has joined #openstack-qa | 03:46 | |
| *** jhesketh has joined #openstack-qa | 03:46 | |
| *** morganfainberg has joined #openstack-qa | 03:46 | |
| *** SpamapS has joined #openstack-qa | 03:46 | |
| *** smurashov has joined #openstack-qa | 03:46 | |
| *** afazekas has joined #openstack-qa | 03:46 | |
| *** gmann_ has joined #openstack-qa | 03:46 | |
| *** ianw has joined #openstack-qa | 03:46 | |
| *** danfinch has joined #openstack-qa | 03:46 | |
| *** jgriffith has joined #openstack-qa | 03:46 | |
| *** sweston has joined #openstack-qa | 03:46 | |
| *** harlowja has joined #openstack-qa | 03:46 | |
| *** kmartin has joined #openstack-qa | 03:46 | |
| *** ekhugen has joined #openstack-qa | 03:46 | |
| *** peterstac has joined #openstack-qa | 03:46 | |
| *** adam_g has joined #openstack-qa | 03:46 | |
| *** wendar has joined #openstack-qa | 03:46 | |
| *** esmute has joined #openstack-qa | 03:46 | |
| *** dhellmann has joined #openstack-qa | 03:46 | |
| *** devananda has joined #openstack-qa | 03:46 | |
| *** nikhil_k has joined #openstack-qa | 03:46 | |
| *** zz_johnthetubagu has joined #openstack-qa | 03:46 | |
| *** gema has joined #openstack-qa | 03:46 | |
| *** sileht has joined #openstack-qa | 03:46 | |
| *** freyes has joined #openstack-qa | 03:46 | |
| *** maurosr has joined #openstack-qa | 03:46 | |
| *** rossella_s has joined #openstack-qa | 03:46 | |
| *** tristanC has joined #openstack-qa | 03:46 | |
| *** eglynn-office has joined #openstack-qa | 03:46 | |
| *** cyeoh has joined #openstack-qa | 03:46 | |
| *** lyxus has joined #openstack-qa | 03:46 | |
| *** smoser has joined #openstack-qa | 03:46 | |
| *** beisner has joined #openstack-qa | 03:46 | |
| *** early has joined #openstack-qa | 03:46 | |
| *** timello has joined #openstack-qa | 03:46 | |
| *** sc68cal has joined #openstack-qa | 03:46 | |
| *** EmilienM has joined #openstack-qa | 03:46 | |
| *** pleia2 has joined #openstack-qa | 03:46 | |
| *** zhiyan has joined #openstack-qa | 03:46 | |
| *** zz_dimtruck has joined #openstack-qa | 03:46 | |
| *** jogo has joined #openstack-qa | 03:46 | |
| *** mattoliverau has joined #openstack-qa | 03:46 | |
| *** chmouel has joined #openstack-qa | 03:46 | |
| *** notmyname has joined #openstack-qa | 03:46 | |
| *** JoshNang has joined #openstack-qa | 03:46 | |
| *** kashyap has joined #openstack-qa | 03:46 | |
| *** SergeyLukjanov has joined #openstack-qa | 03:46 | |
| *** jvrbanac has joined #openstack-qa | 03:46 | |
| *** asselin__ has joined #openstack-qa | 03:46 | |
| *** sdague has joined #openstack-qa | 03:46 | |
| *** akuznetsova has joined #openstack-qa | 03:46 | |
| *** mikal has joined #openstack-qa | 03:46 | |
| *** mugsie has joined #openstack-qa | 03:46 | |
| *** mkoderer has joined #openstack-qa | 03:46 | |
| *** DinaBelova has joined #openstack-qa | 03:46 | |
| *** tteggel has joined #openstack-qa | 03:46 | |
| *** dansmith has joined #openstack-qa | 03:46 | |
| *** zaro has joined #openstack-qa | 03:46 | |
| *** clarkb has joined #openstack-qa | 03:46 | |
| *** mtreinish has joined #openstack-qa | 03:46 | |
| *** lifeless has joined #openstack-qa | 03:46 | |
| *** jeblair has joined #openstack-qa | 03:46 | |
| *** anteaya has joined #openstack-qa | 03:46 | |
| *** zigo has joined #openstack-qa | 03:46 | |
| *** rahmu has joined #openstack-qa | 03:46 | |
| *** mgagne has joined #openstack-qa | 03:46 | |
| *** mordred has joined #openstack-qa | 03:46 | |
| *** hemna_ has joined #openstack-qa | 03:46 | |
| *** SlickNik has joined #openstack-qa | 03:46 | |
| *** arnaud__ has quit IRC | 03:46 | |
| *** jroll has quit IRC | 03:46 | |
| *** jroll has joined #openstack-qa | 03:46 | |
| *** fungi is now known as Guest37209 | 03:46 | |
| *** ZZelle is now known as Guest81941 | 03:46 | |
| *** melwitt1 has quit IRC | 03:47 | |
| *** coolsvap has joined #openstack-qa | 03:47 | |
| *** pcrews has quit IRC | 04:06 | |
| *** HenryG has joined #openstack-qa | 04:07 | |
| *** Poornima has joined #openstack-qa | 04:08 | |
| *** Dafna has joined #openstack-qa | 04:14 | |
| *** Dafna has quit IRC | 04:14 | |
| *** Sukhdev has joined #openstack-qa | 04:19 | |
| *** pcrews has joined #openstack-qa | 04:21 | |
| *** mriedem1 has quit IRC | 04:25 | |
| *** arnaud has quit IRC | 04:29 | |
| *** stevebak` is now known as stevebaker | 04:32 | |
| *** dutsmoc is now known as comstud | 04:33 | |
| *** asselin__ has quit IRC | 04:36 | |
| *** hemna_ has quit IRC | 04:36 | |
| *** hemna_ has joined #openstack-qa | 04:37 | |
| *** pcrews has quit IRC | 04:38 | |
| *** arnaud has joined #openstack-qa | 04:43 | |
| *** neeti has joined #openstack-qa | 04:47 | |
| *** kmartin has quit IRC | 04:47 | |
| *** ildikov has joined #openstack-qa | 04:49 | |
| *** chandankumar has quit IRC | 04:51 | |
| *** Madan has joined #openstack-qa | 05:05 | |
| *** chandankumar has joined #openstack-qa | 05:17 | |
| *** jlibosva has joined #openstack-qa | 05:23 | |
| *** sabeen has quit IRC | 05:26 | |
| *** k4n0 has joined #openstack-qa | 05:33 | |
| *** cmyster_ has joined #openstack-qa | 05:37 | |
| *** yfried has joined #openstack-qa | 05:45 | |
| *** Santosh has joined #openstack-qa | 05:47 | |
| *** harlowja is now known as harlowja_away | 05:47 | |
| Santosh | afazekas: hi | 05:49 |
| Santosh | is there any issue in installing devstack... with dnsmasq version... | 05:54 |
| Santosh | that version should be greater than 2.63... and anypointer how to resolve this ? | 05:55 |
| *** afazekas_ has joined #openstack-qa | 05:55 | |
| yfried | andreaf_: ping | 05:56 |
| *** cmyster_ has quit IRC | 06:13 | |
| *** cmyster_ has joined #openstack-qa | 06:14 | |
| *** cmyster_ has quit IRC | 06:19 | |
| *** Sukhdev has quit IRC | 06:19 | |
| *** cmyster_ has joined #openstack-qa | 06:21 | |
| *** flaper87 has quit IRC | 06:22 | |
| *** flaper87 has joined #openstack-qa | 06:22 | |
| *** evgenyf has joined #openstack-qa | 06:26 | |
| yfried | gmann_: ping | 06:34 |
| gmann_ | yfried: hi | 06:35 |
| yfried | gmann_: https://review.openstack.org/#/c/119670/2/ | 06:36 |
| yfried | gmann_: are you familiar with how secgroups are implemented in Neutron? | 06:36 |
| gmann_ | yfried: from neutron side, not much. | 06:37 |
| gmann_ | yfried: but for this test i think we should keep sec grp tests there and if needed / feasible we can add more networking interfaces in future | 06:38 |
| yfried | gmann_: it's not doing anything, so there's no problem with keeping it | 06:39 |
| yfried | gmann_: secgroup is implemented as part of the neutron link between VMs and network. the purpose of adding secgroup (if I understand the patch you linked to correctly) is to test some stress/scale issues that we've seen in neutron secgroups | 06:40 |
| gmann_ | yfried: yes, and for nova-network as it run for gate-tempest-dsvm-large-ops too | 06:42 |
| yfried | gmann_: in actuality, since we don't create network connections, the secgroups remain only a single line in the DB | 06:42 |
| yfried | gmann_: ok | 06:42 |
| *** arnaud has quit IRC | 06:42 | |
| yfried | gmann_: so if you move it to use novanetwork as well, then I have no idea about what the secgroup would do | 06:42 |
| gmann_ | yfried: for neutron, it will be in use with your VM-network connectivity tests enhancement | 06:43 |
| yfried | gmann_: but the secgroup prefomance issues are neutron related, so that remains un-tested until my patch comes in | 06:43 |
| gmann_ | yfried: may be you are right but from https://review.openstack.org/#/c/50514/2 it looks like sec grp was targeted for novanetwork too (as it run on both gate job) | 06:45 |
| yfried | gmann_: anyway, if you just fix the commit message without removing secgroup, I'm ok | 06:45 |
| gmann_ | yfried: and thats my concern to remove those as those can be used by enhancing the tests | 06:45 |
| gmann_ | yfried: ok let me check | 06:46 |
| gmann_ | yfried: yes, i will do that | 06:46 |
| gmann_ | yfried: just FYI(may be you know) for your idea of having VM network connection in this test - in both gate job this tests runs with 100 VM | 06:48 |
| yfried | gmann_: though, if you could please check to see if adding secgroup without any network has any meaning in Nova (unlike Neutron) it would be nice to make sure we are not doing code stuff without any effect | 06:48 |
| yfried | gmann_: or not thinking we are covering stuff when we actually don't | 06:49 |
| yfried | gmann_: yeah, this is definatly not gonna work with 100 VMs. I'm gonna go for 30 first | 06:50 |
| gmann_ | yfried: ok. I will check nova side | 06:51 |
| yfried | gmann_: tnx | 06:52 |
| gmann_ | yfried: thanks for review. | 06:52 |
| yfried | gmann_: also, why do we must make it work with novanetwork gate? isn't it enough to have only neutron-large-ops? | 06:52 |
| yfried | gmann_: IMO, this is just adds unnecessary gates that check the same thing twice | 06:54 |
| *** gszasz has joined #openstack-qa | 06:56 | |
| gmann_ | yfried: i think till novanetwork exist we should but if it nothing to do with networking services, yes i also agree that its enough to run once | 06:56 |
| gmann_ | yfried: but I am not sure about complete background for - https://review.openstack.org/#/c/50514/2 . | 06:56 |
| *** e0ne has joined #openstack-qa | 06:57 | |
| *** jpich has quit IRC | 06:57 | |
| *** cmyster_ has quit IRC | 06:59 | |
| *** e0ne has quit IRC | 06:59 | |
| *** gfidente has joined #openstack-qa | 07:07 | |
| *** mpaolino has joined #openstack-qa | 07:13 | |
| yfried | jogo: ping | 07:14 |
| yfried | gmann_: I think jogo (the writer of the patch) is a Neutron guy. I know salvatore is. | 07:15 |
| gmann_ | yfried: Jogo is nova core too. | 07:16 |
| yfried | gmann_: then I don't know... | 07:16 |
| gmann_ | yfried: its same in nova too. till we do not attach any network with VM, sec grp is just a DB entry. | 07:17 |
| *** mpavlase has joined #openstack-qa | 07:18 | |
| yfried | gmann_: so I stand by my comment. don't add secgroup if you don't have network | 07:18 |
| gmann_ | yfried: :) | 07:18 |
| gmann_ | yfried: but as it create sec grp and VM so on. it tests the sec grp creation at least | 07:18 |
| gmann_ | yfried :as this test should have volume attaching too, which is future enhancement | 07:19 |
| *** bkopilov has joined #openstack-qa | 07:19 | |
| yfried | gmann_: if we need it in the future, we can add it in the future. for now, it makes people think we are testing secgroup for scale while we actually don't | 07:20 |
| gmann_ | yfried: humm, I am not much hard on keeping it. But i would like to have jogo or other core opinion before removal (as it was added intensionally) . | 07:22 |
| yfried | bkopilov: the Jenkins failed in non-voting gate, so you shouldn't worry about it if it's not related to your code | 07:23 |
| bkopilov | yfried, ok thanks | 07:23 |
| yfried | bkopilov: you got the -1 from Citrix XenServer CI | 07:23 |
| gmann_ | yfried: i agree with your point as if it is in scenario tests it should have some scenario to tests/verify instead just code coverage | 07:24 |
| bkopilov | yfried, What is exactly non voting section ? | 07:24 |
| yfried | bkopilov: there's a button called "Toggle CI" at the bottom of the gerrit screen. if you press it, it will add the CI results as comments to your patch, and you can take a look at the problems | 07:24 |
| yfried | gmann_: so make sure the patch isn't going in before you get their review | 07:25 |
| yfried | bkopilov: non-voting is a gate that's being tested. once they are sure it won't post bad review (like the one you got) it would move to "voting" | 07:26 |
| gmann_ | yfried: yes, that make sense. I will add there. | 07:26 |
| *** Albert has joined #openstack-qa | 07:27 | |
| yfried | bkopilov: in your case, these are a bunch of neutron tests that aren't 100% ready yet | 07:27 |
| yfried | bkopilov: you can get more info on this at #openstack-infra | 07:27 |
| bkopilov | yfried, great thanks | 07:27 |
| yfried | bkopilov: but your problem is Citrix. I'm seeing it too on my patches. don't know who to contact for help with it... | 07:28 |
| yfried | masayukig: andreaf_: do you guys know what's up with the citrix CI? it's voting down patches and failing without even running tests | 07:29 |
| bkopilov | yfried, is there any way to re-run those tests ? | 07:30 |
| yfried | for jenkins "recheck" | 07:33 |
| *** tkammer has joined #openstack-qa | 07:33 | |
| yfried | for other CI you can see in their comment details | 07:33 |
| yfried | bkopilov: ^ | 07:33 |
| *** k4n0_ has joined #openstack-qa | 07:35 | |
| yfried | bkopilov: please talk to clarkb in #openstack-infra | 07:36 |
| bkopilov | yfried, ok | 07:36 |
| *** jlanoux has joined #openstack-qa | 07:38 | |
| *** k4n0 has quit IRC | 07:39 | |
| mkoderer | andreaf: ping | 07:40 |
| flaper87 | chmouel: do you think this makes sense for devstack? https://review.openstack.org/#/c/120022/ | 07:41 |
| flaper87 | not sure what's the best way to achieve that without adding it to devstack | 07:41 |
| chmouel | flaper87: that sounds good to me | 07:42 |
| flaper87 | chmouel: awesome, thanks for the feedback | 07:42 |
| chmouel | flaper87: we already install all oslo.* from git | 07:42 |
| flaper87 | yeah, that's what convinced me | 07:43 |
| flaper87 | I was a bit worried about stable branches | 07:43 |
| flaper87 | they'll still clone from master | 07:43 |
| flaper87 | I mean, glance_store master | 07:43 |
| flaper87 | ah no, | 07:43 |
| flaper87 | for get it | 07:43 |
| flaper87 | :D | 07:43 |
| chmouel | :) | 07:43 |
| flaper87 | I saw we also have stable/icehouse for oslo.messaging | 07:44 |
| flaper87 | so we'll certainly have sstable/juno for glance_store | 07:44 |
| chmouel | that's correct i don't think that would be a problem | 07:44 |
| Albert | Hi ppl, my patch made some tests fail, because it changed the call of some functions, and those tests where expecting a the "old" call, how I shall proceed? shall I change the tests? | 07:46 |
| mkoderer | Albert: which patch? and is it already merged? | 07:50 |
| Albert | One I've submitted yesterday for an improvement | 07:53 |
| Albert | https://review.openstack.org/#/c/119762/ | 07:54 |
| *** dosaboy has quit IRC | 07:54 | |
| *** dosaboy has joined #openstack-qa | 07:54 | |
| Albert | This is the test: http://logs.openstack.org/62/119762/2/check/gate-tempest-python27/184e4d0 | 07:54 |
| Albert | That is failing since I've changed the call for exec.command | 07:54 |
| yfried | bkopilov: citrix gate should be disabled. if you push a new patch it won't post comments on it | 07:56 |
| Albert | My problem is not with citrix, but with gate-tempest-python27 | 07:58 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 08:06 | |
| mkoderer | Albert: you need to adapt the unit tests, too | 08:09 |
| Albert | ok, thanks mkoderer | 08:11 |
| *** safchain has joined #openstack-qa | 08:14 | |
| *** andreaf_ is now known as andreaf | 08:25 | |
| *** Guest81941 has quit IRC | 08:26 | |
| *** jordanP has joined #openstack-qa | 08:28 | |
| *** nmagnezi has joined #openstack-qa | 08:38 | |
| *** beisner has quit IRC | 08:40 | |
| yfried | jordanP: masayukig: andreaf: could you please review https://review.openstack.org/#/c/102700/ ? | 08:41 |
| *** tkammer has quit IRC | 08:42 | |
| *** e0ne has joined #openstack-qa | 08:43 | |
| *** xianghuihui has joined #openstack-qa | 08:45 | |
| *** zaro has quit IRC | 08:49 | |
| *** xianghuihuihui has joined #openstack-qa | 08:49 | |
| *** xianghuihui has quit IRC | 08:49 | |
| *** aix has joined #openstack-qa | 08:53 | |
| *** e0ne has quit IRC | 08:54 | |
| *** Dafna has joined #openstack-qa | 08:55 | |
| *** trinaths has joined #openstack-qa | 08:56 | |
| *** rdekel has joined #openstack-qa | 09:02 | |
| *** beisner has joined #openstack-qa | 09:02 | |
| *** e0ne has joined #openstack-qa | 09:03 | |
| *** cdent has joined #openstack-qa | 09:06 | |
| *** rdekel has quit IRC | 09:10 | |
| *** gfidente has quit IRC | 09:13 | |
| *** safchain has quit IRC | 09:15 | |
| *** safchain has joined #openstack-qa | 09:15 | |
| chandankumar | hello | 09:18 |
| chandankumar | when i am trying to run cinder multibackend test through tempest , http://fpaste.org/132037/25425714/ | 09:18 |
| chandankumar | i have devstack installed on fedora 20 | 09:19 |
| *** openstackgerrit has joined #openstack-qa | 09:19 | |
| chandankumar | How can i debug in order to fix it? | 09:19 |
| mkoderer | chandankumar: please check the cinder logs | 09:20 |
| *** evgenyf has quit IRC | 09:21 | |
| *** trinaths has left #openstack-qa | 09:25 | |
| *** evgenyf has joined #openstack-qa | 09:26 | |
| *** rdekel has joined #openstack-qa | 09:27 | |
| *** rdekel has quit IRC | 09:27 | |
| *** yfried has quit IRC | 09:28 | |
| *** SridharG has joined #openstack-qa | 09:28 | |
| *** yfried has joined #openstack-qa | 09:29 | |
| *** rdekel has joined #openstack-qa | 09:29 | |
| yfried | andreaf: thank you for your review. are you ok with my explanations? | 09:31 |
| *** gmann_ has quit IRC | 09:32 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in scenarios https://review.openstack.org/102700 | 09:32 |
| *** gmann has joined #openstack-qa | 09:32 | |
| *** safchain has quit IRC | 09:33 | |
| *** rdekel has quit IRC | 09:35 | |
| *** yfried_ has joined #openstack-qa | 09:35 | |
| *** chandankumar has quit IRC | 09:35 | |
| *** e0ne has quit IRC | 09:38 | |
| *** yfried has quit IRC | 09:38 | |
| *** mkollaro has joined #openstack-qa | 09:39 | |
| *** e0ne has joined #openstack-qa | 09:44 | |
| andreaf | yfried_, yes some changes I still would have made them in a separate patch, but the change is good quality so it should merge now - thank you for it | 09:45 |
| *** chandankumar has joined #openstack-qa | 09:46 | |
| *** aix has quit IRC | 10:00 | |
| yfried_ | andreaf: tnx | 10:01 |
| yfried_ | andreaf: we can move on the the 3rd one now | 10:01 |
| coolsvap | andreaf, hi can you review https://review.openstack.org/116435 ? | 10:11 |
| *** rdekel has joined #openstack-qa | 10:11 | |
| *** jyuso has quit IRC | 10:14 | |
| *** aix has joined #openstack-qa | 10:16 | |
| *** SridharG has quit IRC | 10:18 | |
| *** yfried__ has joined #openstack-qa | 10:24 | |
| *** pkoniszewski has joined #openstack-qa | 10:27 | |
| *** yfried_ has quit IRC | 10:28 | |
| *** Longgeek has joined #openstack-qa | 10:33 | |
| yfried__ | andreaf: one of your comments suggested to use: "X = X or DEFAULT" instead of "if not X" | 10:36 |
| yfried__ | andreaf: this was discussed and decided that is presents inconsistency issues and also problems with non-explicit False (such as 0) | 10:37 |
| *** Madan has quit IRC | 10:40 | |
| *** xianghuihuihui has quit IRC | 10:40 | |
| *** SridharG has joined #openstack-qa | 10:42 | |
| *** gfidente has joined #openstack-qa | 10:43 | |
| andreaf | yfried__, ok but that is only different if you test for if x is not None - doing if not X is identical to X or Y | 10:47 |
| *** mpaolino has quit IRC | 10:47 | |
| andreaf | yfried__, testing "if not X" means that is is not None, not [], not {}, not 0 etc | 10:49 |
| andreaf | yfried__, depending on the case that may be ok or not - for the scenario test in question it was fine to include the case {} in the check | 10:50 |
| *** e0ne has quit IRC | 10:51 | |
| *** qba73 has joined #openstack-qa | 10:54 | |
| openstackgerrit | Satoru Moriya proposed a change to openstack-dev/devstack: Update parameter checking with provider network https://review.openstack.org/118147 | 10:55 |
| *** ildikov_ has joined #openstack-qa | 11:02 | |
| yfried__ | andreaf: I know. just mentioning | 11:03 |
| *** ildikov has quit IRC | 11:05 | |
| *** jhenner has joined #openstack-qa | 11:06 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 11:06 | |
| *** e0ne has joined #openstack-qa | 11:07 | |
| openstackgerrit | A change was merged to openstack/tempest: Adds the clients and tests for CINDER QoS V1 & V2 APIs https://review.openstack.org/90306 | 11:08 |
| *** dims has joined #openstack-qa | 11:11 | |
| coolsvap | andreaf, reg https://review.openstack.org/116435, f you are wondering its -1, the patch has a -1 from Citrix XenServer CI, jenkins is fine | 11:15 |
| *** marun has joined #openstack-qa | 11:21 | |
| yfried__ | gmann: remember the network scale? | 11:25 |
| yfried__ | marun: could you please review https://review.openstack.org/#/c/102700/ ? | 11:44 |
| *** Madan has joined #openstack-qa | 11:45 | |
| *** mmedvede has joined #openstack-qa | 11:46 | |
| *** nosnos has quit IRC | 11:52 | |
| jordanP | yfried__, hi | 11:52 |
| *** nosnos has joined #openstack-qa | 11:52 | |
| jordanP | i am looking at https://review.openstack.org/#/c/102700/21/tempest/common/waiters.py | 11:52 |
| yfried__ | jordanP: yeah? | 11:53 |
| jordanP | i don"t understand the "# check the status again to avoid a false negative where we hit" part | 11:53 |
| jordanP | oh, I see it's already in wait_for_image_status() but it's strange | 11:54 |
| yfried__ | jordanP: this is copy pasted from wait_for_image_status in the same module. not greate practice. I know. | 11:54 |
| *** dmellado has quit IRC | 11:54 | |
| *** andreaf_ has joined #openstack-qa | 11:54 | |
| yfried__ | jordanP: I plan on merging them down the road. maybe I should add a TODO... | 11:54 |
| *** dmellado has joined #openstack-qa | 11:54 | |
| *** andreaf has quit IRC | 11:56 | |
| *** nosnos has quit IRC | 11:57 | |
| *** amotoki has quit IRC | 11:57 | |
| *** masayukig has quit IRC | 11:58 | |
| *** masayukig_ has joined #openstack-qa | 11:58 | |
| *** luqas has quit IRC | 12:01 | |
| *** rdekel has quit IRC | 12:01 | |
| *** jlanoux has quit IRC | 12:01 | |
| *** Albert has quit IRC | 12:01 | |
| *** torandu has quit IRC | 12:01 | |
| *** yamahata_ has quit IRC | 12:01 | |
| *** bdperkin has quit IRC | 12:01 | |
| *** rfolco has joined #openstack-qa | 12:02 | |
| openstackgerrit | Nir Magnezi proposed a change to openstack/tempest: Adds status check for router Port in scenarios https://review.openstack.org/117454 | 12:02 |
| *** Madan has quit IRC | 12:02 | |
| *** torandu has joined #openstack-qa | 12:03 | |
| yfried__ | sdague: mtreinish: https://review.openstack.org/112199 this needs "Approved", could you please take a look? | 12:04 |
| sdague | yfried__: I'll let mtreinish handle that, I think he had some ordering concerns before | 12:05 |
| yfried__ | gmann: still around? | 12:05 |
| *** rdekel has joined #openstack-qa | 12:05 | |
| *** jlanoux has joined #openstack-qa | 12:05 | |
| *** Albert has joined #openstack-qa | 12:05 | |
| *** yamahata_ has joined #openstack-qa | 12:05 | |
| *** bdperkin has joined #openstack-qa | 12:05 | |
| yfried__ | sdague: genaral concerns? or concerns with my patch? | 12:06 |
| marun | yfried__: the wait function could use some cleanup | 12:06 |
| yfried__ | marun: indeed. any pointers? | 12:07 |
| *** luqas has joined #openstack-qa | 12:07 | |
| marun | yfried__: It should be possible to avoid the second check with the comment block | 12:07 |
| marun | yfried__: The check for error status is also problematic - no check is being performed on the first fetch | 12:07 |
| marun | yfried__: Consider looping on the timer rather than the status. | 12:09 |
| yfried__ | marun: about error, we can assume that if it's in error in the first fetch, it would be this way on the following fetch... | 12:10 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: WIP - Adds network to large scale scenario https://review.openstack.org/120070 | 12:11 |
| *** dims has quit IRC | 12:11 | |
| *** dims has joined #openstack-qa | 12:12 | |
| *** dims_ has joined #openstack-qa | 12:12 | |
| yfried__ | sdague: Can I make sure Jenkins won't run for my WIP patch? | 12:12 |
| marun | yfried__: it seems pretty silly to have to duplicate that fetch | 12:13 |
| sdague | yfried__: no, it runs on WIP as well | 12:14 |
| yfried__ | marun: I agree. I just want to keep the waiter identical to the others as much as possible, so I can fix them all together, and do some code sharing | 12:14 |
| marun | yfried__: ? | 12:14 |
| *** dims_ has quit IRC | 12:14 | |
| marun | yfried__: duplicating terrible code doesn't seem like a good strategy | 12:14 |
| *** dims_ has joined #openstack-qa | 12:15 | |
| *** mpaolino has joined #openstack-qa | 12:15 | |
| yfried__ | marun: I agree. I'll fix that waiter and worry about the others later. would you care to post your comments on gerrit, so I have them later for when I can actually work on it? | 12:15 |
| *** dims has quit IRC | 12:16 | |
| *** dkranz has joined #openstack-qa | 12:16 | |
| marun | yfried__: sure | 12:16 |
| yfried__ | andreaf_: sdague: gmann: marun: I'm trying to get large_scale_test to do some neutron checks as well https://review.openstack.org/#/c/120070/ | 12:17 |
| *** mwagner_lap has quit IRC | 12:17 | |
| yfried__ | but a new patch is making it a regular scenario instead of a network scenario | 12:17 |
| *** yamahata has joined #openstack-qa | 12:18 | |
| yfried__ | which means I can't access the manager.networkscenario methods (create network, check connectivity, etc) | 12:18 |
| yfried__ | any suggestions? | 12:18 |
| *** Poornima has quit IRC | 12:21 | |
| *** weshay has joined #openstack-qa | 12:21 | |
| *** ildikov_ has quit IRC | 12:27 | |
| *** Guest37209 is now known as fungi | 12:32 | |
| *** weshay is now known as weshay_mtg | 12:35 | |
| *** adalbas has joined #openstack-qa | 12:36 | |
| *** mestery_ is now known as mestery | 12:37 | |
| openstackgerrit | Kirill Shileev proposed a change to openstack-dev/devstack: Does chown recursively https://review.openstack.org/120074 | 12:37 |
| *** aysyd has joined #openstack-qa | 12:39 | |
| andreaf_ | yfried__, long term my personal preference would be for the scenario tests structure to be flat - one base class for all - because inherently tests are always working across services | 12:41 |
| yfried__ | andreaf_: me too. I had a spec for scenario refactor. maybe we should revisit it? | 12:42 |
| yfried__ | andreaf_: but for now? | 12:42 |
| andreaf_ | yfried__, on the short term if the patch to change the base class did not land yet you can talk to the author | 12:42 |
| andreaf_ | yfried__, else you can change the base class back in your patch | 12:43 |
| yfried__ | andreaf_: we talked. gmann says there's a request to have this test run on novanetwork gate as well | 12:43 |
| yfried__ | andreaf_: seems like an unfriendly move :) | 12:43 |
| andreaf_ | yfried__, I see it makes sense - is any of the check you're doing neutron specific? | 12:44 |
| andreaf_ | yfried__, it doesn't seem so | 12:45 |
| andreaf_ | yfried__, you could have your extended large opts working with both neutron and nova network | 12:47 |
| *** marun has quit IRC | 12:50 | |
| yfried__ | andreaf_: how? | 12:52 |
| *** chandankumar has quit IRC | 12:56 | |
| openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Run processes without screen https://review.openstack.org/117339 | 13:00 |
| openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Replace screen_it() with run_process() throughout https://review.openstack.org/120077 | 13:00 |
| openstackgerrit | John Davidge proposed a change to openstack-dev/devstack: Add IPv6 support for tenant data network https://review.openstack.org/87987 | 13:00 |
| andreaf_ | yfried__, one way would be to use nova api for doing things - if neutron is installed nova requests will be proxied to neutron | 13:02 |
| andreaf_ | yfried__, you can control floating IPs and SG | 13:02 |
| cdent | sdague, jogo, dtroyer: https://review.openstack.org/#/c/120077/ | 13:02 |
| andreaf_ | yfried__, that should be enough to do connectivity check | 13:03 |
| cdent | that's ^ the run_process change discussed yesterday | 13:03 |
| * cdent lunches | 13:03 | |
| sdague | cdent: looking | 13:03 |
| cdent | it's a fairly extensive change | 13:04 |
| cdent | it if would be better borken up per service or some such, just shout | 13:04 |
| cdent | i'll be back shortly (with a sandwich) | 13:05 |
| sdague | yeh, I'm ok with bulk transition here, let me just get through all the changes | 13:07 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 13:08 | |
| *** dkranz has quit IRC | 13:11 | |
| sdague | cdent: ok, I think I found one issue in lib/swift because it does some of the tails as well for screen sessions | 13:14 |
| cdent | I thought there might be some of those lingering | 13:14 |
| sdague | I would suggest we just do another function for the tailing of log files to screen sessions | 13:14 |
| cdent | that make sense | 13:14 |
| sdague | so the conditional for USE_SCREEN can be buried in that function | 13:14 |
| * cdent nods | 13:14 | |
| sdague | otherwise, it looks good | 13:14 |
| cdent | cool, I'll make that change | 13:15 |
| sdague | so if you respin that after sandwich, should be good to land | 13:15 |
| cdent | ✔ | 13:15 |
| *** mestery has quit IRC | 13:17 | |
| *** mriedem has joined #openstack-qa | 13:17 | |
| cdent | any concer about the removal of the cd commands, sdague ? | 13:17 |
| *** mestery has joined #openstack-qa | 13:17 | |
| openstackgerrit | Kirill Shileev proposed a change to openstack-dev/devstack: DEST from local.conf gets redefined https://review.openstack.org/120085 | 13:19 |
| *** Longgeek has quit IRC | 13:19 | |
| sdague | cdent: I think they are mostly obsolete | 13:28 |
| sdague | I'll leave the final approve to dtroyer, but I think we don't need them any more, they were an artifact | 13:29 |
| cdent | excellent, thanks | 13:29 |
| rdekel | sdague jogo mtreinish: hi all, I wrote a patch https://review.openstack.org/#/c/117520/ which I would like you to review. With this patch we can resume the work on https://review.openstack.org/#/c/102633/ | 13:32 |
| *** dustins has joined #openstack-qa | 13:32 | |
| *** mestery has quit IRC | 13:34 | |
| *** mestery has joined #openstack-qa | 13:34 | |
| *** dmellado has quit IRC | 13:35 | |
| *** sabeen has joined #openstack-qa | 13:36 | |
| *** markmcclain has joined #openstack-qa | 13:38 | |
| *** jaypipes has joined #openstack-qa | 13:40 | |
| *** ChanServ sets mode: +o jaypipes | 13:40 | |
| *** Longgeek has joined #openstack-qa | 13:41 | |
| *** mwagner_lap has joined #openstack-qa | 13:42 | |
| *** nsaje_ has joined #openstack-qa | 13:43 | |
| *** mkollaro has quit IRC | 13:43 | |
| yfried__ | andreaf_: problem is that all connectivity testing method are in NetworkScenario | 13:45 |
| *** cbader has joined #openstack-qa | 13:48 | |
| *** mkollaro has joined #openstack-qa | 13:50 | |
| *** amotoki has joined #openstack-qa | 13:59 | |
| *** k4n0_ has quit IRC | 14:02 | |
| *** neeti has quit IRC | 14:04 | |
| *** marun has joined #openstack-qa | 14:04 | |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: Add docs environment to tox.ini https://review.openstack.org/119907 | 14:07 |
| yfried__ | andreaf_: solved it pretty nicely: https://review.openstack.org/120070 | 14:07 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: WIP - Adds network to large scale scenario https://review.openstack.org/120070 | 14:07 |
| dtroyer | cdent, sdague: here's where using cnder as the example fell down... | 14:09 |
| dtroyer | I broke the new functions up so we could run the log tails with screen_service() and have them either "harmlessly" stay in screen or skip them altogether without screen as I don't really see the point, if we want the logs then we should just capture the logs. | 14:09 |
| *** rbak has joined #openstack-qa | 14:09 | |
| dtroyer | looking back now,, its set up to always leave them in screen, if we want to skip them the USE_SCREEN test needs to be added... | 14:10 |
| sdague | dtroyer: yeh, cdent's change basically does that | 14:11 |
| sdague | with one fix needed in swift that he's going to address | 14:11 |
| *** pcrews has joined #openstack-qa | 14:11 | |
| dtroyer | Removing the cd commands is actually the reason for these changes…they dorced the use of a shell to interpret the commands; one service needed it, cargo cult ensued | 14:11 |
| dtroyer | I'm still working through the changes…he kept the tail in keystone too | 14:12 |
| cdent | I think I've got that now | 14:13 |
| dtroyer | cdent: ok, cool | 14:13 |
| cdent | i've made a tail_log function that does nothing if USE_SCREEN is not True | 14:13 |
| cdent | new patchset will be up in a few minutes | 14:14 |
| *** annegentle_ has joined #openstack-qa | 14:14 | |
| *** annegentle has quit IRC | 14:14 | |
| openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Replace screen_it() with run_process() throughout https://review.openstack.org/120077 | 14:16 |
| openstackgerrit | Roey Dekel proposed a change to openstack/tempest: Enhanced connectivity check using multiple protocols https://review.openstack.org/117520 | 14:19 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: WIP - Adds network to large scale scenario https://review.openstack.org/120070 | 14:19 |
| *** e0ne has quit IRC | 14:19 | |
| *** bkopilov has quit IRC | 14:19 | |
| sdague | cdent: so it looks like trove doesn't work right this way | 14:20 |
| cdent | feh | 14:20 |
| sdague | based on - http://logs.openstack.org/77/120077/1/check/check-tempest-dsvm-full/0d36152/console.html | 14:20 |
| cdent | there's always at least one | 14:20 |
| sdague | yeh | 14:20 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: WIP - Adds network to large scale scenario https://review.openstack.org/120070 | 14:20 |
| sdague | so... maybe just leave trove unconverted, file a critical bug against them, and get through the rest of the changes | 14:20 |
| sdague | we don't do trove in grenade anyway, so it will at least let us make progress there | 14:21 |
| * cdent nods | 14:21 | |
| dtroyer | we had problems getting trove to work to begin with, I forgot exactly why, but they do something different… | 14:21 |
| cdent | I'll take a quick look round for anything obvious | 14:21 |
| cdent | and then move on if no progress | 14:21 |
| *** jecarey has joined #openstack-qa | 14:21 | |
| sdague | on the up side, all the other services seemed to work fine | 14:22 |
| *** nmagnezi has quit IRC | 14:22 | |
| cdent | amaze | 14:22 |
| *** zz_dimtruck is now known as dimtruck | 14:24 | |
| mriedem | network-feature-enabled all_extensions ftw! | 14:25 |
| mriedem | *api_extensions | 14:25 |
| *** yfried__ has quit IRC | 14:25 | |
| mriedem | thanks to whoever put thta in | 14:26 |
| *** toure has joined #openstack-qa | 14:28 | |
| mtreinish | mriedem: I did, almost a year ago I think :) | 14:28 |
| mriedem | mtreinish: saving my ass with filtering out neutron lbaas testing internally L( | 14:28 |
| mriedem | :) | 14:28 |
| mriedem | was getting super fun hangs on fc19 ppc64 systems, stepped back and thought 'why am i even testing this?', and then started using the extension filter | 14:29 |
| mriedem | b/c playing whack a mole with excluding tests is a pita | 14:29 |
| mtreinish | mriedem: yeah, we eventually also need to get the pieces in place where devstack can generate that extension list based on the extensions being used | 14:32 |
| mriedem | yeah that would be nice since i had to do some awk playing around this morning | 14:32 |
| mtreinish | mriedem: oh you could have used http://git.openstack.org/cgit/openstack/tempest/tree/tempest/cmd/verify_tempest_config.py | 14:33 |
| mtreinish | that will generate a list | 14:33 |
| *** sergeysh has quit IRC | 14:33 | |
| *** SridharG has quit IRC | 14:34 | |
| *** e0ne has joined #openstack-qa | 14:34 | |
| mtreinish | mriedem: salv-orlando started working on getting the devstack bits because we actually really need it to add tests for new extensions | 14:34 |
| *** atiwari has joined #openstack-qa | 14:34 | |
| *** bkopilov has joined #openstack-qa | 14:35 | |
| *** dkranz has joined #openstack-qa | 14:38 | |
| mriedem | mtreinish: cool, will have to check that out at some point. we just clone tempest and configure it right now, we don't install it to get the commands but that could easily be changed | 14:39 |
| mriedem | or use the new lib stuff | 14:39 |
| mriedem | i'm actually just moonlighting this week since it's a holiday in china | 14:39 |
| mriedem | but hey i did fix a bug in swift last night, ATC! | 14:39 |
| *** evgenyf has quit IRC | 14:40 | |
| mtreinish | mriedem: I think the py file is executable so you don't actually have to install it, you just have to call it from the tempest root dir.(otherwise it can't find the proper namespace) | 14:42 |
| mtreinish | mriedem: yeah the lib migration will make it so the tool can work by itself, but I think we'll still package it with tempest | 14:43 |
| mtreinish | since it's tightly coupled to the config file | 14:43 |
| mtreinish | cool, now you can vote if there is one:) | 14:43 |
| mriedem | maybe i should run for PTL | 14:44 |
| mriedem | "it's time to clean house!" | 14:44 |
| mtreinish | go for it! :) | 14:47 |
| *** mitz has joined #openstack-qa | 14:49 | |
| *** bkopilov has quit IRC | 14:51 | |
| *** SridharG has joined #openstack-qa | 14:54 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Allow setting the number of workers to be used. https://review.openstack.org/117517 | 14:54 |
| *** keinkurt has joined #openstack-qa | 14:55 | |
| mtreinish | sdague, dtroyer: on https://review.openstack.org/#/c/116121/ I'm thinking it would be better to just add the enabled api extension list detection to devstack rather than doing that | 14:59 |
| mtreinish | any thoughts | 14:59 |
| sdague | mtreinish: sure, I was starting to think about it the other day | 15:01 |
| *** mtreinish changes topic to "All Things QA, Tempest Bug Day: Tues. Sept. 9th" | 15:01 | |
| sdague | mtreinish: I agree just listing extensions in devstack on stable/ is probably the sane thing to do | 15:01 |
| *** bkopilov has joined #openstack-qa | 15:02 | |
| *** zz_jgrimm is now known as jgrimm | 15:03 | |
| *** markmcclain has quit IRC | 15:05 | |
| openstackgerrit | A change was merged to openstack/tempest: Allow new quota types https://review.openstack.org/112474 | 15:06 |
| *** keinkurt has quit IRC | 15:08 | |
| *** bkopilov has quit IRC | 15:09 | |
| mtreinish | jgrimm: just a heads up on: https://bugs.launchpad.net/tempest/+bug/1365136 | 15:10 |
| mtreinish | I assigned that to you since your WIP patch will fix it | 15:11 |
| *** afazekas_ has quit IRC | 15:11 | |
| jgrimm | mtreinish, thanks sir | 15:11 |
| *** mlavalle has joined #openstack-qa | 15:11 | |
| *** mlavalle has quit IRC | 15:12 | |
| *** mlavalle has joined #openstack-qa | 15:12 | |
| *** chandankumar has joined #openstack-qa | 15:12 | |
| *** qba73 has quit IRC | 15:17 | |
| *** e0ne has quit IRC | 15:22 | |
| *** Longgeek has quit IRC | 15:23 | |
| *** qba73 has joined #openstack-qa | 15:24 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack-infra/devstack-gate: Add option for building Ironic deploy ramdisks https://review.openstack.org/119473 | 15:24 |
| *** e0ne has joined #openstack-qa | 15:25 | |
| sdague | lifeless: for whenever you end up awake - someone seriously needs to get launchpad to increase the timeouts, because I'm having to do every launchpad state change 3+ times for it to stick | 15:30 |
| dkranz | mtreinish: I'm looking at https://bugs.launchpad.net/tempest/+bug/1276778 | 15:30 |
| *** jordanP has quit IRC | 15:31 | |
| dkranz | mtreinish: A google search for 'failed to reach ACTIVE status and task state "None"' shows at least 6 bugs that are probably similar | 15:31 |
| dkranz | mtreinish: It is unlikely to be a tempest bug but how do we get more attention to this? | 15:32 |
| *** mkollaro has quit IRC | 15:34 | |
| *** mkollaro has joined #openstack-qa | 15:35 | |
| mtreinish | dkranz: hmm, I think something like that is constantly a top bug on e-r | 15:35 |
| dkranz | mtreinish: right. I closed the tempest bug but somehow feel like I should be doing something more, but not sure what. | 15:36 |
| mtreinish | yeah, I'm not sure what either. It may be that how we're logging the failures is tripping people up | 15:37 |
| *** dimtruck is now known as zz_dimtruck | 15:38 | |
| *** asselin has joined #openstack-qa | 15:38 | |
| mtreinish | jogo, mriedem: ^^^ you probably see this a lot too, any ideas? | 15:38 |
| *** dhellmann is now known as dhellmann_ | 15:38 | |
| *** carl_baldwin has joined #openstack-qa | 15:39 | |
| mriedem | we probably just need better trace logging throug the operation in nova | 15:40 |
| mriedem | but i agree these generic timeout things aren't probably tempest bugs | 15:40 |
| openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Replace screen_it() with run_process() throughout https://review.openstack.org/120077 | 15:41 |
| openstackgerrit | Thomas Bechtold proposed a change to openstack-dev/devstack: Remove python-mysql.connector for SUSE distros https://review.openstack.org/120126 | 15:41 |
| *** qba73 has quit IRC | 15:43 | |
| *** jlibosva has quit IRC | 15:44 | |
| *** jlibosva has joined #openstack-qa | 15:44 | |
| *** qba73 has joined #openstack-qa | 15:44 | |
| carl_baldwin | Hi, over in Neutron we’re close to converging on clean tempest runs in the dvr experiemental job. We’ve got two persistent errors left which are addressed by https://review.openstack.org/#/c/116377/, a patch to openstack/tempest. | 15:45 |
| *** Longgeek has joined #openstack-qa | 15:45 | |
| carl_baldwin | Is there anyone who could have a look at this patch for us? | 15:45 |
| mtreinish | jogo: do you have a bug number for when nova-compute stops during a tempest run? | 15:46 |
| *** Longgeek has quit IRC | 15:46 | |
| *** chandankumar has quit IRC | 15:48 | |
| *** zz_dimtruck is now known as dimtruck | 15:48 | |
| *** cdent has quit IRC | 15:48 | |
| *** e0ne has quit IRC | 15:49 | |
| *** cdent has joined #openstack-qa | 15:49 | |
| mtreinish | carl_baldwin: +2 | 15:49 |
| carl_baldwin | mtreinish: Many thanks for taking a look. | 15:49 |
| *** cdent has quit IRC | 15:50 | |
| *** mpaolino has quit IRC | 15:50 | |
| *** cdent has joined #openstack-qa | 15:51 | |
| *** qba73 has quit IRC | 15:52 | |
| *** ZZelle has joined #openstack-qa | 15:53 | |
| ZZelle | mtreinish, hi | 15:53 |
| *** gszasz has quit IRC | 15:56 | |
| *** Longgeek has joined #openstack-qa | 15:57 | |
| *** carl_baldwin has quit IRC | 15:58 | |
| *** qba73 has joined #openstack-qa | 15:59 | |
| *** qba73 has quit IRC | 16:00 | |
| *** jlanoux has quit IRC | 16:03 | |
| *** rdekel has quit IRC | 16:04 | |
| *** pkoniszewski has quit IRC | 16:05 | |
| *** cdent has quit IRC | 16:05 | |
| *** qba73 has joined #openstack-qa | 16:06 | |
| *** nsaje_ has quit IRC | 16:07 | |
| openstackgerrit | John Davidge proposed a change to openstack-dev/devstack: Add IPv6 support for tenant data network https://review.openstack.org/87987 | 16:08 |
| *** chandankumar has joined #openstack-qa | 16:08 | |
| *** yfried__ has joined #openstack-qa | 16:13 | |
| *** yjiang5 has joined #openstack-qa | 16:13 | |
| *** Longgeek has quit IRC | 16:14 | |
| *** Longgeek has joined #openstack-qa | 16:14 | |
| openstackgerrit | Thomas Bechtold proposed a change to openstack-dev/devstack: Enable services before trying to start them https://review.openstack.org/120141 | 16:14 |
| *** Longgeek has quit IRC | 16:14 | |
| *** masayukig_ has quit IRC | 16:18 | |
| *** mpaolino has joined #openstack-qa | 16:18 | |
| *** masayukig has joined #openstack-qa | 16:24 | |
| afazekas | Santosh: Can you elaborate ti ? BTW: some dnsmasq with older version number what neutron/ nova requires might the the necessary fixes. | 16:25 |
| mtreinish | ZZelle: hi | 16:29 |
| *** mpaolino has quit IRC | 16:29 | |
| *** mpaolino has joined #openstack-qa | 16:30 | |
| *** markmcclain has joined #openstack-qa | 16:30 | |
| *** markmcclain has quit IRC | 16:32 | |
| *** qba73 has quit IRC | 16:32 | |
| *** markmcclain has joined #openstack-qa | 16:33 | |
| *** mpavlase has quit IRC | 16:36 | |
| *** bkopilov has joined #openstack-qa | 16:36 | |
| openstackgerrit | Thomas Bechtold proposed a change to openstack-dev/devstack: Enable services before trying to start them https://review.openstack.org/120141 | 16:37 |
| *** aimon has quit IRC | 16:38 | |
| jogo | mtreinish: not sure what number it was | 16:42 |
| jogo | but there was an oslo sync patch that claimed to help it | 16:42 |
| mtreinish | jogo: ok I think it was an older bug, I just wanted to mark it as a dup | 16:43 |
| *** mpaolino has quit IRC | 16:43 | |
| *** suneelb has joined #openstack-qa | 16:44 | |
| jogo | mtreinish: https://bugs.launchpad.net/nova/+bug/1349452 | 16:44 |
| *** Albert has quit IRC | 16:44 | |
| jogo | mtreinish: https://review.openstack.org/#/c/119586/ | 16:44 |
| mtreinish | jogo: ok cool, thanks | 16:45 |
| *** arnaud__ has joined #openstack-qa | 16:49 | |
| *** wuhg has quit IRC | 16:53 | |
| *** andreaf has joined #openstack-qa | 16:53 | |
| *** mestery has quit IRC | 16:59 | |
| *** mestery has joined #openstack-qa | 17:00 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 17:05 | |
| *** luqas has quit IRC | 17:07 | |
| *** Longgeek has joined #openstack-qa | 17:08 | |
| *** aimon has joined #openstack-qa | 17:09 | |
| *** davidlenwell_ is now known as davidlenwell | 17:10 | |
| *** coolsvap has quit IRC | 17:10 | |
| *** cmyster_ has joined #openstack-qa | 17:12 | |
| *** dhellmann_ is now known as dhellmann | 17:13 | |
| yfried__ | mtreinish: https://review.openstack.org/#/c/112199/ needs +A. could you please take a look? | 17:17 |
| *** coolsvap has joined #openstack-qa | 17:18 | |
| *** eglynn-office is now known as eglynn-officeafk | 17:19 | |
| *** harlowja_away is now known as harlowja | 17:25 | |
| *** SridharG has quit IRC | 17:28 | |
| openstackgerrit | Rob Crittenden proposed a change to openstack-dev/devstack: Configure endpoints to use SSL natively or via proxy https://review.openstack.org/98854 | 17:29 |
| *** amotoki has quit IRC | 17:29 | |
| *** cmyster_ has quit IRC | 17:30 | |
| *** amotoki has joined #openstack-qa | 17:30 | |
| openstackgerrit | Mikhail S Medvedev proposed a change to openstack/tempest: Skip orchestration tests if Neutron is not available https://review.openstack.org/118454 | 17:30 |
| *** amotoki has quit IRC | 17:34 | |
| *** e0ne has joined #openstack-qa | 17:35 | |
| *** SridharG has joined #openstack-qa | 17:44 | |
| *** e0ne has quit IRC | 17:46 | |
| *** jlibosva has quit IRC | 17:49 | |
| *** yfried__ has quit IRC | 17:49 | |
| *** mkollaro has quit IRC | 17:49 | |
| *** ZZelle_ has joined #openstack-qa | 18:01 | |
| *** SridharG has quit IRC | 18:10 | |
| *** dims has joined #openstack-qa | 18:11 | |
| *** dims_ has quit IRC | 18:11 | |
| *** mwagner_lap has quit IRC | 18:11 | |
| *** dims has quit IRC | 18:12 | |
| *** marun has quit IRC | 18:12 | |
| *** dims_ has joined #openstack-qa | 18:16 | |
| *** e0ne has joined #openstack-qa | 18:20 | |
| *** pcrews has quit IRC | 18:24 | |
| *** pcrews has joined #openstack-qa | 18:25 | |
| mtreinish | yfried: sure I'll put it on my list for reviews today | 18:28 |
| dkranz | Any one else doing bug day activity other than mtreinish and myself? | 18:31 |
| *** mestery has quit IRC | 18:32 | |
| *** suneelb has quit IRC | 18:32 | |
| *** mestery has joined #openstack-qa | 18:32 | |
| *** suneelb has joined #openstack-qa | 18:33 | |
| jogo | sdague: https://review.openstack.org/#/c/117339/3 was rebased and your +W was dropped even though your +2 is still there | 18:34 |
| *** kmartin has joined #openstack-qa | 18:35 | |
| *** chandankumar has quit IRC | 18:35 | |
| sdague | jogo: thanks | 18:40 |
| *** Longgeek has quit IRC | 18:40 | |
| jogo | sdague: for Add Ironic sideways migration testing | 18:42 |
| jogo | sdague: the dependant POC patch didn't pass | 18:42 |
| jogo | sdague: so I assume there are some things that need to be worked out, but I take it https://review.openstack.org/#/c/111859/25 is a very good first pass | 18:42 |
| *** Longgeek has joined #openstack-qa | 18:43 | |
| mtreinish | dkranz: I saw triage comments from jordanP this morning | 18:44 |
| jogo | adam_g: ping | 18:44 |
| dkranz | mtreinish: ok. No as much activity as I hoped for :( | 18:45 |
| *** catohornet has joined #openstack-qa | 18:47 | |
| dkranz | mtreinish: Do you know what the default devstack token expiration time is? | 18:47 |
| mtreinish | dkranz: not off the top of my head, I'd have to dig it up | 18:49 |
| mtreinish | dkranz: I suspect it's the keystone default though | 18:49 |
| dkranz | mtreinish: ok. I hope it is longer than tempest could take to run | 18:49 |
| *** e0ne has quit IRC | 18:50 | |
| mtreinish | dkranz: it's not, but we have code in auth.py to refresh a token slightly before, at, or after it's expiration before we use it | 18:50 |
| dkranz | mtreinish: ok, so a tempest test cannot fail due to an unexpectedly expired token | 18:51 |
| dkranz | mtreinish: that answers my question | 18:51 |
| mtreinish | dkranz: nope, it shouldn't. See: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/auth.py#n340 and http://git.openstack.org/cgit/openstack/tempest/tree/tempest/auth.py#n464 | 18:52 |
| *** yjiang5 has quit IRC | 18:55 | |
| *** suneelb has quit IRC | 18:57 | |
| *** suneelb has joined #openstack-qa | 18:57 | |
| lifeless | sdague: I'll ping the devs but doubt the situation has changed | 19:00 |
| adam_g | jogo, yo | 19:00 |
| jogo | adam_g: question on https://review.openstack.org/#/c/111859/25 | 19:01 |
| adam_g | jogo, commenting now | 19:01 |
| *** rockyg has joined #openstack-qa | 19:02 | |
| *** openstackgerrit has quit IRC | 19:02 | |
| *** sabeen has quit IRC | 19:03 | |
| *** sabeen has joined #openstack-qa | 19:04 | |
| *** aix has quit IRC | 19:06 | |
| *** mpavlase has joined #openstack-qa | 19:08 | |
| adam_g | dtroyer, https://review.openstack.org/#/c/107814/ any chance of that +A here if you get a sec? | 19:17 |
| *** mlavalle has quit IRC | 19:17 | |
| *** mlavalle has joined #openstack-qa | 19:18 | |
| dkranz | mtreinish: Do you have an opinion about what to do with bugs that may be real, but we know no one is ever going to look at the ticket and try to debug it. Like https://bugs.launchpad.net/tempest/+bug/1358814 | 19:18 |
| mtreinish | dkranz: heh, well that's probably a nova bug... | 19:20 |
| mtreinish | but, just mark it low priority | 19:20 |
| mtreinish | or wishlist | 19:20 |
| mtreinish | but if it's a real bug I'd use low | 19:20 |
| dkranz | mtreinish: ok :) | 19:21 |
| dkranz | mtreinish: I am not allowed to set the priority of a nova ticket | 19:23 |
| dkranz | mtreinish: But I added nova to that ticket anyway | 19:23 |
| *** dimtruck is now known as zz_dimtruck | 19:23 | |
| dtroyer | adam_g: done, apologies for missing it over the weekend... | 19:24 |
| mtreinish | adam_g: is this really a bug in tempest?: https://bugs.launchpad.net/tempest/+bug/1333852 | 19:24 |
| mtreinish | because I don't see how tempest could do anything differently (besides skipping the trove test) | 19:24 |
| adam_g | mtreinish, ya, sorry. marked i nvalid | 19:27 |
| *** amotoki has joined #openstack-qa | 19:30 | |
| *** zz_dimtruck is now known as dimtruck | 19:33 | |
| *** amotoki has quit IRC | 19:35 | |
| *** openstackgerrit has joined #openstack-qa | 19:41 | |
| *** cbader has quit IRC | 19:45 | |
| *** cbader has joined #openstack-qa | 19:47 | |
| *** cbader has quit IRC | 19:47 | |
| *** cbader has joined #openstack-qa | 19:48 | |
| jogo | mtreinish: whats up with icehouse https://review.openstack.org/#/c/119081/ | 19:51 |
| jogo | anyone else seeing it fail? | 19:51 |
| *** e0ne has joined #openstack-qa | 19:52 | |
| mtreinish | jogo: one was a network issue: http://logs.openstack.org/81/119081/4/check/check-tempest-dsvm-full-icehouse/9c92330/logs/devstacklog.txt.gz#_2014-09-09_04_30_17_406 | 19:52 |
| *** cbader_ has joined #openstack-qa | 19:53 | |
| mtreinish | the postgres job failure is something on heat, I don't recognize it: | 19:53 |
| mtreinish | http://logs.openstack.org/81/119081/4/check/check-tempest-dsvm-postgres-full-icehouse/4b56c46/logs/screen-h-api.txt.gz#_2014-09-09_04_52_27_926 | 19:53 |
| jogo | mtreinish: so we have a bunch of icehouse bugs? | 19:54 |
| mtreinish | err, I guess cinder | 19:54 |
| jogo | not just one | 19:54 |
| *** cbader has quit IRC | 19:54 | |
| mtreinish | http://logs.openstack.org/81/119081/4/check/check-tempest-dsvm-postgres-full-icehouse/4b56c46/logs/screen-h-eng.txt.gz?level=TRACE#_2014-09-09_04_52_27_917 | 19:54 |
| mtreinish | jogo: yeah, probably, although the pip timeout affects everyone | 19:57 |
| jogo | mtreinish: heh | 19:57 |
| mtreinish | and those neutron grenade icehouse jobs, I'm not sure they've ever worked | 19:58 |
| mtreinish | which is why it's non-voting | 19:58 |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack-dev/devstack: Increase CEPH_LOOPBACK_DISK_SIZE_DEFAULT to 4GB https://review.openstack.org/116290 | 19:59 |
| mtreinish | jogo: if we've got a ~12% failure rate and we run 26 jobs on tempest it's lucky we get anything through... | 20:00 |
| jogo | mtreinish: yeah wow | 20:00 |
| mtreinish | well, I guess it's really not 26 voting tempest runs | 20:01 |
| *** gfidente has quit IRC | 20:04 | |
| *** jhenner has quit IRC | 20:07 | |
| *** jhenner has joined #openstack-qa | 20:08 | |
| *** masayukig has quit IRC | 20:09 | |
| *** masayukig has joined #openstack-qa | 20:15 | |
| *** dims_ has quit IRC | 20:18 | |
| *** dims_ has joined #openstack-qa | 20:19 | |
| *** sabeen1 has joined #openstack-qa | 20:20 | |
| *** sabeen has quit IRC | 20:21 | |
| *** dims_ has quit IRC | 20:23 | |
| *** weshay_mtg has quit IRC | 20:23 | |
| *** toure has quit IRC | 20:23 | |
| *** ajo_ has joined #openstack-qa | 20:24 | |
| *** dims_ has joined #openstack-qa | 20:35 | |
| jogo | mtreinish: so https://bugs.launchpad.net/nova/+bug/1353962 | 20:35 |
| jogo | mtreinish: there may be a tempest server leak | 20:35 |
| jogo | or a nova issue | 20:35 |
| jogo | mtreinish: thoughts on adding something to run at the end of tempest to check for leaked servers | 20:36 |
| sdague | lifeless: well the current state of launchpad is that you can no longer triage nova bugs effectively, as you have a 1 in 3 chance that a state change will *not* timeout (they time out more than they don't) | 20:36 |
| jogo | dkranz: ^ | 20:37 |
| sdague | and that's not even cross linked bugs | 20:37 |
| *** e0ne has quit IRC | 20:37 | |
| jogo | sdague: I am having slightly better luck with timeouts | 20:37 |
| jogo | more like 1 in 5 | 20:37 |
| sdague | I have had many instances today of needing 5 tries to set a bug to invalid | 20:37 |
| jogo | sdague: wow lifeless really doesn't want you to invalidate bugs :) | 20:38 |
| dkranz | jogo: https://review.openstack.org/#/c/112581/ | 20:38 |
| dkranz | jogo: That, in the --dry-run mode, can be added to the way we run tempest | 20:39 |
| dkranz | jogo: The author told me he is ready to push another revision that I hope can be approved. | 20:39 |
| jogo | dkranz: awesome | 20:39 |
| *** yjiang5 has joined #openstack-qa | 20:41 | |
| *** ajo_ has quit IRC | 20:42 | |
| jogo | mtreinish: https://bugs.launchpad.net/nova/+bug/1204930 | 20:44 |
| jogo | mtreinish: any inisight into that bug? | 20:44 |
| lifeless | sdague: 66% timeouts? | 20:45 |
| sdague | lifeless: yes | 20:45 |
| lifeless | sdague: wgrant will be up in ~2 hours and he's the person we want to speak with | 20:45 |
| mtreinish | jogo: hmm, that's interesting | 20:47 |
| jogo | sdague: your bug https://bugs.launchpad.net/nova/+bug/1222852 can use another round of triaging | 20:47 |
| *** andreaf has quit IRC | 20:48 | |
| sdague | jogo: I think that was all done | 20:49 |
| sdague | mark it as fix released or invalid | 20:49 |
| jogo | sdague: done, thanks | 20:50 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove fingerprint for bug 1291605 https://review.openstack.org/120220 | 20:52 |
| *** mestery has quit IRC | 20:57 | |
| *** mestery has joined #openstack-qa | 20:57 | |
| *** carl_baldwin has joined #openstack-qa | 20:58 | |
| *** arnaud__ has quit IRC | 20:59 | |
| *** suneelb has quit IRC | 21:00 | |
| *** suneelb has joined #openstack-qa | 21:00 | |
| carl_baldwin | Hi, mtreinish has been kind enough to look at this patch which should allow us to have clean runs in our Neutron dvr experimental job. Is there another tempest core around who could also take a look? https://review.openstack.org/#/c/116377 | 21:01 |
| *** dustins has quit IRC | 21:02 | |
| mtreinish | jogo: yeah the coverage runs seem to be blocking something on the nova unit tests. It might have to do with how the modules are loaded | 21:06 |
| mtreinish | I ran it locally and had 1199 failures with coverage on but everything passed without coverage enabled | 21:07 |
| *** weshay has joined #openstack-qa | 21:07 | |
| *** mpavlase has quit IRC | 21:08 | |
| mtreinish | oh, it might be a bug in run_tests.sh let me take a look | 21:09 |
| *** carl_baldwin has quit IRC | 21:10 | |
| *** aysyd has quit IRC | 21:16 | |
| *** dims__ has joined #openstack-qa | 21:19 | |
| *** dims__ has quit IRC | 21:20 | |
| *** dims__ has joined #openstack-qa | 21:21 | |
| *** dims_ has quit IRC | 21:23 | |
| *** mwagner_lap has joined #openstack-qa | 21:25 | |
| *** dims__ has quit IRC | 21:25 | |
| *** e0ne has joined #openstack-qa | 21:27 | |
| *** mmedvede has quit IRC | 21:27 | |
| *** cbader_ has quit IRC | 21:28 | |
| *** e0ne has quit IRC | 21:30 | |
| *** amotoki has joined #openstack-qa | 21:31 | |
| openstackgerrit | Thang Pham proposed a change to openstack-dev/devstack: Add VMware vmdk cinder backend https://review.openstack.org/120243 | 21:31 |
| openstackgerrit | Emily Hugenbruch proposed a change to openstack/tempest: Remove extra tenant_isolation parameter from config file. https://review.openstack.org/120245 | 21:33 |
| *** amotoki has quit IRC | 21:36 | |
| openstackgerrit | Winnie Tsang proposed a change to openstack/tempest: Add stack-update arguments for patching/reset parameters https://review.openstack.org/120247 | 21:37 |
| *** e0ne has joined #openstack-qa | 21:38 | |
| *** mriedem has quit IRC | 21:38 | |
| mtreinish | jogo: heh, I think the issue might be related to the integrated tests spawning new processes | 21:38 |
| *** dims_ has joined #openstack-qa | 21:39 | |
| *** e0ne has quit IRC | 21:40 | |
| lifeless | mtreinish: whats up? | 21:42 |
| *** dims_ has quit IRC | 21:45 | |
| mtreinish | lifeless: hey, are you referring to my last comment? | 21:45 |
| *** dims_ has joined #openstack-qa | 21:45 | |
| lifeless | yah | 21:45 |
| *** aix has joined #openstack-qa | 21:46 | |
| *** masayukig has quit IRC | 21:46 | |
| mtreinish | it's a coverage thing. Running the nova unit tests w/ coverage fails all the integrated tests (and maybe some others) | 21:47 |
| mtreinish | I'm thinking it's because nova is launching additional processes there for the api server | 21:47 |
| lifeless | do any of the unit tests test the built in coverage thing? | 21:47 |
| lifeless | cause I'm fairly sure coverage isn't reentrant | 21:47 |
| lifeless | or maybe I'm remembering old stuff | 21:48 |
| mtreinish | that got ripped out in icehouse because it didn't work well with eventlet | 21:48 |
| lifeless | ahright | 21:48 |
| clarkb | mtreinish: lifeless: double check on that | 21:48 |
| clarkb | there was a blakclist for some tests for a long time | 21:49 |
| mtreinish | clarkb: https://review.openstack.org/120241 | 21:49 |
| *** suneelb has quit IRC | 21:49 | |
| *** suneelb has joined #openstack-qa | 21:50 | |
| mtreinish | also the bug was against the run_tests.sh script, which was missing that, but it didn't actually make a difference because there are no coverage unit tests anymore | 21:51 |
| *** masayukig has joined #openstack-qa | 21:51 | |
| clarkb | mtreinish: that change isn't quite right, will comment on change | 21:52 |
| mtreinish | clarkb: :) | 21:52 |
| clarkb | done | 21:52 |
| mtreinish | oops missing the {} | 21:52 |
| mtreinish | I got overzealous in my deletes | 21:53 |
| mtreinish | clarkb: heh, that wouldn't have run any tests, because I'm pretty sure posargs doesn't match any test_ids | 21:54 |
| clarkb | mtreinish: :) | 21:54 |
| clarkb | that would have fixed it right? | 21:54 |
| mtreinish | heh, yeah I guess it would have | 21:55 |
| *** rockyg has quit IRC | 21:56 | |
| clarkb | perfect | 21:57 |
| *** flaper87 is now known as flaper87|afk | 22:03 | |
| *** dkranz has quit IRC | 22:04 | |
| *** jgrimm is now known as zz_jgrimm | 22:23 | |
| *** atiwari has quit IRC | 22:23 | |
| clarkb | mtreinish: lifeless: is there a good way to check dict equivalency without worrying about order? assertEqual and PYTHONHASHSEED and different python versions keeps biting us | 22:32 |
| *** jpich has joined #openstack-qa | 22:32 | |
| *** jecarey has quit IRC | 22:34 | |
| *** dimtruck is now known as zz_dimtruck | 22:37 | |
| *** dims_ has quit IRC | 22:42 | |
| lifeless | clarkb: assertEqual isn't order connected? | 22:42 |
| clarkb | it is, so tests break all over when the order of dict keys changes | 22:43 |
| *** dims_ has joined #openstack-qa | 22:43 | |
| jogo | random dstat question: | 22:44 |
| jogo | http://logs.openstack.org/99/115799/7/gate/gate-tempest-dsvm-neutron-pg/6394ec4/logs/screen-dstat.txt.gz | 22:44 |
| jogo | at 28-08 08:48:10 | 22:44 |
| jogo | all the CPU is in wai | 22:44 |
| jogo | does that mean things are waiting for I/O? | 22:44 |
| jogo | mtreinish: ^ | 22:44 |
| *** dims_ has quit IRC | 22:47 | |
| *** dims_ has joined #openstack-qa | 22:49 | |
| *** rbak has quit IRC | 22:56 | |
| *** dims_ has quit IRC | 23:00 | |
| *** dims_ has joined #openstack-qa | 23:00 | |
| *** dims_ has quit IRC | 23:05 | |
| *** amotoki has joined #openstack-qa | 23:32 | |
| *** amotoki has quit IRC | 23:37 | |
| *** ZZelle_ has quit IRC | 23:39 | |
| *** mlavalle has quit IRC | 23:47 | |
| gmann | andreaf_: yfried: changing base class for large ops tests was it should be going to nova for novanetwork or proxy to neutron. | 23:47 |
| gmann | As we had changes the _create_security_group function name to same in both ScenarioTest & NeutronScenarioTest with diff param, it would not allow large ops test to use nova _create_security_group | 23:50 |
| *** gmann_ has joined #openstack-qa | 23:53 | |
| lifeless | clarkb: sorry, was OTP | 23:57 |
| lifeless | clarkb: assertEqual is literally a==b in the code. | 23:57 |
| lifeless | clarkb: it cannot be order sensitive in regular dicts. | 23:57 |
| lifeless | clarkb: got a reference example where it breaks for me to see? | 23:58 |
| clarkb | lifeless: right I am not complaining that it is. Just asking if there is an easy way to check equivalency | 23:58 |
| clarkb | lifeless: run python-glanceclient `tox -epy34` | 23:58 |
| lifeless | clarkb: yes, assertEqual | 23:58 |
| clarkb | no that checks equality | 23:58 |
| lifeless | for dicts that is equivalent | 23:59 |
| *** oomichi has joined #openstack-qa | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!