Thursday, 2016-08-18

*** tonanhngo has quit IRC00:07
*** sdake has joined #openstack-kuryr00:35
*** limao has joined #openstack-kuryr00:37
apuimedolimao: ping00:43
limaohello00:43
limaohi apuimedo00:44
openstackgerritLiping Mao proposed openstack/kuryr: Set up MTU docker interface  https://review.openstack.org/35571200:45
apuimedolimao: thanks for the mtu patches ;-)00:45
apuimedoI just posted two reviews00:45
apuimedoI wanted to ask you if you ran devstack lately00:46
limaoYeah, I addressed one00:46
apuimedocause I got a very funny error00:46
apuimedosaying that it was trying to install kuryr>=lib00:46
apuimedoI wonder if you saw it00:46
limaoYes00:46
limao1. Install Devstack without Kuryr.00:47
limao2. git clone kuryr-lib, and "sudo pip install ."00:47
limao3. git clone kuryr-libnetwor, delete "-e git+https://github.com/vikaschoudhary16/kuryr.git@drop_libnet_specific_code#egg=kuryr-lib" in requirement.txt, then and "sudo pip install ."00:47
limao4. check /usr/lib/docker/plugins/kuryr/kuryr.spec:00:47
limaohttp://127.0.0.1:2375000:47
limao5. add config in /etc/kuryr/kuryr.conf:00:47
limao[DEFAULT]00:47
limaobindir = /usr/libexec/kuryr00:47
limao5. start kuryr with "kuryr-libnetwork/scripts/run_server.py"00:47
limao        if port_active or (vif_plug_timeout > 0 and tries >= vif_plug_timeout):00:47
limaoI did not use devstack to install dev environment00:47
limaoCurrently when kuryr-libnetwork install00:47
limaoit will have problem because kuryr lib did not release00:47
apuimedoirenab: please, repeat the +2 on https://review.openstack.org/#/c/355712/500:48
apuimedoand set the w+100:48
limaoSo I install environment without kuryr first, then install kuryr-lib and kuryr-libnetwork by manual00:48
apuimedook, so still the same I had to do the last time00:49
limaoyeah00:49
apuimedoI thought this was fixed since I saw a patch in the logs for "fix devstack"00:49
apuimedobut I guess there's more things to fix00:49
limaoyeah...00:50
limaoIn fact, I feel a little bit worry about if we really should seperate kuryr-lib at this stage00:51
apuimedoit's certainly slowed us down00:52
apuimedobut I think in the end it will work out well00:52
limaoYeah00:53
limaomy feeling about kuryr-lib is that we should not put too many things into kuryr-lib right now. In fact, we only have kuryr-libnetwork using kuryr-lib right now. many of the things of kuryr-lib is we "thought" they may be will be used in kuryr-k8s then we put to in kuryr-lib, I check neutron-lib and find it only have very basic common function and do not have too much logical.00:54
apuimedofor me it's only the binding and the neutron client config00:55
apuimedothe rpc/rest is pushing it a bit for not having released yet00:55
limaoOK~ get it00:58
*** shashank_hegde has quit IRC00:58
limaobecause if we put too many things in kuryr-lib it will make it a little bit hard to modify something in it00:59
*** sdake has quit IRC00:59
limaothe things in kuryr-lib should be "stable" maybe00:59
apuimedoright01:00
apuimedoit should be like neutron-lib, something almost glacial01:00
limaoAgree, thanks apuimedo, i will also check devstack and docker image later01:02
apuimedolimao: I'm now asking in #openstack-infra about the devstack error01:02
apuimedolimao: about the docker image01:02
apuimedoI want to change it01:02
apuimedoand put the Dockerfile at the root of the repo01:02
apuimedoso that it builds the current state of the repo01:02
*** sdake has joined #openstack-kuryr01:02
limaook, cool01:03
apuimedoand not whatever is on github01:03
apuimedobut first I'm trying to sort out the devstack mess01:03
apuimedoand the configuration01:03
limaohttps://review.openstack.org/#/c/346717/01:03
apuimedovikasc sent https://review.openstack.org/#/c/350406/01:04
apuimedobut I think that it should be different01:04
apuimedoit should be that kuryr.lib has the config options, and then kuryr-libnetwork/config registers them01:04
apuimedoa bit like how neutron registers the plugins/ml2 options01:04
limaoYeah01:04
limaoagree01:05
limaowe should also let https://review.openstack.org/#/c/346717/ merged01:06
apuimedovikasc: please, do as Andreas says about https://review.openstack.org/#/c/346717/201:06
limaoit enable devstack01:06
apuimedoleave jenkins/jobs/kuryr.yaml without renaming01:06
apuimedoand just rename the job and service names01:07
apuimedowith that, I think we can get infra to merge it01:07
*** sdake has quit IRC01:11
openstackgerritLiping Mao proposed openstack/kuryr: Set up MTU docker interface  https://review.openstack.org/35571201:16
*** limao has quit IRC01:19
*** limao has joined #openstack-kuryr01:22
*** huikang has joined #openstack-kuryr01:30
apuimedolimao: regarding the /usr/libexec thing01:40
apuimedoI also prefer it in absolute01:40
limaoyeah01:40
apuimedobut there's some assumption that openstack projects setup.py do not install stuff in absolute location01:41
apuimedohowever01:41
apuimedowe could have the path hardcoded to /usr/libexec/kuryr01:41
apuimedoor rather made configurable without forcing pybasedir01:41
apuimedoand leave it up to the packaging/installing to put the scripts in the right location01:42
limao# Where to store Neutron state files.  This directory must be writable by the01:43
limao# user executing the agent.01:43
limao# state_path = /var/lib/neutron01:43
limaoI see a lot of abs dir in neutron01:44
limao# Paste configuration file01:45
limao# api_paste_config = /usr/share/neutron/api-paste.ini01:45
apuimedolimao: who installs that?01:46
apuimedosetup.py?01:46
apuimedoor manually01:46
limaoLet me check01:46
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr: fix python packaging  https://review.openstack.org/35682401:47
limao[root@ci91hf1ctr001 ~]# rpm -qf /usr/share/neutron/api-paste.ini01:54
limaoopenstack-neutron-2014.2.3-11.el7.noarch01:54
limaofor this case01:54
limaoit is done in rpm spec01:55
limao# Install dist conf01:55
limaoinstall -p -D -m 640 %{SOURCE40} %{buildroot}%{_datadir}/neutron/neutron-dist.conf01:55
limaomv %{buildroot}%{_sysconfdir}/neutron/api-paste.ini %{buildroot}%{_datadir}/neutron/api-paste.ini01:56
limaosetup.conf install this file in /etc/neutron/api-paste.ini (not /usr/share/neutron/api-paste.ini)02:02
apuimedook02:02
apuimedowe can do the same then ;-)02:02
apuimedothanks limao02:03
limaoSo, we can hard code the abs dir for bindir02:04
limaoin setup.conf we still install it02:04
limaowhen package, we should move the scripts into the abs dir02:05
limaoright?02:05
apuimedoright02:06
limaomaybe neutron will check both dir to find the config02:06
limaoif this, kuryr may also need to check both dir02:07
apuimedolimao: no, we should only check the one that is configured02:09
apuimedoit's the responsibility of packaging/installing to put the things on the right place then02:09
limaook~02:09
limaoget it02:10
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr: wheel: build universal wheels  https://review.openstack.org/35683002:11
apuimedolimao: it seems I fixed devstack :-)02:13
apuimedoat least, I was able to do `pip install .` without having to modify anyting or install kuryr-lib manually02:13
limaocool02:14
*** diogogmt has joined #openstack-kuryr02:17
*** diogogmt has quit IRC02:18
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-libnetwork: requirements: fix to point to upstream  https://review.openstack.org/35683102:22
apuimedolimao: can you try devstack with https://review.openstack.org/#/c/356831/02:23
limaosure02:23
apuimedoit should probably work02:23
limaoso can I try it with vagrant up?02:25
apuimedoI guess you can02:26
apuimedommm02:26
apuimedowait02:26
limaoaha02:26
apuimedodoesn't vagrant up use the latest master instead02:26
apuimedoof the current dir?02:26
apuimedoor does it mount02:26
apuimedolet me check02:26
apuimedoit mounts02:27
apuimedoso you should be able to use it02:27
apuimedogotta go to bed, it's 4:27 am02:27
limao!!! bye, I will try it later02:28
openstacklimao: Error: "!!" is not a valid command.02:28
limao;-) bye02:28
openstackgerritSongming Yan proposed openstack/kuryr: Remove execute permission which is added by mistake in midonet  https://review.openstack.org/34592102:32
*** shashank_hegde has joined #openstack-kuryr02:40
*** yuanying_ has quit IRC02:46
*** huikang has quit IRC02:47
*** huikang has joined #openstack-kuryr02:48
*** huikang has quit IRC02:52
*** yuanying has joined #openstack-kuryr02:52
*** limao has quit IRC03:11
*** huikang has joined #openstack-kuryr03:12
*** yuanying has quit IRC03:18
*** yuanying has joined #openstack-kuryr03:19
openstackgerritjanonymous proposed openstack/kuryr-libnetwork: Add option to fetch kuryr uri from env  https://review.openstack.org/35635703:29
*** yuanying has quit IRC03:42
*** yuanying has joined #openstack-kuryr03:42
*** yuanying has quit IRC03:43
*** huikang has quit IRC03:45
*** huikang has joined #openstack-kuryr03:46
*** janki has joined #openstack-kuryr03:46
*** yuanying has joined #openstack-kuryr03:48
*** huikang_ has joined #openstack-kuryr03:50
*** huikang has quit IRC03:50
vikascapuimedo,  read all the channel logs. Please ping me once you back. But since you were here till late, not expecting u will show up very soon :)04:21
*** tonanhngo has joined #openstack-kuryr04:26
openstackgerritMerged openstack/kuryr-libnetwork: vif_plugging_timeout = 0 should mean endless retry  https://review.openstack.org/35460005:13
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-libnetwork: Updated from global requirements  https://review.openstack.org/35197605:27
*** huikang_ has quit IRC06:06
*** yamamoto has quit IRC06:21
*** tonanhngo has quit IRC06:31
*** janki is now known as janki|afk06:46
*** janki|afk is now known as janki06:55
*** janki is now known as janki|afk06:58
*** yamamoto has joined #openstack-kuryr07:00
*** tonanhngo has joined #openstack-kuryr07:02
*** tonanhngo has quit IRC07:02
*** janki|afk is now known as janki07:25
*** yamamoto has quit IRC07:38
openstackgerritMerged openstack/kuryr: fix python packaging  https://review.openstack.org/35682407:41
openstackgerritMerged openstack/kuryr: wheel: build universal wheels  https://review.openstack.org/35683007:41
*** shashank_hegde has quit IRC07:46
*** pablochacin has joined #openstack-kuryr07:59
*** garyloug has joined #openstack-kuryr08:02
apuimedovikasc: pong08:18
vikascapuimedo, hi08:18
apuimedoI'm only half awake08:18
apuimedoso go easy on me :P08:18
apuimedothanks for updating the dvsm patch08:19
vikascapuimedo, what i understood from conversation logs is that you want all config in kuryr-lib08:19
apuimedoyolanda already approved it08:19
vikascapuimedo, :)08:19
apuimedovikasc: no, I want only the neutron and whatever is common in kuryr-lib08:19
apuimedoand then, in kuryr-libnetwork/kuryr_libnetwork/config08:19
apuimedoI want the module to register those options08:19
apuimedoand for them to be listed08:19
vikascapuimedo, ok08:20
apuimedoso that the conf file gets properly generated08:20
vikascapuimedo, so currently when config.py of kuryr-lib is imported, options get registered08:20
apuimedogsagie: please, approve and merge https://review.openstack.org/#/c/355712/ :-)08:20
apuimedoright, my idea was that they only get declared in kuryr-lib08:21
apuimedobut it is then registered by kuryr-libnetwork / kuryr-kubernetes08:21
openstackgerritMerged openstack/kuryr-libnetwork: requirements: fix to point to upstream  https://review.openstack.org/35683108:22
apuimedogotta go for a a lot of hours now08:22
vikascapuimedo,  should we do this by import lib config in kuryr-libnetwork config08:22
vikasc?08:22
apuimedovikasc: my idea was to do it like this08:23
vikascapuimedo,  i think i got your point.. dont want to eat your brain more :)08:24
apuimedohttps://github.com/openstack/neutron/blob/master/neutron/common/config.py08:24
apuimedoas you can see there, it imports from neutron.conf08:24
apuimedoand goes and registers the things one by one08:24
apuimedoI think that's a nice way to do it08:25
apuimedothe way nova does it is quite nice too08:25
*** garyloug has quit IRC08:25
*** garyloug has joined #openstack-kuryr08:25
vikascapuimedo, thanks for the pointers08:25
apuimedothanks for the reviews and the nice morning patches :-)08:26
vikasc:)08:26
apuimedovikasc: keep an eye on #openstack-infra08:26
openstackgerritMerged openstack/kuryr: Set up MTU docker interface  https://review.openstack.org/35571208:26
apuimedoif you see some core being active08:26
vikascapuimedo, sure08:26
apuimedopoint them to your pach08:27
apuimedo*patch08:27
apuimedoso the job gets fixed08:27
vikascsure08:27
apuimedovikasc: also, now that my patches have been merged08:28
apuimedocould you try to run plain devstack08:28
vikascapuimedo, will do08:28
apuimedoto see if all is fixed? If it is not, send me an email with the stack trace and I'll fix it when I go back home08:28
apuimedothanks!08:28
vikascapuimedo, thanks!08:28
apuimedo:-)08:29
apuimedook, as the last thing before I'm dragged by my ears to the mountains08:33
apuimedoI uploaded the lates kuryr-lib master wheel to pypi08:34
apuimedohttps://pypi.python.org/pypi/kuryr-lib08:34
apuimedogsagie: can you give me co-ownership over https://pypi.python.org/pypi/kuryr-libnetwork ?08:35
apuimedoI want to push an updated wheel too08:35
gsagieapuimedo: let me find out how08:36
apuimedothanks08:36
apuimedoI see that you are an owner with gal-sagie username08:36
apuimedoalso openstackci08:36
gsagiesec08:36
gsagieforgot my pass08:36
gsagiewhat is your username?08:38
gsagieapuimedo08:39
apuimedogsagie: celebdor08:39
*** yamamoto has joined #openstack-kuryr08:39
gsagieadded you08:40
apuimedothanks a lot gsagie08:42
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-libnetwork: wheels: support building universal wheels  https://review.openstack.org/35701408:42
apuimedogsagie: irenab: https://review.openstack.org/35701408:42
apuimedoI forgot in the night to fix this for kuryr-libnetwork too08:42
*** limao has joined #openstack-kuryr08:46
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-libnetwork: versioning: let pbr handle versioning from tags  https://review.openstack.org/35701908:50
apuimedogsagie: irenab: vikasc: ^^08:50
apuimedoand now I'm already being dragged out by the ears08:50
apuimedotalk to you all later08:50
*** yamamoto has quit IRC08:52
gsagiecya08:53
vikascapuimedo,  bbye tc08:54
*** yamamoto has joined #openstack-kuryr08:56
*** yamamoto has quit IRC08:56
*** yamamoto has joined #openstack-kuryr08:56
openstackgerritLiping Mao proposed openstack/kuryr: Add UT for binding  https://review.openstack.org/35645408:57
*** yamamoto has quit IRC09:10
*** yamamoto has joined #openstack-kuryr09:12
*** yamamoto has quit IRC09:12
*** yamamoto has joined #openstack-kuryr09:30
*** yamamoto has quit IRC09:30
*** yamamoto has joined #openstack-kuryr09:30
*** yamamoto has quit IRC09:30
*** yamamoto has joined #openstack-kuryr09:31
*** yamamoto has quit IRC09:36
*** janki is now known as janki|afk09:47
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-libnetwork: Updated from global requirements  https://review.openstack.org/35197609:51
*** janki|afk has quit IRC09:52
*** yamamoto has joined #openstack-kuryr10:01
*** yamamoto has quit IRC10:01
*** yamamoto has joined #openstack-kuryr10:32
*** yamamoto has quit IRC10:37
*** yamamoto has joined #openstack-kuryr10:41
openstackgerritLiping Mao proposed openstack/kuryr: Add UT for binding  https://review.openstack.org/35645410:44
*** janki|afk has joined #openstack-kuryr10:56
*** oanson has joined #openstack-kuryr11:00
*** janki|afk has quit IRC11:01
*** janki has joined #openstack-kuryr11:01
*** oanson has quit IRC11:09
*** tonanhngo has joined #openstack-kuryr12:05
*** tonanhngo has quit IRC12:07
*** garyloug has quit IRC12:40
*** janki has quit IRC12:58
*** oanson has joined #openstack-kuryr13:08
*** garyloug has joined #openstack-kuryr13:16
vikascirenab, hello13:25
*** hongbin has joined #openstack-kuryr13:41
*** janki has joined #openstack-kuryr13:42
*** limao has quit IRC13:56
*** limao has joined #openstack-kuryr13:58
*** limao has quit IRC13:58
*** tonanhngo has joined #openstack-kuryr14:00
*** tonanhngo has quit IRC14:01
openstackgerritMerged openstack/kuryr: Enable DeprecationWarning in test environments  https://review.openstack.org/35317914:02
*** janki has quit IRC14:05
*** salv-orlando has joined #openstack-kuryr15:00
*** yamamoto has quit IRC15:01
*** yamamoto has joined #openstack-kuryr15:04
*** yamamoto has quit IRC15:09
*** sdake_ has joined #openstack-kuryr15:29
*** janki has joined #openstack-kuryr15:50
*** oanson has quit IRC16:03
*** yamamoto has joined #openstack-kuryr16:06
*** yamamoto has quit IRC16:11
*** shashank_hegde has joined #openstack-kuryr16:14
*** sdake_ has quit IRC16:16
*** banix has joined #openstack-kuryr16:25
*** pablochacin has quit IRC16:27
openstackgerritMerged openstack/kuryr-libnetwork: versioning: let pbr handle versioning from tags  https://review.openstack.org/35701916:42
*** sdake has joined #openstack-kuryr16:43
apuimedovikasc: gsagie: irenab: banix: I pushed a dev release of kuryr-libnetwork to pypi16:46
apuimedovikasc: did the devstack work?16:46
openstackgerritMerged openstack/kuryr-libnetwork: Remove pep8 exclusion for VMware  https://review.openstack.org/35322816:54
*** shashank_hegde has quit IRC16:54
*** shashank_hegde has joined #openstack-kuryr17:15
*** banix has quit IRC17:28
*** shashank_hegde has quit IRC17:28
*** shashank_hegde has joined #openstack-kuryr17:48
*** salv-orl_ has joined #openstack-kuryr19:15
*** salv-orlando has quit IRC19:18
*** janki has quit IRC19:21
*** huikang has joined #openstack-kuryr19:55
*** tonanhngo has joined #openstack-kuryr20:02
*** diogogmt has joined #openstack-kuryr20:27
*** yamamoto has joined #openstack-kuryr21:09
*** yamamoto has quit IRC21:09
*** pablochacin has joined #openstack-kuryr21:37
*** huikang has quit IRC22:06
*** huikang has joined #openstack-kuryr22:06
*** huikang has quit IRC22:11
*** huikang has joined #openstack-kuryr22:13
*** yamamoto has joined #openstack-kuryr22:42
*** huikang has quit IRC22:48
*** huikang has joined #openstack-kuryr22:48
*** huikang has quit IRC22:53
*** vikasc has quit IRC22:54
*** vikasc has joined #openstack-kuryr22:59
*** hongbin has quit IRC23:09
*** sdake_ has joined #openstack-kuryr23:12
*** sdake has quit IRC23:13
*** garyloug has quit IRC23:17

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