| *** esmute_ has joined #openstack-cue | 00:01 | |
| *** esmute_ has quit IRC | 00:02 | |
| *** esmute_ has joined #openstack-cue | 00:40 | |
| *** esmute_ has quit IRC | 02:22 | |
| *** esmute_ has joined #openstack-cue | 02:33 | |
| *** esmute_ has quit IRC | 03:30 | |
| *** esmute_ has joined #openstack-cue | 03:37 | |
| *** esmute_ has quit IRC | 05:16 | |
| *** esmute_ has joined #openstack-cue | 05:19 | |
| *** esmute_ has quit IRC | 05:20 | |
| *** openstackgerrit_ has joined #openstack-cue | 09:53 | |
| *** openstackgerrit has quit IRC | 11:31 | |
| *** openstackgerrit_ is now known as openstackgerrit | 11:31 | |
| *** openstackgerrit has quit IRC | 11:32 | |
| *** openstackgerrit_ has joined #openstack-cue | 11:32 | |
| *** openstackgerrit_ is now known as openstackgerrit | 11:32 | |
| *** openstackgerrit_ has joined #openstack-cue | 11:33 | |
| *** openstackgerrit has quit IRC | 12:16 | |
| *** openstackgerrit has joined #openstack-cue | 12:17 | |
| *** miqui has joined #openstack-cue | 13:11 | |
| *** esmute_ has joined #openstack-cue | 14:37 | |
| *** sputnik13 has joined #openstack-cue | 15:29 | |
| openstackgerrit | Tom Cocozzello proposed openstack/cue-dashboard: Activate pep8 check that _ is imported https://review.openstack.org/209153 | 16:04 |
|---|---|---|
| *** openstackgerrit_ has quit IRC | 16:29 | |
| *** esmute_ has quit IRC | 16:51 | |
| *** sputnik13 has quit IRC | 17:04 | |
| *** sputnik13 has joined #openstack-cue | 17:05 | |
| *** abitha has joined #openstack-cue | 17:06 | |
| *** esmute_ has joined #openstack-cue | 17:59 | |
| *** dkalleg has joined #openstack-cue | 17:59 | |
| vipul | sputnik13: so what do we mean by that | 19:01 |
| sputnik13 | mean by what | 19:02 |
| vipul | tempest tests are covered by unit/functional | 19:02 |
| sputnik13 | oh, I just mean the things we're doing, I think they are actually covered for the most part | 19:02 |
| sputnik13 | in tempest we're doing a cluster create | 19:03 |
| sputnik13 | we have coverage for that code path | 19:03 |
| vipul | but there is likely some part of that code path that only gets exercised by tempest right | 19:03 |
| vipul | for example.. the cloud init script.. | 19:04 |
| sputnik13 | likely | 19:04 |
| vipul | anyway.. the point i was trying to make is.. if there are other tests that are run.. that catch the things not caught by unit tests.. | 19:05 |
| vipul | then it shouldn't be a bug | 19:05 |
| vipul | if there is functionality that isn't tested by unit / tempest / rally.. | 19:05 |
| vipul | then it's a bug | 19:05 |
| sputnik13 | agreed | 19:06 |
| vipul | it would be nice to have a unit / functional test.. but it's not necessary | 19:06 |
| sputnik13 | at the same time we're aiming for 100% coverage, there's no reason we shouldn't have 100% coverage in the coverage report | 19:06 |
| sputnik13 | anything we've considered and decided that something like tempest or rally is testing, or we decide it's not something possible to really test | 19:07 |
| sputnik13 | or anything we consider not worthwhile testing | 19:07 |
| sputnik13 | should be marked noqa | 19:07 |
| sputnik13 | that shows that someone put conscious thought in to whether this needs to be tested and decided it shouldn't be | 19:07 |
| sputnik13 | anything less than 100% in the coverage report means there's code someone hasn't given that level of scrutiny | 19:08 |
| vipul | sputnik13: yep.. that's a good point.. if it's covered by something else.. then it's fine to exclude from coverage report if that's the best way to test it | 19:08 |
| sputnik13 | on a positive note, we only have 2 bugs now above low and 'new' now :) | 19:08 |
| abitha | the bug about broker add is already fixed. | 19:08 |
| abitha | I am closing that now | 19:09 |
| sputnik13 | please link the patch that fixes the bug | 19:09 |
| abitha | ok | 19:09 |
| vipul | oh btw.. our blueprints should be searchable now via gerrit | 19:09 |
| sputnik13 | now that we're openstack instead of stackforge | 19:12 |
| sputnik13 | the links actually work now | 19:12 |
| sputnik13 | :) | 19:12 |
| vipul | sputnik13: it was one more step ;) | 19:16 |
| vipul | had to add our project to be a 'member' of the openstack launchpad | 19:17 |
| *** esmute_ has quit IRC | 19:28 | |
| openstackgerrit | Davide Agnello proposed openstack/cue: Renaming cue-scenarios.yaml rally tasks file https://review.openstack.org/209229 | 19:33 |
| *** esmute_ has joined #openstack-cue | 19:36 | |
| *** esmute_ has quit IRC | 19:40 | |
| sputnik13 | vipul one step too many! | 20:22 |
| openstackgerrit | Davide Agnello proposed openstack/cue: Renaming cue-scenarios.yaml rally tasks file https://review.openstack.org/209229 | 20:23 |
| *** esmute_ has joined #openstack-cue | 20:43 | |
| *** abitha has quit IRC | 21:10 | |
| *** abitha has joined #openstack-cue | 21:11 | |
| *** miqui has quit IRC | 22:20 | |
| *** sputnik13 has quit IRC | 23:34 | |
| openstackgerrit | Merged openstack/cue: Renaming cue-scenarios.yaml rally tasks file https://review.openstack.org/209229 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!