*** vishwanathj has joined #tacker | 00:27 | |
*** vishwanathj has quit IRC | 00:27 | |
*** bobh has joined #tacker | 00:55 | |
*** bobh has quit IRC | 01:02 | |
*** tbh has joined #tacker | 01:15 | |
tbh | sripriya, Hi, thanks for the comments | 01:46 |
---|---|---|
tbh | sripriya, just gave response to those comments | 01:46 |
sripriya | tbh: reg: flavor name uniqueness i can only then have a 1:1 for vnf: vnfd template with flavor properties | 01:49 |
tbh | sripriya, yes we have unique flavors for each vnf | 01:50 |
tbh | for example for the first time, I got "tacker.vm.infra_drivers.heat.heat_DeviceHeat-0f6d1135-cd2f-4c53-adc9-f9b13b982b59-VDU1_flavor-ue3wixajgwsk" | 01:51 |
tbh | and the second is "tacker.vm.infra_drivers.heat.heat_DeviceHeat-93725825-e2b1-4c52-ad2f-408b6a5ebf6e-VDU1_flavor-pjj3wyqv6lic" | 01:51 |
sripriya | tbh: is this the flavor name? | 01:51 |
tbh | sripriya, so even though I use "VDU1_flavor" as resource name, heat will give a different name | 01:51 |
tbh | sripriya, yes | 01:51 |
sripriya | tbh: how does heat handle this? | 01:52 |
tbh | sripriya, I haven't see the heat code, but ... | 01:53 |
tbh | sripriya, if you see these document, they have removed the flavor_name support in HOT http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Nova::Flavor | 01:54 |
tbh | *seen | 01:54 |
tbh | sripriya, """Note that the current implementation of the Nova Flavor resource does not allow specifying the name and flavorid properties for the resource. This is done to avoid potential naming collision upon flavor creation as all flavor have a global scope.""" | 01:55 |
*** bobh has joined #tacker | 01:55 | |
sripriya | tbh: ah ok cool, so why are we still handling the name then in our code? | 01:55 |
tbh | sripriya, to give a name to the resource, so that I can refer it | 01:56 |
sripriya | tbh: got it! | 01:57 |
sripriya | tbh: thanks | 01:57 |
tbh | np :) | 01:59 |
tbh | sripriya, sometimes gerrit goes down? | 01:59 |
*** vishnoianil has quit IRC | 02:00 | |
sripriya | tbh: gerrit looks fine for me, and there is no scheduled maintainence AFAIK | 02:01 |
tbh | sripriya, oh I see, I am trying to fetch the patch, but it says "network unreachable" | 02:02 |
sripriya | tbh: can you try again in a little time, hopefully intermittent | 02:03 |
sripriya | tbh: if you do not specify unit for disk size, does heat assume it is in GB? | 02:04 |
tbh | sripriya, yes I thought the same | 02:06 |
tbh | sripriya, yes for disk it is GB, and for ram it is MB | 02:06 |
tbh | sripriya, for both, heat expects only integer, it won't accept units | 02:07 |
sripriya | tbh: ok | 02:07 |
sripriya | tbh: i was looking into change_memory_unit method in our patch | 02:08 |
*** prashantD has joined #tacker | 02:08 | |
tbh | sripriya, oh you mean from tosca -> HOT | 02:08 |
sripriya | tbh: for the case of disk if i did not specify any unit in tosca template, the 'to' paramater will have "GB" | 02:08 |
sripriya | tbh: s/our/your | 02:09 |
sripriya | tbh: in L336 i assume it will attach "MB" as default in this case? | 02:10 |
sripriya | tbh: suppose unit is "MB" then eval will evaluate to 1024 | 02:10 |
tbh | sripriya, yes, from tosca template, we accept only MB values if units not specified for both ram and disk | 02:11 |
sripriya | tbh: if i give input as 1 for disk_size | 02:11 |
tbh | sripriya, so if units are not specified this patch will make sure it will be "MB" | 02:11 |
tbh | sripriya, yes it will be "0" | 02:11 |
sripriya | tbh: if i give disk_size as 1 in tosca , will that imply as 1MB? | 02:12 |
tbh | sripriya, 1/1024 is a float value, but HOT won't accept float value | 02:12 |
sripriya | tbh: ok | 02:13 |
tbh | sripriya, yes at L336 it will be 1MB and it will processed in the next steps to return as zero | 02:13 |
sripriya | tbh: and what happens if 0 is passed to heat? | 02:14 |
sripriya | tbh: does it error gracefully? | 02:14 |
tbh | sripriya, "The “0” size is a special case that uses the native base image size as the size of the ephemeral root volume." | 02:15 |
sripriya | tbh: ah ok... cool | 02:15 |
sripriya | tbh: another minor comment sinc you happen to respin, can you provide a doc string for the above method, it will help other devs | 02:17 |
tbh | sripriya, sure, I will do that, I will respin a new patchset, addressing your comments now | 02:17 |
sripriya | tbh: cool thanks | 02:18 |
*** bobh has quit IRC | 02:45 | |
*** vishwanathj has joined #tacker | 02:48 | |
openstackgerrit | bharaththiruveedula proposed openstack/tacker: Implements auto flavor creation https://review.openstack.org/290751 | 03:48 |
openstackgerrit | Liuqing Jing proposed openstack/tacker: Update DevStack README https://review.openstack.org/287228 | 04:14 |
*** vishwanathj is now known as vishwanathj_zzz | 04:33 | |
*** prashantD has quit IRC | 04:43 | |
openstackgerrit | Merged openstack/tacker: py3.x: Use six.text_type() instead of unicode() https://review.openstack.org/293905 | 04:49 |
*** bobh has joined #tacker | 04:50 | |
*** bobh has quit IRC | 04:55 | |
*** janki91 has joined #tacker | 05:19 | |
*** sridhar_ram has joined #tacker | 05:29 | |
*** bobh has joined #tacker | 05:52 | |
*** bobh has quit IRC | 05:56 | |
*** sridhar_ram has quit IRC | 06:12 | |
openstackgerrit | Merged openstack/tacker: Update DevStack README https://review.openstack.org/287228 | 06:16 |
*** Ravikiran_K has joined #tacker | 06:28 | |
janki91 | sridhar_ram: hi. This is regrading the openwrt patch https://review.openstack.org/#/c/294573/ | 06:40 |
*** vishwanathj_zzz has quit IRC | 06:52 | |
*** sripriya has quit IRC | 06:53 | |
*** gongysh has joined #tacker | 06:53 | |
*** dkushwaha has quit IRC | 06:54 | |
*** Liuqing has joined #tacker | 06:59 | |
*** dkushwaha has joined #tacker | 07:07 | |
*** bobh has joined #tacker | 07:52 | |
*** bobh has quit IRC | 07:57 | |
openstackgerrit | Janki Chhatbar proposed openstack/tacker: openwrt image download failure handled https://review.openstack.org/294573 | 07:58 |
*** dkushwaha has quit IRC | 08:46 | |
*** tbh has quit IRC | 08:51 | |
*** dkushwaha has joined #tacker | 08:59 | |
*** tbh has joined #tacker | 09:05 | |
*** tbh_ has joined #tacker | 09:05 | |
*** tbh_ has quit IRC | 09:20 | |
*** tbh_ has joined #tacker | 09:26 | |
*** tbh_ has quit IRC | 09:43 | |
*** zeih has joined #tacker | 09:43 | |
*** bobh has joined #tacker | 09:53 | |
*** bobh has quit IRC | 09:57 | |
*** Liuqing has quit IRC | 10:25 | |
openstackgerrit | Martin Oemke proposed openstack/python-tackerclient: Cleanup vim settings https://review.openstack.org/298192 | 10:31 |
*** mbound has joined #tacker | 10:35 | |
openstackgerrit | bharaththiruveedula proposed openstack/tacker: Implements auto flavor creation https://review.openstack.org/290751 | 10:47 |
*** Liuqing has joined #tacker | 10:51 | |
*** Ravikiran_K has left #tacker | 10:55 | |
*** janki91 has quit IRC | 11:02 | |
openstackgerrit | Martin Oemke proposed openstack/tacker: Cleanup vim settings https://review.openstack.org/298199 | 11:06 |
*** zeih has quit IRC | 11:10 | |
*** Liuqing_ has joined #tacker | 11:15 | |
*** Liuqing has quit IRC | 11:16 | |
*** tbh has quit IRC | 11:49 | |
*** bobh has joined #tacker | 11:54 | |
*** bobh has quit IRC | 11:58 | |
*** Liuqing_ has quit IRC | 12:02 | |
*** tbh has joined #tacker | 12:08 | |
*** zeih has joined #tacker | 12:09 | |
*** mbound has quit IRC | 12:15 | |
*** zeih has quit IRC | 12:16 | |
*** tbh has quit IRC | 12:27 | |
*** mbound has joined #tacker | 12:36 | |
*** mbound has quit IRC | 12:53 | |
*** tbh_ has joined #tacker | 12:56 | |
*** trozet has joined #tacker | 13:20 | |
*** vishwanathj_zzz has joined #tacker | 13:22 | |
*** bobh has joined #tacker | 13:26 | |
*** openstackgerrit has quit IRC | 13:48 | |
*** openstackgerrit has joined #tacker | 13:49 | |
*** mbound has joined #tacker | 13:54 | |
*** tbh_ has quit IRC | 13:56 | |
*** mbound has quit IRC | 13:59 | |
*** zeih has joined #tacker | 14:13 | |
*** zeih has quit IRC | 14:18 | |
*** vishwana_ has joined #tacker | 14:26 | |
*** vishwana_ has quit IRC | 14:26 | |
*** vishwanathj_zzz has quit IRC | 14:29 | |
*** vishwanathj has joined #tacker | 14:29 | |
vishwanathj | bobh good morning | 14:54 |
bobh | vishwanathj: mornin | 14:55 |
vishwanathj | bobh looking forward to your review and merge approval of tbh patchset https://review.openstack.org/#/c/290751/ so that my review and merge can happen after that | 14:56 |
*** tbh has joined #tacker | 15:01 | |
bobh | I'll take a look today - probably after lunch | 15:03 |
vishwanathj | bobh thanks | 15:04 |
*** sripriya has joined #tacker | 15:06 | |
*** sripriya_ has joined #tacker | 15:11 | |
*** sripriya has quit IRC | 15:14 | |
*** Ravikiran_K has joined #tacker | 15:26 | |
*** tbh has quit IRC | 15:28 | |
*** tbh has joined #tacker | 15:29 | |
*** tbh has quit IRC | 15:32 | |
sripriya_ | gongysh: ping | 15:32 |
*** lhcheng has joined #tacker | 16:04 | |
*** zeih has joined #tacker | 16:14 | |
*** mannidi has joined #tacker | 16:16 | |
*** zeih has quit IRC | 16:19 | |
*** sripriya_ has quit IRC | 16:29 | |
*** tbh has joined #tacker | 16:53 | |
*** vishnoianil has joined #tacker | 17:03 | |
*** prashantD has joined #tacker | 17:05 | |
*** sripriya has joined #tacker | 17:13 | |
*** sridhar_ram has joined #tacker | 17:24 | |
openstackgerrit | bharaththiruveedula proposed openstack/tacker: Implements auto image creation https://review.openstack.org/292034 | 17:26 |
*** s3wong has joined #tacker | 18:03 | |
*** zeih has joined #tacker | 18:15 | |
sridhar_ram | bobh: for tbh's patchset https://review.openstack.org/#/c/290751 .. I'll wait for your +2 before merging it | 18:15 |
*** zeih has quit IRC | 18:19 | |
bobh | sridhar_ram: I'll give it one more look this afternoon | 18:28 |
sridhar_ram | bobh: thanks! | 18:28 |
*** sridhar_ram1 has joined #tacker | 19:01 | |
*** sridhar_ram has quit IRC | 19:02 | |
*** mannidi has quit IRC | 19:14 | |
openstackgerrit | Sripriya Seetharam proposed openstack/tacker: Implement multisite VIM feature https://review.openstack.org/285096 | 19:14 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-tackerclient: Updated from global requirements https://review.openstack.org/298395 | 19:32 |
*** Ravikiran_K has quit IRC | 19:40 | |
openstackgerrit | Santosh Kodicherla proposed openstack/tacker: Adding testcase for create and delete VIM https://review.openstack.org/296696 | 19:51 |
openstackgerrit | Santosh Kodicherla proposed openstack/tacker: Adding testcase for create and delete VIM https://review.openstack.org/296696 | 19:53 |
*** zeih has joined #tacker | 20:15 | |
*** tbh has quit IRC | 20:20 | |
*** zeih has quit IRC | 20:21 | |
*** prashantD_ has joined #tacker | 20:54 | |
*** prashantD has quit IRC | 20:57 | |
openstackgerrit | Merged openstack/python-tackerclient: Updated from global requirements https://review.openstack.org/298395 | 21:12 |
openstackgerrit | Santosh Kodicherla proposed openstack/tacker: Adding vnf create/delete testcase with VIM id https://review.openstack.org/296207 | 21:22 |
*** sridhar_ram1 is now known as sridhar_ram | 21:29 | |
openstackgerrit | Santosh Kodicherla proposed openstack/tacker: Adding vnf create/delete testcase with VIM id https://review.openstack.org/296207 | 21:32 |
openstackgerrit | Santosh Kodicherla proposed openstack/tacker: Adding vnf create/delete testcase with VIM id https://review.openstack.org/296207 | 21:35 |
sripriya | sridhar_ram: is there a way we can get testr results for dsvm functional gate? | 21:35 |
sridhar_ram | sripriya: something like this ? http://logs.openstack.org/96/285096/14/check/gate-tacker-dsvm-functional/6e4c1d6/logs/testr_results.html.gz | 21:36 |
sripriya | sridhar_ram: yes | 21:38 |
*** prashantD__ has joined #tacker | 21:40 | |
*** prashantD_ has quit IRC | 21:40 | |
sridhar_ram | bobh: ping | 21:42 |
bobh | sridhar_ram: | 21:42 |
bobh | sridhar_ram: pong | 21:42 |
*** prashantD has joined #tacker | 21:43 | |
sridhar_ram | just checked w/ release team.. we got to request tacker release by Thurs to make it to Mitaka. | 21:43 |
*** zeih has joined #tacker | 21:43 | |
bobh | ok | 21:44 |
sridhar_ram | We only have EOD Wednesday to merge mitaka changes.. rest need to go into Newton. | 21:44 |
bobh | just finished up flavor patchset - couple of nits but otherwise fine - it can go as is | 21:44 |
sridhar_ram | bobh: okay.. cool. We need to tag team to soft land these patchsets.... | 21:45 |
bobh | I'll look at the other patchset tonight | 21:45 |
sridhar_ram | bobh: Thanks.. so you are waiting for my +2 for multisite to merge ? | 21:45 |
*** prashantD__ has quit IRC | 21:46 | |
bobh | sridhar_ram: I think so - I can't recall :-) | 21:47 |
bobh | sridhar_ram: yes - I +1d it before | 21:48 |
*** zeih has quit IRC | 21:49 | |
vishwanathj | bobh do you mind reviewing my patch set as well...thanks | 21:49 |
sridhar_ram | bobh: Okay. I'm waiting for results from https://review.openstack.org/296207 to +2. Otherwise looks good to me | 21:49 |
bobh | vishwanathj: that's on my list for tonight | 21:50 |
sridhar_ram | bobh: let's land tbh's now... | 21:50 |
bobh | vishwanathj: needs a respin for merge conflict, though will need another one after tbh's lands so might as well wait | 21:50 |
vishwanathj | bobh i plan to wait | 21:51 |
bobh | sridhar_ram: ok by me | 21:51 |
openstackgerrit | Santosh Kodicherla proposed openstack/tacker: Adding vnf create/delete testcase with VIM id https://review.openstack.org/296207 | 22:01 |
*** bobh has quit IRC | 22:11 | |
openstackgerrit | Merged openstack/tacker: Implements auto flavor creation https://review.openstack.org/290751 | 22:12 |
openstackgerrit | Sripriya Seetharam proposed openstack/tacker: Implement multisite VIM feature https://review.openstack.org/285096 | 22:34 |
*** mbound has joined #tacker | 22:35 | |
*** mbound has quit IRC | 22:39 | |
*** zeih has joined #tacker | 22:44 | |
openstackgerrit | Merged openstack/python-tackerclient: Help message correction https://review.openstack.org/293297 | 22:48 |
*** zeih has quit IRC | 22:49 | |
vishwanathj | histoty | 23:07 |
*** prashantD has quit IRC | 23:10 | |
*** prashantD has joined #tacker | 23:11 | |
sripriya | vishwanathj: wrong window :-) | 23:18 |
vishwanathj | :) | 23:18 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!