Wednesday, 2017-12-06

*** Guest35678 has quit IRC00:07
*** jmlowe has quit IRC00:22
*** dylanturn has quit IRC00:46
*** hongbin has quit IRC00:49
*** hieulq has joined #heat00:50
*** dylanturn has joined #heat00:51
*** yassine has joined #heat01:00
*** yassine is now known as Guest1893001:00
*** om2 has quit IRC01:05
*** yangyapeng has joined #heat01:14
*** dylanturn has quit IRC01:16
*** dylanturn has joined #heat01:19
*** ayoung has quit IRC01:26
*** liusheng has quit IRC01:32
*** kiennt26 has joined #heat01:36
*** ricolin has joined #heat01:41
*** liusheng has joined #heat01:46
*** namnh has joined #heat01:53
*** wangbo has joined #heat02:10
*** dylanturn has quit IRC02:21
*** pbourke_ has quit IRC02:25
*** fragatin_ has quit IRC02:27
*** pbourke_ has joined #heat02:27
*** ricolin has quit IRC02:30
*** hieulq has quit IRC02:32
*** hieulq has joined #heat02:34
*** elynn has joined #heat02:49
*** hjensas has quit IRC03:07
*** liyi has joined #heat03:08
*** liyi has quit IRC03:09
*** dylanturn has joined #heat03:09
openstackgerritKien Nguyen proposed openstack/heat master: Change transport_url in grenade-multinode job  https://review.openstack.org/51040003:24
*** hjensas has joined #heat03:24
*** hjensas has quit IRC03:24
*** hjensas has joined #heat03:24
openstackgerritKien Nguyen proposed openstack/heat master: Change transport_url in grenade-multinode job  https://review.openstack.org/51040003:46
*** ramishra has joined #heat03:55
*** links has joined #heat03:59
*** links has quit IRC04:01
*** links has joined #heat04:01
*** gouthamr has quit IRC04:36
*** namnh has quit IRC04:41
*** namnh has joined #heat04:42
*** wangbo has quit IRC04:55
*** ricolin has joined #heat04:58
openstackgerritMerged openstack/heat master: Fix reset_stack_status  https://review.openstack.org/52455805:10
*** skramaja has joined #heat05:14
*** dylanturn has quit IRC05:19
*** dylanturn has joined #heat05:22
*** wangbo has joined #heat05:28
*** k_mouza has joined #heat05:32
*** k_mouza has quit IRC05:37
*** mdnadeem has joined #heat05:43
*** fragatina has joined #heat05:53
*** AlexeyAbashkin has joined #heat06:07
*** AlexeyAbashkin has quit IRC06:11
*** AlexeyAbashkin has joined #heat06:12
*** rwsu has quit IRC06:17
*** nmathew has joined #heat06:21
*** AlexeyAbashkin has quit IRC06:26
*** rwsu has joined #heat06:30
openstackgerritMerged openstack/heat master: [policy in code] part3 (resource types)  https://review.openstack.org/50535806:43
*** gkadam has joined #heat06:54
*** rcernin has quit IRC07:02
*** jtomasek has joined #heat07:10
*** jtomasek has quit IRC07:10
*** jtomasek has joined #heat07:11
*** ioggstream has joined #heat07:31
openstackgerritKien Nguyen proposed openstack/heat master: Change transport_url in grenade-multinode job  https://review.openstack.org/51040007:39
*** rcernin has joined #heat07:39
*** ildikov has quit IRC08:03
*** ildikov has joined #heat08:04
*** dbecker has joined #heat08:07
*** mnaser has quit IRC08:08
*** shardy has joined #heat08:08
*** mnaser has joined #heat08:11
openstackgerritKazunori Shinohara proposed openstack/heat-dashboard master: Update plugin.sh to resolve a couple of issues  https://review.openstack.org/52593508:12
*** elynn has quit IRC08:21
*** elynn has joined #heat08:24
*** dbellant has joined #heat08:25
*** AlexeyAbashkin has joined #heat08:32
*** dbellant has quit IRC08:33
*** AlexeyAbashkin has quit IRC08:38
*** dbellant has joined #heat08:44
*** AlexeyAbashkin has joined #heat08:44
*** ramishra has quit IRC08:45
*** jtomasek has quit IRC08:45
*** bubuwifi_ has quit IRC08:48
*** AlexeyAbashkin has quit IRC08:49
*** dbellant has quit IRC08:50
*** bubuwifi_ has joined #heat08:52
*** dbellant has joined #heat08:53
*** ramishra has joined #heat08:57
*** gfidente has joined #heat08:57
*** hieulq_ has joined #heat09:01
*** hieulq_ has quit IRC09:04
*** hieulq has quit IRC09:04
*** hieulq has joined #heat09:06
*** hieulq_ has joined #heat09:07
*** hieulq has quit IRC09:08
*** hieulq_ has quit IRC09:08
*** gfidente has quit IRC09:09
*** k_mouza has joined #heat09:09
*** hieulq has joined #heat09:10
*** gfidente has joined #heat09:10
*** gfidente has quit IRC09:10
*** gfidente has joined #heat09:10
*** bubuwifi__ has joined #heat09:12
*** wangbo has quit IRC09:13
*** vishwanathj has quit IRC09:13
*** wangbo has joined #heat09:14
*** vishwanathj has joined #heat09:14
*** AlexeyAbashkin has joined #heat09:14
*** gfidente has quit IRC09:15
*** bubuwifi_ has quit IRC09:15
*** gfidente has joined #heat09:17
*** armaan has joined #heat09:21
therveramishra, Hi, looking at https://review.openstack.org/#/c/523307/09:26
therveramishra, The test you made seems a bit fragile, did you +1 a template id to get a unknown template?09:26
ramishratherve: yes09:29
therveramishra, Any better idea then? :)09:30
openstackgerritMerged openstack/heat-dashboard master: Update plugin.sh to resolve a couple of issues  https://review.openstack.org/52593509:30
*** gfidente has quit IRC09:31
*** armaan has quit IRC09:31
ramishratherve: I mean we setup dummy db for every test, so I thought there would be no conflict as such, no?09:31
*** gfidente has joined #heat09:33
*** gfidente has quit IRC09:33
*** gfidente has joined #heat09:33
therveHum, maybe09:33
*** serlex has joined #heat09:35
*** d0ugal has quit IRC09:37
*** derekh has joined #heat09:39
*** d0ugal has joined #heat09:41
*** serlex has quit IRC09:44
*** armaan has joined #heat09:53
*** gfidente has quit IRC09:57
*** gfidente has joined #heat10:01
*** gfidente has quit IRC10:01
*** gfidente has joined #heat10:01
*** gfidente has quit IRC10:03
*** ccamacho has joined #heat10:11
*** gfidente has joined #heat10:13
*** gfidente has joined #heat10:13
*** kiennt26 has quit IRC10:15
*** armaan has quit IRC10:16
*** k_mouza has quit IRC10:22
*** wangbo has quit IRC10:22
*** k_mouza has joined #heat10:24
*** Qiming has quit IRC10:24
*** namnh has quit IRC10:25
*** elynn has quit IRC10:28
*** Qiming has joined #heat10:30
openstackgerritKien Nguyen proposed openstack/heat master: Change transport_url in grenade-multinode job  https://review.openstack.org/51040010:32
*** skramaja_ has joined #heat10:33
*** skramaja has quit IRC10:33
*** tosky has joined #heat10:37
*** skramaja has joined #heat10:37
*** skramaja_ has quit IRC10:38
*** d0ugal has quit IRC10:39
*** jtomasek has joined #heat10:40
ricolinramishra, why we ne:qeds to raise NotFound for all delete actions in sqlalchemy api?10:45
ricolins/ne:qeds/needs/10:46
ricolintherve, ^^^10:47
ramishraricolin: I don't understand the question, which delete action?10:56
ricolinlike in https://github.com/openstack/heat/blob/master/heat/db/sqlalchemy/api.py#L43410:57
ricolinresource_data_get_by_key will raise not found if no such result from query10:58
*** d0ugal has joined #heat11:01
*** wangbo has joined #heat11:01
ramishrayeah, that seems incorrect to me though, may be it's taken care of in the object layer, I don't know11:02
ricolinI found this error https://bugs.launchpad.net/heat/+bug/173670611:03
openstackLaunchpad bug 1736706 in OpenStack Heat "purge_db failed with non-exists raw_template" [Medium,New] - Assigned to Rico Lin (rico-lin)11:03
*** tosky_ has joined #heat11:03
*** tosky has quit IRC11:03
*** tosky_ is now known as tosky11:03
ricolinnot sure if I can ignore not found directly in delete api11:04
ramishraI mean that's in convergence, there is probably a race condition there11:06
ricolinyes it is11:06
openstackgerritMerged openstack/heat master: Ignore resources with non-existent template  https://review.openstack.org/52330711:08
*** wangbo has quit IRC11:12
*** tosky has quit IRC11:16
*** wangbo has joined #heat11:23
openstackgerritMerged openstack/heat master: Eager-load properties for nested resource list  https://review.openstack.org/52164311:25
*** links has quit IRC11:34
*** yangyapeng has quit IRC11:36
*** yangyapeng has joined #heat11:36
*** rcernin has quit IRC11:39
*** k_mouza has quit IRC11:40
*** links has joined #heat11:48
*** kiennt26 has joined #heat11:54
*** nmathew has quit IRC11:54
*** AlexeyAbashkin has quit IRC11:58
*** openstackgerrit has quit IRC12:03
*** k_mouza has joined #heat12:03
*** links has quit IRC12:08
*** openstackgerrit has joined #heat12:09
openstackgerritKien Nguyen proposed openstack/heat master: Change transport_url in grenade-multinode job  https://review.openstack.org/51040012:09
*** kiennt26 has quit IRC12:11
*** links has joined #heat12:13
*** xinni9e_ has joined #heat12:16
*** yangyapeng has quit IRC12:18
*** yangyapeng has joined #heat12:18
*** yangyapeng has quit IRC12:25
ricolinHi all ,meeting in 15mins. feel free to add agenda https://wiki.openstack.org/wiki/Meetings/HeatAgenda#Agenda_.282017-12-06_1300_UTC.2912:45
*** xinni9e_ has quit IRC12:46
*** AlexeyAbashkin has joined #heat12:46
*** yangyapeng has joined #heat12:50
*** iurygregory has quit IRC12:52
*** ivve has quit IRC12:55
*** nmathew has joined #heat12:55
*** xinni9e_ has joined #heat12:57
*** wangbo has quit IRC12:58
ricolin#startmeeting heat13:00
openstackMeeting started Wed Dec  6 13:00:22 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:00
*** openstack changes topic to " (Meeting topic: heat)"13:00
openstackThe meeting name has been set to 'heat'13:00
ricolin#topic roll call13:00
*** openstack changes topic to "roll call (Meeting topic: heat)"13:00
*** skramaja has quit IRC13:01
ramishrahi13:01
ricolino/13:01
ricolinzaneb, therve meeting time:)13:01
kazsho/13:01
ricolino/13:01
ricolinkazsh, what you mean by topic -> temporarlly remove admin panel for Heat (heat-dashboard13:03
kazshricolin: current horizon has admin panel for heat apart from orchestration panel group13:03
ricolinoh okay got it13:04
ricolin#topic adding items to agenda13:04
*** openstack changes topic to "adding items to agenda (Meeting topic: heat)"13:04
ricolin#link https://wiki.openstack.org/wiki/Meetings/HeatAgenda#Agenda_.282017-12-06_1300_UTC.2913:04
ricolinmaybe 8:00 is too early US-East13:05
* ricolin never understand the light saving time changes13:05
ramishraricolin: not many around to discuss some important topics13:05
ramishramay be we postpone them to next week13:06
ricolinramishra, we can do others first13:06
zanebo/13:06
ricolinzaneb, good morning!:)13:06
*** ivve has joined #heat13:06
zanebwe could talk about the tempest goal13:07
ricolinzaneb, on the topics:)13:08
ricolinlet's do that one first13:08
zanebso it is :)13:08
ricolin#topic heat tempest plugin13:08
*** openstack changes topic to "heat tempest plugin (Meeting topic: heat)"13:08
*** armaan has joined #heat13:09
zanebso there's been some interesting discussion on https://review.openstack.org/#/c/521602/13:09
zanebI'd encourage everyone to go read that13:09
zanebbut the way it's shaping up is it looks like any stuff we leave in-tree should just not be a tempest plugin13:10
zanebwe can meet the goal just by removing the endpoint from setup.cfg13:11
ricolindidn't ready in that review yet, but can't we said that in tree plugin just for CI usage?13:12
zanebwhat we'll need is some way of running the tests from two different sources (plugin repo plus in-tree) in the gate13:12
ramishraI was looking at how many functional tests we've changed in the last six months for any code changes, they are very very few13:12
*** ioggstream has quit IRC13:12
ramishrait's mostly adding tests for existing functionality or new tests13:12
zanebricolin: yes, but it can't have a tempest plugin endpoint in setup.cfg. that's what breaks other projects just by having heat installed if we break test discovery13:13
*** k_mouza has quit IRC13:13
*** dprince has joined #heat13:14
*** k_mouza has joined #heat13:14
zanebso a mechanism for running the two lots of tests together is still under discussion13:14
zaneband we'll also need to decide which tests belong where13:14
zanebprobably more should go to the plugin than I originally thought13:15
zanebbecause there are several use cases for tempest other than interop13:15
ricolinzaneb, so if we all agree some tests to go to plugin, that means those test in long term should be branchless, right?13:16
zanebe.g. co-gating, running against clouds to check that they're configured correctly13:16
zanebricolin: tbh I'm not completely convinced that all of those use cases want branchless tests. but we might just have to live with that13:17
openstackgerritMerged openstack/heat-dashboard master: Align tox_install.sh with other projects  https://review.openstack.org/52413613:18
zanebanything that is purely testing regression stuff in Heat should stay behind IMO13:18
ricolintotally agree with that13:18
zanebanything with an integration component should _probably_ move to the plugin13:18
zanebI guess I need to make a list, don't I13:19
*** armaan has quit IRC13:20
ricolinwhat you mean by  integration component13:20
zanebI mean it's testing interaction with some other service13:20
ramishrazaneb: Now that we've created the new repo, we can just see how many tests are creating issues and then move them as we go from the new repo, or as u said we can leave all tests in-tree for now, but we've to manage the changes in both repos.13:22
ramishraor may be if we outrightly can identify what tests to have in-tree and how we test them13:23
*** links has quit IRC13:23
zanebramishra: I'd rather try to identify them ahead of time if possible, so that we don't delete them and lose the git history13:24
ramishramy only concern is managing changes.13:25
chandankumarramishra: hello13:25
zanebyeah, we are in limbo at the moment13:25
zanebwith code in two places13:25
chandankumarfor managing changes you can use depends-on in gerrit reviews13:26
ramishrachandankumar: hi13:26
zanebramishra: we're currently running the tests from the plugin not the repo, is that right?13:26
ricolinzaneb, yes13:26
ramishrayeah from the external plugin repo not in-tree tests13:26
zanebyes, that's what I meant :)13:27
* zaneb was extremely ambiguous13:27
zanebok, so rough plan13:27
zaneb1) identify tests we can safely delete from the heat tree13:28
zaneb2) find a way of running tests from both sources13:28
zaneb3) delete tests that are staying behind from the plugin repo13:28
zanebmeanwhile try to avoid landing changes in only one place, I guess13:29
chandankumaris there any problem with branchless nature of plugin?13:29
ramishrawe've to keep a close watch on patches that modify tests, I can see a few around which make changes to tests in-tree.13:30
ricolinIs it possible that keep common test utils in plugin for tests from both sides?13:30
zanebricolin: yes. that's probably a good idea13:30
* zaneb hadn't thought about that13:31
ricolinthat's something after (3) I guess13:31
ricolinramishra, what you think about zaneb's plan13:32
ramishraricolin: yeah, if we can do 1 & 2 quickly, we'll avoid the mess we're in13:33
ricolinzaneb, how you suggest we start on doing (1)13:33
ramishrabut till we do that we would probably have a hard time landing patches that change tests13:34
* zaneb reluctantly volunteers to take an #action13:34
zanebramishra: agree. unfortunately 1 & 2 are the hard ones. 3 is easy13:35
zanebwell, 1 isn't _hard_. it's just work13:35
ricolinzaneb, we absolutely needs you to in part for 1:)13:36
zaneb#link https://etherpad.openstack.org/p/heat-integration-test-categories13:37
ricolin#action zaneb will help on identify tests we can safely delete from the heat tree13:37
zanebI'll start in there13:37
ramishrafor 2 may be we can probably run the tests without tempest, that means revert to the way we tested before we moved to use tempest plugin13:38
ricolinramishra, yeah, if we can't have a extra plugin for heat CI13:39
ramishraricolin: "extra_plugin"?13:39
ramishraAFAIU, it would not be a tempest plugin13:40
zanebramishra: so the trick will be that we want to discover the tests from the plugin and the tests from the local repo and run them all together using the same unittest run13:40
ricolinI mean like second tempest plugin, but it seems any tempest plugin in tree will not happen:)13:40
zanebbut yeah, it may just be a case of doing everything the old way and not treating the tempest plugin as a tempest plugin in our gate13:41
ricolinzaneb, ramishra any concerns or issues when we not using tempest plugin?13:41
zanebricolin: I like the idea of splitting the plugin mechanism out of tempest, so that we can do test suite plugins without interfering with tempest. that was discussed in the review I linked earlier. we'll see where it goes13:41
*** nmathew- has joined #heat13:42
ricolinzaneb, thanks will read that right after meeting13:42
ramishrawe can run them separately at the gate plugin tempest tempest tests + in-tree tests with 'tox -eintegration'13:42
zanebricolin: nope. it appears the plugin mechanism is literally just 50 lines of code that does test discovery13:42
zanebwe can still use tempest-lib stuff from the tempest tree. so really no concerns about how a non-plugin would work13:43
zanebjust a shame we can't use that handy plugin mechanism any more at the moment13:43
zaneb#link https://review.openstack.org/#/c/521602/13:45
*** nmathew has quit IRC13:45
zanebfor the minutes ^ :)13:45
ricolinso we revert first and see if we like to use tempest-lib further for discover?13:46
ramishrawe don't use tempest-lib stuff I think13:48
zanebtempest-lib isn't for discovery13:48
ramishrawe use python*_clients13:49
ramishrawe just use tempest as the test runner13:49
*** Smeared_Beard has joined #heat13:50
*** SmearedBeard has quit IRC13:51
ramishrawe're on this one topic for the whole time it seems:)13:52
zanebramishra: I just got schooled about that :) according to tempest folks, tempest is just a test suite and the testrunning is unvarnished unittest/testr13:52
ramishrazaneb: OK, I think I somewhat understood that though13:54
ricolinsounds like our original design:)13:56
*** dylanturn has quit IRC13:56
zanebpretty much13:56
zanebwe can definitely move to another topic now if folks want to13:56
ricolinwe might still needs a action here I think:)13:57
ramishrawe did not have the test discovery (with plugin) part though13:57
zanebyeah, and that's the part we will need to figure out13:57
ricolinwe still have few topic to go:(13:58
ricolinI propose we extend this meeting13:58
kazsh+113:58
ramishraricolin: I've to leave in 5 mins13:59
ricolinramishra, NP13:59
ricolinanyway that's do this quick13:59
ricolinwe leave rest topic to next14:00
zanebramishra: which topic would you most like to discuss in the next 5 minutes? :)14:00
ramishrazaneb: I've not added anything to the agenda:)14:00
*** nmathew- has quit IRC14:01
ricolinmaybe we go with kazsh's one?14:01
kazshthx!14:01
zaneb+114:01
ricolin#topic temporarlly remove admin panel for Heat (heat-dashboard14:01
*** openstack changes topic to "temporarlly remove admin panel for Heat (heat-dashboard (Meeting topic: heat)"14:01
kazshheat-dashboard is almost ready for queens-214:01
zaneb\o/14:02
ricolinkazsh, I will release rest heat to queens-2 today14:02
kazshresolved points horizon guys commented14:02
kazshricolin: thx!14:02
kazshand14:02
ricolinkazsh, is it possible we release heat-dashboard before Friday?14:02
kazshricolin: yep! it's ok even now :)14:03
kazshbut only one thing14:03
kazshI wanna consult with you14:03
kazshas you might know, current horizon has admin func for heat14:03
kazshjust displaying the result of "openstack orchestration service list"14:03
kazshthe list of engine services14:04
kazshmigrate this func is tricky because the code is not in orchestration panel group's code14:05
kazshsomething tightly coupling with other code14:05
ricolinso where these code are right now?14:05
kazshhttps://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/admin/info/tables.py#L25614:06
kazsharound here14:06
zanebkazsh: if that's all it does then I wouldn't worry about it14:06
ramishrabye guys, got to go, will check the meeting notes later14:06
kazshzaneb: thx I'm on same page as you14:06
zanebramishra: thanks o/14:06
ricolinramishra, bye! thanks for your attend!14:06
kazshhorizon guy also has same thought14:07
ricolinkazsh, any plan after this temporary removal?14:07
kazshas heat team, is there any objection on this approach ?14:07
kazshwell14:07
ricolinkazsh, I have no objection on that:)14:07
zanebno objection from me14:08
kazshone of them is saying we should talk about make admin panels as a plugin14:08
kazshprob..next PTG ?14:08
kazshalong with heat-dashboard code, horizon team is preparing this chage https://review.openstack.org/#/c/523402/14:08
kazshtrying to split out all of heat support from horizon repo14:08
kazshalready got one +2 from the core14:08
kazshzaneb: thx:)14:09
kazshhorizon team is waiting for our answer on this14:09
ricolinwe can always have a PTG session for this14:09
*** mdnadeem has quit IRC14:09
kazshricolin: yep:)14:09
ricolinwe can drop it once we release heat-dashboard queens-2 IMO14:10
kazshyep, the current heat-dashboard code does not have any admin func14:10
kazshand i a few days, 523402 will be merged in horizon side14:11
*** ramishra has quit IRC14:11
kazshok I got a couple of +2 from heat core :)14:11
ricolinso final ask It's fine for me to release heat-dashboard now?14:11
kazshricolin: yes :)14:12
zanebin my experience, at least, ~nobody uses the GUI for admin stuff14:12
kazshzaneb: +114:12
ricolinzaneb, bloody true14:12
*** pbourke_ is now known as pbourke14:12
kazshwell I will inform our decision to Horizon team14:12
ricolinkazsh, thanks14:13
kazshthanks ricolin, zaneb :)14:13
ricolinRabi is away now, I guess we can leave rest for next week:)14:13
ricolin14mins ahead:)14:14
ricolin#action rico will release heat today14:14
ricolinthanks you for join:)14:14
ricolinif no question let's end it14:15
kazsh+114:15
ricolin#endmeeting no sounds means no question:)14:15
*** openstack changes topic to "branchless heat tempest plugin (Meeting topic: heat)"14:15
openstackMeeting ended Wed Dec  6 14:15:45 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:15
openstackMinutes:        http://eavesdrop.openstack.org/meetings/heat/2017/heat.2017-12-06-13.00.html14:15
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/heat/2017/heat.2017-12-06-13.00.txt14:15
openstackLog:            http://eavesdrop.openstack.org/meetings/heat/2017/heat.2017-12-06-13.00.log.html14:15
ricolinzaneb, do you know we can we fix irc topic...14:16
zanebgood question14:16
ricolinI mean our irc topic is broken for a week:(14:16
*** mchlumsky has joined #heat14:16
ricolin#endmeeting14:16
zanebricolin: maybe ask in #infra14:17
*** dbellant has quit IRC14:17
ricolinzaneb, yeah, on the way14:17
zaneber, #openstack-infra14:17
ricolinzaneb, I got a question about our DB APIs14:18
*** gkadam has quit IRC14:18
zanebdon't we all14:18
ricolinso I found this issue in gate and try to figure out what's the best way to approch14:18
ricolinhttps://bugs.launchpad.net/heat/+bug/173670614:18
openstackLaunchpad bug 1736706 in OpenStack Heat "purge_db failed with non-exists raw_template" [Medium,New] - Assigned to Rico Lin (rico-lin)14:18
ricolins/approch/approach/14:18
ricolincan we not raise not found exception when we do a delete action in DB API?14:19
ricolinlike in https://github.com/openstack/heat/blob/master/heat/db/sqlalchemy/api.py#L43414:19
ricolin in resource_data_delete(), resource_data_get_by_key will raise not found if no such result from query14:19
zanebtbh I think this is a result of running a very aggressive purge (i.e. purge anything deleted >0 minutes ago) in the test14:20
zaneband it's getting into a race with another test by the looks of it14:21
*** crose has joined #heat14:21
ricolina race condition I think14:21
zanebyeah, so in the real world this wouldn't happen14:21
zanebit's annoying, but if we're going to change something it should probably be to make the test less racy somehow14:22
therveFamous last words :)14:22
zaneblol14:22
zanebtherve: people _should_ be using ages higher than 0 minutes14:22
thervezaneb, Right, but it doesn't mean it won't delete the stack you look at14:22
zanebmaybe we should have a warning message if you use 014:23
zanebtherve: it does mean that, doesn't it?14:23
* zaneb will bbiab14:24
*** xinni9e_ has quit IRC14:25
*** ayoung has joined #heat14:25
* ricolin might not be back since it's 22:25 already:)14:26
*** hjensas has quit IRC14:34
*** ioggstream has joined #heat14:37
*** iogg has joined #heat14:40
*** ioggstream has quit IRC14:41
*** gouthamr has joined #heat14:47
*** dylanturn has joined #heat14:50
*** armaan has joined #heat15:03
*** ramishra has joined #heat15:07
*** jmlowe has joined #heat15:16
*** armaan has quit IRC15:16
*** k_mouza_ has joined #heat15:16
*** k_mouza has quit IRC15:19
*** dylanturn has quit IRC15:21
*** k_mouza_ has quit IRC15:24
*** k_mouza has joined #heat15:25
*** k_mouza has quit IRC15:25
*** k_mouza has joined #heat15:26
*** kiennt26 has joined #heat15:28
*** dylanturn has joined #heat15:31
*** tiantian has quit IRC15:33
*** liusheng has quit IRC15:33
thervezaneb, Looking at this case more closely, the stack is not deleted yet, so the template shouldn't gone15:33
therveIf purge is indeed the thing interfering, it shouldn't15:33
*** crose has quit IRC15:36
zanebso there is a bug15:40
therveThere is a suspicious exception later on in the logs15:51
*** bubuwifi has quit IRC15:55
*** dylanturn has quit IRC15:58
*** andreaf_ has joined #heat16:10
thervezaneb, Looks like mark_complete is called multiple times16:13
zanebon delete? that's surprising16:14
therveThere is a delete and a concurrent update :/16:16
thervehttp://logs.openstack.org/74/510074/6/check/heat-functional-convg-mysql-lbaasv2-py35/8f7dd27/logs/screen-h-eng.txt.gz?#_Dec_04_18_02_32_83436616:16
thervehttp://logs.openstack.org/74/510074/6/check/heat-functional-convg-mysql-lbaasv2-py35/8f7dd27/logs/screen-h-eng.txt.gz?#_Dec_04_18_02_27_62158816:16
*** andreaf has quit IRC16:17
*** andreaf_ is now known as andreaf16:17
zaneboh, ok, that is legit16:18
therveRight, it doesn't seem to handle it properly though16:19
zanebwe cancel the update and then delete the stack16:19
zanebso we expect _an_ exception in that case16:19
zanebjust not the one we're getting apparently16:19
zanebso we need to handle that one as well16:19
*** frickler has joined #heat16:20
thervezaneb, Shouldn't the thread killed though?16:22
*** fungi has joined #heat16:22
therveIt seems to go through the update16:22
zanebnot in convergence16:22
*** fungi has left #heat16:22
zaneboh, I guess it should be killed when we start the delete16:22
*** gouthamr has quit IRC16:23
zanebhonestly, that test should wait for the update to be cancelled successfully before it ends (and causes the stack to get deleted)16:24
zanebotherwise what are we actually testing?16:24
therveWell. That you can delete while update runs? :)16:25
therveNot reliably I guess16:25
zanebif we were planning to test that we should test it in the test code and not only in the cleanup :)16:26
*** dylanturn has joined #heat16:27
*** iurygregory has joined #heat16:27
openstackgerritKien Nguyen proposed openstack/heat master: Change transport_url in grenade-multinode job  https://review.openstack.org/51040016:30
*** dylanturn has quit IRC16:31
*** ricolin_ has joined #heat16:32
*** ricolin has quit IRC16:34
*** dbecker has quit IRC16:35
*** dylanturn has joined #heat16:37
*** kiennt26 has quit IRC16:38
*** nmathew- has joined #heat16:39
*** nmathew has joined #heat16:41
*** nmathew- has quit IRC16:44
*** nmathew- has joined #heat16:44
*** nmathew- has quit IRC16:45
*** nmathew has quit IRC16:48
*** syjulian has quit IRC16:52
*** hongbin has joined #heat16:54
*** AlexeyAbashkin has quit IRC16:54
*** fragatina has quit IRC16:55
*** ricolin_ has quit IRC16:55
*** ricolin has joined #heat16:56
*** ChanServ changes topic to "OpenStack Heat (Logs: http://eavesdrop.openstack.org/irclogs/%23heat/)"16:56
*** ChanServ changes topic to "OpenStack Heat Team (logs: http://eavesdrop.openstack.org/irclogs/%23heat/)"16:57
*** dprince has quit IRC17:01
zigoHi there! I'm trying to debug my tempest setup, and I'm having issues setting-up heat.conf, apparently. Could someone help me by giving me an example hest.conf from tempest?17:07
zigotherve: ^17:07
*** Guest18930 has quit IRC17:07
*** hjensas has joined #heat17:09
*** hjensas has joined #heat17:09
*** dylanturn has quit IRC17:11
openstackgerritThomas Goirand proposed openstack/heat master: Build heat.conf reproducibly  https://review.openstack.org/52612617:12
zigotherve: I also would love to get this patch merged, if possible! :)17:13
zigoIt's a one liner ...17:14
zigo(no, I'm not trying to vote for every project... :P)17:14
*** fragatina has joined #heat17:14
*** jtomasek has quit IRC17:22
*** dylanturn has joined #heat17:23
*** iogg has quit IRC17:25
*** ricolin has quit IRC17:25
openstackgerritMerged openstack/heat master: Speed up event listing with nested stacks  https://review.openstack.org/52165117:40
*** ramishra has quit IRC17:40
*** dylanturn has quit IRC17:43
*** dylanturn has joined #heat17:44
*** gouthamr has joined #heat17:44
*** ivve_ has joined #heat17:46
*** ivve has quit IRC17:47
openstackgerritMerged openstack/heat master: Generate correct name for Sahara cluster  https://review.openstack.org/50674917:50
*** shardy has quit IRC17:54
*** k_mouza_ has joined #heat17:56
*** k_mouza has quit IRC18:00
*** k_mouza_ has quit IRC18:01
*** derekh has quit IRC18:01
*** gkadam has joined #heat18:32
*** dprince has joined #heat18:36
*** frickler has left #heat18:37
*** hjensas has quit IRC18:47
*** hjensas has joined #heat18:56
*** hjensas has quit IRC19:13
*** harlowja has quit IRC19:32
*** harlowja has joined #heat19:36
*** rwsu has quit IRC19:51
*** crose has joined #heat19:51
*** Sandy619 has joined #heat19:56
*** k_mouza has joined #heat19:59
*** crose has quit IRC20:05
*** k_mouza has quit IRC20:07
*** gkadam has quit IRC20:14
*** crose has joined #heat20:18
*** dprince has quit IRC20:21
openstackgerritMerged openstack/heat-templates master: Add example for using external resources  https://review.openstack.org/41759420:28
*** crose has quit IRC20:31
*** kei-ichi has quit IRC20:31
*** zhenguo has quit IRC20:31
*** dgonzalez has quit IRC20:31
*** Tyrantelf_ has quit IRC20:31
*** hongbin has quit IRC20:31
*** jmlowe has quit IRC20:31
*** nwonknu has quit IRC20:31
*** FL1SK has quit IRC20:31
*** jnettlet[m] has quit IRC20:35
*** syjulian has joined #heat20:38
*** mikal has quit IRC20:38
*** mikal has joined #heat20:43
*** kei-ichi has joined #heat20:47
*** zhenguo has joined #heat20:47
*** dgonzalez has joined #heat20:47
*** Tyrantelf_ has joined #heat20:47
*** hongbin has joined #heat20:47
*** jmlowe has joined #heat20:47
*** nwonknu has joined #heat20:47
*** FL1SK has joined #heat20:47
*** livelace-link has quit IRC20:49
*** livelace-link has joined #heat20:51
*** livelace-link has quit IRC20:53
*** livelace-link has joined #heat20:54
*** livelace-link has quit IRC20:56
*** livelace-link has joined #heat20:57
*** livelace-link has quit IRC20:59
*** harlowja has quit IRC21:00
*** harlowja has joined #heat21:03
jdandrea_For a WaitHandle (in heat-newton) I get "Can't find role heat_stack_user" ... but don't we no longer need that role in latter versions of heat? Or are there some install steps that might have been overlooked?21:14
jdandrea_(meaning when folks installed heat-newton)21:14
*** Smeared_Beard has quit IRC21:14
*** higgins has quit IRC21:16
*** higgins has joined #heat21:16
*** hjensas has joined #heat21:21
*** hjensas has quit IRC21:21
*** hjensas has joined #heat21:21
*** liusheng has joined #heat21:30
*** harlowja has quit IRC21:32
*** gouthamr has quit IRC21:36
*** AlexeyAbashkin has joined #heat21:38
*** openstackgerrit has quit IRC21:38
*** sai has quit IRC21:38
*** dyasny_ has quit IRC21:38
*** lifeless has quit IRC21:38
*** mfisch has quit IRC21:38
*** afazekas has quit IRC21:38
*** hjensas has quit IRC21:45
*** gfidente has quit IRC21:57
*** openstackgerrit has joined #heat22:01
*** sai has joined #heat22:01
*** dyasny_ has joined #heat22:01
*** lifeless has joined #heat22:01
*** mfisch has joined #heat22:01
*** afazekas has joined #heat22:01
*** rcernin has joined #heat22:01
*** rcernin has quit IRC22:03
*** rcernin has joined #heat22:03
*** shadower has quit IRC22:12
*** shadower has joined #heat22:13
zanebjdandrea_: yeah, you might have skipped creating the stack user domain + roles in keystone22:16
*** yassine has joined #heat22:26
*** yassine is now known as Guest9919622:27
*** gouthamr has joined #heat22:28
*** harlowja has joined #heat22:36
*** AlexeyAbashkin has quit IRC22:43
*** gouthamr has quit IRC22:53
*** gouthamr has joined #heat22:56
*** Sandy619 has quit IRC23:25
*** armaan has joined #heat23:26
*** openstack has joined #heat23:31
*** ChanServ sets mode: +o openstack23:31

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!