*** jwcroppe has joined #openstack-zun | 00:22 | |
*** zhurong has joined #openstack-zun | 00:28 | |
*** kevinz has joined #openstack-zun | 01:22 | |
*** yanyanhu has joined #openstack-zun | 01:23 | |
*** zhangjl has joined #openstack-zun | 01:36 | |
*** Wenzhi has joined #openstack-zun | 01:53 | |
caowei | hi all | 02:00 |
---|---|---|
*** jianyi has joined #openstack-zun | 02:01 | |
jianyi | hi caowei | 02:01 |
*** zhangjl has left #openstack-zun | 02:10 | |
*** zhangjl has joined #openstack-zun | 02:23 | |
*** hongbin_ has joined #openstack-zun | 02:33 | |
*** jianyi has quit IRC | 03:16 | |
*** zhurong has quit IRC | 03:39 | |
*** prameswar has joined #openstack-zun | 03:45 | |
*** hongbin_ has quit IRC | 03:55 | |
*** hongbin_ has joined #openstack-zun | 04:21 | |
*** diga has joined #openstack-zun | 04:21 | |
*** pksingh has joined #openstack-zun | 04:28 | |
*** sudipto has joined #openstack-zun | 04:29 | |
*** sudipto_ has joined #openstack-zun | 04:29 | |
*** gb21 has joined #openstack-zun | 04:53 | |
*** pksingh has quit IRC | 05:07 | |
*** hongbin_ has quit IRC | 05:13 | |
*** sudipto has quit IRC | 05:14 | |
*** sudipto_ has quit IRC | 05:14 | |
openstackgerrit | Merged openstack/zun: Support multiple compute hosts https://review.openstack.org/415554 | 05:14 |
*** adisky_ has joined #openstack-zun | 05:18 | |
openstackgerrit | guo yunxian proposed openstack/zun: Remove unused function https://review.openstack.org/419280 | 05:19 |
*** pksingh has joined #openstack-zun | 05:46 | |
openstackgerrit | Kevin Zhao proposed openstack/python-zunclient: Add support for restart policy when create/run https://review.openstack.org/419288 | 05:51 |
openstackgerrit | prameswar proposed openstack/zun: [WIP] Added support for Auto remove the container https://review.openstack.org/418942 | 05:54 |
openstackgerrit | Kevin Zhao proposed openstack/zun: Add support for container restart policy in server side https://review.openstack.org/419289 | 05:54 |
*** sudipto has joined #openstack-zun | 05:58 | |
*** sudipto_ has joined #openstack-zun | 05:58 | |
openstackgerrit | prameswar proposed openstack/python-zunclient: [WIP] Autoremove container added to zun run client https://review.openstack.org/418943 | 05:58 |
openstackgerrit | Pradeep Kumar Singh proposed openstack/zun: Remove the Container class https://review.openstack.org/417658 | 06:00 |
*** zhurong has joined #openstack-zun | 06:23 | |
*** pksingh has quit IRC | 06:47 | |
*** janki has joined #openstack-zun | 06:49 | |
openstackgerrit | Wenzhi Yu proposed openstack/zun: Add etcd DB backend support for multiple compute env https://review.openstack.org/419302 | 06:57 |
openstackgerrit | Wenzhi Yu proposed openstack/zun: Add etcd DB backend support for multiple compute env https://review.openstack.org/419302 | 07:00 |
*** pksingh has joined #openstack-zun | 07:16 | |
openstackgerrit | Kevin Zhao proposed openstack/zun: Remove redundant Connection Class in zun/db/api.py https://review.openstack.org/419310 | 07:20 |
*** janki has quit IRC | 07:22 | |
*** jchhatbar has joined #openstack-zun | 07:22 | |
openstackgerrit | Wenzhi Yu proposed openstack/zun: Add etcd DB backend support for multiple compute env https://review.openstack.org/419302 | 07:29 |
*** sudipto has quit IRC | 07:36 | |
*** sudipto_ has quit IRC | 07:36 | |
*** hongbin has quit IRC | 08:16 | |
*** diga has quit IRC | 08:26 | |
*** strigazi_AFK is now known as strigazi | 08:28 | |
*** sudipto_ has joined #openstack-zun | 08:31 | |
*** sudipto has joined #openstack-zun | 08:31 | |
*** caowei has quit IRC | 08:39 | |
*** jwcroppe has quit IRC | 08:41 | |
*** jwcroppe has joined #openstack-zun | 08:42 | |
*** Guest16073 is now known as jayaura | 08:45 | |
*** jwcroppe has quit IRC | 08:46 | |
*** prameswar has quit IRC | 09:03 | |
pksingh | zhangjl: Hello | 09:14 |
zhangjl | pksingh:hello | 09:14 |
pksingh | zhangjl: https://bugs.launchpad.net/zun/+bug/1655868, nice catch :) | 09:14 |
openstack | Launchpad bug 1655868 in Zun "API cannot work as expected because of the boolean params" [High,Confirmed] - Assigned to zhangjialong (zhangjl) | 09:14 |
zhangjl | pksingh:thanks | 09:14 |
pksingh | You need to convert the query parameters to Boolean, like below | 09:15 |
pksingh | https://github.com/openstack/zun/blob/master/zun/api/controllers/v1/containers.py#L337 | 09:15 |
zhangjl | pksingh:yes | 09:15 |
pksingh | zhangjl: ^^, i think above should work | 09:15 |
zhangjl | pksingh:i working like above | 09:15 |
pksingh | zhangjl: please submit the patch when you have time, thanks for reporting | 09:15 |
pksingh | zhangjl: we missed it in review :( | 09:16 |
zhangjl | pksingh:my pleasure | 09:16 |
adisky_ | hi all, i was working on the bug https://bugs.launchpad.net/zun/+bug/1654010, i need some suggestions from team to imporve status field of zun list command | 09:21 |
openstack | Launchpad bug 1654010 in Zun "Improve the container status" [Medium,Triaged] - Assigned to aditi sharma (adi-sky17) | 09:21 |
pksingh | yes adisky_ , what can we help? | 09:25 |
adisky_ | There are two possible solutions which i can think of | 09:26 |
adisky_ | 1) currently we are using inspect_container to get the list of containers from docker which does not returns from how long it has been running or stopped, we can calculate it with the other timestamp details returned. | 09:26 |
adisky_ | 2) we can use 'containers' from docker api to directly get the status from how long it is being running or stopped | 09:26 |
adisky_ | for second approach we need to call docker api twice fro conatiner list operation | 09:27 |
adisky_ | one more query should we save status_detail to database, as this field will keep on changing?? | 09:28 |
pksingh | let me check the code | 09:28 |
adisky_ | @pksingh thnx | 09:28 |
pksingh | https://github.com/openstack/zun/blob/master/zun/container/docker/driver.py#L119, inspect command gives result of the state | 09:31 |
pksingh | adisky_: see https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/ | 09:32 |
pksingh | adisky_: https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/#/inspect-a-container | 09:32 |
pksingh | adisky_: i think using that u can derive the state | 09:32 |
adisky_ | ya... the 1 st approach we should derive it | 09:34 |
adisky_ | pksingh : https://github.com/openstack/zun/blob/master/zun/compute/manager.py | 09:35 |
adisky_ | i think the method container_list is not used anywhere | 09:36 |
pksingh | you need to add status_detail in method https://github.com/openstack/zun/blob/master/zun/container/docker/driver.py#L129, | 09:37 |
adisky_ | ya i know.. | 09:38 |
adisky_ | i m just asking, has this method is not being used anywhere we should remove it?? | 09:39 |
pksingh | adisky_: i need to check, | 09:40 |
adisky_ | ok | 09:41 |
adisky_ | thnx for your help :) | 09:41 |
pksingh | adisky_: yes its not being used | 09:42 |
pksingh | adisky_: if you want more information then plz check https://github.com/docker/docker/blob/master/cli/command/formatter/container.go | 09:42 |
pksingh | adisky_: may be there you can get, its the docker client code where its format the output | 09:42 |
*** zhurong has quit IRC | 09:44 | |
adisky_ | ok i will look on to it, thnx :) | 09:45 |
*** Wenzhi has quit IRC | 09:52 | |
*** yanyanhu has quit IRC | 09:54 | |
openstackgerrit | zhangjl proposed openstack/zun: Ensure api can work as expected with boolean query params https://review.openstack.org/419373 | 09:55 |
*** zhangjl has quit IRC | 10:02 | |
*** prameswar has joined #openstack-zun | 10:04 | |
*** kevinz has quit IRC | 10:11 | |
*** zhurong has joined #openstack-zun | 10:25 | |
*** janki has joined #openstack-zun | 10:28 | |
*** jchhatbar has quit IRC | 10:28 | |
*** Wenzhi has joined #openstack-zun | 11:05 | |
*** Wenzhi has quit IRC | 11:07 | |
*** zhurong has quit IRC | 11:15 | |
*** sudipto has quit IRC | 11:22 | |
*** sudipto_ has quit IRC | 11:22 | |
*** pksingh has quit IRC | 11:25 | |
*** sheel has joined #openstack-zun | 12:19 | |
*** gb21 has quit IRC | 13:00 | |
*** sudipto_ has joined #openstack-zun | 13:37 | |
*** sudipto has joined #openstack-zun | 13:37 | |
*** janki has quit IRC | 14:35 | |
*** hongbin has joined #openstack-zun | 14:59 | |
*** sheel has quit IRC | 15:07 | |
*** jwcroppe has joined #openstack-zun | 15:28 | |
*** sudipto has quit IRC | 16:05 | |
*** sudipto_ has quit IRC | 16:05 | |
*** sudipto_ has joined #openstack-zun | 16:05 | |
*** sudipto has joined #openstack-zun | 16:05 | |
hongbin | hi folks, sorry i left the office without logging out my IRC. if you sent me private messages in the last 12 hours, i possibly missed that. | 16:14 |
*** sudswas__ has joined #openstack-zun | 17:07 | |
*** sudipto_ has quit IRC | 17:07 | |
*** sudipto has quit IRC | 17:08 | |
*** sudipto has joined #openstack-zun | 17:08 | |
*** sheel has joined #openstack-zun | 17:10 | |
*** openstackgerrit has quit IRC | 18:33 | |
*** prameswar has quit IRC | 18:36 | |
*** openstackgerrit has joined #openstack-zun | 19:02 | |
openstackgerrit | Hongbin Lu proposed openstack/zun: Add container-update API https://review.openstack.org/391107 | 19:02 |
*** sudipto has quit IRC | 19:05 | |
*** sudswas__ has quit IRC | 19:05 | |
openstackgerrit | Hongbin Lu proposed openstack/zun: Ensure api can work as expected with boolean query params https://review.openstack.org/419373 | 19:51 |
openstackgerrit | Hongbin Lu proposed openstack/zun: Add container-update API https://review.openstack.org/391107 | 20:07 |
-openstackstatus- NOTICE: Gerrit will be offline between now and 20:30 for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html | 20:10 | |
*** ChanServ changes topic to "Gerrit will be offline between now and 20:30 for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html" | 20:10 | |
-openstackstatus- NOTICE: Updated: Gerrit will be offline until 20:45 for scheduled maintenance (running longer than anticipated): http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html | 20:35 | |
*** ChanServ changes topic to "Updated: Gerrit will be offline until 20:45 for scheduled maintenance (running longer than anticipated): http://lists.openstack.org/pipermail/openstack-dev/2017-January/109910.html" | 20:35 | |
*** ChanServ changes topic to "Container management service for OpenStack" | 20:54 | |
*** jwcroppe has quit IRC | 21:01 | |
*** openstackgerrit has quit IRC | 21:18 | |
*** adisky_ has quit IRC | 21:19 | |
*** sheel has quit IRC | 21:47 | |
*** openstackgerrit has joined #openstack-zun | 22:00 | |
openstackgerrit | Hongbin Lu proposed openstack/zun: Remove redundant Connection Class in zun/db/api.py https://review.openstack.org/419310 | 22:00 |
openstackgerrit | Hongbin Lu proposed openstack/zun: Remove redundant Connection Class in zun/db/api.py https://review.openstack.org/419310 | 22:12 |
*** jwcroppe has joined #openstack-zun | 22:26 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!