Thursday, 2017-01-12

*** jwcroppe has joined #openstack-zun00:22
*** zhurong has joined #openstack-zun00:28
*** kevinz has joined #openstack-zun01:22
*** yanyanhu has joined #openstack-zun01:23
*** zhangjl has joined #openstack-zun01:36
*** Wenzhi has joined #openstack-zun01:53
caoweihi all02:00
*** jianyi has joined #openstack-zun02:01
jianyihi caowei02:01
*** zhangjl has left #openstack-zun02:10
*** zhangjl has joined #openstack-zun02:23
*** hongbin_ has joined #openstack-zun02:33
*** jianyi has quit IRC03:16
*** zhurong has quit IRC03:39
*** prameswar has joined #openstack-zun03:45
*** hongbin_ has quit IRC03:55
*** hongbin_ has joined #openstack-zun04:21
*** diga has joined #openstack-zun04:21
*** pksingh has joined #openstack-zun04:28
*** sudipto has joined #openstack-zun04:29
*** sudipto_ has joined #openstack-zun04:29
*** gb21 has joined #openstack-zun04:53
*** pksingh has quit IRC05:07
*** hongbin_ has quit IRC05:13
*** sudipto has quit IRC05:14
*** sudipto_ has quit IRC05:14
openstackgerritMerged openstack/zun: Support multiple compute hosts  https://review.openstack.org/41555405:14
*** adisky_ has joined #openstack-zun05:18
openstackgerritguo yunxian proposed openstack/zun: Remove unused function  https://review.openstack.org/41928005:19
*** pksingh has joined #openstack-zun05:46
openstackgerritKevin Zhao proposed openstack/python-zunclient: Add support for restart policy when create/run  https://review.openstack.org/41928805:51
openstackgerritprameswar proposed openstack/zun: [WIP] Added support for Auto remove the container  https://review.openstack.org/41894205:54
openstackgerritKevin Zhao proposed openstack/zun: Add support for container restart policy in server side  https://review.openstack.org/41928905:54
*** sudipto has joined #openstack-zun05:58
*** sudipto_ has joined #openstack-zun05:58
openstackgerritprameswar proposed openstack/python-zunclient: [WIP] Autoremove container added to zun run client  https://review.openstack.org/41894305:58
openstackgerritPradeep Kumar Singh proposed openstack/zun: Remove the Container class  https://review.openstack.org/41765806:00
*** zhurong has joined #openstack-zun06:23
*** pksingh has quit IRC06:47
*** janki has joined #openstack-zun06:49
openstackgerritWenzhi Yu proposed openstack/zun: Add etcd DB backend support for multiple compute env  https://review.openstack.org/41930206:57
openstackgerritWenzhi Yu proposed openstack/zun: Add etcd DB backend support for multiple compute env  https://review.openstack.org/41930207:00
*** pksingh has joined #openstack-zun07:16
openstackgerritKevin Zhao proposed openstack/zun: Remove redundant Connection Class in zun/db/api.py  https://review.openstack.org/41931007:20
*** janki has quit IRC07:22
*** jchhatbar has joined #openstack-zun07:22
openstackgerritWenzhi Yu proposed openstack/zun: Add etcd DB backend support for multiple compute env  https://review.openstack.org/41930207:29
*** sudipto has quit IRC07:36
*** sudipto_ has quit IRC07:36
*** hongbin has quit IRC08:16
*** diga has quit IRC08:26
*** strigazi_AFK is now known as strigazi08:28
*** sudipto_ has joined #openstack-zun08:31
*** sudipto has joined #openstack-zun08:31
*** caowei has quit IRC08:39
*** jwcroppe has quit IRC08:41
*** jwcroppe has joined #openstack-zun08:42
*** Guest16073 is now known as jayaura08:45
*** jwcroppe has quit IRC08:46
*** prameswar has quit IRC09:03
pksinghzhangjl: Hello09:14
zhangjlpksingh:hello09:14
pksinghzhangjl: https://bugs.launchpad.net/zun/+bug/1655868, nice catch :)09:14
openstackLaunchpad bug 1655868 in Zun "API cannot work as expected because of the boolean params" [High,Confirmed] - Assigned to zhangjialong (zhangjl)09:14
zhangjlpksingh:thanks09:14
pksinghYou need to convert the query parameters to Boolean, like below09:15
pksinghhttps://github.com/openstack/zun/blob/master/zun/api/controllers/v1/containers.py#L33709:15
zhangjlpksingh:yes09:15
pksinghzhangjl: ^^, i think above should work09:15
zhangjlpksingh:i working like above09:15
pksinghzhangjl: please submit the patch when you have time, thanks for reporting09:15
pksinghzhangjl: we missed it in review :(09:16
zhangjlpksingh:my pleasure09: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 command09:21
openstackLaunchpad bug 1654010 in Zun "Improve the container status" [Medium,Triaged] - Assigned to aditi sharma (adi-sky17)09:21
pksinghyes adisky_ , what can we help?09:25
adisky_There are two possible solutions which i can think of09: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 stopped09:26
adisky_for second approach we need to call docker api twice fro conatiner list operation09:27
adisky_one more query should we save status_detail to database, as this field will keep on changing??09:28
pksinghlet me check the code09:28
adisky_@pksingh thnx09:28
pksinghhttps://github.com/openstack/zun/blob/master/zun/container/docker/driver.py#L119, inspect command gives result of the state09:31
pksinghadisky_: see https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/09:32
pksinghadisky_: https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/#/inspect-a-container09:32
pksinghadisky_: i think using that u can derive the state09:32
adisky_ya... the 1 st approach we should derive it09:34
adisky_pksingh : https://github.com/openstack/zun/blob/master/zun/compute/manager.py09:35
adisky_i think the method container_list  is not used anywhere09:36
pksinghyou 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
pksinghadisky_: i need to check,09:40
adisky_ok09:41
adisky_thnx for your help :)09:41
pksinghadisky_: yes its not being used09:42
pksinghadisky_: if you want more information then plz check  https://github.com/docker/docker/blob/master/cli/command/formatter/container.go09:42
pksinghadisky_: may be there you can get, its the docker client code where its format the output09:42
*** zhurong has quit IRC09:44
adisky_ok i will look on to it, thnx :)09:45
*** Wenzhi has quit IRC09:52
*** yanyanhu has quit IRC09:54
openstackgerritzhangjl proposed openstack/zun: Ensure api can work as expected with boolean query params  https://review.openstack.org/41937309:55
*** zhangjl has quit IRC10:02
*** prameswar has joined #openstack-zun10:04
*** kevinz has quit IRC10:11
*** zhurong has joined #openstack-zun10:25
*** janki has joined #openstack-zun10:28
*** jchhatbar has quit IRC10:28
*** Wenzhi has joined #openstack-zun11:05
*** Wenzhi has quit IRC11:07
*** zhurong has quit IRC11:15
*** sudipto has quit IRC11:22
*** sudipto_ has quit IRC11:22
*** pksingh has quit IRC11:25
*** sheel has joined #openstack-zun12:19
*** gb21 has quit IRC13:00
*** sudipto_ has joined #openstack-zun13:37
*** sudipto has joined #openstack-zun13:37
*** janki has quit IRC14:35
*** hongbin has joined #openstack-zun14:59
*** sheel has quit IRC15:07
*** jwcroppe has joined #openstack-zun15:28
*** sudipto has quit IRC16:05
*** sudipto_ has quit IRC16:05
*** sudipto_ has joined #openstack-zun16:05
*** sudipto has joined #openstack-zun16:05
hongbinhi 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-zun17:07
*** sudipto_ has quit IRC17:07
*** sudipto has quit IRC17:08
*** sudipto has joined #openstack-zun17:08
*** sheel has joined #openstack-zun17:10
*** openstackgerrit has quit IRC18:33
*** prameswar has quit IRC18:36
*** openstackgerrit has joined #openstack-zun19:02
openstackgerritHongbin Lu proposed openstack/zun: Add container-update API  https://review.openstack.org/39110719:02
*** sudipto has quit IRC19:05
*** sudswas__ has quit IRC19:05
openstackgerritHongbin Lu proposed openstack/zun: Ensure api can work as expected with boolean query params  https://review.openstack.org/41937319:51
openstackgerritHongbin Lu proposed openstack/zun: Add container-update API  https://review.openstack.org/39110720: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.html20: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.html20: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 IRC21:01
*** openstackgerrit has quit IRC21:18
*** adisky_ has quit IRC21:19
*** sheel has quit IRC21:47
*** openstackgerrit has joined #openstack-zun22:00
openstackgerritHongbin Lu proposed openstack/zun: Remove redundant Connection Class in zun/db/api.py  https://review.openstack.org/41931022:00
openstackgerritHongbin Lu proposed openstack/zun: Remove redundant Connection Class in zun/db/api.py  https://review.openstack.org/41931022:12
*** jwcroppe has joined #openstack-zun22:26

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