Monday, 2014-12-29

*** openstack has joined #storyboard16:02
*** openstackstatus has joined #storyboard16:17
*** ChanServ sets mode: +v openstackstatus16:17
*** wuhg has quit IRC16:44
krotscheckNikitaKonovalov: What is aleksey’s IRC handle?17:32
krotscheckNikitaKonovalov: I want to make sure we file a webclient story for https://review.openstack.org/#/c/143984/17:33
openstackgerritMerged openstack-infra/storyboard: Fix i18n in exception module.  https://review.openstack.org/14399217:34
openstackgerritMerged openstack-infra/storyboard: Extended titles  https://review.openstack.org/14398417:35
openstackgerritMerged openstack-infra/storyboard: All aborts now have message  https://review.openstack.org/14346517:36
*** yolanda has quit IRC18:27
*** yolanda has joined #storyboard18:29
*** NikitaKonovalov has quit IRC18:29
*** NikitaKonovalov has joined #storyboard18:31
openstackgerritMichael Krotscheck proposed openstack-infra/storyboard: Corrected type casting in OAuth Response  https://review.openstack.org/14434118:51
openstackgerritMichael Krotscheck proposed openstack-infra/storyboard: Corrected type casting in OAuth Response  https://review.openstack.org/14434118:59
openstackgerritMerged openstack-infra/storyboard: Corrected type casting in OAuth Response  https://review.openstack.org/14434119:07
yolandakrotscheck, i was thinking about the failure caused by six. Would it be possible to have same test we have with the frontend, where we can test in a draft frontend, and be sure that everything works, but on the backend? have same test, but use master frontend, and the backend of the change19:14
krotscheckyolanda: I think that’s called functional tests :)19:26
yolandabut we are replacing that functional tests at the moment in the frontend by human tests :)19:26
krotscheckAnd our human tests are actual users :)19:27
krotscheckI think we both agree that bugs like that one should not crop up.19:27
krotscheckI feel that the amount of effort to build coverage for that component in our functional tests is probably equivalent to building a test in the frontend and gating on that.19:28
yolandaso adding more tests in the backend, ensuring that basic operations, is needed19:29
yolandawe all gave positive feedback on that change, it was quite simple, but even simple things can cause trouble19:29
krotscheckYep19:30
krotscheckBut we caught it quick.19:30
krotscheckSo that’s good19:30
yolandayou were fast :)19:31
yolandaagh, i'm going to throw Furby through the window :)19:33
*** reed has joined #storyboard19:36
krotscheckDid you get it for them?19:46
yolandayes :(19:48
yolandait has been singing, farting and laughing all day :(19:49
openstackgerrityolanda.robla proposed openstack-infra/storyboard-webclient: Add timeout to the blur event for story inline edition  https://review.openstack.org/14435419:57
yolandakrotscheck, eternal diff in that review about newline. Shall we leave it like that, until i create the lintian warn for it and we force newlines?19:58
yolandait seems to happen only in js files, created by other people, as my editors are always adding a trailing newline19:59
krotscheckYou have created your own hell :)19:59
yolandaso, my changes are adding the right newline there :)19:59
openstackgerritMichael Krotscheck proposed openstack-infra/storyboard: Added cron plugin to clean old access tokens.  https://review.openstack.org/14053620:01
openstackgerritMichael Krotscheck proposed openstack-infra/storyboard: Consolidated OAuth Configuration, added enable flag to oauth cleaner.  https://review.openstack.org/14435520:01
*** Piet has joined #storyboard21:00
*** mase_x200 has joined #storyboard21:22
*** mase_x200 has quit IRC21:31
*** Piet has quit IRC22:18
openstackgerritKhai Do proposed openstack-infra/storyboard: Allow overriding test database from an environment variable  https://review.openstack.org/14437523:04
openstackgerritKhai Do proposed openstack-infra/storyboard: Allow overriding test database from an environment variable  https://review.openstack.org/14437523:24
openstackgerritKhai Do proposed openstack-infra/storyboard: Allow overriding test database from an environment variable  https://review.openstack.org/14437523:30

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