| *** hongbin has quit IRC | 00:19 | |
| *** zhurong has joined #openstack-zun | 00:55 | |
| *** zhangjl has joined #openstack-zun | 01:11 | |
| *** kevinz has joined #openstack-zun | 01:13 | |
| *** yanyanhu has joined #openstack-zun | 01:37 | |
| *** yanyanhu has quit IRC | 01:53 | |
| *** yanyanhu has joined #openstack-zun | 01:54 | |
| openstackgerrit | zhangjl proposed openstack/zun: Ensure api can work as expected with boolean query params https://review.openstack.org/419373 | 02:03 |
|---|---|---|
| openstackgerrit | Kevin Zhao proposed openstack/python-zunclient: Add support for restart policy when create/run https://review.openstack.org/419288 | 02:50 |
| *** yuanying has quit IRC | 02:52 | |
| *** hongbin has joined #openstack-zun | 02:59 | |
| openstackgerrit | Kevin Zhao proposed openstack/python-zunclient: Add support for restart policy when create/run https://review.openstack.org/419288 | 03:00 |
| *** gb21 has joined #openstack-zun | 03:02 | |
| *** hongbin_ has joined #openstack-zun | 03:03 | |
| *** hongbin has quit IRC | 03:03 | |
| openstackgerrit | Digambar proposed openstack/zun: Implement Cinder integration with zun https://review.openstack.org/417747 | 03:15 |
| *** diga has joined #openstack-zun | 03:15 | |
| diga | hongbin_: please go through the latest PS on Cinder-integration | 03:16 |
| *** prameswar has joined #openstack-zun | 03:21 | |
| hongbin_ | diga: ack | 03:24 |
| *** zhurong has quit IRC | 03:39 | |
| *** yuanying has joined #openstack-zun | 03:52 | |
| *** pksingh has joined #openstack-zun | 04:04 | |
| pksingh | zhangjl: Hello, Good morning | 04:05 |
| zhangjl | pksingh:hello | 04:05 |
| zhangjl | pksingh:sorry for my late reply | 04:05 |
| pksingh | zhangjl: its okay | 04:05 |
| zhangjl | pksingh: i push a new patch, pls review again | 04:06 |
| pksingh | zhangjl: yes sure, i lloked into it. | 04:06 |
| zhangjl | pksingh:thanks :-D | 04:06 |
| pksingh | zhangjl: although not related to this patch, but if you handle the ValueError exception in delete also then it would be good | 04:07 |
| pksingh | zhangjl: what do you think | 04:07 |
| zhangjl | pksingh: oh ,yes, it`s my mistack | 04:07 |
| zhangjl | pksingh:i will push a new one, pls wait | 04:08 |
| pksingh | zhangjl: sure, thanks sir :) | 04:08 |
| pksingh | zhangjl: if possible please add UTs for delete too | 04:09 |
| zhangjl | pksingh:np. And then , i will leave for lunch, good luck | 04:10 |
| pksingh | zhangjl: sure, thanks :) | 04:12 |
| openstackgerrit | Merged openstack/zun: Add etcd DB backend support for multiple compute env https://review.openstack.org/419302 | 04:21 |
| *** janki has joined #openstack-zun | 04:28 | |
| *** diga has quit IRC | 05:01 | |
| *** hongbin_ has quit IRC | 05:02 | |
| *** sudipto has joined #openstack-zun | 05:18 | |
| *** sudipto_ has joined #openstack-zun | 05:18 | |
| openstackgerrit | zhangjl proposed openstack/zun: Ensure api can work as expected with boolean query params https://review.openstack.org/419373 | 05:38 |
| *** adisky_ has joined #openstack-zun | 05:38 | |
| Qiming | hi, do we have an API document so far? | 05:41 |
| adisky_ | No, I have filed a blueprint for API doc | 05:43 |
| adisky_ | https://blueprints.launchpad.net/zun/+spec/zun-api-ref-document | 05:43 |
| Qiming | thanks, adisky_ | 05:51 |
| adisky_ | Qiming: :) | 05:57 |
| Qiming | I was thinking if it is possible to create and manage containers using senlin by invoking zun apis | 06:00 |
| adisky_ | ok | 06:01 |
| adisky_ | so how can i help you?? | 06:02 |
| Qiming | :D, get that blueprint implemented, | 06:03 |
| *** yuanying has quit IRC | 06:06 | |
| *** prameswar has quit IRC | 06:07 | |
| *** yuanying has joined #openstack-zun | 06:07 | |
| *** yuanying has quit IRC | 06:11 | |
| *** yuanying_ has joined #openstack-zun | 06:11 | |
| *** yuanying_ has quit IRC | 06:12 | |
| *** yuanying has joined #openstack-zun | 06:12 | |
| *** yuanying has quit IRC | 06:17 | |
| openstackgerrit | Namrata proposed openstack/python-zunclient: Add OSC Plugin for openstack appcontainer exec https://review.openstack.org/417964 | 06:18 |
| *** jwcroppe has quit IRC | 06:23 | |
| *** prameswar has joined #openstack-zun | 06:30 | |
| *** prameswar has quit IRC | 06:47 | |
| adisky_ | Qiming: OK :) | 06:53 |
| *** prameswar has joined #openstack-zun | 06:59 | |
| *** zhurong has joined #openstack-zun | 07:05 | |
| *** yuanying has joined #openstack-zun | 07:19 | |
| openstackgerrit | Merged openstack/zun: Remove redundant Connection Class in zun/db/api.py https://review.openstack.org/419310 | 07:25 |
| openstackgerrit | Kevin Zhao proposed openstack/zun: Add support for container restart policy in server side https://review.openstack.org/419289 | 07:29 |
| *** adisky_ has quit IRC | 07:29 | |
| *** adisky_ has joined #openstack-zun | 07:36 | |
| *** janki has quit IRC | 07:42 | |
| openstackgerrit | Kevin Zhao proposed openstack/zun: Remove get_container_id functions https://review.openstack.org/419796 | 07:43 |
| *** janki has joined #openstack-zun | 07:43 | |
| pksingh | Hello kevinz , | 07:59 |
| kevinz | pksingh:Hi | 08:04 |
| pksingh | kevinz: i looked into your patch, can we move the validation of values to API, what do you think? | 08:05 |
| kevinz | pksingh: Good afternoon | 08:05 |
| pksingh | kevinz: good afternoon | 08:05 |
| pksingh | :) | 08:05 |
| pksingh | kevinz: i am talking about the validation in docker.py | 08:06 |
| kevinz | pksingh: You mean the restart policy patch, the parameter in docker.py? | 08:06 |
| pksingh | yes where you raise the exception due to wrong users input of retry count | 08:07 |
| pksingh | if something is wrong in users input may we we can validate that at API, instaed of creating all the entries in DB and all | 08:08 |
| kevinz | pksingh: OK ,I will move to API , that will make the validation more reasonable | 08:08 |
| kevinz | pksingh: Thanks for your valuable advice :-) | 08:09 |
| pksingh | kevinz: and more thing i would like to suggest, please add more test regarding the vaidation of restart_policy schema in test_validation.py | 08:10 |
| pksingh | may be all positive and negative tests | 08:11 |
| pksingh | what do you think? | 08:11 |
| kevinz | Agree with you~ I will add all related tests | 08:12 |
| pksingh | kevinz: thanks a lot :) | 08:19 |
| kevinz | pksingh: :-) | 08:20 |
| *** prameswar has quit IRC | 08:57 | |
| *** zhurong has quit IRC | 09:16 | |
| *** gb21 has quit IRC | 09:22 | |
| *** sheel has joined #openstack-zun | 09:56 | |
| *** kevinz has quit IRC | 10:08 | |
| *** openstackgerrit has quit IRC | 10:18 | |
| *** yanyanhu has quit IRC | 10:19 | |
| *** prameswar has joined #openstack-zun | 10:21 | |
| *** aloga has joined #openstack-zun | 10:46 | |
| aloga | hi | 10:46 |
| *** sudipto_ has quit IRC | 11:05 | |
| *** sudipto has quit IRC | 11:05 | |
| *** zhangjl has left #openstack-zun | 11:15 | |
| *** pksingh has quit IRC | 11:24 | |
| *** gb21 has joined #openstack-zun | 12:03 | |
| *** adisky_ has quit IRC | 12:09 | |
| *** catintheroof has joined #openstack-zun | 12:30 | |
| *** catintheroof has quit IRC | 12:31 | |
| *** catintheroof has joined #openstack-zun | 12:32 | |
| *** gb21 has quit IRC | 12:38 | |
| *** openstackgerrit has joined #openstack-zun | 12:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/zun: Updated from global requirements https://review.openstack.org/419943 | 12:53 |
| *** kevinz has joined #openstack-zun | 12:55 | |
| *** kevinz has quit IRC | 13:00 | |
| *** sudipto_ has joined #openstack-zun | 13:19 | |
| *** sudipto has joined #openstack-zun | 13:19 | |
| openstackgerrit | prameswar proposed openstack/python-zunclient: Corrected wrong env variable in --zun-api-version https://review.openstack.org/419966 | 13:38 |
| *** kevinz has joined #openstack-zun | 14:24 | |
| *** kevinz has quit IRC | 14:31 | |
| *** janki has quit IRC | 14:52 | |
| *** sheel has quit IRC | 14:57 | |
| *** hongbin has joined #openstack-zun | 15:00 | |
| *** jwcroppe has joined #openstack-zun | 15:17 | |
| openstackgerrit | Merged openstack/python-zunclient: Move out the "-i" and "--image" flags from shell https://review.openstack.org/418207 | 15:18 |
| openstackgerrit | Merged openstack/zun: Ensure api can work as expected with boolean query params https://review.openstack.org/419373 | 15:31 |
| openstackgerrit | Merged openstack/zun: Remove get_container_id functions https://review.openstack.org/419796 | 16:02 |
| sudipto_ | hongbin, hello, sorry i got swayed into other things...I was wondering what's the best way to find out the capabilities of a host, esp cpus...any thoughts? | 16:29 |
| hongbin | sudipto_: docker info could show some basic information | 16:30 |
| openstackgerrit | Merged openstack/zun: Updated from global requirements https://review.openstack.org/419943 | 16:30 |
| hongbin | sudipto_: there might be some libraries/tools to query the detailed capabilities | 16:30 |
| sudipto_ | hongbin, I was trying to find out how kurbernetes does it - they seem to represent all cpus in milliseconds. | 16:31 |
| sudipto_ | another approach is to read it directly from proc/sysfs | 16:31 |
| openstackgerrit | Merged openstack/python-zunclient: Corrected wrong env variable in --zun-api-version https://review.openstack.org/419966 | 16:31 |
| hongbin | sudipto_: perhaps, we could look at how nova-libvirt driver did it | 16:32 |
| sudipto_ | nova-libvirt reads it from virsh capabilities | 16:33 |
| sudipto_ | and libvirt reads it from syses | 16:33 |
| sudipto_ | afaik | 16:33 |
| hongbin | sudipto_: i see | 16:34 |
| hongbin | there are two problems here: | 16:36 |
| hongbin | 1. how to represent the cpu allocation (milliseconds? vcpus?) | 16:36 |
| hongbin | 2. how to query the cpu capabilities | 16:36 |
| hongbin | for #2, i think copying from nova/libvirt/k8s would be a good start | 16:37 |
| hongbin | for #1, i am not sure | 16:37 |
| sudipto_ | hongbin, i am not sure if kubernetes supports the cpusets configs or numa...i am guessing they don't | 16:38 |
| hongbin | sudipto_: AFAIK, supporting numa is in their roadmap, but not for cpuset | 16:39 |
| sudipto_ | hongbin, ok something we can in parallel develop i guess. but anyway, let me spend some more time thinking what they have done to get to 2. | 16:40 |
| sudipto_ | i got swayed with some sahara work | 16:40 |
| hongbin | sudipto_: ack | 16:40 |
| *** prameswar has quit IRC | 16:57 | |
| *** sudipto has quit IRC | 17:01 | |
| *** sudipto_ has quit IRC | 17:01 | |
| *** prameswar has joined #openstack-zun | 17:13 | |
| *** jwcroppe has quit IRC | 17:56 | |
| *** jwcroppe has joined #openstack-zun | 17:59 | |
| *** jwcroppe has quit IRC | 18:48 | |
| *** jwcroppe has joined #openstack-zun | 18:53 | |
| *** openstack has joined #openstack-zun | 19:00 | |
| *** prameswar has quit IRC | 19:13 | |
| *** openstackstatus has quit IRC | 19:57 | |
| *** openstack has joined #openstack-zun | 20:00 | |
| openstackgerrit | Hongbin Lu proposed openstack/zun: Add container-update API https://review.openstack.org/391107 | 20:08 |
| *** openstackstatus has quit IRC | 20:27 | |
| *** openstack has joined #openstack-zun | 20:30 | |
| *** vikasc has joined #openstack-zun | 20:32 | |
| openstackgerrit | Hongbin Lu proposed openstack/zun: Add container-update API https://review.openstack.org/391107 | 21:05 |
| openstackgerrit | Hongbin Lu proposed openstack/zun: Add container-update API https://review.openstack.org/391107 | 22:28 |
| *** catintheroof has quit IRC | 22:38 | |
| openstackgerrit | Hongbin Lu proposed openstack/python-zunclient: Add OSC Plugin for openstack appcontainer exec https://review.openstack.org/417964 | 22:39 |
| *** jwcroppe has quit IRC | 23:47 | |
| *** harlowja has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!