| *** achanda has quit IRC | 00:01 | |
| *** lpmulligan has joined #heat | 00:02 | |
| *** sergmelikyan has quit IRC | 00:06 | |
| *** dims_ has joined #heat | 00:06 | |
| *** dims has quit IRC | 00:07 | |
| *** Drago has quit IRC | 00:12 | |
| *** dims has joined #heat | 00:12 | |
| *** sdake_ has joined #heat | 00:13 | |
| *** dims_ has quit IRC | 00:15 | |
| *** sdake has quit IRC | 00:16 | |
| *** kebray has quit IRC | 00:17 | |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #heat | 00:23 | |
| *** kebray has joined #heat | 00:23 | |
| *** openstack has joined #heat | 00:34 | |
| *** pm90_ has quit IRC | 00:35 | |
| *** vahidh has quit IRC | 00:36 | |
| *** sergmelikyan has joined #heat | 00:38 | |
| *** kebray has quit IRC | 00:38 | |
| *** kebray has joined #heat | 00:41 | |
| *** sergmelikyan has quit IRC | 00:43 | |
| *** pm90_ has joined #heat | 00:50 | |
| openstackgerrit | Yingzhe Zeng proposed openstack/python-heatclient: Improve the format hint of -Pf option https://review.openstack.org/216515 | 00:51 |
|---|---|---|
| *** liusheng has quit IRC | 00:52 | |
| *** sdake has joined #heat | 00:56 | |
| *** sdake_ has quit IRC | 00:59 | |
| *** sergmelikyan has joined #heat | 01:00 | |
| *** huangtianhua has joined #heat | 01:01 | |
| openstackgerrit | Angus Salkeld proposed openstack/heat: Make the TestResource a bit smarter https://review.openstack.org/215489 | 01:02 |
| *** sergmelikyan has quit IRC | 01:03 | |
| *** ricolin has joined #heat | 01:03 | |
| *** sergmelikyan has joined #heat | 01:08 | |
| *** Qiming has joined #heat | 01:10 | |
| *** pm90_ has quit IRC | 01:10 | |
| *** sergmelikyan has quit IRC | 01:10 | |
| *** zhenguo has joined #heat | 01:17 | |
| *** pm90_ has joined #heat | 01:19 | |
| *** bnemec has quit IRC | 01:26 | |
| *** erkules has joined #heat | 01:30 | |
| *** erkules_ has quit IRC | 01:33 | |
| openstackgerrit | Rico Lin proposed openstack/heat: unify REST word https://review.openstack.org/216919 | 01:33 |
| *** junhongl has joined #heat | 01:35 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Don't metadata_update all resources for deployment signals https://review.openstack.org/216920 | 01:36 |
| *** Yanyanhu has joined #heat | 01:45 | |
| *** neelashah has joined #heat | 01:58 | |
| *** elynn has joined #heat | 02:01 | |
| *** neelashah has quit IRC | 02:01 | |
| elynn | morning | 02:06 |
| *** Marga_ has quit IRC | 02:09 | |
| *** vijayagurug has joined #heat | 02:10 | |
| ricolin | morning | 02:17 |
| *** boris-42 has quit IRC | 02:20 | |
| *** hdd has quit IRC | 02:23 | |
| *** achanda has joined #heat | 02:23 | |
| jamespd | I'm seeing a stack consistently get stuck in CREATE_IN_PROGRESS when referencing "{ get_attr: [ this_host_port, fixed_ips ] }" in user_data/str_replace/template/params of an OS::Nova::Server resource. (Juno) | 02:25 |
| jamespd | Am I introducing some cyclical dependency? by referencing the server's own port like this? | 02:26 |
| jamespd | heat engine logs just say 'creating Server "webserver" Stack "teststack" [523463ed-209b-4049-80d7-d9a4d106f7ca]' | 02:26 |
| elynn | use heat resource-list teststack to see which resource this stack are stuck in. | 02:27 |
| *** gberginc has quit IRC | 02:32 | |
| *** sabeen has joined #heat | 02:34 | |
| *** achanda has quit IRC | 02:38 | |
| jamespd | elynn: I have a couple of servers and a floating ip association in INIT_COMPLETE. http://paste.openstack.org/show/427677/ | 02:38 |
| jamespd | resource-show keeps telling me stack or resource not found. | 02:40 |
| elynn | jamespd, seems your stack stuck in floating ip association | 02:40 |
| jamespd | I guess resource-show won't work until they are CREATE_COMPLETE? | 02:41 |
| *** dims has quit IRC | 02:46 | |
| ramishra | stevebaker: hi | 02:59 |
| *** vijayagurug has quit IRC | 03:00 | |
| *** alex_xu has quit IRC | 03:00 | |
| ramishra | morning all | 03:00 |
| *** hdd has joined #heat | 03:17 | |
| openstackgerrit | Qiming Teng proposed openstack/heat: Split engine service test cases (10) https://review.openstack.org/216940 | 03:17 |
| *** alex_xu has joined #heat | 03:17 | |
| *** jdandrea has quit IRC | 03:23 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Don't metadata_update all resources for deployment signals https://review.openstack.org/216920 | 03:24 |
| *** tochi has quit IRC | 03:25 | |
| *** tochi has joined #heat | 03:29 | |
| *** achanda has joined #heat | 03:30 | |
| ramishra | stevebaker: https://bugs.launchpad.net/heat/+bug/1488475 is probably not a duplicate, I still see the error on master. | 03:32 |
| openstack | Launchpad bug 1486182 in heat "duplicate for #1488475 empty output validation bug" [High,Fix committed] - Assigned to Steve Baker (steve-stevebaker) | 03:32 |
| *** alex_xu has quit IRC | 03:32 | |
| *** alex_xu has joined #heat | 03:33 | |
| stevebaker | ramishra: oh, ok | 03:33 |
| stevebaker | ramishra: i've unduped | 03:33 |
| *** achanda has quit IRC | 03:35 | |
| ramishra | stevebaker: may be the way they are constructing the list in the template. But we may have to support the existing templates, I suppose;) | 03:35 |
| openstackgerrit | huangtianhua proposed openstack/heat: Set FAILED state if resource in *_IN_PROGRESS when updating https://review.openstack.org/215486 | 03:40 |
| stevebaker | ramishra: ideally yes | 03:41 |
| *** Marga_ has joined #heat | 03:41 | |
| *** links has joined #heat | 03:45 | |
| *** boris-42 has joined #heat | 03:46 | |
| *** Marga_ has quit IRC | 03:49 | |
| *** gberginc has joined #heat | 03:49 | |
| *** hdd has quit IRC | 03:55 | |
| *** hdd has joined #heat | 04:06 | |
| *** liusheng has joined #heat | 04:09 | |
| *** KanagarajM has joined #heat | 04:24 | |
| KanagarajM | good morning all ! | 04:24 |
| *** lpmulligan has quit IRC | 04:36 | |
| *** vijayagurug has joined #heat | 04:36 | |
| *** Marga_ has joined #heat | 04:45 | |
| openstackgerrit | Angus Salkeld proposed openstack/heat: Make the TestResource a bit smarter https://review.openstack.org/215489 | 04:46 |
| *** Marga_ has quit IRC | 04:49 | |
| *** achanda has joined #heat | 05:01 | |
| *** sergmelikyan has joined #heat | 05:07 | |
| *** sergmelikyan has quit IRC | 05:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/216561 | 05:14 |
| *** huangtianhua has quit IRC | 05:22 | |
| *** Yanyanhu has quit IRC | 05:26 | |
| *** Yanyanhu has joined #heat | 05:27 | |
| *** sabeen has quit IRC | 05:28 | |
| *** dims has joined #heat | 05:36 | |
| *** dims has quit IRC | 05:42 | |
| *** Marga_ has joined #heat | 05:46 | |
| *** hdd has quit IRC | 05:50 | |
| *** ameybhide has joined #heat | 05:50 | |
| *** Marga_ has quit IRC | 05:50 | |
| *** SamYaple has joined #heat | 06:02 | |
| sdake | hey folks runnign heat in kolla get following results: http://paste.fedoraproject.org/259356/56852414/ | 06:02 |
| sdake | AttributeError: 'NoneType' object has no attribute 'isoformat' | 06:02 |
| sdake | any bones to trhow my way ? :) | 06:02 |
| sdake | stevebaker asalkeld yo uaroudn at this hour? :) | 06:04 |
| SamYaple | this only occurs when trying to do a delete (and the delete is not successful) | 06:05 |
| asalkeld | sdake: hi | 06:07 |
| *** rakesh_hs has joined #heat | 06:07 | |
| asalkeld | looking | 06:07 |
| sdake | thanks asalkeld | 06:08 |
| sdake | appreciate it alot :) | 06:08 |
| sdake | have demoware in morning | 06:08 |
| sdake | you know how that goes :( | 06:08 |
| *** gberginc has quit IRC | 06:08 | |
| *** jtomasek has joined #heat | 06:09 | |
| asalkeld | sdake: a bit odd that, the stack object seems to have disapeared | 06:09 |
| asalkeld | it's at the end of a delete | 06:10 |
| asalkeld | but still should exist | 06:10 |
| sdake | this is using master | 06:10 |
| sdake | it is possible we hae config wrong in some way | 06:10 |
| *** ameybhide has quit IRC | 06:11 | |
| sdake | SamYaple can you paste the config file loaded in? | 06:11 |
| SamYaple | yea sure one moment | 06:11 |
| SamYaple | http://paste.fedoraproject.org/259359/14405695/ | 06:12 |
| SamYaple | heat-engine conf ^^ | 06:12 |
| asalkeld | notification_driver = nova.openstack.common.notifier.rpc_notifier | 06:12 |
| *** kebray has quit IRC | 06:12 | |
| asalkeld | should be | 06:13 |
| asalkeld | notification_driver = messagingv2 | 06:13 |
| sdake | nice | 06:13 |
| * sdake would kiss asalkeld if that fixes it :) | 06:13 | |
| SamYaple | asalkeld: is that just for heat, or does that need to change for most services in liberty? | 06:13 |
| asalkeld | SamYaple: that's the new oslo.messaging setup for notifcations | 06:13 |
| SamYaple | cool | 06:14 |
| asalkeld | see: https://github.com/openstack/heat/blob/master/heat_integrationtests/pre_test_hook.sh#L29 | 06:14 |
| SamYaple | we noop everywhere since we dont have ceilometer yet in kolla | 06:14 |
| SamYaple | will test thanks asalkeld | 06:14 |
| sdake | rpc notifier is for more then just ceilomter afaik | 06:15 |
| SamYaple | sdake: im not so sure about that | 06:16 |
| asalkeld | sdake: SamYaple -> https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/notify/notifier.py#L30-L34 | 06:16 |
| asalkeld | (just incase you don't believe me) | 06:17 |
| SamYaple | asalkeld: ill always believe you! | 06:17 |
| asalkeld | ;) | 06:17 |
| SamYaple | it is not my who wrote this config | 06:17 |
| * SamYaple eyes sdake | 06:17 | |
| sdake | hehe :) | 06:17 |
| sdake | so SamYaple whats the verdict | 06:17 |
| SamYaple | testing | 06:18 |
| SamYaple | hey while i have people, what is 'heat_watch_server_url' for? | 06:18 |
| SamYaple | im a heat noob | 06:19 |
| sdake | that is where watchpiont communication gets sent | 06:19 |
| SamYaple | we arent setting that | 06:19 |
| SamYaple | should we be? | 06:19 |
| SamYaple | it defaults to blank | 06:19 |
| sdake | ya our waitcidionts are busted | 06:19 |
| sdake | it should be set yes | 06:19 |
| SamYaple | ill set it? | 06:19 |
| sdake | to where heat-api-cfn is running I think | 06:19 |
| sdake | or maybe its heat-api now | 06:19 |
| SamYaple | i thought cfn was just the ec2 compatibility layer? | 06:20 |
| asalkeld | SamYaple: yeah to something that the vm can access heat-api-cfn | 06:20 |
| sdake | nah its cfn - dont ask me why we never put it in heat api :) | 06:20 |
| asalkeld | before software config | 06:20 |
| sdake | i think there are native waitcitions in heat now | 06:20 |
| SamYaple | alrighty | 06:20 |
| sdake | as my keyboard fails to keyboard | 06:20 |
| sdake | the vm being able to access heat-api-cfn is a painful requirement as well samyaple | 06:21 |
| sdake | i had to go through magic contortions to set my network up to get that to work in the compose days | 06:21 |
| SamYaple | watch_server and waitcondition are two different options | 06:21 |
| asalkeld | the new stuff should use https://github.com/openstack/heat/blob/master/heat/api/openstack/v1/resources.py#L141 | 06:21 |
| SamYaple | im confused as to the difference | 06:21 |
| asalkeld | (signals) | 06:21 |
| * SamYaple is in over his head | 06:22 | |
| SamYaple | ill just continue to test | 06:22 |
| asalkeld | watch server is a very simplistic aws style Cloud Watch | 06:22 |
| SamYaple | i have 0 heat experince outside of setting this up for kolla | 06:22 |
| sdake | i dont care if wait conditions work ;) | 06:22 |
| sdake | samyaple we dont have a container for the watch server iirc | 06:23 |
| sdake | sameyaple is create and delete work i'm golden ;) | 06:25 |
| sdake | is/if | 06:25 |
| sdake | samyaple see private msgs plz | 06:25 |
| sdake | when you aren't swamped :) | 06:25 |
| asalkeld | so if you guys don't use the notifcations, you could just remove the notification_driver =.. | 06:25 |
| *** jpeeler has quit IRC | 06:26 | |
| sdake | asalkeld aren't notifications used besides ceilometer? | 06:26 |
| SamYaple | asalkeld: yea thats what i was saying, we noop it everywhere else | 06:26 |
| asalkeld | afaik only ceilometer | 06:26 |
| SamYaple | either way, same stack trace | 06:26 |
| sdake | ok noop it | 06:26 |
| SamYaple | i tested it with and without | 06:26 |
| asalkeld | unless you have something special receiving the notifications | 06:26 |
| sdake | samyaple with it totally removed still a backtrace? | 06:27 |
| SamYaple | sdake: correct | 06:27 |
| SamYaple | i doubt this is related to messaging | 06:27 |
| sdake | maybe asalkeld could post his config | 06:27 |
| sdake | maybe we need to set some timestamps or something | 06:28 |
| sdake | in the loggingconfig | 06:28 |
| sdake | since it looks like a iso time function is missing | 06:28 |
| asalkeld | is seems stack.created_time is None | 06:28 |
| asalkeld | which is odd | 06:29 |
| SamYaple | asalkeld: maybe it was from a bad create | 06:29 |
| SamYaple | im dropping the database and rebootstraping | 06:29 |
| sdake | oh you had old db? | 06:29 |
| sdake | ya that may do it | 06:29 |
| *** Marga_ has joined #heat | 06:29 | |
| SamYaple | ive done a few iterations fo dropping database | 06:29 |
| *** Marga_ has quit IRC | 06:29 | |
| sdake | i cleanup-containers -fv every time i make a change ust to be sure ;) | 06:29 |
| asalkeld | aaa, I have never seen that bug before | 06:30 |
| *** Marga_ has joined #heat | 06:30 | |
| sdake | asalkeld we aren't setting anything related to heat domains in heat-engine.conf either iiuc | 06:30 |
| sdake | i thought it was a heat domain issue | 06:30 |
| SamYaple | ohh a new stack trace has been found (it was there before i just didnt catc hit) | 06:30 |
| SamYaple | http://paste.fedoraproject.org/259368/40570630/ | 06:30 |
| sdake | ya you have to creat ean ssh key called mykey | 06:31 |
| sdake | that demo script probably needs love | 06:31 |
| asalkeld | ha steak_node | 06:31 |
| SamYaple | oh | 06:31 |
| sdake | hasn't been run since mykey was default in compose | 06:31 |
| SamYaple | well same isoformat stack trace on delete anyhow | 06:31 |
| asalkeld | SamYaple: can you raise a bug | 06:32 |
| *** jprovazn has joined #heat | 06:32 | |
| sdake | asalkeld I dont think its a regressoin tis been like this for a month | 06:32 |
| SamYaple | agree with sdake there | 06:32 |
| asalkeld | sure, still should fix it tho' | 06:32 |
| SamYaple | I can raise a bug though | 06:32 |
| asalkeld | thanks | 06:32 |
| sdake | samyaple try creating an ssh key with mykey and tyr a new stack ;) | 06:33 |
| asalkeld | there is a regression | 06:33 |
| SamYaple | yea sdake thats why the stack wasn't creating, i did a fresh kick of opentack during this whole thing and forgot to create the key, either way, same issue | 06:34 |
| sdake | samyaple got it | 06:34 |
| sdake | asalkeld have you ideentified the offending commit - perhap we could revert it for the day until a fix is merged | 06:36 |
| asalkeld | getting there, hang on - raise the bug in the mean time | 06:36 |
| SamYaple | asalkeld: https://bugs.launchpad.net/heat/+bug/1488779 | 06:36 |
| openstack | Launchpad bug 1488779 in heat "stacktrace on heat stack-delete complaining about isotime()" [Undecided,New] | 06:36 |
| sdake | asalkeld cool thanks for your help :) | 06:36 |
| asalkeld | np | 06:37 |
| asalkeld | https://github.com/openstack/heat/commit/720c86549a05d044ecdf7d4ad3761db4dba1a88e | 06:38 |
| asalkeld | 3 jun | 06:38 |
| asalkeld | timeutils.isotime() checked for None, now we don't | 06:38 |
| SamYaple | asalkeld: yea | 06:39 |
| SamYaple | ill try a patch | 06:39 |
| SamYaple | i wish sdake knew about heat | 06:39 |
| * SamYaple runs and hides | 06:39 | |
| asalkeld | :-) | 06:39 |
| *** eprasad has joined #heat | 06:40 | |
| sdake | hey thats my birthday ;-) | 06:40 |
| *** achanda has quit IRC | 06:40 | |
| asalkeld | thanks for report the issue guys | 06:40 |
| asalkeld | reporting | 06:40 |
| SamYaple | its funny because i told sdake this looks like a bug in heat and he said "its highly unlikely" | 06:41 |
| SamYaple | when will he trust me | 06:41 |
| sdake | i just have high degree of faith in the heat team :) | 06:41 |
| sdake | what do you want | 06:41 |
| sdake | software is pretty mature at this point | 06:42 |
| SamYaple | thats well placed, i just didnt want to deal with the issue so i yelled "BUG" | 06:42 |
| asalkeld | too much reworking existing working code :-( | 06:42 |
| sdake | asalkeld ya refacotirng not always a good thing :( | 06:42 |
| SamYaple | we dont have that problem for Kolla at all! | 06:42 |
| asalkeld | just wait... | 06:42 |
| sdake | asalkeld but it keeps people busy ;-) | 06:42 |
| SamYaple | asalkeld: i was kidding, we keep reworking it | 06:42 |
| sdake | asalkeld ya kolla is on about its 10-15th rewrite | 06:43 |
| sdake | from scratch ;) | 06:43 |
| SamYaple | asalkeld: would you like me to submit a heat patch? | 06:43 |
| asalkeld | SamYaple: sure, that would help | 06:44 |
| SamYaple | caaaaaan do | 06:44 |
| asalkeld | (if you got time) | 06:44 |
| * sdake submits a git revert patch | 06:44 | |
| sdake | those are the best ;) | 06:44 |
| SamYaple | sdake will just bug me about this until i do it | 06:44 |
| sdake | one of you two ;) | 06:45 |
| * sdake wtbs heat in t-8 hours | 06:45 | |
| asalkeld | revert is not so good in this case as timeutils.isotime is deprected | 06:46 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Validate support_status while searching resource registry https://review.openstack.org/213107 | 06:46 |
| SamYaple | yea im pretty sure i can fix it without a revert ;) | 06:46 |
| sdake | samyaple if you could make https://review.openstack.org/#/c/205528/19/docker/common/heat/heat-api/start.sh configurable while your at it that would be grand ;-) | 06:46 |
| asalkeld | just need to check for none | 06:46 |
| SamYaple | sdake: already done | 06:46 |
| sdake | oh its not in the queue | 06:46 |
| sdake | but cool :) | 06:46 |
| SamYaple | sdake: yea ive got the heat.conf's merged again too | 06:48 |
| asalkeld | SamYaple: when you raised the bug can you mention it here? | 06:48 |
| asalkeld | maybe i can add some info to it | 06:48 |
| SamYaple | I posted it already | 06:48 |
| SamYaple | let me find it | 06:48 |
| SamYaple | 06:37 < SamYaple> asalkeld: https://bugs.launchpad.net/heat/+bug/1488779 | 06:49 |
| openstack | Launchpad bug 1488779 in heat "stacktrace on heat stack-delete complaining about isotime()" [Undecided,New] - Assigned to Sam Yaple (s8m) | 06:49 |
| asalkeld | doh | 06:49 |
| *** e0ne has joined #heat | 06:52 | |
| *** jpeeler has joined #heat | 06:53 | |
| *** jpeeler has joined #heat | 06:53 | |
| *** inc0 has joined #heat | 06:56 | |
| *** cyli has quit IRC | 06:56 | |
| *** pbrooko has joined #heat | 07:00 | |
| *** asalkeld has quit IRC | 07:11 | |
| *** zaneb has quit IRC | 07:11 | |
| *** asalkeld has joined #heat | 07:11 | |
| *** dims has joined #heat | 07:24 | |
| *** zaneb has joined #heat | 07:25 | |
| *** dims has quit IRC | 07:29 | |
| *** ricolin_ has joined #heat | 07:44 | |
| *** ricolin has quit IRC | 07:45 | |
| *** e0ne has quit IRC | 07:45 | |
| *** tlashchova has joined #heat | 07:48 | |
| *** vijayagurug has quit IRC | 07:49 | |
| *** vijayagurug has joined #heat | 07:50 | |
| *** andrearo1 is now known as andrearosa | 07:50 | |
| *** shardy has joined #heat | 07:52 | |
| *** skraynev_ has joined #heat | 07:52 | |
| *** pas-ha has joined #heat | 07:55 | |
| *** Marga_ has quit IRC | 07:59 | |
| *** Marga_ has joined #heat | 08:04 | |
| *** Marga_ has quit IRC | 08:06 | |
| *** Marga_ has joined #heat | 08:07 | |
| SamYaple | asalkeld: you still around? | 08:08 |
| SamYaple | i fixed the first stack trace almost immediately, but ive been banging my head against http://paste.fedoraproject.org/259404/40576508/ | 08:08 |
| SamYaple | so i probably didnt fix it correctly ;) | 08:09 |
| *** derekh has joined #heat | 08:09 | |
| *** pas-ha has quit IRC | 08:15 | |
| *** sorantis has joined #heat | 08:15 | |
| *** gberginc has joined #heat | 08:16 | |
| *** pas-ha has joined #heat | 08:18 | |
| *** Yanyanhu has quit IRC | 08:25 | |
| *** Yanyanhu has joined #heat | 08:25 | |
| sdake | samyaple asalkeld is in au he may be gone now | 08:26 |
| SamYaple | yea | 08:29 |
| SamYaple | im running down the code now | 08:29 |
| therve | SamYaple, What's the database that you use? | 08:36 |
| SamYaple | therve: mysql/mariadb | 08:37 |
| therve | OK | 08:37 |
| *** zeih has joined #heat | 08:38 | |
| *** jistr has joined #heat | 08:39 | |
| *** huangtianhua has joined #heat | 08:45 | |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Provide signal_url for DELETE action when using other transports https://review.openstack.org/216651 | 08:49 |
| *** junhongl has quit IRC | 08:52 | |
| *** junhongl has joined #heat | 08:52 | |
| SamYaple | sdake: heat is unlikely to happen tonight. i cant figure out this latest stacktrace | 08:53 |
| *** vijayagurug has quit IRC | 08:54 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: Merge Server os-collect-config metadata https://review.openstack.org/217040 | 08:54 |
| ramishra | therve: hi | 08:55 |
| therve | ramishra, Hi | 08:55 |
| ramishra | therve: a quick question on signaling with zaqar. now that it supports presigned urls, can we not change zaqar signaling to use that? | 08:56 |
| therve | ramishra, We could yeah. | 08:57 |
| therve | Maybe next cycle | 08:57 |
| ramishra | ok..not good enough this cycle? | 08:59 |
| therve | There is no client support. v2 client needs to happen, and then signed url support needs to happen. | 09:00 |
| ramishra | oh yeah.. you're right. thanks | 09:01 |
| *** inc0 has quit IRC | 09:01 | |
| therve | It's also super hard to land anything in osc apparently: https://review.openstack.org/#/c/185882/ (3 months woo) | 09:02 |
| *** junhongl has quit IRC | 09:02 | |
| SamYaple | therve: regarding http://paste.fedoraproject.org/259404/40576508/ when i select from the database they all have timestamps, any guidance? | 09:02 |
| therve | I don't know if I should throw a birthday party tomorrow | 09:03 |
| shardy | therve: I thought that was why we (and other projects AIUI) are moving towards providing an oscplugin in their python-*client | 09:03 |
| therve | SamYaple, Nope. It feels there is something weird in your environment though. | 09:04 |
| therve | shardy, "we". As in without telling me? :) | 09:04 |
| therve | No idea about that plan | 09:04 |
| SamYaple | therve: most probably this is the initial setup of heat with Kolla ;) | 09:04 |
| SamYaple | any ideas? | 09:04 |
| SamYaple | asalkeld seems to be gone now | 09:04 |
| *** inc0 has joined #heat | 09:05 | |
| sdake | shardy if you could throw us a debug bone here i'd super apprecaite it | 09:05 |
| sdake | samyaple can you post the config - shardy is super pro with heat | 09:05 |
| sdake | so is therve :) | 09:05 |
| SamYaple | config: http://paste.fedoraproject.org/259436/40579973/ | 09:06 |
| shardy | therve: stevebaker's got a patch up https://review.openstack.org/#/c/195867/ | 09:06 |
| SamYaple | initial stacktrace that ive fixed locally, but ive probably fixed wrong: https://bugs.launchpad.net/heat/+bug/1488779 | 09:06 |
| openstack | Launchpad bug 1488779 in heat "stacktrace on heat stack-delete complaining about isotime()" [High,Triaged] - Assigned to Sam Yaple (s8m) | 09:06 |
| shardy | I thought I saw a wiki somewhere saying other folks were doing the same | 09:07 |
| therve | OH | 09:07 |
| * shardy will try to find the link shortly | 09:07 | |
| therve | shardy, I'm taling about os-collect-config | 09:07 |
| shardy | therve: isn't your patch os-collect-config? | 09:07 |
| *** junhongl has joined #heat | 09:07 | |
| therve | I guess I used the wrong acronym | 09:07 |
| shardy | therve: occ vs osc :) | 09:07 |
| therve | Right sorry :) I'm aware of that other plan indeed | 09:07 |
| shardy | therve: I'll review it now, sometimes stuff gets missed | 09:07 |
| shardy | therve: sorry it's been hanging around so long :( | 09:08 |
| *** e0ne has joined #heat | 09:08 | |
| therve | shardy, No worries. I was a bit light on the harassing. | 09:08 |
| therve | SamYaple, What's the template you're trying? | 09:10 |
| openstackgerrit | Merged openstack/heat: Ensure traversal ID is updated https://review.openstack.org/216683 | 09:11 |
| sdake | shardy can you look at that backtrace and config that samyaple posted | 09:11 |
| sdake | samyaple can you post your git diff too | 09:11 |
| sdake | of heat | 09:11 |
| sdake | it is entirely possible our config is wrong | 09:11 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/heat: Fix some files permissions https://review.openstack.org/215125 | 09:12 |
| SamYaple | therve: https://github.com/stackforge/kolla/blob/master/demos/heat/steak-rg.yaml | 09:12 |
| SamYaple | thats teh template | 09:12 |
| *** h00327910__ has joined #heat | 09:13 | |
| shardy | therve: lgtm, I'd have approved it except the CI run is 2 months old, just rechecked | 09:14 |
| sdake | samyaple there is another template as well | 09:14 |
| *** pbrooko has quit IRC | 09:16 | |
| shardy | SamYaple: does just reverting the timeutils patch work for you? | 09:16 |
| SamYaple | shardy: reverting the patch will throw that stack trace http://paste.fedoraproject.org/259404/40576508/ | 09:17 |
| SamYaple | shardy: fixing it correct without a revert also throw that stackttrace | 09:17 |
| SamYaple | my question is why am i the only one seeing this anywhere? | 09:17 |
| SamYaple | maybe i coud fix what im doing to workaround hte issue for now | 09:17 |
| * shardy pulls latest master and tests | 09:17 | |
| *** e0ne has quit IRC | 09:18 | |
| *** pbrooko has joined #heat | 09:18 | |
| therve | SamYaple, I suspect that timeutils patch is not the issue | 09:20 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Search based on the resource type name https://review.openstack.org/213108 | 09:21 |
| SamYaple | therve: youre probably right because the code looks sound. created_time = resource.created_time or timeutils.utcnow() | 09:21 |
| therve | created_at should always be present | 09:21 |
| SamYaple | created_time = stack.created_time or timeutils.utcnow() | 09:22 |
| SamYaple | therve: it is present, but set to none | 09:22 |
| therve | That's not the one that's failing for you initially though I believe | 09:22 |
| SamYaple | the isotime() function check if None, and if it was none set it to utcnow() | 09:22 |
| SamYaple | no itially its the stacktrace in the bug https://bugs.launchpad.net/heat/+bug/1488779 | 09:22 |
| openstack | Launchpad bug 1488779 in heat "stacktrace on heat stack-delete complaining about isotime()" [High,Triaged] - Assigned to Sam Yaple (s8m) | 09:22 |
| therve | I thought you checked the db and said the values are present | 09:22 |
| SamYaple | therve: correct | 09:23 |
| therve | So there ought to be something else going on | 09:23 |
| SamYaple | agreed, but im no heat guy | 09:23 |
| SamYaple | i defer to your judgements | 09:23 |
| *** e0ne has joined #heat | 09:23 | |
| therve | SamYaple, What's your version of oslo.versionedobjects? | 09:23 |
| SamYaple | will check | 09:24 |
| SamYaple | 0.8.0 | 09:25 |
| *** pfreund_ has joined #heat | 09:25 | |
| therve | Looks like the latest | 09:26 |
| therve | At that point I don't see any other way than pdb | 09:27 |
| SamYaple | therve: yea im all pdb'd up in it | 09:27 |
| SamYaple | not familiar with the heat code at all | 09:27 |
| SamYaple | so its slow going | 09:27 |
| SamYaple | the attibute exists, its just set to None :( | 09:28 |
| *** pfreund has quit IRC | 09:28 | |
| *** pfreund_ is now known as pfreund | 09:28 | |
| therve | SamYaple, Is stack.id set? | 09:29 |
| openstackgerrit | huangtianhua proposed openstack/heat: Format id instead name when making credentials from context https://review.openstack.org/217050 | 09:30 |
| SamYaple | verfied id is set correctly, yes therve | 09:34 |
| SamYaple | shardy: im assuming master worked for you just fine? | 09:34 |
| openstackgerrit | Sam Yaple proposed openstack/heat: WIP: created_time is not correctly checked if None https://review.openstack.org/217058 | 09:40 |
| SamYaple | therve: shardy: for what its worth that was the small change i made to get past the initial stacktrace i pasted in the bug report | 09:40 |
| therve | SamYaple, Did you try reverting the whole patch? | 09:41 |
| SamYaple | yea | 09:41 |
| SamYaple | reverting the patch gets me the versioned objects stacktrace | 09:42 |
| SamYaple | this also gets me that | 09:42 |
| openstackgerrit | Sam Yaple proposed openstack/heat: WIP: created_time is not correctly checked if None https://review.openstack.org/217058 | 09:42 |
| SamYaple | actually here would be the correct code i suppose, since we would only want to set time to now if created_time was None | 09:42 |
| therve | That's nowhere near correct, but whatever :) | 09:43 |
| SamYaple | :) i know my codebase, does that make you feel better? | 09:43 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Search resource type based on version https://review.openstack.org/214983 | 09:43 |
| SamYaple | tonight has been my first experince with heat in whole, so im not the best guy to be doing this. sdake just asked me to look into it | 09:44 |
| shardy | SamYaple: Yes, my basic tests work fine | 09:47 |
| shardy | the question isn't the test for created_time, but why is it None on delete? | 09:47 |
| shardy | I suspect it's a regression elsewhere which wasn't previously visible | 09:48 |
| SamYaple | asking the entirely wrong guy here | 09:48 |
| * shardy was just generally thinking aloud :) | 09:48 | |
| therve | SamYaple, And you don't have any other traceback? | 09:48 |
| SamYaple | therve: just teh two i have pasted | 09:48 |
| SamYaple | I suspect if i fix the first one correct, the second one would go away | 09:48 |
| *** dims has joined #heat | 09:48 | |
| therve | shardy, It looks a bit like the secondary error tripleo got when sd attributes were broken | 09:49 |
| therve | shardy, Where stack.id was None | 09:49 |
| therve | SamYaple, Do you still have a hand on a pdb somewhere in the stack context? | 09:50 |
| SamYaple | no | 09:51 |
| SamYaple | im rebilding the containers now | 09:51 |
| *** zengyingzhe has quit IRC | 10:03 | |
| *** zeih has quit IRC | 10:03 | |
| SamYaple | ok so according to the database all of the created_at fields are set correctly | 10:14 |
| SamYaple | but on delete i get the stacktrace fro mthe bug | 10:14 |
| *** Kennan2 has joined #heat | 10:16 | |
| *** Kennan has quit IRC | 10:16 | |
| *** sorantis has quit IRC | 10:17 | |
| *** Yanyanhu has quit IRC | 10:20 | |
| shardy | SamYaple: random question, you didn't update heat and forget to run heat-manage db_sync did you? | 10:21 |
| shardy | what does heat-manage db_version say? | 10:21 |
| *** ricolin_ has quit IRC | 10:21 | |
| shardy | I'm assuming not if it's running in a container, but there's a recent change messing with the datetime dialect | 10:22 |
| SamYaple | no thats not possible, its all automated deploy stuff with Kolla | 10:22 |
| SamYaple | root@server02:/# heat-manage db_version | 10:22 |
| SamYaple | 2015-08-26 10:22:21.220 41 WARNING oslo_config.cfg [-] Option "verbose" from group "DEFAULT" is deprecated for removal. Its value may be silently ignored in the future. | 10:22 |
| SamYaple | 65 | 10:22 |
| shardy | SamYaple: Ok, cool | 10:23 |
| SamYaple | im kaing some headway i think | 10:23 |
| SamYaple | making* | 10:23 |
| shardy | SamYaple: I'm not that familiar with kolla atm, how hard would it be to spin up a deployment identical to yours so we can attempt a reproduce? | 10:23 |
| shardy | I've failed to reproduce any issues on devstack | 10:23 |
| SamYaple | shardy: that depends on how familiar with docker and ansible you are (we do not have prebuilt images on master, those will only be on the stable branches) | 10:24 |
| SamYaple | I mean i could walk you through it without much trouble, but i think that might be more time than you want to invest at the moment, like 2-3 hours | 10:25 |
| shardy | SamYaple: Ah, yeah I was hoping more like "run this script and grab some coffee" ;) | 10:26 |
| SamYaple | yea ive been asking for a heat template but that guy on our team that knows heat hasn't done it yet ;) | 10:26 |
| *** Qiming has quit IRC | 10:26 | |
| shardy | SamYaple: I've been helping out a little bit with the kolla->TripleO integration rhallisey and Slow are doing, that should be a great way to spin up a multi-node containerized deployment when it lands :) | 10:27 |
| SamYaple | it shouldnt be too hard to do since it could automate the image building and deploying into virtualmachines | 10:28 |
| SamYaple | but sdake is the heat guy, not i | 10:28 |
| SamYaple | im the ansible guy (apparently) | 10:29 |
| SamYaple | i should really learn this heat thing, its been around long enough ;) | 10:29 |
| sdake | samyaple still blocked? | 10:32 |
| SamYaple | indeed | 10:32 |
| SamYaple | but im learning heat as i go. its starting to make sense | 10:32 |
| SamYaple | the code base that is | 10:32 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Raise exception if asg in cooldown https://review.openstack.org/215080 | 10:41 |
| *** huangtianhua has quit IRC | 10:43 | |
| *** pal has joined #heat | 10:44 | |
| sdake | samyaple this needs to merge if you have a moment https://review.openstack.org/#/c/215065/ | 10:46 |
| *** sorantis has joined #heat | 10:46 | |
| SamYaple | k | 10:46 |
| *** zeih has joined #heat | 11:06 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: Merge Server os-collect-config metadata https://review.openstack.org/217040 | 11:07 |
| SamYaple | therve: im back in the debugger | 11:07 |
| SamYaple | (Pdb) print(stack.created_at) | 11:08 |
| SamYaple | None | 11:08 |
| SamYaple | anything you want me to check? | 11:08 |
| SamYaple | shardy: you too | 11:08 |
| SamYaple | ? | 11:08 |
| therve | SamYaple, Yeah | 11:08 |
| therve | If you can do a up | 11:08 |
| SamYaple | ok | 11:08 |
| therve | And then stack_object.Stack.get_by_id(self.context, self.id) | 11:09 |
| SamYaple | stack_object is not defined | 11:09 |
| *** pbrooko has quit IRC | 11:10 | |
| SamYaple | i probably wasnt broke where you though I was broken at | 11:10 |
| SamYaple | up one more gets me to where i think you want me | 11:10 |
| therve | stack.py yeah | 11:10 |
| SamYaple | yup | 11:10 |
| SamYaple | ok what do you want from there? | 11:11 |
| therve | So if you can paste the object returned | 11:11 |
| therve | Maybe its dict or something | 11:11 |
| SamYaple | im in rpdb due to container working, so its not returning the full values it seems | 11:13 |
| SamYaple | http://paste.fedoraproject.org/259490/05876071/ | 11:13 |
| therve | If you do print(stack_object.Stack.get_by_id(self.context, self.id).__dict__) it may be better | 11:17 |
| therve | Checking the corresponding database object would be good | 11:17 |
| SamYaple | today i learned | 11:18 |
| SamYaple | http://paste.fedoraproject.org/259491/58790014/ | 11:18 |
| SamYaple | therve: anymore suggeestion? | 11:21 |
| *** pas-ha has quit IRC | 11:21 | |
| therve | SamYaple, Yeah check the db object ? | 11:21 |
| SamYaple | ah sorry i see you said that, many irc windows :) | 11:22 |
| therve | Like Select * from stack where id='5cb8ade6-6091-4194-943c-5b44f7cf9b5e' if I'm not mistaken | 11:22 |
| *** pal has quit IRC | 11:27 | |
| SamYaple | therve: http://paste.fedoraproject.org/259496/05884461/ | 11:27 |
| therve | Spooky | 11:28 |
| SamYaple | indeed | 11:28 |
| SamYaple | i double check that result | 11:28 |
| therve | Everything is the same except created_at AFAICT | 11:31 |
| SamYaple | totally looks like that | 11:31 |
| SamYaple | maybe someone placed an and instead of an or somewhere | 11:31 |
| SamYaple | but that owuldn't explain why im the only one seeing this (or Kolla rather) | 11:32 |
| *** elynn has quit IRC | 11:34 | |
| *** pbrooko has joined #heat | 11:34 | |
| therve | SamYaple, Can we try yet another thing? | 11:35 |
| SamYaple | of course | 11:35 |
| therve | print(stack_object.db_api.stack_get(self.context, self.id).__dict__) | 11:36 |
| SamYaple | http://paste.fedoraproject.org/259499/89049144/ | 11:37 |
| *** liusheng has quit IRC | 11:38 | |
| *** shardy_ has joined #heat | 11:38 | |
| *** shardy has quit IRC | 11:39 | |
| therve | Urg | 11:40 |
| therve | SamYaple, stack_object.db_api.stack_get(self.context, self.id).refresh() | 11:40 |
| therve | And then do the print again? | 11:40 |
| SamYaple | 'created_at': None, | 11:41 |
| therve | OK, I'm at loss | 11:41 |
| SamYaple | =D | 11:41 |
| SamYaple | fantastic! | 11:41 |
| SamYaple | let me dig some more | 11:42 |
| SamYaple | ill be around | 11:42 |
| SamYaple | thanks for all your help so far | 11:42 |
| therve | Np, sorry for the bug | 11:43 |
| *** shardy_ has quit IRC | 11:43 | |
| *** shardy has joined #heat | 11:44 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add get_attr which returns all attributes https://review.openstack.org/199442 | 11:46 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add information about new get_attr functionality https://review.openstack.org/199443 | 11:46 |
| therve | SamYaple, OK I have one last idea. Create the stack, shutdown heat, restart, and then delete the stack | 11:48 |
| *** dims_ has joined #heat | 11:48 | |
| *** steveg_afk has joined #heat | 11:49 | |
| SamYaple | haha god one therve | 11:49 |
| *** hdd has joined #heat | 11:49 | |
| SamYaple | no its still broken | 11:49 |
| SamYaple | also tried that long before pdb | 11:49 |
| therve | So for some reason it doesn't manage to get the value from the db | 11:49 |
| SamYaple | yea im digging into that right now | 11:50 |
| SamYaple | getting pdb into the database fetch atm | 11:50 |
| *** dims has quit IRC | 11:52 | |
| *** h00327910__ has quit IRC | 11:53 | |
| *** Qiming has joined #heat | 11:53 | |
| *** dkusidlo has joined #heat | 11:55 | |
| *** KanagarajM has quit IRC | 12:00 | |
| *** MVenesio has joined #heat | 12:02 | |
| *** Qlawy has quit IRC | 12:09 | |
| *** zeih has quit IRC | 12:09 | |
| *** lpmulligan has joined #heat | 12:12 | |
| *** zeih has joined #heat | 12:13 | |
| *** hdd has quit IRC | 12:17 | |
| *** lpmulligan has quit IRC | 12:18 | |
| *** hdd has joined #heat | 12:19 | |
| *** hdd has quit IRC | 12:23 | |
| *** pbrooko has quit IRC | 12:28 | |
| *** sergmelikyan has joined #heat | 12:28 | |
| *** pbrooko has joined #heat | 12:32 | |
| *** pbrooko has quit IRC | 12:33 | |
| *** shardy_ has joined #heat | 12:34 | |
| *** shardy has quit IRC | 12:35 | |
| *** pal has joined #heat | 12:36 | |
| *** zeih has quit IRC | 12:36 | |
| *** pbrooko has joined #heat | 12:37 | |
| *** lkarm has joined #heat | 12:37 | |
| *** shardy_ has quit IRC | 12:39 | |
| *** shardy has joined #heat | 12:40 | |
| *** shardy has quit IRC | 12:44 | |
| *** shardy has joined #heat | 12:45 | |
| *** tochi has quit IRC | 12:50 | |
| *** pal has quit IRC | 12:58 | |
| *** pal has joined #heat | 13:01 | |
| *** pm90__ has joined #heat | 13:09 | |
| *** blomquisg has joined #heat | 13:10 | |
| *** pm90_ has quit IRC | 13:12 | |
| *** gberginc has quit IRC | 13:13 | |
| *** links has quit IRC | 13:15 | |
| *** rpothier has joined #heat | 13:16 | |
| *** jaime49 has joined #heat | 13:16 | |
| *** junhongl has quit IRC | 13:26 | |
| *** jasond has joined #heat | 13:27 | |
| *** inc0 has quit IRC | 13:28 | |
| *** jonesbr has joined #heat | 13:29 | |
| *** pbrooko has quit IRC | 13:33 | |
| *** Piet has quit IRC | 13:34 | |
| *** jprovazn has quit IRC | 13:34 | |
| *** neelashah has joined #heat | 13:34 | |
| openstackgerrit | Zane Bitter proposed openstack/heat: Refactor InstanceGroup batch size check https://review.openstack.org/213549 | 13:35 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Refactor InstanceGroup batch sizing logic https://review.openstack.org/213548 | 13:35 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Invert logic of counting in InstanceGroup _get_batches() https://review.openstack.org/213551 | 13:35 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Split out calculation of the next batch into a separate function https://review.openstack.org/217133 | 13:35 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Respect rolling update params on the initial batch https://review.openstack.org/217134 | 13:35 |
| *** jdandrea has joined #heat | 13:44 | |
| SamYaple | therve: wierd | 13:45 |
| SamYaple | i maunally set updated at, same issue | 13:45 |
| SamYaple | i set deleted at and _that_ comes through | 13:45 |
| *** jaime49 has quit IRC | 13:52 | |
| *** dyasny has joined #heat | 13:52 | |
| *** inc0 has joined #heat | 13:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/216561 | 13:55 |
| *** blomquisg has quit IRC | 13:55 | |
| *** dkusidlo has quit IRC | 13:56 | |
| *** eprasad has quit IRC | 13:57 | |
| *** zeih has joined #heat | 13:59 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Make encryption script more robust in presence of malformed data https://review.openstack.org/216536 | 14:00 |
| *** zeih has quit IRC | 14:01 | |
| *** Kennan2 has quit IRC | 14:01 | |
| *** Kennan has joined #heat | 14:02 | |
| pm90__ | hey folks, I was hoping to understand if the way I've used logger in this patch is idiomatic: https://review.openstack.org/#/c/216536/4/heat/cmd/manage.py | 14:04 |
| pm90__ | basically, I'm trying to capture all the logs created by the script in a log file | 14:04 |
| pm90__ | I was wondering if using logging.getLogger('heat') is the right idiom to get hold of the root logger, so all other modules' log may be recorded. Is this way the right way to use logger? | 14:05 |
| *** links has joined #heat | 14:06 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: SoftwareDeploymentGroup - make path based attributes work https://review.openstack.org/217191 | 14:08 |
| openstackgerrit | Steven Hardy proposed openstack/heat: SoftwareDeploymentGroup allow arbitrary keys for attributes https://review.openstack.org/217192 | 14:08 |
| *** eprasad has joined #heat | 14:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-heatclient: Updated from global requirements https://review.openstack.org/217201 | 14:10 |
| *** blomquisg has joined #heat | 14:10 | |
| *** sergmelikyan has quit IRC | 14:12 | |
| *** sergmelikyan has joined #heat | 14:16 | |
| *** vijendar has joined #heat | 14:18 | |
| *** thumpba has joined #heat | 14:19 | |
| *** thumpba has quit IRC | 14:20 | |
| *** thumpba has joined #heat | 14:20 | |
| *** inc0 has quit IRC | 14:21 | |
| *** hdd has joined #heat | 14:21 | |
| *** lifeless1 has joined #heat | 14:23 | |
| *** thedodd has joined #heat | 14:23 | |
| *** lifeless has quit IRC | 14:24 | |
| *** jprovazn has joined #heat | 14:26 | |
| *** jaime49 has joined #heat | 14:28 | |
| *** Piet has joined #heat | 14:29 | |
| *** jaime49 has quit IRC | 14:30 | |
| *** jcoufal has joined #heat | 14:30 | |
| *** jaime49 has joined #heat | 14:30 | |
| *** Raj1 has joined #heat | 14:31 | |
| *** eprasad has quit IRC | 14:34 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add floating_ip property to Nova::Server nics https://review.openstack.org/217240 | 14:37 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add subnet property to Nova::Server nic https://review.openstack.org/217241 | 14:37 |
| *** Raj1 has quit IRC | 14:39 | |
| *** Raj1 has joined #heat | 14:39 | |
| *** jaime49 has quit IRC | 14:40 | |
| *** jaime49 has joined #heat | 14:41 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Make encryption script more robust in presence of malformed data https://review.openstack.org/216536 | 14:46 |
| *** eprasad has joined #heat | 14:47 | |
| *** Qiming has quit IRC | 14:54 | |
| *** eprasad has quit IRC | 14:56 | |
| *** Drago has joined #heat | 14:58 | |
| *** sdake has quit IRC | 14:58 | |
| *** Drago has quit IRC | 14:58 | |
| *** links has quit IRC | 14:59 | |
| *** Drago has joined #heat | 14:59 | |
| *** daneyon has joined #heat | 15:01 | |
| *** Marga_ has quit IRC | 15:02 | |
| *** crose has joined #heat | 15:05 | |
| *** rakesh_hs has quit IRC | 15:06 | |
| *** blomquisg has quit IRC | 15:07 | |
| *** boris-42 has quit IRC | 15:10 | |
| *** jistr is now known as jistr|mtg | 15:12 | |
| openstackgerrit | Joe D'Andrea proposed openstack/heat: Adds resource uuid, volume support to lifecycle scheduler hints https://review.openstack.org/203823 | 15:15 |
| *** bnemec has joined #heat | 15:17 | |
| *** sabeen has joined #heat | 15:19 | |
| *** Marga_ has joined #heat | 15:19 | |
| *** blomquisg has joined #heat | 15:20 | |
| *** jaime49 has quit IRC | 15:22 | |
| *** jistr|mtg is now known as jistr | 15:22 | |
| *** Marga_ has quit IRC | 15:25 | |
| *** Marga_ has joined #heat | 15:25 | |
| *** jaime49 has joined #heat | 15:26 | |
| *** MVenesio has quit IRC | 15:31 | |
| *** Marga_ has quit IRC | 15:33 | |
| *** jaime49 has quit IRC | 15:33 | |
| *** Marga_ has joined #heat | 15:34 | |
| *** jaime49 has joined #heat | 15:34 | |
| *** pal has quit IRC | 15:36 | |
| *** Raj1 has quit IRC | 15:38 | |
| *** ochuprykov has joined #heat | 15:40 | |
| *** Marga_ has quit IRC | 15:42 | |
| SamYaple | therve: you still around | 15:42 |
| *** Raj1 has joined #heat | 15:42 | |
| therve | Yeah | 15:43 |
| -openstackstatus- NOTICE: restarting gerrit due to a slow memory leak | 15:43 | |
| SamYaple | therve: https://github.com/openstack/heat/blob/master/heat/db/api.py#L133-L135 | 15:44 |
| SamYaple | I have confirmed what is returned from that _is_ created_at = None | 15:44 |
| SamYaple | that goes straight to sqlalchemy | 15:44 |
| SamYaple | s/goes/comes | 15:44 |
| *** ochuprykov has quit IRC | 15:47 | |
| therve | Okay, That's super weird. | 15:48 |
| *** blues-man has joined #heat | 15:48 | |
| *** blues-man has joined #heat | 15:48 | |
| SamYaple | totally | 15:48 |
| SamYaple | | created_at | datetime(6) | YES | | NULL | | | 15:49 |
| SamYaple | | updated_at | datetime(6) | YES | | NULL | | | 15:49 |
| SamYaple | | deleted_at | datetime | YES | | NULL | | | 15:49 |
| SamYaple | the Type is the only difference here | 15:49 |
| SamYaple | could it be that is why its not returnign correctly... | 15:49 |
| SamYaple | but why | 15:49 |
| SamYaple | created and updated do not return values | 15:50 |
| SamYaple | deleted_at does | 15:50 |
| *** sorantis has quit IRC | 15:50 | |
| shardy | https://github.com/openstack/heat/blob/master/heat/db/sqlalchemy/migrate_repo/versions/065_update_at_fsp.py | 15:51 |
| shardy | SamYaple: I think that migration I mentioned earlier is the reason | 15:51 |
| shardy | it messes with the types of updated_at and created_at but not deleted_at | 15:52 |
| SamYaple | shardy: clearly this is the issue | 15:52 |
| SamYaple | im running mariadb10.2? | 15:52 |
| SamYaple | oops, 10.0.21 | 15:53 |
| SamYaple | so that would evalutate to newer that 5 6 4 | 15:53 |
| SamYaple | let me check that this is the issue | 15:53 |
| *** Marga_ has joined #heat | 15:54 | |
| SamYaple | grrrr | 15:54 |
| SamYaple | my eyes are bleeding | 15:54 |
| SamYaple | so much pdb stepping | 15:54 |
| *** sabeen2 has joined #heat | 15:55 | |
| *** sabeen has quit IRC | 15:58 | |
| *** dims_ has quit IRC | 16:00 | |
| SamYaple | ugh thank god | 16:00 |
| SamYaple | shardy: therve thats it | 16:00 |
| shardy | SamYaple: Ok, maybe we just revert that migration then? | 16:01 |
| SamYaple | mariadb 10.0.21 is newer than that server_version and gets it | 16:01 |
| SamYaple | shardy: yea | 16:01 |
| SamYaple | or we could filter > 10 | 16:01 |
| SamYaple | ? | 16:01 |
| *** dims has joined #heat | 16:01 | |
| *** Nepoc has joined #heat | 16:01 | |
| SamYaple | shardy: if we filter > 10 this would work and we could keep the migration intact | 16:01 |
| SamYaple | my guess is you didn't see this because you arent running mariadb10 :) | 16:02 |
| *** Marga_ has quit IRC | 16:02 | |
| SamYaple | shardy: i need to sleep, do you think a patch to revert the migrate entirely, or filter unsupported mariabd versions? | 16:04 |
| shardy | SamYaple: If we can filter in a way which works for both mysql and mariadb then +1 otherwise lets revert and work it out later | 16:05 |
| SamYaple | im almost positive i can filter it, testing now | 16:05 |
| openstackgerrit | Sergey Kraynev proposed openstack/heat: Refactor _get_network_matches method https://review.openstack.org/209437 | 16:06 |
| openstackgerrit | Sergey Kraynev proposed openstack/heat: Use _get_network_id in other methods of mixin class https://review.openstack.org/217284 | 16:06 |
| SamYaple | the migrate wil lneed an update once mariadb10 pulls in that feature | 16:09 |
| shardy | we can't detect the minor version? | 16:10 |
| SamYaple | it isn't in any minor version yet | 16:10 |
| SamYaple | i cant _predict_ the minor version | 16:10 |
| shardy | haha, yeah I see ;) | 16:10 |
| SamYaple | im just filtering all > 10 | 16:10 |
| *** jistr has quit IRC | 16:12 | |
| SamYaple | here goes nothing | 16:13 |
| *** erkules has quit IRC | 16:13 | |
| *** erkules has joined #heat | 16:13 | |
| SamYaple | a-ok | 16:15 |
| SamYaple | patch incoming | 16:15 |
| shardy | \o/ | 16:15 |
| shardy | good work tracking it down :) | 16:16 |
| SamYaple | my eyes really are bleeding | 16:16 |
| SamYaple | i need ot go to sleep | 16:16 |
| *** inc0_ has joined #heat | 16:17 | |
| *** inc0_ has quit IRC | 16:17 | |
| openstackgerrit | Sam Yaple proposed openstack/heat: MariaDB10 breaks with migrate version 065 https://review.openstack.org/217058 | 16:18 |
| SamYaple | therve: https://review.openstack.org/217058 if you could remove your -2 i would appreciate it | 16:19 |
| *** sdake has joined #heat | 16:20 | |
| *** Piet has quit IRC | 16:20 | |
| SamYaple | for the record, everyone should be able to reproduce this at this very moment by using mariadb10 | 16:21 |
| shardy | SamYaple: I'm running 10.0.20-1 on F21 and for some reason I don't see it | 16:23 |
| SamYaple | :( | 16:23 |
| SamYaple | does it apply dialects? | 16:23 |
| SamYaple | the migration i mean | 16:23 |
| openstackgerrit | Kairat Kushaev proposed openstack/heat: Resolve attributes for Software Deployment Group https://review.openstack.org/210306 | 16:25 |
| openstackgerrit | Kairat Kushaev proposed openstack/heat: Resolve dynamic attributes for RG, ASG, template resource https://review.openstack.org/209544 | 16:25 |
| openstackgerrit | Kairat Kushaev proposed openstack/heat: Replace FnGetAttr from Stack resource with _resolve_attribute https://review.openstack.org/206098 | 16:25 |
| SamYaple | jeez im so tired | 16:30 |
| SamYaple | not dialects | 16:30 |
| *** endorphin has joined #heat | 16:31 | |
| SamYaple | shardy: further looking shows that is clearly support his feature shardy | 16:31 |
| SamYaple | im seeting 6 places past the second | 16:31 |
| SamYaple | which is what that migration allows | 16:32 |
| SamYaple | i can set them as well | 16:32 |
| SamYaple | let me try version 10.0.20 | 16:32 |
| *** skraynev_ has quit IRC | 16:32 | |
| *** derekh has quit IRC | 16:46 | |
| *** kencjohnston has joined #heat | 16:50 | |
| *** lkarm has quit IRC | 16:52 | |
| *** achanda has joined #heat | 16:52 | |
| *** lkarm has joined #heat | 16:53 | |
| *** Raj1 has quit IRC | 16:54 | |
| *** lkarm has quit IRC | 16:55 | |
| *** lkarm has joined #heat | 16:55 | |
| *** thedodd has quit IRC | 16:56 | |
| SamYaple | shardy: well ive hit a wall | 17:07 |
| *** boris-42 has joined #heat | 17:07 | |
| *** pm90_ has joined #heat | 17:09 | |
| *** pm90_ has quit IRC | 17:10 | |
| *** pm90__ has quit IRC | 17:11 | |
| *** blues-man has quit IRC | 17:12 | |
| *** e0ne has quit IRC | 17:15 | |
| openstackgerrit | Merged openstack/heat: Add tests for show_resource function https://review.openstack.org/201094 | 17:15 |
| *** Piet has joined #heat | 17:26 | |
| *** daneyon has quit IRC | 17:27 | |
| *** kebray has joined #heat | 17:34 | |
| *** kebray has quit IRC | 17:35 | |
| *** kebray has joined #heat | 17:35 | |
| *** achanda_ has joined #heat | 17:39 | |
| *** achanda has quit IRC | 17:39 | |
| *** dims_ has joined #heat | 17:41 | |
| *** lkarm has quit IRC | 17:41 | |
| *** lkarm has joined #heat | 17:41 | |
| *** dims has quit IRC | 17:42 | |
| *** kebray has quit IRC | 17:42 | |
| *** cyli has joined #heat | 17:43 | |
| *** lkarm_ has joined #heat | 17:43 | |
| *** lkarm has quit IRC | 17:46 | |
| *** achanda_ has quit IRC | 17:48 | |
| *** achanda has joined #heat | 17:51 | |
| *** pm90_ has joined #heat | 17:55 | |
| *** Nepoc has quit IRC | 17:58 | |
| *** thedodd has joined #heat | 17:58 | |
| *** jasond has quit IRC | 18:01 | |
| *** Nepoc has joined #heat | 18:07 | |
| *** Raj1 has joined #heat | 18:09 | |
| SamYaple | shardy: well its not working for 10.0.20 or any other versions ive tested | 18:15 |
| SamYaple | leaving me with only dependancies to think about | 18:15 |
| *** Raj1 has quit IRC | 18:15 | |
| *** achanda has quit IRC | 18:16 | |
| *** openstackgerrit has quit IRC | 18:17 | |
| *** openstackgerrit has joined #heat | 18:17 | |
| *** neelashah has quit IRC | 18:18 | |
| *** ameybhide has joined #heat | 18:22 | |
| *** zhenguo has quit IRC | 18:23 | |
| *** neelashah has joined #heat | 18:24 | |
| *** e0ne has joined #heat | 18:25 | |
| *** lifeless1 is now known as lifeless | 18:39 | |
| *** vahidh has joined #heat | 18:47 | |
| *** jasond has joined #heat | 18:48 | |
| *** e0ne has quit IRC | 18:50 | |
| *** kencjohnston has quit IRC | 18:51 | |
| *** dims_ has quit IRC | 18:53 | |
| *** Raj1 has joined #heat | 18:54 | |
| *** dims has joined #heat | 18:54 | |
| *** jaime49 has quit IRC | 19:01 | |
| *** kencjohnston has joined #heat | 19:02 | |
| *** gberginc has joined #heat | 19:03 | |
| *** jaime49 has joined #heat | 19:06 | |
| openstackgerrit | Bryan Jones proposed openstack/heat: Consolidate keystone v3 hacks https://review.openstack.org/217354 | 19:06 |
| pm90_ | hello, is a template such as https://gist.github.com/pratikmallya/d1ce45257afe96b6ddb0 in valid HOT syntax? | 19:07 |
| openstackgerrit | Bryan Jones proposed openstack/heat: Consolidate keystone v3 hacks https://review.openstack.org/217354 | 19:07 |
| pm90_ | I'm wondering if the default value of the flavor parameter can be 4353 or has to be specified as "4353" | 19:08 |
| *** dsneddon is now known as dsneddon_bbiab | 19:12 | |
| *** tlashchova_ has joined #heat | 19:13 | |
| *** e0ne has joined #heat | 19:13 | |
| *** kebray has joined #heat | 19:13 | |
| *** rpothier has quit IRC | 19:16 | |
| *** Nepoc has quit IRC | 19:16 | |
| *** achanda has joined #heat | 19:16 | |
| *** kebray has quit IRC | 19:22 | |
| *** jprovazn has quit IRC | 19:23 | |
| *** achanda has quit IRC | 19:25 | |
| *** dsneddon_bbiab is now known as dsneddon | 19:29 | |
| *** daneyon has joined #heat | 19:29 | |
| *** tlashchova_ has quit IRC | 19:29 | |
| *** Raj1 has quit IRC | 19:33 | |
| *** daneyon has quit IRC | 19:33 | |
| *** Marga_ has joined #heat | 19:38 | |
| *** kraynevs has joined #heat | 19:42 | |
| *** Raj1 has joined #heat | 19:45 | |
| *** daneyon has joined #heat | 19:48 | |
| *** kraynevs is now known as skraynev_ | 19:53 | |
| *** Raj1 has quit IRC | 19:54 | |
| *** Raj1 has joined #heat | 19:54 | |
| *** daneyon has quit IRC | 19:54 | |
| *** thumpba has quit IRC | 19:55 | |
| *** daneyon has joined #heat | 19:55 | |
| Drago | pm90_: should be https://github.com/openstack/heat/blob/master/heat/engine/properties.py#L258-L259 | 19:57 |
| *** pas-ha has joined #heat | 19:59 | |
| stevebaker | meeting time | 20:00 |
| *** DandyPandy has joined #heat | 20:02 | |
| *** achanda has joined #heat | 20:02 | |
| *** jecarey has joined #heat | 20:05 | |
| *** kebray has joined #heat | 20:07 | |
| *** kencjohnston has quit IRC | 20:08 | |
| *** achanda has quit IRC | 20:08 | |
| *** daneyon has quit IRC | 20:09 | |
| *** dyasny has quit IRC | 20:16 | |
| *** Raj1 has quit IRC | 20:19 | |
| *** vahidh has quit IRC | 20:21 | |
| *** Marga_ has quit IRC | 20:21 | |
| *** daneyon has joined #heat | 20:24 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Don't encrypt default parameters https://review.openstack.org/217377 | 20:26 |
| *** achanda has joined #heat | 20:26 | |
| *** Raj1 has joined #heat | 20:28 | |
| *** e0ne has quit IRC | 20:28 | |
| *** thumpba has joined #heat | 20:29 | |
| openstackgerrit | Jason Dunsmore proposed openstack/heat: Add a preview endpoint for stack updates https://review.openstack.org/122473 | 20:31 |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Don't encrypt default parameters https://review.openstack.org/217377 | 20:33 |
| *** endorphin has quit IRC | 20:35 | |
| *** kebray has quit IRC | 20:36 | |
| openstackgerrit | Drago proposed openstack/heat: Add crypt algorithm to digest https://review.openstack.org/217384 | 20:38 |
| *** kebray has joined #heat | 20:42 | |
| *** jaime49 has quit IRC | 20:42 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Don't encrypt default parameters https://review.openstack.org/217377 | 20:47 |
| *** fesp has joined #heat | 20:52 | |
| *** vahidh has joined #heat | 20:53 | |
| *** vijendar has quit IRC | 20:53 | |
| *** vijendar has joined #heat | 20:53 | |
| *** ameybhide has quit IRC | 20:55 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Don't encrypt default parameters https://review.openstack.org/217377 | 20:58 |
| *** kebray has quit IRC | 20:59 | |
| *** pas-ha has quit IRC | 21:00 | |
| stevebaker | shardy, zaneb: could you review this? I believe it fixes the tripleo deployment race https://review.openstack.org/#/c/216920/ | 21:00 |
| shardy | stevebaker: sure, will do | 21:00 |
| zaneb | oh that's pure evil | 21:00 |
| stevebaker | zaneb: the fix or the bug? | 21:01 |
| zaneb | the bug | 21:01 |
| *** ochuprykov has joined #heat | 21:01 | |
| stevebaker | yeah, its been my nemesis | 21:01 |
| zaneb | maybe both. I'll let you know in a minute ;) | 21:01 |
| *** openstackgerrit has quit IRC | 21:01 | |
| shardy | stevebaker: hey, if you get a moment, can you check out https://bugs.launchpad.net/heat/+bug/1489028 | 21:01 |
| openstack | Launchpad bug 1489028 in heat "str_replace and list_join only accept strings" [Undecided,New] - Assigned to Steven Hardy (shardy) | 21:01 |
| shardy | stevebaker: I can probably take on the implementation but wanted to discuss the approach | 21:02 |
| *** hdd has quit IRC | 21:02 | |
| *** openstackgerrit has joined #heat | 21:02 | |
| shardy | particularly wrt SoftwareDeployment and input_values types | 21:02 |
| shardy | I've been having a hard time getting map/json parameters and attributes into nodes in a format that is easily consumable | 21:02 |
| *** skraynev_ has quit IRC | 21:03 | |
| shardy | all our current interfaces either refuse those formats or mangle them into __repr__ of dict | 21:03 |
| shardy | which is nearly useless if you want to consume the data via jq etc | 21:03 |
| stevebaker | shardy: I suppose an implicit conversion to json would be preferable to the current behaviour | 21:04 |
| shardy | tl;dr we need a way to pass stuff as json (e.g actually json vs parameters called json which really arent) | 21:04 |
| shardy | stevebaker: I was thinking with str_replace, it currently won't accept map/list params at all, so that would be an easy backwards compatible way to enable conversion | 21:05 |
| shardy | e.g just document that list/map types are json encoded into the template | 21:06 |
| *** achanda has quit IRC | 21:07 | |
| *** jecarey has quit IRC | 21:08 | |
| *** jcoufal has quit IRC | 21:10 | |
| *** ochuprykov has quit IRC | 21:11 | |
| *** crose has quit IRC | 21:11 | |
| *** MasterPiece has joined #heat | 21:13 | |
| *** daneyon_ has joined #heat | 21:16 | |
| *** jcoufal has joined #heat | 21:18 | |
| *** daneyon has quit IRC | 21:19 | |
| openstackgerrit | Jay_Clark proposed openstack/heat: enabling the stevedore.sphinxext extension. https://review.openstack.org/217401 | 21:19 |
| *** Piet has quit IRC | 21:19 | |
| *** jasondotstar has joined #heat | 21:20 | |
| *** lkarm_ has quit IRC | 21:22 | |
| *** thedodd has quit IRC | 21:23 | |
| stevebaker | shardy: that sounds like a good start anyway | 21:27 |
| *** MasterPiece has quit IRC | 21:30 | |
| openstackgerrit | Merged openstack/heat-specs: Add nested-validation spec https://review.openstack.org/197199 | 21:31 |
| *** alex_xu has quit IRC | 21:33 | |
| *** Raj1 has quit IRC | 21:34 | |
| *** alex_xu has joined #heat | 21:35 | |
| *** jamielennox has quit IRC | 21:36 | |
| *** Raj1 has joined #heat | 21:37 | |
| *** jonesbr has left #heat | 21:37 | |
| *** jamielennox has joined #heat | 21:38 | |
| *** thumpba has quit IRC | 21:40 | |
| *** thumpba has joined #heat | 21:40 | |
| *** Raj1 has quit IRC | 21:43 | |
| *** fesp has quit IRC | 21:45 | |
| *** neelashah has quit IRC | 21:46 | |
| *** shardy has quit IRC | 21:46 | |
| *** Raj1 has joined #heat | 21:47 | |
| *** thumpba has quit IRC | 21:47 | |
| *** hdd has joined #heat | 21:47 | |
| *** thumpba has joined #heat | 21:49 | |
| *** thumpba has quit IRC | 21:51 | |
| *** pm90__ has joined #heat | 21:51 | |
| *** pm90__ has quit IRC | 21:51 | |
| *** pm90__ has joined #heat | 21:52 | |
| *** pm90__ has quit IRC | 21:52 | |
| *** jasond has quit IRC | 21:53 | |
| openstackgerrit | Drago proposed openstack/heat: Add crypt algorithm to digest https://review.openstack.org/217384 | 21:55 |
| *** pm90_ has quit IRC | 21:55 | |
| *** dims_ has joined #heat | 21:56 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Use the official mission statement in index.rst https://review.openstack.org/216486 | 21:59 |
| openstackgerrit | Steve Baker proposed openstack/heat: Use code-block:: yaml for all template snippets https://review.openstack.org/216455 | 21:59 |
| openstackgerrit | Steve Baker proposed openstack/heat: Use latest stable template version in snippets https://review.openstack.org/198169 | 21:59 |
| openstackgerrit | Steve Baker proposed openstack/heat: Use sections for individual properties and attributes https://review.openstack.org/216456 | 21:59 |
| openstackgerrit | Steve Baker proposed openstack/heat: flake8 check doc/source https://review.openstack.org/216458 | 21:59 |
| *** dims has quit IRC | 21:59 | |
| *** blomquisg has quit IRC | 22:00 | |
| *** achanda has joined #heat | 22:06 | |
| openstackgerrit | Drago proposed openstack/heat: Add crypt algorithm to digest https://review.openstack.org/217384 | 22:11 |
| openstackgerrit | Merged openstack/heat: Fix some files permissions https://review.openstack.org/215125 | 22:12 |
| *** sdake_ has joined #heat | 22:15 | |
| *** sdake has quit IRC | 22:18 | |
| *** vijendar has quit IRC | 22:22 | |
| *** dims_ has quit IRC | 22:24 | |
| *** dims has joined #heat | 22:24 | |
| *** Raj1 has quit IRC | 22:26 | |
| *** hdd has quit IRC | 22:27 | |
| *** achanda_ has joined #heat | 22:28 | |
| *** achanda has quit IRC | 22:29 | |
| *** hdd has joined #heat | 22:30 | |
| *** jcoufal has quit IRC | 22:31 | |
| *** Piet has joined #heat | 22:32 | |
| *** dims has quit IRC | 22:34 | |
| *** pm90_ has joined #heat | 22:39 | |
| *** pm90__ has joined #heat | 22:40 | |
| *** pm90_ has quit IRC | 22:41 | |
| *** sdake_ is now known as sdake | 22:41 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Don't encrypt default parameters https://review.openstack.org/217377 | 22:43 |
| *** hdd has quit IRC | 22:44 | |
| *** pm90_ has joined #heat | 22:44 | |
| asalkeld | morning | 22:46 |
| *** pm90__ has quit IRC | 22:47 | |
| *** sabeen2 has quit IRC | 22:50 | |
| *** Raj1 has joined #heat | 23:02 | |
| *** tyson has joined #heat | 23:03 | |
| *** tyson has quit IRC | 23:04 | |
| *** sergmelikyan has quit IRC | 23:09 | |
| pm90_ | hey stevebaker, if the default value has been specified in the template, it will be available in the raw_template form anyways | 23:11 |
| pm90_ | which kinda defeats the purpose of encrypting | 23:11 |
| pm90_ | ref: https://review.openstack.org/#/c/217377/4 | 23:12 |
| stevebaker | pm90_: I realise that. I just worry about conditional paths in what to encrypt will lead to incorrectly decrypting in some cases. | 23:12 |
| stevebaker | pm90_: what is the harm of encrypting a default value? | 23:13 |
| pm90_ | stevebaker: there's no harm, except that it gives a sense of security when there is none | 23:13 |
| stevebaker | I'm fine with that | 23:14 |
| jamespd | Are there weird dependency issues in Juno Heat? I can get my stack to not hang in CREATE_IN_PROGRESS state if I change "get_attr: [ HOST_port, fixed_ips ]" to "get_attr: [HOST, networks, NETWORK-NAME, 0]" | 23:16 |
| stevebaker | jamespd: either should work, but the latter is valid anyway. | 23:18 |
| jamespd | cool, ta. | 23:18 |
| pm90_ | stevebaker: in fact, the patch actually reduces the code paths to encryption/decryption: we are only encrypting the user specified parameters and nothing else | 23:18 |
| *** achanda has joined #heat | 23:18 | |
| stevebaker | jamespd: so if you want to dig into the port case could you raise a bug? | 23:18 |
| *** achanda_ has quit IRC | 23:18 | |
| *** arif-ali has quit IRC | 23:19 | |
| stevebaker | pm90_: how about the case where the value goes to/from the default value on a stack-update? | 23:19 |
| *** arif-ali has joined #heat | 23:19 | |
| stevebaker | pm90_: actually, looking at the decrypt code that wouldn't be an issue | 23:21 |
| pm90_ | stevebaker: won't it be more straightforward in that case? In fact, the way it stands now, we are storing default parameters as environment parameters, which seems illogical | 23:22 |
| stevebaker | pm90_: ok, I've withdrawn my -1 | 23:25 |
| pm90_ | stevebaker: thanks! I've been facing an issue where the encryption script fails because the default value is an integer, even when the parameter type is string. Still investigating why it behaves that way | 23:26 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Calculate ResourceGroup batch sizes using scaling library https://review.openstack.org/213552 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Split out calculation of the next batch into a separate function https://review.openstack.org/217133 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Respect rolling update params on the initial batch https://review.openstack.org/217134 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Refactor InstanceGroup batch size check https://review.openstack.org/213549 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Refactor InstanceGroup batch sizing logic https://review.openstack.org/213548 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Invert logic of counting in InstanceGroup _get_batches() https://review.openstack.org/213551 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Refactor ResourceGroup batched updates and add tests https://review.openstack.org/217430 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Return the correct size for the ResourceGroup from _get_batches() https://review.openstack.org/217431 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Don't do unnecessary updates in ResourceGroup rolling updates https://review.openstack.org/217432 | 23:31 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Don't repeatedly overwrite definitions in RG rolling update https://review.openstack.org/217433 | 23:31 |
| asalkeld | stevebaker: if you had to give an indicator of server create/update/delete times what would be a reasonable guess? | 23:35 |
| asalkeld | (i am faking it in TestResource) | 23:36 |
| asalkeld | and running rally | 23:36 |
| stevebaker | asalkeld: between 1ms and 1year | 23:36 |
| asalkeld | I currently have create: 15, update: 5, delete: 8 | 23:36 |
| asalkeld | (seconds) | 23:36 |
| asalkeld | hah stevebaker , be useful ... | 23:36 |
| stevebaker | that sounds reasonable | 23:36 |
| asalkeld | ok | 23:36 |
| stevebaker | create could sometimes be longer if the glance image has to be transferred to the compute, then shorter | 23:37 |
| asalkeld | and volume/attachement | 23:38 |
| asalkeld | 1-3 secs | 23:38 |
| *** Raj1 has quit IRC | 23:38 | |
| *** achanda has quit IRC | 23:38 | |
| *** tochi has joined #heat | 23:39 | |
| stevebaker | asalkeld: does TestResource have a random variance option? | 23:39 |
| *** Qiming has joined #heat | 23:39 | |
| stevebaker | that would be good for triggering races | 23:39 |
| asalkeld | https://review.openstack.org/#/c/215489/ | 23:39 |
| asalkeld | smarter ^ | 23:39 |
| asalkeld | also now actually can ping the server | 23:40 |
| asalkeld | services | 23:40 |
| stevebaker | cool | 23:40 |
| asalkeld | i am using a resource group with this in it: http://paste.openstack.org/show/428903/ | 23:42 |
| * asalkeld just happy to not have to worry about no-such-host | 23:42 | |
| jamespd | stevebaker: I am happy to use the latter method to get the IP. Would you rather have the bug report against juno, or I can wait and re-test it against Kilo in six-ish weeks after we upgrade. | 23:43 |
| stevebaker | jamespd: yes, wait for kilo I think | 23:43 |
| jamespd | rad; will do. | 23:43 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Clarify name of scaling resource_templates() function https://review.openstack.org/213553 | 23:58 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Allow a custom new ID function for member_definitions() https://review.openstack.org/213554 | 23:58 |
| openstackgerrit | Zane Bitter proposed openstack/heat: Allow member_definitions() to customise each member that it defines https://review.openstack.org/213555 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!