Wednesday, 2018-08-08

« Tuesday, 2018-08-07 Index Thursday, 2018-08-09 »
*** longkb has joined #openstack-fwaas00:42
*** yushiro has joined #openstack-fwaas01:19
*** reedip has joined #openstack-fwaas01:33
*** bbbbzhao_ has joined #openstack-fwaas01:38
yushiroping hoangcx -c 101:45
hoangcxyushiro: pong01:45
yushirohoangcx, Hi.  Could you paste changelog again?01:46
hoangcxyushiro: Do you mean log of my discussion with infra team?01:47
yushirohoangcx, sorry,  I mean the log was build-sphinx-job01:47
hoangcxyushiro: http://logs.openstack.org/92/532792/27/check/build-openstack-sphinx-docs/2efad8e/job-output.txt.gz#_2018-08-07_23_13_58_33262801:48
hoangcxamotoki: ping, Do you have any idea why the job still failed and it is not reflected to latest neutron?01:49
hoangcxamotoki: I mean sphinx-doc job01:49
yushirohoangcx, thansk01:50
openstackgerritMerged openstack/neutron-fwaas master: Add notification callback events for FWaaS v2  https://review.openstack.org/57871802:25
openstackgerritMerged openstack/neutron-fwaas master: Add neutron as required-projects  https://review.openstack.org/58942102:47
*** yushiro has quit IRC03:48
*** yushiro has joined #openstack-fwaas04:06
openstackgerritNguyen Phuong An proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373804:19
*** yamamoto has joined #openstack-fwaas04:29
*** yamamoto has quit IRC04:46
*** yamamoto has joined #openstack-fwaas04:50
yushiroping annp04:54
yushirohoangcx, HI.  do you have any update?05:23
hoangcxyushiro: https://review.openstack.org/#/c/589708/05:27
hoangcxyushiro: Please wait, I'm still thinking and discussing05:28
yushirohoangcx, Looked. OK.05:28
* hoangcx enjoy lunch time05:28
yushiroPlease enjoy!05:28
reediplol05:35
reedipyushiro : https://review.openstack.org/#/c/440331/ ??05:42
yushiroreedip, Hi.  Ah, you need review ?  I think it's better to rebase it :)05:43
reedipok05:43
openstackgerritCao Xuan Hoang proposed openstack/neutron-fwaas master: Add log validator for FWaaS v2 logging  https://review.openstack.org/53279205:45
hoangcxyushiro: ^^ wait to see if the depended on works05:46
yushirohoangcx, thanks.  Can you paste super quick page for zuul job ?05:49
*** yamamoto has quit IRC05:49
hoangcxyushiro: http://zuul.openstack.org/05:49
hoangcxyushiro: it is still failed :(05:53
hoangcxyushiro: http://logs.openstack.org/92/532792/28/check/build-openstack-sphinx-docs/fb2bea5/job-output.txt.gz#_2018-08-08_05_48_49_93892205:57
hoangcxyushiro: This show that the patch already pulled latest master of neutron05:57
openstackgerritReedip proposed openstack/neutron-fwaas master: Add new protocols in Firewalls  https://review.openstack.org/44033105:58
reedipyushiro : done05:58
openstackgerritNguyen Phuong An proposed openstack/neutron-fwaas master: Add python binding for libnetfilter_log  https://review.openstack.org/53069406:04
yushiroreedip, OK, thanks!06:16
yushirohoangcx, Oh... OK.  So, we faced another bug ??06:17
yushiroannp, Is it really correct usage for @validators ?06:18
hoangcxyushiro: I'm still investigating06:18
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Add new protocols in Firewalls  https://review.openstack.org/44033106:20
yushirohoangcx, thank you06:20
annpyushiro, It's correct!06:20
yushiroannp, I see.06:23
yushiroreedip, I have 1 question.  This DCCP/SCTP work only iptables based impl, right ?06:27
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Add log validator for FWaaS v2 logging  https://review.openstack.org/53279206:32
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log]: Add rpc stuff for logging  https://review.openstack.org/53071506:32
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Adding resources callback handler for FWaaS v2 logging  https://review.openstack.org/58097606:32
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Add python binding for libnetfilter_log  https://review.openstack.org/53069406:32
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373806:32
reedipshouldnt it work on netfilter?06:46
yushirohoangcx, we cloned master branch of neutron on sphinx-docs job.  However, did this job run 'sudo python setup.py install' in neutron directory??06:55
yushirohoangcx, Did you discuss with infra team about that?  If not,  I'll try to ask about it.06:55
hoangcxyushiro: not yet06:55
yushirohoangcx, OK, I see.06:56
yushirohoangcx, did you try to modify stephenfin ?06:58
hoangcxyushiro: What do you mean?06:58
yushirohoangcx, Oh, sorry. I missed.06:58
hoangcxyushiro: np. Maybe you are right about 'sudo python setup.py install' for neutron06:59
yushiroSo, we need to run setup command before running apidoc...  How can we edit this job..07:05
hoangcxyushiro: another temporarily solution would be ignore sphinx-docs build for log path07:06
yushirohoangcx, Yes by adding ignore list..  I think it's OK for "temporary" solution.07:10
yushiroUnfortunately, Andreas looks busy and no reply from me.07:11
openstackgerritCao Xuan Hoang proposed openstack/neutron-fwaas master: Ignore sphinx-docs job for logapi path  https://review.openstack.org/58973507:15
yushiroquick :)07:17
openstackgerritCao Xuan Hoang proposed openstack/neutron-fwaas master: Add log validator for FWaaS v2 logging  https://review.openstack.org/53279207:18
yushiroHowever, hoangcx and annp  I tested my local env with adding ignore list for conf.py07:18
yushiroBut it failed with same exception.07:18
yushiroDid you success on your local env ?07:19
hoangcxyushiro: yes. Let me double check07:19
yushiroAh, I was typo 'logpi' :(07:19
hoangcxyushiro: :p07:20
yushirohoangcx, could you edit your commit message?  I'll comment it.07:21
yushiroon 30 seconds.07:21
hoangcxyushiro:   docs: commands succeeded07:21
hoangcx  congratulations :)07:21
hoangcxyushiro: double checked07:21
hoangcxyushiro: let wait to see what happen with zuul07:22
yushiroYes, of course. After that, please update.07:22
hoangcxyushiro: please comment on the gerrit07:23
hoangcxyushiro: I will update07:23
yushiroDONE07:23
yushiroreedip, I'm sorry for I was late reply.  Could you tell me your answer?07:24
reedipshouldnt it work on netfilter?07:26
yushiroreedip, sorry.  What's mean netfilter?  Does DCCP/SCTP work on OVS flow ?07:27
reedipThat I need to check ,....07:27
reedipfor now, iptables07:27
yushiroOK, if so, it's better to write commit msg.07:28
yushiroAnd, we should insert validation logic to avoid setting into L2 port of these rule.07:29
hoangcxyushiro: PASSED07:29
yushirohoangcx, I tested on my local env again and PASSED.07:29
yushiroOK, so please update commit msg.07:29
yushiroah, sorry hoangcx07:31
yushiroI forgot to mention to insert #TODO message in conf.py07:31
yushirodone07:32
openstackgerritCao Xuan Hoang proposed openstack/neutron-fwaas master: Adding 'logapi' in apidoc_excluded_paths for sphinx-docs job  https://review.openstack.org/58973507:35
hoangcxyushiro: DONE07:36
*** velizarx has joined #openstack-fwaas07:38
yushiroping longkb07:40
longkbo/ yushiro07:41
yushirolongkb, Regarding RPC stuff patch,07:41
yushiroArguments of _make_log_info_dict make me confuse.  What is 'ports_id' ??  port_ids  or port_id ??07:42
yushirolongkb, ports_id is starnge.07:43
longkbIt is ports_id, a list of port_id07:44
longkbThe format of log_info should follow: http://paste.openstack.org/show/727604/07:45
yushiroIf so, it seems 'a list of port IDs'.  It should be 'port_ids'.07:45
yushiroI'll comment on it.07:45
longkbOk :D Please do that07:46
longkbI will update the variable name in the next patch07:46
openstackgerritNguyen Phuong An proposed openstack/neutron-fwaas master: Add python binding for libnetfilter_log  https://review.openstack.org/53069407:48
yushirolongkb, DNE07:48
yushiroDONE07:48
longkbThanks, rushiro :D07:50
longkbI am writing UTs the patches now07:50
longkbPlease help me to check the main logic.07:51
*** velizarx has quit IRC07:56
yushiroannp, for libnetfilter_log patch,07:57
yushiroannp, plz ignore my comment.  I understood that libnetfilter_log doesn't exist on pip package.07:57
yushiroannp, however, why did you remove installing libnetfilter-log1 ?07:58
*** velizarx has joined #openstack-fwaas08:01
yushiroannp, Ah, OK I understood why you removed.08:03
annpyushiro, thanks for your understanding. :)08:05
yushirobecause we have centOS or RHEL08:07
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373808:08
yushirohoangcx, your patch looks passed.08:51
yushiroby watching zuul job08:52
hoangcxyushiro: Absolutely :)08:52
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log]: Add rpc stuff for logging  https://review.openstack.org/53071509:03
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Adding resources callback handler for FWaaS v2 logging  https://review.openstack.org/58097609:29
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log]: Add rpc stuff for logging  https://review.openstack.org/53071510:07
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Adding resources callback handler for FWaaS v2 logging  https://review.openstack.org/58097610:07
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Add python binding for libnetfilter_log  https://review.openstack.org/53069410:07
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373810:07
*** yushiro has quit IRC10:22
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Add log validator for FWaaS v2 logging  https://review.openstack.org/53279211:05
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: [log]: Add rpc stuff for logging  https://review.openstack.org/53071511:06
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Adding resources callback handler for FWaaS v2 logging  https://review.openstack.org/58097611:06
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Add python binding for libnetfilter_log  https://review.openstack.org/53069411:06
*** yushiro has joined #openstack-fwaas11:06
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373811:06
yushirohoangcx, Thanks for helping to pass sphinx-docs job!11:07
yushiroNow, I just rebase all patches and ready for test again.11:08
longkbthanks yushiro :)11:16
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log]: Add rpc stuff for logging  https://review.openstack.org/53071512:19
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Adding resources callback handler for FWaaS v2 logging  https://review.openstack.org/58097612:19
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Add python binding for libnetfilter_log  https://review.openstack.org/53069412:19
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373812:19
*** longkb has quit IRC12:20
*** velizarx has quit IRC12:52
*** velizarx has joined #openstack-fwaas12:56
yushiroxgerman_, thanks for review, temporary solution13:15
xgerman_yep, it’s messy though13:15
xgerman_we couldn’t pull a depends-on ?13:15
yushiroxgerman_, Yup. Depends-on and required-project don't work correctly regarding sphinx-docs job.13:16
xgerman_:-(13:16
yushiroI assume that we need 'sudo python setup.py install' after cloned neutron on sphinx-docs job.13:17
yushiroTomorrow, I'll ask about infra team :(13:17
yushiros/ :( / :) Haha13:18
yushiroannp, ping13:43
*** annp_ has joined #openstack-fwaas14:07
*** hongbin has joined #openstack-fwaas14:18
*** longkb has joined #openstack-fwaas14:20
openstackgerritMerged openstack/neutron-fwaas master: Adding 'logapi' in apidoc_excluded_paths for sphinx-docs job  https://review.openstack.org/58973514:24
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: Add python binding for libnetfilter_log  https://review.openstack.org/53069414:33
*** reedip is now known as reedip|afk14:35
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373815:03
*** velizarx has quit IRC15:20
*** annp_ has quit IRC17:39
*** longkb_ has joined #openstack-fwaas17:45
*** longkb has quit IRC17:47
*** annp_ has joined #openstack-fwaas17:51
openstackgerritOpenStack Release Bot proposed openstack/neutron-fwaas-dashboard master: Update reno for stable/rocky  https://review.openstack.org/58998217:54
*** annp_ has quit IRC18:05
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas master: Add releasenote for FWaaS v2 logging  https://review.openstack.org/58999118:34
*** yushiro has quit IRC18:35
openstackgerritKim Bao Long proposed openstack/neutron-fwaas master: [log] FWaaS L3 Logging driver based iptables  https://review.openstack.org/55373818:40
*** longkb_ has quit IRC18:50
*** SumitNaiksatam has joined #openstack-fwaas20:25
*** SumitNaiksatam has left #openstack-fwaas21:52
*** hongbin has quit IRC22:54
*** amotoki_ has joined #openstack-fwaas23:01
*** amotoki has quit IRC23:02
*** SridarK has joined #openstack-fwaas23:27
« Tuesday, 2018-08-07 Index Thursday, 2018-08-09 »

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