Wednesday, 2018-06-27

*** bobh has quit IRC02:10
*** e0ne has joined #openstack-vitrage05:35
*** e0ne has quit IRC05:36
*** eyalb has joined #openstack-vitrage06:25
*** eyalb1 has joined #openstack-vitrage07:18
*** ifat_afek has joined #openstack-vitrage07:21
*** brault has quit IRC07:30
*** brault has joined #openstack-vitrage07:30
*** eyalb has quit IRC07:39
*** eyalb has joined #openstack-vitrage07:40
*** eyalb has quit IRC07:40
*** eyalb has joined #openstack-vitrage07:41
*** e0ne has joined #openstack-vitrage07:53
*** pava_ has quit IRC07:57
*** pava_ has joined #openstack-vitrage07:58
openstackgerritMerged openstack/vitrage master: Add upgrade support for Vitrage  https://review.openstack.org/57511507:58
*** e0ne has quit IRC08:46
*** ifat_afek has quit IRC09:23
*** ifat_afek has joined #openstack-vitrage10:03
openstackgerritYuval Adar proposed openstack/vitrage-dashboard master: Add templates panel to admin section  https://review.openstack.org/57835010:16
*** e0ne has joined #openstack-vitrage10:41
openstackgerritMuhamad Najjar proposed openstack/vitrage-dashboard master: fix time format to show in 24h  https://review.openstack.org/57835811:27
*** e0ne has quit IRC11:29
openstackgerritMuhamad Najjar proposed openstack/vitrage-dashboard master: fix time format to show in 24h  https://review.openstack.org/57835811:34
openstackgerritYuval Adar proposed openstack/vitrage-dashboard master: Add templates panel to admin section  https://review.openstack.org/57836011:34
openstackgerritYuval Adar proposed openstack/vitrage-dashboard master: Add templates panel to admin section  https://review.openstack.org/57836011:36
*** e0ne has joined #openstack-vitrage11:42
*** ifat_afek has quit IRC12:00
openstackgerritOleksiy Petrenko proposed openstack/vitrage master: Refactor vitrage to use global CONF instead of local one  https://review.openstack.org/57738812:01
openstackgerritMerged openstack/vitrage-dashboard master: fix time format to show in 24h  https://review.openstack.org/57835812:18
*** e0ne has quit IRC13:26
*** e0ne has joined #openstack-vitrage13:38
e0neeyalb: hi. are you around? I've got few questions about vitrage tempest plugin13:42
*** bobh has joined #openstack-vitrage13:55
eyalbi am here13:56
e0newe discussed a bit last week on the meeting that tempest plugin imports some things from vitrage directly14:02
e0nedo you know about any plans on fixing ?i14:03
e0ne*fixing it14:03
eyalbit is on our road map but nopthing concreate14:10
eyalbsome fixes are easy some are more complicated14:10
eyalbyou have all the constants that are taken from vitrage this is easy to fix14:11
eyalbeither duplicate of create a common library14:11
eyalbthe complicated is that there are some tests that use vitrage inner implementations which is wrong and should be refactored14:12
e0neyea, I see14:14
e0newe tried to re-factore conf object usage14:15
e0neand now tempest plugin fails because of it14:15
e0neI'm  trying to figure out the best way to fix it without huge refactoring if possible14:16
eyalbI have no problem with huge refactoring better now then later14:18
e0nefair enough :)14:19
eyalbthis something that should have been done before but we simply didn't have enough resources14:19
eyalbor even the knowledge of how things should be done the "OpenStack" way14:19
e0neI totally understand the reasons why it didn't happen14:20
e0neeyalb: honestly, 'openstack way' is not always the best way14:20
eyalbI agree14:20
*** ifat_afek has joined #openstack-vitrage14:21
eyalbbut sometimes you have no choice :-(14:21
e0ne+114:21
e0ne:(14:21
eyalbso if you have the to refactor then go ahead ...14:22
eyalb*time*14:22
e0neit's a hard question14:23
e0neI'll try to find some resources after rocky-3 milestone14:23
eyalbyes this can be good14:24
eyalbmaybe for now we can just have the upgrade without tests and work on it after rocky-3 milestone14:25
e0nenot a bad plan14:30
*** eyalb has quit IRC14:31
e0neI'll try to have a fast fix: disable tests (patch #1) => implement conf changes (patch #2) => implement tempest changes (patch #3) => enable tests (patch #4) =>  implement db migrations (patch #5) == have a chain with 5 patches:(. after it we can start refactoring  tempest plugin without rush14:32
ifat_afeke0ne: Hi :-) eyalb just left, but I followed part of the discussion14:33
ifat_afekWhat do you mean by ‘disable tests’? the new upgrade tests?14:33
ifat_afekAnd what are the parts that you plan to do after rocky-3?14:34
e0neifat_afek: hi. I mean move current jobs in non-voting mode to get tempest fix landed14:40
e0neI'm not sure that it's a good solution, TBH :(14:40
e0neifat_afek: I'll try to find some resources to refactor tempest plugin as discussed above after rocky-314:41
ifat_afeke0ne: this would be great. We know this is something we must fix… and we understand if it’s currently blocking you14:56
e0neifat_afek: thanks for supporting my efforts! I appreciate it14:57
ifat_afekThanks for helping :-) we have a common interest14:59
e0ne:)14:59
*** e0ne has quit IRC16:13
*** ifat_afek has quit IRC17:06
*** e0ne has joined #openstack-vitrage18:16
*** bobh has quit IRC18:22
*** bobh has joined #openstack-vitrage18:23
*** e0ne has quit IRC18:32
*** e0ne has joined #openstack-vitrage18:44
*** ifat_afek has joined #openstack-vitrage19:45
*** ifat_afek has quit IRC19:56
*** e0ne has quit IRC20:42
*** ifat_afek has joined #openstack-vitrage21:13
*** ifat_afek has quit IRC21:14
*** bobh has quit IRC22:00
*** brault has quit IRC23:54

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