15:00:57 #startmeeting heat 15:01:01 Meeting started Wed May 24 15:00:57 2017 UTC and is due to finish in 60 minutes. The chair is ricolin. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:01:02 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:01:04 The meeting name has been set to 'heat' 15:01:08 #topic roll call 15:01:53 Hi 15:01:55 greetings 15:02:15 hi 15:02:24 therve, zaneb ramishra hi guys:) 15:03:33 #topic adding items to agenda 15:03:39 #link https://wiki.openstack.org/wiki/Meetings/HeatAgenda#Agenda_.282017-05-24_1500_UTC.29 15:05:02 #topic weekly report 15:05:15 only got six new medium bug this week 15:06:14 also it's closer to feature freeze so would be great if any help on reviewing specs 15:06:14 #link https://review.openstack.org/#/q/status:open+project:openstack/heat-specs 15:07:23 ricolin: Is there a patch in queue to implement that? Approving the spec would probably be of very little use. 15:08:43 can we approve https://review.openstack.org/#/c/267770/ now? the patch is close to landing 15:08:48 ramishra, yeah, agree with you. But do wondering any of those guys try to wait for spec approved before take action 15:09:25 ramishra, maybe we should leave them message to encourage doing implement and spec at some time 15:11:15 #topic seperators encoding https://review.openstack.org/#/c/462216/ 15:11:46 this is about ramishra 's ML:) 15:11:48 #link http://lists.openstack.org/pipermail/openstack-dev/2017-May/117355.html 15:12:25 I did not get any solution from the devstack or infra team yet, little surprising 15:13:29 may be someone would look at it soon 15:13:39 ramishra: with the client change does everything work? 15:13:51 zaneb: I've responded to your comments 15:13:55 #link https://review.openstack.org/#/c/463510/ 15:14:20 regardless of what else we do, we should stop %-encoding the / between the stack name and stack UUID 15:14:45 zaneb: It works locally for me, but don'te about that route not found error for waitconditions 15:14:52 ramishra: on the ML you mean? because you haven't responded on that review 15:15:09 zaneb: yeah, I responded on the ML 15:15:37 ok cool, I saw that 15:16:13 ramishra, Maybe the cfn client still does that encoding? 15:16:38 ramishra: I think we're in agreement, because I said the same thing in the very next paragraph after the one you replied to ;) 15:17:02 therve: it does not look like http://logs.openstack.org/16/462216/11/check/gate-heat-dsvm-functional-convg-mysql-lbaasv2-non-apache-ubuntu-xenial/e7d9e90/console.html#_2017-05-20_07_04_30_500696 15:17:30 so https://review.openstack.org/#/c/463510/ sounds like the solution to all, right?:) 15:17:57 I don't really understand why port 80 could not be reached on the host 15:18:15 so all tests where we signal from the vm are failing 15:18:53 You can see the AodhAlarm test passed. 15:19:23 I think autoscaling stuff is also working fine 15:19:38 o/ 15:19:53 Btw, all the tests failing at the gate work fine for me locally 15:20:00 LanceHaig, o/ 15:20:03 so it's surely some infra thing 15:21:58 ramishra, can't figure out too 15:22:52 okay, at least we can land the other fix:) 15:22:58 #topic py3.5 for heat-agents https://review.openstack.org/#/c/425057/ 15:23:30 I can propose a gate for py35 in heat-agent 15:23:48 ramishra, It's failing with could not connect, it doesn't reach the cfn api 15:23:50 just think might be fine to let https://review.openstack.org/#/c/425057/ land first? 15:23:57 therve: yeah 15:24:12 ramishra, http://10.0.1.78/heat-api-cfn/ looks like a apache endpoint? And it's the non apache job 15:24:27 therve: I'm just testing one job 15:24:58 so I've just used the flag we aleady have HEAT_USE_MOD_WSGI 15:25:13 I'll change that later, once it works:) 15:25:21 ramishra, Sure, but it doesn't look like the issue is an encoding issue? 15:25:24 * therve is lost 15:26:02 therve: If the flag is not set then we're using apache+uwsgi 15:26:23 therve: yeah, the reamaining issue is not an encoding issue 15:26:25 ramishra, Which flag? 15:26:40 You're just setting heat_waitcondition_server_url incorrectly 15:26:40 therve, HEAT_USE_MOD_WSGI 15:27:07 But the logic in the patch looks backward 15:27:30 See http://logs.openstack.org/16/462216/11/check/gate-heat-dsvm-functional-orig-mysql-lbaasv2-ubuntu-xenial/e9e5a03/logs/etc/heat/heat.conf.txt.gz 15:27:35 It should use the apache endpoint 15:27:44 therve: yeah, I just mentioned, I wanted to just test one job 15:28:05 so the non-apache job is using apache+uwsgi in the last patch 15:28:12 I'll change that once it works 15:28:43 ramishra, It's not using apache? There is a cfn screen log 15:29:11 therve: yes, it's using systemd services+apache 15:29:18 and the proxy_pass 15:30:10 http://logs.openstack.org/16/462216/11/check/gate-heat-dsvm-functional-convg-mysql-lbaasv2-non-apache-ubuntu-xenial/e7d9e90/logs/apache_config/heat-wsgi-api.conf.txt.gz 15:30:40 OK but 15:30:43 ramishra, https://review.openstack.org/gitweb?p=openstack/heat.git;a=blob;f=devstack/lib/heat;h=bbbb9bab54e201772b3c813b94942984d5a1da8f;hb=539c24231570dc0031399fc6bd6e81eaa028a132#l132 15:30:48 That logic is backward 15:31:19 therve: yes, I just wanted to make it work first 15:31:25 We should connect to /heat-api-cfn when MOD_WSGI si true, not the other way around 15:31:45 Make what work? :) 15:31:51 OK I'll shut up 15:32:43 mod_wsgi and mod_proxy_uwsgi are different 15:33:10 so I'll add a new flag USE_UWSGI 15:33:52 so we've options 1. no apache 2. apache_mod_wsgi 3. apache+uwsgi(with mod_proxy_uwsgi) 15:34:10 We'll have different jobs for all three later 15:34:40 Does that help clear up the confusion? 15:35:23 therve: ? 15:35:30 let's say that it does and move on ;) 15:36:15 ramishra, therve said he will shut up:) 15:36:19 lol 15:36:37 so back to py35 15:37:20 Is it fine to let https://review.openstack.org/#/c/425057/ land first? And I will change the gate after 15:38:16 or we can insist to put the gate in first, and it's going to be a non voting period 15:38:56 I think it's fine to land it 15:39:22 cool 15:39:43 therve: I think it would be something like this https://review.openstack.org/#/c/462216/4/devstack/lib/heat 15:40:13 ricolin: sorry for continuing with the earlier topic 15:40:59 ramishra, np! we got to solve that issue:) 15:41:56 doesn't the actual encoding of stdout/stderr depend on the locale? 15:44:29 Yes 15:45:31 it seems like that decoding will be a possible source of exceptions 15:48:50 zaneb: are we stilltalking about the earlier topic? 15:49:04 ramishra, yes:) 15:49:30 ramishra: we're talking about https://review.openstack.org/#/c/425057/6 (py3.5) 15:49:41 zaneb: ok 15:49:57 I added a comment to the review 15:50:26 ricolin: next topic? 15:50:32 10 mins left 15:50:44 #topic heat-template future 15:51:07 #link http://lists.openstack.org/pipermail/openstack-dev/2017-May/117080.html 15:51:42 first would like to see the fake client landing 15:51:43 #link https://review.openstack.org/#/c/425057/6 15:52:03 therve, also can we also add magnum in fake client?:) 15:52:43 wait wrong link 15:52:49 ricolin, Are there magnum templates? 15:53:04 #link https://review.openstack.org/#/c/465860/ 15:53:29 #link https://review.openstack.org/#/c/455673/ 15:53:43 not in current repo 15:53:58 but a patch for it:) 15:54:13 OK 15:54:33 also LanceHaig got some thread in ML 15:54:34 #link http://lists.openstack.org/pipermail/openstack-dev/2017-May/117080.html 15:55:00 Can someone explain to me how this fake client would work? 15:55:04 about some cool suggestion for heat 15:55:07 Or do w ego that on the list rather? 15:55:38 sorry for the typo's it has been a long day 15:55:54 LanceHaig, Btw, you can put openstack version names in template versions 15:56:01 Forgot to answer that 15:56:25 therve, thanks. 15:57:07 LanceHaig, I like what we have in heat-tutorial, would be greate to make those in heat repo. Also you might consider to use os-collect-config to replace cloud-init in tutorial, since that's more recommanded to do so. 15:57:38 Hum 15:57:52 If things work with cloud init, not sure there is a reason to use os collect config 15:58:04 occ requires a custom image, whereas cloud init is everywhere 15:58:24 what therve said 15:58:49 I am wanting to keep the examples as easy to use for all heat versions 15:59:09 Not everyone can run a new version of OS and some still have very old versions 15:59:59 therve, sure, but like Magnum team, they might facing metadata size limit though 16:00:05 I know if does not make sense to support old versions but at least we can get a set of working templates for the older versions and then mark it as unsupported so that they have something to work with 16:00:14 I am happy to put in the work to get this done 16:00:21 I just want to see that everyone agrees 16:00:27 LanceHaig, therve and keep it cloud-init might actually better:) 16:01:00 just thinking about the tutorial only 16:01:09 right 16:01:12 sorry 16:01:46 Alright, EOM 16:01:53 LanceHaig, I think we can make a set for that:) 16:01:58 therve, right! 16:02:10 shall we move back to heat:) 16:02:21 yes 16:02:22 #endmeeting