Monday, 2015-03-30

*** mahito has joined #openstack-qa00:00
*** achanda has quit IRC00:00
*** salv-orl_ has quit IRC00:06
*** dmorita has joined #openstack-qa00:24
*** yamamoto has joined #openstack-qa00:49
*** jhesketh has quit IRC00:50
*** jhesketh has joined #openstack-qa00:53
*** achanda has joined #openstack-qa01:01
*** achanda has quit IRC01:06
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for horizon integration test bug 1436903  https://review.openstack.org/16875301:07
openstackbug 1436903 in OpenStack Dashboard (Horizon) "integration tests failing blocking gate" [Critical,Triaged] https://launchpad.net/bugs/1436903 - Assigned to David Lyle (david-lyle)01:07
*** salv-orlando has joined #openstack-qa01:07
*** salv-orlando has quit IRC01:12
*** Longgeek has joined #openstack-qa01:15
*** yamamoto has quit IRC01:15
openstackgerritMerged openstack/tempest: Handle fixed_network edge cases gracefully  https://review.openstack.org/16835301:18
lifelesssdague: 261401:24
lifelessbah01:24
lifelesssdague: https://github.com/pypa/pip/issues/261201:24
openstackgerritMerged openstack/tempest: Remove hyphen from rand_name calls in identity tests  https://review.openstack.org/16667601:24
lifelesssdague: if you have the time to put a reproducer together that would be helpful01:25
openstackgerritGhanshyam Mann proposed openstack-dev/devstack: Add Nova v2 API option  https://review.openstack.org/16371801:27
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Remove hyphen from rand_name calls in network tests  https://review.openstack.org/16666901:28
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Add a rule for blocking "-" from rand_name call  https://review.openstack.org/16667901:28
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Remove hyphen from rand_name calls in scenario tests  https://review.openstack.org/16667801:28
*** annashen has joined #openstack-qa01:37
*** annashen has quit IRC01:41
*** yamamoto has joined #openstack-qa01:46
*** mhorban has quit IRC01:49
*** mhorban has joined #openstack-qa01:49
*** salv-orlando has joined #openstack-qa01:53
*** salv-orlando has quit IRC01:58
*** sputnik13 has quit IRC02:05
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: WIP: Add hacking rule for "GET /resources"  https://review.openstack.org/16876202:09
*** salv-orlando has joined #openstack-qa02:10
*** salv-orlando has quit IRC02:12
*** Longgeek has quit IRC02:12
*** Longgeek has joined #openstack-qa02:16
openstackgerritKen'ichi Ohmichi proposed openstack/qa-specs: Add consistent-service-method-names spec  https://review.openstack.org/15931002:25
*** Longgeek has quit IRC02:26
*** Longgeek has joined #openstack-qa02:26
*** mriedem1 has joined #openstack-qa02:37
*** annashen has joined #openstack-qa02:38
*** mriedem has quit IRC02:39
openstackgerritMerged openstack/tempest: Update description of Enable block_migrate_cinder_iscsi  https://review.openstack.org/16518202:41
*** annashen has quit IRC02:42
*** sputnik13 has joined #openstack-qa02:51
*** david-ly_ is now known as david-lyle02:52
*** chandankumar has joined #openstack-qa02:58
*** mriedem1 has quit IRC02:58
*** dims has quit IRC02:59
*** dims has joined #openstack-qa02:59
*** dims has quit IRC03:00
*** haigang has joined #openstack-qa03:04
*** Longgeek_ has joined #openstack-qa03:07
*** Longgeek has quit IRC03:10
*** sabeen has joined #openstack-qa03:11
*** chandankumar has quit IRC03:13
*** ajmiller has quit IRC03:13
*** sabeen has quit IRC03:14
openstackgerritMerged openstack/tempest: Add test for the various config cases of is_admin_available()  https://review.openstack.org/16422503:18
*** salv-orlando has joined #openstack-qa03:25
*** salv-orlando has quit IRC03:30
openstackgerritYuki Nishiwaki proposed openstack-dev/devstack: Restart rabbitmq if rabbitmq-server isn’t running  https://review.openstack.org/16870503:33
*** Madan has joined #openstack-qa03:33
*** dpaterson has quit IRC03:46
*** sabeen1 has joined #openstack-qa03:58
*** sabeen1 has quit IRC04:05
openstackgerritJamie Lennox proposed openstack-dev/devstack: Remove keystone public/admin_endpoint options  https://review.openstack.org/16877004:13
*** dims has joined #openstack-qa04:15
*** woodster_ has quit IRC04:20
*** dims has quit IRC04:21
*** pal_ has joined #openstack-qa04:22
*** achanda has joined #openstack-qa04:24
openstackgerritDean Troyer proposed openstack-dev/devstack: Add inc/rootwrap  https://review.openstack.org/16877304:25
*** jecarey has quit IRC04:27
*** chandankumar has joined #openstack-qa04:27
*** annashen has joined #openstack-qa04:39
*** annashen has quit IRC04:44
*** pal__ has joined #openstack-qa04:53
*** pal_ has quit IRC04:56
*** BharatK has joined #openstack-qa04:57
openstackrecheckopenstack/heat change: https://review.openstack.org/168568 failed check-heat-dsvm-functional-mysql in the gate queue with an unrecognized error05:09
*** pal__ has quit IRC05:10
*** pal_ has joined #openstack-qa05:12
*** k4n0 has joined #openstack-qa05:12
*** mrodden has quit IRC05:17
*** achanda has quit IRC05:22
*** achanda_ has joined #openstack-qa05:22
*** jlibosva has joined #openstack-qa05:25
*** achanda has joined #openstack-qa05:29
*** achanda_ has quit IRC05:30
*** mrodden has joined #openstack-qa05:32
*** pal__ has joined #openstack-qa05:32
*** pal_ has quit IRC05:34
*** woodster_ has joined #openstack-qa05:37
*** pal_ has joined #openstack-qa06:02
*** pal__ has quit IRC06:05
*** neeti has joined #openstack-qa06:06
*** e0ne has joined #openstack-qa06:19
*** jamielennox is now known as jamielennox|away06:20
*** e0ne has quit IRC06:23
*** yfried has joined #openstack-qa06:26
*** gamado has joined #openstack-qa06:27
*** e0ne has joined #openstack-qa06:28
*** e0ne is now known as e0ne_06:35
*** e0ne_ is now known as e0ne06:35
*** itzikb has joined #openstack-qa06:36
*** e0ne has quit IRC06:36
*** Poornima has joined #openstack-qa06:37
mkodererandreaf: ping06:38
openstackgerritJames Polley proposed openstack-dev/devstack: Add a target for, and link to, minimal config docs  https://review.openstack.org/16879406:41
*** hichihara has quit IRC06:54
*** salv-orlando has joined #openstack-qa06:54
openstackgerritBharat Kumar Kobagana proposed openstack-dev/devstack: Clone external plugins before overriding defaults  https://review.openstack.org/16879606:57
*** salv-orl_ has joined #openstack-qa06:58
*** salv-orlando has quit IRC06:58
*** afazekas has joined #openstack-qa07:02
*** pal__ has joined #openstack-qa07:02
*** pal_ has quit IRC07:05
*** sergsh has joined #openstack-qa07:07
*** achanda has quit IRC07:08
*** ihrachyshka has joined #openstack-qa07:22
*** ibiris_away is now known as ibiris07:26
*** jlibosva has quit IRC07:27
*** shardy has joined #openstack-qa07:32
*** jordanP has joined #openstack-qa07:38
*** amotoki has joined #openstack-qa07:38
*** markus_z has joined #openstack-qa07:39
*** woodster_ has quit IRC07:40
*** annashen has joined #openstack-qa07:42
*** salv-orl_ has quit IRC07:42
*** chandankumar has quit IRC07:42
*** chandankumar has joined #openstack-qa07:43
*** jlanoux has joined #openstack-qa07:43
*** liusheng has quit IRC07:44
*** jyuso1 has joined #openstack-qa07:46
*** annashen has quit IRC07:46
*** cody-somerville has quit IRC07:47
*** mpaolino has joined #openstack-qa07:52
*** dims has joined #openstack-qa07:53
*** dkusidlo has joined #openstack-qa07:53
*** gfidente has joined #openstack-qa07:54
*** gfidente has quit IRC07:54
*** gfidente has joined #openstack-qa07:54
*** pal_ has joined #openstack-qa07:55
*** yamamoto has quit IRC07:56
*** pal__ has quit IRC07:57
*** dims has quit IRC07:58
*** tkammer has joined #openstack-qa07:58
*** gamado_ has joined #openstack-qa07:59
*** gszasz has joined #openstack-qa08:00
*** cody-somerville has joined #openstack-qa08:02
*** gamado_ has quit IRC08:03
*** gamado has quit IRC08:04
*** vigneshvar has joined #openstack-qa08:05
*** vigneshvar_ has joined #openstack-qa08:05
*** SridharG has joined #openstack-qa08:06
*** jlibosva has joined #openstack-qa08:09
*** safchain has joined #openstack-qa08:12
*** haigang has quit IRC08:18
*** haigang has joined #openstack-qa08:18
*** yamamoto has joined #openstack-qa08:19
*** haigang has quit IRC08:19
*** pal__ has joined #openstack-qa08:25
*** yamamoto has quit IRC08:25
*** yfried is now known as yfried|afk08:25
andreafmkoderer: pong - what's up08:26
*** pal_ has quit IRC08:28
* andreaf will be afk for ~30min08:29
*** yfried|afk is now known as yfried08:31
*** sergsh has quit IRC08:37
*** ihrachyshka has quit IRC08:39
*** salv-orlando has joined #openstack-qa08:43
*** e0ne has joined #openstack-qa08:43
openstackgerritJoseph Lanoux proposed openstack/tempest: WIP - Add new multi ssh configuration options  https://review.openstack.org/14653508:46
*** salv-orlando has quit IRC08:47
openstackgerritJoseph Lanoux proposed openstack/tempest: Define validation_resources function for ssh validation  https://review.openstack.org/14922908:48
*** sergsh has joined #openstack-qa08:49
openstackgerritJoseph Lanoux proposed openstack/tempest: WIP - Add create_test_server function  https://review.openstack.org/15160108:49
openstackgerritJoseph Lanoux proposed openstack/tempest: Create shared ssh verification by extending RemoteClient  https://review.openstack.org/15389908:50
*** e0ne is now known as e0ne_08:51
*** dkusidlo has quit IRC08:53
*** Longgeek_ has quit IRC08:53
*** e0ne_ is now known as e0ne08:54
*** haigang has joined #openstack-qa08:55
*** vigneshvar has quit IRC08:56
*** vigneshvar_ has quit IRC08:56
*** jlibosva has quit IRC08:58
mkodererandreaf: I added a comment on https://review.openstack.org/#/c/168253/, I would really like to get it merged if this is ok for you08:59
*** gszasz has quit IRC09:02
*** jlibosva has joined #openstack-qa09:02
*** jlibosva has quit IRC09:06
*** zz_johnthetubagu is now known as johnthetubaguy09:07
*** jlibosva has joined #openstack-qa09:09
*** jlibosva has quit IRC09:10
*** jlibosva has joined #openstack-qa09:11
*** bkopilov has quit IRC09:18
*** pal_ has joined #openstack-qa09:21
*** ihrachyshka has joined #openstack-qa09:21
*** tkammer has quit IRC09:22
*** apevec has joined #openstack-qa09:23
*** apevec has quit IRC09:23
*** apevec has joined #openstack-qa09:23
*** bkopilov has joined #openstack-qa09:23
*** pal__ has quit IRC09:23
*** ajo has joined #openstack-qa09:25
*** mpaolino has quit IRC09:25
*** mahito_ has joined #openstack-qa09:25
openstackgerritJoe Hakim Rahme proposed openstack/tempest: Allow Javelin to create volumes without servers  https://review.openstack.org/16883009:25
*** mahito has quit IRC09:28
*** tkammer has joined #openstack-qa09:28
*** ajo_ has joined #openstack-qa09:30
*** jlibosva has quit IRC09:33
*** ajo has quit IRC09:33
andreafmkoderer: sure I wanted to lift my -1 on Friday but I forgot, sorry - I'll do a review today and if ok I'll approve it09:34
mkodererandreaf: ok cool thx alot09:35
*** bkopilov has quit IRC09:37
*** ajo_ is now known as ajo09:38
*** bkopilov has joined #openstack-qa09:39
*** dkusidlo has joined #openstack-qa09:39
*** ajo has quit IRC09:40
*** ajo has joined #openstack-qa09:40
*** dkusidlo has quit IRC09:42
*** annashen has joined #openstack-qa09:43
*** dkusidlo has joined #openstack-qa09:45
*** annashen has quit IRC09:48
*** pal__ has joined #openstack-qa09:53
*** dims has joined #openstack-qa09:54
*** tosky has joined #openstack-qa09:54
*** pal_ has quit IRC09:56
*** vigneshvar has joined #openstack-qa09:58
*** dratushnyy has quit IRC09:58
*** Longgeek has joined #openstack-qa09:58
*** vigneshvar_ has joined #openstack-qa09:59
*** mahito_ has quit IRC10:04
openstackgerritJoe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin  https://review.openstack.org/16577510:07
*** yamamoto has joined #openstack-qa10:12
openstackgerritAlex proposed openstack-infra/subunit2sql: Extend database schema of run_metadata  https://review.openstack.org/16884010:17
*** dmorita has quit IRC10:17
*** Longgeek has quit IRC10:17
*** gamado has joined #openstack-qa10:18
*** pal_ has joined #openstack-qa10:23
*** salv-orlando has joined #openstack-qa10:23
*** jlibosva has joined #openstack-qa10:26
*** pal__ has quit IRC10:26
*** gamado has quit IRC10:29
*** dratushnyy has joined #openstack-qa10:32
*** yfried is now known as yfried|afk10:32
*** yfried|afk is now known as yfried10:34
*** yamamoto has quit IRC10:35
*** pc_m has joined #openstack-qa10:36
*** bkopilov has quit IRC10:37
*** haigang has quit IRC10:40
*** johnthetubaguy is now known as zz_johnthetubagu10:41
*** jhenner has joined #openstack-qa10:41
*** yamamoto has joined #openstack-qa10:42
*** bkopilov has joined #openstack-qa10:47
*** pal_ has quit IRC10:52
*** mwagner_lap has quit IRC10:52
*** yamamoto has quit IRC10:52
*** pal_ has joined #openstack-qa10:52
*** pal_ has quit IRC11:01
*** vigneshvar_ has quit IRC11:04
*** vigneshvar has quit IRC11:04
*** vigneshvar has joined #openstack-qa11:05
*** vigneshvar_ has joined #openstack-qa11:05
*** sergsh has quit IRC11:11
*** Longgeek has joined #openstack-qa11:15
*** jokke_ has quit IRC11:15
*** jaypipes has joined #openstack-qa11:17
*** ChanServ sets mode: +o jaypipes11:17
*** sergsh has joined #openstack-qa11:26
openstackgerritJoe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin  https://review.openstack.org/16577511:26
*** markus_z has quit IRC11:27
*** BharatK has quit IRC11:33
*** rdekel has joined #openstack-qa11:37
*** neeti has quit IRC11:39
*** Longgeek has quit IRC11:40
*** neeti has joined #openstack-qa11:42
*** annashen has joined #openstack-qa11:45
*** vigneshvar has quit IRC11:49
openstackgerritJoseph Lanoux proposed openstack/tempest: Add new multi ssh configuration options  https://review.openstack.org/14653511:49
*** annashen has quit IRC11:49
*** vigneshvar_ has quit IRC11:50
*** yamamoto has joined #openstack-qa11:50
openstackgerritJoseph Lanoux proposed openstack/tempest: Define validation_resources function for ssh validation  https://review.openstack.org/14922911:51
openstackgerritJoseph Lanoux proposed openstack/tempest: Add create_test_server function  https://review.openstack.org/15160111:55
*** woodster_ has joined #openstack-qa12:00
openstackgerritJoseph Lanoux proposed openstack/tempest: Create shared ssh verification by extending RemoteClient  https://review.openstack.org/15389912:01
*** Poornima has quit IRC12:08
*** cdent has joined #openstack-qa12:08
*** tkammer has quit IRC12:11
*** dims has quit IRC12:17
*** dims has joined #openstack-qa12:18
*** dkranz has joined #openstack-qa12:18
*** krtaylor has quit IRC12:19
*** yfried is now known as yfried|afk12:20
*** tkammer has joined #openstack-qa12:23
*** aysyd has joined #openstack-qa12:23
*** salv-orlando has quit IRC12:27
*** salv-orlando has joined #openstack-qa12:30
*** sputnik13 has quit IRC12:30
*** weshay has joined #openstack-qa12:30
*** yfried|afk is now known as yfried12:31
openstackgerritRichard Winters proposed openstack/tempest: Adding multi-node test case for ML2 driver verification  https://review.openstack.org/16888012:32
*** pal_ has joined #openstack-qa12:32
*** rfolco has joined #openstack-qa12:32
*** Longgeek has joined #openstack-qa12:36
*** gamado has joined #openstack-qa12:41
*** e0ne is now known as e0ne_12:44
*** tochi has quit IRC12:45
*** e0ne_ is now known as e0ne12:51
*** dkranz has quit IRC12:53
*** mwagner_lap has joined #openstack-qa12:53
*** dboik has quit IRC12:56
*** Madan has quit IRC12:59
*** krtaylor has joined #openstack-qa13:00
*** k4n0 has quit IRC13:01
*** marrusl has joined #openstack-qa13:03
*** e0ne is now known as e0ne_13:03
*** dkranz has joined #openstack-qa13:04
*** bknudson has quit IRC13:05
*** jecarey has joined #openstack-qa13:09
*** Longgeek has quit IRC13:10
*** haigang has joined #openstack-qa13:12
*** e0ne_ has quit IRC13:14
*** neeti has quit IRC13:15
*** dboik has joined #openstack-qa13:16
*** e0ne has joined #openstack-qa13:16
*** zz_johnthetubagu is now known as johnthetubaguy13:21
*** dustins has joined #openstack-qa13:24
*** johnthetubaguy is now known as zz_johnthetubagu13:34
*** apevec has quit IRC13:36
*** rfolco has quit IRC13:40
*** jlanoux has quit IRC13:40
*** rfolco has joined #openstack-qa13:42
*** mriedem has joined #openstack-qa13:42
*** markus_z has joined #openstack-qa13:43
*** e0ne is now known as e0ne_13:43
*** jgrimm is now known as zz_jgrimm13:45
*** bknudson has joined #openstack-qa13:45
*** e0ne_ is now known as e0ne13:46
*** rfolco has quit IRC13:47
*** annashen has joined #openstack-qa13:47
*** rfolco has joined #openstack-qa13:48
*** tkammer has quit IRC13:48
*** apevec has joined #openstack-qa13:48
*** chandankumar has quit IRC13:50
openstackgerritAbhishek G M proposed openstack/tempest: Test Creeation of server in a shared_network  https://review.openstack.org/16891013:50
andreafmtreinish: hi - you around?13:50
*** rbak has quit IRC13:50
*** annashen has quit IRC13:52
mtreinishandreaf: yep13:52
andreafmtreinish: regarding subuni2sql, I was thinking about how to get more details about the system under test in the DB13:53
andreafmtreinish: the problem I see is that there's no ID that identifies a specific combination of d-g flags, like multinode, mysql or postgres, neutron or n-net etc\13:54
andreafmtreinish: so assuming we get that info into the subunit stream, we would need to store it for each run again and again13:55
mtreinishandreaf: so we store whatever flags logstash would have in the run_metadata table13:56
*** superdan is now known as dansmith13:56
*** sabeen1 has joined #openstack-qa13:56
mtreinishso if they're fields that apply to the whole run, like the db backend, that's probably where they make sense to go13:56
andreafmtreinish: yes but there's kind of a limited number or run types we have13:58
andreafmtreinish: so it would be interesting to be able know that a run uses the same combination of flags as another one, without the need of matching 10 or 15 different fields13:59
*** pal_ has quit IRC14:01
mtreinishandreaf: well that's kinda how we do it with infra. Only the job name is stored in the run_metadata table, which can be used to indicate which flags were used.14:01
*** yamamoto has quit IRC14:03
*** yamamoto has joined #openstack-qa14:03
*** dboik_ has joined #openstack-qa14:03
openstackgerritAbhishek G M proposed openstack/tempest: Test Creeation of server in a shared_network  https://review.openstack.org/16891014:04
*** jyuso1 has quit IRC14:05
andreafmtreinish: yes the job name helps, but that works in upstream only because the test job has full control over the deployment on the cloud as well, so the job name implies the cloud configuration - but when I use subunit2sql for my internal post CI tests I cannot rely on that assumption14:05
*** dboik has quit IRC14:06
mtreinishandreaf: so what are you looking for then? Because you could also just make a comma separate string as the value for the specific flags you wanted too.14:06
andreafmtreinish: well I need to know for instance all test runs that have the same target configuration in common - including software version, configuration options, deployment type etc14:09
*** yamamoto has quit IRC14:10
andreafmtreinish: but I would also like to read the data in a different way, e.g. see the trends for tests across different versions but with the same deployment type - so having a comma separate list for that does not help - unless I store both individual items and the list14:10
*** dkranz has quit IRC14:11
*** rbak has joined #openstack-qa14:12
mtreinishandreaf: oh, ok yeah I follow now. But, my only concern with adding columns for things like this is that they are very use case specific. Which is why I went arbitrary key value pair for that table, because every use case will have different needs14:13
andreafmtreinish: yes I was thinking more about a new table of target systems with its key-value metadata - the problem is that there's no ID14:15
*** rbak has quit IRC14:15
andreafmtreinish: not at the moment at least14:16
mtreinishandreaf: an id for what?14:16
mtreinishandreaf: my other thought is that you can overload the key value for a particular run_id fk in the run_metadata table. (it's not like it has a unique constraint for a given fk)14:17
andreafmtreinish: target system, i.e. combination of deployment type and configuration14:17
openstackgerritKirill Shileev proposed openstack/tempest: Check IPv6 mupltiprefix feature  https://review.openstack.org/15781614:17
*** mtanino has joined #openstack-qa14:18
*** Alex______ has joined #openstack-qa14:19
*** rbak has joined #openstack-qa14:19
mtreinishandreaf: oh, so you're thinking about adding a new table that would map your specific configuration data to an arbitrary id which can be associated to a given run?14:20
andreafmtreinish: yes14:21
*** tkammer has joined #openstack-qa14:21
andreafmtreinish: but there's no such ID -  getting one would be having some algorithm in d-g to create it in a repeatable way, so that when the subunit stream is injected if the ID already exists it can be used and the the rest of the metadata associated with it discareded14:23
mtreinishandreaf: in that case why not just store that id in the run_metadata table then? You could write a small script to query your table against a set of params you pass it (from d-g or where ever) and return the id or create one and return that14:25
mtreinishthen use that as a field in the run_metadata table with the subunit2sql util14:25
*** jlanoux has joined #openstack-qa14:26
*** dkranz has joined #openstack-qa14:27
andreafmtreinish: so create a kind of hash of the metadata I'm interested into, add store it in the metadata table as an additional key/value, and search by that?14:28
*** ajo has quit IRC14:28
*** ChuckC has quit IRC14:29
mtreinishandreaf: yeah, although it doesn't even need to be a hash, you could just use an int or uuid too which is the primary key in your new table.14:30
andreafmtreinish: oh I see what you mean - so get the ID by searching in the new table and add it into the run_metadata as a fk14:31
*** jlanoux_ has joined #openstack-qa14:31
mtreinishandreaf: yeah except it probably won't be a real fk because I don't think you could add that relationship to the value column in run_metadata14:32
andreafmtreinish: indeed14:33
*** achanda has joined #openstack-qa14:33
openstackgerritJoe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin  https://review.openstack.org/16577514:34
*** jlanoux has quit IRC14:34
*** jlibosva has quit IRC14:37
*** cbader has quit IRC14:37
*** asselin has quit IRC14:37
*** cbader has joined #openstack-qa14:37
*** asselin has joined #openstack-qa14:37
andreafmtreinish: I could prepare a draft subunit2sql patch to show you what the table would look like - for me it would be interesting to this as long as it's supported by subunit2sql14:38
*** ilyashakhat has joined #openstack-qa14:38
*** zz_dimtruck is now known as dimtruck14:38
mtreinishandreaf: sure you can push it up, but my concern will be that it will be too specific to your use case14:39
*** cbader has quit IRC14:39
*** cbader has joined #openstack-qa14:39
*** asselin has quit IRC14:40
*** asselin has joined #openstack-qa14:41
andreafmtreinish: my use case seems to be quite general to me,  I agree that we should not have anything specific in there14:41
*** asselin has quit IRC14:41
*** asselin_ has joined #openstack-qa14:41
andreafmtreinish: I'll think a bit more about it a make a draft proposal,  I think I can keep it generic enough and that it would then be beneficial for everyone14:42
*** asselin_ has quit IRC14:42
*** patrickeast has joined #openstack-qa14:42
*** cbader has quit IRC14:42
mtreinishandreaf: ok cool, I'll keep an eye out for it then14:43
*** cbader has joined #openstack-qa14:43
*** asselin_ has joined #openstack-qa14:43
*** asselin_ has quit IRC14:44
*** achanda has quit IRC14:44
*** asselin_ has joined #openstack-qa14:44
*** zz_jgrimm is now known as jgrimm14:44
*** cbader has quit IRC14:45
*** asselin_ has quit IRC14:45
*** cbader has joined #openstack-qa14:45
*** asselin_ has joined #openstack-qa14:45
mtreinishandreaf: also unrelated to subunit2sql I was thinking a bit more about the non-locking providers. Is there a reason we would ever want to use it with the accounts.yaml file?14:46
*** cbader has quit IRC14:46
mtreinishthe locking provider will work fine in the serial case (with a limited number of creds in the file)14:46
*** tsekiyama has joined #openstack-qa14:46
*** asselin_ has quit IRC14:46
*** asselin has joined #openstack-qa14:47
*** cbader has joined #openstack-qa14:47
*** asselin has quit IRC14:47
andreafmtreinish: I see no reason for that, it was designed for the accounts in tempest.conf - so we might want to explicitly forbid that combination14:47
*** cbader has quit IRC14:47
andreafnon-locking provider and accounts.yaml14:47
*** cbader has joined #openstack-qa14:48
*** asselin has joined #openstack-qa14:48
mtreinishandreaf: yeah that's what I was thinking when I was reviewing: https://review.openstack.org/16837314:48
andreafmtreinish: and once we get rid of the configured accounts completely, we can also drop the non-locking account provider14:48
*** cbader has quit IRC14:48
mtreinishyep14:48
*** yfried has quit IRC14:48
mtreinishhogepodge: ^^^ since it's your patch in question any thoughts?14:49
*** asselin has quit IRC14:49
*** cbader has joined #openstack-qa14:49
*** asselin_ has joined #openstack-qa14:49
*** rdekel has quit IRC14:49
*** asselin_ has quit IRC14:50
*** cbader has quit IRC14:50
*** asselin_ has joined #openstack-qa14:50
*** cbader has joined #openstack-qa14:50
*** asselin_ has quit IRC14:51
*** cbader has quit IRC14:51
*** cbader has joined #openstack-qa14:51
*** asselin_ has joined #openstack-qa14:51
*** hemnafk is now known as hemna14:52
*** cbader has quit IRC14:52
*** asselin_ has quit IRC14:52
*** cbader has joined #openstack-qa14:52
*** asselin_ has joined #openstack-qa14:52
*** haigang has quit IRC14:53
hogepodgemtreinish: andreaf: We have two issues, the short term bug that causes the crash, and the longer term strategy for handling accounts.14:54
hogepodgemtreinish: andreaf: also, kind of related, this: https://etherpad.openstack.org/p/accounts-yaml-generation14:55
*** Longgeek has joined #openstack-qa14:55
mtreinishhogepodge: but we could fix both if we re-spun your patch to just remove the accounts.yaml path from the non-locking provider and removed the explicity option for enabling locking (and just made it the check for the accounts file)14:56
*** Guest96413 is now known as redrobot14:58
hogepodgemtreinish: what do you expect the amount of work would be for the latter? I think it's better to do the formally correct thing, but a short term patch until we get there would be nice. I understand that short term solutions have a nasty habit of becoming long term solutions though.14:59
mtreinishhogepodge: heh, well that list cast didn't even fix that code path IIRC which was part of the problem14:59
mtreinishhogepodge: it would be pretty easy I think to disallow that case14:59
hogepodgeIt fixes part of the code path, but not the complete path15:00
*** rwsu has joined #openstack-qa15:01
hogepodgegoing into the mirantis meeting15:01
hogepodgebrb15:01
mtreinishhogepodge: I think http://paste.openstack.org/show/197563/ is all that would be needed along with dropping the locking accounts option (and a unit test to verify that path probably)15:03
*** dboik_ has quit IRC15:04
*** dboik has joined #openstack-qa15:04
openstackgerritMerged openstack-dev/devstack: Move back isset to the functions-common  https://review.openstack.org/15868315:06
andreafmtreinish, hogepodge: the only problem I have with that approach is that it is less explicit - if I do want to use accounts.yaml, and thus set my testr to parallel - but for whatever reason the file is not there - the tests will still run, but there will be problems because of concurrency15:07
*** jlanoux_ has quit IRC15:07
andreafmtreinish, hogepodge: so I was thinking about keeping the option, but ignore accounts.yaml in the non-locking provider, just always used configured credentials15:08
*** jlanoux has joined #openstack-qa15:08
andreafmtreinish, hogepodge: but maybe less configuration options is better, easier to use :)15:09
*** catohornet has joined #openstack-qa15:10
hogepodgeandreaf: mtreinish: I'm much less of an expert, so my feeling is to defer to your judgment. I do have a very short-term interest though, since testing will start soon.15:11
*** ChuckC has joined #openstack-qa15:12
*** Longgeek has quit IRC15:13
mtreinishandreaf: well maybe, because don't we default the path to None? in which case you'd have to explicitly set it to use it15:13
mtreinishhogepodge: well the implementation is simple either way, it's more of about end-user friendliness on which approach we go. Which you definitely can weigh in on15:14
*** chandankumar has joined #openstack-qa15:15
*** afazekas has quit IRC15:16
andreafmtreinish: no, we default to etc/accounts.yaml http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n3815:16
mtreinishandreaf: oh, maybe we don't want to do that then15:16
andreafmtreinish: but by default the file does not exists15:16
andreafmtreinish: which is almost like defaulting to None15:17
mtreinishexcept that if you don't set the option to anything and that file was created it would still try to use it. (which seems less than desirable)15:18
andreafmtreinish: ok, right15:19
*** dkranz has quit IRC15:22
openstackgerritJoseph Lanoux proposed openstack/tempest: DO NOT MERGE- Test ssh validation in the gate  https://review.openstack.org/16550415:22
*** sputnik13 has joined #openstack-qa15:23
*** itzikb has quit IRC15:27
*** dboik_ has joined #openstack-qa15:30
hogepodgemtreinish: trying to access a set as a list is still a straightforward fix that removes an immediate crash, and fixing the larger behavior is desirable too, so do the first then do the second would be my preference, but I may be over-complicating things15:30
hogepodgebut regards to settings, I need to think. It's not clear in my mind yet, but it's early and I just started thinking about it in parallel with another meeting. ;-)15:31
mtreinishhogepodge: heh ok15:32
*** e0ne is now known as e0ne_15:32
*** dboik has quit IRC15:32
mtreinishhogepodge: my issue with the list casting patch as it sits now is that we have no way to verify it fixes anything right now15:33
mtreinishalso IIRC the non-locking provider with the accounts file will just fail slightly later in that code path (which is why we switched your test setup to use the locking provider)15:34
hogepodgeandreaf mtreinish My preference is to be explicit in what behavior I want. If I ask for accounts.yaml, I should get it regardless of parallel or serial execution. If it's a bug to run in serial, an error describing the state should be thrown.15:34
*** eugene_bag has joined #openstack-qa15:34
hogepodgemtreinish: agree. Not sure exactly how to test it, but that's me being a bit fuzzy on how the system works.15:35
*** SridharG has left #openstack-qa15:35
mtreinishhogepodge: well the issue now is that accounts.yaml is usable between 2 providers right now the locking and non-locking (which doesn't work and your patch begins to try and fix)15:35
mtreinishbut I think we've realized that there isn't a reason to use accounts.yaml with the non-locking provider15:35
mtreinishbecause the locking provider works fine when running serially (which is the use case for the non-locking provider)15:36
hogepodgemtreinish: what are the use cases for locking vs non-locking? What's a summary of the fix for the non-locking case?15:37
mtreinishhogepodge: I think we're in agreement on the config approach, which is just have a single accounts.yaml option (which defaults to None) and if specified it'll be used with locking provider15:37
*** e0ne_ is now known as e0ne15:38
mtreinishhogepodge: the non-locking provider is used like the original config one was (just using the same internal api as the locking and isolated creds)15:38
mtreinishright now it supports accounts.yaml because it was simple to add, but there isn't any advantage to doing that15:39
mtreinishthat is the code path your patch is trying to fix15:39
mtreinishas for the full fix for that path I'm not sure what it would entail I haven't ever tried to run in that combination, we would need to push up a test patch to verify it15:39
mtreinishbut instead of bothering with that I think it would be easier to disable that path and only allow the non-locking provider to be used with config options15:40
mtreinishsince there isn't a benefit to ever using it with the accounts.yaml file (even when running serially)15:40
mtreinishthe locking provider works fine serially and it would make the code and configuration around this a bit simpler15:42
*** dboik_ has quit IRC15:42
*** dboik has joined #openstack-qa15:43
*** zz_johnthetubagu is now known as johnthetubaguy15:46
*** tsekiyama has quit IRC15:49
*** tsekiyama has joined #openstack-qa15:49
openstackgerritMerged openstack-infra/elastic-recheck: Add query for horizon integration test bug 1436903  https://review.openstack.org/16875315:53
openstackbug 1436903 in OpenStack Dashboard (Horizon) "integration tests failing blocking gate" [Critical,Triaged] https://launchpad.net/bugs/1436903 - Assigned to David Lyle (david-lyle)15:53
* andreaf afk for ~1h15:56
*** markus_z has quit IRC15:57
openstackgerritMatthew Treinish proposed openstack/tempest: Only use accounts.yaml with locking provider  https://review.openstack.org/16898115:59
mtreinishhogepodge, andreaf: ^^^ this is what I was talking about15:59
*** edmondsw has joined #openstack-qa16:02
*** openstackrecheck has quit IRC16:02
*** openstackrecheck has joined #openstack-qa16:02
*** Alex______ has quit IRC16:03
openstackgerritJoe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin  https://review.openstack.org/16577516:04
edmondswcheck-grenade-dsvm is failing for https://review.openstack.org/#/c/167370/ with a ceilometer error that seems to indicate using a different policy.json than what is in that ceilometer patch set... how do I figure out where the policy.json is coming from that is used in this test?16:04
*** annashen has joined #openstack-qa16:05
edmondswseems to be using this, which doesn't match the patch set:  http://logs.openstack.org/70/167370/5/check/check-grenade-dsvm/7767ba3/logs/etc/ceilometer/policy.json.txt.gz16:06
mtreinishedmondsw: it's being caused by your patch changing the policy.json file and permissions. Grenade takes the previous stable release deploys it runs tests then upgrades to the current master (plus your proposed patch) without changing any of the config from the stable deployment (which includes the policy file)16:08
mtreinishedmondsw: you'll have to change your patch to make it backwards compat and ensure it works using an older version of the policy file16:09
edmondswmtreinish: Thanks.16:10
mtreinishedmondsw: the alternative would be to backport the fixes to fix the policy enforcement behavior in the stable branches so the upgrade path works. (but I don't think that would be a backportable change)16:11
edmondswmtreinish: ceilometer policy enforcement is a completely mess (non-existent in juno and previous, still incomplete in kilo), so being backward compatible may not really be possible16:13
*** yfried has joined #openstack-qa16:13
*** riwinters has joined #openstack-qa16:13
mtreinishedmondsw: well there is a mechanism to manual upgrade config in grenade (using the upgrade_ceilometer file in this case) but you really should try to avoid using it at all costs.16:14
edmondswmtreinish: we'll do what we can and come back with questions about that if necessary... Thanks16:14
mtreinishedmondsw: also before a patch adding a manual upgrade step can land in grenade there needs to be buy in from the ceilo team and something added to the release notes reflecting that all users will have to make the manual step when upgrading16:15
edmondswmtreinish: sure16:15
rahmumtreinish: hello. I started working on Javelin unit tests here https://review.openstack.org/#/c/165775/6. It's still a WIP.I will continue pushing code progressively as I write it, but any early feedback and comment I can get would help me a lot (before I replicate all this on all the resources Javelin deals with).16:16
openstackgerritJordan Pittier proposed openstack-dev/devstack: Swift PyEClib build requires 'make'  https://review.openstack.org/16899016:17
mtreinishrahmu: cool thanks, I saw that you started pushing that. I'll put it on my review list and try to take a prelim look at it today16:17
rahmuthat would be great thanks :)16:18
*** ajmiller has joined #openstack-qa16:28
hogepodgemtreinish: it lgtm at first glance. Let me run it through some paces.16:28
*** jlibosva has joined #openstack-qa16:28
riwintersHi all, I uploaded a patch this morning that failed miserably - looks like import of data_utils from tempest.common.utils16:28
dtroyersdague, jogo: https://review.openstack.org/#/c/168581/ teaches Grenade how to deal with the coming neutron rework in DevStack, and https://review.openstack.org/168591 gets rid of the turns-out-to-be-trouble-in-gerrit symlink16:28
riwintersshould I be using tempest_lib data utils ?16:28
*** dkranz has joined #openstack-qa16:28
mtreinishriwinters: yes, we moved the data_utils module over to tempest_lib and removed the local copy of it from tempest soon after it was in a released version16:30
riwintersAh ok - thanks16:30
*** e0ne has quit IRC16:30
mtreinishriwinters: fyi: https://review.openstack.org/#/c/158012/16:31
openstackgerritJoseph Lanoux proposed openstack/tempest: DO NOT MERGE- Test ssh validation in the gate  https://review.openstack.org/16550416:31
sdaguedtroyer: +A16:31
*** Nic has joined #openstack-qa16:32
*** aysyd has quit IRC16:32
*** jlibosva has quit IRC16:33
*** marun has joined #openstack-qa16:33
mtreinishsdague: so on: https://review.openstack.org/#/c/166993/ I think the project config change landed that was assuming this was in d-g (it didn't have a depends on in the commit msg)16:35
mtreinishthe project-config patch was: https://review.openstack.org/#/c/166964/16:35
*** dratushnyy has quit IRC16:35
*** aysyd has joined #openstack-qa16:35
sdaguemtreinish: ok, can you just fix project-config to use pass through?16:36
mtreinishprobably, it shouldnt be too hard16:36
*** sergsh has quit IRC16:41
*** jlvillal has left #openstack-qa16:41
*** yamahata has joined #openstack-qa16:44
*** Nic has quit IRC16:46
*** jaypipes has quit IRC16:49
openstackgerritRichard Winters proposed openstack/tempest: Adding multi-node test case for ML2 driver verification  https://review.openstack.org/16888016:57
*** erlon has joined #openstack-qa16:59
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: WIP: Trying Nova with oslo.db trunk  https://review.openstack.org/16900416:59
*** dkranz has quit IRC17:01
*** adalbas has joined #openstack-qa17:04
*** bknudson has quit IRC17:04
*** safchain has quit IRC17:11
*** dkusidlo has quit IRC17:11
*** Nic has joined #openstack-qa17:12
*** pal_ has joined #openstack-qa17:13
*** dkranz has joined #openstack-qa17:16
*** yamahata has quit IRC17:18
openstackgerritMatthew Treinish proposed openstack/tempest: Only use accounts.yaml with locking provider  https://review.openstack.org/16898117:19
*** eugene_bag1 has joined #openstack-qa17:19
*** eugene_bag1 has quit IRC17:19
*** jgrimm is now known as zz_jgrimm17:22
*** eugene_bag has quit IRC17:23
*** achanda has joined #openstack-qa17:24
*** melwitt has joined #openstack-qa17:25
*** harlowja_away is now known as harlowja17:25
*** chandankumar has quit IRC17:28
*** dkranz has quit IRC17:28
*** e0ne has joined #openstack-qa17:28
*** mriedem has quit IRC17:34
*** dboik has quit IRC17:35
*** e0ne is now known as e0ne_17:35
*** dboik has joined #openstack-qa17:35
*** amotoki has quit IRC17:36
*** yamahata has joined #openstack-qa17:38
*** e0ne_ is now known as e0ne17:39
openstackgerritMerged openstack-dev/grenade: Add new DevStack Neutron requirements  https://review.openstack.org/16858117:39
openstackgerritMerged openstack-dev/devstack: Remove symlink for lib/neutron compat  https://review.openstack.org/16859117:40
openstackgerritClark Boylan proposed openstack-dev/devstack: Update libvirt cpu map before starting nova  https://review.openstack.org/16840717:40
*** soumit has joined #openstack-qa17:41
*** gamado has quit IRC17:41
*** dkranz has joined #openstack-qa17:41
clarkbdansmith: jogo sdague ^ that updates two things, first only affects libvirt if we are using libvirt and second attempts to use the core2duo cpu as a base since I Think the scenario tests may have failed due to the pentiumpro + lm not be "real" enough17:42
*** jaypipes has joined #openstack-qa17:42
*** ChanServ sets mode: +o jaypipes17:42
clarkbI think I am doing the feature disabling correctly, hopefully that is valid in cpu_map.xml17:42
clarkbkashyap: ^17:42
kashyapclarkb: Looking17:42
dansmithclarkb: what about the scenario tests care about the cpu model?17:42
dansmithclarkb: that seems odd and potentially broken17:43
kashyapHmm, that's likely -- "not being real". /me will experiment locally a bit in a few.17:43
clarkbdansmith: I think its the compiled binaries for eg ssh expecting more instructions to exist on the 64bit build of cirros17:43
clarkbdansmith: because ssh seems to not be working properly17:44
dansmithclarkb: ah17:44
clarkbhttps://jenkins03.openstack.org/job/check-tempest-dsvm-aiopcpu-full/10/console is the job run17:44
*** dpaterson has joined #openstack-qa17:45
hogepodgemtreinish: can you verify my commit message on this patch for accuracy? https://review.openstack.org/#/c/169021/17:45
*** annashen has quit IRC17:46
*** catohornet has quit IRC17:46
clarkbbut if this works then I can figure out a better way to deal with xml in devstack and make this cpu_map update a bit more optional17:47
*** ihrachyshka has quit IRC17:47
*** annashen has joined #openstack-qa17:47
*** Sukhdev has joined #openstack-qa17:48
*** e0ne is now known as e0ne_17:49
clarkbjogo: https://jenkins03.openstack.org/job/check-tempest-dsvm-aiopcpu-full/10/console just failed on the ssh thing :/17:49
clarkbjogo: heisenbug17:49
clarkbkashyap: dansmith https://jenkins05.openstack.org/job/check-tempest-dsvm-neutron-aiopcpu-full/23/ is a better one to follow, the earlier job failed due to a known issue we haven't been able to track down17:49
kashyapclarkb: Thanks, I was trying to follow the other one.17:50
*** mriedem has joined #openstack-qa17:51
openstackgerritAndrey Pavlov proposed openstack-dev/devstack: install euca2ools for tempest preparation  https://review.openstack.org/16902717:52
*** chandankumar has joined #openstack-qa17:53
*** e0ne_ has quit IRC17:54
*** jordanP has quit IRC17:54
*** pal_ has quit IRC17:55
*** dustins_ has joined #openstack-qa17:56
openstackgerritSean Dague proposed openstack-dev/devstack: add back python-guestfs  https://review.openstack.org/16902917:57
*** dustins_ has quit IRC17:57
dansmithclarkb: after this merges are you going to work on making the devstacks run in parallel?17:58
clarkbdansmith: ya I think that is going to have to be the next improvement17:58
dansmithokay cool17:58
*** dustins has quit IRC17:59
sdaguedtroyer: https://review.openstack.org/169029 - a few too many things removed in doing the nova venv17:59
*** dustins has joined #openstack-qa18:01
dtroyerah, that was more of a 'see if we can get away without it' that I forgot about.  we'll beed to add it to the wheel build list18:01
kashyapclarkb: I'm off for the night, will check the status tomm.18:04
clarkbkashyap: thanks18:04
*** greghaynes has quit IRC18:07
openstackgerritRichard Winters proposed openstack/tempest: Adding multi-node test case for ML2 driver verification  https://review.openstack.org/16888018:08
*** soumit has quit IRC18:08
*** soumit has joined #openstack-qa18:09
*** jlanoux has quit IRC18:09
*** bknudson has joined #openstack-qa18:10
*** chandankumar has quit IRC18:10
*** bknudson has left #openstack-qa18:12
*** bknudson has joined #openstack-qa18:14
mtreinishhogepodge: well, it's a subset of tests that require ssh. It's anything that actually honors the run_ssh config flag18:19
mtreinishthere are other tests which use ssh but don't take into account that flag18:19
mtreinishfor example, most of the scenario tests18:19
hogepodgemtreinish: can I have you leave a comment that clarifies that so I can update the commit message18:22
hogepodge?18:22
mtreinishhogepodge: sure, one sec18:22
hogepodgemtreinish: thanks18:23
*** johnthetubaguy is now known as zz_johnthetubagu18:23
clarkbhrm nova seems unhappy with my latest cpu update will dig in once logs come back18:23
mtreinishhogepodge: ok, left a comment18:27
*** eugene_bag has joined #openstack-qa18:27
*** eugene_bag1 has joined #openstack-qa18:30
*** eugene_bag has quit IRC18:33
*** e0ne has joined #openstack-qa18:34
*** achanda has quit IRC18:38
*** pal_ has joined #openstack-qa18:40
*** tosky has quit IRC18:40
*** dustins has quit IRC18:42
*** chandankumar has joined #openstack-qa18:43
*** catohornet has joined #openstack-qa18:45
*** dustins has joined #openstack-qa18:48
*** soumit has quit IRC18:48
*** Nic has quit IRC18:51
*** pal_ has quit IRC18:51
openstackgerritClark Boylan proposed openstack-dev/devstack: Update libvirt cpu map before starting nova  https://review.openstack.org/16840718:55
*** gfidente has quit IRC18:55
*** Rockyg has joined #openstack-qa18:57
*** achanda has joined #openstack-qa18:57
*** achanda has quit IRC18:57
*** achanda has joined #openstack-qa18:58
*** nikil22 has joined #openstack-qa19:07
nikil22hi is there a way to restart all openstack service or at-least neutron related service in devstack through a script. Because i use to kill all screen and rejoin.sh or manually move to particular screen ctrl+c up arror and start again. But i need to make a script to restart all service . Is it possible without starting a screen?19:09
openstackrecheckopenstack/nova change: https://review.openstack.org/152997 failed gate-grenade-dsvm-ironic-sideways in the gate queue with an unrecognized error19:09
openstackgerritMatthew Treinish proposed openstack/tempest: Embed network resoruces and credentials in TestResources  https://review.openstack.org/16783019:10
openstackgerritMatthew Treinish proposed openstack/tempest: Add network support to the accounts providers  https://review.openstack.org/16798819:10
mtreinishandreaf: ^^^ my patch still won't work because I need to init a client to get the network list, but I think that's the last piece19:12
mtreinishandreaf: also I had to rebase everything on https://review.openstack.org/16898119:13
*** Nic has joined #openstack-qa19:15
*** Sukhdev has quit IRC19:20
*** annashen has quit IRC19:22
openstackrecheckConsole logs not available after 13:27s for gate-glance-docs 119174,71,0a8d82019:27
*** cdent has quit IRC19:30
*** adalbas has quit IRC19:31
*** annashen has joined #openstack-qa19:34
*** adalbas has joined #openstack-qa19:45
andreafmtreinish: ok great, I'll start with reviewing https://review.openstack.org/16898119:50
lifelesssdague: ping (re https://github.com/pypa/pip/issues/2612)19:58
*** jecarey has quit IRC20:24
openstackgerritMerged openstack-dev/devstack: add back python-guestfs  https://review.openstack.org/16902920:26
*** afazekas has joined #openstack-qa20:27
*** melwitt has quit IRC20:33
*** melwitt_ has joined #openstack-qa20:33
*** afazekas has quit IRC20:34
*** artom has joined #openstack-qa20:36
*** afazekas has joined #openstack-qa20:37
*** mwagner_lap has quit IRC20:40
*** afazekas has quit IRC20:44
*** e0ne has quit IRC20:46
*** Sukhdev has joined #openstack-qa20:46
*** Rockyg has quit IRC20:48
*** jecarey has joined #openstack-qa20:50
*** rfolco has quit IRC20:54
*** adalbas has quit IRC20:56
*** yamamoto has joined #openstack-qa20:58
*** amotoki has joined #openstack-qa20:59
*** melwitt_ has quit IRC21:01
*** melwitt has joined #openstack-qa21:01
openstackgerritMerged openstack/tempest: Merge availability_zone response schema into one file  https://review.openstack.org/16281621:01
*** melwitt has quit IRC21:02
mtreinishdkranz: if you're around any thoughts on: https://review.openstack.org/#/c/168981/ ?21:02
dkranzmtreinish: I saw that go by and it made sense to me. I'll review it in a few minutes21:03
dkranzmtreinish: I suppose the README has to be updated as well, right?21:04
mtreinishis there something in the readme about that?21:04
*** afazekas has joined #openstack-qa21:06
*** melwitt has joined #openstack-qa21:09
*** chandankumar has quit IRC21:11
*** dustins has quit IRC21:11
dkranzmtreinish: oh, sorry. It was in doc/source/configuration21:13
*** riwinters has quit IRC21:17
*** dpaterson has quit IRC21:18
*** boris-42 has quit IRC21:18
*** afazekas has quit IRC21:18
openstackgerritDavid Kranz proposed openstack/tempest: Prevent failures when running as non-admin  https://review.openstack.org/16695521:18
mriedemmtreinish: have you seen this one? http://logs.openstack.org/46/168646/1/check/check-grenade-dsvm-ironic-sideways/59aadb5/console.html#_2015-03-30_19_31_10_31121:21
mtreinishmriedem: https://bugs.launchpad.net/grenade/+bug/143841521:21
openstackLaunchpad bug 1438415 in grenade "Some network tests failing with IndexError" [Undecided,New]21:21
mtreinishmriedem: yep just a few mins ago21:21
mtreinishalthough I'm wondering if we should just skip lbaas on ironic21:22
mtreinishI mean who's going to do that combo21:22
mtreinishadam_g: ^^^21:22
adam_gmtreinish, hey, i just noticed the same and looking into it now21:22
mriedemhmm, what changed recently?21:22
mtreinishmriedem: we dropped the devstack network name as the default value for compute.fixed_network_name21:23
mtreinishmriedem: in http://git.openstack.org/cgit/openstack/tempest/commit/?id=03feae04e670e0f4f7a950628ce0c9ec3025fba121:24
mriedemmtreinish: why isn't this trace in tempest.txt.gz? http://logs.openstack.org/46/168646/1/check/check-grenade-dsvm-ironic-sideways/59aadb5/console.html#_2015-03-30_19_31_10_31121:27
*** ibiris is now known as ibiris_away21:28
openstackgerritAdam Gandelman proposed openstack-dev/devstack: Set fixed_network_name in Tempest config  https://review.openstack.org/16909521:28
mtreinishmriedem: either it's happening before the logging fixture is started, or the fixtures are misconfigured and stealing the traceback21:29
*** eugene_bag1 has quit IRC21:29
adam_ghttps://review.openstack.org/169095 and its juno cousin should fix it21:29
*** boris-42 has joined #openstack-qa21:30
mtreinishadam_g: a corresponding tempest fix would be to catch that and print out a useful InvalidConfiguration exception. (if you're so inclined :) )21:30
*** cbader has quit IRC21:30
adam_gmtreinish, sure21:30
mtreinishadam_g: also yoy might need it on icehouse21:31
adam_gmtreinish, why for? tempest doesn't run against ironic icehouse anywhere21:31
mtreinishadam_g: well a couple tests get skipped if it's not set, not a big deal either way21:32
mtreinishI don't really care21:32
*** dkranz has quit IRC21:34
*** yamahata has quit IRC21:34
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for ironic sideways bug 1438415  https://review.openstack.org/16909921:35
openstackbug 1438415 in devstack "Some network tests failing with IndexError" [Undecided,In progress] https://launchpad.net/bugs/143841521:35
openstackgerritAdam Gandelman proposed openstack/tempest: Print an error message and fail if network could not be found by name  https://review.openstack.org/16910021:36
*** jamielennox|away is now known as jamielennox21:36
*** ajmiller_ has joined #openstack-qa21:45
*** mriedem is now known as mriedem_away21:45
*** jamielennox is now known as jamielennox|away21:46
openstackgerritMerged openstack-infra/elastic-recheck: Add query for ironic sideways bug 1438415  https://review.openstack.org/16909921:48
openstackbug 1438415 in devstack "Some network tests failing with IndexError" [Undecided,In progress] https://launchpad.net/bugs/143841521:48
*** ajmiller has quit IRC21:48
mtreinishmriedem_away: ^^^21:48
*** jamielennox|away is now known as jamielennox21:53
mriedem_away<321:54
*** aysyd has quit IRC21:55
andreafmtreinish: I will wait for https://review.openstack.org/#/c/168981/ to merge, then I'll submit a new version of https://review.openstack.org/#/c/167830/13 that addresses masayukig's  comment21:56
openstackgerritAaron Rosen proposed openstack-infra/devstack-gate: Allow SERVICE_TIMEOUT to be configured  https://review.openstack.org/16910321:56
*** annashen has quit IRC21:57
mtreinishandreaf: oh, I didn't even notice it had a -121:57
*** arosen has joined #openstack-qa22:00
arosenHi I'm curious if you guys see a lot of CI failures like this:22:00
arosen2015-03-30 21:23:31.683 | [ERROR] /opt/stack/new/devstack/lib/ceilometer:351 ceilometer-api did not start22:00
aroseni see the same thing for cinder-api22:00
arosenand sometimes glance.22:00
arosenthough if i look at the ceilometer-api logs It seems like it's running: http://logs2.aaronorosen.com/11/169011/1/check/dsvm-tempest-full-congress-nodepool/b779c78/logs/screen-ceilometer-api.txt.gz#_2015-03-30_21_23_37_36022:00
clarkbsdague: dtroyer https://review.openstack.org/#/c/168407/7/extras.d/99-cpu_map.sh is working now and the aiopcpu tests pass with that change in place. So question for you all is how do I make that more devstacky? I don't think we need to update the cpu_map.xml by default and suggestions on a better way to parse XML? though the current method seems to work finally22:02
dtroyerclarkb: sdague and I talked about that a bit last week.  One conclusion was to use a real XML parser to make those changes rather than attempt to do it in shell.  although the options there need to be considered, ie, I don't want to wind up with lxml as a DevStack prereq.22:04
clarkbdtroyer: python's stdlib xml module should be sufficient22:05
andreafmtreinish, masayukig, sdague: regarding ssh-auth-bp, the WIP has been lifted: https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bp/ssh-auth-strategy,n,z22:05
clarkbdtroyer: I could write a tiny python script to do it and embed that in devstack22:05
dtroyerclarkb: I think that is what sdague had in mind.  we'll still want to call it from a script like you have though to do the phase checking22:06
dtroyerclarkb: does that need to be re-run on every stack.sh run?22:06
dtroyeror is it a one-time setup thing?22:06
clarkbdtroyer: its a one time setup thing, which sdague also mentioned I needed, idempotency which is doable with python script22:07
clarkbdtroyer: basically if gate64 cpu model exists then return else add it22:07
dtroyerclarkb: ok, I'll leave a couple of other comments on some details22:09
clarkbdtroyer: awesome, thank you22:09
*** mhorban has quit IRC22:10
openstackgerritMerged openstack/tempest: Only use accounts.yaml with locking provider  https://review.openstack.org/16898122:10
*** amotoki has quit IRC22:10
mtreinishandreaf: ^^^22:11
*** openstackrecheck has quit IRC22:14
*** openstackrecheck has joined #openstack-qa22:14
*** dboik_ has joined #openstack-qa22:15
*** yamamoto has quit IRC22:15
*** dboik has quit IRC22:18
*** sabeen1 has quit IRC22:18
*** dboik_ has quit IRC22:19
openstackgerritAndrea Frittoli  proposed openstack/tempest: Initial class creds creation in test base class  https://review.openstack.org/15368122:21
openstackgerritAndrea Frittoli  proposed openstack/tempest: Embed network resoruces and credentials in TestResources  https://review.openstack.org/16783022:21
nikil22@clarkb is there any available script to restart all service in devstack , without using screen?22:21
*** mhorban has joined #openstack-qa22:22
andreafmtreinish: ^^^22:22
clarkbnikil22: unstack then stack?22:26
nikil22clarkb: If I do that all bridge settings will go22:27
*** pc_m has quit IRC22:34
openstackgerritJoe Hakim Rahme proposed openstack/tempest: Allows Javelin to specify Swift role  https://review.openstack.org/16910822:37
openstackgerritAdam Gandelman proposed openstack/tempest: Print an error message and fail if network could not be found by name  https://review.openstack.org/16910022:42
*** annashen has joined #openstack-qa22:42
*** dboik_ has joined #openstack-qa22:43
*** harlowja has quit IRC22:45
*** harlowja_ has joined #openstack-qa22:45
*** Sukhdev has quit IRC22:46
*** marrusl has quit IRC22:48
*** dimtruck is now known as zz_dimtruck22:58
*** annashen has quit IRC23:00
*** marrusl has joined #openstack-qa23:01
*** rbak has quit IRC23:02
mtreinishdkranz, andreaf: btw we need: https://review.openstack.org/169042 to make the non-admin jobs do what we expect23:04
mtreinishright now they'll run with admin enabled23:04
openstackgerritIan Wienand proposed openstack-dev/devstack: [WIP] Testing pip version of pyOpenSSL  https://review.openstack.org/16911223:06
*** patrickeast_ has joined #openstack-qa23:08
*** patrickeast has quit IRC23:08
*** patrickeast_ is now known as patrickeast23:08
*** melwitt has quit IRC23:17
*** dratushnyy has joined #openstack-qa23:19
*** hemna is now known as hemnafk23:20
arosenmtreinish:  if you have a sec do you have any idea why i'm getting this? http://paste.openstack.org/show/197632/23:26
mtreinisharosen: oh, because we changed the default there to None23:27
mtreinishfacepalm that's an obvious bug how did that get through23:27
arosenmtreinish: default where?23:27
arosenmtreinish:  I might need to make a change to the congress tempest files23:28
mtreinisharosen: a default in the tempest config file for auth.test_accounts_file: https://review.openstack.org/#/c/168981/23:28
mtreinisharosen: it's an obvious bug in the code, in the is_admin_available function we never take into account that CONF.auth.test_accounts_file could be None23:29
*** mattoliverau has quit IRC23:29
mtreinishso it's trying to do an isfile(None) which causes that error23:29
arosenmtreinish: gotcha23:29
*** mattoliverau has joined #openstack-qa23:29
mtreinisharosen: it's an easy patch if you want the honors23:30
arosensure23:30
mtreinisharosen: qq though you're not using tenant isolation in the congress stuff?23:30
arosenmtreinish:  we will soon though we aren't because a badly written tempest test we have that i haven't rewritten.23:31
arosenwe need to rewrite our congress tempest tests to do so23:31
mtreinishah, ok. Well you caught a legit bug because of that23:31
arosenif we don't use tenant isolation doesn't it run in parallel?23:31
mtreinisharosen: you need tenant isolation to run in parallel (or use an accounts.yaml file)23:32
mtreinisharosen: http://docs.openstack.org/developer/tempest/configuration.html#auth-credentials23:32
arosenmtreinish:  right so we want to run in serial right now for the congress tests.23:32
arosenbut i'll be fixing that soon so we don't need to  (just need to update the tests).23:33
openstackgerritAaron Rosen proposed openstack/tempest: test_accounts_file can be None  https://review.openstack.org/16911523:34
arosenmtreinish: ^23:34
mtreinisharosen: reviewed, I think we just need that in one more place23:36
openstackgerritAaron Rosen proposed openstack/tempest: test_accounts_file can be None  https://review.openstack.org/16911523:37
arosenmtreinish:  ah good catch. Fixed ^23:38
*** dims has quit IRC23:38
mtreinishhogepodge: ^^^ sry I think I broke things again...23:39
*** tochi has joined #openstack-qa23:45
openstackgerritMerged openstack/tempest: Add API tests for Neutron DVR extension  https://review.openstack.org/9823523:49
*** hichihara has joined #openstack-qa23:50
*** weshay has quit IRC23:58
*** mahito has joined #openstack-qa23:58

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