| *** Drago has quit IRC | 00:02 | |
| stevebaker | dangerous | 00:15 |
|---|---|---|
| *** EricGonc_ has joined #heat | 00:17 | |
| *** devlaps has quit IRC | 00:18 | |
| *** pratikmallya has joined #heat | 00:19 | |
| *** dims_ has quit IRC | 00:20 | |
| *** bapalm_ has joined #heat | 00:20 | |
| *** EricGonczer_ has quit IRC | 00:21 | |
| *** bapalm has quit IRC | 00:21 | |
| *** jdob has quit IRC | 00:21 | |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #heat | 00:23 | |
| *** pratikmallya has quit IRC | 00:23 | |
| *** manishg_ has quit IRC | 00:29 | |
| *** slagle_ has joined #heat | 00:31 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Save previous template on non convergence updates https://review.openstack.org/222675 | 00:32 |
| *** ricolin has joined #heat | 00:33 | |
| *** htruta` has joined #heat | 00:33 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Allow map/list items for list_join https://review.openstack.org/221765 | 00:35 |
| *** slagle has quit IRC | 00:35 | |
| *** dyasny has quit IRC | 00:35 | |
| *** ahonda has quit IRC | 00:35 | |
| *** tochi has quit IRC | 00:35 | |
| *** blomquisg has quit IRC | 00:35 | |
| *** htruta has quit IRC | 00:36 | |
| *** mikal has quit IRC | 00:36 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Add test for _show_resource https://review.openstack.org/223026 | 00:36 |
| *** mikal has joined #heat | 00:36 | |
| *** hdd has quit IRC | 00:38 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Remove policies from mistral types https://review.openstack.org/218633 | 00:38 |
| *** blomquisg has joined #heat | 00:38 | |
| *** vahidh has quit IRC | 00:40 | |
| openstackgerrit | Steve Baker proposed openstack/heat: Allow map/list items for str_replace https://review.openstack.org/221766 | 00:40 |
| *** dyasny has joined #heat | 00:42 | |
| *** ahonda has joined #heat | 00:42 | |
| *** tochi has joined #heat | 00:42 | |
| *** vahidh has joined #heat | 00:56 | |
| *** rwsu has quit IRC | 00:57 | |
| *** EricGonc_ has quit IRC | 00:59 | |
| *** dims_ has joined #heat | 01:05 | |
| *** jasond has quit IRC | 01:05 | |
| *** jdandrea has quit IRC | 01:05 | |
| *** liusheng has joined #heat | 01:06 | |
| *** jasond has joined #heat | 01:09 | |
| stevebaker | asalkeld, huangtianhua: potentially awake cores, we really need a heatclient release soon, ideally with these changes. Could you review? https://review.openstack.org/#/c/222325/ https://review.openstack.org/#/c/205755/ | 01:10 |
| *** jasond has quit IRC | 01:13 | |
| asalkeld | sure | 01:15 |
| *** crose has joined #heat | 01:16 | |
| asalkeld | stevebaker: the second one needs this heat patch https://review.openstack.org/#/c/205754/ | 01:21 |
| asalkeld | and it didn't merge | 01:21 |
| asalkeld | needs rebase/merge ... | 01:22 |
| asalkeld | (gave both +2) | 01:23 |
| *** Yanyanhu has joined #heat | 01:28 | |
| *** jasond has joined #heat | 01:30 | |
| *** dims_ has quit IRC | 01:34 | |
| *** manishg has joined #heat | 01:36 | |
| *** eliqiao has joined #heat | 01:39 | |
| *** eliqiao has quit IRC | 01:42 | |
| *** eliqiao has joined #heat | 01:43 | |
| *** jasond has quit IRC | 01:46 | |
| *** manishg has quit IRC | 01:46 | |
| *** eliqiao1 has joined #heat | 01:50 | |
| *** zhenguo has joined #heat | 01:50 | |
| stevebaker | asalkeld: it doesn't *need* the patch, it just won't do anything on a heat without it. | 01:51 |
| *** eliqiao has quit IRC | 01:51 | |
| *** htruta` is now known as htruta | 01:53 | |
| *** jasond has joined #heat | 01:54 | |
| *** EricGonczer_ has joined #heat | 02:03 | |
| asalkeld | ok | 02:04 |
| *** sasukeh has joined #heat | 02:07 | |
| *** manishg has joined #heat | 02:08 | |
| *** tiantian has joined #heat | 02:12 | |
| *** jasond has joined #heat | 02:13 | |
| *** vijayagurug has joined #heat | 02:14 | |
| *** sasukeh has quit IRC | 02:14 | |
| *** jasond has quit IRC | 02:15 | |
| *** jasond has joined #heat | 02:17 | |
| *** pratikmallya has joined #heat | 02:23 | |
| *** eliqiao1 has quit IRC | 02:25 | |
| *** eliqiao has joined #heat | 02:25 | |
| *** pratikmallya has quit IRC | 02:27 | |
| *** zhenguo has quit IRC | 02:28 | |
| *** tochi has quit IRC | 02:28 | |
| *** dyasny has quit IRC | 02:28 | |
| *** ahonda has quit IRC | 02:28 | |
| *** dyasny has joined #heat | 02:28 | |
| *** EricGonczer_ has quit IRC | 02:29 | |
| *** ahonda has joined #heat | 02:29 | |
| *** crose has quit IRC | 02:31 | |
| *** zhenguo has joined #heat | 02:33 | |
| *** liusheng is now known as 64MADUTTG | 02:34 | |
| *** liusheng has joined #heat | 02:34 | |
| *** tochi has joined #heat | 02:34 | |
| *** achanda has joined #heat | 02:35 | |
| *** 64MADUTTG has quit IRC | 02:36 | |
| *** sergmelikyan has joined #heat | 02:38 | |
| *** junhongl has joined #heat | 02:38 | |
| *** eliqiao1 has joined #heat | 02:47 | |
| *** eliqiao has quit IRC | 02:48 | |
| *** eliqiao has joined #heat | 02:54 | |
| *** eliqiao1 has quit IRC | 02:57 | |
| *** hdd has joined #heat | 02:57 | |
| *** vijayagurug has quit IRC | 03:09 | |
| *** jasond has quit IRC | 03:14 | |
| openstackgerrit | Qiming Teng proposed openstack/heat: Fix filter name inconsistency in stack_list https://review.openstack.org/188411 | 03:15 |
| *** Qiming has joined #heat | 03:17 | |
| *** vijayagurug has joined #heat | 03:23 | |
| *** mragupat has joined #heat | 03:24 | |
| *** mragupat_ has joined #heat | 03:25 | |
| *** tochi has quit IRC | 03:27 | |
| asalkeld | heat.tests.clients.test_manila_client.ManilaClientPluginTests.test_create | 03:28 |
| *** manishg has quit IRC | 03:28 | |
| asalkeld | that seems to be failing lots ^ | 03:28 |
| *** mragupat has quit IRC | 03:29 | |
| *** manishg has joined #heat | 03:29 | |
| stevebaker | asalkeld: this is the fix, its next in line but a long way down the queue https://review.openstack.org/#/c/223747/ | 03:29 |
| *** boris-42 has quit IRC | 03:30 | |
| asalkeld | ok, well at least it's on the way | 03:30 |
| asalkeld | stevebaker: https://goo.gl/igwqqB | 03:34 |
| *** david-lyle has joined #heat | 03:41 | |
| *** sergmelikyan has quit IRC | 03:45 | |
| *** boris-42 has joined #heat | 03:49 | |
| *** boris-42 has quit IRC | 03:49 | |
| *** boris-42 has joined #heat | 03:49 | |
| *** tochi has joined #heat | 03:50 | |
| *** vijayagurug has quit IRC | 03:51 | |
| *** manishg has quit IRC | 03:52 | |
| *** elynn has joined #heat | 04:03 | |
| *** akuznetsova has quit IRC | 04:07 | |
| *** akuznetsova has joined #heat | 04:07 | |
| *** sasukeh has joined #heat | 04:16 | |
| *** Marga_ has quit IRC | 04:16 | |
| *** rwsu has joined #heat | 04:21 | |
| *** sasukeh has quit IRC | 04:21 | |
| *** pratikmallya has joined #heat | 04:24 | |
| *** sasukeh_ has joined #heat | 04:26 | |
| *** pratikmallya has quit IRC | 04:26 | |
| *** pratikmallya has joined #heat | 04:26 | |
| *** sasukeh_ has quit IRC | 04:27 | |
| *** coolsvap has joined #heat | 04:28 | |
| *** blomquisg has quit IRC | 04:28 | |
| *** gberginc has joined #heat | 04:30 | |
| *** sdake has joined #heat | 04:33 | |
| *** sdake_ has joined #heat | 04:34 | |
| *** KanagarajM has joined #heat | 04:35 | |
| KanagarajM | good morning all ! | 04:35 |
| *** kebray has joined #heat | 04:37 | |
| *** sdake has quit IRC | 04:38 | |
| *** kebray has quit IRC | 04:39 | |
| *** kebray has joined #heat | 04:41 | |
| *** david-lyle has quit IRC | 04:42 | |
| *** david-lyle has joined #heat | 04:45 | |
| *** achanda has quit IRC | 04:46 | |
| *** achanda has joined #heat | 04:51 | |
| *** sasukeh has joined #heat | 04:52 | |
| *** sdake_ has quit IRC | 04:56 | |
| *** sasukeh has quit IRC | 04:57 | |
| *** sdake has joined #heat | 04:59 | |
| *** LimorStotland has joined #heat | 05:07 | |
| *** sdake_ has joined #heat | 05:10 | |
| LimorStotland | Hi all,I see test of test_create(share_type) failing on MismatchError: 'http://server.test:5000/v3' != 'http://server.test:5000/' does anyone know this issue? | 05:12 |
| *** sdake has quit IRC | 05:14 | |
| *** sergmelikyan has joined #heat | 05:15 | |
| *** sergmelikyan has quit IRC | 05:19 | |
| stevebaker | LimorStotland: the fix is in the queue, it will take a while to get to the front | 05:19 |
| LimorStotland | Ok Thanks | 05:21 |
| *** mragupat_ has quit IRC | 05:24 | |
| *** gberginc has quit IRC | 05:30 | |
| *** vijayagurug has joined #heat | 05:32 | |
| cmyster | morning | 05:38 |
| asalkeld | o/ | 05:40 |
| *** lsmola has joined #heat | 05:42 | |
| *** hdd has quit IRC | 05:43 | |
| stevebaker | huangtianhua: ping | 05:44 |
| *** KanagarajM has quit IRC | 05:44 | |
| *** KanagarajM_ has joined #heat | 05:44 | |
| *** vgridnev has joined #heat | 05:45 | |
| *** LimorStotland has quit IRC | 05:46 | |
| *** tshtilma has joined #heat | 05:59 | |
| *** zeih has joined #heat | 06:07 | |
| tiantian | stevebaker, hi | 06:13 |
| stevebaker | tiantian: hey | 06:13 |
| stevebaker | tiantian: we really need a heatclient release soon, ideally with these changes. Could you review and approve if you think they are ready? https://review.openstack.org/#/c/222325/ https://review.openstack.org/#/c/205755/ | 06:15 |
| tiantian | stevebaker, looking | 06:15 |
| *** pas-ha has joined #heat | 06:23 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat-specs: Fixes bug in the designate spec https://review.openstack.org/223934 | 06:23 |
| tiantian | stevebaker, approve:) | 06:24 |
| *** jprovazn has joined #heat | 06:28 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Template-validate fails when resources are swaped with None resource https://review.openstack.org/223528 | 06:31 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat-specs: Fixes bug in the designate spec https://review.openstack.org/223934 | 06:35 |
| *** e0ne has joined #heat | 06:37 | |
| *** e0ne has quit IRC | 06:39 | |
| stevebaker | tiantian: \o/ | 06:39 |
| *** sasukeh has joined #heat | 06:44 | |
| *** LimorStotland has joined #heat | 06:47 | |
| *** KanagarajM__ has joined #heat | 06:47 | |
| *** KanagarajM_ has quit IRC | 06:51 | |
| *** tspatzier has joined #heat | 06:52 | |
| *** ricolin has quit IRC | 06:53 | |
| *** sasukeh has quit IRC | 06:55 | |
| *** vgridnev has quit IRC | 06:57 | |
| *** sasukeh has joined #heat | 06:58 | |
| *** shardy has joined #heat | 06:59 | |
| skraynev | meeting time ? :) | 06:59 |
| skraynev | stevebaker: ^ ? | 06:59 |
| shardy | morning all | 06:59 |
| skraynev | shardy: morning ;) | 07:00 |
| stevebaker | righto | 07:00 |
| *** zeih has quit IRC | 07:00 | |
| *** sasukeh has quit IRC | 07:03 | |
| *** achanda has quit IRC | 07:06 | |
| asalkeld | something to give everyone nightmares: http://www.abc.net.au/news/2015-09-15/snakes-in-loos-search-for-water-in-townsville-houses/6778608 | 07:08 |
| *** vgridnev has joined #heat | 07:11 | |
| *** dkusidlo has joined #heat | 07:11 | |
| *** shardy_ has joined #heat | 07:16 | |
| *** shardy has quit IRC | 07:18 | |
| *** boris-42 has quit IRC | 07:19 | |
| *** liusheng has quit IRC | 07:19 | |
| *** Qiming has quit IRC | 07:19 | |
| *** zhenguo has quit IRC | 07:19 | |
| openstackgerrit | Merged openstack/python-heatclient: stack-update -x tolerate no template https://review.openstack.org/205755 | 07:20 |
| *** tspatzier_ has joined #heat | 07:21 | |
| *** dkusidlo has quit IRC | 07:21 | |
| *** shardy_ has quit IRC | 07:21 | |
| *** liusheng has joined #heat | 07:22 | |
| *** shardy has joined #heat | 07:22 | |
| *** kebray has quit IRC | 07:23 | |
| *** Qiming has joined #heat | 07:24 | |
| *** zhenguo has joined #heat | 07:24 | |
| *** boris-42 has joined #heat | 07:25 | |
| Qiming | what happened? suddenly everyone disappeared, then in a few minutes, everyone is back ... | 07:29 |
| *** Yanyanhu has quit IRC | 07:33 | |
| *** dkusidlo has joined #heat | 07:33 | |
| *** Yanyanhu has joined #heat | 07:33 | |
| asalkeld | Qiming: netsplit | 07:35 |
| asalkeld | irc network issue | 07:36 |
| Qiming | asalkeld, I was shocked, I was feeling that I became the only man in the world | 07:36 |
| asalkeld | ;) | 07:37 |
| tspatzier_ | is there still some irc network issue? I am having problems connecting to freenode from xchat on the Mac ... but only when at the office. seemed to work from home yesterday | 07:41 |
| *** liusheng has quit IRC | 07:41 | |
| *** liusheng has joined #heat | 07:42 | |
| openstackgerrit | Merged openstack/python-heatclient: Ability to specify show_nested for template validation https://review.openstack.org/222325 | 07:43 |
| *** dkusidlo has quit IRC | 07:44 | |
| *** zeih has joined #heat | 07:48 | |
| *** tochi has quit IRC | 07:50 | |
| *** LimorStotland has quit IRC | 07:50 | |
| *** LimorStotland has joined #heat | 07:50 | |
| *** Shangxdy has joined #heat | 07:52 | |
| shardy | Qiming: lol! If it helps I also got dropped off IRC halfway through the meeting so I know how you feel ;) | 07:52 |
| *** tspatzier has left #heat | 07:52 | |
| stevebaker | meeting partition | 07:57 |
| *** vgridnev has quit IRC | 07:58 | |
| *** tochi has joined #heat | 07:59 | |
| Qiming | shardy, it helps, haha | 07:59 |
| *** e0ne has joined #heat | 08:03 | |
| stevebaker | shardy: https://review.openstack.org/#/c/223747/ will need a recheck, py27 unit tests failed https://jenkins04.openstack.org/job/gate-heat-python27/1168/console | 08:04 |
| *** KanagarajM__ has quit IRC | 08:05 | |
| shardy | stevebaker: Ok cheers will check it out | 08:06 |
| stevebaker | alright, night all | 08:07 |
| shardy | o/ | 08:07 |
| asalkeld | didn't we have a bp at some point that described a desire to stop processing on a node before it gets shutdown? | 08:09 |
| shardy | asalkeld: that is already possibly via a DELETE only SoftwareDeployment? | 08:10 |
| shardy | asalkeld: or do you mean something different? | 08:10 |
| asalkeld | ok, i'll check that out | 08:10 |
| asalkeld | thanks | 08:10 |
| shardy | asalkeld: sec, here's an example from TripleO: | 08:11 |
| *** sdake has joined #heat | 08:11 | |
| *** sorantis has joined #heat | 08:11 | |
| *** gberginc has joined #heat | 08:11 | |
| shardy | https://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/post_deploy/rhel-registration/rhel-registration.yaml#L111 | 08:11 |
| shardy | asalkeld: we use that approach to unregister nodes from a satellite just before they're deleted | 08:12 |
| shardy | but you could do anything there really | 08:12 |
| asalkeld | cool, thanks | 08:12 |
| *** sdake_ has quit IRC | 08:12 | |
| *** yassine_ has joined #heat | 08:13 | |
| *** jistr has joined #heat | 08:19 | |
| *** dkusidlo has joined #heat | 08:22 | |
| *** e0ne has quit IRC | 08:24 | |
| *** alexpilotti has joined #heat | 08:35 | |
| *** derekh has joined #heat | 08:42 | |
| *** e0ne has joined #heat | 08:46 | |
| *** dkusidlo has quit IRC | 08:47 | |
| *** kbyrne has joined #heat | 08:53 | |
| *** zeih has quit IRC | 08:59 | |
| *** tochi has quit IRC | 09:03 | |
| *** dkusidlo has joined #heat | 09:04 | |
| *** tspatzier_ is now known as tspatzier | 09:08 | |
| *** dkusidlo has quit IRC | 09:21 | |
| *** dkusidlo has joined #heat | 09:27 | |
| *** sdake has quit IRC | 09:30 | |
| *** dkusidlo has quit IRC | 09:32 | |
| *** dims_ has joined #heat | 09:35 | |
| *** vgridnev has joined #heat | 09:39 | |
| *** liusheng has quit IRC | 09:40 | |
| *** dkusidlo has joined #heat | 09:41 | |
| *** shardy_ has joined #heat | 09:47 | |
| *** shardy has quit IRC | 09:49 | |
| openstackgerrit | Tetiana Lashchova proposed openstack/heat: Add node_processes validation to sahara node group https://review.openstack.org/221673 | 09:51 |
| *** shardy_ has quit IRC | 09:52 | |
| *** shardy has joined #heat | 09:53 | |
| *** zengyingzhe has quit IRC | 10:04 | |
| *** dkusidlo has quit IRC | 10:05 | |
| *** LimorStotland has quit IRC | 10:19 | |
| *** heatuser has joined #heat | 10:20 | |
| heatuser | Hi everybody...can somebody help me with generating simple Heat with windows instance ? | 10:21 |
| *** gberginc has quit IRC | 10:22 | |
| *** sorantis has quit IRC | 10:22 | |
| heatuser | I did preparation of heat with fresh windows install and works fine...now i want to get a file from http repository and run it on windows instance after creation...can somebody help me with it ? because get_file doesn't work | 10:23 |
| *** huangtianhua_ has joined #heat | 10:25 | |
| openstackgerrit | Merged openstack/heat: Use endpoint_url instead of base_url https://review.openstack.org/223747 | 10:28 |
| *** tiantian has quit IRC | 10:29 | |
| openstackgerrit | huangtianhua proposed openstack/heat: No need to detach if there is no attachment https://review.openstack.org/224015 | 10:29 |
| *** KanagarajM has joined #heat | 10:31 | |
| *** Yanyanhu has quit IRC | 10:33 | |
| *** Qiming has quit IRC | 10:33 | |
| pas-ha | heat user - you run Heat standalone on windows? or you deploy windows with heat stack? | 10:34 |
| pas-ha | what do you use to launch stacks - CLI or Horizon? (the latter indeed does not support get_file) | 10:34 |
| pas-ha | heatuser, ^ | 10:35 |
| heatuser | pas-ha CLI | 10:37 |
| *** gberginc has joined #heat | 10:38 | |
| pas-ha | CLI on which host system? windows might require path to be provided in some specially formatted way (AFAIR) | 10:38 |
| pas-ha | also, you use the get_file to attach a script to be executed on a Windows VM? does your windows VM has a cloud-init compatible thing installed? | 10:39 |
| heatuser | pas_ha : I have installed cloudbase on it, so I run heat template both with CLI and also as text input in Horizon with no luck...I think i have wrong code because on cloudbase logs i have outputs like Cundefinedscript.bat where should be C:\script.bat, i've tried with "" also | 10:42 |
| *** tochi has joined #heat | 10:43 | |
| *** elynn has quit IRC | 10:43 | |
| *** dkusidlo has joined #heat | 10:44 | |
| *** sorantis has joined #heat | 10:46 | |
| *** Shangxdy has quit IRC | 10:52 | |
| *** dkusidlo has quit IRC | 10:54 | |
| *** e0ne has quit IRC | 10:56 | |
| *** sorantis_ has joined #heat | 11:02 | |
| openstackgerrit | Merged openstack/heat: Validate TemplateResource schema when based on other templates https://review.openstack.org/197402 | 11:03 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Convergence: small fixes in tests https://review.openstack.org/224034 | 11:03 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Remove tests for _show_resource in manila https://review.openstack.org/223032 | 11:04 |
| *** sorantis has quit IRC | 11:04 | |
| *** sorantis_ is now known as sorantis | 11:04 | |
| *** e0ne has joined #heat | 11:13 | |
| *** vijayagurug has quit IRC | 11:19 | |
| *** vgridnev has quit IRC | 11:30 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Ceilometer) https://review.openstack.org/221024 | 11:35 |
| *** vgridnev has joined #heat | 11:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/223620 | 11:42 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Do not lose traceback while raising exc https://review.openstack.org/223662 | 11:48 |
| *** boris-42 has quit IRC | 11:50 | |
| *** Qiming has joined #heat | 11:51 | |
| *** Qiming has quit IRC | 11:52 | |
| *** Qiming has joined #heat | 11:52 | |
| *** EricGonczer_ has joined #heat | 11:53 | |
| *** Qiming has quit IRC | 11:54 | |
| *** Qiming has joined #heat | 11:55 | |
| *** blomquisg has joined #heat | 12:02 | |
| *** sergmelikyan has joined #heat | 12:08 | |
| *** rakesh_hs has joined #heat | 12:10 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat-templates: Fixes build failure due to conditional validation of service availability https://review.openstack.org/224051 | 12:14 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat-templates: Sample auto-scaling template using monasca https://review.openstack.org/217654 | 12:15 |
| *** tshtilma has quit IRC | 12:16 | |
| *** aweiteka has quit IRC | 12:17 | |
| *** KanagarajM has quit IRC | 12:21 | |
| *** jasond has joined #heat | 12:22 | |
| *** aweiteka has joined #heat | 12:27 | |
| *** MVenesio has joined #heat | 12:28 | |
| *** dkusidlo has joined #heat | 12:28 | |
| *** manas has joined #heat | 12:28 | |
| *** dkusidlo_ has joined #heat | 12:31 | |
| *** dkusidlo has quit IRC | 12:32 | |
| *** dkusidlo_ is now known as dkusidlo | 12:32 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Cinder) https://review.openstack.org/221029 | 12:33 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Glance) https://review.openstack.org/221030 | 12:33 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Magnum) https://review.openstack.org/221223 | 12:33 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Manila) https://review.openstack.org/221224 | 12:34 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Mistral) https://review.openstack.org/221225 | 12:34 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Nova) https://review.openstack.org/221226 | 12:35 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Delete children handle_delete testing (Sahara) https://review.openstack.org/221227 | 12:35 |
| *** jdob has joined #heat | 12:36 | |
| *** jprovazn has quit IRC | 12:36 | |
| *** dkusidlo has quit IRC | 12:38 | |
| *** jasond has quit IRC | 12:38 | |
| *** e0ne has quit IRC | 12:38 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Fix translating for props with get_param value https://review.openstack.org/223137 | 12:38 |
| *** slagle_ is now known as slagle | 12:39 | |
| *** dkusidlo has joined #heat | 12:41 | |
| *** jasond has joined #heat | 12:41 | |
| *** sorantis_ has joined #heat | 12:42 | |
| *** sorantis has quit IRC | 12:42 | |
| *** sorantis_ is now known as sorantis | 12:42 | |
| *** tiantian has joined #heat | 12:46 | |
| *** wendar has quit IRC | 12:47 | |
| *** serg_melikyan has joined #heat | 12:47 | |
| *** wendar has joined #heat | 12:47 | |
| *** dims_ has quit IRC | 12:47 | |
| *** dsneddon has quit IRC | 12:48 | |
| *** huangtianhua_ has quit IRC | 12:48 | |
| *** gondoi has quit IRC | 12:49 | |
| *** dsneddon has joined #heat | 12:49 | |
| *** sergmelikyan has quit IRC | 12:51 | |
| *** gondoi has joined #heat | 12:52 | |
| *** serg_melikyan has quit IRC | 12:54 | |
| *** sergmelikyan has joined #heat | 12:54 | |
| *** cumulus has quit IRC | 12:55 | |
| *** jonesbr has joined #heat | 12:57 | |
| *** rpothier has joined #heat | 12:58 | |
| *** dims_ has joined #heat | 12:58 | |
| *** lkarm has joined #heat | 12:58 | |
| *** EricGonczer_ has quit IRC | 12:59 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Do not lose traceback while raising exc https://review.openstack.org/223662 | 12:59 |
| *** tiantian has quit IRC | 13:00 | |
| *** tiantian has joined #heat | 13:01 | |
| *** dkusidlo has quit IRC | 13:01 | |
| *** dsneddon has quit IRC | 13:02 | |
| *** e0ne has joined #heat | 13:09 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: Allow selectively disabling resource validation https://review.openstack.org/223737 | 13:10 |
| openstackgerrit | Steven Hardy proposed openstack/heat: Add initial functional test for template-validate https://review.openstack.org/223579 | 13:10 |
| openstackgerrit | Steven Hardy proposed openstack/heat: Wire in files to template validate https://review.openstack.org/223580 | 13:10 |
| openstackgerrit | Steven Hardy proposed openstack/heat: WIP convert validate_template validation path https://review.openstack.org/223581 | 13:10 |
| openstackgerrit | Steven Hardy proposed openstack/heat: WIP Allow nested validation for template-validate https://review.openstack.org/224078 | 13:10 |
| shardy | jdob: ^^ Hey FYI that's a first pass at enabling recursive validation via the stack-preview method | 13:10 |
| shardy | needs tweaking next to allow multiple levels of nesting, but hopefully gets the idea over, as we build the entire tree like with stack preview, then walk it | 13:11 |
| jdob | cool, I'll take a look right now actually | 13:11 |
| shardy | jdob: I made FIXME note in the test, as I've deviated from the spec, would welcome your thoughts re that | 13:11 |
| shardy | I've added the NestedParameters per-resource, not per-resource-type, which I think we'll need if we want to express "value is passed from parent" vs undefined mandatory param that requires definition in parameter_defaults | 13:12 |
| *** hdd has joined #heat | 13:13 | |
| shardy | I also add a Type: key, as it seems helpful to map what type alias the params refer to, e.g what resource_registry entry | 13:13 |
| shardy | as without that info you're back into having to parse the template all over again I think | 13:13 |
| jdob | interesting. i wasnt sold on the value from parentt thing, but i also dont see any harm in more information if we have it | 13:13 |
| jdob | easy enough to filter out client-side | 13:13 |
| *** jaime49 has joined #heat | 13:14 | |
| *** MVenesio has quit IRC | 13:14 | |
| shardy | jdob: Ok, I'll add it and we can see how it looks | 13:14 |
| jdob | and agree on the type thing; if we move to per-resource, my first thought was that it'd be useful to resolve the types as well | 13:14 |
| shardy | I was thinking of e.g a third-party ExtraConfig template, where you want to distinguish the params that are unique to the implementation from those always passed via the parent | 13:14 |
| *** jprovazn has joined #heat | 13:14 | |
| jdob | especially since we dont have those sorts of introspection calls yet | 13:15 |
| *** MVenesio has joined #heat | 13:15 | |
| jdob | ya, it will be useful to have parent and optional side by side | 13:15 |
| shardy | jdob: Yeah, I can imagine this may get renamed to something other than "validate" when we reach a v2 API | 13:15 |
| *** dsneddon has joined #heat | 13:16 | |
| *** sergmelikyan has quit IRC | 13:17 | |
| *** neelashah has joined #heat | 13:17 | |
| jdob | shardy: dumb question, is there a way to pull the last commit in that dep tree and have it bring over all of the deps, or do i have to cherry pick them individually? | 13:17 |
| shardy | jdob: I think you can just cut/paste the "checkout" line from the gerrit page | 13:18 |
| shardy | unless there's anything local you want to rebase on | 13:18 |
| jdob | i'll try that | 13:18 |
| jdob | will be easy enough to verify | 13:18 |
| *** junhongl has quit IRC | 13:19 | |
| shardy | jdob: Ok, cool, any test feedback would be great - I'll crack on and get the fully recursive version going | 13:19 |
| shardy | jdob: Oh, the other way is git-review -d <Change Id> | 13:21 |
| shardy | which checks out the branch for you IIRC | 13:21 |
| *** dkusidlo has joined #heat | 13:23 | |
| *** sergmelikyan has joined #heat | 13:23 | |
| *** heatuser has quit IRC | 13:27 | |
| jdob | makes sense. i've used that before, just never checked to make sure deps come down, but it makes sense theyw ould | 13:28 |
| *** pratikma_ has joined #heat | 13:29 | |
| *** sergmelikyan has quit IRC | 13:32 | |
| *** pratikmallya has quit IRC | 13:32 | |
| *** sergmelikyan has joined #heat | 13:33 | |
| *** sergmelikyan has quit IRC | 13:33 | |
| *** LimorStotland has joined #heat | 13:33 | |
| *** sergmelikyan has joined #heat | 13:33 | |
| *** hdd has quit IRC | 13:33 | |
| *** tochi has quit IRC | 13:34 | |
| *** sergmelikyan has quit IRC | 13:36 | |
| *** blomquisg has quit IRC | 13:37 | |
| *** sergmelikyan has joined #heat | 13:38 | |
| *** dkusidlo has quit IRC | 13:39 | |
| *** achanda has joined #heat | 13:40 | |
| jdob | shardy: when you say "needs tweaking with multiple levels", are you saying it doesnt work or just needs some clean up? | 13:40 |
| *** achanda has quit IRC | 13:41 | |
| jdob | seems to infinitely loop for me when trying a ResourceGroup -> nested1 -> nested2 | 13:41 |
| shardy | jdob: the for loop in service.py needs to go in a function, so yeah multiple levels don't currently work | 13:41 |
| * shardy is fixing that now | 13:41 | |
| *** pas-ha has quit IRC | 13:41 | |
| jdob | gotcha | 13:42 |
| *** zhenguo has quit IRC | 13:43 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add floating_ip property to Nova::Server nics https://review.openstack.org/217754 | 13:47 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add intrinsic creating port for server https://review.openstack.org/223169 | 13:47 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Add port_extra_properties to Nova::Server nics https://review.openstack.org/224091 | 13:47 |
| *** sasukeh has joined #heat | 13:49 | |
| *** jdandrea has joined #heat | 13:50 | |
| *** blomquisg has joined #heat | 13:53 | |
| *** pas-ha has joined #heat | 13:56 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: WIP Allow nested validation for template-validate https://review.openstack.org/224078 | 14:00 |
| shardy | jdob: ^^ does recursion, now for cleanups and more tests | 14:01 |
| *** sergmelikyan has quit IRC | 14:02 | |
| openstackgerrit | Sergey Kraynev proposed openstack/heat: Store external ports in Server data https://review.openstack.org/224096 | 14:03 |
| jdob | shardy: cool, I got pulled off (again) for some last minute 7.1 fires, so i'll check in a bit | 14:03 |
| shardy | jdob: No worries, looking like I may get it fairly well sorted by later today, then you can try to break it ;) | 14:04 |
| openstackgerrit | Sergey Kraynev proposed openstack/heat: Copy resource data in _delete_backup_stack https://review.openstack.org/220986 | 14:05 |
| jdob | shardy: CHALLENGE ACCEPTED | 14:05 |
| openstackgerrit | Sergey Kraynev proposed openstack/heat: Handle replace and rollback cases for Port https://review.openstack.org/220502 | 14:05 |
| shardy | lol | 14:05 |
| *** pas-ha has quit IRC | 14:06 | |
| *** hdd has joined #heat | 14:07 | |
| *** sergmelikyan has joined #heat | 14:07 | |
| *** mragupat has joined #heat | 14:09 | |
| *** tshtilma has joined #heat | 14:09 | |
| *** mragupat has quit IRC | 14:10 | |
| *** tshtilma has quit IRC | 14:10 | |
| jdob | shardy: at first glance, that looks better, it didn't infinitely loop :) | 14:12 |
| shardy | jdob: cool, I'll add a functional with a ResourceGroup to help prove that pattern is OK | 14:13 |
| jdob | shardy: how much is left to do? is it worthwhile for me to start reviewing the code itself yet? | 14:14 |
| *** Marga_ has joined #heat | 14:16 | |
| *** jecarey has joined #heat | 14:16 | |
| *** tshtilma has joined #heat | 14:17 | |
| *** LimorStotland has quit IRC | 14:17 | |
| shardy | jdob: not much other than tests, the only change I may make is adding the "Value" for value-from-parent, and maybe refactor some of the stuff in service.py into stack.py | 14:17 |
| shardy | jdob: so, review feedback would be welcome :) | 14:18 |
| *** pas-ha has joined #heat | 14:18 | |
| *** vijayagurug has joined #heat | 14:19 | |
| openstackgerrit | Rakesh H S proposed openstack/heat: Convergence: Fix FnGetRefId to fetch value from cache_data https://review.openstack.org/223090 | 14:19 |
| jdob | shardy: kk | 14:19 |
| *** LimorStotland has joined #heat | 14:23 | |
| *** sdake has joined #heat | 14:25 | |
| *** jruano has joined #heat | 14:25 | |
| *** sdake_ has joined #heat | 14:25 | |
| *** sdake_ has quit IRC | 14:27 | |
| *** sdake_ has joined #heat | 14:27 | |
| *** vijendar has joined #heat | 14:27 | |
| *** crose has joined #heat | 14:28 | |
| *** alexpilotti has quit IRC | 14:29 | |
| *** sdake has quit IRC | 14:29 | |
| *** EricGonczer_ has joined #heat | 14:30 | |
| *** openstackgerrit has quit IRC | 14:31 | |
| *** openstackgerrit has joined #heat | 14:32 | |
| *** blomquisg has quit IRC | 14:34 | |
| *** Marga_ has quit IRC | 14:36 | |
| *** Marga_ has joined #heat | 14:37 | |
| *** david-lyle has quit IRC | 14:37 | |
| *** jecarey has quit IRC | 14:42 | |
| *** LimorStotland has quit IRC | 14:43 | |
| *** blomquisg has joined #heat | 14:45 | |
| *** charlesr has joined #heat | 14:46 | |
| *** LimorStotland has joined #heat | 14:48 | |
| *** tshtilma has quit IRC | 14:52 | |
| *** tshtilma has joined #heat | 14:53 | |
| *** Drago has joined #heat | 14:53 | |
| *** shardy_ has joined #heat | 14:54 | |
| *** Drago has quit IRC | 14:54 | |
| *** Drago has joined #heat | 14:54 | |
| *** shardy has quit IRC | 14:55 | |
| *** dkusidlo has joined #heat | 14:58 | |
| *** dims_ has quit IRC | 14:58 | |
| *** dims_ has joined #heat | 14:58 | |
| *** kebray has joined #heat | 14:58 | |
| *** jistr is now known as jistr|call | 14:58 | |
| openstackgerrit | Rakesh H S proposed openstack/heat: Convergence: Assign current_template_id when resource fails https://review.openstack.org/223091 | 14:59 |
| *** shardy_ has quit IRC | 14:59 | |
| *** sorantis has quit IRC | 14:59 | |
| *** shardy has joined #heat | 15:00 | |
| *** boris-42 has joined #heat | 15:01 | |
| *** openstackgerrit has quit IRC | 15:01 | |
| *** rakesh_hs has quit IRC | 15:01 | |
| *** openstackgerrit has joined #heat | 15:02 | |
| *** dkusidlo has quit IRC | 15:02 | |
| *** vijayagurug has left #heat | 15:03 | |
| *** manishg has joined #heat | 15:09 | |
| *** manishg_ has joined #heat | 15:10 | |
| *** manishg has quit IRC | 15:13 | |
| *** kebray has quit IRC | 15:15 | |
| *** ricolin has joined #heat | 15:16 | |
| *** dkusidlo has joined #heat | 15:16 | |
| *** dkusidlo has quit IRC | 15:16 | |
| *** jasond` has joined #heat | 15:18 | |
| *** jasond has quit IRC | 15:19 | |
| *** kebray has joined #heat | 15:22 | |
| *** manas has quit IRC | 15:23 | |
| *** Marga_ has quit IRC | 15:24 | |
| *** sasukeh has quit IRC | 15:27 | |
| *** Qiming has quit IRC | 15:28 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Fix translating for props with get_param value https://review.openstack.org/223137 | 15:29 |
| *** jistr|call is now known as jistr | 15:30 | |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: No need to stub is_service_available https://review.openstack.org/224153 | 15:31 |
| openstackgerrit | Jason Dunsmore proposed openstack/heat: Additional unit tests for update-preview https://review.openstack.org/222345 | 15:31 |
| *** vgridnev has quit IRC | 15:31 | |
| *** sasukeh has joined #heat | 15:34 | |
| *** david-ly_ has joined #heat | 15:35 | |
| *** david-ly_ is now known as david-lyle | 15:41 | |
| *** kfox1111 has left #heat | 15:41 | |
| *** Piet has joined #heat | 15:43 | |
| *** hdd has quit IRC | 15:43 | |
| *** hdd has joined #heat | 15:50 | |
| *** blomquisg has quit IRC | 15:51 | |
| *** KanagarajM has joined #heat | 15:52 | |
| *** jistr has quit IRC | 15:54 | |
| *** devlaps has joined #heat | 15:54 | |
| *** inc0 has joined #heat | 15:56 | |
| *** jecarey has joined #heat | 15:58 | |
| *** klindgren has joined #heat | 16:05 | |
| *** blomquisg has joined #heat | 16:05 | |
| *** Qiming has joined #heat | 16:05 | |
| klindgren | hello - I have a heat question :-) | 16:06 |
| klindgren | we use keystone backed by a read-only ad domain | 16:06 |
| klindgren | we created a seperate domain for heat so that heat can add in the stack users to that domain | 16:06 |
| klindgren | this was all working great | 16:07 |
| klindgren | until it came time to change the user domain password | 16:07 |
| klindgren | now we can't delete the stack that was created because thesotred password no longer matches the current user password | 16:07 |
| shardy | klindgren: to clarify, you changed the password of the stack_domain_admin configured in heat.conf? | 16:08 |
| *** Marga_ has joined #heat | 16:09 | |
| shardy | and you updated the stack_domain_admin_password to match the new password, and restarted all heat-engine services? | 16:09 |
| klindgren | and its failing to remove wait-states with an unauthorized error. I attempted to redefine the stack from a new template and i can set a new password, however, this also fails. | 16:09 |
| klindgren | shardy - no | 16:09 |
| klindgren | I change the password of the user who created the stack | 16:09 |
| klindgren | IE in horizon whne you spinup a new stack it wasn't the password of the current user | 16:09 |
| klindgren | I assume to do future operations | 16:09 |
| klindgren | so now I login to openstack with password2 | 16:10 |
| klindgren | and heat still has password1 stored in the db | 16:10 |
| klindgren | and I can't delete the stack because everything is unauthorized | 16:10 |
| shardy | klindgren: are you sure the domain stuff was working, e.g the heat stack users were definitely getting created in the heat domain? | 16:11 |
| klindgren | shardy - yes when I create a stack user, the keystone db gets a user added under it | 16:11 |
| shardy | klindgren: Heat storing a password is related to deferred_auth_method being password instead of trusts (now the default) | 16:12 |
| klindgren | since we do auth against read-only AD, none of our other stuff creates users in the domain | 16:12 |
| shardy | that's not actually related to the domain isolated users, those credentials are used e.g for AutoScaling and other deferred operations | 16:12 |
| shardy | which will indeed break if you use deferred_auth_method=password then change the password | 16:12 |
| shardy | but it shouldn't cause you to fail on stack-delete | 16:13 |
| klindgren | deferred_auth_method=password indeed it is | 16:13 |
| klindgren | changing that to trust should fix it? | 16:13 |
| shardy | klindgren: Ok, so I'm still not sure why that's causing a delete failure, but yes trusts has been reccomended for some time | 16:13 |
| *** Qiming has quit IRC | 16:13 | |
| klindgren | shardy well it does for sure when trying to delete. Let me check that. | 16:14 |
| shardy | it means we no longer store a password, instead we store a trust ID, which won't change when you change your keystone password | 16:14 |
| shardy | klindgren: what openstack/heat version are you on? | 16:14 |
| klindgren | we installed heat a long time ago (havana) and ran into the read-only dc issue | 16:14 |
| klindgren | and only recently has keystone supported multiple auth domains - so we are revisting in kilo | 16:15 |
| shardy | http://hardysteven.blogspot.co.uk/2014/04/heat-auth-model-updates-part-1-trusts.html | 16:15 |
| shardy | klindgren: that explains the trust usage in heat, which may be useful context | 16:15 |
| shardy | one thing I should change is recent heat versions don't require a special role, you can set trusts_delegated_roles to an empty list (now the default) and we just delegate all the user roles | 16:16 |
| shardy | klindgren: on Kilo I'd definitely recommend deferred_auth_method = trusts and trusts_delegated_roles = <empty> which are both now the defaults, e.g you can just remove both from your heat.conf | 16:17 |
| shardy | klindgren: that should fix your issues for new stacks, but we need more info to resolve the password delete problem | 16:17 |
| *** KanagarajM has quit IRC | 16:17 | |
| shardy | if you can raise a bug with a backtrace and reproducer I can take a look | 16:18 |
| *** ricolin has quit IRC | 16:18 | |
| *** sdake_ is now known as sdake | 16:23 | |
| *** sasukeh has quit IRC | 16:26 | |
| *** pratikma_ has quit IRC | 16:27 | |
| *** jaime49 has quit IRC | 16:30 | |
| klindgren | shardy, https://bugs.launchpad.net/heat/+bug/1496485 | 16:33 |
| openstack | Launchpad bug 1496485 in heat "Heat stack can't delete when current user password no long matches stored user password" [Undecided,New] | 16:33 |
| *** sasukeh has joined #heat | 16:34 | |
| *** jruano has quit IRC | 16:36 | |
| shardy | klindgren: the backtrace shows an error getting a token for the domain-admin user - is it at all possible that password changed as well? | 16:40 |
| shardy | klindgren: and, can you definitely create new stacks containing WaitConditionHandle resources? | 16:40 |
| *** vgridnev has joined #heat | 16:41 | |
| klindgren | I can create and delete stacks as long aas the sotred password matches my current password | 16:41 |
| klindgren | but let me make sure thats 100% ture - its possible someone has been making changes to this env behind my back | 16:42 |
| *** jasond`` has joined #heat | 16:43 | |
| shardy | klindgren: Ok, I am confused then, as that backtrace shouldn't use the credentials stored via deferred_auth_method=password at all, it should use stack_domain* from heat.conf | 16:43 |
| shardy | If you can confirm that stack_domain_admin_password definitely has not changed, I'll try a reproduce as something strange is happening | 16:44 |
| *** sasukeh has quit IRC | 16:45 | |
| *** jasond` has quit IRC | 16:46 | |
| *** neelashah has quit IRC | 16:48 | |
| openstackgerrit | Merged openstack/heat: Fix scenario names in rolling update test https://review.openstack.org/222220 | 16:48 |
| klindgren | and pebkac :( | 16:50 |
| shardy | lol :) | 16:50 |
| klindgren | so apparently a co-worker did change the stack domain admin account yesterday | 16:50 |
| shardy | boom | 16:50 |
| shardy | Ok, well easy fix then :) | 16:50 |
| openstackgerrit | Merged openstack/heat: Add test for _show_resource https://review.openstack.org/223026 | 16:51 |
| shardy | klindgren: if you're routinely changing passwords I'd defintely reccomend moving to trusts though, especially if you're using AutoScalingGroups | 16:52 |
| klindgren | yea - we are reconfiguring for trusts now :-) | 16:52 |
| *** sasukeh has joined #heat | 16:52 | |
| *** sasukeh has quit IRC | 16:55 | |
| *** derekh is now known as derekh_afk | 16:59 | |
| *** lkarm has quit IRC | 17:02 | |
| *** sasukeh has joined #heat | 17:02 | |
| *** kebray has quit IRC | 17:03 | |
| *** neelashah has joined #heat | 17:04 | |
| *** kebray has joined #heat | 17:05 | |
| *** pratikmallya has joined #heat | 17:06 | |
| *** lkarm has joined #heat | 17:07 | |
| *** yassine_ has quit IRC | 17:07 | |
| *** sasukeh has quit IRC | 17:07 | |
| *** inc0 has quit IRC | 17:08 | |
| *** zeih has joined #heat | 17:08 | |
| *** zeih has quit IRC | 17:08 | |
| *** achanda has joined #heat | 17:09 | |
| *** sasukeh has joined #heat | 17:12 | |
| *** lsmola has quit IRC | 17:13 | |
| *** e0ne has quit IRC | 17:19 | |
| openstackgerrit | Merged openstack/heat: Allow map/list items for list_join https://review.openstack.org/221765 | 17:20 |
| *** sasukeh has quit IRC | 17:27 | |
| *** lkarm has quit IRC | 17:29 | |
| *** sasukeh has joined #heat | 17:30 | |
| *** pas-ha has quit IRC | 17:32 | |
| *** sasukeh has quit IRC | 17:35 | |
| shardy | zaneb: Hey, I have a question about ResourceDefinition - I see there are a bunch of asserts in the constructor, is there any reason not to just raise StackValidation failed there instead? | 17:35 |
| zaneb | shardy: yes | 17:35 |
| zaneb | StackValidationFailed = user screwed up | 17:35 |
| zaneb | AssertionError = programmer screwed up | 17:36 |
| zaneb | they're completely different | 17:36 |
| shardy | zaneb: I'm hitting an issue with the new template-validate where an invalid deletion_policy is incorrectly reported, because we assert here: | 17:36 |
| shardy | https://github.com/openstack/heat/blob/master/heat/engine/stack.py#L627 | 17:36 |
| shardy | because now we create a Stack object to to the pre-validate stack.validate() | 17:37 |
| *** lkarm has joined #heat | 17:37 | |
| shardy | and it's possible to hit the constructor error path depending on the template being malformed | 17:37 |
| shardy | Maybe I screwed up, let me see if it fails better on create ;) | 17:37 |
| zaneb | shardy: I'm confused, because there is no assert at the link you pasted ;) | 17:38 |
| shardy | zaneb: the iterkeys creates all teh resource objects and thus we hit the ResourceDefinition constructor | 17:39 |
| shardy | sec, let me paste a reproduce on latest master | 17:39 |
| zaneb | if there's code that creates a ResourceDefinition with data that causes ResourceDefinition to assert, that code is passing the wrong data | 17:39 |
| shardy | zaneb: http://paste.openstack.org/show/465000/ | 17:42 |
| shardy | I raised bug #1496502 | 17:43 |
| openstack | bug 1496502 in heat "Invalid deletion policy not correctly validated" [Undecided,New] https://launchpad.net/bugs/1496502 - Assigned to Steven Hardy (shardy) | 17:43 |
| zaneb | shardy: we should never pass user-supplied data as the deletion policy | 17:45 |
| zaneb | the deletion policy should be ResourceDefinition.DELETE, ResourceDefinition.RETAIN or ResourceDefinition.SNAPSHOT | 17:46 |
| shardy | zaneb: I was thinking maybe we should just check that's a string or function, and punt the validation to Resource.validate_deletion_policy | 17:46 |
| zaneb | and all code should work equally well if those are defined to be "Delete", "Retain" and "Snapshot" or "foo", "wibble" and "quux" | 17:46 |
| shardy | zaneb: Yeah, well the issue is we're hitting that before we validate the user-supplied deletion_policy from the template | 17:47 |
| zaneb | shardy: we should explicitly translate it in HOTemplate.resource_definitions() | 17:47 |
| shardy | aha, I see it's missing | 17:48 |
| zaneb | https://github.com/openstack/heat/blob/master/heat/engine/hot/template.py#L258 <- this line is wrong | 17:48 |
| zaneb | I'm sure git blame will show it's my fault :) | 17:49 |
| zaneb | but this is why we have asserts | 17:49 |
| *** charlesr has quit IRC | 17:52 | |
| *** crose has quit IRC | 17:52 | |
| *** crose has joined #heat | 17:52 | |
| *** pratikma_ has joined #heat | 17:53 | |
| *** lkarm has quit IRC | 17:53 | |
| *** tshtilma has quit IRC | 17:54 | |
| *** sdake_ has joined #heat | 17:54 | |
| *** pratikmallya has quit IRC | 17:56 | |
| shardy | zaneb: Ok, sorry, it's getting late, but I'm still confused - are you saying we should do the value validation in HOTemplate instead of ResourceDefinition (or resource.Resource)? | 17:58 |
| *** sdake has quit IRC | 17:58 | |
| *** e0ne has joined #heat | 17:58 | |
| *** MVenesio has quit IRC | 17:59 | |
| zaneb | in HOTemplate we should have a map from valid HOT values to the ResourceDefinition API: {'Delete': ResourceDefinition.DELETE, 'Retain': ResourceDefintion.RETAIN, ...} | 17:59 |
| zaneb | and use that to get the input value for ResourceDefinition | 18:00 |
| zaneb | right now it is correctly asserting because we (I) didn't do this | 18:00 |
| shardy | zaneb: Ok, and same for the Cfn ResourceDefinition, and probably remove the validate_deletion_policy in resource.Resource because we then can't ever hit it | 18:00 |
| zaneb | I guess we're effectively validating it earlier now | 18:01 |
| shardy | Yeah, I think that's where I got confused, validating the same thing in multiple places | 18:01 |
| zaneb | but we should be careful not to start moving validation code into the template loading process | 18:02 |
| zaneb | because that leaves us with old, unvalidated stacks in the DB that can never be shown/deleted | 18:02 |
| zaneb | we just got away from all that | 18:02 |
| shardy | zaneb: Ok, well thanks for the clarification - feel free to grab that bug if you feel like it ;) I probably won't get to it today now | 18:04 |
| zaneb | I likely won't have time either, but I left a comment case someone does | 18:05 |
| shardy | Ok no worries, I'll pick it up in the morning unless someone beats me to it, cheers | 18:05 |
| openstackgerrit | Steven Hardy proposed openstack/heat: Allow map/list items for str_replace https://review.openstack.org/221766 | 18:07 |
| *** zeih has joined #heat | 18:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Imported Translations from Zanata https://review.openstack.org/224235 | 18:11 |
| *** jaime49 has joined #heat | 18:15 | |
| *** zeih has quit IRC | 18:15 | |
| *** openstackgerrit has quit IRC | 18:16 | |
| *** openstackgerrit has joined #heat | 18:17 | |
| *** jprovazn has quit IRC | 18:17 | |
| *** e0ne has quit IRC | 18:18 | |
| *** e0ne has joined #heat | 18:20 | |
| *** achanda has quit IRC | 18:25 | |
| klindgren | shardy, one more question. Is it possible to now run multiple heat-engines at the same time on different machines? (kilo version) | 18:27 |
| *** jaime49 has quit IRC | 18:27 | |
| *** jaime49 has joined #heat | 18:27 | |
| shardy | klindgren: Yes, provided they can all be reached via RPC | 18:29 |
| *** david-lyle has quit IRC | 18:30 | |
| *** boris-42 has quit IRC | 18:30 | |
| shardy | klindgren: recent versions of heat support the same scale-out model as other openstack services (I know a long while ago that was not the case) | 18:30 |
| klindgren | kk - I knew a while back that you could only run heat-engine on one server | 18:31 |
| klindgren | so we were still running that in pacemaker - happy to hear I can kill that with fire | 18:31 |
| shardy | klindgren: Yeah, IIRC we actually fixed that in Icehouse ;) | 18:33 |
| klindgren | well - like I said - we bascially stopped looking at heat until the multiple domain thing was fixed in keystone - or atleast thats the blocker we identified and have been waiting on to re-look at it | 18:35 |
| *** vgridnev has quit IRC | 18:38 | |
| *** Drago has quit IRC | 18:43 | |
| *** jruano has joined #heat | 18:50 | |
| *** nijaba_ has joined #heat | 18:54 | |
| *** nijaba_ has quit IRC | 18:54 | |
| *** nijaba_ has joined #heat | 18:54 | |
| *** nijaba has quit IRC | 18:55 | |
| *** manishg_ has quit IRC | 19:00 | |
| *** jasond`` has quit IRC | 19:00 | |
| *** jasond has joined #heat | 19:01 | |
| *** manishg has joined #heat | 19:01 | |
| *** manishg has quit IRC | 19:01 | |
| *** vgridnev has joined #heat | 19:01 | |
| *** manishg has joined #heat | 19:02 | |
| *** blomquisg has quit IRC | 19:03 | |
| *** Drago has joined #heat | 19:04 | |
| *** achanda has joined #heat | 19:12 | |
| *** lkarm has joined #heat | 19:15 | |
| *** blomquisg has joined #heat | 19:19 | |
| *** protoz has joined #heat | 19:27 | |
| *** jasond has quit IRC | 19:34 | |
| *** jasond has joined #heat | 19:34 | |
| *** e0ne has quit IRC | 19:35 | |
| *** achanda_ has joined #heat | 19:40 | |
| *** achanda has quit IRC | 19:40 | |
| *** manishg has quit IRC | 19:46 | |
| *** sdake has joined #heat | 19:49 | |
| *** manishg_ has joined #heat | 19:50 | |
| *** sdake_ has quit IRC | 19:52 | |
| *** pratikma_ has quit IRC | 19:53 | |
| *** hdd has quit IRC | 19:55 | |
| *** hdd has joined #heat | 19:57 | |
| *** openstackstatus has quit IRC | 20:02 | |
| *** openstackstatus has joined #heat | 20:02 | |
| *** ChanServ sets mode: +v openstackstatus | 20:02 | |
| *** derekh_afk is now known as derekh | 20:06 | |
| *** hdd has quit IRC | 20:09 | |
| *** cody-somerville has joined #heat | 20:10 | |
| *** cody-somerville has quit IRC | 20:10 | |
| *** cody-somerville has joined #heat | 20:10 | |
| *** derekh has quit IRC | 20:11 | |
| *** pratikmallya has joined #heat | 20:14 | |
| *** pratikma_ has joined #heat | 20:17 | |
| *** Piet has quit IRC | 20:20 | |
| *** pratikmallya has quit IRC | 20:21 | |
| *** gberginc has quit IRC | 20:22 | |
| *** hdd has joined #heat | 20:29 | |
| openstackgerrit | Miguel Grinberg proposed openstack/heat: Inherit SwiftSignal behavior from WaitCondition https://review.openstack.org/222844 | 20:32 |
| openstackgerrit | Miguel Grinberg proposed openstack/heat: Add signal_transport property to Heat wait conditions https://review.openstack.org/219996 | 20:32 |
| openstackgerrit | Steven Hardy proposed openstack/heat: Validate deletion policy during template parsing https://review.openstack.org/224292 | 20:40 |
| *** lkarm_ has joined #heat | 20:41 | |
| *** lkarm has quit IRC | 20:44 | |
| *** shardy has quit IRC | 20:44 | |
| *** jasond has quit IRC | 20:48 | |
| *** jasond has joined #heat | 20:56 | |
| *** crose has quit IRC | 20:58 | |
| *** jaime49 has quit IRC | 21:00 | |
| *** Piet has joined #heat | 21:00 | |
| *** protoz has quit IRC | 21:08 | |
| *** cody-somerville has quit IRC | 21:09 | |
| *** jasond has quit IRC | 21:10 | |
| *** jasond has joined #heat | 21:10 | |
| *** vgridnev has quit IRC | 21:13 | |
| openstackgerrit | Jason Dunsmore proposed openstack/heat: Add _needs_update() to AWS::EC2::EIPAssociation https://review.openstack.org/224309 | 21:14 |
| *** crose has joined #heat | 21:16 | |
| *** jaime49 has joined #heat | 21:20 | |
| *** jaime49 has quit IRC | 21:20 | |
| *** jaime49 has joined #heat | 21:20 | |
| *** jonesbr has left #heat | 21:24 | |
| *** boris-42 has joined #heat | 21:26 | |
| *** lkarm_ has quit IRC | 21:27 | |
| *** jasond has quit IRC | 21:32 | |
| *** cody-somerville has joined #heat | 21:35 | |
| *** EricGonczer_ has quit IRC | 21:40 | |
| *** Drago has quit IRC | 21:41 | |
| *** vijendar has quit IRC | 21:42 | |
| *** achanda_ has quit IRC | 21:45 | |
| *** rpothier has quit IRC | 21:48 | |
| openstackgerrit | Jason Dunsmore proposed openstack/heat: Add _needs_update() to AWS::AutoScaling::LaunchConfiguration https://review.openstack.org/224321 | 21:49 |
| *** blomquisg has quit IRC | 21:50 | |
| *** david-lyle has joined #heat | 21:56 | |
| *** achanda has joined #heat | 21:59 | |
| *** MasterPiece has joined #heat | 22:01 | |
| *** jecarey has quit IRC | 22:02 | |
| *** neelashah has quit IRC | 22:02 | |
| *** klindgren_ has joined #heat | 22:02 | |
| *** klindgren has quit IRC | 22:03 | |
| *** Drago has joined #heat | 22:04 | |
| *** Drago has quit IRC | 22:04 | |
| *** kebray has quit IRC | 22:04 | |
| *** Drago has joined #heat | 22:04 | |
| *** achanda has quit IRC | 22:07 | |
| *** achanda has joined #heat | 22:09 | |
| *** zeih has joined #heat | 22:12 | |
| *** achanda_ has joined #heat | 22:17 | |
| *** achanda has quit IRC | 22:17 | |
| *** zeih has quit IRC | 22:17 | |
| *** david-lyle has quit IRC | 22:29 | |
| *** jruano has quit IRC | 22:31 | |
| *** pratikmallya has joined #heat | 22:32 | |
| *** jpeeler has quit IRC | 22:32 | |
| *** pratikma_ has quit IRC | 22:32 | |
| *** kebray has joined #heat | 22:35 | |
| *** dims_ has quit IRC | 22:48 | |
| *** cody-somerville has quit IRC | 22:53 | |
| *** kebray has quit IRC | 22:54 | |
| *** pratikmallya has quit IRC | 23:10 | |
| asalkeld | o/ | 23:16 |
| *** jruano has joined #heat | 23:27 | |
| *** dims_ has joined #heat | 23:32 | |
| openstackgerrit | Merged openstack/heat: Save previous template on non convergence updates https://review.openstack.org/222675 | 23:37 |
| *** chlong has quit IRC | 23:41 | |
| *** tochi has joined #heat | 23:42 | |
| *** KanagarajM has joined #heat | 23:51 | |
| *** steveg_afk has quit IRC | 23:55 | |
| KanagarajM | stevebaker: hi, I tried to fix the build failure in template project , https://review.openstack.org/224051 | 23:59 |
| KanagarajM | stevebaker: could you please review it? | 23:59 |
| stevebaker | KanagarajM: sure, thanks | 23:59 |
| stevebaker | hey look, it passes! | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!