| *** iceyao has quit IRC | 00:24 | |
| *** iceyao has joined #openstack-zun | 01:21 | |
| openstackgerrit | duhu proposed openstack/zun: Replace assertEqual(None, *) with assertIsNone https://review.openstack.org/392125 | 01:26 |
|---|---|---|
| *** yanyanhu has joined #openstack-zun | 01:34 | |
| *** zhurong has joined #openstack-zun | 01:35 | |
| openstackgerrit | Merged openstack/zun: Updated from global requirements https://review.openstack.org/391172 | 01:51 |
| *** iceyao has quit IRC | 02:44 | |
| *** iceyao has joined #openstack-zun | 02:44 | |
| *** coolsvap has joined #openstack-zun | 04:08 | |
| *** vikasc has joined #openstack-zun | 04:17 | |
| *** yuanying has quit IRC | 04:24 | |
| *** yuanying has joined #openstack-zun | 04:36 | |
| *** mkrai has joined #openstack-zun | 05:00 | |
| openstackgerrit | Merged openstack/python-zunclient: Add Python 3.5 classifier and venv https://review.openstack.org/390706 | 05:08 |
| openstackgerrit | Sheel Rana proposed openstack/zun: Add coverage configuration https://review.openstack.org/361765 | 05:16 |
| openstackgerrit | Pradeep Kumar Singh proposed openstack/zun: Move all conf option to centralize conf directory https://review.openstack.org/391210 | 05:17 |
| openstackgerrit | Sheel Rana proposed openstack/python-zunclient: Add coverage configuration https://review.openstack.org/361766 | 05:19 |
| openstackgerrit | Merged openstack/zun: Add some functional test case https://review.openstack.org/390821 | 05:23 |
| openstackgerrit | Merged openstack/zun: Replace assertEqual(None, *) with assertIsNone https://review.openstack.org/392125 | 05:24 |
| openstackgerrit | Merged openstack/zun: Enable DeprecationWarning in test environments https://review.openstack.org/390751 | 05:24 |
| *** pksingh has joined #openstack-zun | 05:24 | |
| *** iceyao_ has joined #openstack-zun | 05:25 | |
| *** iceyao has quit IRC | 05:25 | |
| openstackgerrit | Merged openstack/zun: Adding support for container image driver https://review.openstack.org/383678 | 05:28 |
| *** sheel has joined #openstack-zun | 05:33 | |
| openstackgerrit | Merged openstack/zun: Add coverage configuration https://review.openstack.org/361765 | 05:38 |
| openstackgerrit | Madhuri Kumari proposed openstack/zun: [WIP]Add container-update API https://review.openstack.org/391107 | 05:40 |
| openstackgerrit | Merged openstack/python-zunclient: Add more details to service-list command https://review.openstack.org/374605 | 05:44 |
| openstackgerrit | Merged openstack/zun-ui: Drop MANIFEST.in - it's not needed by pbr https://review.openstack.org/391475 | 05:51 |
| openstackgerrit | Merged openstack/zun-ui: Don't include openstack/common in flake8 exclude list https://review.openstack.org/391488 | 05:51 |
| openstackgerrit | Merged openstack/zun-ui: Enable DeprecationWarning in test environments https://review.openstack.org/390757 | 05:53 |
| *** iceyao_ has quit IRC | 06:04 | |
| *** iceyao has joined #openstack-zun | 06:06 | |
| openstackgerrit | Wenzhi Yu proposed openstack/zun: Run etcd server in docker container https://review.openstack.org/389584 | 06:22 |
| openstackgerrit | Pradeep Kumar Singh proposed openstack/zun: Set status_reason when container creation fails https://review.openstack.org/392428 | 06:32 |
| *** pksingh has quit IRC | 06:36 | |
| *** yuanying has quit IRC | 06:46 | |
| *** yuanying has joined #openstack-zun | 06:49 | |
| *** trinaths has joined #openstack-zun | 06:59 | |
| openstackgerrit | Merged openstack/zun: Set status_reason when container creation fails https://review.openstack.org/392428 | 07:01 |
| *** pksingh has joined #openstack-zun | 07:17 | |
| *** yuywz has joined #openstack-zun | 07:34 | |
| *** yuywz is now known as Wenzhi | 07:38 | |
| openstackgerrit | Pradeep Kumar Singh proposed openstack/zun: Move all conf option to centralize conf directory https://review.openstack.org/391210 | 07:43 |
| openstackgerrit | Madhuri Kumari proposed openstack/zun: Fix issue in image-create API https://review.openstack.org/392447 | 07:47 |
| *** openstackgerrit has quit IRC | 07:48 | |
| *** openstackgerrit has joined #openstack-zun | 07:48 | |
| *** mkrai has quit IRC | 08:02 | |
| *** Wenzhi has quit IRC | 08:06 | |
| *** Wenzhi has joined #openstack-zun | 08:07 | |
| *** Wenzhi has quit IRC | 08:11 | |
| *** yanyanhu has quit IRC | 08:13 | |
| *** Wenzhi has joined #openstack-zun | 08:16 | |
| *** trinaths has quit IRC | 08:28 | |
| *** Wenzhi has quit IRC | 08:34 | |
| *** GB21 has joined #openstack-zun | 08:55 | |
| pksingh | Hello guys, can there be many image drivers applicable at a time in system? | 09:04 |
| pksingh | eliqiao, ^ | 09:05 |
| pksingh | i thin for single compute service it would be one, am i right? | 09:06 |
| pksingh | Qiming, | 09:06 |
| *** zhurong has quit IRC | 09:06 | |
| *** mkrai has joined #openstack-zun | 09:07 | |
| pksingh | mkrai, can there be many image drivers applicable at a time in system? | 09:07 |
| pksingh | mkrai, for a single compute service? | 09:07 |
| mkrai | pksingh: yes | 09:08 |
| mkrai | operators can define the list of image_drivers in conf | 09:08 |
| pksingh | yes they can be define, but for a single instance, what is the use case | 09:09 |
| openstackgerrit | Madhuri Kumari proposed openstack/zun: Fix issue in image-create API https://review.openstack.org/392447 | 09:09 |
| mkrai | pksingh if the image is not found in glance for instance, zun will look into dockerhub | 09:11 |
| pksingh | mkrai, found https://bugs.launchpad.net/zun/+bug/1638494, was thinking how to fix | 09:11 |
| openstack | Launchpad bug 1638494 in Zun "container creation fails with error if driver is glance" [Undecided,New] - Assigned to Pradeep Kumar Singh (pradeep-singh-u) | 09:11 |
| pksingh | mkrai, i think nobody sets multiple repositories for a single image, right? | 09:12 |
| mkrai | which API gave you this error? | 09:12 |
| pksingh | mkrai, sun create | 09:12 |
| pksingh | ^zun | 09:13 |
| mkrai | image_driver conf is list and one can specify like glancedriver,dockerdriver | 09:13 |
| mkrai | I haven't got this error | 09:13 |
| mkrai | Can you please paste your config and command you ran? | 09:13 |
| pksingh | i just installed devstack, and executed 'zun create --name test --image nginx' | 09:14 |
| *** GB21 has quit IRC | 09:15 | |
| mkrai | pksingh: how does your conf look like? | 09:16 |
| pksingh | mkrai, https://github.com/openstack/zun/blob/master/zun/image/driver.py#L114, will come when image is not found in glance | 09:16 |
| pksingh | i have only one driver specified | 09:16 |
| mkrai | ok | 09:16 |
| mkrai | yes | 09:17 |
| pksingh | its better to set docker driver in default list | 09:17 |
| mkrai | Yes, we can for now. I didn't set docker as default because we aim to support multiple container runtime backends | 09:19 |
| mkrai | Ok I got the error, it is when we set only one driver and image is not found there | 09:20 |
| mkrai | pksingh: Right? | 09:20 |
| pksingh | mkrai, correct | 09:21 |
| mkrai | Thanks for reporting the bug pksingh | 09:22 |
| pksingh | mkrai, :) | 09:22 |
| mkrai | I missed this case | 09:22 |
| pksingh | mkrai, one more thing, if driver list order is [docker, glance] and image is found in docker , then it will pull image from glance, right? | 09:31 |
| *** iceyao has quit IRC | 09:31 | |
| pksingh | mkrai, because pull from docker driver returns nothing, right? | 09:31 |
| mkrai | no it will pull from docker only | 09:31 |
| mkrai | Yes it returns nothing. | 09:33 |
| mkrai | We are trying to return path of image tar when we download image from glance | 09:33 |
| mkrai | In docker driver case, we don't need this path | 09:33 |
| mkrai | that's why returning none | 09:33 |
| mkrai | pksingh: https://github.com/openstack/zun/blob/master/zun/container/docker/driver.py#L50 | 09:34 |
| mkrai | this explains what is image_path | 09:35 |
| pksingh | mkrai, http://paste.openstack.org/show/587617/ | 09:37 |
| pksingh | mkrai, my driver list is like default=['docker.driver.DockerDriver', 'glance.driver.GlanceDriver'], | 09:37 |
| pksingh | mkrai, is driver list correct? | 09:38 |
| mkrai | yes | 09:38 |
| pksingh | then there is issue,https://github.com/openstack/zun/blob/master/zun/image/driver.py#L111 because docker driver returns none | 09:39 |
| pksingh | mkrai, and loop never breaks | 09:39 |
| pksingh | until it found the image | 09:40 |
| mkrai | pksingh: yes right | 09:41 |
| mkrai | my bad :( | 09:41 |
| pksingh | no problem, | 09:42 |
| mkrai | pksingh: I will watch for your patch | 09:43 |
| mkrai | Thanks again | 09:43 |
| pksingh | thanks, sure | 09:43 |
| pksingh | no prob | 09:44 |
| *** strigazi_AFK is now known as strigazi | 09:44 | |
| pksingh | mkrai, regarding https://review.openstack.org/#/c/392447/2/zun/tests/unit/api/controllers/v1/test_images.py, i wanted to say can we add anothet test that calls api with repo like 'nginx:lates' | 09:45 |
| pksingh | mkrai, currently its calling with 'hello-world' only | 09:46 |
| pksingh | mkrai, am i missing something? | 09:46 |
| mkrai | Ok got it | 09:46 |
| pksingh | :) | 09:46 |
| mkrai | pksingh: I thought you are asking to include this case in parse_image_tag test | 09:47 |
| mkrai | I will add it | 09:47 |
| pksingh | nope | 09:47 |
| pksingh | its already there in parse_image_tag | 09:47 |
| mkrai | pksingh: i will do that. Thanks | 09:47 |
| pksingh | mkrai, :) | 09:47 |
| pksingh | mkrai, what are meeting timings for zun weekly meeting in IST? | 09:49 |
| mkrai | tuesday, 8:30 am | 09:49 |
| pksingh | ok thanks | 09:49 |
| *** deepak_jon has joined #openstack-zun | 09:51 | |
| *** exploreshaifali has joined #openstack-zun | 09:52 | |
| *** pksingh has quit IRC | 09:58 | |
| *** pksingh has joined #openstack-zun | 10:08 | |
| *** pksingh has quit IRC | 10:13 | |
| *** exploreshaifali has quit IRC | 10:22 | |
| *** exploreshaifali has joined #openstack-zun | 10:25 | |
| *** exploreshaifali has quit IRC | 10:43 | |
| *** exploreshaifali has joined #openstack-zun | 10:57 | |
| *** GB21 has joined #openstack-zun | 11:15 | |
| *** mkrai has quit IRC | 11:31 | |
| *** GB21 has quit IRC | 11:33 | |
| *** iceyao has joined #openstack-zun | 12:06 | |
| *** sheel has quit IRC | 13:00 | |
| *** sudipto has joined #openstack-zun | 13:10 | |
| *** sudipto_ has joined #openstack-zun | 13:10 | |
| *** coolsvap has quit IRC | 13:18 | |
| openstackgerrit | Pradeep Kumar Singh proposed openstack/zun: Change Default Image Driver to DockerDriver https://review.openstack.org/392661 | 13:35 |
| openstackgerrit | Pradeep Kumar Singh proposed openstack/zun: Change Default Image Driver to DockerDriver https://review.openstack.org/392661 | 13:37 |
| *** deepak_jon has quit IRC | 13:55 | |
| *** GB21 has joined #openstack-zun | 14:01 | |
| *** mkrai has joined #openstack-zun | 14:32 | |
| *** iceyao has quit IRC | 15:04 | |
| *** exploreshaifali has quit IRC | 15:22 | |
| openstackgerrit | Madhuri Kumari proposed openstack/zun: Fix issue in image-create API https://review.openstack.org/392447 | 15:54 |
| openstackgerrit | Madhuri Kumari proposed openstack/zun: Add docker API name in logs https://review.openstack.org/392886 | 17:00 |
| *** harlowja has quit IRC | 17:21 | |
| *** harlowja has joined #openstack-zun | 17:44 | |
| *** mkrai has quit IRC | 18:06 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** openstackgerrit has joined #openstack-zun | 18:18 | |
| *** exploreshaifali has joined #openstack-zun | 18:19 | |
| *** GB21 has quit IRC | 18:24 | |
| *** sudipto has quit IRC | 18:52 | |
| *** sudipto_ has quit IRC | 18:52 | |
| *** yuanying has quit IRC | 19:03 | |
| *** exploreshaifali has quit IRC | 19:28 | |
| *** openstack has joined #openstack-zun | 20:00 | |
| *** openstackgerrit has quit IRC | 20:18 | |
| *** openstackgerrit has joined #openstack-zun | 20:18 | |
| *** exploreshaifali has joined #openstack-zun | 21:25 | |
| *** harlowja has quit IRC | 22:02 | |
| *** flaper87 has quit IRC | 22:29 | |
| *** exploreshaifali has quit IRC | 22:31 | |
| *** harlowja has joined #openstack-zun | 23:08 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!