*** tonanhngo has quit IRC | 00:07 | |
*** sdake has joined #openstack-kuryr | 00:35 | |
*** limao has joined #openstack-kuryr | 00:37 | |
apuimedo | limao: ping | 00:43 |
---|---|---|
limao | hello | 00:43 |
limao | hi apuimedo | 00:44 |
openstackgerrit | Liping Mao proposed openstack/kuryr: Set up MTU docker interface https://review.openstack.org/355712 | 00:45 |
apuimedo | limao: thanks for the mtu patches ;-) | 00:45 |
apuimedo | I just posted two reviews | 00:45 |
apuimedo | I wanted to ask you if you ran devstack lately | 00:46 |
limao | Yeah, I addressed one | 00:46 |
apuimedo | cause I got a very funny error | 00:46 |
apuimedo | saying that it was trying to install kuryr>=lib | 00:46 |
apuimedo | I wonder if you saw it | 00:46 |
limao | Yes | 00:46 |
limao | 1. Install Devstack without Kuryr. | 00:47 |
limao | 2. git clone kuryr-lib, and "sudo pip install ." | 00:47 |
limao | 3. 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 |
limao | 4. check /usr/lib/docker/plugins/kuryr/kuryr.spec: | 00:47 |
limao | http://127.0.0.1:23750 | 00:47 |
limao | 5. add config in /etc/kuryr/kuryr.conf: | 00:47 |
limao | [DEFAULT] | 00:47 |
limao | bindir = /usr/libexec/kuryr | 00:47 |
limao | 5. 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 |
limao | I did not use devstack to install dev environment | 00:47 |
limao | Currently when kuryr-libnetwork install | 00:47 |
limao | it will have problem because kuryr lib did not release | 00:47 |
apuimedo | irenab: please, repeat the +2 on https://review.openstack.org/#/c/355712/5 | 00:48 |
apuimedo | and set the w+1 | 00:48 |
limao | So I install environment without kuryr first, then install kuryr-lib and kuryr-libnetwork by manual | 00:48 |
apuimedo | ok, so still the same I had to do the last time | 00:49 |
limao | yeah | 00:49 |
apuimedo | I thought this was fixed since I saw a patch in the logs for "fix devstack" | 00:49 |
apuimedo | but I guess there's more things to fix | 00:49 |
limao | yeah... | 00:50 |
limao | In fact, I feel a little bit worry about if we really should seperate kuryr-lib at this stage | 00:51 |
apuimedo | it's certainly slowed us down | 00:52 |
apuimedo | but I think in the end it will work out well | 00:52 |
limao | Yeah | 00:53 |
limao | my 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 |
apuimedo | for me it's only the binding and the neutron client config | 00:55 |
apuimedo | the rpc/rest is pushing it a bit for not having released yet | 00:55 |
limao | OK~ get it | 00:58 |
*** shashank_hegde has quit IRC | 00:58 | |
limao | because if we put too many things in kuryr-lib it will make it a little bit hard to modify something in it | 00:59 |
*** sdake has quit IRC | 00:59 | |
limao | the things in kuryr-lib should be "stable" maybe | 00:59 |
apuimedo | right | 01:00 |
apuimedo | it should be like neutron-lib, something almost glacial | 01:00 |
limao | Agree, thanks apuimedo, i will also check devstack and docker image later | 01:02 |
apuimedo | limao: I'm now asking in #openstack-infra about the devstack error | 01:02 |
apuimedo | limao: about the docker image | 01:02 |
apuimedo | I want to change it | 01:02 |
apuimedo | and put the Dockerfile at the root of the repo | 01:02 |
apuimedo | so that it builds the current state of the repo | 01:02 |
*** sdake has joined #openstack-kuryr | 01:02 | |
limao | ok, cool | 01:03 |
apuimedo | and not whatever is on github | 01:03 |
apuimedo | but first I'm trying to sort out the devstack mess | 01:03 |
apuimedo | and the configuration | 01:03 |
limao | https://review.openstack.org/#/c/346717/ | 01:03 |
apuimedo | vikasc sent https://review.openstack.org/#/c/350406/ | 01:04 |
apuimedo | but I think that it should be different | 01:04 |
apuimedo | it should be that kuryr.lib has the config options, and then kuryr-libnetwork/config registers them | 01:04 |
apuimedo | a bit like how neutron registers the plugins/ml2 options | 01:04 |
limao | Yeah | 01:04 |
limao | agree | 01:05 |
limao | we should also let https://review.openstack.org/#/c/346717/ merged | 01:06 |
apuimedo | vikasc: please, do as Andreas says about https://review.openstack.org/#/c/346717/2 | 01:06 |
limao | it enable devstack | 01:06 |
apuimedo | leave jenkins/jobs/kuryr.yaml without renaming | 01:06 |
apuimedo | and just rename the job and service names | 01:07 |
apuimedo | with that, I think we can get infra to merge it | 01:07 |
*** sdake has quit IRC | 01:11 | |
openstackgerrit | Liping Mao proposed openstack/kuryr: Set up MTU docker interface https://review.openstack.org/355712 | 01:16 |
*** limao has quit IRC | 01:19 | |
*** limao has joined #openstack-kuryr | 01:22 | |
*** huikang has joined #openstack-kuryr | 01:30 | |
apuimedo | limao: regarding the /usr/libexec thing | 01:40 |
apuimedo | I also prefer it in absolute | 01:40 |
limao | yeah | 01:40 |
apuimedo | but there's some assumption that openstack projects setup.py do not install stuff in absolute location | 01:41 |
apuimedo | however | 01:41 |
apuimedo | we could have the path hardcoded to /usr/libexec/kuryr | 01:41 |
apuimedo | or rather made configurable without forcing pybasedir | 01:41 |
apuimedo | and leave it up to the packaging/installing to put the scripts in the right location | 01:42 |
limao | # Where to store Neutron state files. This directory must be writable by the | 01:43 |
limao | # user executing the agent. | 01:43 |
limao | # state_path = /var/lib/neutron | 01:43 |
limao | I see a lot of abs dir in neutron | 01:44 |
limao | # Paste configuration file | 01:45 |
limao | # api_paste_config = /usr/share/neutron/api-paste.ini | 01:45 |
apuimedo | limao: who installs that? | 01:46 |
apuimedo | setup.py? | 01:46 |
apuimedo | or manually | 01:46 |
limao | Let me check | 01:46 |
openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr: fix python packaging https://review.openstack.org/356824 | 01:47 |
limao | [root@ci91hf1ctr001 ~]# rpm -qf /usr/share/neutron/api-paste.ini | 01:54 |
limao | openstack-neutron-2014.2.3-11.el7.noarch | 01:54 |
limao | for this case | 01:54 |
limao | it is done in rpm spec | 01:55 |
limao | # Install dist conf | 01:55 |
limao | install -p -D -m 640 %{SOURCE40} %{buildroot}%{_datadir}/neutron/neutron-dist.conf | 01:55 |
limao | mv %{buildroot}%{_sysconfdir}/neutron/api-paste.ini %{buildroot}%{_datadir}/neutron/api-paste.ini | 01:56 |
limao | setup.conf install this file in /etc/neutron/api-paste.ini (not /usr/share/neutron/api-paste.ini) | 02:02 |
apuimedo | ok | 02:02 |
apuimedo | we can do the same then ;-) | 02:02 |
apuimedo | thanks limao | 02:03 |
limao | So, we can hard code the abs dir for bindir | 02:04 |
limao | in setup.conf we still install it | 02:04 |
limao | when package, we should move the scripts into the abs dir | 02:05 |
limao | right? | 02:05 |
apuimedo | right | 02:06 |
limao | maybe neutron will check both dir to find the config | 02:06 |
limao | if this, kuryr may also need to check both dir | 02:07 |
apuimedo | limao: no, we should only check the one that is configured | 02:09 |
apuimedo | it's the responsibility of packaging/installing to put the things on the right place then | 02:09 |
limao | ok~ | 02:09 |
limao | get it | 02:10 |
openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr: wheel: build universal wheels https://review.openstack.org/356830 | 02:11 |
apuimedo | limao: it seems I fixed devstack :-) | 02:13 |
apuimedo | at least, I was able to do `pip install .` without having to modify anyting or install kuryr-lib manually | 02:13 |
limao | cool | 02:14 |
*** diogogmt has joined #openstack-kuryr | 02:17 | |
*** diogogmt has quit IRC | 02:18 | |
openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-libnetwork: requirements: fix to point to upstream https://review.openstack.org/356831 | 02:22 |
apuimedo | limao: can you try devstack with https://review.openstack.org/#/c/356831/ | 02:23 |
limao | sure | 02:23 |
apuimedo | it should probably work | 02:23 |
limao | so can I try it with vagrant up? | 02:25 |
apuimedo | I guess you can | 02:26 |
apuimedo | mmm | 02:26 |
apuimedo | wait | 02:26 |
limao | aha | 02:26 |
apuimedo | doesn't vagrant up use the latest master instead | 02:26 |
apuimedo | of the current dir? | 02:26 |
apuimedo | or does it mount | 02:26 |
apuimedo | let me check | 02:26 |
apuimedo | it mounts | 02:27 |
apuimedo | so you should be able to use it | 02:27 |
apuimedo | gotta go to bed, it's 4:27 am | 02:27 |
limao | !!! bye, I will try it later | 02:28 |
openstack | limao: Error: "!!" is not a valid command. | 02:28 |
limao | ;-) bye | 02:28 |
openstackgerrit | Songming Yan proposed openstack/kuryr: Remove execute permission which is added by mistake in midonet https://review.openstack.org/345921 | 02:32 |
*** shashank_hegde has joined #openstack-kuryr | 02:40 | |
*** yuanying_ has quit IRC | 02:46 | |
*** huikang has quit IRC | 02:47 | |
*** huikang has joined #openstack-kuryr | 02:48 | |
*** huikang has quit IRC | 02:52 | |
*** yuanying has joined #openstack-kuryr | 02:52 | |
*** limao has quit IRC | 03:11 | |
*** huikang has joined #openstack-kuryr | 03:12 | |
*** yuanying has quit IRC | 03:18 | |
*** yuanying has joined #openstack-kuryr | 03:19 | |
openstackgerrit | janonymous proposed openstack/kuryr-libnetwork: Add option to fetch kuryr uri from env https://review.openstack.org/356357 | 03:29 |
*** yuanying has quit IRC | 03:42 | |
*** yuanying has joined #openstack-kuryr | 03:42 | |
*** yuanying has quit IRC | 03:43 | |
*** huikang has quit IRC | 03:45 | |
*** huikang has joined #openstack-kuryr | 03:46 | |
*** janki has joined #openstack-kuryr | 03:46 | |
*** yuanying has joined #openstack-kuryr | 03:48 | |
*** huikang_ has joined #openstack-kuryr | 03:50 | |
*** huikang has quit IRC | 03:50 | |
vikasc | apuimedo, 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-kuryr | 04:26 | |
openstackgerrit | Merged openstack/kuryr-libnetwork: vif_plugging_timeout = 0 should mean endless retry https://review.openstack.org/354600 | 05:13 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/kuryr-libnetwork: Updated from global requirements https://review.openstack.org/351976 | 05:27 |
*** huikang_ has quit IRC | 06:06 | |
*** yamamoto has quit IRC | 06:21 | |
*** tonanhngo has quit IRC | 06:31 | |
*** janki is now known as janki|afk | 06:46 | |
*** janki|afk is now known as janki | 06:55 | |
*** janki is now known as janki|afk | 06:58 | |
*** yamamoto has joined #openstack-kuryr | 07:00 | |
*** tonanhngo has joined #openstack-kuryr | 07:02 | |
*** tonanhngo has quit IRC | 07:02 | |
*** janki|afk is now known as janki | 07:25 | |
*** yamamoto has quit IRC | 07:38 | |
openstackgerrit | Merged openstack/kuryr: fix python packaging https://review.openstack.org/356824 | 07:41 |
openstackgerrit | Merged openstack/kuryr: wheel: build universal wheels https://review.openstack.org/356830 | 07:41 |
*** shashank_hegde has quit IRC | 07:46 | |
*** pablochacin has joined #openstack-kuryr | 07:59 | |
*** garyloug has joined #openstack-kuryr | 08:02 | |
apuimedo | vikasc: pong | 08:18 |
vikasc | apuimedo, hi | 08:18 |
apuimedo | I'm only half awake | 08:18 |
apuimedo | so go easy on me :P | 08:18 |
apuimedo | thanks for updating the dvsm patch | 08:19 |
vikasc | apuimedo, what i understood from conversation logs is that you want all config in kuryr-lib | 08:19 |
apuimedo | yolanda already approved it | 08:19 |
vikasc | apuimedo, :) | 08:19 |
apuimedo | vikasc: no, I want only the neutron and whatever is common in kuryr-lib | 08:19 |
apuimedo | and then, in kuryr-libnetwork/kuryr_libnetwork/config | 08:19 |
apuimedo | I want the module to register those options | 08:19 |
apuimedo | and for them to be listed | 08:19 |
vikasc | apuimedo, ok | 08:20 |
apuimedo | so that the conf file gets properly generated | 08:20 |
vikasc | apuimedo, so currently when config.py of kuryr-lib is imported, options get registered | 08:20 |
apuimedo | gsagie: please, approve and merge https://review.openstack.org/#/c/355712/ :-) | 08:20 |
apuimedo | right, my idea was that they only get declared in kuryr-lib | 08:21 |
apuimedo | but it is then registered by kuryr-libnetwork / kuryr-kubernetes | 08:21 |
openstackgerrit | Merged openstack/kuryr-libnetwork: requirements: fix to point to upstream https://review.openstack.org/356831 | 08:22 |
apuimedo | gotta go for a a lot of hours now | 08:22 |
vikasc | apuimedo, should we do this by import lib config in kuryr-libnetwork config | 08:22 |
vikasc | ? | 08:22 |
apuimedo | vikasc: my idea was to do it like this | 08:23 |
vikasc | apuimedo, i think i got your point.. dont want to eat your brain more :) | 08:24 |
apuimedo | https://github.com/openstack/neutron/blob/master/neutron/common/config.py | 08:24 |
apuimedo | as you can see there, it imports from neutron.conf | 08:24 |
apuimedo | and goes and registers the things one by one | 08:24 |
apuimedo | I think that's a nice way to do it | 08:25 |
apuimedo | the way nova does it is quite nice too | 08:25 |
*** garyloug has quit IRC | 08:25 | |
*** garyloug has joined #openstack-kuryr | 08:25 | |
vikasc | apuimedo, thanks for the pointers | 08:25 |
apuimedo | thanks for the reviews and the nice morning patches :-) | 08:26 |
vikasc | :) | 08:26 |
apuimedo | vikasc: keep an eye on #openstack-infra | 08:26 |
openstackgerrit | Merged openstack/kuryr: Set up MTU docker interface https://review.openstack.org/355712 | 08:26 |
apuimedo | if you see some core being active | 08:26 |
vikasc | apuimedo, sure | 08:26 |
apuimedo | point them to your pach | 08:27 |
apuimedo | *patch | 08:27 |
apuimedo | so the job gets fixed | 08:27 |
vikasc | sure | 08:27 |
apuimedo | vikasc: also, now that my patches have been merged | 08:28 |
apuimedo | could you try to run plain devstack | 08:28 |
vikasc | apuimedo, will do | 08:28 |
apuimedo | to 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 home | 08:28 |
apuimedo | thanks! | 08:28 |
vikasc | apuimedo, thanks! | 08:28 |
apuimedo | :-) | 08:29 |
apuimedo | ok, as the last thing before I'm dragged by my ears to the mountains | 08:33 |
apuimedo | I uploaded the lates kuryr-lib master wheel to pypi | 08:34 |
apuimedo | https://pypi.python.org/pypi/kuryr-lib | 08:34 |
apuimedo | gsagie: can you give me co-ownership over https://pypi.python.org/pypi/kuryr-libnetwork ? | 08:35 |
apuimedo | I want to push an updated wheel too | 08:35 |
gsagie | apuimedo: let me find out how | 08:36 |
apuimedo | thanks | 08:36 |
apuimedo | I see that you are an owner with gal-sagie username | 08:36 |
apuimedo | also openstackci | 08:36 |
gsagie | sec | 08:36 |
gsagie | forgot my pass | 08:36 |
gsagie | what is your username? | 08:38 |
gsagie | apuimedo | 08:39 |
apuimedo | gsagie: celebdor | 08:39 |
*** yamamoto has joined #openstack-kuryr | 08:39 | |
gsagie | added you | 08:40 |
apuimedo | thanks a lot gsagie | 08:42 |
openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-libnetwork: wheels: support building universal wheels https://review.openstack.org/357014 | 08:42 |
apuimedo | gsagie: irenab: https://review.openstack.org/357014 | 08:42 |
apuimedo | I forgot in the night to fix this for kuryr-libnetwork too | 08:42 |
*** limao has joined #openstack-kuryr | 08:46 | |
openstackgerrit | Antoni Segura Puimedon proposed openstack/kuryr-libnetwork: versioning: let pbr handle versioning from tags https://review.openstack.org/357019 | 08:50 |
apuimedo | gsagie: irenab: vikasc: ^^ | 08:50 |
apuimedo | and now I'm already being dragged out by the ears | 08:50 |
apuimedo | talk to you all later | 08:50 |
*** yamamoto has quit IRC | 08:52 | |
gsagie | cya | 08:53 |
vikasc | apuimedo, bbye tc | 08:54 |
*** yamamoto has joined #openstack-kuryr | 08:56 | |
*** yamamoto has quit IRC | 08:56 | |
*** yamamoto has joined #openstack-kuryr | 08:56 | |
openstackgerrit | Liping Mao proposed openstack/kuryr: Add UT for binding https://review.openstack.org/356454 | 08:57 |
*** yamamoto has quit IRC | 09:10 | |
*** yamamoto has joined #openstack-kuryr | 09:12 | |
*** yamamoto has quit IRC | 09:12 | |
*** yamamoto has joined #openstack-kuryr | 09:30 | |
*** yamamoto has quit IRC | 09:30 | |
*** yamamoto has joined #openstack-kuryr | 09:30 | |
*** yamamoto has quit IRC | 09:30 | |
*** yamamoto has joined #openstack-kuryr | 09:31 | |
*** yamamoto has quit IRC | 09:36 | |
*** janki is now known as janki|afk | 09:47 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/kuryr-libnetwork: Updated from global requirements https://review.openstack.org/351976 | 09:51 |
*** janki|afk has quit IRC | 09:52 | |
*** yamamoto has joined #openstack-kuryr | 10:01 | |
*** yamamoto has quit IRC | 10:01 | |
*** yamamoto has joined #openstack-kuryr | 10:32 | |
*** yamamoto has quit IRC | 10:37 | |
*** yamamoto has joined #openstack-kuryr | 10:41 | |
openstackgerrit | Liping Mao proposed openstack/kuryr: Add UT for binding https://review.openstack.org/356454 | 10:44 |
*** janki|afk has joined #openstack-kuryr | 10:56 | |
*** oanson has joined #openstack-kuryr | 11:00 | |
*** janki|afk has quit IRC | 11:01 | |
*** janki has joined #openstack-kuryr | 11:01 | |
*** oanson has quit IRC | 11:09 | |
*** tonanhngo has joined #openstack-kuryr | 12:05 | |
*** tonanhngo has quit IRC | 12:07 | |
*** garyloug has quit IRC | 12:40 | |
*** janki has quit IRC | 12:58 | |
*** oanson has joined #openstack-kuryr | 13:08 | |
*** garyloug has joined #openstack-kuryr | 13:16 | |
vikasc | irenab, hello | 13:25 |
*** hongbin has joined #openstack-kuryr | 13:41 | |
*** janki has joined #openstack-kuryr | 13:42 | |
*** limao has quit IRC | 13:56 | |
*** limao has joined #openstack-kuryr | 13:58 | |
*** limao has quit IRC | 13:58 | |
*** tonanhngo has joined #openstack-kuryr | 14:00 | |
*** tonanhngo has quit IRC | 14:01 | |
openstackgerrit | Merged openstack/kuryr: Enable DeprecationWarning in test environments https://review.openstack.org/353179 | 14:02 |
*** janki has quit IRC | 14:05 | |
*** salv-orlando has joined #openstack-kuryr | 15:00 | |
*** yamamoto has quit IRC | 15:01 | |
*** yamamoto has joined #openstack-kuryr | 15:04 | |
*** yamamoto has quit IRC | 15:09 | |
*** sdake_ has joined #openstack-kuryr | 15:29 | |
*** janki has joined #openstack-kuryr | 15:50 | |
*** oanson has quit IRC | 16:03 | |
*** yamamoto has joined #openstack-kuryr | 16:06 | |
*** yamamoto has quit IRC | 16:11 | |
*** shashank_hegde has joined #openstack-kuryr | 16:14 | |
*** sdake_ has quit IRC | 16:16 | |
*** banix has joined #openstack-kuryr | 16:25 | |
*** pablochacin has quit IRC | 16:27 | |
openstackgerrit | Merged openstack/kuryr-libnetwork: versioning: let pbr handle versioning from tags https://review.openstack.org/357019 | 16:42 |
*** sdake has joined #openstack-kuryr | 16:43 | |
apuimedo | vikasc: gsagie: irenab: banix: I pushed a dev release of kuryr-libnetwork to pypi | 16:46 |
apuimedo | vikasc: did the devstack work? | 16:46 |
openstackgerrit | Merged openstack/kuryr-libnetwork: Remove pep8 exclusion for VMware https://review.openstack.org/353228 | 16:54 |
*** shashank_hegde has quit IRC | 16:54 | |
*** shashank_hegde has joined #openstack-kuryr | 17:15 | |
*** banix has quit IRC | 17:28 | |
*** shashank_hegde has quit IRC | 17:28 | |
*** shashank_hegde has joined #openstack-kuryr | 17:48 | |
*** salv-orl_ has joined #openstack-kuryr | 19:15 | |
*** salv-orlando has quit IRC | 19:18 | |
*** janki has quit IRC | 19:21 | |
*** huikang has joined #openstack-kuryr | 19:55 | |
*** tonanhngo has joined #openstack-kuryr | 20:02 | |
*** diogogmt has joined #openstack-kuryr | 20:27 | |
*** yamamoto has joined #openstack-kuryr | 21:09 | |
*** yamamoto has quit IRC | 21:09 | |
*** pablochacin has joined #openstack-kuryr | 21:37 | |
*** huikang has quit IRC | 22:06 | |
*** huikang has joined #openstack-kuryr | 22:06 | |
*** huikang has quit IRC | 22:11 | |
*** huikang has joined #openstack-kuryr | 22:13 | |
*** yamamoto has joined #openstack-kuryr | 22:42 | |
*** huikang has quit IRC | 22:48 | |
*** huikang has joined #openstack-kuryr | 22:48 | |
*** huikang has quit IRC | 22:53 | |
*** vikasc has quit IRC | 22:54 | |
*** vikasc has joined #openstack-kuryr | 22:59 | |
*** hongbin has quit IRC | 23:09 | |
*** sdake_ has joined #openstack-kuryr | 23:12 | |
*** sdake has quit IRC | 23:13 | |
*** garyloug has quit IRC | 23:17 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!