Friday, 2017-06-30

*** amotoki is now known as amotoki_away00:17
*** amotoki_away is now known as amotoki00:18
*** amotoki is now known as amotoki_away00:27
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Remove log translations  https://review.openstack.org/47910200:50
*** cuongnv has joined #openstack-fwaas00:58
*** amotoki_away is now known as amotoki01:23
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Remove log translations  https://review.openstack.org/47910201:56
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Handle rehome of firewall exceptions  https://review.openstack.org/47836302:39
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Use API Definitions from Neutron-Lib  https://review.openstack.org/47888302:42
reedipyushiro :good morning :)  I have made the neutron-lib ON TOP of the exceptions patch ( you can see it on the right hand side ) Thats why I didnt need the Depends-On02:44
yushiroreedip, oh, sorry.  I didn't noticed.02:46
reedipwas there a pep8 error ? I didnt see it yesterday :P)02:46
reedipno issues yushiro02:47
yushiroreedip, yes, 1 nit pep8 error.  just fixed.02:47
reedipLets first complete the neutron-lib and fwaas patch for exception, then we will push the neutron-lib migration and midonet patch02:47
yushiroreedip, OK, thanks.02:47
reedipyushiro : but we need to change the parent revision for neutron-lib02:48
openstackgerritReedip proposed openstack/neutron-fwaas master: Use API Definitions from Neutron-Lib  https://review.openstack.org/47888302:49
reedipdone :)02:49
yushirothanks02:51
openstackgerritVan Hung Pham proposed openstack/neutron-fwaas master: Switch from oslosphinx to openstackdocstheme  https://review.openstack.org/47875002:59
*** vks1 has joined #openstack-fwaas03:01
*** yamamoto has joined #openstack-fwaas03:05
reedipyushiro : the fwaas patches cannot understand neutron_lib.exceptions03:45
reedipseems like the depends-on didnt work03:45
yushiroyes, something wrong?  but it looks depends-on doesn't work correctly.03:46
yushiroI'm checking now03:46
yushiroabout that.03:46
yushirohttp://logs.openstack.org/02/479102/2/check/gate-neutron-fwaas-v2-dsvm-tempest/ee4ed95/logs/devstack-early.txt.gz03:49
yushiroGITREPO["neutron-lib"]=git://git.openstack.org/openstack/neutron-lib.git03:49
yushiroGITBRANCH["neutron-lib"]=master03:49
yushirowow03:50
yamamotohi, do you want me help something?  i just saw a few occurrences of "midonet" in the backlog.03:53
reedipyamamoto : we are trying to migrate to neutron-lib again .04:08
reedipand trying to check its impact on midonet04:08
reedipdont want things to be broken like they were earlier :)04:08
*** vks1 has quit IRC04:49
*** vks1 has joined #openstack-fwaas05:06
yamamotoreedip: did you figure out "race in midonet"? did you file a bug?05:33
reedipyamamoto : no, I didnt. I will, if the new patch fails . I didnt think it to be useful once the revert was complete05:36
reedipyamamoto san , can you confirm if this issue is still occuring ? https://bugs.launchpad.net/networking-midonet/+bug/170045405:37
openstackLaunchpad bug 1700454 in networking-midonet ""Quota exceeded for instances" in fwaas tempest tests" [Critical,In progress] - Assigned to YAMAMOTO Takashi (yamamoto)05:37
reedipafter the neutron-lib patch was reverted for fwaas05:37
reedipjust want to make sure that the neutron-lib change was not a reason for this bug05:37
yushiroreedip, Hi.  about fwaas exception in neutron-lib05:57
yushiroreedip, I got a comment from akihiro with some concering about separating v1 and v2.05:58
yushiro"Don't we need to have separate exception modules for FWaaS v1 and v2? I am afraid that consumers of the exception modules might be confused and FWaaS v1/v2 exceptions are used in a mixed way.05:59
amotokiyushiro: reedip: I am not sure how much v1 and v2 has common stuffs... my comment is generic one, so I am not sure my comment is reasonable or not.06:00
yushiroamotoki, OK. but I'd like to consider your comment.  Though v1 will be deprecated, currently it is still maintained.  So, I think we should separate file(e.g. fwaas_v1.py, fwaas_v2.py in neutron_lib/exceptions)06:02
reedipamotoki : I think that the exception messages are pretty generic which can be used by both v1 and v2 . However, the exceptions raised by fwaas_V1 and fwaas_v2 implementation are different.so yes, combining them together may cause the new consumers of fwaas some effort to read the actual fwaas implementation06:12
yushiroreedip, I see.  I'll update it.06:12
amotokireedip: if the expected usages are differnt between v1 and v2, i think it is better to be separate module.06:13
openstackgerritRitesh proposed openstack/neutron-fwaas master: Replace the usage of 'admin_manager' with 'os_admin'  https://review.openstack.org/47918306:49
*** vks1 has quit IRC07:19
reedipamotoki: the exceptions defined for fwaas v1 , can be used by fwaas v2 as well. the exceptions are generic.07:35
amotokireedip: if the usage (the condition where an individual exception is raised) is exactly same, you can share both versions.07:36
amotokireedip: if the usage is different and you re-use existing one, i do no think it is a good idea.07:36
amotokiit is based on the fact the exception description defines exact confidition when an exception is raised.07:37
yamamotoreedip: i haven't seen the symptom laterly. maybe it's fixed in somewhere. i don't think it's related to the fwaas revert.08:46
*** yamamoto has quit IRC09:35
*** yamamoto has joined #openstack-fwaas09:36
*** yamamoto has quit IRC09:38
*** yamamoto has joined #openstack-fwaas09:42
*** yamamoto has quit IRC09:42
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas-dashboard master: Imported Translations from Zanata  https://review.openstack.org/47923009:44
*** vks1 has joined #openstack-fwaas09:45
*** yamamoto has joined #openstack-fwaas09:46
*** yamamoto has quit IRC09:51
*** vks1 has quit IRC10:00
*** trungnv has quit IRC10:03
*** trungnv has joined #openstack-fwaas10:05
*** yamamoto has joined #openstack-fwaas10:10
*** yamamoto has quit IRC10:12
*** yamamoto has joined #openstack-fwaas10:13
*** yamamoto has quit IRC10:16
*** yamamoto has joined #openstack-fwaas10:16
*** yamamoto has quit IRC10:18
*** yushiro has quit IRC10:33
*** yamamoto has joined #openstack-fwaas10:52
*** cuongnv has quit IRC10:54
*** amotoki is now known as amotoki_away11:11
openstackgerritRitesh proposed openstack/neutron-fwaas master: Drop MANIFEST.in - it's not needed by pbr  https://review.openstack.org/47926011:15
openstackgerritMerged openstack/neutron-fwaas master: use service type constants from neutron_lib plugins  https://review.openstack.org/47824011:32
*** yamamoto has quit IRC11:34
*** yamamoto has joined #openstack-fwaas11:36
openstackgerritMerged openstack/neutron-fwaas master: Enable to set neutron's local repository for testing  https://review.openstack.org/46839811:44
*** yamamoto has quit IRC11:58
*** amotoki_away is now known as amotoki12:19
*** yamamoto has joined #openstack-fwaas12:26
openstackgerritMerged openstack/neutron-fwaas master: Replace the usage of 'manager' with 'os_primary'  https://review.openstack.org/47632912:43
*** yamamoto has quit IRC12:55
*** yamamoto has joined #openstack-fwaas12:56
*** yamamoto has quit IRC13:01
*** yamamoto has joined #openstack-fwaas13:58
*** reedip_ has joined #openstack-fwaas13:59
*** amotoki is now known as amotoki_away14:04
*** yamamoto has quit IRC14:10
*** reedip_ has quit IRC14:18
*** vks1 has joined #openstack-fwaas14:22
*** vks1 has quit IRC14:50
*** vks1 has joined #openstack-fwaas15:04
*** lnicolas has quit IRC15:24
*** Tim_Eberhard has joined #openstack-fwaas15:57
*** reedip_ has joined #openstack-fwaas17:35
*** vks1 has quit IRC18:02
*** reedip_ has quit IRC18:29
*** Tim_Eberhard has quit IRC21:08
*** Tim_Eberhard has joined #openstack-fwaas22:29
*** Tim_Eberhard has quit IRC22:41
*** Tim_Eberhard has joined #openstack-fwaas23:27
*** lnicolas has joined #openstack-fwaas23:45
*** Tim_Eberhard has quit IRC23:50

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