| *** sthaha has joined #openstack-qa | 00:23 | |
| *** sthaha has joined #openstack-qa | 00:23 | |
| openstackgerrit | andrea-frittoli proposed a change to openstack/tempest: Whitebox server expects deleted > 0 https://review.openstack.org/40129 | 00:24 |
|---|---|---|
| openstackgerrit | Ivan-Zhu proposed a change to openstack-dev/grenade: add nova v3 api support in upgrade https://review.openstack.org/40134 | 01:03 |
| *** woodspa has quit IRC | 02:19 | |
| openstackgerrit | Ian Wienand proposed a change to openstack-dev/devstack: Use a dummy rpm for Fedora https://review.openstack.org/39862 | 03:02 |
| *** HenryG has quit IRC | 03:17 | |
| *** prekarat has joined #openstack-qa | 03:18 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack-dev/grenade: add nova v3 api support in upgrade https://review.openstack.org/40134 | 03:30 |
| *** prekarat1 has joined #openstack-qa | 03:30 | |
| *** prekarat has quit IRC | 03:33 | |
| *** SergeyLukjanov has joined #openstack-qa | 03:39 | |
| openstackgerrit | Prem Karat proposed a change to openstack/tempest: Add nova v3 tests for test_servers https://review.openstack.org/40147 | 03:51 |
| *** timello is now known as timello_vac | 03:55 | |
| openstackgerrit | Prem Karat proposed a change to openstack/tempest: Add nova-v3-api tests for create_servers https://review.openstack.org/40154 | 04:01 |
| *** prekarat1 has quit IRC | 04:10 | |
| *** prekarat has joined #openstack-qa | 04:13 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Convert Nova's virt driver to hypervisor plugins https://review.openstack.org/40156 | 04:49 |
| openstackgerrit | raiesmh08 proposed a change to openstack/tempest: Add network api test cases https://review.openstack.org/39023 | 04:59 |
| *** raies has joined #openstack-qa | 05:02 | |
| raies | jaypipes: I have added patch 7, you had approved patch 6 , please check - https://review.openstack.org/#/c/39023/ | 05:03 |
| *** marun has joined #openstack-qa | 05:05 | |
| raies | jaypipes: r u there ? | 05:06 |
| *** sourabh_ has joined #openstack-qa | 05:08 | |
| *** sourabh_ has quit IRC | 05:11 | |
| *** afazekas_ has joined #openstack-qa | 05:17 | |
| *** Ater has joined #openstack-qa | 05:18 | |
| *** Ater has left #openstack-qa | 05:20 | |
| openstackgerrit | A change was merged to openstack/tempest: Negative tests added for network https://review.openstack.org/40049 | 05:28 |
| raies | jaypipes: hi | 05:39 |
| *** SergeyLukjanov has quit IRC | 05:40 | |
| openstackgerrit | Prem Karat proposed a change to openstack/tempest: Add nova-v3-api tests for flavors https://review.openstack.org/40166 | 06:41 |
| openstackgerrit | Prem Karat proposed a change to openstack/tempest: Add nova-v3-api tests for flavors https://review.openstack.org/40166 | 06:43 |
| *** bashok has joined #openstack-qa | 06:43 | |
| *** tkammer has joined #openstack-qa | 06:52 | |
| afazekas_ | raies: looks like it is not trivial rebase now | 06:56 |
| afazekas_ | git checkout master | 06:57 |
| afazekas_ | git pull | 06:57 |
| afazekas_ | git review -d 39023 | 06:57 |
| afazekas_ | git rebase master | 06:58 |
| afazekas_ | raies: I can try to approve it , but I think it will have merge conflicts | 07:00 |
| afazekas_ | git add <conflicting file after edit> | 07:03 |
| afazekas_ | git rebase --continue | 07:04 |
| afazekas_ | dkranz,sdague: FIXME: after a rebase issue one +2 vote is enough | 07:05 |
| afazekas_ | Can I approve it alone in this case ? | 07:06 |
| *** jhenner has joined #openstack-qa | 07:06 | |
| Anju | afazekas: may ia dd atest case for start and stopping a server | 07:07 |
| *** jhenner has quit IRC | 07:08 | |
| afazekas_ | Anju: I do not know about any existing test case which starts the server after a stop | 07:08 |
| afazekas_ | So it can be ok | 07:09 |
| Anju | yeah it is not currently avaialble | 07:10 |
| *** giulivo has joined #openstack-qa | 07:15 | |
| afazekas_ | raies: If you want I can try to re-approve it: https://review.openstack.org/#/c/39023/7 | 07:28 |
| afazekas_ | Jenkis said it is a trivial rebase, I hope he is correct | 07:29 |
| kashyap | afazekas_, Hey, do you know why the Cirros image boot is failing? -- http://paste.openstack.org/show/43157/ | 07:30 |
| kashyap | This is on F19, Havana m-2 bits | 07:30 |
| * kashyap goes to lunch, be back in a bit | 07:30 | |
| afazekas_ | kashyap: what was the boot command ? | 07:31 |
| *** psedlak has joined #openstack-qa | 07:31 | |
| afazekas_ | kashyap: One nova change miss uses the libguestfs , so if you try to inject a file it may fail | 07:32 |
| raies | afazekas_: yes please if u can please approve | 07:32 |
| *** psedlak has quit IRC | 07:35 | |
| *** psedlak has joined #openstack-qa | 07:35 | |
| afazekas_ | kashyap: you need to disable the new hard link security feature and run qemu as the same as the nova user, I am still waiting for info about which parts need to change, but it is very likely the nova is the sin | 07:36 |
| afazekas_ | raies: jenkins was incorrect, so you need to rebase it | 07:39 |
| *** fbo_away is now known as fbo | 07:39 | |
| afazekas_ | raies: The previous QA meeting just allows me to approve on trivial rebase, I will ask the rules about less trivial rebase | 07:41 |
| kashyap | nova boot --flavor 2 --image \ | 07:47 |
| kashyap | 38aa7e40-2672-4174-b6cf-a524945d4bec cirros-t1 | 07:47 |
| kashyap | afazekas, ^ | 07:47 |
| kashyap | No file injection here | 07:48 |
| kashyap | Why I was wondering was - this worked for me a couple of days ago with Havana m2 bits. | 07:48 |
| kashyap | afazekas_, ^ Now, checking the permissions of qemu.. | 07:49 |
| afazekas_ | kashyap: Why do you show me AMQP messages ? | 07:50 |
| kashyap | afazekas_, Um, that was the last bits from the compute.log. Didn't mean to point to AMQP messages. | 07:50 |
| afazekas_ | kashyap: you can see many AMQP messages in the log even if nothing happening :) | 07:52 |
| kashyap | afazekas_, Ah, grep for 'failure' string in that pastebin :-) | 07:52 |
| afazekas_ | Try to find the related ERROR message if there is any. | 07:53 |
| afazekas_ | "failure": null | 07:53 |
| kashyap | Yes, looking | 07:53 |
| kashyap | egrep -i 'ERROR|CRITICAL' /var/log/nova/compute.log | 07:54 |
| kashyap | 2013-08-05 02:23:39.337 9715 DEBUG nova.openstack.common.service [-] fatal_exception_format_errors = False log_opt_values /usr/lib/python2.7/site-packages/oslo/config/cfg.py:1899 | 07:54 |
| kashyap | 2013-08-05 02:23:39.947 9715 DEBUG nova.openstack.common.service [-] publish_errors = False log_opt_values /usr/lib/python2.7/site-packages/oslo/config/cfg.py:1899 | 07:54 |
| kashyap | That's pretty much what I see. | 07:56 |
| afazekas_ | kashyap: is the instance in ERROR state ? | 07:57 |
| kashyap | afazekas_, Yes. | 07:57 |
| kashyap | afazekas_, And, qemu is not running either. SELinux in permissive | 07:58 |
| kashyap | Let me try to boot a Fedora image | 07:58 |
| afazekas_ | kashyap: check the scheduler log | 07:58 |
| kashyap | Oh, this VM is small, it wouldn't work | 07:58 |
| afazekas_ | is it in immediate fail ? | 07:58 |
| afazekas_ | The image does not matters | 07:59 |
| afazekas_ | you can try to boot the password.txt :) | 07:59 |
| kashyap | afazekas_, Yes, looking. Right, it's failing in 10 secs or so (maybe because, it's using qemu) | 07:59 |
| afazekas_ | no | 07:59 |
| afazekas_ | How much memeory do you have ? | 08:00 |
| afazekas_ | is it a vm ? | 08:00 |
| kashyap | afazekas_, You know? Next I ping, just ignore me. It's the memory :( | 08:00 |
| kashyap | afazekas_, Thanks for the Scheduler log pointer. | 08:01 |
| kashyap | afazekas_, While I have your attention, do you know the resolution to that 'pip' issue with devstack? | 08:02 |
| kashyap | If I manually install it, devstack removes it. Not sure what to do here :-) | 08:02 |
| afazekas_ | git revert --no-edit 0392a10a635a8befb13ff242e2ed5d5be4b23560 | 08:04 |
| * kashyap goes to read the above commit | 08:06 | |
| *** afazekas_ is now known as afazekas_food | 08:07 | |
| *** jhenner has joined #openstack-qa | 08:11 | |
| *** jhenner has quit IRC | 08:26 | |
| *** jhenner has joined #openstack-qa | 08:32 | |
| *** jhenner has quit IRC | 08:46 | |
| openstackgerrit | Zhang Hua proposed a change to openstack-dev/devstack: enable apache2 server as front end for keystone https://review.openstack.org/36474 | 08:48 |
| openstackgerrit | raiesmh08 proposed a change to openstack/tempest: Add network api test cases https://review.openstack.org/39023 | 08:50 |
| openstackgerrit | Zhang Hua proposed a change to openstack-dev/devstack: enable apache2 server as front end for keystone https://review.openstack.org/36474 | 08:51 |
| openstackgerrit | aguzikova proposed a change to openstack-dev/devstack: Add keystoneclient support for cinder. https://review.openstack.org/39747 | 08:52 |
| raies | afazekas_food : | 08:53 |
| raies | I have added patch 8 | 08:53 |
| raies | please check | 08:53 |
| *** prekarat1 has joined #openstack-qa | 09:06 | |
| *** prekarat has quit IRC | 09:06 | |
| *** tkammer has quit IRC | 09:08 | |
| *** lifeless has quit IRC | 09:10 | |
| *** fbo is now known as fbo_away | 09:11 | |
| *** afazekas_ has joined #openstack-qa | 09:13 | |
| *** fbo_away is now known as fbo | 09:14 | |
| raies | afazekas_: please check https://review.openstack.org/#/c/39023/ | 09:17 |
| afazekas_ | since neutron is non-voting job I need wait until the full gate says it is ok | 09:18 |
| *** lifeless has joined #openstack-qa | 09:20 | |
| Anju | afazekas: are u able to run this cli neutron net-update ?? | 09:21 |
| Anju | according to help i gave the arguments to update a network id and its new name | 09:22 |
| Anju | but afazekas: it is saying :Must specify new values to update network | 09:22 |
| raies | afazekas: I have added not patch 8 which includes codes which were causing conflict | 09:26 |
| afazekas_ | raies: after the full neutron gate job says this part is ok I'll check it | 09:27 |
| *** tkammer has joined #openstack-qa | 09:28 | |
| raies | afazekas_: ok | 09:29 |
| raies | I will let u know soon | 09:29 |
| *** SergeyLukjanov has joined #openstack-qa | 09:38 | |
| raies | afazekas_: you can now check | 09:38 |
| *** dosaboy_ has joined #openstack-qa | 09:40 | |
| *** dosaboy has quit IRC | 09:42 | |
| afazekas_ | Anju: are you trying to change the network uuid ? | 09:46 |
| Anju | noooo | 09:47 |
| Anju | i select a network using its id | 09:47 |
| Anju | and try to upadte it | 09:47 |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Added a test for stop and start a server https://review.openstack.org/40191 | 10:11 |
| mkoderer | afazekas: in which case assertTrue() should be avoided.. always? | 10:13 |
| mkoderer | afazekas_: ^^ | 10:13 |
| *** tkammer has quit IRC | 10:15 | |
| *** prekarat1 has quit IRC | 10:26 | |
| *** tkammer has joined #openstack-qa | 10:28 | |
| raies | jaypipes: please check for approval https://review.openstack.org/#/c/39023/8 | 10:35 |
| *** key has joined #openstack-qa | 10:41 | |
| *** prekarat has joined #openstack-qa | 10:47 | |
| *** prekarat has quit IRC | 11:03 | |
| afazekas_ | Anju: what is your exact command, is the network really exists , do you have admin permissions ? | 11:19 |
| afazekas_ | mkoderer: For example if the response is a boolean , in this case it is ok | 11:20 |
| afazekas_ | mkoderer: for sets and list it is not ok | 11:20 |
| mkoderer | ok I already put a command here https://review.openstack.org/#/c/37179/5/tempest/api/identity/admin/v3/test_roles.py | 11:21 |
| mkoderer | I think this is a case where it is not ok | 11:21 |
| mkoderer | since it is a set | 11:21 |
| mkoderer | afazekas_: ^^ | 11:21 |
| mkoderer | ^comment not command ;) | 11:23 |
| raies | jaypipes: please check for approval https://review.openstack.org/#/c/39023/8 | 11:26 |
| *** woodspa has joined #openstack-qa | 11:34 | |
| afazekas_ | mkoderer: that expression should be translated to testtools matcher | 11:35 |
| afazekas_ | we do not have to much in the code yet... | 11:36 |
| afazekas_ | The one is not used fully correctly | 11:36 |
| * afazekas_ I'll be back in 1 hour | 11:36 | |
| mkoderer | giulivo: ping | 11:48 |
| giulivo | pong mkoderer | 11:50 |
| mkoderer | giulivo: what if we introduce a new decorator for stress test | 11:50 |
| mkoderer | so if a author want's to add the certain test to the stress test framework he just could use this decorator | 11:51 |
| mkoderer | like @stress-test | 11:51 |
| *** mkollaro has joined #openstack-qa | 11:52 | |
| giulivo | I've no idea how to implement that, but would +1 the feature | 11:53 |
| mkoderer | giulivo: I mean we just add this decorator and we keep the json files | 11:54 |
| mkoderer | if you specify a class/function without this decorator you get a warning | 11:55 |
| mkoderer | we could add a script that generates one json file with all possible actions by search for the decorator | 11:55 |
| *** weshay has joined #openstack-qa | 11:56 | |
| giulivo | mkoderer, only I wonder, what is the purpose of the decorator_ | 11:56 |
| giulivo | ? | 11:57 |
| giulivo | I thought one wouldn't have to create any json file for that to work (maybe we could pass the threads number as argument to the decorator) | 11:58 |
| mkoderer | yes right.. thats also possible | 11:58 |
| mkoderer | the decorator is the simply define that this method runs as stress test (and makes sense as stress test) | 11:59 |
| mkoderer | giulivo: but yes maybe we could skip the json file completely | 12:00 |
| mkoderer | ok it's just an idea - let me create a prototype and we will see ;) | 12:01 |
| *** HenryG has joined #openstack-qa | 12:25 | |
| *** adalbas has quit IRC | 12:28 | |
| *** krtaylor has quit IRC | 12:40 | |
| *** jasondotstar has joined #openstack-qa | 12:43 | |
| *** adalbas has joined #openstack-qa | 12:44 | |
| openstackgerrit | Sascha Peilicke proposed a change to openstack-dev/devstack: Don't install nodejs, horizon moved to lesscpy https://review.openstack.org/40218 | 12:50 |
| *** dosaboy_ has quit IRC | 12:51 | |
| *** dosaboy has joined #openstack-qa | 12:52 | |
| *** prekarat has joined #openstack-qa | 13:08 | |
| *** mriedem has joined #openstack-qa | 13:24 | |
| *** krtaylor has joined #openstack-qa | 13:32 | |
| openstackgerrit | Prem Karat proposed a change to openstack/tempest: Add nova v3 tests for keypairs https://review.openstack.org/40224 | 13:38 |
| openstackgerrit | A change was merged to openstack/tempest: Add network api test cases https://review.openstack.org/39023 | 13:41 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port instance_actions and server_list tests into v3 part1 https://review.openstack.org/40225 | 13:46 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port instance_actions and server_list tests into v3 part2 https://review.openstack.org/40226 | 13:48 |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Test cases for Roles V3 Actions https://review.openstack.org/37179 | 13:57 |
| *** adalbas has quit IRC | 14:02 | |
| *** bashok has quit IRC | 14:05 | |
| *** fbo is now known as fbo_away | 14:12 | |
| *** afazekas_ has quit IRC | 14:13 | |
| *** adalbas has joined #openstack-qa | 14:18 | |
| *** fbo_away is now known as fbo | 14:21 | |
| *** dkranz has quit IRC | 14:21 | |
| *** burt has joined #openstack-qa | 14:26 | |
| *** key4 has joined #openstack-qa | 14:27 | |
| *** key has quit IRC | 14:28 | |
| giulivo | jaypipes, thanks for the vote but don't approve it yet please | 14:32 |
| giulivo | I'd really like other people to jump in that to see what is the general idea of what we should or should not do in a scenario test | 14:32 |
| jaypipes | giulivo: I did not :) | 14:37 |
| *** dosaboy has quit IRC | 14:55 | |
| *** dosaboy has joined #openstack-qa | 14:55 | |
| *** mlavalle has joined #openstack-qa | 14:56 | |
| *** AT has joined #openstack-qa | 15:00 | |
| *** dkranz has joined #openstack-qa | 15:19 | |
| *** sthaha has quit IRC | 15:22 | |
| mtreinish | sdague, dkranz, jaypipes, afazekas: can I get some eyes on: https://review.openstack.org/39099 and https://review.openstack.org/39539 they're the 2 big failures in the testr runs right now. | 15:29 |
| jaypipes | mtreinish: yup. | 15:29 |
| mtreinish | jaypipes: great thanks | 15:30 |
| jaypipes | mtreinish: "A security group rule can only be added the same to security group once" ? | 15:30 |
| jaypipes | mtreinish: do you mean "A security group rule can only be added to the same security group once"? | 15:30 |
| mtreinish | jaypipes: oops, ok I'll fix the typo and repush it | 15:31 |
| jaypipes | mtreinish: ok, sounds good... | 15:33 |
| jaypipes | mtreinish: sorry, just wanted to make sure that I understood you properly. | 15:33 |
| sdague | mtreinish: ok, reviewed | 15:34 |
| dkranz | mtreinish: Looking... | 15:35 |
| mtreinish | jaypipes: nothing to apologize for it needed to be fixed. It wouldn't be a problem if I could write well... | 15:36 |
| mtreinish | sdague, dkranz: ok thanks | 15:36 |
| sdague | mtreinish: ok, I'm +2 on one and -1 on the other, for reasons inline | 15:38 |
| dkranz | mtreinish: For the host_aggregates, the last negative test involves actually adding a host before the "negative" part. Doesn't that one need the same treatment? | 15:40 |
| mtreinish | sdague: actually your -1 comment led me to a bug. OfficialClientManager isn't a test class so tearDownClass won't automatically run... (I'll fix that) | 15:42 |
| mriedem | afazekas: ping | 15:42 |
| mtreinish | dkranz: not sure let me look | 15:43 |
| mtreinish | dkranz: yep looks that way, I'll respin it and add the lock | 15:44 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-dev/devstack: Run Neutron migrations after creating schema https://review.openstack.org/40243 | 15:56 |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/grenade: add oslo and infra from git to the upgrade https://review.openstack.org/39574 | 15:59 |
| *** prekarat has quit IRC | 16:00 | |
| *** prekarat has joined #openstack-qa | 16:05 | |
| giulivo | ping mkoderer | 16:17 |
| *** tkammer has quit IRC | 16:17 | |
| *** SergeyLukjanov has quit IRC | 16:17 | |
| *** psedlak has quit IRC | 16:22 | |
| *** dosaboy has quit IRC | 16:25 | |
| *** dosaboy has joined #openstack-qa | 16:27 | |
| openstackgerrit | James Kyle proposed a change to openstack-dev/devstack: Ensures cross compatibility for zsh and bash. https://review.openstack.org/39971 | 16:32 |
| *** fbo is now known as fbo_away | 16:33 | |
| *** jhenner has joined #openstack-qa | 16:36 | |
| *** weshay has quit IRC | 16:40 | |
| *** jasondotstar has quit IRC | 16:48 | |
| *** bearhands is now known as comstud | 16:54 | |
| *** psedlak has joined #openstack-qa | 16:55 | |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/grenade: add oslo and infra from git to the upgrade https://review.openstack.org/39574 | 17:01 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add tenant isolation for scenario tests https://review.openstack.org/39099 | 17:04 |
| *** AT has left #openstack-qa | 17:06 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add locks for all aggregates tests with hosts https://review.openstack.org/39539 | 17:06 |
| *** jasondotstar has joined #openstack-qa | 17:20 | |
| *** SergeyLukjanov has joined #openstack-qa | 17:22 | |
| *** giulivo has quit IRC | 17:25 | |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/grenade: add oslo and infra from git to the upgrade https://review.openstack.org/39574 | 17:27 |
| mtreinish | fun looks like there is another new aggregates fail now. This ones even stranger: http://logs.openstack.org/23/39723/11/check/gate-tempest-devstack-vm-testr-full/ea37975/console.html | 17:29 |
| dkranz | sdague: I can never remember this. Is it ok to approve a patch before the initial Jenkins +1? | 17:29 |
| sdague | dkranz: no :) | 17:29 |
| sdague | it can break the gate | 17:30 |
| sdague | let jenkins +1 first | 17:30 |
| dkranz | sdague: OK. Perhaps not possible but it would be nice if the messages about new patches in the channel showed after that +1 | 17:31 |
| mtreinish | jog0: ^^ any ideas on that one. I don't know why add_host wouldn't exist for some tests but other ones pass. (also the code quite clearly has add_host in the _actions dict) | 17:32 |
| mtreinish | dkranz: yeah I think that you should be able to change the gerritbot config to enable that. But, I think that may be bordering on being a bit too chatty. | 17:33 |
| mtreinish | dkranz: oh, nm I misread what you were looking for. I don't know if gerritbot can conditionally print new patchset given a jenkins response. | 17:35 |
| *** bdperkin_gone has joined #openstack-qa | 17:40 | |
| *** bdperkin_gone is now known as bdperkin | 17:40 | |
| *** bdperkin has quit IRC | 17:42 | |
| *** bdperkin has joined #openstack-qa | 17:42 | |
| mtreinish | jog0: what's even more fun about that fail is that enabling locking on all tests with add_host fixes it | 17:42 |
| *** burt has quit IRC | 17:43 | |
| jog0 | mtreinish: will look ina moment | 17:45 |
| jog0 | mtreinish: very strange | 18:12 |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/grenade: add oslo and infra from git to the upgrade https://review.openstack.org/39574 | 18:14 |
| jog0 | mtreinish: I think the KeyError exception logic is too general | 18:14 |
| jog0 | and is catching something else | 18:15 |
| jog0 | mtreinish: to test I would just remove the try except block in nova/api/openstack/compute/contrib/aggregates.py:228 | 18:15 |
| jog0 | and rerun | 18:15 |
| jog0 | and see the error really is | 18:15 |
| openstackgerrit | Kyle Mestery proposed a change to openstack-dev/devstack: Correctly setup ML2 mechanism_drivers https://review.openstack.org/40260 | 18:25 |
| *** krtaylor has quit IRC | 18:26 | |
| *** fbo_away is now known as fbo | 18:35 | |
| openstackgerrit | A change was merged to openstack/tempest: Remove skip code for test_servers_whitebox as bug had been fixed https://review.openstack.org/37610 | 18:40 |
| *** krtaylor has joined #openstack-qa | 18:51 | |
| *** burt has joined #openstack-qa | 18:55 | |
| openstackgerrit | A change was merged to openstack/tempest: Skip secgroup rules invalid id tests until bug 1182384 is fixed https://review.openstack.org/40015 | 18:58 |
| *** mkollaro has quit IRC | 19:02 | |
| *** hemnafk is now known as hemna | 19:08 | |
| *** mkollaro has joined #openstack-qa | 19:20 | |
| openstackgerrit | A change was merged to openstack-dev/grenade: Use pip -e in setup_develop https://review.openstack.org/40022 | 19:22 |
| *** psedlak has quit IRC | 19:32 | |
| *** mriedem has quit IRC | 19:34 | |
| openstackgerrit | A change was merged to openstack-dev/grenade: add oslo and infra from git to the upgrade https://review.openstack.org/39574 | 19:47 |
| *** mriedem has joined #openstack-qa | 19:49 | |
| openstackgerrit | A change was merged to openstack/tempest: Add locks for all aggregates tests with hosts https://review.openstack.org/39539 | 19:54 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Add support for running multiple n-cpus for performance testing https://review.openstack.org/39851 | 19:55 |
| mtreinish | jog0: looks like it was from https://review.openstack.org/#/c/36651/ see: http://paste.openstack.org/show/43228/ | 20:01 |
| mtreinish | I'll push out a fix to handle that | 20:01 |
| openstackgerrit | A change was merged to openstack/tempest: Whitebox server expects deleted > 0 https://review.openstack.org/40129 | 20:03 |
| jog0 | mtreinish: ahh thanks | 20:05 |
| jog0 | mtreinish: yeah I didn't like those patches too much but they were in the name of user friendliness | 20:05 |
| jog0 | mtreinish: you fix should also get rid of that overly general try except block | 20:06 |
| mtreinish | jog0: ok, I'll change it to directly check for the action in the list up front. | 20:07 |
| jog0 | mtreinish: ++ | 20:07 |
| *** SergeyLukjanov has quit IRC | 20:07 | |
| dkranz | lifeless: Could you give (or not) final +1 to https://review.openstack.org/#/c/39716/ please? | 20:08 |
| openstackgerrit | Monty Taylor proposed a change to openstack-dev/devstack: Update projects to global requirements before setup https://review.openstack.org/35705 | 20:17 |
| mtreinish | jog0: about https://review.openstack.org/39539 it definitely does mask an issue with the logic added by https://review.openstack.org/#/c/36651/ but I'd like to get the testr runs back on track while fixing it in nova | 20:18 |
| dkranz | sdague: Can you comment on https://review.openstack.org/#/c/37194/ | 20:19 |
| jog0 | mtreinish: reasonable enough, can you file a bug or two and put comments in tempest saying so | 20:19 |
| jog0 | I just want to make sure the locks in tempest get pulled as soon as its safe | 20:19 |
| mtreinish | jog0: yeah I'll push a patch out adding comments. (maybe I can figure out a way to get it picked up by the skip tracker...) | 20:20 |
| jog0 | mtreinish: that would be nice, I was thinking that this is a new type of 'skip' that the tracker should watch | 20:22 |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: Update projects to global requirements before setup https://review.openstack.org/35705 | 20:22 |
| mtreinish | jog0: yeah, I was just thinking the same thing. I'll need to think of a good way to document in the code but at the same time make it easy to be picked up the tracker and add it as an additional check. | 20:26 |
| mtreinish | for the tracker that is | 20:26 |
| jog0 | right | 20:27 |
| openstackgerrit | A change was merged to openstack/tempest: White space after # in thirdparty https://review.openstack.org/39726 | 20:44 |
| *** fbo is now known as fbo_away | 20:46 | |
| *** fbo_away is now known as fbo | 20:47 | |
| dkranz | lifeless: Thanks for the review. Odd though. It actually failed for me with the colons. | 20:49 |
| lifeless | dkranz: I may have had them slightly wrong. Shrug :) | 20:49 |
| dkranz | lifeless: No worries. | 20:49 |
| *** rhsu has joined #openstack-qa | 20:55 | |
| *** key4 has quit IRC | 20:57 | |
| *** key has joined #openstack-qa | 20:57 | |
| *** psedlak has joined #openstack-qa | 20:58 | |
| *** psedlak has quit IRC | 20:59 | |
| afazekas | mriedem: pong | 20:59 |
| mriedem | afazekas: hey, was just going to see if i could add you to this review? https://review.openstack.org/#/c/40070/ | 21:03 |
| mriedem | afazekas: probably isn't necessary though, lots of +1's already | 21:04 |
| afazekas | mriedem: yes that review fixes an issue, but there is more there (more complex), but it makes it better than it was originally | 21:06 |
| afazekas | +1 | 21:07 |
| mriedem | afazekas: ok, thanks | 21:07 |
| * afazekas zzzzzzz | 21:11 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Convert Nova's virt driver to hypervisor plugins https://review.openstack.org/40156 | 21:12 |
| *** jasondotstar has quit IRC | 21:20 | |
| *** woodspa has quit IRC | 21:24 | |
| *** dkranz has quit IRC | 21:25 | |
| *** krtaylor has quit IRC | 21:38 | |
| *** mriedem has quit IRC | 21:43 | |
| *** fbo is now known as fbo_away | 21:45 | |
| *** mkollaro has quit IRC | 21:48 | |
| openstackgerrit | A change was merged to openstack/tempest: Make testr full runs skip tests with 'slow' tag https://review.openstack.org/39716 | 21:49 |
| openstackgerrit | aguzikova proposed a change to openstack-dev/devstack: Add keystoneclient support for cinder. https://review.openstack.org/39747 | 21:56 |
| *** burt has quit IRC | 22:04 | |
| *** jasondotstar has joined #openstack-qa | 22:12 | |
| *** mlavalle has quit IRC | 22:32 | |
| *** krtaylor has joined #openstack-qa | 22:40 | |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/devstack-gate: Use Jenkins credentials store if specified https://review.openstack.org/40310 | 22:42 |
| *** rhsu has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!