| *** atiwari has quit IRC | 00:00 | |
| *** jasondotstar has joined #openstack-qa | 00:02 | |
| *** nosnos has joined #openstack-qa | 00:18 | |
| *** giulivo has quit IRC | 00:21 | |
| *** malini_afk is now known as malini | 00:22 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack/tempest: Modify token revocation test to accept 401 or 404 https://review.openstack.org/46341 | 00:32 |
|---|---|---|
| *** jasondotstar has quit IRC | 00:57 | |
| *** zzs has joined #openstack-qa | 01:00 | |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Remove claim that scenario tests need 2 services https://review.openstack.org/47067 | 01:01 |
| *** zzs has left #openstack-qa | 01:04 | |
| *** weshay has joined #openstack-qa | 01:29 | |
| *** mriedem has joined #openstack-qa | 01:52 | |
| *** marun has quit IRC | 01:56 | |
| *** rfolco has joined #openstack-qa | 01:56 | |
| *** adalbas has quit IRC | 01:59 | |
| *** mriedem has quit IRC | 02:02 | |
| *** mriedem has joined #openstack-qa | 02:03 | |
| *** zhikunliu has joined #openstack-qa | 02:05 | |
| *** rfolco has quit IRC | 02:15 | |
| openstackgerrit | zhu zhu proposed a change to openstack/tempest: Add tempest tests for os-host/{host-name} api https://review.openstack.org/46914 | 02:22 |
| openstackgerrit | zhu zhu proposed a change to openstack/tempest: Add tempest tests for os-host/{host-name} api https://review.openstack.org/46914 | 02:25 |
| openstackgerrit | Christopher Yeoh proposed a change to openstack/tempest: Adds test_update_all_metadata_field_not_included negative test https://review.openstack.org/46941 | 02:47 |
| openstackgerrit | ayoung proposed a change to openstack-dev/devstack: make mysql user be defaulted to other than root https://review.openstack.org/40676 | 02:59 |
| *** mriedem has quit IRC | 03:08 | |
| *** mkoderer_ has joined #openstack-qa | 03:13 | |
| *** jamielennox has quit IRC | 03:13 | |
| *** adarazs has quit IRC | 03:13 | |
| *** mkoderer has quit IRC | 03:13 | |
| *** adarazs_ has joined #openstack-qa | 03:14 | |
| *** mkoderer_ is now known as mkoderer | 03:14 | |
| *** jamielennox_ has joined #openstack-qa | 03:18 | |
| *** jamielennox_ is now known as jamielennox | 03:18 | |
| *** weshay has quit IRC | 03:19 | |
| *** masayukig has quit IRC | 03:32 | |
| openstackgerrit | Xiang Hui proposed a change to openstack/tempest: Enhance Neutron ports update test https://review.openstack.org/46894 | 03:35 |
| *** malini is now known as malini_afk | 03:38 | |
| openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates https://review.openstack.org/47076 | 03:50 |
| openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates https://review.openstack.org/47076 | 03:59 |
| *** marun has joined #openstack-qa | 04:12 | |
| openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates https://review.openstack.org/47076 | 04:17 |
| *** SergeyLukjanov has joined #openstack-qa | 04:32 | |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Add 'Field' to the title of the Field Guides https://review.openstack.org/46976 | 04:33 |
| raies | mkoderer: hi gm | 04:38 |
| *** madan has joined #openstack-qa | 04:40 | |
| raies | jaypipes: hi | 04:46 |
| openstackgerrit | Abhijeet Malawade proposed a change to openstack/tempest: GET Volume tests for APIs that accept params (p1) https://review.openstack.org/47078 | 04:48 |
| *** masayukig has joined #openstack-qa | 04:49 | |
| openstackgerrit | Abhijeet Malawade proposed a change to openstack/tempest: GET Volume tests for APIs that accept params (p2) https://review.openstack.org/47079 | 04:51 |
| *** yfried has joined #openstack-qa | 05:00 | |
| openstackgerrit | Jamie Lennox proposed a change to openstack-dev/devstack: Allow deploying keystone with SSL certificates https://review.openstack.org/47076 | 05:06 |
| raies | sdague: please check for approval https://review.openstack.org/#/c/45602/ , https://review.openstack.org/#/c/45627/ and https://review.openstack.org/#/c/45675/ | 05:12 |
| raies | mkoderer: ^^ | 05:19 |
| *** prekarat has joined #openstack-qa | 05:27 | |
| raies | afazekas: hi | 05:41 |
| openstackgerrit | Abhijeet Malawade proposed a change to openstack/tempest: GET Volume tests for APIs that accept params (p1) https://review.openstack.org/47078 | 05:49 |
| *** jhenner has joined #openstack-qa | 06:21 | |
| openstackgerrit | ChangBo Guo proposed a change to openstack/tempest: Use built-in print() instead of print statement https://review.openstack.org/47089 | 06:37 |
| raies | hello abyone from tempest core reviewer ? | 06:48 |
| raies | can someone help me in devstack installation ???? | 07:00 |
| Anju | yes | 07:03 |
| *** fbo_away is now known as fbo | 07:03 | |
| openstackgerrit | zhu zhu proposed a change to openstack/tempest: Add tempest tests for os-host/{host-name} api https://review.openstack.org/46914 | 07:04 |
| *** tkammer has joined #openstack-qa | 07:05 | |
| *** SergeyLukjanov has quit IRC | 07:20 | |
| *** jhenner has quit IRC | 07:20 | |
| *** openstack has joined #openstack-qa | 09:17 | |
| *** zhikunliu has quit IRC | 09:20 | |
| *** SergeyLukjanov has joined #openstack-qa | 09:20 | |
| *** prekarat has quit IRC | 09:21 | |
| giulivo | raies, I think all three are great | 09:21 |
| giulivo | and I'm sure also very welcomed | 09:21 |
| giulivo | it's just the assertIn I'd chage | 09:21 |
| *** prekarat has joined #openstack-qa | 09:22 | |
| *** yfried has joined #openstack-qa | 09:22 | |
| raies | giulivo: I will go as per your direction. You will see new patch very soon | 09:22 |
| giulivo | raies, I know it's boring and I'm sorry about that but we even encourage the use of "more complex" assertions in the HACKING file | 09:24 |
| raies | np. thanks for input | 09:26 |
| giulivo | thanks for all the great stuff | 09:28 |
| raies | giulivo: please let me know what would be n['id'] in assertIn(n[id], [p[id] for p in pools]) | 09:33 |
| openstackgerrit | Qing Xin Meng proposed a change to openstack/tempest: Add update-volume test https://review.openstack.org/47100 | 09:35 |
| *** prekarat has quit IRC | 09:36 | |
| *** thomasbiege has joined #openstack-qa | 09:36 | |
| giulivo | raies, see pvt | 09:36 |
| raies | pvt ? | 09:37 |
| *** prekarat has joined #openstack-qa | 09:37 | |
| *** ken1ohmichi has quit IRC | 09:43 | |
| *** thomasbiege has quit IRC | 09:44 | |
| *** sthaha has quit IRC | 10:16 | |
| *** masayukig has left #openstack-qa | 10:18 | |
| raies | giulivo: taking msg as parameter there is an error saying that I am using 4 parameters while I am using 3 | 10:18 |
| *** nosnos has quit IRC | 10:18 | |
| raies | self.assertIn(self.pool['id'], [p['id'] for p in pools], msg) | 10:18 |
| raies | I am using ^^ | 10:18 |
| raies | I think I should proceed without message. wht do you think ? | 10:19 |
| raies | giulivo: are you there ? | 10:19 |
| *** mkollaro has quit IRC | 10:21 | |
| giulivo | pong raies , yeah drop the msg , sorry | 10:22 |
| raies | ok | 10:23 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: LBaaS client functions and pool testcases https://review.openstack.org/45602 | 10:24 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding member operation test cases https://review.openstack.org/45627 | 10:25 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding member operation test cases https://review.openstack.org/45627 | 10:30 |
| openstackgerrit | A change was merged to openstack/tempest: fix pep8 errors: E231, E128 https://review.openstack.org/46982 | 10:30 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding health monitoring test case https://review.openstack.org/45675 | 10:31 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding health monitoring test case https://review.openstack.org/45675 | 10:36 |
| openstackgerrit | A change was merged to openstack/tempest: assertEquals is deprecated, use assertEqual https://review.openstack.org/46850 | 10:36 |
| raies | giulivo : now you can check for approval: https://review.openstack.org/#/c/45602/ , https://review.openstack.org/#/c/45627/ and https://review.openstack.org/#/c/45675/ | 10:38 |
| raies | giulivo : please ensure approval order as given above as all these are dependent in given order | 10:39 |
| openstackgerrit | A change was merged to openstack/tempest: Use common create_server method for advanced_ops https://review.openstack.org/44197 | 10:48 |
| giulivo | raies, in the future, before committing changes please give an eye to the blueprints and try to tie your submissions to the blueprints because there were a few mentioning missing tests for these features | 11:12 |
| raies | ok | 11:15 |
| raies | thanks for suggestion | 11:16 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding health monitoring test case https://review.openstack.org/45675 | 11:19 |
| giulivo | you'd just have to use the topic appropriately using bp/blueprint-name | 11:19 |
| *** adalbas has joined #openstack-qa | 11:19 | |
| raies | giulivo: I had not submitted any blueprint for member | 11:20 |
| raies | I will take care of this at lease please consider this | 11:21 |
| giulivo | yeah I know, it's stuff from other people | 11:21 |
| *** sthaha has joined #openstack-qa | 11:21 | |
| *** sthaha has joined #openstack-qa | 11:21 | |
| *** sthaha has quit IRC | 11:21 | |
| giulivo | sure np at all, you're doing fine I was just suggesting to give it an eye because it will help tidy up things | 11:21 |
| raies | :) yes it is. Thanks | 11:21 |
| raies | Now you can look member and health patch there for approval | 11:23 |
| raies | I have many more things to implement but thinking to let it approve these things first | 11:23 |
| *** sthaha has joined #openstack-qa | 11:31 | |
| *** sthaha has joined #openstack-qa | 11:31 | |
| *** sthaha has quit IRC | 11:33 | |
| *** sthaha has joined #openstack-qa | 11:35 | |
| *** tkammer has quit IRC | 11:36 | |
| openstackgerrit | A change was merged to openstack/tempest: LBaaS client functions and pool testcases https://review.openstack.org/45602 | 11:38 |
| *** mkollaro has joined #openstack-qa | 11:46 | |
| Anju | giulivo: ping | 11:58 |
| *** adalbas has quit IRC | 12:02 | |
| *** jhenner1 has joined #openstack-qa | 12:06 | |
| *** mkollaro has quit IRC | 12:06 | |
| *** mkollaro1 has joined #openstack-qa | 12:06 | |
| *** mkollaro1 is now known as mkollaro | 12:06 | |
| *** jhenner has quit IRC | 12:07 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding LB member operation test cases https://review.openstack.org/45627 | 12:10 |
| *** adalbas has joined #openstack-qa | 12:10 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding health monitoring test case https://review.openstack.org/45675 | 12:10 |
| *** hootjr has joined #openstack-qa | 12:13 | |
| *** mriedem has joined #openstack-qa | 12:14 | |
| *** thomasbiege1 has joined #openstack-qa | 12:18 | |
| *** madan has quit IRC | 12:18 | |
| *** adalbas has quit IRC | 12:19 | |
| *** rfolco has joined #openstack-qa | 12:26 | |
| *** thomasbiege1 has quit IRC | 12:30 | |
| *** yassine_ has joined #openstack-qa | 12:35 | |
| hootjr | Hi all, I have a neutron question to see if anyone has any insight into.. | 12:36 |
| hootjr | I have a single eth0 for testing purposes, which is connected to my public net | 12:36 |
| hootjr | the network that eth0 is on is a flat /24 with a gateway of 192.168.100.254 | 12:36 |
| hootjr | when I add a subnet to the public, should I be adding it as 192.168.100.0/24 with that gateway? Or should I be subnetting that /24 to maybe a /26 and using that range? so maybe it would be 192.168.100.128/26 and then set it's gateway to 192.168.100.190 | 12:37 |
| *** yassine has quit IRC | 12:37 | |
| *** mkollaro1 has joined #openstack-qa | 12:46 | |
| *** mkollaro has quit IRC | 12:46 | |
| *** weshay has joined #openstack-qa | 12:47 | |
| *** raies has quit IRC | 12:51 | |
| *** jhenner1 has quit IRC | 12:54 | |
| *** adalbas has joined #openstack-qa | 13:03 | |
| *** jhenner has joined #openstack-qa | 13:08 | |
| openstackgerrit | A change was merged to openstack/tempest: Add "region" config for each service https://review.openstack.org/41026 | 13:25 |
| openstackgerrit | A change was merged to openstack/tempest: Adding LB member operation test cases https://review.openstack.org/45627 | 13:25 |
| openstackgerrit | A change was merged to openstack/tempest: Adding health monitoring test case https://review.openstack.org/45675 | 13:25 |
| *** malini_afk is now known as malini | 13:38 | |
| *** ken1ohmichi has joined #openstack-qa | 13:46 | |
| *** ken1ohmichi has quit IRC | 13:46 | |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Increase default swift storage https://review.openstack.org/46770 | 13:47 |
| *** ken1ohmichi has joined #openstack-qa | 13:47 | |
| *** ken1ohmichi_ has joined #openstack-qa | 13:49 | |
| *** yfried has quit IRC | 13:50 | |
| *** ken1ohmichi has quit IRC | 13:51 | |
| malini | mordred: ping | 13:52 |
| *** yfried has joined #openstack-qa | 13:55 | |
| mordred | malini: morning! | 13:55 |
| *** prekarat has quit IRC | 13:55 | |
| malini | mordred: Good Morning!! | 13:55 |
| *** afazekas has joined #openstack-qa | 13:56 | |
| malini | I am working on Marconi tests, trying to move away from ddt based on your feedback | 13:56 |
| mordred | malini: woot! | 13:57 |
| malini | mordered: Can you point me to some of the OS projects taht use testscenarios? So far I just checked tempest & nova | 13:57 |
| mordred | malini: yes - there are actually examples in openstack-dev/pbr | 13:57 |
| mordred | (there are others, I just happen to know about that one) | 13:57 |
| mordred | I think neutron has some too | 13:57 |
| malini | I did a grep thru neutron & cudnt find any..But might be my eyes :) | 13:58 |
| mordred | might also be my brain | 13:59 |
| mordred | malini: ah - ceilometer has a bunch | 13:59 |
| malini | mordered: thanks a lot!! that gives me enough resources to get familiar. testscenarios docs werent much help :) | 14:00 |
| mordred | malini: also oslo.config and python-heatclient and oslo.messaging | 14:00 |
| mordred | malini: yeah - I agree - I think it's much easier to get started with testscenarios with an example | 14:01 |
| mordred | malini: if you look in ceilomter - you'll see in each file that uses testscenarios, a line "load_tests = testscenarios.load_tests_apply_scenarios" | 14:01 |
| afazekas | giulivo: https://review.openstack.org/#/c/46770/ if the devstack patch merged I change the devstack-gate version to remove the config option on the devstack-gate side | 14:02 |
| mordred | be sure to put that line into your test file - it affects how the test loader runs | 14:02 |
| malini | thanks again mordred..You just made my life easier :) | 14:02 |
| *** yassine has joined #openstack-qa | 14:03 | |
| mordred | malini: don't thank me until you're done :) | 14:03 |
| mordred | malini: you might hate me eventually | 14:03 |
| malini | mordered: :D Related to this, does openstack have any history with py.test? | 14:03 |
| mordred | malini: I'm going to send in a couple of other test related patches, now that I'm looking at the repo | 14:04 |
| mordred | malini: only if by history you mean "we do not use it" | 14:04 |
| *** ken1ohmichi_ has quit IRC | 14:04 | |
| mordred | :) | 14:04 |
| malini | mordred: did we ever try using it ? For some reason, I am tempted to think we dont use it for a reason.. | 14:04 |
| mordred | malini: you are tempted correctly | 14:05 |
| *** EmilienM has quit IRC | 14:05 | |
| mordred | malini: we don't use it for a couple of reasons | 14:05 |
| mordred | first of all, when I investigated it about a year ago, it could not run any of the nova unittests | 14:05 |
| *** hootjr has quit IRC | 14:05 | |
| mordred | more importantly now though, we're moving all of openstack to use testr | 14:05 |
| mordred | (testrepository) | 14:05 |
| mordred | which has several features we need/use/will use in the CI system | 14:06 |
| *** hootjr has joined #openstack-qa | 14:06 | |
| giulivo | afazekas, agreed | 14:06 |
| giulivo | but out of curiosity | 14:06 |
| *** yassine_ has quit IRC | 14:06 | |
| giulivo | so if tempest is enabled we expect the default devstack config to change and ensure the tests pass | 14:07 |
| giulivo | isn't this the very same purpose of devstack-gate ? | 14:07 |
| giulivo | where is the distinguo? | 14:07 |
| *** EmilienM has joined #openstack-qa | 14:07 | |
| mordred | giulivo: devstack-gate drives the input config of devstack so that it works for our gate | 14:07 |
| mordred | but a user may want to run tempest in their devsack locally with a different config | 14:07 |
| giulivo | mordred, so this https://review.openstack.org/#/c/34102/ should have actually been posted into devstack, not devstack-gate, correct? | 14:08 |
| giulivo | I mean that was intended to cover the requirements for tempest to succeed, it wasn't related to the actual gating job in particular | 14:09 |
| mordred | giulivo: eeee... tough to say - I'm not sure about enough of the context or if that should have been a change to defaults | 14:09 |
| giulivo | makes sense? | 14:09 |
| afazekas | https://review.openstack.org/#/c/46663/ the gate version :) | 14:10 |
| mordred | what you say makes sense - but I think I need more context on the intent of that change | 14:10 |
| malini | mordred: thanks for the info ! | 14:10 |
| giulivo | mordred, afazekas, so I'd +1 the change into devstack and strip out the setting from devstack-gate completely | 14:10 |
| mordred | malini: sure thing! I'm going to work while I'm stuck on this airplane on getting you migrated to testr - let's see if your tests are clean... | 14:10 |
| malini | mordred: yayyy!!! Thanks ..You are going to get pop tarts from the entire marconi team :) | 14:11 |
| afazekas | giulivo: http://logstash.openstack.org/#eyJzZWFyY2giOiJAbWVzc2FnZTpcIkRldGFpbHM6IFRpbWUgTGltaXQgRXhjZWVkZWQhICg0MDBzKXdoaWxlIHdhaXRpbmcgZm9yIGFjdGl2ZSwgYnV0IHdlIGdvdCBraWxsZWQuXCIgQU5EIEBmaWVsZHMuZmlsZW5hbWU6XCJjb25zb2xlLmh0bWxcIiBBTkQgQGZpZWxkcy5idWlsZF9zdGF0dXM6XCJGQUlMVVJFXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjYwNDgwMCIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwic3RhbXAiOjEzNzk1MTMzNDE0NDh9 , | 14:12 |
| afazekas | other swift/snapshot related test failure can be related as well | 14:12 |
| giulivo | thanks for clarifying about that mordred :) | 14:12 |
| giulivo | mordred, afazekas one more question though, what is the reason to check for tempest in devstack and not just assume that by default the tempest tests should all pass? | 14:17 |
| mordred | giulivo: different tests currently get run based on different configs - clouds actually behave differently (makes me sad) if you use kvm vs. xen, for instance | 14:18 |
| mordred | or if you have the floating-ips extension enabled | 14:18 |
| giulivo | so you mean there could be no 'default' settings for both tempest and devstack which can accomodate a clean run? | 14:19 |
| afazekas | I do not see what is the issue with increasing the value on gate side, 1 change merge can fix the issue sooner than 2 | 14:19 |
| mordred | giulivo: I would be very much in favor of that myself | 14:19 |
| mordred | giulivo: I think that sensible defaults are a thing we ignore around here at times | 14:20 |
| giulivo | mordred, np I'm only trying to get a better idea of how things should be 'used' | 14:21 |
| giulivo | afazekas, I think changing it in devstack-gate would only duplicate the effort and the risks to maintain two different values | 14:22 |
| afazekas | mkoderer: I am ok with any kill -9 version, if it works properly :) | 14:24 |
| openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: Let devstack to decide about swift storage https://review.openstack.org/46663 | 14:37 |
| *** thomasbiege1 has joined #openstack-qa | 14:40 | |
| *** yfried has quit IRC | 14:52 | |
| *** thomasbiege1 has quit IRC | 14:53 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Increase default swift storage https://review.openstack.org/46770 | 14:54 |
| *** fbo is now known as fbo_away | 14:56 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Allow replacing a user variable in a swift template https://review.openstack.org/46856 | 14:57 |
| openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: makes passing the client optional to utilities in scenario/manager.py https://review.openstack.org/47182 | 15:09 |
| mordred | malini: I submitted a couple of patches around tests to marconi - I think the testr patch will need to wait on the ddt patch you're working on | 15:14 |
| mordred | malini: which is sort of funny - because I think that the ddt patch you are working on is going to require the testr patch (nose doesn't honor the load_tests protocol) | 15:14 |
| malini | mordred: testr doesnt work with ddt ? | 15:15 |
| mordred | malini: doesn't look that way, no | 15:15 |
| malini | darn :( | 15:16 |
| mordred | malini: I'm going to dig in to what ddt is doing real quick | 15:18 |
| mordred | because now I'm curious | 15:18 |
| mordred | lifeless: ^^ fyi | 15:18 |
| malini | mordered: ddt lets us define test data on a per test case level, while with testscenarios it needs to be at a class level. Tht is what is holding us back now :( | 15:20 |
| mordred | malini: split tests into more classes? | 15:21 |
| malini | mordered: worst case, thts what we'll do | 15:21 |
| malini | we have a logical grouping of tests based on functionality now | 15:22 |
| mordred | malini: so - the problem right now is that ddt does not sanitize the test names | 15:23 |
| mordred | that it generates | 15:23 |
| openstackgerrit | gongysh proposed a change to openstack-dev/devstack: Record the L3 sevice plugin of ML2 into Q_SERVICE_PLUGIN_CLASSES https://review.openstack.org/47186 | 15:23 |
| *** thomasbiege1 has joined #openstack-qa | 15:24 | |
| malini | mordred: we have some really long ugly names, that we were planning to get rid of. probably testr doesnt like those? | 15:24 |
| mordred | (I think) | 15:24 |
| mordred | I don't think it's testr - I think it might be subunit | 15:24 |
| mordred | that's unhappy | 15:24 |
| mordred | test_name = getattr(v, "__name__", "{0}_{1}".format(name, v)) | 15:25 |
| mordred | is what ddt does | 15:25 |
| *** thomasbiege1 has quit IRC | 15:25 | |
| mordred | and I think the data being fed in to some of them is causing something to be unhappy | 15:25 |
| mordred | tests/functional/wsgi/v1/test_queues.py: @ddt.data('汉字漢字', '@$@^qw', 'i' | 15:25 |
| mordred | for instance | 15:25 |
| mordred | or tests/unit/transport/wsgi/test_messages.py: @ddt.data(None, '[', '[]', '{}', | 15:26 |
| malini | hmm testr doesnt support non ASCII ? | 15:26 |
| *** burt has quit IRC | 15:27 | |
| mordred | this is actually not testr that's breaking | 15:27 |
| mordred | subunit is not pleased -actually, lemme try something | 15:27 |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Remove claim that scenario tests need 2 services https://review.openstack.org/47067 | 15:27 |
| mordred | yup. it's subunit.run that's unhappy. testtools.run is fine | 15:28 |
| mordred | lifeless: ^^ | 15:28 |
| mordred | lifeless: something in how they're using ddt for test multiplication is causing subunit to vomit | 15:28 |
| *** bnemec_ has quit IRC | 15:28 | |
| mordred | malini: btw - I get 4 config file errors when not running via nose | 15:29 |
| *** afazekas has quit IRC | 15:29 | |
| mordred | malini: do this: .tox/py27/bin/python -m testtools.run discover -t ./ ./ | 15:29 |
| mordred | malini: I believe there is an assumption in the test suite about working directories | 15:29 |
| mordred | found it | 15:30 |
| mordred | malini: ok - I updated the testr change with the testsuite changes that make it work under the plain testtools.run runner. as soon as we sort out the ddt issue, I think we'll be somewhere | 15:32 |
| malini | awesome..Thanks! | 15:32 |
| mkoderer | sdague: ping | 15:35 |
| mkoderer | sdague: I'd like to talk about https://review.openstack.org/#/c/46087 if you have time | 15:36 |
| *** marun has quit IRC | 15:37 | |
| mordred | lifeless: seems that ddt does something with test name construction that may be interfering with something in subunit.run not doing something with binary output | 15:39 |
| timello | mkoderer: IIRC, sdague is on LinuxCon, he may not able to respond | 15:39 |
| mkoderer | timello: your right... I forgot about it | 15:41 |
| mkoderer | timello: thx | 15:41 |
| mordred | lifeless: also, the marconi folks are currently enjoying ddt's ability to do multiplication on a per-method rather than per-class basis - any chance a similar feature would be acceptable for testscenarios? or should we be looking at re-orging classes? | 15:42 |
| lifeless | ddt? | 15:44 |
| mordred | lifeless: library marconi currently uses to do data-driven test cases (similar to testscenarios) | 15:44 |
| lifeless | anyhow, testscenarios doing per-method would be entirely fine | 15:45 |
| *** thomasbiege1 has joined #openstack-qa | 15:45 | |
| lifeless | but I don't have the cycles to implement it; I can review it. | 15:45 |
| mordred | that's what I was thinking | 15:45 |
| *** hootjr has left #openstack-qa | 15:45 | |
| mordred | it might be easier to port the per-method values thing to testscenarios than to re-org all of marconi's tests | 15:45 |
| mordred | malini: ^^ | 15:45 |
| lifeless | test name construction - that's plausible, if the test name was already a bytestring rather than a unicode string, you could trigger implicit decode under py2 and fail badly | 15:45 |
| mordred | I'm landing - will reocnnect in a bit | 15:46 |
| *** thomasbiege1 has quit IRC | 15:46 | |
| *** mlavalle has joined #openstack-qa | 15:48 | |
| *** mlavalle has quit IRC | 15:48 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Do not review this https://review.openstack.org/47194 | 15:55 |
| *** hemnafk is now known as hemna | 15:56 | |
| giulivo | dkranz, can you point me to the job which is running the full tempest (not only smoke or gate) periodically^ | 16:00 |
| giulivo | ? | 16:00 |
| dkranz | giulivo: Just a sec.. | 16:00 |
| *** thomasbiege1 has joined #openstack-qa | 16:01 | |
| *** thomasbiege1 has quit IRC | 16:02 | |
| giulivo | but how do we know about it anyway? from the automated emails I get this one linked https://jenkins01.openstack.org/job/periodic-tempest-devstack-vm-all/ | 16:02 |
| dkranz | giulivo: https://jenkins01.openstack.org/job/periodic-tempest-devstack-vm-all/ | 16:03 |
| dkranz | giulivo: Oh, I see you already found it. | 16:03 |
| dkranz | giulivo: What do you mean how do we know about it? | 16:03 |
| giulivo | yeah but I was trying to figure why it isn't on jenkins.openstack.org | 16:03 |
| dkranz | giulivo: jenkins is now spread across jenkins, jenkins01, jenkins02 | 16:05 |
| giulivo | need to learn about zuul, one more reason for it | 16:05 |
| dkranz | giulivo: THere doesn't seem to be a way to "search" all of them. | 16:05 |
| giulivo | thanks! | 16:05 |
| *** yassine has quit IRC | 16:12 | |
| *** thomasbiege1 has joined #openstack-qa | 16:13 | |
| *** marun has joined #openstack-qa | 16:13 | |
| mordred | dkranz: there is not | 16:18 |
| mordred | reporting on periodic jobs is something we need a better answer for | 16:18 |
| dkranz | mordred: I understand. I was just trying to describe the "state of the world as it is now" | 16:18 |
| mordred | ++ | 16:18 |
| malini | mordred, lifeless: Sorry, I had to step away in the middle of the discussion earlier | 16:19 |
| dkranz | mordred: This was announced on the dev list but a lot of folks didn't see it. | 16:19 |
| malini | mordred, lifeless: is the plan to support per method values in testscenarios? | 16:20 |
| mordred | malini: tl;dr - there may be an issue with unicode encoding issues getting tripped in python2 which may need to get sorted in any case | 16:20 |
| mordred | malini: but also, it might be less work to add per-method values to testscenarios than to re-arrange your test right now | 16:20 |
| dkranz | mordred: Perhaps there could be a little blurb at the top of jenkins.openstack.org listing the other places. | 16:20 |
| dkranz | mordred: At least until something better comes along. | 16:20 |
| malini | mordered: the per method value support will be great to have ! tht is the only reason we have now for sticking with ddt | 16:22 |
| mordred | malini, lifeless: this is what re-working a couple to use class-based looks like: http://paste.openstack.org/show/47222 | 16:22 |
| mordred | which seems to be a little extra verbose | 16:23 |
| malini | mordred: yeap..we'll end up creating separate classes for a lot many tests | 16:26 |
| lifeless | mordred: so you're in SFO or something now? | 16:29 |
| mordred | lifeless: SLC - but yeah | 16:34 |
| lifeless | woger wilco | 16:34 |
| mordred | lifeless: funny enough - I'm in the sky club we were in last time we dug in to testscenarios | 16:34 |
| lifeless | mordred: don't miss this flight | 16:34 |
| mordred | lifeless: ++ | 16:34 |
| lifeless | mordred: I'm currently hacking a lung out in my hotel room | 16:35 |
| lifeless | mordred: I hope to be down at the office before you | 16:35 |
| mordred | I'm so happy I'm coming to visit you all | 16:35 |
| *** SergeyLukjanov has quit IRC | 16:39 | |
| *** thomasbiege1 has quit IRC | 16:40 | |
| giulivo | it's a bit obscure to me how we could run the checks from hacking/checks.py without tox, anyone willing to help? | 16:45 |
| mordred | giulivo: testr run | 16:47 |
| mordred | lifeless: first sketch: http://paste.openstack.org/show/47224 | 16:47 |
| giulivo | mordred, I see but that won't help cause I still don't get how it is doing | 16:50 |
| *** sthaha has quit IRC | 16:52 | |
| *** thomasbiege1 has joined #openstack-qa | 16:52 | |
| openstackgerrit | A change was merged to openstack/tempest: Add missing import of 'subunit' in test-requirements.txt https://review.openstack.org/46619 | 16:53 |
| openstackgerrit | A change was merged to openstack/tempest: Add update-snapshot test https://review.openstack.org/46860 | 16:53 |
| lifeless | mordred: interesting; I think I need to see it in action. | 16:57 |
| giulivo | mordred, I see there is a 'hacking' module installed which takes care of that | 16:58 |
| *** jasondotstar has joined #openstack-qa | 17:08 | |
| *** jasondotstar has quit IRC | 17:10 | |
| *** jasondotstar has joined #openstack-qa | 17:11 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Do not review this https://review.openstack.org/47194 | 17:13 |
| *** fbo_away is now known as fbo | 17:18 | |
| *** EmilienM has quit IRC | 17:46 | |
| *** RecheckWatchBot has joined #openstack-qa | 17:47 | |
| mtreinish | jog0: ^^^ | 17:47 |
| *** EmilienM has joined #openstack-qa | 17:49 | |
| *** jasondotstar has quit IRC | 18:19 | |
| *** EmilienM has quit IRC | 18:29 | |
| *** EmilienM has joined #openstack-qa | 18:32 | |
| mordred | lifeless: well, I need to get it to work too | 18:32 |
| *** thomasbiege1 has quit IRC | 18:33 | |
| *** jasondotstar has joined #openstack-qa | 18:35 | |
| RecheckWatchBot | openstack/neutron change: https://review.openstack.org/47218 failed tempest because of: https://bugs.launchpad.net/bugs/1211915 | 18:37 |
| jog0 | \o/ | 18:39 |
| jog0 | ^ automated recheckwatching | 18:40 |
| openstackgerrit | A change was merged to openstack/tempest: Add 'Field' to the title of the Field Guides https://review.openstack.org/46976 | 18:41 |
| openstackgerrit | A change was merged to openstack/tempest: Move _check_tenant_network_connectivity to the end https://review.openstack.org/46791 | 18:42 |
| *** thomasbiege1 has joined #openstack-qa | 18:43 | |
| *** dosaboy has quit IRC | 18:43 | |
| *** dosaboy has joined #openstack-qa | 18:43 | |
| *** dosaboy has quit IRC | 18:44 | |
| *** RecheckWatchBot has quit IRC | 18:45 | |
| *** RecheckWatchBot has joined #openstack-qa | 18:45 | |
| *** thomasbiege1 has quit IRC | 18:46 | |
| *** dosaboy has joined #openstack-qa | 18:46 | |
| mordred | jog0: woot! | 18:51 |
| *** annegentle has quit IRC | 18:56 | |
| RecheckWatchBot | openstack/python-keystoneclient change: https://review.openstack.org/30043 failed tempest with an unrecognized error | 18:56 |
| *** malini is now known as malini_afk | 19:03 | |
| lifeless | mordred: heh indeed. | 19:12 |
| lifeless | mordred: where areyou? | 19:12 |
| RecheckWatchBot | openstack/ceilometer change: https://review.openstack.org/47221 failed tempest because of: https://bugs.launchpad.net/bugs/1211915 | 19:20 |
| *** jasondotstar has quit IRC | 19:31 | |
| RecheckWatchBot | openstack/python-keystoneclient change: https://review.openstack.org/30043 failed tempest with an unrecognized error | 19:42 |
| *** Luz_ has joined #openstack-qa | 19:42 | |
| Luz_ | Hello, I'm configuring tempest to run it for an openstack implementation... right now I have some test cases related to volume that is failing, and I was wondering what information should go on this parameters: | 19:43 |
| Luz_ | # Protocol and vendor of volume backend to target when testing volume-types. | 19:44 |
| Luz_ | # You should update to reflect those exported by configured backend driver. | 19:44 |
| *** jasondotstar has joined #openstack-qa | 19:44 | |
| Luz_ | storage_protocol = iSCSI | 19:44 |
| Luz_ | vendor_name = Open Source | 19:44 |
| *** zzs has joined #openstack-qa | 19:45 | |
| Luz_ | any thoughts? | 19:45 |
| Luz_ | does anyone knows where can I get this info? | 19:47 |
| *** malini_afk is now known as malini | 19:47 | |
| RecheckWatchBot | openstack/requirements change: https://review.openstack.org/46081 failed tempest with an unrecognized error | 19:51 |
| *** adalbas has quit IRC | 19:57 | |
| adam_g | Luz_, those defaults should match the default lvm backend | 20:00 |
| adam_g | Luz_, if you're using a different storage backend in cinder, you'll need those updated to match what the driver exports | 20:02 |
| openstackgerrit | A change was merged to openstack/tempest: Add a function for creating a server on a given network https://review.openstack.org/46899 | 20:02 |
| Luz_ | ok, thank you Adam... we are using ceph as backend... | 20:17 |
| Luz_ | where do I look the driver info? | 20:18 |
| Luz_ | at cinder.conf? | 20:18 |
| RecheckWatchBot | openstack-dev/devstack change: https://review.openstack.org/46816 failed tempest because of: https://bugs.launchpad.net/bugs/1218391 | 20:18 |
| *** annegentle has joined #openstack-qa | 20:29 | |
| *** mkollaro1 has quit IRC | 20:30 | |
| *** SergeyLukjanov has joined #openstack-qa | 20:31 | |
| RecheckWatchBot | openstack/nova change: https://review.openstack.org/46379 failed tempest because of: https://bugs.launchpad.net/bugs/1211915 | 20:34 |
| *** mkollaro has joined #openstack-qa | 20:39 | |
| morganfainberg | any tempest folks around? I'm looking to get some eyes on https://review.openstack.org/#/c/46341/ since it blocks a fix for keystone tagged to RC1. | 20:41 |
| mtreinish | morganfainberg: we normally don't change tests that way. If it needs to be changed to get a fix through we use the skip decorator with the bug id so we can track the skip. | 20:42 |
| mtreinish | and then the expected response is updated when the skip is removed | 20:43 |
| morganfainberg | mtreinish, ah ok, i'll go and update it | 20:43 |
| morganfainberg | mtreinish, that is much clearer than the comments i saw there. | 20:43 |
| morganfainberg | :) | 20:43 |
| morganfainberg | mtreinish, should i also make the change to accept the new http status at the same time as adding the skip? or is it better to add it when we un-skip? | 20:44 |
| mtreinish | morganfainberg: it's probably better to do it when we un-skip it | 20:44 |
| morganfainberg | mtreinish, works for me. | 20:44 |
| morganfainberg | mtreinish i'll have a new patch with the proper skip in place up in a few minutes. thanks! | 20:45 |
| mtreinish | morganfainberg: sure np, yeah I guess those comments aren't the most clear if you're not familiar with skip tests to change an api response. | 20:48 |
| openstackgerrit | A change was merged to openstack/tempest: Fix a test case of 35chars+ https://review.openstack.org/46852 | 20:53 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack/tempest: Skip test tokens https://review.openstack.org/46341 | 20:54 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack/tempest: Skip test tokens https://review.openstack.org/46341 | 20:59 |
| *** jasondotstar has quit IRC | 21:01 | |
| *** RecheckWatchBot has quit IRC | 21:02 | |
| *** RecheckWatchBot has joined #openstack-qa | 21:03 | |
| *** RecheckWatchBot has quit IRC | 21:03 | |
| *** RecheckWatchBot has joined #openstack-qa | 21:03 | |
| *** rfolco has quit IRC | 21:07 | |
| *** jamielennox is now known as jamielennox|away | 21:15 | |
| RecheckWatchBot | openstack/nova change: https://review.openstack.org/46379 failed tempest because of: https://bugs.launchpad.net/bugs/1191960 | 21:18 |
| mtreinish | jog0: ^^^ looks like your reworked wait patch works \0/ | 21:19 |
| jog0 | mtreinish: woot! | 21:20 |
| jog0 | mtreinish: I think with this we are definitly ready to move this over to openstack-infra tomorrow | 21:20 |
| mtreinish | jog0: yeah now I just need to figure out daemon context and I'll push the fixes out | 21:21 |
| *** weshay has quit IRC | 21:23 | |
| jog0 | mtreinish: awesome | 21:23 |
| *** giulivo has quit IRC | 21:24 | |
| mtreinish | jog0: quick question does stream auto reconnect if gerrit goes down? Or should I handle that in the bot? | 21:26 |
| jog0 | I don't think it does but not sure | 21:27 |
| jog0 | I use the gerritlib to do that | 21:27 |
| jog0 | ask jeblair and infra I think the wrote that | 21:27 |
| mtreinish | ok I just based the irc bot on gerritbot and in gerritbot they have an exception that'll reconnect if it catches an exception in the main loop | 21:28 |
| jog0 | mtreinish: it can't hurt | 21:28 |
| morganfainberg | mtreinish, ok patchset passes now. thanks for the help earlier, got it all under the decorator now. | 21:39 |
| mtreinish | morganfainberg: ok I'll take a look in a sec | 21:41 |
| morganfainberg | mtreinish, appriciate it | 21:42 |
| mtreinish | morganfainberg: lgtm | 21:43 |
| *** jasondotstar has joined #openstack-qa | 22:06 | |
| *** zzs has left #openstack-qa | 22:14 | |
| *** mriedem has quit IRC | 22:17 | |
| *** jasondotstar has quit IRC | 22:23 | |
| *** RecheckWatchBot has quit IRC | 22:39 | |
| *** Luz_ has quit IRC | 22:56 | |
| *** RecheckWatchBot has joined #openstack-qa | 23:08 | |
| *** jasondotstar has joined #openstack-qa | 23:08 | |
| *** jamielennox|away is now known as jamielennox | 23:13 | |
| *** malini is now known as malini_afk | 23:34 | |
| *** mriedem has joined #openstack-qa | 23:35 | |
| *** ken1ohmichi has joined #openstack-qa | 23:36 | |
| *** fbo is now known as fbo_away | 23:40 | |
| *** hemna is now known as hemnafk | 23:42 | |
| *** jasondotstar has quit IRC | 23:44 | |
| *** jasondotstar has joined #openstack-qa | 23:55 | |
| *** mkollaro has quit IRC | 23:58 | |
| *** mkollaro has joined #openstack-qa | 23:58 | |
| *** jhenner has quit IRC | 23:59 | |
| *** mkollaro has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!