Thursday, 2017-04-27

*** hongbin has quit IRC00:33
*** Shunli has joined #openstack-zun00:55
openstackgerrithaobing proposed openstack/python-zunclient master: Fix confusing error message on interactive run  https://review.openstack.org/45909101:00
openstackgerritfeng.shengqin proposed openstack/python-zunclient master: Introduce API micro version  https://review.openstack.org/45959001:01
openstackgerritShunli Zhou proposed openstack/zun master: Add host operating system info to compute node  https://review.openstack.org/46036101:06
*** zhenguo has joined #openstack-zun01:19
*** vikasc has quit IRC01:46
*** vikasc has joined #openstack-zun01:49
*** yanyanhu has joined #openstack-zun01:50
*** zhurong has joined #openstack-zun02:02
openstackgerritMerged openstack/zun master: Optimize the link address  https://review.openstack.org/45998902:13
*** zhurong has quit IRC02:18
*** hongbin has joined #openstack-zun02:33
hongbinShunli: there?02:36
Shunlihi, hongbin02:36
hongbinShunli: hey02:37
hongbinfor your comments about port deletion02:37
ShunliI found that the kuryr-libnework deleted the port.02:37
hongbinyes, my best guess is your kuryr-libenetwork doesn't have this patch yet: https://review.openstack.org/#/c/441024/02:37
hongbinor it has this patch, but the port extension is disabled02:38
hongbinif it is the case, the fix in zun would be simply catching the portnotfound exception and ignore it02:38
Shunlione minutes, let me check the patch.02:39
openstackgerritWayne Chan proposed openstack/python-zunclient master: Optimize the link address  https://review.openstack.org/46037502:44
Shunliyes, do not have  https://review.openstack.org/#/c/441024/02:44
*** zhurong has joined #openstack-zun02:44
hongbini see02:44
hongbinthat is fine, zun needs to handle that02:45
ShunliBut the patch is for using existing port. I think my case does not use the existing port.02:46
ShunliSorry for not go through the patch totally.02:46
hongbinzun is using exsiting port02:46
hongbinin details02:46
hongbinzun will pre-create a neutron port, and pass the port to docker, docker pass it to kruyr02:47
ShunliI found when creating a container, a new port will show in neutron port-list02:47
ShunliOh, I see. for kuryr-libnetwork, it's an existing port, right?02:47
hongbinyes02:48
hongbinsee here: https://github.com/openstack/zun/blob/master/zun/network/kuryr_network.py#L12202:48
openstackgerritMerged openstack/zun master: Add docker-ce in check_docker and updated docker daemon with proxy configuration  https://review.openstack.org/45964302:49
hongbinzun used the client contex to create that port, then pass it to kuryr02:49
hongbinso from kuryr's point of view, it is an existing port02:49
hongbini don't want kuryr to create that port because kuryr-created port will belong to a different tenant02:50
hongbinif the port in differenent tenant, floating ip association is impossible02:50
ShunliOK, I will catch the portnotfound exception and ignore it.02:51
hongbinthx02:51
ShunliI may need to go through the https://github.com/openstack/zun/blob/master/zun/network/kuryr_network.py#L122 first :)02:51
hongbinthanks for catching this issue :002:51
hongbin:)02:51
Shunlinp :)02:52
*** caowei has quit IRC03:06
*** caowei has joined #openstack-zun03:14
*** zhurong has quit IRC03:42
*** caowei has quit IRC04:06
*** diga has joined #openstack-zun04:06
*** caowei has joined #openstack-zun04:27
*** hongbin has quit IRC04:37
*** adisky_ has joined #openstack-zun04:56
*** janki has joined #openstack-zun05:31
openstackgerritShunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist.  https://review.openstack.org/45997805:50
*** janki has quit IRC05:51
openstackgerritShunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist.  https://review.openstack.org/45997805:56
*** janki has joined #openstack-zun05:56
*** janki has quit IRC05:56
*** janki has joined #openstack-zun05:57
*** zhurong has joined #openstack-zun06:11
*** janki has quit IRC06:27
*** janki has joined #openstack-zun06:31
openstackgerritShunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist.  https://review.openstack.org/45997806:36
*** yanyanhu has quit IRC06:51
*** zhurong has quit IRC07:02
*** zhurong has joined #openstack-zun07:05
*** zhurong has quit IRC07:37
*** jchhatbar has joined #openstack-zun07:58
*** janki has quit IRC08:00
*** janki has joined #openstack-zun08:04
*** jchhatbar has quit IRC08:05
*** vikasc has quit IRC08:12
*** vikasc has joined #openstack-zun08:15
*** himani has quit IRC08:19
*** zhurong has joined #openstack-zun08:19
openstackgerritMerged openstack/zun master: Set 'addresses' field at an earlier stage  https://review.openstack.org/46032408:45
*** janki has quit IRC08:48
*** janki has joined #openstack-zun08:50
openstackgerritKevin Zhao proposed openstack/zun master: Add support for webproxy  https://review.openstack.org/45908608:54
*** himani has joined #openstack-zun08:56
openstackgerritKevin Zhao proposed openstack/zun master: Add support for webproxy  https://review.openstack.org/45908608:56
openstackgerritMerged openstack/zun master: Add host operating system info to compute node  https://review.openstack.org/46036108:56
openstackgerritMerged openstack/zun master: Avoiding setting container to error on list  https://review.openstack.org/46028908:56
openstackgerritMerged openstack/zun master: Set timeout on executing command in container  https://review.openstack.org/45908109:03
openstackgerritShunli Zhou proposed openstack/zun master: Fix zun delete fail as port not exist.  https://review.openstack.org/45997809:05
*** sharatss has joined #openstack-zun09:17
*** sharatss has left #openstack-zun09:18
openstackgerritMerged openstack/zun master: Persist to DB if the object has changed  https://review.openstack.org/45677509:19
*** Shunli has quit IRC10:05
*** kevinz has quit IRC10:19
*** zhurong has quit IRC10:20
*** zhurong has joined #openstack-zun10:27
*** cool06 has joined #openstack-zun10:31
*** diga has quit IRC10:48
*** caowei has quit IRC10:57
*** caowei has joined #openstack-zun10:57
*** caowei has quit IRC11:18
*** zhurong has quit IRC12:25
*** zhurong has joined #openstack-zun12:58
*** janki is now known as janki|afk13:16
*** catintheroof has joined #openstack-zun13:17
*** janki|afk is now known as janki13:46
*** adisky_ has quit IRC13:49
*** kevinz has joined #openstack-zun13:53
*** kevinz has quit IRC13:59
*** kevinz has joined #openstack-zun14:00
openstackgerritKevin Zhao proposed openstack/zun master: Add support for webproxy  https://review.openstack.org/45908614:02
*** hongbin has joined #openstack-zun14:13
*** cool06 has quit IRC14:15
*** jchhatbar has joined #openstack-zun14:35
*** janki has quit IRC14:36
*** zhurong has quit IRC14:39
openstackgerritHongbin Lu proposed openstack/zun master: Avoid raising error on 'force' deletion  https://review.openstack.org/46031114:57
*** kevinz has quit IRC15:07
*** kevinz has joined #openstack-zun15:49
*** jchhatbar has quit IRC15:53
*** kevinz has quit IRC15:54
openstackgerritMerged openstack/zun master: Add Apache License Content in index.rst  https://review.openstack.org/45527716:59
*** harlowja has quit IRC17:08
openstackgerritMerged openstack/zun master: Fix zun delete fail as port not exist.  https://review.openstack.org/45997817:18
*** lakerzhou has joined #openstack-zun17:23
*** harlowja has joined #openstack-zun17:51
*** lakerzhou has quit IRC18:33
*** lakerzhou has joined #openstack-zun18:34
*** openstackgerrit has quit IRC18:48
*** openstackgerrit has joined #openstack-zun21:22
openstackgerritHongbin Lu proposed openstack/zun master: Add more information to 'addresses' fields  https://review.openstack.org/45984621:22
*** lakerzhou has quit IRC21:37
*** catintheroof has quit IRC22:00
*** harlowja has quit IRC22:14
*** harlowja has joined #openstack-zun23:23
*** hongbin has quit IRC23:29
openstackgerritMerged openstack/python-zunclient master: Fix confusing error message on interactive run  https://review.openstack.org/45909123:49

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