Monday, 2019-04-15

*** kopecmartin|off is now known as kopecmartin06:17
*** tosky has joined #refstack07:20
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Introduce post_configuration method  https://review.openstack.org/65186608:11
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: DNM testing converting of images  https://review.openstack.org/65087408:33
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add heat_plugin support  https://review.openstack.org/64550008:34
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add a job for tempest plugins  https://review.openstack.org/64524008:35
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Configure options for neutron plugin  https://review.openstack.org/65094508:38
toskyarxcruz, kopecmartin : no taiga thing in the commit message08:39
kopecmartinarxcruz, and here we are, i thought i had a reason for that, but i couldn't remember which one, i was searching in old reviews for a discussion about it but i didn't find anything08:41
kopecmartinaand let's recheck all of the reviews again, i love my work ;)08:42
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Introduce post_configuration method  https://review.openstack.org/65186608:44
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add heat_plugin support  https://review.openstack.org/64550008:45
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add heat_plugin support  https://review.openstack.org/64550008:45
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add a job for tempest plugins  https://review.openstack.org/64524008:46
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add a job for tempest plugins  https://review.openstack.org/64524008:46
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Configure options for neutron plugin  https://review.openstack.org/65094508:47
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Configure options for neutron plugin  https://review.openstack.org/65094508:47
toskykopecmartin: right, but that does not exempt you from creating stories on storyboard08:48
toskykopecmartin: so I'm going to -1 again later today unless I see a proper story with tasks or more stories which track the ongoing effort08:48
toskyso that we have a more general vision of what's going on - with the proper tooling08:49
kopecmartintosky, is it really necessary? simple changes ..08:49
toskythey are not simple08:49
kopecmartinfor general vision you can see the history on github for exmaple08:49
tosky- add a new (big) job to test something08:49
tosky- run more tests in an existing jobs - and there is a release note for it, so it's not small08:50
kopecmartinstill, storyboard makes more sense for tracking bugs08:50
tosky- post_config is another step in the ongoing refactoring to remove services; we had a story for it, this could be a new task, or, if the scope changed, a new story with subtasks08:50
toskyno.08:50
toskythe history is useful after-the-facts08:50
toskystoryboard was born to replace also blueprints; bugs are kind of afterthought08:51
toskyso features should be tracked there08:51
toskymust, I'd say08:51
toskysame for the dreaded image conversion08:51
kopecmartinthis is bureaucracy, nothing else08:52
toskyno08:52
toskyit's understanding what's going on08:53
toskythe proper way to do it is storyboard08:53
toskyfor example, when I created https://storyboard.openstack.org/#!/story/2002787 I *had to* think about what I wanted to achive08:54
toskyit's a way to plan the work before writing code - of course it's possible that something will need to change on the way, but the discussion for example on the post_config that happened on the review may have been avoided if we had a story08:55
toskyif may have discussed the approach there before writing the code08:55
kopecmartinthe changes are there for weeks, and only now we realize that storyboard is needed, great ...08:56
toskysure, because I'm stupid and dump08:56
toskyI should have noticed it immediately, and I didn't08:57
toskyso that's my fault08:57
toskywhich does not excuse the fact that it should be fixed08:57
kopecmartintosky, it's not your fault08:58
toskyoh, it totally is08:58
kopecmartinthere are at least 4 core-reviewers (including me) and all of us are working on different things, so we missed these little things08:59
kopecmartinand i can't say i wasn't lazy , because i was and I am :D08:59
kopecmartini like shortcuts08:59
kopecmartinok, anyway , i'm gonna create stories and asks in existing stories .. copy some info there and then hopefully we'll merge those reviews finally09:00
toskykopecmartin: but not just a story for each review - see my upcoming comments09:02
toskythe story should group related tasks together09:02
kopecmartintosky, i understand09:03
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add option for converting images  https://review.openstack.org/64812109:23
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add option for converting images  https://review.openstack.org/64812109:27
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Introduce post_configuration method  https://review.openstack.org/65186609:35
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add heat_plugin support  https://review.openstack.org/64550009:47
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add a job for tempest plugins  https://review.openstack.org/64524009:51
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Add a job for tempest plugins  https://review.openstack.org/64524009:56
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Configure options for neutron plugin  https://review.openstack.org/65094510:12
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: DNM testing converting of images  https://review.openstack.org/65087411:29
*** tosky has quit IRC12:36
*** tosky has joined #refstack15:08
*** markvoelker has joined #refstack15:20
*** kopecmartin is now known as kopecmartin|off16:50
*** markvoelker has quit IRC18:26
*** markvoelker has joined #refstack19:04
*** eglute has joined #refstack21:27
*** markvoelker has quit IRC22:57
*** markvoelker has joined #refstack23:08

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!