*** hongbin has quit IRC | 00:33 | |
*** Shunli has joined #openstack-zun | 00:55 | |
openstackgerrit | haobing proposed openstack/python-zunclient master: Fix confusing error message on interactive run https://review.openstack.org/459091 | 01:00 |
---|---|---|
openstackgerrit | feng.shengqin proposed openstack/python-zunclient master: Introduce API micro version https://review.openstack.org/459590 | 01:01 |
openstackgerrit | Shunli Zhou proposed openstack/zun master: Add host operating system info to compute node https://review.openstack.org/460361 | 01:06 |
*** zhenguo has joined #openstack-zun | 01:19 | |
*** vikasc has quit IRC | 01:46 | |
*** vikasc has joined #openstack-zun | 01:49 | |
*** yanyanhu has joined #openstack-zun | 01:50 | |
*** zhurong has joined #openstack-zun | 02:02 | |
openstackgerrit | Merged openstack/zun master: Optimize the link address https://review.openstack.org/459989 | 02:13 |
*** zhurong has quit IRC | 02:18 | |
*** hongbin has joined #openstack-zun | 02:33 | |
hongbin | Shunli: there? | 02:36 |
Shunli | hi, hongbin | 02:36 |
hongbin | Shunli: hey | 02:37 |
hongbin | for your comments about port deletion | 02:37 |
Shunli | I found that the kuryr-libnework deleted the port. | 02:37 |
hongbin | yes, my best guess is your kuryr-libenetwork doesn't have this patch yet: https://review.openstack.org/#/c/441024/ | 02:37 |
hongbin | or it has this patch, but the port extension is disabled | 02:38 |
hongbin | if it is the case, the fix in zun would be simply catching the portnotfound exception and ignore it | 02:38 |
Shunli | one minutes, let me check the patch. | 02:39 |
openstackgerrit | Wayne Chan proposed openstack/python-zunclient master: Optimize the link address https://review.openstack.org/460375 | 02:44 |
Shunli | yes, do not have https://review.openstack.org/#/c/441024/ | 02:44 |
*** zhurong has joined #openstack-zun | 02:44 | |
hongbin | i see | 02:44 |
hongbin | that is fine, zun needs to handle that | 02:45 |
Shunli | But the patch is for using existing port. I think my case does not use the existing port. | 02:46 |
Shunli | Sorry for not go through the patch totally. | 02:46 |
hongbin | zun is using exsiting port | 02:46 |
hongbin | in details | 02:46 |
hongbin | zun will pre-create a neutron port, and pass the port to docker, docker pass it to kruyr | 02:47 |
Shunli | I found when creating a container, a new port will show in neutron port-list | 02:47 |
Shunli | Oh, I see. for kuryr-libnetwork, it's an existing port, right? | 02:47 |
hongbin | yes | 02:48 |
hongbin | see here: https://github.com/openstack/zun/blob/master/zun/network/kuryr_network.py#L122 | 02:48 |
openstackgerrit | Merged openstack/zun master: Add docker-ce in check_docker and updated docker daemon with proxy configuration https://review.openstack.org/459643 | 02:49 |
hongbin | zun used the client contex to create that port, then pass it to kuryr | 02:49 |
hongbin | so from kuryr's point of view, it is an existing port | 02:49 |
hongbin | i don't want kuryr to create that port because kuryr-created port will belong to a different tenant | 02:50 |
hongbin | if the port in differenent tenant, floating ip association is impossible | 02:50 |
Shunli | OK, I will catch the portnotfound exception and ignore it. | 02:51 |
hongbin | thx | 02:51 |
Shunli | I may need to go through the https://github.com/openstack/zun/blob/master/zun/network/kuryr_network.py#L122 first :) | 02:51 |
hongbin | thanks for catching this issue :0 | 02:51 |
hongbin | :) | 02:51 |
Shunli | np :) | 02:52 |
*** caowei has quit IRC | 03:06 | |
*** caowei has joined #openstack-zun | 03:14 | |
*** zhurong has quit IRC | 03:42 | |
*** caowei has quit IRC | 04:06 | |
*** diga has joined #openstack-zun | 04:06 | |
*** caowei has joined #openstack-zun | 04:27 | |
*** hongbin has quit IRC | 04:37 | |
*** adisky_ has joined #openstack-zun | 04:56 | |
*** janki has joined #openstack-zun | 05:31 | |
openstackgerrit | Shunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist. https://review.openstack.org/459978 | 05:50 |
*** janki has quit IRC | 05:51 | |
openstackgerrit | Shunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist. https://review.openstack.org/459978 | 05:56 |
*** janki has joined #openstack-zun | 05:56 | |
*** janki has quit IRC | 05:56 | |
*** janki has joined #openstack-zun | 05:57 | |
*** zhurong has joined #openstack-zun | 06:11 | |
*** janki has quit IRC | 06:27 | |
*** janki has joined #openstack-zun | 06:31 | |
openstackgerrit | Shunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist. https://review.openstack.org/459978 | 06:36 |
*** yanyanhu has quit IRC | 06:51 | |
*** zhurong has quit IRC | 07:02 | |
*** zhurong has joined #openstack-zun | 07:05 | |
*** zhurong has quit IRC | 07:37 | |
*** jchhatbar has joined #openstack-zun | 07:58 | |
*** janki has quit IRC | 08:00 | |
*** janki has joined #openstack-zun | 08:04 | |
*** jchhatbar has quit IRC | 08:05 | |
*** vikasc has quit IRC | 08:12 | |
*** vikasc has joined #openstack-zun | 08:15 | |
*** himani has quit IRC | 08:19 | |
*** zhurong has joined #openstack-zun | 08:19 | |
openstackgerrit | Merged openstack/zun master: Set 'addresses' field at an earlier stage https://review.openstack.org/460324 | 08:45 |
*** janki has quit IRC | 08:48 | |
*** janki has joined #openstack-zun | 08:50 | |
openstackgerrit | Kevin Zhao proposed openstack/zun master: Add support for webproxy https://review.openstack.org/459086 | 08:54 |
*** himani has joined #openstack-zun | 08:56 | |
openstackgerrit | Kevin Zhao proposed openstack/zun master: Add support for webproxy https://review.openstack.org/459086 | 08:56 |
openstackgerrit | Merged openstack/zun master: Add host operating system info to compute node https://review.openstack.org/460361 | 08:56 |
openstackgerrit | Merged openstack/zun master: Avoiding setting container to error on list https://review.openstack.org/460289 | 08:56 |
openstackgerrit | Merged openstack/zun master: Set timeout on executing command in container https://review.openstack.org/459081 | 09:03 |
openstackgerrit | Shunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist. https://review.openstack.org/459978 | 09:05 |
*** sharatss has joined #openstack-zun | 09:17 | |
*** sharatss has left #openstack-zun | 09:18 | |
openstackgerrit | Merged openstack/zun master: Persist to DB if the object has changed https://review.openstack.org/456775 | 09:19 |
*** Shunli has quit IRC | 10:05 | |
*** kevinz has quit IRC | 10:19 | |
*** zhurong has quit IRC | 10:20 | |
*** zhurong has joined #openstack-zun | 10:27 | |
*** cool06 has joined #openstack-zun | 10:31 | |
*** diga has quit IRC | 10:48 | |
*** caowei has quit IRC | 10:57 | |
*** caowei has joined #openstack-zun | 10:57 | |
*** caowei has quit IRC | 11:18 | |
*** zhurong has quit IRC | 12:25 | |
*** zhurong has joined #openstack-zun | 12:58 | |
*** janki is now known as janki|afk | 13:16 | |
*** catintheroof has joined #openstack-zun | 13:17 | |
*** janki|afk is now known as janki | 13:46 | |
*** adisky_ has quit IRC | 13:49 | |
*** kevinz has joined #openstack-zun | 13:53 | |
*** kevinz has quit IRC | 13:59 | |
*** kevinz has joined #openstack-zun | 14:00 | |
openstackgerrit | Kevin Zhao proposed openstack/zun master: Add support for webproxy https://review.openstack.org/459086 | 14:02 |
*** hongbin has joined #openstack-zun | 14:13 | |
*** cool06 has quit IRC | 14:15 | |
*** jchhatbar has joined #openstack-zun | 14:35 | |
*** janki has quit IRC | 14:36 | |
*** zhurong has quit IRC | 14:39 | |
openstackgerrit | Hongbin Lu proposed openstack/zun master: Avoid raising error on 'force' deletion https://review.openstack.org/460311 | 14:57 |
*** kevinz has quit IRC | 15:07 | |
*** kevinz has joined #openstack-zun | 15:49 | |
*** jchhatbar has quit IRC | 15:53 | |
*** kevinz has quit IRC | 15:54 | |
openstackgerrit | Merged openstack/zun master: Add Apache License Content in index.rst https://review.openstack.org/455277 | 16:59 |
*** harlowja has quit IRC | 17:08 | |
openstackgerrit | Merged openstack/zun master: Fix zun delete fail as port not exist. https://review.openstack.org/459978 | 17:18 |
*** lakerzhou has joined #openstack-zun | 17:23 | |
*** harlowja has joined #openstack-zun | 17:51 | |
*** lakerzhou has quit IRC | 18:33 | |
*** lakerzhou has joined #openstack-zun | 18:34 | |
*** openstackgerrit has quit IRC | 18:48 | |
*** openstackgerrit has joined #openstack-zun | 21:22 | |
openstackgerrit | Hongbin Lu proposed openstack/zun master: Add more information to 'addresses' fields https://review.openstack.org/459846 | 21:22 |
*** lakerzhou has quit IRC | 21:37 | |
*** catintheroof has quit IRC | 22:00 | |
*** harlowja has quit IRC | 22:14 | |
*** harlowja has joined #openstack-zun | 23:23 | |
*** hongbin has quit IRC | 23:29 | |
openstackgerrit | Merged openstack/python-zunclient master: Fix confusing error message on interactive run https://review.openstack.org/459091 | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!