13:00:45 #startmeeting heat 13:00:46 Meeting started Wed Nov 29 13:00:45 2017 UTC and is due to finish in 60 minutes. The chair is ricolin. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:00:47 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:00:49 The meeting name has been set to 'heat' 13:00:57 #topic roll call 13:01:42 o/ 13:01:48 o/ 13:01:52 o/ 13:02:26 hi 13:03:21 zaneb, therve meeting time:) 13:04:05 #topic adding items to agenda 13:04:11 #link https://wiki.openstack.org/wiki/Meetings/HeatAgenda#Agenda_.282017-11-29_1300_UTC.29 13:04:49 Yeah aroundish 13:07:05 #topic CloudWatch and HARestarter 13:08:23 We already have ML and patch for both resources, and didn't get any objection about remove those stuff 13:08:30 so on the cloudwatch stuff, we should decide quickly on what to do, ceiloemter and ceilometerclient both gone now 13:09:40 one of the propose to remove cloudwatch: https://review.openstack.org/#/c/514515/ and https://review.openstack.org/#/c/517189/ 13:11:27 ramishra, any other resource or API still using `CeilometerClientPlugin`? 13:11:28 we use ceilometer sample api with cloudwatch service to upload custom metrics, so keeping it does not make much sense, unless we come up with another way of doing it 13:12:14 ricolin: AFAIK only AodhAlarm uses it other than cloudwatch 13:14:06 Not sure how we remove client plugins? Have we done one yet? 13:15:19 not for these few years:) 13:16:45 therve: any idea? third party resources may be using it. if the ceilometerclient is not there, there is probably no point keeping it? 13:16:47 is that mean if we directly hidden AodhAlarm(and mark as placeholder resource) and remove cloudwatch, we can directly remove ceilometer client? 13:17:13 ramishra, Probably remove it 13:17:26 therve: yep 13:18:08 remove it +1 13:19:24 ok it seems we all agree, unless there is an disagreement later:) 13:19:54 ricolin: let's move on, I'll send some more patches to clean it up 13:20:08 ramishra, nice 13:20:44 #link https://review.openstack.org/#/c/511278/ 13:21:17 As for HaRestarter I propose a patch to directly Hidden it and mark it as placeholder 13:21:36 Any Objection about it? 13:21:44 or concern about it?:) 13:21:51 +1 13:23:32 cool, so I think we can move on and do it:) 13:23:43 I know zaneb is quick support to remove those code out:) 13:24:24 #topic branchless heat tempest plugin 13:25:16 We need to think about ways in future to allow adding new tests and keep our tempest branchless 13:25:37 Any idea what will be the best way to do it? 13:26:44 one way I can think about is to add a branch tag for new join integration and scenairo test 13:27:05 s/scenairo/scenario/ 13:27:26 ricolin: AFAIK, the test should be able to query it using some kind of API from the deployment, like microversions 13:28:36 because u don't know what's the heat deployed version when you run the tests with the tempest plugin 13:28:43 deployed version of heat 13:28:47 ramishra, can you give me a sample code? 13:29:08 ricolin: we don't have microversions 13:29:44 I don't know if there is any other simple way to do it 13:29:48 ramishra, than we can't use it:/ 13:30:31 yes, if we don't find a way then we can't make it branchless 13:30:40 ramishra, we needs to figure this our before we release queens 13:30:53 making branchless has number of issues, I see plenty of issues with tempest being branchless 13:32:19 sure, may be starting a ML thread about it to collect some ideas? 13:32:43 ramishra, I will also go ask QA teams about this as well 13:33:07 better got a way for all non-microversion projects to do it 13:34:03 we needs to know what other issues we're going to face on doing this 13:34:57 ML+1 13:36:20 ramishra, you want to do the ML or I do it?:) 13:36:22 may be we can use config entries like https://github.com/openstack/tempest/blob/master/tempest/config.py#L427 13:37:21 tempest uses it for some features and skips tests based on whether they are enabled or not 13:37:29 not a great way, but would work 13:38:24 ramishra, maybe from different source of config file(not in heat.conf)? 13:39:19 it's not in heat.conf but would be in heat_plugin section in tempest.conf 13:39:58 ramishra, yeah, that will be better and sync with other heat tempest config:) 13:40:13 example https://github.com/openstack/tempest/blob/5fc2548eae225ebe84ab378a6136f1be9cb8b443/tempest/api/volume/admin/test_volume_snapshot_quotas_negative.py#L30 13:40:56 workable solution and tempest uses it 13:41:30 ramishra, there we at least have a way to do it:) 13:42:28 andreaf, around?