| mriedem | ah nevermind, well py26 is junk now for tempest | 00:06 |
|---|---|---|
| openstackgerrit | Nachi Ueno proposed a change to openstack-dev/devstack: Added functions for get a ip on an instance https://review.openstack.org/41799 | 00:26 |
| *** woodspa has joined #openstack-qa | 00:52 | |
| openstackgerrit | Steve Baker proposed a change to openstack/tempest: Heat autoscaling scenario test https://review.openstack.org/36367 | 01:21 |
| *** stevebaker has joined #openstack-qa | 01:22 | |
| openstackgerrit | Shawn Hartsock proposed a change to openstack-dev/devstack: vmware - vsphere v5.0 - WSDL cleanup script. https://review.openstack.org/41824 | 01:46 |
| openstackgerrit | Shawn Hartsock proposed a change to openstack-dev/devstack: vmware - vsphere v5.0 - WSDL cleanup script. https://review.openstack.org/41824 | 01:48 |
| openstackgerrit | Shawn Hartsock proposed a change to openstack-dev/devstack: vmware - vsphere v5.0 - WSDL cleanup script. https://review.openstack.org/41824 | 01:55 |
| *** LiangZheng has joined #openstack-qa | 02:01 | |
| openstackgerrit | Shawn Hartsock proposed a change to openstack-dev/devstack: vmware - vsphere v5.0 - WSDL cleanup script. https://review.openstack.org/41824 | 02:01 |
| mriedem | mtreinish: i found the problem - it's not the switch from nose to testr, it's the addition of using tearDownClass in base.py here: | 02:04 |
| mriedem | https://github.com/openstack/tempest/commit/b86cda92e2560ef69bcceef367fbf323be0e8c6e | 02:04 |
| mriedem | https://github.com/openstack/tempest/commit/b86cda92e2560ef69bcceef367fbf323be0e8c6e#L1R113 | 02:04 |
| *** woodspa has quit IRC | 02:13 | |
| mriedem | mtreinish: bug opened: https://bugs.launchpad.net/tempest/+bug/1212071 | 02:14 |
| *** mriedem has quit IRC | 02:17 | |
| *** anteaya has quit IRC | 02:40 | |
| *** LiangZhe_ has joined #openstack-qa | 02:49 | |
| *** LiangZheng has quit IRC | 02:52 | |
| *** LiangZhe_ has quit IRC | 03:06 | |
| *** LiangZheng has joined #openstack-qa | 03:07 | |
| openstackgerrit | Steve Baker proposed a change to openstack/tempest: Heat autoscaling scenario test https://review.openstack.org/36367 | 03:40 |
| *** SergeyLukjanov has joined #openstack-qa | 03:43 | |
| openstackgerrit | raiesmh08 proposed a change to openstack/tempest: Adding network api xml support https://review.openstack.org/41673 | 04:35 |
| *** yfried has joined #openstack-qa | 05:18 | |
| *** prekarat has joined #openstack-qa | 05:22 | |
| *** SergeyLukjanov has quit IRC | 05:55 | |
| *** SergeyLukjanov has joined #openstack-qa | 05:56 | |
| *** SergeyLukjanov has quit IRC | 06:02 | |
| openstackgerrit | raiesmh08 proposed a change to openstack/tempest: Adding network api xml support https://review.openstack.org/41673 | 06:23 |
| *** tkammer has joined #openstack-qa | 06:33 | |
| *** afazekas has joined #openstack-qa | 06:50 | |
| *** tkammer has quit IRC | 07:00 | |
| *** afazekas_ has joined #openstack-qa | 07:08 | |
| *** afazekas_ has quit IRC | 07:13 | |
| *** raies has joined #openstack-qa | 07:14 | |
| raies | afazekas: I have added xml api support please check https://review.openstack.org/#/c/41673/ | 07:15 |
| raies | afazekas: r u there ? | 07:16 |
| afazekas | yes | 07:16 |
| raies | afazekas: Please check the link what i shared :) | 07:17 |
| afazekas | https://review.openstack.org/#/c/41673/3/tempest/api/network/base.py can the port name be an optional argument ? | 07:17 |
| *** jhenner has joined #openstack-qa | 07:18 | |
| raies | afazekas: I have made it necessary here | 07:18 |
| raies | to check name to be updated | 07:18 |
| afazekas | https://github.com/openstack/tempest/blob/master/tempest/services/network/json/network_client.py#L98 | 07:20 |
| afazekas | the json version has a different signature | 07:21 |
| *** tkammer has joined #openstack-qa | 07:21 | |
| Anju_ | afazekas: i am getting an error while running a cli nova attach-volume | 07:22 |
| afazekas | raises: create_port(self, network_id, state=None, name=None): should be the signature | 07:23 |
| raies | afazekas: I did not understand the meaning of signature | 07:24 |
| raies | what's it ? | 07:24 |
| afazekas | Anju_: please copy paste the command and the error , and the volume and instance list | 07:24 |
| afazekas | raies: it means the json and xml functions should have the same argument ordering and meaning | 07:24 |
| afazekas | you can extended the create port by a name in both version | 07:25 |
| afazekas | the json in your patch is ' def create_port(self, network_id, name, state=None):' | 07:26 |
| afazekas | the xml 'def create_port(self, net_uuid, name):' | 07:26 |
| Anju_ | afazeaks : http://pastebin.com/wtphyMB3 | 07:27 |
| raies | afazekas: u mean that i should add one more parameter state ? | 07:27 |
| raies | or I should remove state from json ? | 07:27 |
| afazekas | IMHO this should be the definition of both: create_port(self, network_id, state=None, name=None): | 07:28 |
| raies | ok | 07:29 |
| raies | I will amend it | 07:29 |
| afazekas | if it does not cause an issue to pending patch you may change order .. | 07:29 |
| Anju_ | afazekas : check the last error : | 07:30 |
| afazekas | raies: Thank you, if those are None -s, you should exclude them from the requests | 07:30 |
| raies | I think better would be to remove state from both the xml and json | 07:31 |
| *** raies has quit IRC | 07:31 | |
| Anju_ | afazekas : are u there | 07:34 |
| afazekas | Anju_: yes, but I did found your pastebin about the error | 07:36 |
| Anju_ | error is in the last | 07:37 |
| Anju_ | ERROR: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-4ff723ac-44c2-45e6-ace2-fc38a5effb19) | 07:37 |
| Anju_ | afazekas : ERROR: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-4ff723ac-44c2-45e6-ace2-fc38a5effb19) | 07:39 |
| *** jgallard has joined #openstack-qa | 07:39 | |
| afazekas | you should check the service logs for more detail, | 07:41 |
| afazekas | is the volume state active ? | 07:41 |
| Anju_ | volume status is avalible | 07:42 |
| afazekas | check the screen-n-cpu and screen-n-api log for details | 07:44 |
| afazekas | probably you will see a related error log entry | 07:45 |
| Anju_ | afzekas : i forgot where the logs reside | 07:46 |
| Anju_ | afazekas : i forgot where the logs reside | 07:47 |
| afazekas | by default they are in the /opt/stack/new/screen-logs | 07:47 |
| Anju_ | afazeaks : if not there ? | 07:51 |
| *** fbo_away is now known as fbo | 07:52 | |
| Anju_ | afazkas : if not there ? | 07:55 |
| Anju_ | afazekas : if not there ? | 07:55 |
| afazekas | you can join to screen and check the output of the processes | 07:56 |
| afazekas | or you can check the ps aux or ps -eo pid,args for the service start command, and figure out the log file | 07:57 |
| Anju_ | okk | 07:58 |
| *** SergeyLukjanov has joined #openstack-qa | 08:04 | |
| *** tkammer has quit IRC | 08:06 | |
| *** raies has joined #openstack-qa | 08:14 | |
| raies | afazekas: sorry network connectivity was lost | 08:15 |
| *** prekarat has quit IRC | 08:15 | |
| *** dosaboy has joined #openstack-qa | 08:15 | |
| raies | afazekas: I will update create port in order please check others also | 08:16 |
| raies | afazekas: r u there ? | 08:21 |
| afazekas | raies: pong | 08:23 |
| afazekas | ok | 08:23 |
| raies | afazekas : thanks , please check others also | 08:23 |
| afazekas | https://review.openstack.org/#/c/41673/3/tempest/api/network/test_networks.py L117-L182 removed, why ? | 08:25 |
| *** fbo is now known as fbo_away | 08:26 | |
| raies | afazekas: yes I will also check this once again | 08:27 |
| raies | total 2 reviews to amend :) | 08:27 |
| afazekas | https://review.openstack.org/#/c/41673/3/tempest/api/network/test_networks.py additional code removal from here | 08:28 |
| afazekas | L 119-L125, L140,-L146 | 08:29 |
| *** giulivo has joined #openstack-qa | 08:30 | |
| raies | afazekas : yes these are related i will check and verify please suggest others | 08:30 |
| *** prekarat has joined #openstack-qa | 08:33 | |
| afazekas | Nice to have xml style sheet (xsd) reference in the xml request , but if it is working without that it is not required IMHO | 08:34 |
| raies | afazekas : I did not understand please elaborate a bit | 08:36 |
| *** tkammer has joined #openstack-qa | 08:37 | |
| afazekas | check the XMLNS_11 usage, (incorrect), so not needed to copy | 08:38 |
| afazekas | XMLNS another one | 08:38 |
| *** SergeyLukjanov has quit IRC | 08:44 | |
| *** fbo_away is now known as fbo | 08:46 | |
| *** jhenner has quit IRC | 08:48 | |
| *** SergeyLukjanov has joined #openstack-qa | 08:49 | |
| *** psedlak has quit IRC | 08:56 | |
| raies | afazekas: I have issue in adding code for removed what you suggested. Previously I was having same problem please check http://paste.fedoraproject.org/31996/13764705/ | 08:58 |
| afazekas | Anju_ : rejoin_stack.sh press Ctrl+A and " , and select the app | 08:59 |
| afazekas | raises: looks like n is string instead of a dict like object | 09:01 |
| afazekas | raies: ^^ | 09:01 |
| afazekas | the xml responses frequently contains an additional root element which can be removed .. | 09:02 |
| afazekas | raies: can you copy paste a (unified diff) patch which contains the xml code and those tricky codes ? output of the git diff master | 09:05 |
| Anju_ | hey afzazeks : i have done sorry i forgot to tell u | 09:05 |
| *** jgallard has quit IRC | 09:10 | |
| raies | afazekas : which specific diff u need ? | 09:10 |
| *** jgallard has joined #openstack-qa | 09:10 | |
| raies | afazekas : I have created diff, but it is 400+ lines | 09:11 |
| raies | so what specific u need ? | 09:11 |
| afazekas | raies: you can use one oft the pastebin sites | 09:13 |
| afazekas | I would like to apply the patch and see what happening actually | 09:14 |
| afazekas | raies: can you find out what is the issue and fix it ? | 09:27 |
| raies | afazekas: please find diff http://paste.fedoraproject.org/31999/72485137/ | 09:28 |
| *** prekarat has quit IRC | 09:29 | |
| raies | afazekas : got ? | 09:33 |
| afazekas | http://www.fpaste.org/32003/37647305/ | 09:37 |
| *** psedlak has joined #openstack-qa | 09:39 | |
| afazekas | I got it | 09:43 |
| *** prekarat has joined #openstack-qa | 09:43 | |
| raies | afazekas : I tried your command on my system I found another error "can't find file to patch at input line 5 Perhaps you used the wrong -p or --strip option?" | 09:49 |
| raies | by the way what do u want to indicate from your paste | 09:50 |
| afazekas | so the response should be dict with 'subnets' key which contains a list of subnets | 09:59 |
| afazekas | probably it requires a similar method like the list_snapshots, but with 'subnets' "main" dict entry | 10:04 |
| raies | ok | 10:05 |
| raies | i will see it | 10:05 |
| *** prekarat has quit IRC | 10:06 | |
| *** prekarat has joined #openstack-qa | 10:11 | |
| *** LiangZheng has quit IRC | 10:33 | |
| *** prekarat has quit IRC | 10:45 | |
| *** prekarat has joined #openstack-qa | 10:48 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: setUpClass/tearDownClass full chain https://review.openstack.org/39346 | 10:52 |
| *** LiangZheng has joined #openstack-qa | 11:04 | |
| *** prekarat has quit IRC | 11:11 | |
| *** LiangZheng has quit IRC | 11:12 | |
| *** jhenner has joined #openstack-qa | 11:13 | |
| openstackgerrit | raiesmh08 proposed a change to openstack/tempest: Adding network api xml support https://review.openstack.org/41673 | 11:14 |
| raies | afazekas: I have added modified create port please check https://review.openstack.org/#/c/41673/ | 11:16 |
| raies | still I was unable to sort out of the problem of missing code | 11:16 |
| *** weshay has joined #openstack-qa | 11:22 | |
| *** prekarat has joined #openstack-qa | 11:26 | |
| afazekas | raies: I think the name should be optional https://github.com/openstack/tempest/blob/master/tempest/services/compute/json/servers_client.py#L79 | 11:28 |
| afazekas | Usually just the parameters which MUST present are without a default argument | 11:30 |
| *** woodspa has joined #openstack-qa | 11:31 | |
| afazekas | The optional things can be passwd in the kwargs as well, but in this case the doc string should contain the the valid parameter names | 11:31 |
| afazekas | in the server case the name is a MUST parameter | 11:32 |
| afazekas | will the port name be required parameter in the future ? | 11:32 |
| *** jasondotstar has quit IRC | 11:34 | |
| raies | afazekas: yes in future during port update (during port name update) | 11:36 |
| afazekas | raies: I mean on the neutron API level. AFAIK it remains an optional parameter | 11:37 |
| afazekas | in test cases using the name perfectly makes sense | 11:37 |
| raies | afazekas: I think it will not create any issue. Still it is creating an issue then I should make it optional ? | 11:38 |
| afazekas | but not always requred | 11:38 |
| afazekas | create_port(self, net_uuid, name=None, state='True') | 11:39 |
| afazekas | create_port(self, net_uuid, state='True', name=None) | 11:39 |
| afazekas | create_port(self, net_uuid, **kwargs) | 11:40 |
| afazekas | sounds good to me | 11:40 |
| raies | hmm it is good thanks | 11:40 |
| raies | I will update it soon | 11:40 |
| raies | thank u very much | 11:40 |
| raies | will you please give some input for the what I have pasted on pastebin ? | 11:41 |
| *** Anju_ has quit IRC | 11:45 | |
| *** raies has quit IRC | 11:47 | |
| *** jhenner has quit IRC | 11:53 | |
| *** psedlak_ has joined #openstack-qa | 11:54 | |
| *** psedlak has quit IRC | 11:58 | |
| afazekas | raies: I will came back in 1 hour (meeting) | 11:58 |
| *** afazekas has quit IRC | 11:58 | |
| *** jhenner has joined #openstack-qa | 11:58 | |
| *** mkollaro has joined #openstack-qa | 12:03 | |
| *** afazekas has joined #openstack-qa | 12:07 | |
| *** prekarat has left #openstack-qa | 12:07 | |
| *** LiangZheng has joined #openstack-qa | 12:09 | |
| *** HenryG has quit IRC | 12:11 | |
| *** LiangZheng has quit IRC | 12:13 | |
| *** adalbas has joined #openstack-qa | 12:15 | |
| *** afazekas is now known as afazekas_mtg | 12:15 | |
| *** HenryG has joined #openstack-qa | 12:23 | |
| *** dosaboy has quit IRC | 12:27 | |
| *** SergeyLukjanov has quit IRC | 12:33 | |
| *** afazekas_mtg has quit IRC | 12:44 | |
| *** dosaboy has joined #openstack-qa | 12:44 | |
| *** weshay has quit IRC | 12:52 | |
| *** jasondotstar has joined #openstack-qa | 12:53 | |
| *** SergeyLukjanov has joined #openstack-qa | 12:59 | |
| *** psedlak__ has joined #openstack-qa | 13:01 | |
| *** dosaboy has quit IRC | 13:02 | |
| *** mkollaro has quit IRC | 13:04 | |
| *** weshay has joined #openstack-qa | 13:05 | |
| *** mkollaro has joined #openstack-qa | 13:05 | |
| *** weshay has quit IRC | 13:05 | |
| *** psedlak_ has quit IRC | 13:05 | |
| *** jhenner1 has joined #openstack-qa | 13:05 | |
| *** weshay has joined #openstack-qa | 13:06 | |
| *** mkollaro has quit IRC | 13:06 | |
| *** dosaboy has joined #openstack-qa | 13:06 | |
| *** mkollaro has joined #openstack-qa | 13:06 | |
| *** jhenner has quit IRC | 13:06 | |
| *** LiangZheng has joined #openstack-qa | 13:10 | |
| *** afazekas has joined #openstack-qa | 13:11 | |
| *** LiangZheng has quit IRC | 13:14 | |
| *** mriedem has joined #openstack-qa | 13:25 | |
| *** burt has joined #openstack-qa | 13:35 | |
| yfried | can anyone help me? i'm trying to run a simple test of tempest and get this error: | 13:46 |
| yfried | File "/root/tempest/tempest/common/rest_client.py", line 478, in _error_checker | 13:46 |
| yfried | raise exceptions.RestClientException(str(resp.status)) | 13:46 |
| yfried | RestClientException: An unknown exception occurred | 13:46 |
| yfried | Details: 503 | 13:46 |
| *** yfried has quit IRC | 13:52 | |
| mriedem | afazekas: testing your setup/teardown patch now | 14:04 |
| mriedem | on py26 | 14:04 |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Configure bash completion https://review.openstack.org/41928 | 14:07 |
| mriedem | afazekas: it works, resolves bug 1212071 | 14:07 |
| mtreinish | afazekas: cool you might want to add the bug to the commit message then while you're fixing up the typo mriedem pointed out anyway. | 14:09 |
| *** LiangZheng has joined #openstack-qa | 14:10 | |
| mriedem | if it weren't for the drugs i took last night i would have lost sleep over this py26 thing :) | 14:11 |
| afazekas | ok | 14:13 |
| *** LiangZheng has quit IRC | 14:15 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: setUpClass/tearDownClass full chain https://review.openstack.org/39346 | 14:15 |
| afazekas | is it ok now ? | 14:16 |
| afazekas | mriedem, mtreinish: ^^^ | 14:16 |
| mtreinish | afazekas: I'm not sure that it got picked up the bug listing doesn't show as in progress. | 14:17 |
| mriedem | afazekas: i think you should just have: Closes-Bug: #1212071 | 14:17 |
| mriedem | right | 14:17 |
| mriedem | drop the "This also" | 14:17 |
| *** anteaya has joined #openstack-qa | 14:22 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: setUpClass/tearDownClass full chain https://review.openstack.org/39346 | 14:27 |
| mriedem | afazekas: beautiful | 14:27 |
| *** adalbas has quit IRC | 14:27 | |
| *** SergeyLukjanov has quit IRC | 14:28 | |
| *** SergeyLukjanov has joined #openstack-qa | 14:29 | |
| mkoderer | afazekas: ping | 14:32 |
| afazekas | mkoderer: pong | 14:33 |
| afazekas | mtreinish: looks like now it picks up | 14:34 |
| mkoderer | afazekas: what's our current rule for skipped test - do we accept them to be merged, or do we wait until the bug is fixed? | 14:34 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add an option to print out nova request ids https://review.openstack.org/41934 | 14:34 |
| afazekas | the hacking guide warns everybody if the change just contains a spiked test it could be rejected | 14:35 |
| afazekas | mkoderer: does it answers your question ? Do you have a specific review ? | 14:37 |
| mkoderer | afazekas: yes this one https://review.openstack.org/#/c/41635/5 | 14:37 |
| mkoderer | it introduces a skip | 14:37 |
| mkoderer | I would -1 it | 14:38 |
| mriedem | mkoderer: i would too at least on the groups that the commit doesn't say it does | 14:38 |
| mriedem | *does a skip | 14:38 |
| mriedem | not really adding a test - they are taking one away | 14:38 |
| mkoderer | mriedem: that's right.... | 14:38 |
| mriedem | mkoderer: i -1'ed it | 14:40 |
| mkoderer | mriedem: ok, I will add a link to the hacking guide | 14:41 |
| afazekas | As mriedem pointed out it is not just an adding a skiped test case, it removes one | 14:42 |
| mriedem | mkoderer: as for an example of a patch that skips tests: https://review.openstack.org/#/c/40666/ | 14:43 |
| *** yfried has joined #openstack-qa | 14:43 | |
| *** adalbas has joined #openstack-qa | 14:43 | |
| mkoderer | mriedem: I see - so if a existing test is broken we allow it but only in that case | 14:45 |
| afazekas | mriedem: as per the previous QA meeting, probably we will skip some other test cases in the neutron job | 14:45 |
| afazekas | mkoderer: I would be ok adding 3 tests with one skipped one, but may be I am alone with this. | 14:46 |
| mriedem | mkoderer: i think different people have different opinions on when it's OK to skip, i know sdague has had different criteria when i've asked him before | 14:48 |
| mriedem | mkoderer: for the ones i pushed for those broken neutron tests, i thought it was justified because (1) they had been broken for awhile (2) there were already other nova tests skipped if neutron was available and (3) there were abandoned reviews to fix the bug based on arguments over how to fix, i.e. no end in sight for the bug fix. | 14:49 |
| mkoderer | Ok I see :) | 14:55 |
| *** tkammer has quit IRC | 14:56 | |
| mkoderer | I remember that we had a discussion in a QA meeting about the isssue | 14:56 |
| mkoderer | mtreinish: ping | 14:56 |
| mtreinish | mkoderer: pong | 14:56 |
| mkoderer | about https://review.openstack.org/#/c/41934/1 | 14:56 |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: added new tag for development packages https://review.openstack.org/38127 | 14:57 |
| mkoderer | what is the reason of using "print" and not the logger? | 14:57 |
| mkoderer | since you need it on stdout? | 14:57 |
| mtreinish | mkoderer: yeah its for the failure case | 14:58 |
| mtreinish | with testr we capture stdout and it only prints on failure | 14:58 |
| mtreinish | so if we have a test failure it'll print the request ids vercy nicely right before the traceback | 14:58 |
| mtreinish | I also changed the logging to make it more explicit in there, but even then it's still tricky to figure out which tests go with the request id | 14:59 |
| mkoderer | mtreinish: ok I thought using "print" is a evil thing but need to have a closer look | 14:59 |
| mtreinish | mkoderer: in general it should be avoided, but this is a special case :) | 15:00 |
| mtreinish | but that is why I made it a config option | 15:00 |
| *** SergeyLukjanov has quit IRC | 15:01 | |
| *** SergeyLukjanov has joined #openstack-qa | 15:03 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Cleanup try/except/finally blocks in several tests https://review.openstack.org/40005 | 15:03 |
| mriedem | mtreinish: should that go in the logging.conf? | 15:05 |
| mkoderer | mriedem: that what I am thinking about | 15:07 |
| mriedem | i honestly don't know if that would work though | 15:07 |
| mkoderer | I mean you could register the stream handler just for this class | 15:08 |
| *** mkollaro has quit IRC | 15:08 | |
| mkoderer | as I did for the stress test thing | 15:09 |
| mkoderer | but this will log every output to stdout for the whole package | 15:09 |
| mkoderer | mtreinish: ^ | 15:10 |
| mkoderer | maybe a "print" is the easiest solution | 15:10 |
| *** LiangZheng has joined #openstack-qa | 15:11 | |
| mtreinish | mkoderer: yeah that's why I did it with a print. We just want the req-id on stdout for failures. | 15:11 |
| mtreinish | we don't want to just dump everything into stdout | 15:12 |
| mriedem | yeah, we already had that problem a few weeks ago :( | 15:12 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/devstack-gate: Add ability to do smart backwards compat testing https://review.openstack.org/41945 | 15:13 |
| *** LiangZheng has quit IRC | 15:15 | |
| *** dosaboy has quit IRC | 15:24 | |
| *** dosaboy has joined #openstack-qa | 15:26 | |
| mriedem | cyeoh: ping | 15:39 |
| mriedem | i don't see any other cores around but this already had a +2 from cyeoh and sdague, and has a +2 from mtreinish: https://review.openstack.org/#/c/39346/ | 15:41 |
| mriedem | it had to be rebased and clean up a few nits | 15:41 |
| mtreinish | mriedem: I'm waiting for jenkins to come back on it | 15:42 |
| mriedem | mtreinish: ah, ok | 15:42 |
| mriedem | thanks | 15:42 |
| mtreinish | jaypipes: ^^^ unless you want to do it | 15:42 |
| mtreinish | mriedem: both sdague and dkranz are out this week so some reviews might be slow | 15:43 |
| mriedem | mtreinish: k, i knew sean was out | 15:44 |
| mriedem | i swear this is the only thing i'll care about the rest of the week :) | 15:44 |
| afazekas | cyeoh: https://review.openstack.org/#/c/39346/ | 15:59 |
| afazekas | can you review it | 15:59 |
| *** torandu has joined #openstack-qa | 16:01 | |
| *** LiangZheng has joined #openstack-qa | 16:12 | |
| *** hemnafk is now known as hemna | 16:13 | |
| *** mlavalle has joined #openstack-qa | 16:14 | |
| *** LiangZheng has quit IRC | 16:16 | |
| afazekas | mtreinish: just on comment change you usually loose the +1 from jenkins, but it will not test it again automatically | 16:19 |
| mriedem | afazekas: i have a feeling that cyeoh is gone by now | 16:20 |
| *** yfried has quit IRC | 16:24 | |
| *** jgallard has quit IRC | 16:24 | |
| *** SergeyLukjanov has quit IRC | 16:26 | |
| *** fbo is now known as fbo_away | 16:49 | |
| *** SergeyLukjanov has joined #openstack-qa | 16:58 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: added new tag for testonly packages in lists https://review.openstack.org/41975 | 16:58 |
| openstackgerrit | Christian Berendt proposed a change to openstack-dev/devstack: added new tag for testonly packages in lists https://review.openstack.org/38127 | 17:00 |
| *** jhenner1 has quit IRC | 17:11 | |
| *** LiangZheng has joined #openstack-qa | 17:12 | |
| *** LiangZheng has quit IRC | 17:17 | |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Use a global virtualenv https://review.openstack.org/40534 | 17:21 |
| *** yfried has joined #openstack-qa | 17:35 | |
| mriedem | mtreinish: jaypipes: this passed jenkins now: https://review.openstack.org/#/c/39346/ | 17:39 |
| jaypipes | mriedem: k, lemme review... thx for the heads up | 17:40 |
| jaypipes | mriedem, afazekas: approved above. very nice work, Attila. | 17:45 |
| mriedem | jaypipes: thanks | 17:45 |
| afazekas | jaypipes: thank you | 17:46 |
| *** dosaboy has quit IRC | 18:08 | |
| *** LiangZheng has joined #openstack-qa | 18:13 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Unit tests as stress tests https://review.openstack.org/38980 | 18:15 |
| *** LiangZheng has quit IRC | 18:18 | |
| openstackgerrit | A change was merged to openstack/tempest: Cleanup try/except/finally blocks in several tests https://review.openstack.org/40005 | 18:20 |
| *** dosaboy_1 has joined #openstack-qa | 18:23 | |
| *** afazekas has quit IRC | 18:25 | |
| *** dosaboy__ has quit IRC | 18:25 | |
| *** lifeless has quit IRC | 18:28 | |
| *** psedlak__ has quit IRC | 18:29 | |
| *** jerryz has joined #openstack-qa | 18:29 | |
| jerryz | hi everyone. my devstack has been breaking for a few days. updating code of each project does not help. either nova-api can not start due to AttributeError: 'module' object has no attribute 'DeprecatedOpt' or when i unsintalled my oslo.config , keystone-mange sync-db failed 2013-08-09 18:55:16 from oslo.config import cfg 2013-08-09 18:55:16 ImportError: No module named config 2013-08-09 18:55:16 + [[ PKI == \P\K\I ] | 18:33 |
| jerryz | could anyone help? thanks | 18:33 |
| openstackgerrit | Derek Morton proposed a change to openstack-dev/devstack: Fixes install on XenServer/XCP https://review.openstack.org/41985 | 18:35 |
| *** thomasbiege has joined #openstack-qa | 18:41 | |
| *** thomasbiege has quit IRC | 18:48 | |
| *** psedlak__ has joined #openstack-qa | 18:53 | |
| *** psedlak__ has quit IRC | 18:59 | |
| *** torandu has quit IRC | 19:04 | |
| *** torandu has joined #openstack-qa | 19:11 | |
| *** torandu has quit IRC | 19:13 | |
| *** torandu has joined #openstack-qa | 19:13 | |
| *** LiangZheng has joined #openstack-qa | 19:14 | |
| *** LiangZheng has quit IRC | 19:19 | |
| *** dosaboy_1 is now known as dosaboy | 19:23 | |
| *** thomasbiege has joined #openstack-qa | 19:25 | |
| mriedem | new gate check bug: https://bugs.launchpad.net/tempest/+bug/1212405 | 19:28 |
| *** thomasbiege has quit IRC | 19:32 | |
| *** lifeless has joined #openstack-qa | 19:39 | |
| *** afazekas has joined #openstack-qa | 19:41 | |
| *** SergeyLukjanov has quit IRC | 19:58 | |
| *** afazekas has quit IRC | 20:02 | |
| mtreinish | mriedem: i saw that once on a parallel run and I just assumed it was a parallel race | 20:10 |
| mtreinish | but it looks like something else is going on | 20:10 |
| mriedem | mtreinish: i've seen it 3 times today | 20:10 |
| mriedem | http://i582.photobucket.com/albums/ss265/zknizek/Family-Guy-Puke-A-Thon-family-guy-2.gif | 20:10 |
| mtreinish | mriedem: heh | 20:11 |
| mtreinish | man, that test is pretty simple too, it just tries to login to horizon using the config provided creds | 20:12 |
| mriedem | hence the puking | 20:13 |
| mriedem | she's failing hard | 20:13 |
| mriedem | mtreinish: i raised the bug over in #openstack-horizon and no one bit | 20:14 |
| *** LiangZheng has joined #openstack-qa | 20:15 | |
| *** whayutin_ has joined #openstack-qa | 20:16 | |
| *** LiangZheng has quit IRC | 20:20 | |
| *** weshay has quit IRC | 20:20 | |
| *** whayutin_ is now known as weshay | 20:23 | |
| *** adalbas has quit IRC | 20:23 | |
| *** thomasbiege has joined #openstack-qa | 20:36 | |
| *** thomasbiege has quit IRC | 20:36 | |
| *** adalbas has joined #openstack-qa | 20:40 | |
| *** jhenner has joined #openstack-qa | 20:57 | |
| *** woodspa has quit IRC | 21:03 | |
| *** LiangZheng has joined #openstack-qa | 21:15 | |
| *** thomasbiege has joined #openstack-qa | 21:20 | |
| *** LiangZheng has quit IRC | 21:20 | |
| *** thomasbiege has quit IRC | 21:23 | |
| *** fbo_away is now known as fbo | 21:27 | |
| *** thomasbiege has joined #openstack-qa | 21:31 | |
| *** mriedem has quit IRC | 21:42 | |
| *** thomasbiege has quit IRC | 21:46 | |
| openstackgerrit | Maru Newby proposed a change to openstack/tempest: Remove skip of neutron connectivity check. https://review.openstack.org/42018 | 21:57 |
| marun | ^^ Reviews appreciated so that the neutron gate can run again. | 21:59 |
| *** anteaya has quit IRC | 22:00 | |
| mtreinish | marun: ok it looks fine to me, but I'll wait for jenkins to come back before giving it +2 | 22:00 |
| marun | mtreinish: sounds good. | 22:00 |
| marun | mtreinish: It tests out fine locally, but I definitely don't want to merge something that doesn't pass the gate. | 22:01 |
| *** burt has quit IRC | 22:05 | |
| *** yfried has quit IRC | 22:07 | |
| *** weshay has quit IRC | 22:09 | |
| *** jerryz has quit IRC | 22:12 | |
| *** jasondotstar has quit IRC | 22:13 | |
| openstackgerrit | Maru Newby proposed a change to openstack/tempest: Remove skip of neutron connectivity check. https://review.openstack.org/42018 | 22:15 |
| *** LiangZheng has joined #openstack-qa | 22:16 | |
| *** LiangZheng has quit IRC | 22:20 | |
| mtreinish | marun: ^^^ actually that'll probably fail pep8 because of the period in the commit title | 22:24 |
| marun | *sigh* | 22:25 |
| * marun shakes fist at jog0 | 22:25 | |
| marun | Style nazis can bit my shiny metal ass. | 22:26 |
| openstackgerrit | Maru Newby proposed a change to openstack/tempest: Remove skip of neutron connectivity check https://review.openstack.org/42018 | 22:27 |
| openstackgerrit | Maru Newby proposed a change to openstack/tempest: Remove skip of neutron connectivity check https://review.openstack.org/42018 | 22:27 |
| marun | If we had half as much attention to actual quality issues, we'd be laughing. :D | 22:27 |
| marun | (thanks for the heads up, though, mtreinish) | 22:28 |
| *** giulivo has quit IRC | 22:29 | |
| *** jhenner has quit IRC | 22:31 | |
| mtreinish | marun: yeah I've been caught in that trap too many times myself since we moved to the new hacking in tempest | 22:31 |
| mtreinish | I now flinch and triple check the commit message before I push things | 22:32 |
| marun | mtreinish: I think this kind of rigor has negative value, frankly. If this is enabled in neutron, I'm going to vote to have the checking disabled. | 22:34 |
| marun | It's pointless mechanical effort that contributes exactly nothing to quality. | 22:34 |
| marun | Warnings, fine. But it shouldn't result in a failure. | 22:35 |
| marun | And it shouldn't require running tox post-commit. | 22:35 |
| marun | mtreinish: Ah, you know where this would actually be useful? | 22:36 |
| marun | mtreinish: The commit message should be checked by git-review. | 22:36 |
| marun | mtreinish: No mechanical checking or expensive tox runs required. Just do a mechanical check automatically before posting. | 22:37 |
| *** mlavalle has quit IRC | 22:40 | |
| mtreinish | marun: that's not a bad idea. We might be able to do it at the project level by forcing a git hook too I think. | 22:42 |
| marun | mtreinish: +1 | 22:43 |
| *** jasondotstar has joined #openstack-qa | 23:10 | |
| *** LiangZheng has joined #openstack-qa | 23:16 | |
| *** LiangZheng has quit IRC | 23:21 | |
| *** jerryz_ has joined #openstack-qa | 23:27 | |
| *** sthaha has joined #openstack-qa | 23:28 | |
| *** sthaha has joined #openstack-qa | 23:28 | |
| *** fbo is now known as fbo_away | 23:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
