| *** kiennt has joined #openstack-zun | 00:31 | |
| *** hongbin has joined #openstack-zun | 00:43 | |
| *** zhurong has joined #openstack-zun | 00:45 | |
| *** Shunli has joined #openstack-zun | 00:50 | |
| *** kevzha01 has joined #openstack-zun | 01:35 | |
| *** kevzha01 is now known as kevinz | 01:36 | |
| openstackgerrit | Merged openstack/zun master: Make some config options work https://review.openstack.org/489012 | 01:44 |
|---|---|---|
| *** caowei has joined #openstack-zun | 02:08 | |
| *** deep-book-gk has joined #openstack-zun | 02:16 | |
| *** deep-book-gk has left #openstack-zun | 02:18 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 02:51 | |
| openstackgerrit | Merged openstack/zun-ui master: Fix invalid UTF-8 https://review.openstack.org/488994 | 03:01 |
| *** kiennt_ has joined #openstack-zun | 04:14 | |
| *** hongbin has quit IRC | 05:01 | |
| *** janki has joined #openstack-zun | 05:10 | |
| *** caowei has quit IRC | 05:23 | |
| *** zhurong has quit IRC | 05:29 | |
| *** c06 has joined #openstack-zun | 05:39 | |
| *** kiennt_ has quit IRC | 05:49 | |
| *** kiennt has quit IRC | 05:49 | |
| openstackgerrit | Namrata proposed openstack/zun master: Image pull policy support for zun pull Closes Bug: #1647434 https://review.openstack.org/489093 | 05:50 |
| openstack | bug 1647434 in Zun "Implement image_pull policy for image clis" [Medium,In progress] https://launchpad.net/bugs/1647434 - Assigned to Namrata (sitlani-namrata) | 05:50 |
| openstackgerrit | Namrata proposed openstack/python-zunclient master: Image pull policy support for zun pull https://review.openstack.org/489094 | 05:52 |
| openstackgerrit | Namrata proposed openstack/zun master: Image pull policy support for zun pull https://review.openstack.org/489093 | 05:53 |
| *** kiennt has joined #openstack-zun | 06:02 | |
| *** kiennt_ has joined #openstack-zun | 06:02 | |
| *** kiennt__ has joined #openstack-zun | 06:05 | |
| *** kiennt__ has quit IRC | 06:05 | |
| *** kiennt_ has quit IRC | 06:06 | |
| *** kiennt has quit IRC | 06:06 | |
| *** kiennt has joined #openstack-zun | 06:06 | |
| *** caowei has joined #openstack-zun | 06:15 | |
| openstackgerrit | Merged openstack/zun master: Remove unused requirement: greenlet https://review.openstack.org/488928 | 06:19 |
| openstackgerrit | Merged openstack/zun master: Add missing configuration index https://review.openstack.org/488565 | 06:19 |
| *** zsli_ has joined #openstack-zun | 06:40 | |
| openstackgerrit | Namrata proposed openstack/python-zunclient master: Image pull policy support for zun pull https://review.openstack.org/489094 | 06:41 |
| *** Shunli has quit IRC | 06:43 | |
| *** zhurong has joined #openstack-zun | 06:47 | |
| openstackgerrit | cooldharma06 proposed openstack/zun master: Allow glance image name/id support https://review.openstack.org/487381 | 06:52 |
| openstackgerrit | Shunli Zhou proposed openstack/zun master: Add filter scheduler doc. https://review.openstack.org/489109 | 07:08 |
| c06 | @hongbin: ^^ i did and submitted the patch as [WIP], help me in troubleshooting the unit test error. http://paste.openstack.org/show/616854/ | 07:11 |
| *** c06_ has joined #openstack-zun | 07:16 | |
| *** c06 has quit IRC | 07:19 | |
| *** c06_ is now known as c06 | 07:21 | |
| kiennt | c06: You change to use _load_image. In this method, it call self.load_image only if driver = glance. | 07:22 |
| kiennt | In the unittest, image driver is driver, so load_image is never called. I think so. | 07:22 |
| kiennt | Oh, sorry, I see you change driver: glance too. | 07:23 |
| kiennt | I will test it and hope I can help you. | 07:24 |
| c06 | kiennt: ty. i think the call hierarchy like this. load_image(driver.py) -> _load_image(utils.py) -> dockerapi - load_image(). if you found any errors means kindly let me know | 07:25 |
| c06 | kiennt: also _load_image(utils.py) doesn't return any values. | 07:28 |
| kiennt | c06: I think I catched your trouble. Wait me a minute, I paste the explanation and solution to paste.openstack | 07:38 |
| c06 | great and thank you, kiennt | 07:39 |
| kiennt | c06: http://paste.openstack.org/show/616972/ | 07:42 |
| kiennt | The unittest will be passed now. | 07:43 |
| kiennt | But if image['driver'] is not glance, now image will be loaded? | 07:44 |
| *** egonzalez has joined #openstack-zun | 07:44 | |
| kiennt | s/now/no/g | 07:45 |
| c06 | kiennt: i need to check that one | 07:45 |
| c06 | means /s/now/no/g.. ??? | 07:47 |
| kiennt | Oh, this one for correct the previous sentence (But if image['driver'] is not glance, now image will be loaded? --> But if image['driver'] is not glance, **no** image will be loaded?) | 07:48 |
| kiennt | vim syntax :) | 07:48 |
| c06 | kiennt: i thought it, i will check for without glance image ( image from docker hub) | 07:50 |
| kiennt | c06: One more thing. You change to self.driver.load(image) in zun/compute/manager.py, image is returned from image_driver.pull_image() | 07:52 |
| kiennt | But image_driver.pull_image() will "return {'image': repo, 'path': out_path}, image_loaded" | 07:53 |
| kiennt | https://github.com/openstack/zun/blob/master/zun/image/glance/driver.py#L113 | 07:53 |
| kiennt | or return {'image': repo, 'path': None}, image_loaded | 07:53 |
| kiennt | https://github.com/openstack/zun/blob/master/zun/image/docker/driver.py#L72 | 07:53 |
| kiennt | It mean, image = {'image': repo, 'path': None} without any driver key | 07:54 |
| c06 | kiennt: zun run --name "cir2" cirros | 07:54 |
| c06 | zun trying to check in glance repo if its not there means doing image_pull-> creating_container | 07:55 |
| kiennt | c06: I mean the check if image['driver'] == 'glance' will be failed and may raise exception like KeyError. because image doesn't have key 'driver' | 07:57 |
| c06 | kiennt: no it haven't and my container got created and running. docker repo also have cirros image. if you want i will paste the log outputs | 07:58 |
| c06 | ** cirros image downloaded (new one) | 07:58 |
| c06 | kiennt: http://paste.openstack.org/show/616974/ | 08:00 |
| openstackgerrit | Merged openstack/python-zunclient master: Fixed description in README.rst https://review.openstack.org/488946 | 08:02 |
| kiennt | c06, Wow, nice. I see, {'path': None, 'image': u'cirros', 'driver': 'docker'}. I will test your new patch when you push it. | 08:02 |
| c06 | kiennt: i think, it mightbe only workaround, hongbin suggested the way. | 08:06 |
| *** kevzha01_ has joined #openstack-zun | 08:11 | |
| c06 | kiennt: thanks for your help.. :) | 08:12 |
| kiennt | c06: you are welcome | 08:15 |
| *** kevinz has quit IRC | 08:15 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 08:27 | |
| *** zhurong has quit IRC | 08:41 | |
| openstackgerrit | feng.shengqin proposed openstack/python-zunclient master: Add <> for metavar info https://review.openstack.org/489147 | 09:20 |
| openstackgerrit | feng.shengqin proposed openstack/python-zunclient master: Reserve links info when showing image https://review.openstack.org/489149 | 09:27 |
| *** zsli_ has quit IRC | 09:31 | |
| *** nicolasbock has joined #openstack-zun | 09:35 | |
| *** zhurong has joined #openstack-zun | 09:37 | |
| *** janki is now known as janki|meeting | 09:37 | |
| *** janki|meeting is now known as janki | 09:39 | |
| openstackgerrit | feng.shengqin proposed openstack/python-zunclient master: Showing contain info after updating by osc command https://review.openstack.org/489158 | 09:54 |
| *** kiennt has quit IRC | 10:01 | |
| openstackgerrit | Merged openstack/python-zunclient master: Autoremove container added to zun run client https://review.openstack.org/418943 | 10:24 |
| *** kevzha01_ has quit IRC | 10:24 | |
| *** zhurong has quit IRC | 10:40 | |
| c06 | In docker/driver.py -> create(). image(parameter) not used inside the function. can i do trace and remove it..? | 10:48 |
| *** zhurong has joined #openstack-zun | 10:52 | |
| openstackgerrit | cooldharma06 proposed openstack/zun master: Allow glance image name/id support https://review.openstack.org/487381 | 10:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/zun-ui master: Imported Translations from Zanata https://review.openstack.org/489183 | 11:36 |
| *** caowei has quit IRC | 11:38 | |
| *** mkrai has quit IRC | 12:00 | |
| *** catintheroof has joined #openstack-zun | 12:29 | |
| *** lakerzhou has joined #openstack-zun | 12:31 | |
| *** lakerzhou1 has joined #openstack-zun | 12:37 | |
| *** lakerzhou2 has joined #openstack-zun | 12:38 | |
| *** lakerzhou has quit IRC | 12:40 | |
| *** lakerzhou has joined #openstack-zun | 12:41 | |
| *** lakerzhou1 has quit IRC | 12:42 | |
| *** lakerzhou2 has quit IRC | 12:42 | |
| *** zhurong has quit IRC | 13:00 | |
| openstackgerrit | wangzhh proposed openstack/zun master: Imply compute node related Etcd API https://review.openstack.org/489222 | 13:36 |
| *** c06 has quit IRC | 14:04 | |
| *** adisky__ has quit IRC | 14:16 | |
| *** janki has quit IRC | 14:17 | |
| *** coolsvap has quit IRC | 14:31 | |
| *** openstackgerrit has quit IRC | 14:33 | |
| *** mkrai has joined #openstack-zun | 14:40 | |
| *** hongbin has joined #openstack-zun | 14:52 | |
| *** lakerzhou has quit IRC | 14:58 | |
| *** lakerzhou has joined #openstack-zun | 15:10 | |
| *** lakerzhou has quit IRC | 15:21 | |
| *** openstackgerrit has joined #openstack-zun | 15:24 | |
| openstackgerrit | Hongbin Lu proposed openstack/zun master: Move get_available_network to API layer https://review.openstack.org/483113 | 15:24 |
| *** mriedem has joined #openstack-zun | 15:50 | |
| mriedem | hongbin: did you see the reply on https://bugs.launchpad.net/nova/+bug/1707319 ? | 15:50 |
| openstack | Launchpad bug 1707319 in OpenStack Compute (nova) "Security group doesn't apply to existing port" [Wishlist,Opinion] - Assigned to hongbin (hongbin034) | 15:50 |
| hongbin | mriedem: checking it out | 15:51 |
| hongbin | mriedem: i agree with his remark as well, it seems this is a controversial topic | 15:55 |
| mriedem | i'll push a patch to update the compute api reference | 15:56 |
| mriedem | since that's not controversial | 15:56 |
| hongbin | ok | 15:56 |
| hongbin | just let me know if nova team want to abondon my patch for move it forward | 15:57 |
| * hongbin is going to lunch | 16:02 | |
| *** egonzalez has quit IRC | 16:03 | |
| mriedem | hongbin: yeah probably abandon the nova change, i -2'ed it since it's an API change that would require a microversion. | 16:05 |
| *** mriedem has left #openstack-zun | 16:40 | |
| hongbin | ack | 17:43 |
| *** harlowja has joined #openstack-zun | 18:11 | |
| openstackgerrit | Hongbin Lu proposed openstack/zun master: Implement create with existing neutron port https://review.openstack.org/481861 | 18:46 |
| *** catintheroof has quit IRC | 18:59 | |
| *** catintheroof has joined #openstack-zun | 19:00 | |
| *** catintheroof has quit IRC | 19:00 | |
| *** catintheroof has joined #openstack-zun | 19:01 | |
| *** nicolasbock has quit IRC | 19:33 | |
| *** catintheroof has quit IRC | 21:17 | |
| openstackgerrit | Hongbin Lu proposed openstack/zun master: [WIP] Prepare for py35 tempest tests https://review.openstack.org/489425 | 22:33 |
| *** hongbin has quit IRC | 23:11 | |
| *** catintheroof has joined #openstack-zun | 23:24 | |
| *** catintheroof has quit IRC | 23:43 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!