| zaneb | meeting time, y'all | 00:00 |
|---|---|---|
| *** andrew_plunk has quit IRC | 00:01 | |
| zaneb | radix SpamapS asalkeld ^ | 00:04 |
| *** arbylee1 has quit IRC | 00:05 | |
| *** WinnieTsang has joined #heat | 00:05 | |
| radix | thanks | 00:05 |
| *** mestery has joined #heat | 00:07 | |
| *** jcru has quit IRC | 00:08 | |
| *** mestery has quit IRC | 00:08 | |
| *** wchrisj_ has quit IRC | 00:11 | |
| *** yogesh has joined #heat | 00:11 | |
| *** wchrisj has joined #heat | 00:11 | |
| *** andrew_plunk has joined #heat | 00:14 | |
| *** wchrisj has quit IRC | 00:16 | |
| *** wchrisj has joined #heat | 00:17 | |
| *** matsuhashi has joined #heat | 00:20 | |
| *** jergerber has quit IRC | 00:22 | |
| *** david-lyle has quit IRC | 00:24 | |
| *** yogesh has quit IRC | 00:34 | |
| *** wchrisj has quit IRC | 00:37 | |
| *** tango|2 has quit IRC | 00:38 | |
| *** wchrisj has joined #heat | 00:50 | |
| *** andrew_plunk has quit IRC | 00:52 | |
| zaneb | I got me one of these ergonometric keyboards | 00:56 |
| *** TravT has quit IRC | 00:56 | |
| *** nosnos has joined #heat | 00:56 | |
| zaneb | at least I am getting plenty of practice in learning how to type again :D | 00:56 |
| radix | zaneb: which one? | 00:59 |
| *** nanjj has quit IRC | 01:00 | |
| *** randallburt has quit IRC | 01:05 | |
| radix | alright, I'm out for the night, later :) | 01:08 |
| *** achampion has joined #heat | 01:19 | |
| *** achampion has quit IRC | 01:23 | |
| *** pvaneck has quit IRC | 01:23 | |
| *** cody-somerville has quit IRC | 01:25 | |
| *** achampion has joined #heat | 01:29 | |
| *** Invent has joined #heat | 01:37 | |
| *** Invent has quit IRC | 01:42 | |
| *** TravT has joined #heat | 01:43 | |
| *** bada_ has quit IRC | 01:45 | |
| *** kfox1111 has quit IRC | 01:46 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/heat: Create heat database with default character set utf-8 https://review.openstack.org/71437 | 01:48 |
| *** arbylee has joined #heat | 01:49 | |
| openstackgerrit | Ton Ngo proposed a change to openstack/python-heatclient: Adds CLI suppport for troubleshooting failed stack https://review.openstack.org/71438 | 01:51 |
| *** wchrisj has quit IRC | 01:53 | |
| aweiteka | stevebaker, you around? I just learned I should wish you happy Waitangi day. :) | 01:54 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/58641 | 01:55 |
| *** mestery has joined #heat | 01:57 | |
| *** gokrokve has quit IRC | 02:02 | |
| *** IlyaE has quit IRC | 02:11 | |
| *** mestery has quit IRC | 02:15 | |
| *** WinnieTsang has quit IRC | 02:21 | |
| *** erkules has quit IRC | 02:34 | |
| *** nkhare_ has joined #heat | 02:40 | |
| *** dims has joined #heat | 02:44 | |
| *** harlowja has quit IRC | 02:44 | |
| *** erkules has joined #heat | 02:48 | |
| *** gokrokve has joined #heat | 02:50 | |
| *** asalkeld has quit IRC | 02:52 | |
| *** harlowja has joined #heat | 02:55 | |
| *** gokrokve has quit IRC | 02:55 | |
| *** gokrokve has joined #heat | 02:56 | |
| *** randallburt has joined #heat | 02:56 | |
| *** spzala has quit IRC | 02:58 | |
| *** gokrokve has quit IRC | 03:00 | |
| *** gokrokve has joined #heat | 03:00 | |
| *** gokrokve has quit IRC | 03:06 | |
| *** aignatov_ is now known as aignatov | 03:11 | |
| *** coolsvap_away has quit IRC | 03:13 | |
| *** asalkeld has joined #heat | 03:13 | |
| *** randallburt has quit IRC | 03:18 | |
| *** randallburt has joined #heat | 03:18 | |
| *** yogesh has joined #heat | 03:20 | |
| *** yogesh has quit IRC | 03:25 | |
| *** lcheng_ has joined #heat | 03:26 | |
| *** Invent has joined #heat | 03:38 | |
| *** Invent has quit IRC | 03:42 | |
| *** WinnieTsang has joined #heat | 03:45 | |
| *** david-lyle has joined #heat | 03:51 | |
| *** achampion has quit IRC | 03:54 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/71345 | 03:56 |
| *** arbylee has quit IRC | 04:04 | |
| *** lcheng_ has quit IRC | 04:06 | |
| *** wchrisj has joined #heat | 04:17 | |
| *** randallburt has quit IRC | 04:18 | |
| *** wchrisj has quit IRC | 04:23 | |
| *** gokrokve has joined #heat | 04:24 | |
| *** arbylee1 has joined #heat | 04:36 | |
| *** achampion has joined #heat | 04:36 | |
| *** TravT has quit IRC | 04:37 | |
| *** sgordon has quit IRC | 04:37 | |
| *** lcheng_ has joined #heat | 04:38 | |
| *** coolsvap has joined #heat | 04:44 | |
| *** akuznetsov has joined #heat | 04:45 | |
| *** arbylee1 has quit IRC | 04:48 | |
| *** gokrokve has quit IRC | 04:50 | |
| *** gokrokve has joined #heat | 04:52 | |
| *** arbylee has joined #heat | 04:52 | |
| *** gokrokve has quit IRC | 04:53 | |
| *** achampion has quit IRC | 04:56 | |
| *** nosnos_ has joined #heat | 05:02 | |
| *** dims has quit IRC | 05:03 | |
| *** dims has joined #heat | 05:04 | |
| *** jrist has quit IRC | 05:04 | |
| *** munx has quit IRC | 05:05 | |
| *** harlowja has quit IRC | 05:05 | |
| *** nosnos has quit IRC | 05:06 | |
| *** arbylee has quit IRC | 05:07 | |
| *** pablosan has quit IRC | 05:09 | |
| *** pablosan has joined #heat | 05:11 | |
| *** harlowja has joined #heat | 05:14 | |
| *** arbylee has joined #heat | 05:16 | |
| *** arbylee has quit IRC | 05:18 | |
| *** blamar has quit IRC | 05:21 | |
| *** cmyster has joined #heat | 05:25 | |
| *** cmyster has joined #heat | 05:25 | |
| *** aignatov is now known as aignatov_ | 05:27 | |
| *** edmund1 has quit IRC | 05:29 | |
| openstackgerrit | Steven Dake proposed a change to openstack/heat-templates: Add a F20 WordPress_Native template https://review.openstack.org/71459 | 05:32 |
| *** akuznetsov has quit IRC | 05:34 | |
| *** akuznetsov has joined #heat | 05:36 | |
| *** jrist has joined #heat | 05:39 | |
| *** Invent has joined #heat | 05:39 | |
| *** e0ne has joined #heat | 05:40 | |
| *** jrist has quit IRC | 05:43 | |
| *** jrist has joined #heat | 05:43 | |
| *** Invent has quit IRC | 05:43 | |
| cmyster | morning | 05:47 |
| *** jrist has quit IRC | 05:52 | |
| *** sergmelikyan has quit IRC | 05:58 | |
| *** dteselkin_ has quit IRC | 05:59 | |
| *** dteselkin_ has joined #heat | 05:59 | |
| *** matsuhashi has quit IRC | 06:07 | |
| *** matsuhashi has joined #heat | 06:07 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/70517 | 06:08 |
| *** matsuhas_ has joined #heat | 06:09 | |
| *** matsuhashi has quit IRC | 06:09 | |
| *** topol has quit IRC | 06:24 | |
| skraynev | Morning! | 06:25 |
| *** e0ne has quit IRC | 06:27 | |
| *** slong has quit IRC | 06:33 | |
| *** IlyaE has joined #heat | 06:34 | |
| *** nosnos has joined #heat | 06:40 | |
| *** yogesh has joined #heat | 06:42 | |
| *** nosnos_ has quit IRC | 06:42 | |
| *** IlyaE has quit IRC | 06:45 | |
| *** coolsvap_away has joined #heat | 06:51 | |
| *** coolsvap has quit IRC | 06:52 | |
| *** nkhare_ has quit IRC | 06:57 | |
| *** IlyaE has joined #heat | 06:58 | |
| *** coolsvap_away has quit IRC | 07:02 | |
| *** yogesh has quit IRC | 07:03 | |
| *** funzo has quit IRC | 07:05 | |
| *** nati_ueno has quit IRC | 07:06 | |
| *** akuznetsov has quit IRC | 07:06 | |
| *** coolsvap has joined #heat | 07:07 | |
| *** akuznetsov has joined #heat | 07:09 | |
| *** IlyaE has quit IRC | 07:09 | |
| *** coolsvap_away has joined #heat | 07:11 | |
| *** nkhare_ has joined #heat | 07:11 | |
| *** bvandenh has joined #heat | 07:12 | |
| *** coolsvap has quit IRC | 07:12 | |
| *** funzo has joined #heat | 07:13 | |
| *** lcheng_ has quit IRC | 07:14 | |
| *** lcheng_ has joined #heat | 07:15 | |
| *** jprovazn has joined #heat | 07:20 | |
| *** IlyaE has joined #heat | 07:21 | |
| *** harlowja is now known as harlowja_away | 07:23 | |
| *** alexheneveld has quit IRC | 07:30 | |
| *** tspatzier has joined #heat | 07:31 | |
| *** pasquier-s_ has quit IRC | 07:32 | |
| *** pasquier-s_ has joined #heat | 07:32 | |
| *** IlyaE has quit IRC | 07:33 | |
| *** coolsvap_away is now known as coolsvap | 07:37 | |
| *** Invent has joined #heat | 07:40 | |
| *** IlyaE has joined #heat | 07:41 | |
| *** Invent has quit IRC | 07:44 | |
| *** asalkeld has quit IRC | 07:46 | |
| *** e0ne has joined #heat | 07:47 | |
| *** IlyaE has quit IRC | 07:54 | |
| *** kevinbenton has joined #heat | 08:00 | |
| therve | Good morning! | 08:01 |
| *** killer_prince has quit IRC | 08:03 | |
| *** viktors has joined #heat | 08:05 | |
| *** lcheng_ has quit IRC | 08:12 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/python-heatclient: Enable default in bash completion https://review.openstack.org/71307 | 08:13 |
| *** jistr has joined #heat | 08:14 | |
| *** jamieh has joined #heat | 08:14 | |
| *** tspatzier has quit IRC | 08:36 | |
| *** nati_ueno has joined #heat | 08:40 | |
| *** tspatzier has joined #heat | 08:44 | |
| *** ifarkas has joined #heat | 08:45 | |
| shardy | morning all | 08:49 |
| cmyster | morning | 08:52 |
| skraynev | hi folks | 08:53 |
| cmyster | hi | 08:54 |
| cmyster | I'm looking for an image called Fedora-x86_64-20-20131211.1-sda (in sdake's proposed template) where might I find it ? | 09:07 |
| shardy | It's probably a Fedora cloud image, but not sure which one: | 09:09 |
| shardy | http://fedoraproject.org/en/get-fedora#clouds | 09:09 |
| shardy | I'd suggest using the latest F20 qcow image, as it already contains heat-cfntools | 09:09 |
| cmyster | probably a better option indeed. | 09:10 |
| *** aignatov_ is now known as aignatov | 09:10 | |
| shardy | http://openstack.redhat.com/forum/discussion/959/get-fedora-20-now#Item_1 | 09:10 |
| cmyster | google took me to that exactl page :) | 09:11 |
| cmyster | exact even | 09:11 |
| cmyster | same version btw | 09:11 |
| shardy | google ftw :) | 09:11 |
| cmyster | sad, but true | 09:11 |
| cmyster | gogoduck is still lacking | 09:11 |
| *** e0ne_ has joined #heat | 09:12 | |
| *** e0ne has quit IRC | 09:14 | |
| pshchelo | morning all :) | 09:14 |
| cmyster | morning | 09:14 |
| *** tspatzier has joined #heat | 09:19 | |
| *** nati_ueno has quit IRC | 09:23 | |
| *** tspatzier has quit IRC | 09:26 | |
| *** klwilson has quit IRC | 09:28 | |
| *** cody-somerville has joined #heat | 09:28 | |
| *** cody-somerville has quit IRC | 09:28 | |
| *** cody-somerville has joined #heat | 09:28 | |
| *** nati_ueno has joined #heat | 09:29 | |
| *** klwilson has joined #heat | 09:29 | |
| *** nati_ueno has quit IRC | 09:36 | |
| *** derekh has joined #heat | 09:38 | |
| *** Invent has joined #heat | 09:41 | |
| *** Invent has quit IRC | 09:45 | |
| shardy | therve: Hi | 09:56 |
| shardy | therve: IIRC, I saw you discussing some objections to the docs changes which are proposed, can you remind me what the problem was please? | 09:57 |
| shardy | We have a long queue of docs stuff, all in a chain so wondering how we can get the queue moving into the tree: | 09:57 |
| shardy | https://review.openstack.org/#/q/status:open+project:openstack/heat+branch:master+topic:bug/1271226,n,z | 09:57 |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/heat: Implements resource type NetworkGateway https://review.openstack.org/62287 | 09:57 |
| therve | shardy, https://review.openstack.org/#/c/68746/ I think is the point of contention | 10:00 |
| therve | It seems it was changed though | 10:00 |
| shardy | Oh right, I see andersonvom pasted your discussion, thanks | 10:02 |
| shardy | sucks that we need a hack like that just to generate docs :\ | 10:04 |
| shardy | why can't the code which generates the docs know which directories to look in and force-register them dynamically? | 10:05 |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Delete fixed_ips if it's empty list https://review.openstack.org/68648 | 10:07 |
| therve | shardy, That's what I tried to suggest | 10:08 |
| therve | I don't know what's involved though | 10:08 |
| *** tspatzier has joined #heat | 10:09 | |
| shardy | therve: me neither but it seems like a nicer solution | 10:10 |
| *** jprovazn has quit IRC | 10:10 | |
| *** ruhe has joined #heat | 10:21 | |
| *** aignatov is now known as aignatov_ | 10:22 | |
| *** e0ne_ has quit IRC | 10:26 | |
| *** viktors has left #heat | 10:27 | |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack/python-heatclient: Port to python requests https://review.openstack.org/65704 | 10:36 |
| *** jistr has quit IRC | 10:44 | |
| *** denis_makogon has joined #heat | 10:46 | |
| *** alexheneveld has joined #heat | 10:48 | |
| openstackgerrit | Dmitry Shulyak proposed a change to openstack/heat: Transition to alembic migration with oslo migration tool https://review.openstack.org/63767 | 10:49 |
| *** bgorski has joined #heat | 10:53 | |
| *** aignatov_ is now known as aignatov | 10:58 | |
| *** scollier has joined #heat | 11:00 | |
| *** tspatzier has quit IRC | 11:00 | |
| *** ScottL1 has quit IRC | 11:02 | |
| *** ScottL has joined #heat | 11:04 | |
| *** tspatzier has joined #heat | 11:04 | |
| *** jistr has joined #heat | 11:10 | |
| *** jprovazn has joined #heat | 11:11 | |
| *** scollier has quit IRC | 11:13 | |
| *** david-lyle has quit IRC | 11:17 | |
| *** scollier has joined #heat | 11:21 | |
| *** nkhare_ has quit IRC | 11:24 | |
| *** coolsvap_away has joined #heat | 11:29 | |
| *** coolsvap has quit IRC | 11:30 | |
| *** wchrisj has joined #heat | 11:38 | |
| *** tspatzier has quit IRC | 11:39 | |
| *** Invent has joined #heat | 11:41 | |
| *** coolsvap_away is now known as coolsvap | 11:46 | |
| *** Invent has quit IRC | 11:46 | |
| *** Invent has joined #heat | 12:10 | |
| *** asalkeld has joined #heat | 12:17 | |
| *** alexpilotti has joined #heat | 12:18 | |
| *** Invent has quit IRC | 12:23 | |
| *** alexheneveld has quit IRC | 12:24 | |
| openstackgerrit | Dmitry Shulyak proposed a change to openstack/heat: Transition to alembic migration with oslo migration tool https://review.openstack.org/63767 | 12:26 |
| *** e0ne has joined #heat | 12:29 | |
| *** tomek_adamczewsk has joined #heat | 12:32 | |
| *** coolsvap has quit IRC | 12:34 | |
| *** coolsvap has joined #heat | 12:34 | |
| *** rpothier has quit IRC | 12:35 | |
| *** tspatzier has joined #heat | 12:41 | |
| *** jcru has joined #heat | 12:44 | |
| *** asalkeld has quit IRC | 12:44 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (1/9) https://review.openstack.org/71511 | 12:46 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (2/9) https://review.openstack.org/71512 | 12:46 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (3/9) https://review.openstack.org/71513 | 12:47 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (4/9) https://review.openstack.org/71514 | 12:47 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (5/9) https://review.openstack.org/71515 | 12:48 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (6/9) https://review.openstack.org/71516 | 12:48 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (7/9) https://review.openstack.org/71517 | 12:48 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Order imports in alphabetical order (8/9) https://review.openstack.org/71518 | 12:48 |
| pshchelo | ^^^ flood | 12:49 |
| *** tspatzier has quit IRC | 12:49 | |
| *** tomek_adamczewsk has quit IRC | 12:50 | |
| *** sgordon has joined #heat | 12:54 | |
| *** coolsvap has quit IRC | 12:55 | |
| *** raushan has joined #heat | 12:56 | |
| *** anand has joined #heat | 12:57 | |
| raushan | when i use heat in openstack and create a tamplet i got a error msg: Error: Resource create failed: Error: Server wordpress-DatabaseTemplate-fc3lzgb2kukm-MySqlDatabaseServer-7n5dr5wj727t length 72 > 63 characters, please reduce the length of stack or resource names | 12:59 |
| raushan | any one can solve this issue | 12:59 |
| therve | raushan, rename your resources | 13:00 |
| therve | eg MySqlDatabaseServer -> mydb | 13:00 |
| raushan | k | 13:00 |
| skraynev | pshchelo: you are crazy man!!! 9 parts.... Aaaa... | 13:04 |
| pshchelo | skraynev: the last one's gonna be when all of the above merge | 13:05 |
| pshchelo | then I'll actually change the tox.ini | 13:05 |
| pshchelo | and anything else that will break this check | 13:06 |
| skraynev | pshchelo: lol)) I hope it will happen soon))) | 13:06 |
| pshchelo | me too | 13:06 |
| skraynev | pshchelo: could you upload last and mark it WIP? | 13:06 |
| pshchelo | it either nine parts on one patch with ~170 files. the latter one as I see it no chances for review at all | 13:07 |
| *** aweiteka has quit IRC | 13:07 | |
| skraynev | pshchelo: ok. I only want get way to test it. | 13:07 |
| pshchelo | good idea actually | 13:07 |
| pshchelo | no, you can't | 13:08 |
| pshchelo | that's the problem. | 13:08 |
| skraynev | pshchelo: hm.. it's big problem | 13:08 |
| pshchelo | If you change tox.ini, anythingh else would break | 13:08 |
| pshchelo | kind of a check is that all other tests are running fine. that means I haven't removed any import by accident ^) | 13:09 |
| skraynev | pshchelo: How about other strategy for making it available for testing? | 13:09 |
| skraynev | pshchelo: bad news... | 13:10 |
| skraynev | pshchelo: May be you create sequence of depending patches? | 13:10 |
| skraynev | And last will be contain tox changes? | 13:11 |
| skraynev | or is it impossible too? | 13:11 |
| pshchelo | the last -yes, that's where tox.ini gets changed | 13:11 |
| pshchelo | along with anything else which is introduced while this behemoth is in review | 13:12 |
| pshchelo | so the last one will actually enable this check and then it's probably merge race :) | 13:13 |
| pshchelo | for testing single files all i can think of is some sort of bash script that will take the files changed from git, run pep8 on them and grep that no these files appear in the output | 13:13 |
| pshchelo | s/single files/single patches/ | 13:14 |
| skraynev | pshchelo: hm... I offer to ask opinion of core team's guys. | 13:15 |
| therve | pshchelo, fwiw I don't think it's the best time in the release cycle do to this kind of stuff | 13:15 |
| therve | It doesn't have any user facing benefits, it takes CI time and reviewer time... | 13:15 |
| pshchelo | therve: well, I can nuke it of course, but the bp is up icehouse-3 | 13:16 |
| therve | Moving the blueprint is not really a problem | 13:17 |
| therve | As it will happen on several others | 13:17 |
| *** scollier has quit IRC | 13:18 | |
| skraynev | "therve: Moving the blueprint is not really a problem" --- he-he))) | 13:18 |
| pshchelo | therve: so you suggest moving it to next (j-1)? | 13:19 |
| *** pafuent has joined #heat | 13:19 | |
| therve | I'm just sharing my feelings for now. | 13:20 |
| skraynev | therve: what do you think about testing approach? | 13:20 |
| skraynev | for these patches | 13:20 |
| therve | I don't know what that means | 13:21 |
| skraynev | These changes are needed for one of pep8 rule. Unfortunately they have not dependencies between each other. So each patch we should test carefully using pep8 only for files in patch, because cann't start tox test at all. | 13:25 |
| skraynev | I suggest make it in one sequence with tox changes in the last patch. | 13:26 |
| *** topol has joined #heat | 13:27 | |
| *** cmyster has quit IRC | 13:28 | |
| therve | skraynev, Isn't that exactly what's done here? | 13:29 |
| therve | I don't see any other way to do it | 13:30 |
| skraynev | yes. But all patches are independent now | 13:31 |
| *** topol has quit IRC | 13:31 | |
| *** tspatzier has joined #heat | 13:32 | |
| *** blomquisg has joined #heat | 13:33 | |
| skraynev | and it makes testing each of them boring and more difficult then downloading all tree of patches and testing it whole. | 13:34 |
| *** rpothier has joined #heat | 13:37 | |
| shardy | skraynev: I agree in this case a chain of dependent patches probably makes sense, but you could also just cherry-pick all the patches into a local tree for testing | 13:39 |
| shardy | therve: I also agree that gigantic cosmetic refactoring isn't a really great use of reviewer or gate resources atm | 13:39 |
| shardy | but as asalkeld said in response to my similar comment on the ML "people will do what they are interested in" | 13:40 |
| skraynev | shardy: yeap. It makes sense, but it takes more time ;) | 13:40 |
| shardy | so whatever I guess | 13:40 |
| therve | shardy, I'm not forced to review it though :) (yet) | 13:41 |
| *** aweiteka has joined #heat | 13:41 | |
| shardy | therve: well I did review a lot of the last batch of cosmetic test polishing and it took a significant amount of time | 13:41 |
| shardy | so your concern is valid IMO | 13:41 |
| shardy | but pshchelo thanks for your efforts nonetheless :) | 13:42 |
| skraynev | shardy: do you mean moving to j-1? | 13:42 |
| *** Tross has quit IRC | 13:42 | |
| shardy | I have no opinion on that, it's up to stevebaker | 13:42 |
| shardy | I see it as low priority, but lets see if folks have time to review it | 13:43 |
| raushan | #heat | 13:43 |
| *** raushan has left #heat | 13:44 | |
| skraynev | shardy: ok | 13:45 |
| *** jdob has joined #heat | 13:49 | |
| *** e0ne_ has joined #heat | 13:51 | |
| *** ktbenton has quit IRC | 13:52 | |
| *** arbylee has joined #heat | 13:53 | |
| *** e0ne has quit IRC | 13:54 | |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack/python-heatclient: Add HTTP proxy support to heat client https://review.openstack.org/67153 | 13:59 |
| *** rbrady has joined #heat | 14:03 | |
| *** rbrady is now known as rbrady-brb | 14:07 | |
| openstackgerrit | A change was merged to openstack/heat: Verify setenforce is executable https://review.openstack.org/71019 | 14:08 |
| openstackgerrit | A change was merged to openstack/heat: Ensure stack lock is released after stack-delete https://review.openstack.org/71390 | 14:08 |
| openstackgerrit | A change was merged to openstack/heat: A cloud-config resource based on SoftwareConfig https://review.openstack.org/63214 | 14:08 |
| *** edmund has joined #heat | 14:12 | |
| *** alexheneveld has joined #heat | 14:12 | |
| *** anand has quit IRC | 14:13 | |
| sdake | morning | 14:17 |
| *** rbrady-brb is now known as rbrady | 14:17 | |
| arbylee | shardy: regarding https://review.openstack.org/#/c/66485/, I'm not sure what I'm missing but I see the changes in the patch are already applied onto https://review.openstack.org/#q,bb754dc,n,z | 14:18 |
| arbylee | https://review.openstack.org/#q,bb754dc,n,z * | 14:19 |
| arbylee | grrr | 14:19 |
| shardy | arbylee: Oh, sorry, that's my mistake, I missed the dependency | 14:20 |
| arbylee | shardy: ah ok. no worries | 14:21 |
| openstackgerrit | A change was merged to openstack/heat: Fix event_type names for stack CRUD notifications https://review.openstack.org/69372 | 14:24 |
| openstackgerrit | A change was merged to openstack/heat: Create heat database with default character set utf-8 https://review.openstack.org/71437 | 14:25 |
| openstackgerrit | A change was merged to openstack/heat: test_signal add missing VerifyAll calls https://review.openstack.org/70717 | 14:25 |
| *** andersonvom has joined #heat | 14:25 | |
| shardy | arbylee: approved | 14:25 |
| *** jrist has joined #heat | 14:31 | |
| *** edmund has quit IRC | 14:32 | |
| *** wchrisj has quit IRC | 14:32 | |
| openstackgerrit | Bartosz Górski proposed a change to openstack/heat: Adding option in config to specify region name https://review.openstack.org/71540 | 14:33 |
| *** wchrisj has joined #heat | 14:33 | |
| *** jasond has quit IRC | 14:33 | |
| *** mkollaro has joined #heat | 14:34 | |
| *** nkhare has joined #heat | 14:34 | |
| *** Invent has joined #heat | 14:34 | |
| sdake | hey shardy would you expect event-list to sort by timestamp? | 14:34 |
| *** edmund has joined #heat | 14:35 | |
| *** bgorski has quit IRC | 14:36 | |
| *** achampion has joined #heat | 14:37 | |
| arbylee | shardy: thanks! | 14:37 |
| *** dims has quit IRC | 14:38 | |
| *** wchrisj has quit IRC | 14:40 | |
| *** dims has joined #heat | 14:44 | |
| *** sgran has joined #heat | 14:44 | |
| sgran | hello | 14:44 |
| sgran | I think I'm doing something wrong, but I thought I'd drop in and ask | 14:45 |
| sgran | I've created a stack, and then gone back and added a security group rule to an existing security group | 14:45 |
| sgran | (aws secgroup) | 14:45 |
| sgran | heat seems to be trying to delete the existing group and recreate it, which of course fails since that group is in use | 14:45 |
| *** jasond has joined #heat | 14:46 | |
| sdake | how did you add the security groupe rule to the existing security gruop? | 14:47 |
| sdake | via nova cli? | 14:47 |
| sgran | by editing the json and running heat stack-update -f foo.json | 14:47 |
| sdake | i would expect delete to work in that case | 14:47 |
| sgran | the security group is in use by the neutron ports attached to the instances | 14:48 |
| *** alexheneveld has quit IRC | 14:48 | |
| *** SnowDust has joined #heat | 14:48 | |
| sgran | so you can't delete it, but you can update it | 14:48 |
| sgran | it just seems that heat is trying to delete the secgroup rather than update it | 14:48 |
| sdake | sounds like a heat bug related to neutron integration | 14:48 |
| *** alexheneveld has joined #heat | 14:48 | |
| sgran | it's also via the ec2 compatible resources | 14:49 |
| sgran | so yeah, maybe | 14:49 |
| sdake | ec2 compatible resources will use neutron if its setup | 14:49 |
| sgran | ok, good | 14:49 |
| sdake | nova networking and neutron have different behaviors | 14:49 |
| therve | sgran, AWS::EC2::SecurityGroup doesn't implement update | 14:49 |
| SnowDust | sdake can you look at this paste : Paste #62811 | 14:49 |
| sgran | oh, yes, so they do - the exception is a neutron exception | 14:49 |
| sdake | I think delete is fine for nova network | 14:49 |
| sgran | therve: ah, that would be it | 14:49 |
| therve | So it will delete/recreate when you try to do one | 14:50 |
| SnowDust | its giving me error for greenlet .. did a devstack install | 14:50 |
| sdake | snowdust can you post the link? | 14:50 |
| sgran | ok, that's a pain | 14:50 |
| SnowDust | http://paste.openstack.org/show/62811/ | 14:50 |
| sgran | can I attach neutron security groups to an aws autoscaling resource? | 14:50 |
| sgran | (if the native neutron version supports update) | 14:50 |
| sdake | snowdust is that the only data for the error? | 14:51 |
| SnowDust | what i can give as background that greenlet is not getting pip installed | 14:51 |
| SnowDust | for python2.7 | 14:51 |
| SnowDust | trying on ubuntu precise | 14:51 |
| sdake | sounds like a devstack problem to me | 14:51 |
| *** bgorski has joined #heat | 14:52 | |
| SnowDust | for all the api s | 14:53 |
| SnowDust | its giving the same error | 14:53 |
| SnowDust | i can understand .. the requirement.txt is same for all of them | 14:53 |
| therve | sgran, neutron sg do support update. I don't know about using them with autoscaling. | 14:53 |
| sgran | I'll give it a try and see what happens | 14:54 |
| *** blamar has joined #heat | 14:55 | |
| shadower | sdake: so regarding this: https://bugs.launchpad.net/heat/+bug/1257410 | 14:57 |
| *** cody-somerville has quit IRC | 14:57 | |
| uvirtbot | Launchpad bug 1257410 in heat "Heat disables selinux by default making selinux enabled instances nonfunctional" [High,Confirmed] | 14:57 |
| shadower | sdake: why do we even disable selinux in the first place? | 14:57 |
| SnowDust | sdake : will u be a more adventurous helping me :) | 14:57 |
| SnowDust | i need it badly | 14:58 |
| SnowDust | can u just spawn a devstack | 14:58 |
| SnowDust | with heat standalone | 14:58 |
| sgran | sigh, my heat is too old | 14:58 |
| shardy | shadower: IIRC because everything cloud-init related breaks in enforcing mode (or did at least) | 14:59 |
| *** bgorski has quit IRC | 15:00 | |
| shardy | shadower: The template author could easily setenforce 1 at the end of their user-data so it seems like a reasonable default for development to avoid endless non-obvious template failures | 15:00 |
| shardy | shadower: but maybe we should make disabling it optional in the heat.conf | 15:00 |
| shadower | shardy: right, thanks | 15:03 |
| *** bgorski has joined #heat | 15:03 | |
| *** SnowDust has quit IRC | 15:08 | |
| *** mestery has joined #heat | 15:09 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Parse stack_adopt_data https://review.openstack.org/71553 | 15:09 |
| *** bgorski has quit IRC | 15:11 | |
| *** jamieh has quit IRC | 15:11 | |
| *** jamieh has joined #heat | 15:14 | |
| *** matsuhas_ has quit IRC | 15:17 | |
| *** vijendar has joined #heat | 15:17 | |
| *** tims has joined #heat | 15:18 | |
| *** rcleere has joined #heat | 15:18 | |
| *** mestery has quit IRC | 15:20 | |
| *** nosnos has quit IRC | 15:21 | |
| *** mestery has joined #heat | 15:21 | |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Restructure Rackspace resources https://review.openstack.org/68747 | 15:22 |
| *** bgorski has joined #heat | 15:24 | |
| tims | ping tspatzier | 15:25 |
| *** sergmelikyan has joined #heat | 15:25 | |
| tspatzier | Hi tims | 15:26 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Reorganize docker-plugin for consistency https://review.openstack.org/68748 | 15:26 |
| tims | hey, just making sure I'm not misunderstanding something | 15:26 |
| tims | https://review.openstack.org/#/c/61601/ | 15:26 |
| tims | are you referring to the Parameter Groups being in the hot spec? | 15:26 |
| tspatzier | Let me check ... | 15:27 |
| tspatzier | right. since a new construct is being added to the template language, I think it should be described in the spec | 15:28 |
| tims | ok, I added it to the spec in a previous patch | 15:28 |
| tims | unless you think there are changes that I need to make | 15:28 |
| tims | https://github.com/openstack/heat/blob/master/doc/source/template_guide/hot_spec.rst#parameter-groups-section | 15:28 |
| tspatzier | ah, I must have missed it. Is it already merged | 15:29 |
| tims | oh ok, yeah it merged a while back | 15:29 |
| tims | this patch has been lingering | 15:29 |
| tspatzier | oh, sorry for missing this. then I'll revert my -1 in a second | 15:30 |
| tims | ok thanks | 15:30 |
| jasond | arbylee: i can't tell if something changed in your "Restructure Rackspace resources" patch | 15:30 |
| arbylee | jasond nothing changed, we're splitting out non dependent patches | 15:30 |
| *** bgorski has quit IRC | 15:31 | |
| tspatzier | tims: I fixed my score. sorry again. | 15:31 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Separate the Marconi client from the resource https://review.openstack.org/71350 | 15:31 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Rename Marconi plugin package https://review.openstack.org/71141 | 15:31 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Restructure Marconi plugin directory structure https://review.openstack.org/71351 | 15:31 |
| tims | tspatzier: no worries, thanks | 15:31 |
| sdake | shadower cloudinit has selinux support built in | 15:32 |
| *** spzala has joined #heat | 15:33 | |
| jasond | arbylee: you may want to note that so people don't have to scan for changes. it's hard to tell. | 15:33 |
| sdake | shadower basically heat does a useradd operation in boothook.sh | 15:33 |
| arbylee | jasond: good point. we'll do that | 15:33 |
| sdake | useradd also has selinux support | 15:33 |
| jasond | arbylee: thanks | 15:33 |
| sdake | but if the ec2-user isn't added with the right context cloud-init wont work | 15:33 |
| *** bgorski has joined #heat | 15:34 | |
| shardy | sdake: That was broken on the older version of cloud-init which we (still?) support tho right? | 15:34 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make Server compatible with Nova Key Pair resource https://review.openstack.org/68230 | 15:34 |
| sdake | i wrote a patch some time back that removed the setenforce 0 and changed the useradd -m with the proper selinux credentials | 15:34 |
| sdake | shardy iirc we are up to 0.71 of cloudinit on all fedora/rhel/centos | 15:34 |
| sdake | ubuntu12.04 is on 0.69 (which may or may not have the selinux support in the cloudconfig file) | 15:35 |
| sdake | we add a user in cloudconfig and also add in boothook.sh | 15:35 |
| shardy | sdake: >= 0.7.2 IIRC but we were blocked because of ubuntu LTS still shipping 0.6xx | 15:35 |
| sdake | it only needs to be added once, and in that case the user needs to be added with selinux creds | 15:35 |
| *** david-lyle has joined #heat | 15:36 | |
| sdake | shardy yup but not sure what the version has to do with adding the user with selinux creds | 15:36 |
| *** Invent has quit IRC | 15:36 | |
| sdake | shardy it may be that lts doesn't support adding a user with creds, which is fine, we can just take that out of cloudconfig and do it in boothook.sh as we do now | 15:36 |
| *** Invent has joined #heat | 15:36 | |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Rename rackspace.rst to contrib.rst https://review.openstack.org/68752 | 15:37 |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/71345 | 15:38 |
| *** david_lyle_ has joined #heat | 15:39 | |
| radix | morning heaters | 15:40 |
| radix | therve: thanks for the reviews | 15:40 |
| *** ifarkas_ has joined #heat | 15:40 | |
| *** ifarkas__ has joined #heat | 15:41 | |
| *** david-lyle has quit IRC | 15:43 | |
| *** ifarkas has quit IRC | 15:43 | |
| *** ifarkas has joined #heat | 15:43 | |
| *** mestery has quit IRC | 15:47 | |
| radix | therve: honestly I didn't want verifying / number of calls tested at all, the assertions made in the tests are much clearer than a mocking error, especially since they're (almost) pure function calls | 15:47 |
| radix | but yeah I'm looking at mock | 15:47 |
| *** aignatov is now known as aignatov_ | 15:48 | |
| *** jistr has quit IRC | 15:48 | |
| *** bgorski has quit IRC | 15:48 | |
| *** ifarkas has quit IRC | 15:49 | |
| *** ifarkas__ has quit IRC | 15:49 | |
| shadower | sdake: can I see the patch? What are the correct selinux creds? | 15:50 |
| sdake | shadower I looked for it some time ago | 15:50 |
| sdake | and seeemed to have lost it unfortunately | 15:50 |
| sdake | I dont know what the correct selinux creds are | 15:50 |
| shadower | ok | 15:50 |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack/python-heatclient: Add HTTP proxy support to heat client https://review.openstack.org/67153 | 15:51 |
| shadower | sdake: so the ideal fix would be to remove setenforce 0 from boothook.sh, remove the useradd and sudoers commands, move all that to confic using cloudinit's native syntax and verify it works on ubuntu lts? | 15:51 |
| sdake | that wouold be ideal,but I doubt selinux syntax works on cloudinit 0.69 | 15:52 |
| sdake | but if it can work, that would be the way to solve it | 15:52 |
| shadower | sdake: right. And if it doesn't we should at least document it in bootlog.sh | 15:52 |
| sdake | if it doesn't work, then the best solution is to do it all in boothook.sh | 15:53 |
| sdake | and remove the user addition from cloudconfig | 15:53 |
| shadower | okay | 15:53 |
| sdake | while also remvoing the setenforce thing | 15:53 |
| shadower | yeah | 15:54 |
| *** jistr has joined #heat | 15:54 | |
| sdake | dropping off irc for 1 hour, i'll be back | 15:54 |
| shadower | I'll update the bug description and get on it (it wasn't all clear to me) | 15:54 |
| *** shakayumi has joined #heat | 15:54 | |
| sdake | sweet thanks shadower :) | 15:54 |
| *** bgorski has joined #heat | 15:55 | |
| therve | radix, That's fine, but It's better to call VerifyAll when you create the test, it's harder to spot if we change it later while caring about the calls | 15:57 |
| radix | well, switching to mock is going to get rid of that VerifyAll behavior anyway :P | 15:58 |
| radix | anyway, I'm doing that | 15:58 |
| therve | Sweet :) | 15:59 |
| openstackgerrit | Christopher Armstrong proposed a change to openstack/heat: Add unit tests for heat.scaling.template https://review.openstack.org/71399 | 16:01 |
| radix | it's pretty annoying that git review gives me that scary warning about pushing multiple reviews when it's not actually going to push multiple reviews | 16:01 |
| *** che-arne has joined #heat | 16:01 | |
| radix | HeatTestCase should probably just mock out short_id.generate_id globally | 16:03 |
| radix | man, is the zuul check queue really 87 hours long? | 16:03 |
| radix | I notice that the oldest stuff is all tripleo stuff with unfamiliar test suites, maybe they are deprioritized or something | 16:04 |
| shardy | radix: some stuff I approved this morning "only" took 4 hours to merge | 16:05 |
| radix | yeah the gate queue actually looks unusually tiny... | 16:06 |
| radix | but check queue is huge | 16:06 |
| therve | Yeah that's strange | 16:07 |
| therve | It took about 1h on a client change though | 16:08 |
| therve | Yeah same on a heat patch | 16:09 |
| radix | ok hooray, I'll just ignore the status page ;) | 16:09 |
| *** bgorski_ has joined #heat | 16:10 | |
| *** randallburt has joined #heat | 16:10 | |
| andersonvom | shardy: we added a comment re the doc + resource loading on the issue https://review.openstack.org/#/c/68746/ | 16:11 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Add personality files property to Server resource https://review.openstack.org/68685 | 16:11 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Native Nova Server compatibility for Cloud Server https://review.openstack.org/58043 | 16:11 |
| *** bgorski has quit IRC | 16:12 | |
| *** achudnovets has quit IRC | 16:14 | |
| *** shakayumi has quit IRC | 16:14 | |
| aliljet | heat is quite cool! can you modify the default user that's used by modifying the param instance_user under a server's properties? | 16:16 |
| *** akuznetsov has quit IRC | 16:16 | |
| *** nkhare has quit IRC | 16:17 | |
| *** mestery has joined #heat | 16:18 | |
| jasond | aliljet: here is a list of all Server properties http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Nova::Server | 16:18 |
| aliljet | woot, thanks jasond :) | 16:18 |
| jasond | np | 16:18 |
| aliljet | that was exactly what I was looking for | 16:18 |
| *** dklyle has joined #heat | 16:19 | |
| *** randallburt has quit IRC | 16:19 | |
| *** randallburt has joined #heat | 16:20 | |
| *** sabeen has joined #heat | 16:21 | |
| *** david_lyle_ has quit IRC | 16:22 | |
| *** jprovazn is now known as jprovazn_afk | 16:23 | |
| openstackgerrit | Randall Burt proposed a change to openstack/heat: Rackspace Cloud Networks resource https://review.openstack.org/68790 | 16:27 |
| *** wchrisj has joined #heat | 16:28 | |
| *** akuznetsov has joined #heat | 16:29 | |
| *** scollier has joined #heat | 16:35 | |
| *** dklyle is now known as david-lyle | 16:37 | |
| *** pablosan has quit IRC | 16:41 | |
| *** pablosan has joined #heat | 16:41 | |
| *** sdake has quit IRC | 16:41 | |
| *** mestery has quit IRC | 16:44 | |
| *** bvandenh has quit IRC | 16:44 | |
| andersonvom | shardy: also, on this patch as well: https://review.openstack.org/#/c/65576/ ;) | 16:46 |
| *** pshchelo has quit IRC | 16:46 | |
| *** jistr has quit IRC | 16:49 | |
| *** vijendar has quit IRC | 16:51 | |
| *** lsmola has quit IRC | 16:53 | |
| *** ifarkas_ has quit IRC | 16:55 | |
| *** julienvey has quit IRC | 16:56 | |
| *** mestery has joined #heat | 16:56 | |
| *** akuznetsov has quit IRC | 17:02 | |
| *** TravT has joined #heat | 17:04 | |
| *** e0ne_ has quit IRC | 17:05 | |
| *** e0ne has joined #heat | 17:05 | |
| *** lcheng_ has joined #heat | 17:06 | |
| *** pvaneck has joined #heat | 17:07 | |
| *** e0ne has quit IRC | 17:08 | |
| *** sdake has joined #heat | 17:09 | |
| *** sdake has quit IRC | 17:09 | |
| *** sdake has joined #heat | 17:09 | |
| *** scollier has quit IRC | 17:22 | |
| *** WinnieTsang has quit IRC | 17:23 | |
| *** alexheneveld has quit IRC | 17:39 | |
| openstackgerrit | Daneyon Hansen proposed a change to openstack/heat-templates: Added HOT Template for 2 Servers Without Floating IPs https://review.openstack.org/65552 | 17:40 |
| *** tango|2 has joined #heat | 17:40 | |
| *** vijendar has joined #heat | 17:42 | |
| *** akuznetsov has joined #heat | 17:42 | |
| *** Invent has quit IRC | 17:47 | |
| *** Invent has joined #heat | 17:47 | |
| *** WinnieTsang has joined #heat | 17:48 | |
| *** jamieh has quit IRC | 17:50 | |
| *** jamieh has joined #heat | 17:50 | |
| *** cadenzajon has joined #heat | 17:51 | |
| *** aignatov_ is now known as aignatov | 17:53 | |
| *** nati_ueno has joined #heat | 17:54 | |
| *** john-n-seattle has quit IRC | 17:55 | |
| *** john-n-seattle has joined #heat | 17:55 | |
| *** harlowja_away is now known as harlowja | 17:59 | |
| *** david_lyle_ has joined #heat | 18:00 | |
| *** derekh has quit IRC | 18:02 | |
| *** david-lyle has quit IRC | 18:02 | |
| *** lcheng_ has quit IRC | 18:19 | |
| *** IlyaE has joined #heat | 18:20 | |
| *** mestery has quit IRC | 18:20 | |
| *** david_lyle_ is now known as david_lyle | 18:23 | |
| *** tspatzier has quit IRC | 18:27 | |
| *** yogesh has joined #heat | 18:32 | |
| *** bvandenh has joined #heat | 18:34 | |
| *** akuznetsov has quit IRC | 18:37 | |
| *** bvandenh has quit IRC | 18:40 | |
| *** mestery has joined #heat | 18:41 | |
| *** e0ne has joined #heat | 18:43 | |
| *** wchrisj has quit IRC | 18:44 | |
| *** spzala has quit IRC | 18:44 | |
| *** e0ne has quit IRC | 18:45 | |
| *** gokrokve has joined #heat | 18:46 | |
| *** jprovazn_afk has quit IRC | 18:50 | |
| *** wchrisj has joined #heat | 18:53 | |
| *** lcheng_ has joined #heat | 18:55 | |
| *** tango|2 has quit IRC | 18:57 | |
| *** bgorski_ has quit IRC | 18:58 | |
| *** pvaneck has quit IRC | 18:58 | |
| *** pvaneck has joined #heat | 18:58 | |
| *** alexheneveld has joined #heat | 19:00 | |
| *** WinnieTsang has quit IRC | 19:01 | |
| *** WinnieTsang has joined #heat | 19:02 | |
| *** alexheneveld has quit IRC | 19:10 | |
| *** jamieh has quit IRC | 19:11 | |
| *** spzala has joined #heat | 19:14 | |
| *** e0ne has joined #heat | 19:14 | |
| *** mestery has quit IRC | 19:19 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add API support for stack preview https://review.openstack.org/68406 | 19:21 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add StackResource specific preview behavior https://review.openstack.org/68407 | 19:21 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for LoadBalancer https://review.openstack.org/68412 | 19:22 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add engine preview stack with Template and Params https://review.openstack.org/65576 | 19:22 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for NestedStack https://review.openstack.org/68408 | 19:22 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for ResourceGroup https://review.openstack.org/68409 | 19:22 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for InstanceGroup https://review.openstack.org/68410 | 19:22 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for TemplateResource https://review.openstack.org/68411 | 19:22 |
| stevebaker | morning! | 19:23 |
| *** alexheneveld has joined #heat | 19:24 | |
| andersonvom | morning! | 19:24 |
| *** achampion has quit IRC | 19:27 | |
| *** spzala has quit IRC | 19:28 | |
| *** achampion has joined #heat | 19:28 | |
| *** alexheneveld has quit IRC | 19:28 | |
| *** spzala has joined #heat | 19:29 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement HOT intrinsic function get_file https://review.openstack.org/66160 | 19:33 |
| *** gokrokve has quit IRC | 19:36 | |
| jasond | c | 19:37 |
| jasond | oops | 19:37 |
| *** Invent has quit IRC | 19:47 | |
| larsks | stevebaker: should comparisions against basestring be using six.string_types instead? | 19:48 |
| openstackgerrit | A change was merged to openstack/python-heatclient: Allow environment to not have resource_registry https://review.openstack.org/71379 | 19:49 |
| larsks | (I have seen both in different places in openstack code and I'm just curious re: heat in particular) | 19:49 |
| *** che-arne has quit IRC | 19:49 | |
| stevebaker | larsks: for heatclient, always. for heat somebody will need to convert all the existing basestring checks at some time, and I'm happy for that conversion to happen | 19:51 |
| larsks | stevebaker: Thanks, got it. | 19:51 |
| stevebaker | I wonder how far off heatclient is from working on python3 | 19:51 |
| *** aignatov is now known as aignatov_ | 19:56 | |
| *** lcheng_ has quit IRC | 20:03 | |
| *** mkollaro has quit IRC | 20:04 | |
| *** lcheng_ has joined #heat | 20:06 | |
| *** pvaneck has quit IRC | 20:07 | |
| *** Invent has joined #heat | 20:08 | |
| *** WinnieTsang has quit IRC | 20:08 | |
| *** pvaneck has joined #heat | 20:10 | |
| *** WinnieTsang has joined #heat | 20:10 | |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to HOT spec and parameter schema https://review.openstack.org/62418 | 20:10 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to template validate call https://review.openstack.org/65325 | 20:10 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to HOT spec and parameter schema https://review.openstack.org/62418 | 20:13 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to template validate call https://review.openstack.org/65325 | 20:13 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to template validate call https://review.openstack.org/65325 | 20:14 |
| *** mkollaro has joined #heat | 20:16 | |
| openstackgerrit | A change was merged to openstack/heat: Make context available during constraint validation https://review.openstack.org/66304 | 20:17 |
| *** TravT has quit IRC | 20:17 | |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to HOT spec and parameter schema https://review.openstack.org/62418 | 20:20 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to template validate call https://review.openstack.org/65325 | 20:20 |
| openstackgerrit | Andrew Plunk proposed a change to openstack/heat: raw_template hot parameter type data migration https://review.openstack.org/70050 | 20:27 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Handle API 500 response in OS::Nova::Server https://review.openstack.org/71644 | 20:30 |
| *** kfox1111 has joined #heat | 20:37 | |
| kfox1111 | how usable are the openshift heat templates these days? Anyone ever get them working? | 20:37 |
| *** cadenzajon has quit IRC | 20:40 | |
| stevebaker | kfox1111: I believe they are being actively developed, but I haven't been following them | 20:44 |
| kfox1111 | k. thanks. :) | 20:46 |
| *** scollier has joined #heat | 20:50 | |
| john-n-seattle | stupid question: are there any automated test environments/harnesses that leverage heat? thanks. | 20:53 |
| *** mattgriffin has quit IRC | 20:54 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Handle API limit exception in OS::Trove::Instance https://review.openstack.org/71082 | 20:54 |
| stevebaker | john-n-seattle: tempest has heat tests, under "orchestration" | 20:54 |
| larsks | john-n-seattle: do you mean something for testing heat? Or something that uses heat to build test environments for testing something else? | 20:55 |
| john-n-seattle | correct | 20:55 |
| john-n-seattle | (@larsks) | 20:55 |
| larsks | john-n-seattle: That was an either/or question :) | 20:55 |
| *** asalkeld has joined #heat | 20:55 | |
| john-n-seattle | sorry, i misread. the latter. | 20:56 |
| kfox1111 | john-n-seattle: I usually just fire up a few examples from heat-templates. Something more automated would be nice though... | 20:56 |
| larsks | That's why I thought. The tempest tests that stevebaker mentioned are for testing heat itself. I'm not aware of anything that uses heat right now to do what you want (but that would be pretty cool). | 20:56 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Handle API 500 response in OS::Nova::Server https://review.openstack.org/71644 | 20:57 |
| stevebaker | john-n-seattle: I see what you mean. heat would be ideal for describing large test scenarios. My hope is that more tempest scenarios will use heat over time. | 20:58 |
| *** andrew_plunk has joined #heat | 20:58 | |
| stevebaker | john-n-seattle: there has been talk of using heat for some neutron scenarios (possibly load balancing?) | 20:58 |
| *** TravT has joined #heat | 20:59 | |
| john-n-seattle | the use case i'm thinking about is automated testing. assign resources, images, boot them, execute tests. | 20:59 |
| *** asalkeld has quit IRC | 21:00 | |
| kfox1111 | john-n-seattle. We've really close to using jenkins with the jclouds plugin for that. | 21:00 |
| *** denis_makogon has quit IRC | 21:01 | |
| john-n-seattle | @kfox1111: interesting | 21:01 |
| kfox1111 | It has one bug in that if you are using neutron with multiple networks, it cant launch jobs since you can't specify a network id. but once that's done, jenkins should "Just work" (TM) :) | 21:01 |
| *** dmakogon_ has joined #heat | 21:01 | |
| *** slong has joined #heat | 21:01 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Do not log error on software config/deployment NotFound https://review.openstack.org/71653 | 21:02 |
| stevebaker | kfox1111: jclouds heat bindings? | 21:02 |
| kfox1111 | no, currently jclouds for launching clean build hosts. From there, you can do whatever you want though, including launching heat stacks to test the resultant artifacts. | 21:03 |
| kfox1111 | heat support in jclouds itself would be awesome though. | 21:04 |
| *** sgordon has quit IRC | 21:04 | |
| kfox1111 | the jenkins build slaves are simple enough to just be a single vm, so heat for that part of it wouldn't make much sense. | 21:05 |
| kfox1111 | but to thest the resultant artifacts at scale would be awesome to integrate heat. "Fire up these three heat stacks on the result and ensure all launch ok" | 21:05 |
| *** pafuent has left #heat | 21:07 | |
| *** mattgriffin has joined #heat | 21:09 | |
| *** slong_ has joined #heat | 21:10 | |
| *** mattgriffin has quit IRC | 21:11 | |
| *** asalkeld has joined #heat | 21:14 | |
| *** gokrokve has joined #heat | 21:14 | |
| *** slong has quit IRC | 21:14 | |
| *** blomquisg has quit IRC | 21:15 | |
| john-n-seattle | thanks | 21:16 |
| *** cadenzajon has joined #heat | 21:29 | |
| *** e0ne has quit IRC | 21:36 | |
| openstackgerrit | A change was merged to openstack/python-heatclient: Added missed files to create venv with run_tests.sh https://review.openstack.org/69945 | 21:37 |
| stevebaker | radix: you about? | 21:41 |
| radix | stevebaker: yo | 21:41 |
| stevebaker | radix: is there a reason why _resource_templates returns a list of tuples rather than a dict? | 21:42 |
| radix | =( | 21:42 |
| radix | because zaneb didn't like dicts :) | 21:42 |
| radix | stevebaker: oh, actually, maybe there's two different things... | 21:43 |
| zaneb | stevebaker: because they have to be in order iirc | 21:43 |
| radix | right | 21:43 |
| stevebaker | yes, that is a thing | 21:43 |
| radix | the order determines which ones are deleted upon shrinking | 21:43 |
| zaneb | and we're still supporting python 2.6 for some reason | 21:44 |
| zaneb | so no OrderedDict for us :( | 21:44 |
| stevebaker | zaneb: I'm about to +2 https://review.openstack.org/#/c/71143 so get in quick | 21:44 |
| zaneb | ok, I will look now | 21:44 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Handle API limit exception in OS::Nova::Server https://review.openstack.org/71660 | 21:46 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Handle API 500 response in OS::Nova::Server https://review.openstack.org/71644 | 21:46 |
| radix | there are also two small dependent patches after that one | 21:48 |
| *** ScottL is now known as Guest84184 | 21:48 | |
| openstackgerrit | A change was merged to openstack/python-heatclient: Enable default in bash completion https://review.openstack.org/71307 | 21:52 |
| openstackgerrit | A change was merged to openstack/python-heatclient: Enable hacking H233 rule https://review.openstack.org/68534 | 21:52 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to template validate call https://review.openstack.org/65325 | 21:53 |
| openstackgerrit | Tim Schnell proposed a change to openstack/heat: Adds Parameter Label to template validate call https://review.openstack.org/65325 | 21:56 |
| zaneb | radix: I added a question about where we truncate the list of old resources... don't we want that in the library function? | 21:58 |
| radix | yeah, good call, I'll move that in | 21:59 |
| radix | zaneb: as to the second point, I think the nested function is pointless, I'm not sure why it was done that way, I just didn't want to change the behavior | 22:00 |
| *** tims has quit IRC | 22:00 | |
| radix | returning the generator would also be fine, in the worst case the caller would have to list() if they want to iterate multiple times | 22:00 |
| zaneb | I think it's done that way because this is Python, not C ;P | 22:00 |
| radix | well, s/change the behavior/change the syntax too much, risking someone complaining about the patch size/ | 22:00 |
| zaneb | btw python 3 is all iterators all the time | 22:02 |
| radix | huh?? | 22:02 |
| zaneb | y'all had better get used to it ;) | 22:02 |
| radix | I don't even know what we're talking about now | 22:02 |
| zaneb | hi asalkeld :D | 22:02 |
| zaneb | radix: so this business of creating an empty dict or less, and then having a for-loop add elements to it one at a time... | 22:03 |
| zaneb | s/less/list/ | 22:03 |
| zaneb | that's not considered Pythonic | 22:04 |
| asalkeld | yawn | 22:04 |
| radix | oh god, Pythonic | 22:04 |
| *** shakayumi has joined #heat | 22:05 | |
| *** tango|3 has joined #heat | 22:05 | |
| *** wchrisj has quit IRC | 22:06 | |
| zaneb | iterators are easier to read, more efficient and can handle arbitrarily large data sets (since only one item is in memory at a time) | 22:07 |
| *** achampion has quit IRC | 22:07 | |
| radix | zaneb: so, now that I'm refactoring this function, do you think it would be ok to get rid of the nested function and just make _resource_templates a generator? | 22:08 |
| radix | or should I do that in a separate patch? | 22:08 |
| radix | or not do it at all? | 22:08 |
| *** IlyaE has quit IRC | 22:08 | |
| *** david_lyle has quit IRC | 22:09 | |
| zaneb | yeah, since you don't need to convert it to a dict inside that function any more, I think you could just get rid of the nested function and make the whole thing a generator | 22:10 |
| radix | in this patch? | 22:10 |
| zaneb | in the same patch would be fine, it's just changing the indent so the diff should be easy to read | 22:10 |
| radix | ok | 22:10 |
| *** rpothier has quit IRC | 22:11 | |
| * radix is going to discover the process of rebasing dependent patches now... | 22:11 | |
| *** shakayumi has quit IRC | 22:11 | |
| zaneb | the important thing when you're refactoring is to make it as easy as possible for reviewers to verify that you haven't accidentally changed any behaviour | 22:12 |
| zaneb | radix: http://procode.org/stgit/ | 22:12 |
| radix | zaneb: oh cool | 22:13 |
| *** IlyaE has joined #heat | 22:14 | |
| zaneb | that reminds me, I should submit my patches to make that work with the commit hook that adds in the Change-Id | 22:14 |
| zaneb | makes my life _so_ much easier | 22:14 |
| *** jcru has quit IRC | 22:15 | |
| stevebaker | asalkeld, zaneb fancy looking at the heatclient http requests port? https://review.openstack.org/#/c/65704/ | 22:16 |
| openstackgerrit | Randall Burt proposed a change to openstack/python-heatclient: Add more defalt CA paths https://review.openstack.org/68836 | 22:19 |
| zaneb | I'm more interested in that comment, where someone is trying to hack Gerrit | 22:20 |
| randallburt | whoops. should have fixed the commit comment first. | 22:20 |
| randallburt | mine, not Chmouel's that is. | 22:21 |
| clarkb | zaneb: ? | 22:21 |
| *** tims has joined #heat | 22:22 | |
| zaneb | clarkb: https://review.openstack.org/#/c/65704 about halfway down the list of comments | 22:22 |
| stevebaker | clarkb: feb 2 | 22:22 |
| clarkb | ah yes, we poked at that, apparently that was the only comment that was left and the injection seems to have failed | 22:23 |
| zaneb | yeah, it doesn't seem to have worked, so that's good | 22:23 |
| *** jdob has quit IRC | 22:23 | |
| clarkb | and fungi poked launchpad | 22:23 |
| *** lbragstad has joined #heat | 22:24 | |
| zaneb | clarkb: not much gets past you guys! I'm impressed :) | 22:24 |
| clarkb | we have such attentive users that keep us informed :) | 22:24 |
| clarkb | I think dims pointed that out to us | 22:25 |
| fungi | you are all part of our distributed global monitoring system ;) | 22:25 |
| fungi | and yeah, i did mention it in #launchpad but nobody responded. i gave them the openid url and e-mail address for the account, but i was unable to get the lp api to resolve that to an lp account page url | 22:26 |
| fungi | i think the launchpad devs and admins have fallen into an eternal slumber. the sooner we stop using it the better if you ask me | 22:26 |
| zaneb | +1 | 22:26 |
| zaneb | I never liked it | 22:27 |
| zaneb | (Red Hat's bugzilla is probably worse though) | 22:27 |
| fungi | well, openstack did drag away most of their dev talent, so we're sort of to blame ;) | 22:27 |
| zaneb | lol | 22:27 |
| zaneb | can we please get rid of the spurious distinction between bugs and blueprints while we're at it? | 22:28 |
| clarkb | zaneb: pretty sure that is happening | 22:28 |
| clarkb | everything is a 'story' in storyboard | 22:28 |
| zaneb | w00t | 22:29 |
| clarkb | http://storyboard.openstack.org/#!/ I have no idea how close to functional that is | 22:29 |
| clarkb | so don't get to attached or angry about it :) | 22:29 |
| zaneb | "It all begins with a story. A story is a bug report or proposed feature." yaaaaaay :)) | 22:29 |
| clarkb | the about page has decent stuff | 22:30 |
| clarkb | you found it :) | 22:30 |
| lbragstad | hey all, quick question on heat's use of the keystoneclient; just wanted to make sure if this is changed http://paste.openstack.org/show/62897/ (trust_create method signature in the keystone client) it won't break heat. I was going to submit a change so that keystoneclient is consistent with the documentation https://github.com/openstack/identity-api/blob/master/openstack-identity-api/v3/src/markdown/identity-api-v3-os-trust-ext.md#trusts | 22:30 |
| *** vijendar has quit IRC | 22:33 | |
| stevebaker | lbragstad: you should check with shardy when he is awake | 22:33 |
| zaneb | lbragstad: https://github.com/openstack/heat/blob/master/heat/common/heat_keystoneclient.py#L187 looks safe | 22:33 |
| lbragstad | stevebaker: sounds good, do you happen to know what time zone he is in? | 22:34 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Do not log error on software config/deployment NotFound https://review.openstack.org/71653 | 22:34 |
| stevebaker | lbragstad: UTC +0 | 22:34 |
| *** jasond has quit IRC | 22:34 | |
| lbragstad | stevebaker: thank you | 22:34 |
| openstackgerrit | Andrew Plunk proposed a change to openstack/heat: Fix default raw_template files value in migration https://review.openstack.org/71672 | 22:35 |
| zaneb | lbragstad: judging by that code I linked, he's already aware of the discrepancy and has taken care to work around it in such a way that things won't break if you change it | 22:35 |
| lbragstad | zaneb: cool, looks like he's using kwargs to get around it, I had to do the same thing in openstackclient | 22:36 |
| lbragstad | just wanted to double check before submitting something and not working with you guys if it was going to break you | 22:37 |
| lbragstad | s/you/heat | 22:39 |
| zaneb | lbragstad: thanks :) I think you're safe, go ahead | 22:39 |
| openstackgerrit | Christopher Armstrong proposed a change to openstack/heat: Refactor _create_template to not rely on instances https://review.openstack.org/71143 | 22:40 |
| *** asalkeld has quit IRC | 22:41 | |
| *** sabeen has quit IRC | 22:46 | |
| openstackgerrit | A change was merged to openstack/python-heatclient: Add pagination (limit and marker) to stack-list https://review.openstack.org/66484 | 22:48 |
| openstackgerrit | A change was merged to openstack/python-heatclient: Add filter option to stack-list https://review.openstack.org/66485 | 22:48 |
| *** jasond has joined #heat | 22:50 | |
| *** aweiteka has quit IRC | 22:51 | |
| radix | oh god that hurt | 22:54 |
| SpamapS | shardy: FYI, https://review.openstack.org/#/c/65576/5..6/heat/tests/test_engine_service.py <-- comments there regarding usine set_override in tests. I think we need to get that train back on the rails. | 22:54 |
| SpamapS | using | 22:54 |
| openstackgerrit | Andrew Plunk proposed a change to openstack/heat: raw_template hot parameter type data migration https://review.openstack.org/70050 | 22:55 |
| * SpamapS starts a --isolated run to prove the point. :-P | 22:55 | |
| SpamapS | so... slow.. :-P | 22:56 |
| harlowja | props to the zaneb :-P | 22:56 |
| harlowja | haha | 22:56 |
| *** mattgriffin has joined #heat | 22:57 | |
| zaneb | it wasn't me, I was never there, you can't prove anything | 22:57 |
| *** david-lyle has joined #heat | 22:59 | |
| * radix crosses his fingers... | 23:00 | |
| openstackgerrit | Christopher Armstrong proposed a change to openstack/heat: Move resource_templates to heat.scaling https://review.openstack.org/71168 | 23:00 |
| radix | whew | 23:01 |
| *** lbragstad has left #heat | 23:02 | |
| harlowja | zaneb, ha | 23:03 |
| *** tims has quit IRC | 23:04 | |
| zaneb | harlowja: but seriously, what are we talking about? ;) | 23:04 |
| harlowja | idk | 23:04 |
| harlowja | oh | 23:04 |
| harlowja | lol | 23:04 |
| harlowja | zaneb http://lists.openstack.org/pipermail/openstack-dev/2014-February/026588.html | 23:04 |
| harlowja | *that thread | 23:04 |
| harlowja | bb | 23:04 |
| openstackgerrit | Christopher Armstrong proposed a change to openstack/heat: Add unit tests for heat.scaling.template https://review.openstack.org/71399 | 23:04 |
| openstackgerrit | Andrew Plunk proposed a change to openstack/heat: Fix default raw_template files value in migration https://review.openstack.org/71672 | 23:04 |
| radix | alright, all up-to-date | 23:05 |
| radix | --skip is my friend | 23:05 |
| zaneb | harlowja: ah, thanks. I hadn't seen that. I should go have a read | 23:05 |
| SpamapS | zaneb: just more whining from me because my feeble mind has trouble with coroutines. ;) | 23:06 |
| zaneb | lol | 23:06 |
| SpamapS | nothing to see there. ;) | 23:06 |
| zaneb | SpamapS: imo that missing "yield from" syntax is the most confusing part | 23:09 |
| SpamapS | zaneb: Perhaps, I admit to not having looked into yield from | 23:10 |
| zaneb | SpamapS: it's really hard to explain to folks when you need the decorator and what it does | 23:10 |
| *** Invent has quit IRC | 23:15 | |
| SpamapS | zaneb: I think I understand it now actually. I'm just not _SURE_ I understand. ;) | 23:17 |
| *** asalkeld has joined #heat | 23:19 | |
| *** asalkeld has quit IRC | 23:19 | |
| *** asalkeld has joined #heat | 23:20 | |
| *** rcleere has quit IRC | 23:24 | |
| *** dims has quit IRC | 23:30 | |
| zaneb | SpamapS: http://lwn.net/Articles/544522/ ' It is "an incredibly cool, but also brain-exploding thing"' - GvR on yield from :D | 23:33 |
| zaneb | 'A yield from and a Future just kind of cancel out and the value is what would be returned from the equivalent blocking function. That is the "best I can say it without bursting into tears", he said.' | 23:34 |
| *** achampion has joined #heat | 23:34 | |
| *** yogesh has quit IRC | 23:36 | |
| clarkb | yield from is a py3k thing right? | 23:37 |
| clarkb | so in about 10 years everything will be better | 23:37 |
| *** andrew_plunk has quit IRC | 23:38 | |
| SpamapS | clarkb: you're assuming coroutines are better | 23:39 |
| *** asalkeld has quit IRC | 23:39 | |
| clarkb | SpamapS: they make a lot of code much easier to read | 23:39 |
| SpamapS | I don't agree with the go fans on everything. But thinking in linear terms the way go does is, IMO, better. | 23:39 |
| zaneb | clarkb: correct | 23:39 |
| clarkb | assuming you get the special sugar for it | 23:39 |
| zaneb | SpamapS: I'm not familiar with Go, but I am familiar with eventlet and it's pure evil | 23:40 |
| *** lcheng_ has quit IRC | 23:42 | |
| *** scollier has quit IRC | 23:42 | |
| SpamapS | zaneb: yeah. So the go way is to have syntax sugar for lightweight threading called goroutines, and IPC via channels. | 23:44 |
| *** andersonvom has quit IRC | 23:44 | |
| SpamapS | zaneb: so you end up writing code that looks more like forward work-flow, and sync is handled by the go runtime. | 23:45 |
| zaneb | how do they make goroutines thread-safe? | 23:45 |
| SpamapS | zaneb: As somebody was just poking out onto openstack-dev ... python threads would be fine and achieve the same as eventlet, we'd just wonder about the GIL instead of knowing we don't have it. ;) | 23:46 |
| *** dims has joined #heat | 23:46 | |
| SpamapS | zaneb: goroutines can only communicate outside of goroutines through channels, so everything is thread-local. | 23:46 |
| zaneb | e.g. tulip makes it threadsafe by making the yield points explicit | 23:46 |
| zaneb | and eventlet makes it threadsafe by HEY, LOOK OVER THERE | 23:47 |
| stevebaker | yep | 23:47 |
| SpamapS | zaneb: hidden from the user, often goroutines are just one OS thread using poll to coordinate things between the running threads. | 23:47 |
| SpamapS | zaneb: the point is "think about concurrency, not parallelism" | 23:47 |
| SpamapS | which I know is also how a pure coroutine model wants you to think too | 23:48 |
| zaneb | SpamapS: the point is to know when data can change from under you and when it can't, surely? | 23:48 |
| SpamapS | zaneb: data can't change from under you. Changes will arrive via a channel. | 23:49 |
| zaneb | even changes to global data? | 23:50 |
| SpamapS | as I said, goroutines have no access to global data | 23:50 |
| SpamapS | they receive a copy via the input channel | 23:50 |
| SpamapS | and then they spit out their results on the output channel | 23:50 |
| zaneb | ah, ok, they're pure functions | 23:50 |
| radix | I don't think pure is the right word | 23:51 |
| SpamapS | http://golangtutorials.blogspot.com/2011/06/channels-in-go.html | 23:51 |
| zaneb | radix: I think you're right | 23:51 |
| radix | they still have side-effects, just not directly mutating external state | 23:51 |
| *** arbylee has quit IRC | 23:51 | |
| *** asalkeld has joined #heat | 23:52 | |
| SpamapS | right, in the example, the goroutine mutates a global variable, but IIRC, there are compile time checks for this or something.. maybe I've gotten confused again. | 23:53 |
| radix | well, I guess mutating global state is at least discouraged :) | 23:55 |
| * zaneb -> food | 23:55 | |
| SpamapS | ok now I'm remembering.. | 23:56 |
| SpamapS | it is not that you can't reach out to global state | 23:56 |
| SpamapS | it is that channels are the only sync mechanism. | 23:56 |
| radix | ah | 23:57 |
| SpamapS | which keeps things fairly simple | 23:58 |
| SpamapS | there's no "oops I have to lock that" | 23:58 |
| radix | so yeah | 23:58 |
| *** kfox1111 has quit IRC | 23:59 | |
| radix | in Python you can of course use threads and Queues, which is a great way to do it | 23:59 |
| radix | but a ton of Python code still does other random state sharing, which sucks | 23:59 |
| *** che-arne has joined #heat | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!