| *** nosnos has joined #openstack-qa | 01:22 | |
| *** Leo-away has left #openstack-qa | 01:48 | |
| *** jasondot1 has joined #openstack-qa | 01:49 | |
| *** jasondotstar has quit IRC | 01:50 | |
| *** jasondot1 is now known as jasondotstar | 01:50 | |
| *** jasondotstar has quit IRC | 01:50 | |
| *** jasondotstar has joined #openstack-qa | 01:50 | |
| *** matsuhashi has joined #openstack-qa | 01:56 | |
| *** jasondotstar has quit IRC | 02:44 | |
| *** anteaya has quit IRC | 02:54 | |
| *** HenryG has quit IRC | 03:30 | |
| *** yfried has joined #openstack-qa | 03:55 | |
| *** yfried has quit IRC | 04:08 | |
| *** mtreinish has quit IRC | 04:14 | |
| *** mtreinish has joined #openstack-qa | 04:15 | |
| *** ken1ohmichi has joined #openstack-qa | 04:22 | |
| *** mtreinish has quit IRC | 04:24 | |
| *** mtreinish has joined #openstack-qa | 04:26 | |
| *** raies has joined #openstack-qa | 04:38 | |
| raies | anybody (reviewer) from tempest ? | 04:42 |
|---|---|---|
| *** matsuhas_ has joined #openstack-qa | 05:00 | |
| *** matsuhashi has quit IRC | 05:00 | |
| *** yfried has joined #openstack-qa | 05:18 | |
| *** rdekel has joined #openstack-qa | 05:20 | |
| *** matsuhas_ has quit IRC | 05:26 | |
| *** matsuhashi has joined #openstack-qa | 05:29 | |
| *** nosnos has quit IRC | 05:32 | |
| *** nosnos has joined #openstack-qa | 05:32 | |
| *** matsuhashi has quit IRC | 05:34 | |
| *** matsuhashi has joined #openstack-qa | 05:39 | |
| *** dkorn has joined #openstack-qa | 05:58 | |
| *** ken1ohmichi has quit IRC | 06:18 | |
| *** ken1ohmichi has joined #openstack-qa | 06:18 | |
| *** thomasbiege has joined #openstack-qa | 06:32 | |
| *** thomasbiege has quit IRC | 06:35 | |
| *** matsuhashi has quit IRC | 06:57 | |
| *** tkammer has joined #openstack-qa | 06:59 | |
| *** nosnos_ has joined #openstack-qa | 07:00 | |
| *** nosnos has quit IRC | 07:01 | |
| *** matsuhashi has joined #openstack-qa | 07:01 | |
| mkoderer | raies: not a core reviewer but a reviewer ;) | 07:27 |
| *** jgallard has joined #openstack-qa | 07:36 | |
| *** giulivo has joined #openstack-qa | 07:39 | |
| *** yassine has joined #openstack-qa | 08:01 | |
| *** yassine has quit IRC | 08:01 | |
| *** yassine has joined #openstack-qa | 08:02 | |
| *** rdekel has quit IRC | 08:10 | |
| *** rdekel has joined #openstack-qa | 08:16 | |
| *** afazekas has joined #openstack-qa | 08:40 | |
| *** ken1ohmichi has quit IRC | 08:48 | |
| raies | afazekas: please check https://review.openstack.org/#/c/44870/ | 09:24 |
| *** matsuhashi has quit IRC | 09:26 | |
| *** matsuhashi has joined #openstack-qa | 09:26 | |
| afazekas | raies: nowadays we use the type=smoke for neutron tests | 09:28 |
| *** raies has quit IRC | 09:30 | |
| *** rdekel has quit IRC | 09:34 | |
| *** mkollaro has joined #openstack-qa | 09:41 | |
| *** raies has joined #openstack-qa | 09:51 | |
| raies | afazekas: hi | 09:51 |
| afazekas | raies: hi | 09:52 |
| afazekas | I am still thinking about would not be better to pass list of structs (dict) to the create_bulk_subnet instead of 4 list | 09:54 |
| afazekas | list({name='name-1', cidr='192.168.2.0/24'} {name='name-2', cidr='192.168.1.0/24'}) | 09:59 |
| afazekas | the count seams unnecessary parameter , you can know the list size | 10:00 |
| *** rdekel has joined #openstack-qa | 10:01 | |
| raies | afazekas: then any suggestion ? | 10:02 |
| afazekas | create_bulk_subnet(self, net_id, list_name_cidr) | 10:13 |
| giulivo | afazekas, thanks for pointing to the stable branch maint, I had no idea about the cherry pick rules | 10:14 |
| afazekas | for name_cidr in list_name_cidr: | 10:14 |
| afazekas | giulivo: https://review.openstack.org/#/c/42621/9/tempest/services/volume/xml/volumes_client.py I think it is good if it checked on the service side, we had a different opinion, so we can discuss it now | 10:16 |
| afazekas | giulivo: you are welcome | 10:18 |
| mkoderer | afazekas: giulivo: could we have a discussion about https://review.openstack.org/#/c/45465/2 | 10:19 |
| mkoderer | I think it would be a good idea to split it into a separate class | 10:20 |
| mkoderer | since the backup functionality can be disabled | 10:20 |
| mkoderer | dosaboy: ^^^ | 10:20 |
| mkoderer | I will put my comment on it - maybe you could have a look | 10:21 |
| afazekas | mkoderer: we do not need to separate the service classes, actually they should be merged to bigger files in the compute case as well | 10:21 |
| mkoderer | afazekas: ok, but you will have function that aren't available | 10:22 |
| mkoderer | so if you disable it you have function that you shouldn't call | 10:23 |
| mkoderer | IMHO thats confusing | 10:23 |
| afazekas | mkoderer: ok the backup/.* url pattern might get new class | 10:24 |
| mkoderer | afazekas: yes exactly | 10:25 |
| *** jgallard has quit IRC | 10:25 | |
| afazekas | the snapshots also got one | 10:25 |
| mkoderer | afazekas: ok, and is there a rule that a new test should have at least one "gate" test, are two smoke tests are sufficient for the beginning? | 10:26 |
| mkoderer | afazekas: I mean smoke test are gating anyway... | 10:27 |
| afazekas | mkoderer: no, there is no rule | 10:28 |
| giulivo | afazekas, to be honest I think this https://review.openstack.org/#/c/42621/9/tempest/services/volume/xml/volumes_client.py is more like an hack unneeded in the json client because of json.loads | 10:28 |
| afazekas | We should create rule for what should be smoke and add it to the documentation | 10:28 |
| giulivo | but I agree that tests should run unchanged, you're absolutely right on that, I'll +1 the change | 10:29 |
| afazekas | IMHO we should find a set which contains a tests from api (at least one from all sub dir) scenario and third party , and has good coverage | 10:29 |
| *** matsuhashi has quit IRC | 10:30 | |
| afazekas | giulivo: json is able to distinguish boolean from string, xml just able to do that with a formal specification , for example with xsd (the lower case true,false,0,1 are valid) | 10:31 |
| afazekas | We are able to use the same test code with two transport (xml, json) backend because they wants and provides the same 'python data structure' | 10:32 |
| afazekas | adding a backend dependent 'if' statement to the test cases is worse IMHO | 10:33 |
| giulivo | afazekas, agreed on gerrit, sorry about that | 10:35 |
| afazekas | giulivo: np | 10:36 |
| *** nosnos_ has quit IRC | 10:39 | |
| *** HenryG has joined #openstack-qa | 10:51 | |
| *** HenryG_ has joined #openstack-qa | 10:52 | |
| *** HenryG_ has quit IRC | 10:54 | |
| *** HenryG has quit IRC | 10:55 | |
| *** HenryG has joined #openstack-qa | 10:58 | |
| *** afazekas has quit IRC | 11:01 | |
| giulivo | mkoderer, thanks for adding me here https://review.openstack.org/#/c/45465/ ! | 11:01 |
| giulivo | (pointing me there) | 11:01 |
| *** jgallard has joined #openstack-qa | 11:13 | |
| *** plomakin_ has quit IRC | 11:15 | |
| dosaboy | mkoderer, giulivo: hey | 11:16 |
| giulivo | ping dosaboy | 11:17 |
| dosaboy | thanks fro poining out the bp | 11:17 |
| dosaboy | agh can't type today | 11:17 |
| giulivo | dosaboy, thank you for helping with that | 11:17 |
| dosaboy | np, i'll see if I can get it fixed up later | 11:17 |
| dosaboy | i had not noticed that BP | 11:17 |
| giulivo | dosaboy, it's nice you joined on IRC because I think mkoderer prefers that route and I should also use it more in place of gerrit comments | 11:19 |
| dosaboy | gerrit comments are fine too | 11:20 |
| dosaboy | it is easier to lose irc comments | 11:20 |
| *** plomakin has joined #openstack-qa | 11:25 | |
| giulivo | ping sdague | 11:25 |
| *** afazekas has joined #openstack-qa | 11:27 | |
| *** afazekas_ has joined #openstack-qa | 11:27 | |
| giulivo | ping afazekas afazekas_ | 11:27 |
| afazekas_ | giulivo: pong | 11:28 |
| giulivo | I see there is a proposal to run the tempest bug triage this week instead of the initial date proposed (the 17th) | 11:28 |
| giulivo | I was going to send the announcement so I'm asking if you have preferences between this friday (13rd) and the 17th | 11:29 |
| afazekas_ | both can work for me | 11:29 |
| giulivo | mkoderer, ^^ ? | 11:30 |
| giulivo | sdague, ^^ ? | 11:30 |
| giulivo | jaypipes, ^^ ^ | 11:30 |
| * giulivo oops | 11:30 | |
| giulivo | cyeoh, ^^ ? | 11:30 |
| * mkoderer is in a meeting | 11:30 | |
| sdague | giulivo: hey | 11:30 |
| giulivo | I just wanted to ensure the core was available for both dats | 11:31 |
| sdague | so my availability over the next couple of weeks is a little goofy. Friday is my wife's birthday, and next week I'm at Linux Con presenting. | 11:31 |
| sdague | I'm taking Friday off so we can do a long bike ride together | 11:31 |
| *** afazekas has quit IRC | 11:32 | |
| sdague | which means everything looks bad for me, so don't take me into account :) | 11:32 |
| giulivo | mtreinish, ^^ ? | 11:34 |
| cyeoh | giulivo: either is fine with me, but your friday is my saturday so I'm not around as much | 11:37 |
| cyeoh | well probably not at all on saturday and only a bit during fri night maybe | 11:37 |
| giulivo | oh didn't know, so that is a point for the 17th, thanks | 11:38 |
| sdague | giulivo: mtreinish probably isn't awake yet, I'll make sure to nudge him in the office about it | 11:40 |
| dkranz | giulivo: I could do any of those. | 11:48 |
| giulivo | dkranz, sorry for missing to ping you and thanks for helping | 11:53 |
| *** anteaya has joined #openstack-qa | 11:53 | |
| dkranz | giulivo: np | 11:53 |
| *** weshay has joined #openstack-qa | 12:08 | |
| *** weshay has quit IRC | 12:14 | |
| *** dkorn1 has joined #openstack-qa | 12:18 | |
| *** dkorn has quit IRC | 12:19 | |
| *** weshay has joined #openstack-qa | 12:29 | |
| openstackgerrit | Dirk Mueller proposed a change to openstack-dev/devstack: Install schema-image.json https://review.openstack.org/45672 | 12:34 |
| *** dkorn1 has quit IRC | 12:39 | |
| *** dkorn has joined #openstack-qa | 12:42 | |
| *** dkorn has quit IRC | 12:45 | |
| *** dkorn has joined #openstack-qa | 12:45 | |
| *** prekarat has joined #openstack-qa | 12:55 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack-dev/devstack: Wrong arguments for die() call https://review.openstack.org/45468 | 13:01 |
| openstackgerrit | Maru Newby proposed a change to openstack-dev/devstack: Add option to minimize neutron ovs l2 polling https://review.openstack.org/45680 | 13:13 |
| *** rustlebee is now known as russellb | 13:14 | |
| *** mriedem has joined #openstack-qa | 13:15 | |
| openstackgerrit | adalbas proposed a change to openstack/tempest: Fix race condition for test_flavors_extra_specs https://review.openstack.org/45586 | 13:37 |
| adalbas | afazekas_, ping | 13:48 |
| mtreinish | giulivo: I should be around on both days. I think that the 17th would be a bit more convenient for me though. | 13:52 |
| giulivo | mtreinish, thanks, that is one more point for the 17th | 13:52 |
| afazekas_ | adalbas: pong | 13:54 |
| *** afazekas_ is now known as afazekas | 13:55 | |
| *** marun has quit IRC | 13:56 | |
| *** dkranz has quit IRC | 14:00 | |
| adalbas | afazekas, about your review https://review.openstack.org/#/c/35165/, i think it also affects the bug 1218391 , which is causing a lot of rechecks in the gate. | 14:02 |
| *** burt has joined #openstack-qa | 14:04 | |
| afazekas | adalbas: probably yes, and some other might be related | 14:06 |
| afazekas | adalbas: thx for the review | 14:07 |
| adalbas | afazekas, sure. I'm interested in that so i'd like to test it when a new patchset is sent. | 14:08 |
| * afazekas priority ++ of 35165 | 14:10 | |
| adalbas | afazekas, lol. tell me if you need any help ;) | 14:11 |
| afazekas | I need a time machine :) | 14:11 |
| *** adalbas has quit IRC | 14:22 | |
| *** prekarat has quit IRC | 14:34 | |
| *** adalbas has joined #openstack-qa | 14:38 | |
| *** SergeyLu_ has joined #openstack-qa | 14:42 | |
| *** SergeyLu_ has quit IRC | 14:46 | |
| *** yfried has quit IRC | 14:52 | |
| openstackgerrit | Kyle Mestery proposed a change to openstack-dev/devstack: Augment instead of override extra ML2 options. https://review.openstack.org/45697 | 14:53 |
| *** dkorn has quit IRC | 14:54 | |
| openstackgerrit | Tal Kammer proposed a change to openstack/tempest: Initial basic setup of openstack and tempest config file https://review.openstack.org/42920 | 14:54 |
| *** rdekel has quit IRC | 14:56 | |
| *** krtaylor has quit IRC | 15:02 | |
| sdague | mtreinish: can I get you to reconsider your -1 here - https://review.openstack.org/#/c/45342/, as I think the future is to take it into fixtures - https://bugs.launchpad.net/python-fixtures/+bug/1221724 is what lifeless asked me to register for it | 15:10 |
| openstackgerrit | Tal Kammer proposed a change to openstack/tempest: Initial basic setup of openstack and tempest config file https://review.openstack.org/42920 | 15:11 |
| mtreinish | sdague: fine, approved | 15:13 |
| sdague | thanks | 15:13 |
| *** dkranz has joined #openstack-qa | 15:16 | |
| *** afazekas has quit IRC | 15:17 | |
| tkammer | As discussed in the meeting, I have changed the auto configuration script I wrote originally in BASH to python | 15:18 |
| tkammer | If you have time to review it, that would be great :) https://review.openstack.org/42920 | 15:18 |
| *** mlavalle has joined #openstack-qa | 15:24 | |
| *** mkollaro has quit IRC | 15:36 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix tempest config usage in test_attach_volume https://review.openstack.org/45706 | 15:54 |
| openstackgerrit | A change was merged to openstack/tempest: fix race condition between addCleanup and lockutils https://review.openstack.org/45342 | 16:01 |
| *** amscanne has quit IRC | 16:04 | |
| *** mkollaro has joined #openstack-qa | 16:06 | |
| *** mkollaro has quit IRC | 16:07 | |
| openstackgerrit | Tal Kammer proposed a change to openstack/tempest: Initial basic setup of openstack and tempest config file https://review.openstack.org/42920 | 16:10 |
| flashgordon | I have a bug that was hit 24 times in tempest in the last 24 hours: https://bugs.launchpad.net/nova/+bug/1221987 | 16:30 |
| *** flashgordon is now known as jog0 | 16:30 | |
| jog0 | as confirmed via logstash | 16:30 |
| *** malini has joined #openstack-qa | 16:34 | |
| *** jgallard has quit IRC | 16:37 | |
| *** bmqq123_away is now known as yjiang5 | 16:41 | |
| dkranz | sdague: Are we enforcing that negative tests should specify the 'negative' tag? | 16:42 |
| dkranz | sdague: Or is it optional? | 16:42 |
| *** tkammer has quit IRC | 16:44 | |
| dkranz | sdague: I think we should approve https://review.openstack.org/#/c/44299/ but it has your -1 on it to which I commented. | 16:45 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Provide a means of setting vmdk custom properties via image filename https://review.openstack.org/45181 | 17:03 |
| *** openstackgerrit has quit IRC | 17:05 | |
| *** openstackgerrit has joined #openstack-qa | 17:05 | |
| sdague | dkranz: so I thought the policy we set out in stable was it had to be fixed in master first, like other projects, so we don't end up with a big diversion | 17:17 |
| *** wramthun has joined #openstack-qa | 17:18 | |
| dkranz | sdague: I thought it was fixed in master | 17:18 |
| dkranz | sdague: There is a a cherry pick mentioned in the commit message | 17:18 |
| sdague | ok, I'll change my vote | 17:20 |
| dkranz | sdague: Thanks | 17:21 |
| dkranz | mtreinish: Regarding https://review.openstack.org/#/c/45706/ | 17:26 |
| dkranz | mtreinish: config is set as a top-level class variable in BaseTestCase | 17:26 |
| dkranz | mtreinish: I'm not enough of a python expert to understand all the initialization orderings going on here. | 17:27 |
| dkranz | mtreinish: Obviously we need to make this more bulletproof as you suggest. | 17:27 |
| mtreinish | dkranz: yeah I can't remember the exact reason either, but the skip decorators can't access config from the base class | 17:28 |
| mtreinish | the way I did it the patch was wrong too | 17:28 |
| mtreinish | I've got an update coming in a sec | 17:28 |
| mtreinish | but we should move to using config the inteded way | 17:29 |
| *** mlavalle has quit IRC | 17:29 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix tempest config usage in test_attach_volume https://review.openstack.org/45706 | 17:32 |
| *** SergeyLukjanov has joined #openstack-qa | 17:39 | |
| openstackgerrit | Alex Rudenko proposed a change to openstack-dev/devstack: blueprint devstack-support-for-keystone-mixbackend https://review.openstack.org/44605 | 17:57 |
| *** HenryG_ has joined #openstack-qa | 18:01 | |
| *** HenryG has quit IRC | 18:05 | |
| *** mriedem has quit IRC | 18:18 | |
| *** mriedem has joined #openstack-qa | 18:20 | |
| openstackgerrit | Alex Rudenko proposed a change to openstack-dev/devstack: Additional variable APACHE_SERVER_NAME added https://review.openstack.org/44593 | 18:22 |
| *** marun has joined #openstack-qa | 18:26 | |
| openstackgerrit | Bob Melander proposed a change to openstack-dev/devstack: Changes to make Devstack work with Neutron L3 plugin patch. https://review.openstack.org/20909 | 18:29 |
| *** mlavalle has joined #openstack-qa | 18:46 | |
| openstackgerrit | Clint Byrum proposed a change to openstack-dev/devstack: Lower Heat max_template_size for Tempest https://review.openstack.org/45730 | 18:57 |
| openstackgerrit | Clint Byrum proposed a change to openstack/tempest: Test that Heat max_template_size is applied https://review.openstack.org/45732 | 19:01 |
| openstackgerrit | Clint Byrum proposed a change to openstack/tempest: Test Heat max_resources_per_stack limit https://review.openstack.org/45733 | 19:01 |
| *** mriedem1 has joined #openstack-qa | 19:04 | |
| *** mriedem has quit IRC | 19:07 | |
| *** krtaylor has joined #openstack-qa | 19:08 | |
| mtreinish | sdague: for the service tags for the list of allowed services should I include incubated projects? (trove, ironic, marconi) | 19:13 |
| sdague | mtreinish: yeh, though we're using function tags right, not code name tags? | 19:23 |
| mtreinish | sdague: I was using code names but I can change that: http://paste.openstack.org/show/46356/ | 19:26 |
| sdague | yeh, lets use functional names | 19:26 |
| mtreinish | sdague: even for ones that have 2 words? ('block storage', 'object storage', etc...) | 19:27 |
| mtreinish | we used code names for service_available configs | 19:28 |
| sdague | ok | 19:32 |
| openstackgerrit | Clint Byrum proposed a change to openstack/tempest: Test that Heat max_template_size is applied https://review.openstack.org/45732 | 19:46 |
| openstackgerrit | Clint Byrum proposed a change to openstack/tempest: Test Heat max_resources_per_stack limit https://review.openstack.org/45733 | 19:46 |
| *** jhenner has quit IRC | 20:01 | |
| *** marun has quit IRC | 20:16 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack-dev/devstack: Swift: configure Ceilometer when it is enabled https://review.openstack.org/45201 | 20:18 |
| openstackgerrit | Akihiro Motoki proposed a change to openstack-dev/devstack: Configure VPNaaS Horizon panel if q-vpn is enabled https://review.openstack.org/45751 | 20:23 |
| *** HenryG_ has quit IRC | 20:25 | |
| *** HenryG has joined #openstack-qa | 20:26 | |
| *** yassine has quit IRC | 20:28 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Missing pxelinux.0 on RedHat systems https://review.openstack.org/44024 | 20:42 |
| openstackgerrit | Edward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests https://review.openstack.org/45465 | 20:47 |
| *** marun has joined #openstack-qa | 21:03 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Enable multi-threaded nova API servers https://review.openstack.org/45314 | 21:07 |
| openstackgerrit | A change was merged to openstack/tempest: Fix tempest config usage in test_attach_volume https://review.openstack.org/45706 | 21:07 |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/devstack: Enable multi-threaded nova-conductor https://review.openstack.org/45766 | 21:23 |
| *** malini is now known as malini_afk | 21:38 | |
| *** mriedem1 has quit IRC | 21:38 | |
| *** dkranz has quit IRC | 21:42 | |
| openstackgerrit | Edward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests https://review.openstack.org/45465 | 21:51 |
| *** SergeyLukjanov has quit IRC | 22:01 | |
| *** hemnafk is now known as hemna | 22:02 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix import grouping in scenario tests https://review.openstack.org/45779 | 22:08 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Remove order dependence from network_basic_ops https://review.openstack.org/45780 | 22:08 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add @services decorator https://review.openstack.org/45781 | 22:08 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add service tags to api.volume https://review.openstack.org/45782 | 22:08 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add services tags to scenario tests https://review.openstack.org/45783 | 22:08 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add hacking check for service tags in scenario https://review.openstack.org/45784 | 22:08 |
| *** burt has quit IRC | 22:08 | |
| *** weshay has quit IRC | 22:11 | |
| *** krtaylor has quit IRC | 22:45 | |
| *** mlavalle has quit IRC | 23:00 | |
| yjiang5 | sdague: We are adding PCI passthrough support to nova, and maybe in future PCI NIC to neutron. I'm considering adding some integration test to tempest in I release, but that will be tricky because PCI passthrough has special requirement on real hardware, do you have any suggestion/hints on it? | 23:21 |
| *** afazekas has joined #openstack-qa | 23:40 | |
| *** afazekas is now known as afazekas_zz | 23:43 | |
| sdague | yjiang5: we have the ability to have conditional bits in the config file, so you code set whether it was supported. | 23:46 |
| sdague | but that might make for a good design summit session | 23:46 |
| giulivo | ping dosaboy you around? | 23:53 |
| giulivo | ping mtreinish you around? | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!