Monday, 2019-02-18

*** eyalb1 has joined #openstack-vitrage06:03
*** e0ne has joined #openstack-vitrage06:41
*** e0ne has quit IRC06:47
*** ifat_afek has joined #openstack-vitrage07:18
*** e0ne has joined #openstack-vitrage07:35
*** e0ne has quit IRC07:38
openstackgerrithewei proposed openstack/vitrage master: optimize aodh/ceilometer/kubernetes datasource  https://review.openstack.org/63748407:52
*** e0ne has joined #openstack-vitrage09:08
ifat_afeke0ne: hi09:12
e0neifat_afek: hi09:12
ifat_afekRegarding the storyboard tasks - I’m not sure if there is a need to create several tasks. I somehow assumed you would know ;-)09:13
ifat_afekI just think that once one change will be merged, the task will be marked as “merged”09:13
ifat_afekBut I don’t really care, it’s up to you09:13
e0neifat_afek: I just to not create small task if there is few lines change09:14
ifat_afekOk, so I’ll approve it as is09:14
e0neifat_afek: thanks09:14
ifat_afekAnd another question - I want to release today all vitrage deliverables. I need a new version of python-vitrageclient for my change in Heat. Do you have any reason not to release today? anything unstable or WIP?09:15
ifat_afekOf course, we will release again in a few weeks in the feature freeze09:15
e0neifat_afek: honestly, I just returned from PTO and didn't work with vitrage last week09:15
ifat_afekOk, thanks09:16
ifat_afekJust wanted to make sure everyone is ok with releasing today09:16
e0nehere is an important change https://review.openstack.org/#/c/627016/ to decouple tempest plugin from vitrage09:16
ifat_afekCool, I’ll review it today. Which reminds me that I may need to release the tempest plugin as well… will think about it09:17
e0neI'm going to remove vitrage.os_clients usage one by one in a separate change requests to make changes and reviews easier09:17
openstackgerritEyal proposed openstack/vitrage master: fix minor spelling mistakes  https://review.openstack.org/63749609:21
ifat_afeke0ne: sounds good09:22
openstackgerritMerged openstack/vitrage-tempest-plugin master: Do not use vitrage.utils.file module  https://review.openstack.org/63743209:33
openstackgerritMerged openstack/vitrage-tempest-plugin master: Do not use VitrageError exception  https://review.openstack.org/63744009:34
openstackgerrithewei proposed openstack/vitrage master: optimize aodh/ceilometer/kubernetes datasource  https://review.openstack.org/63748409:36
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Use Cinder API v3 by default.  https://review.openstack.org/63750409:41
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Use Cinder API v3 by default  https://review.openstack.org/63750409:42
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config in _get_session_for_user  https://review.openstack.org/63750910:10
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Do Not Review. Test tempest changes  https://review.openstack.org/61265210:12
openstackgerritEyal proposed openstack/vitrage master: fix minor spelling mistakes  https://review.openstack.org/63749610:13
openstackgerritEyal proposed openstack/vitrage master: use the right assertion  https://review.openstack.org/63751310:20
e0neifat_afek: according tempest-plugin refactoring10:20
e0neit's really hard to refactor it with backward compatibility10:22
e0nebecause we need to use config options from tempest.conf, not from vitrage.conf10:22
ifat_afekAre you sure we need backward compatibility? we are using an old tempest-plugin version for Rocky anyway10:22
e0neifat_afek: that's my suggestion. once my patches with refactoring will be merged, I propose to release vitrage-tempest-plugin v.2.010:23
ifat_afekOk, if it’s a huge change in the tempest it makes sense to release v2.010:24
e0nein such case, if anybody want to use old config - they could use v.1.110:24
ifat_afekwhy should people want to do that?10:25
e0neifat_afek: ok, it's good to get agreement for this :)10:25
ifat_afek:-)10:25
e0neifat_afek: not everybody use the latest versions :(10:25
ifat_afekok… but as I said we are using an older tempest for Rocky anyway, so I don’t see a problem here10:26
ifat_afekand we definitely want to make the change in the temepst10:26
e0neI agree with you10:27
*** ifat_afek has quit IRC10:27
openstackgerritNoor Yameen proposed openstack/vitrage-dashboard master: Support  Version 3 template format displaying on template view  https://review.openstack.org/63752110:43
*** ifat_afek has joined #openstack-vitrage10:56
*** ifat_afek has quit IRC10:59
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config in _get_session_for_user  https://review.openstack.org/63750911:27
*** ifat_afek has joined #openstack-vitrage11:37
openstackgerritIdan Hefetz proposed openstack/vitrage-tempest-plugin master: Test template v3 causing_alarm.  https://review.openstack.org/63752611:47
openstackgerritIdan Hefetz proposed openstack/vitrage master: Template v3 causing alarm implemented.  https://review.openstack.org/63752911:50
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config in _get_session_for_user  https://review.openstack.org/63750912:03
openstackgerritIdan Hefetz proposed openstack/vitrage master: Template v3 causing alarm implemented.  https://review.openstack.org/63752913:11
openstackgerritMerged openstack/vitrage-dashboard master: Make the alarm list in Vitrage Topology view  related to the selected entity  https://review.openstack.org/63348713:13
openstackgerritMerged openstack/vitrage master: use the right assertion  https://review.openstack.org/63751313:14
openstackgerritMerged openstack/vitrage-tempest-plugin master: Test template v3 causing_alarm.  https://review.openstack.org/63752613:14
*** ifat_afek has quit IRC13:19
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config in _get_session_for_user  https://review.openstack.org/63750913:23
openstackgerritMerged openstack/vitrage-dashboard master: Support  Version 3 template format displaying on template view  https://review.openstack.org/63752113:24
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use admin credentials for vitrageclient from tempest.conf  https://review.openstack.org/63754513:59
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Do not review. Tempest change test  https://review.openstack.org/63743314:00
openstackgerritIdan Hefetz proposed openstack/vitrage master: WIP - remove end messages  https://review.openstack.org/63755314:18
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config  https://review.openstack.org/62701614:43
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config in _get_session_for_user  https://review.openstack.org/63750914:43
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use admin credentials for vitrageclient from tempest.conf  https://review.openstack.org/63754514:43
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: WIP.Do not use keystoneclient from Vitrage  https://review.openstack.org/63755614:43
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: WIP. Do not use OpenStack services clients from Vitrage  https://review.openstack.org/63755714:43
openstackgerritEyal proposed openstack/vitrage master: fix minor spelling mistakes  https://review.openstack.org/63749614:54
*** e0ne has quit IRC15:42
*** e0ne has joined #openstack-vitrage15:48
openstackgerritMerged openstack/vitrage master: Template v3 causing alarm implemented.  https://review.openstack.org/63752915:55
*** eyalb1 has quit IRC15:55
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Do not review. Tempest change test  https://review.openstack.org/63743316:00
*** e0ne has quit IRC16:34
*** e0ne has joined #openstack-vitrage18:24
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config in _get_session_for_user  https://review.openstack.org/63750919:22
*** e0ne has quit IRC19:30
*** e0ne has joined #openstack-vitrage19:37
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use admin credentials for vitrageclient from tempest.conf  https://review.openstack.org/63754519:39
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: WIP.Do not use keystoneclient from Vitrage  https://review.openstack.org/63755619:39
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: WIP. Do not use OpenStack services clients from Vitrage  https://review.openstack.org/63755719:39
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Do not review. One more tempest change test  https://review.openstack.org/63761719:42
*** ifat_afek has joined #openstack-vitrage20:15
*** ifat_afek has quit IRC20:29
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Fix exception logging  https://review.openstack.org/63763121:03
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use Tempest config for user sessions  https://review.openstack.org/63750921:06
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Do not review. One more tempest change test  https://review.openstack.org/63761721:09
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use admin credentials for vitrageclient from tempest.conf  https://review.openstack.org/63754521:10
openstackgerritMerged openstack/vitrage master: Use Cinder API v3 by default  https://review.openstack.org/63750421:15
*** e0ne has quit IRC22:03
*** e0ne has joined #openstack-vitrage22:04
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: Use admin credentials for vitrageclient from tempest.conf  https://review.openstack.org/63754522:52
openstackgerritIvan Kolodyazhny proposed openstack/vitrage-tempest-plugin master: WIP.Do not use keystoneclient from Vitrage  https://review.openstack.org/63755622:53
openstackgerritIvan Kolodyazhny proposed openstack/vitrage master: Do not review. One more tempest change test  https://review.openstack.org/63761722:53
*** e0ne has quit IRC23:12
*** e0ne has joined #openstack-vitrage23:23
*** e0ne has quit IRC23:30
*** spsurya has quit IRC23:58

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