Wednesday, 2016-06-08

*** EricGonczer_ has joined #heat00:05
*** cwolferh has quit IRC00:14
*** dyasny has joined #heat00:18
*** EricGonczer_ has quit IRC00:21
*** cwolferh has joined #heat00:31
*** ricolin has joined #heat00:37
*** jasond has joined #heat01:07
openstackgerrithuangtianhua proposed openstack/heat: Make sure glance image create success with tags  https://review.openstack.org/32631001:10
openstackgerrithuangtianhua proposed openstack/heat: Ignore notfound error when delete image tags  https://review.openstack.org/32635001:11
*** bana_k has quit IRC01:14
*** zz_dimtruck is now known as dimtruck01:16
*** marcin1234 has quit IRC01:19
*** ramishra has joined #heat01:26
*** p01nt3r75 has quit IRC01:31
*** morion has joined #heat01:34
*** yanyanhu has joined #heat01:44
*** jasond has quit IRC01:45
*** elynn has joined #heat01:55
*** elynn_ has joined #heat01:59
*** elynn has quit IRC02:00
*** akshai has quit IRC02:01
*** morion has quit IRC02:05
*** akshai has joined #heat02:12
*** morion has joined #heat02:14
openstackgerrithuangtianhua proposed openstack/heat: Support resource/output condition  https://review.openstack.org/28336602:19
cwolferhanybody have any clues as to how to get heat.db.sqlalchemy.api LOG messages to show up (anywhere) in a devstack env? kind of perplexed why they are not showing up, at least in the functional test_purge case02:30
*** sdake has joined #heat02:32
*** ramishra_ has joined #heat02:33
*** sdake has quit IRC02:34
openstackgerritRabi Mishra proposed openstack/python-heatclient: Serialize files when using SessionClient  https://review.openstack.org/32598402:36
*** ramishra has quit IRC02:36
cwolferhneverming, it just looks like heat-manage LOG messages get sent to stdout or stderr, and those are just not stashed away anywhere in the functional test02:41
*** openstackgerrit has quit IRC02:47
*** openstackgerrit has joined #heat02:48
*** rpothier has joined #heat02:57
*** rpothier has quit IRC03:06
*** KanagarajM has joined #heat03:06
*** morion has quit IRC03:06
*** rpothier has joined #heat03:07
*** om has joined #heat03:13
*** om2 has joined #heat03:13
*** om is now known as Guest7308703:13
*** fragatina has quit IRC03:19
*** sdake has joined #heat03:28
*** dimtruck is now known as zz_dimtruck03:28
*** links has joined #heat03:35
*** randallburt1 has quit IRC03:43
*** tochi has quit IRC03:46
*** tochi has joined #heat03:50
*** sdake has quit IRC03:53
*** cwolferh has quit IRC04:14
*** fragatina has joined #heat04:17
*** fragatina has quit IRC04:22
*** tochi has quit IRC04:24
*** elynn_ has quit IRC04:28
*** bana_k has joined #heat04:33
*** fragatina has joined #heat04:38
*** fragatina has quit IRC04:40
*** fragatina has joined #heat04:40
*** fragatin_ has joined #heat04:41
*** om2 has quit IRC04:42
*** Guest73087 has quit IRC04:42
*** fragatina has quit IRC04:44
*** jaosorior has joined #heat04:50
*** vishwanathj has joined #heat04:51
*** sshnaidm|afk has quit IRC04:51
*** elynn_ has joined #heat04:54
*** tochi has joined #heat04:54
openstackgerritAnant Patil proposed openstack/heat: Enable convergence by default  https://review.openstack.org/32579804:57
*** elynn_ has quit IRC04:58
*** elynn_ has joined #heat04:59
*** sdake has joined #heat05:01
*** mrpointy has quit IRC05:08
*** mrpointy has joined #heat05:08
*** sdake has quit IRC05:32
*** fragatin_ has quit IRC05:36
skraynevtherve: hi. sure, I will take a look.05:36
skraynevcurrently it is similar on some issue from infra side, but I need to ask guys from rally before05:37
skraynevthank you for the pointing on it05:37
*** jtomasek has quit IRC06:01
*** KanagarajM has quit IRC06:02
*** rcernin has joined #heat06:12
*** lifeless has quit IRC06:13
therveskraynev, Thanks!06:20
*** om has joined #heat06:21
*** om2 has joined #heat06:21
*** om is now known as Guest4821006:21
*** vishwanathj has quit IRC06:36
*** vishwanathj has joined #heat06:36
*** cwolferh has joined #heat06:39
*** Guest48210 has quit IRC06:44
*** om2 has quit IRC06:44
*** om2 has joined #heat06:44
*** om3 has joined #heat06:45
*** om3 has quit IRC06:46
*** wangbo has quit IRC06:46
*** jpeeler has quit IRC06:47
*** openstackgerrit has quit IRC06:48
*** openstackgerrit has joined #heat06:49
*** wangbo has joined #heat06:50
AJaegerthere's a project-config change for heat, I'd like some heat cores or the PTL (hi therve ) to +1 this before we merge it: https://review.openstack.org/31781706:50
*** bana_k has quit IRC06:51
therveAJaeger, Fuzzy on the details, but shouldn't https://review.openstack.org/#/c/317599/ land first?06:51
AJaegertherve: I wasn't aware of that change ;(06:53
AJaegertherve: good that I asked here ;)06:53
AJaegertherve: I suggest you -1 and ask for a depends-on so that we merge in the proper order06:53
therveAJaeger, Thanks, sound like a good idea06:53
*** sdake has joined #heat06:54
*** jpeeler has joined #heat06:55
*** jpeeler has quit IRC06:55
*** jpeeler has joined #heat06:55
* AJaeger changed his +2 to a -1 now as well...06:55
*** sorantis has joined #heat06:56
*** priteau has joined #heat06:59
*** sdake has quit IRC07:10
skraynevtherve , AJaeger: heh. I wanted to suggest the same about Depends-On. It will be the best confirmation of correct work07:13
skraynevfor devstack plugin in Heat tree07:13
therveskraynev, unfortunately that won't really test that it works07:14
therveBut yeah :)07:14
AJaegerIt will not test that it works. I overlooked one change, proper order is:07:14
AJaegerLand 317599, land https://review.openstack.org/#/c/317811, and then 317817 - with some time between 811 and 817.07:15
openstackgerritAnant Patil proposed openstack/heat: Don't hard code subnet  https://review.openstack.org/32634107:15
*** zeih has joined #heat07:15
AJaegerSo, I suggest to WIP 817 for now until 811 proves to be stable07:15
AJaegerBut first 317599 needs to merge and 811 (and that one should have the depends-on)07:15
AJaegerIs Di XiaoLi around?07:16
*** vincentll has joined #heat07:16
*** dixiaoli has joined #heat07:17
dixiaolihi AJaeger07:18
AJaegerdixiaoli: can you read scrollback on eavesdrop for the last 20 mins?07:18
dixiaoliAJaeger, OK07:19
AJaegerThis is regarding your devstack heat changes - I've advised the way forward...07:19
* AJaeger just commented in reviews as well07:19
AJaegerdixiaoli: If you have questions on my suggestion, please ask. It'S crucial to have the dpeends in so that changes merge in the right order, otherwise you break all of heat testing ;(07:20
*** ricolin has quit IRC07:21
dixiaoliAJaeger, Thanks, I will update my patch soon and add the Depends-On07:21
*** ananta has joined #heat07:26
*** elynn_ has quit IRC07:26
*** elynn_ has joined #heat07:28
*** KanagarajM has joined #heat07:29
*** ccamacho has joined #heat07:32
*** ricolin has joined #heat07:34
*** shardy has joined #heat07:39
*** evilt0ne has quit IRC07:43
*** evilt0ne has joined #heat07:45
*** evilt0ne has joined #heat07:45
*** mrpointy has quit IRC07:45
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #heat07:49
*** evilt0ne has quit IRC07:50
*** evilt0ne has joined #heat07:51
*** evilt0ne has joined #heat07:51
openstackgerritRico Lin proposed openstack/heat: DNM Test patch  https://review.openstack.org/32688907:52
*** sorantis has quit IRC07:57
*** sorantis has joined #heat08:01
openstackgerrithuangtianhua proposed openstack/heat: Add aodh client plugin  https://review.openstack.org/32504408:03
openstackgerrithuangtianhua proposed openstack/heat: Migrate to aodh for OS::Ceilometer::Alarm  https://review.openstack.org/32689608:03
*** jtomasek has joined #heat08:05
*** om2 has quit IRC08:07
*** om has joined #heat08:07
*** om is now known as Guest3569008:07
*** Guest35690 has quit IRC08:08
*** om2 has joined #heat08:08
*** om3 has joined #heat08:08
*** om3 has quit IRC08:11
*** vishwanathj has quit IRC08:14
*** jaosorior has quit IRC08:25
*** jaosorior has joined #heat08:25
openstackgerrithuangtianhua proposed openstack/heat: Use to_dict() instead for cinder resources  https://review.openstack.org/32691508:31
*** om2 has quit IRC08:32
openstackgerritRico Lin proposed openstack/heat: DNM Test patch  https://review.openstack.org/32688908:32
*** tlashchova has joined #heat08:35
*** dmk0202 has joined #heat08:37
*** derekh has joined #heat08:42
*** yanyanhu has quit IRC08:43
*** yassine has joined #heat08:44
anantaHi folks! Should I be creating issue for something like https://review.openstack.org/#/c/326341 ?08:45
*** openstackgerrit has quit IRC08:48
therveananta, Nope08:48
*** openstackgerrit has joined #heat08:49
anantatherve: ok, thanks!08:49
*** vishwanathj has joined #heat08:55
*** AJaeger has left #heat08:59
*** sorantis has quit IRC09:02
*** jtomasek has quit IRC09:04
*** om2 has joined #heat09:10
*** om3 has joined #heat09:14
*** om2 has quit IRC09:15
openstackgerrithuangtianhua proposed openstack/heat: Add a Macro class to heat.engine.function  https://review.openstack.org/32224809:15
*** sdake has joined #heat09:16
*** jtomasek has joined #heat09:18
*** wangbo has quit IRC09:22
openstackgerrithuangtianhua proposed openstack/heat: Provides 'if' intrinsic function  https://review.openstack.org/28347909:28
*** wangbo has joined #heat09:34
openstackgerrithuangtianhua proposed openstack/heat: Provides 'if' intrinsic function  https://review.openstack.org/28347909:34
*** sorantis has joined #heat09:44
openstackgerritMerged openstack/heat: Don't hard code subnet  https://review.openstack.org/32634109:45
skraynevtherve: so guys from infra team guide me what need to do, I hope to upload patch on review today or tomorrow. When it be ready I will ping you.09:45
skraynevabout rally periodic job09:46
*** sdake has quit IRC09:49
therveskraynev, OK awesome09:49
*** tlashchova has quit IRC09:49
openstackgerrithuangtianhua proposed openstack/heat: Support resource/output condition  https://review.openstack.org/28336610:02
*** vishwanathj has quit IRC10:08
*** jamie_h has joined #heat10:12
*** sorantis has quit IRC10:15
*** tlashchova has joined #heat10:22
openstackgerritRico Lin proposed openstack/heat: DNM Test patch  https://review.openstack.org/32688910:23
*** om3 has quit IRC10:25
*** prazumovsky has joined #heat10:27
shardytherve: Hey, would be interested in your feedback re the ML thread I started re environments when you get a moment10:28
shardybasically I'm thinking of adding several things to the environment, and wondering about backwards compatibility10:28
*** elynn_ has quit IRC10:29
shardyI guess we actually have added a bunch of stuff (hooks, event sinks etc) fairly recently, so I'm inclined to just hack on the patches and add the stuff10:30
*** ricolin has quit IRC10:30
prazumovskytherve, hi! As I remember, you're debugged improve translations (https://review.openstack.org/#/c/286098/), is there any progress? Sorry for bothering.10:34
*** dixiaoli has quit IRC10:34
*** lifeless has joined #heat10:35
*** vincentll has quit IRC10:38
*** lifeless has quit IRC10:47
*** lifeless_ has joined #heat10:47
*** d0ugal has quit IRC10:59
*** d0ugal has joined #heat11:00
*** afazekas is now known as __afazekas11:05
*** d0ugal has quit IRC11:09
*** KanagarajM has quit IRC11:09
*** KanagarajM_ has joined #heat11:09
therveprazumovsky, Rabi already mentioned that the issue was unrelated11:10
openstackgerritSteven Hardy proposed openstack/heat-templates: Add resource_group index lookup example  https://review.openstack.org/30162311:12
prazumovskyI know:) In previous time I asked you after learning Rabi's comment11:13
Abluwhat is the default way to develop a standalone heat resource plugin? I mean how do I get code completion for the heat modules? It works within the heat repository of course, but outside of it I find no installable heat version...11:14
*** KanagarajM_ has quit IRC11:14
*** KanagarajM__ has joined #heat11:14
Abluor do i need to install heat on my development machine too?11:14
therveprazumovsky, Well if you know why do you ask? :)11:14
shardyAblu: https://github.com/openstack/heat/blob/master/heat/common/config.py#L9211:15
prazumovskyI thought, that you have deeper investigation of this patch:)11:15
therveAblu, I don't know what you use for code completion, but I can't see how it would complete without heat installed11:16
shardyyou need to install the plugin somewhere heat looks for it, or modify plugin_dirs in heat.conf so it searches wherever you've installed it11:16
*** KanagarajM__ has quit IRC11:16
*** KanagarajM_ has joined #heat11:16
shardyAblu: you need access to wherever the heat service is running11:16
prazumovskytherve, shardy, ananta, stevebaker, KanagarajM_ and others: Honestly, I just want some review from community, so could you take a look https://review.openstack.org/#/c/286098/11:17
Ablutherve: i use pycharm.11:17
* shardy possibly misinterpreted Ablu's question11:17
Ablui know how to run the plugin. I only ask about a development setup11:18
openstackgerritPeter Razumovsky proposed openstack/heat: Fix creating SDGroup with no properties  https://review.openstack.org/32700611:18
Ablui have a working test environment, but that is on a server, not on my dev machine11:18
therveshardy, I've seen your mail yeah. I think adding sections is okay. Not sure what we could do better anyway11:18
shardyAblu: ah, sorry, I can't help, I use vim ;)11:18
therveLOL11:18
Abluheh :P11:18
Abluwell i could live without code completion / code checking... just asking11:18
shardytherve: cool, thanks - I kinda wonder if we might want to introduce versioning at some point11:19
shardye.g like (or exactly the same as) the hot version11:19
therveYeah11:19
therveWell versioning matters when you have different behavior of the same functions11:19
therveFor new stuff it's not that useful11:19
shardyYeah, I guess in this case it should fail in a fairly obvious way, but rejecting the new interfaces on old heat's11:20
shardys/but/by11:20
therveTying the env version to the template version would make some sense11:20
therveNot sure we'd want to specify it again11:21
shardytherve: Yeah, but then what happens when parent/nested stacks specify different versions11:21
shardyI suppose we could pick the newest11:21
therveEnv is tied to the top level stack, no?11:22
*** ananta has quit IRC11:23
shardytherve: well it's passed to all the stacks, and the top-level stack might not have the current/latest hot version specified11:23
shardyyou only really need to bump the version in templates where e.g you want to use some new function11:23
therveprazumovsky, Could skraynev or ochuprykov have a look?11:24
shardyI guess for now documenting the versions clearly in the environment docs, and ensuring sane failure messages on old versions should be OK11:24
therveDon't want to push the potato around, but that patch is really super tricky11:24
*** sorantis has joined #heat11:24
therveshardy, Yeah we're pretty strict on section available, so I think it should be okay11:24
shardytherve: ack, thanks for the feedback, I'll hack on some patches and see what folks say11:25
*** tochi has quit IRC11:26
prazumovskyI understand, I ping them, but I pinged them for several months with no response:)11:27
*** vincentll has joined #heat11:29
*** d0ugal has joined #heat11:32
*** om has joined #heat11:32
*** om is now known as Guest9623911:32
*** ccamacho is now known as ccamacho|lunch11:34
thervehuangtianhua, Another thing about that condition patch. I think we tried to be a bit more strict to make sure it only exists on the latest template version11:37
therveRight now I believe your patch makes it available everywhere11:37
thervehuangtianhua, See https://review.openstack.org/#/c/135492/ for example11:39
*** sdake has joined #heat11:44
*** __afazekas is now known as afazekas11:49
openstackgerritSteven Hardy proposed openstack/heat: Add stack files retrieval API  https://review.openstack.org/32704411:57
shardyjdob: ^^ FYI11:58
*** zeih has quit IRC11:59
*** akuznetsov has joined #heat12:00
*** mrpointy has joined #heat12:01
*** vincentll has quit IRC12:05
*** rpothier has quit IRC12:06
*** ccamacho|lunch is now known as ccamacho12:06
*** vincentll has joined #heat12:10
shardyAnyone know how to get python-heatclient install from source to register new osc plugins?12:11
therveshardy, pip install -e .12:11
shardyI've added it to setup.cfg, run python setup.py install, but it's not seen by openstackclient (or any of the new plugins added in master)12:11
shardytherve: aha, thanks12:12
*** akuznetsov has quit IRC12:12
*** sdake has quit IRC12:13
openstackgerritMerged openstack/heat-templates: Add resource_group index lookup example  https://review.openstack.org/30162312:15
*** dprince has joined #heat12:15
*** sshnaidm|afk has joined #heat12:21
*** jcoufal has joined #heat12:23
skraynevtherve: yes ;) I can do it ;)12:23
therveyay12:24
openstackgerritMerged openstack/heat: Make sure glance image create success with tags  https://review.openstack.org/32631012:26
*** zeih has joined #heat12:27
*** pradk has quit IRC12:30
*** jdob has quit IRC12:32
*** zeih has quit IRC12:32
*** jdob has joined #heat12:33
*** fultonj has quit IRC12:36
*** fultonj has joined #heat12:37
*** dyasny has quit IRC12:40
*** vishwanathj has joined #heat12:41
openstackgerritMerged openstack/heat: Ignore notfound error when delete image tags  https://review.openstack.org/32635012:46
AbluI now got this hot file: http://sprunge.us/MWUK and this resource plugin: http://sprunge.us/JAFe. However if I now try to create a stack with that I get: "ERROR: Neither image nor bootable volume is specified for instance my_instance"12:49
AbluIf I however use the line I commented in the hot file it works12:50
Ablu the string is the same, so what is the difference?12:50
*** jdob has quit IRC12:50
shardyAblu: I'd add an output to prove the get_att actually works and isn't returning None12:50
Ablushardy: can i simply print?12:51
*** jdob has joined #heat12:51
*** rpothier has joined #heat12:51
shardyhttp://paste.openstack.org/show/508919/12:52
shardyAblu: No, something like that12:52
shardythen use heat stack-show to look at the "image" output12:52
Abluah12:52
Abluone sec...12:52
Ablu"output_value": null,12:54
Ablushardy: hm ok.12:54
Abluit does not seem to be set. But thanks for the output trick. Did not know that. Really useful12:54
shardyAblu: I think you need to implement the _resolve_attribute function in your plugin12:56
shardyjust setting the data isn't enough for it to resolve12:56
Abluhm i tried to copy that one from the random_string resource before...12:56
Ablulet me try again12:56
Ablushardy: ah. that worked. Thanks12:57
AbluI guess I somehow did not restart or tried too many things at once before...12:58
shardyDoes anyone happen to know how to drive the cliff yaml formatter so it doesn't corrupt the output with Field: Value: stuff?13:02
Ablushardy: hm. When I left out the Nova::Server part it worked, but if I add that again I run into "ERROR: Neither image nor bootable volume is specified for instance my_instance" again. If i comment out the Server again I still get the right result: http://sprunge.us/JSQY13:02
*** zeih has joined #heat13:03
*** links has quit IRC13:03
*** priteau has quit IRC13:14
*** priteau has joined #heat13:15
*** Guest96239 has quit IRC13:16
*** links has joined #heat13:19
*** Guest96239 has joined #heat13:19
*** zz_dimtruck is now known as dimtruck13:22
Ablui tried to add some logging to a file... it does not look like the plugin is called at all...13:22
*** Drago has joined #heat13:22
*** Drago has quit IRC13:22
*** Drago has joined #heat13:23
*** vincentll has quit IRC13:23
*** vincentll has joined #heat13:23
Abluonly the mapping function is being called13:23
*** dimtruck is now known as zz_dimtruck13:34
*** blues-man has joined #heat13:36
*** zz_dimtruck is now known as dimtruck13:36
*** pradk has joined #heat13:41
*** mrpointy has quit IRC13:48
*** jdob has quit IRC13:49
*** jdob has joined #heat13:49
*** jdob has quit IRC13:59
*** jdob has joined #heat13:59
*** dimtruck is now known as zz_dimtruck14:03
*** prazumovsky has quit IRC14:03
*** jasond has joined #heat14:04
*** ricolin has joined #heat14:05
*** zz_dimtruck is now known as dimtruck14:05
*** p01nt3r75 has joined #heat14:05
*** spotz_zzz is now known as spotz14:12
*** zeih has quit IRC14:13
*** dimtruck is now known as zz_dimtruck14:16
jdandreaReality check: With an AutoScalingGroup of VMs, each of them reporting stats to Ceilometer, a CPU utilization alarm is triggered if the avg measurement of ALL VMs hits a threshold, not if any one group member hits it. (Thinking yes?)14:18
*** EricGonczer_ has joined #heat14:19
jdandreaActually, with Gnocchi and aggregation_method we could look at the last CPU, so ... :)14:19
*** ramishra_ has quit IRC14:19
*** ramishra has joined #heat14:20
*** jcoufal has quit IRC14:25
*** jaosorior has quit IRC14:26
*** ananta has joined #heat14:26
*** pt_15 has joined #heat14:28
*** spzala has joined #heat14:28
*** jasond has quit IRC14:29
*** links has quit IRC14:29
*** jasond has joined #heat14:30
*** rajpatel has joined #heat14:30
*** p01nt3r75 has quit IRC14:32
*** zeih has joined #heat14:37
therveMeeting in 20 in #openstack-meeting-414:40
therveAgenda at https://wiki.openstack.org/wiki/Meetings/HeatAgenda#Agenda_.282016-06-08_1500_UTC.29 if you want to add stuff14:40
shardyjdob: Hey I just posted https://review.openstack.org/327149 which changes the output format for stack environment show14:43
Abluok. I tried to make my heat-engine remote debuggable with pycharm... It kind of works. But somehow break points in the resource plugins do not fire... Is there any guide available about how to debug heat?14:43
shardywould be interested in your feedback - IMO it makes it much more usable14:43
jdobcool, will look in a bit14:46
jasondAblu: try adding "num_engine_workers = 1" in the DEFAULT section of /etc/heat/heat.conf14:49
*** zeih has quit IRC14:49
*** KanagarajM_ has quit IRC14:49
Ablujasond: hm... it still does not break14:51
Ablui guess i will try to break higher and try to step into14:51
jasondAblu: does it work locally?14:51
Ablumight take a while since i have zero clue about the code14:51
Ablujasond: i do not have heat installed locally14:51
jasondAblu: i'd start by installing heat via devstack, getting the breakpoints to work locally, *then* testing remotely14:52
anantaAblu: Not sure if you know this or even if it will work for you; but have you tried setting gevent compatible debugging in pycharm?14:54
*** thumpba has joined #heat14:54
Ablujasond: hm...14:54
Abluananta: not sure what gevent is...?14:54
anantaAblu: I used to use pycharm 2 years back and I vaguely remember doing something around that and it worked14:54
jasondAblu: (locally, as in a box you can log into where heat is running)14:54
anantathere should be an option in python debugger settings in pycharm14:55
*** randallburt has joined #heat14:55
*** zhurong has joined #heat14:55
*** jcoufal has joined #heat14:55
jasondananta: hi, would you mind taking a look at https://review.openstack.org/#/c/324898/ ?14:56
Abluok I will first try to find that setting14:56
anantajasond: sure14:56
*** ochuprykov has joined #heat14:56
jasondananta: thanks14:57
Abluotherwise I will debug on a higher level first and try to figure out why the lower breakpoints do not work. Otherwise I will try devstack on the same machine...14:57
anantaAblu: life would be easier if you just use pdb with vim ;)14:57
*** links has joined #heat14:57
jasondananta: long story short, this template results in a stack that cannot be updated http://paste.openstack.org/show/507328/  the convergence fix was pretty involved (for the non-convergence fix, see patchset 1)14:58
Ablui tried to use pdb. but i have no idea how to set breakpoints there...14:58
AbluI only know gdb :/14:58
Abluwell i know how to set break points, but it keeps telling me the files do not exist :P14:58
anantajasond: ok, I will review it14:58
*** ochuprykov has quit IRC15:00
Abluif anyone by the way finds the reason why the plugin is not working I mentioned earlier today... Please let me know :P Then I do not need to do this debugging :P15:01
*** Wenzhi has quit IRC15:01
therveshardy, skraynev, meeting?15:01
*** Wenzhi has joined #heat15:02
shardytherve: joining, thanks15:02
*** zhurong has quit IRC15:02
*** zz_dimtruck is now known as dimtruck15:03
jdandreaNot seeing anyone in #openstack-meeting-4 ... hmm.15:05
jdandreanm, Textual is being silly.15:05
*** blues-man has quit IRC15:05
Abluananta: well pdb with vim does not break either :P15:06
Abluargh... I am stupid. My devstack of course is not equal to the local git code :(15:09
*** Guest96239 has quit IRC15:12
*** dprince has quit IRC15:13
*** dyasny has joined #heat15:14
*** sorantis has quit IRC15:18
*** vijendar has joined #heat15:18
*** blues-man has joined #heat15:21
*** akshai has quit IRC15:21
*** akshai has joined #heat15:22
*** Guest96239 has joined #heat15:23
*** randallburt has quit IRC15:27
*** dmk0202 has quit IRC15:29
*** rajpatel1 has joined #heat15:31
*** rajpatel has quit IRC15:33
*** mohankumar_ has joined #heat15:33
*** randallburt has joined #heat15:38
*** randallburt1 has joined #heat15:40
*** randallburt has quit IRC15:42
*** zeih has joined #heat15:43
*** vincentll has quit IRC15:43
*** zeih has quit IRC15:43
*** zeih has joined #heat15:44
*** blues-man has quit IRC15:45
*** jasond has quit IRC15:48
*** zeih has quit IRC15:48
thervecwolferh, I'll have a look. But this test *is* extremely useful, as we broke purge time and time again in the past and functional tests are the only way to actually verify that it works okay15:56
zanebcwolferh: do you have a theory of how an event referencing the props data is being added while we are in the process of purging a stack>15:56
zaneb?15:56
*** sabeen has joined #heat15:57
cwolferhno, that is the theory, i'm not sure how the extra event is added15:58
*** chlong has quit IRC15:58
cwolferh(by some external test doing its own deletion)15:58
zanebcwolferh: IIUC the event has to be associated with a resource (in order to be using this table). I'm struggling to think of a way that could get added after the stack is already marked DELETE_COMPLETE15:59
cwolferhactually...16:00
cwolferheh, nevermind :-)16:01
cwolferhif DELETE_COMPLETE is a kind of flag i can go off of, maybe i should make sure all resources and events have that?16:02
cwolferhotherwise, don't touch the stack yet?16:02
*** Guest96239 has quit IRC16:05
*** jasond has joined #heat16:06
zanebcwolferh: I guess, but we should be checking the stack is DELETE_COMPLETE before purging it anyway, right?16:09
openstackgerritSteven Hardy proposed openstack/python-heatclient: Add support for stack files show  https://review.openstack.org/32720516:11
*** ananta has quit IRC16:12
cwolferhzaneb, hmm, maybe we should be, but we are just checking stack.c.deleted_at < time_line16:12
zanebah, ok. should amount to the same thing though16:12
zanebpresumably we don't set deleted_at at least until we mark it DELETE_COMPLETE16:13
AbluOk. Got lost during debugging the GetAddr... something goes weird there... will continue at home...16:14
*** links has quit IRC16:15
zanebhmm, interestingly only stack is soft-deleted. resources are hard-deleted immediately16:17
* cwolferh nods16:19
*** jtomasek has quit IRC16:20
cwolferhi'm thinking it may be as easy as check as for DELETE_COMPLETE in the stack16:21
shardyjasond: Hey, can you help me with some details re encrypted_param_names?16:24
shardyI see it's stored in the environment, and it's returned via the environment show API call, but the client doesn't allow passing that section16:24
shardywhere do we set that data?16:24
jasondshardy: sure, let me check16:24
cwolferhsince  this line would set the stack's deleted_at https://review.openstack.org/gitweb?p=openstack/heat.git;a=blob;f=heat/engine/stack.py;h=28f146e46e4d9a95437d75df98564cbe35eb72fa;hb=HEAD#l1711 for a non-FAILED status (but possibly non-COMPLETE as well)16:25
shardyit looks like it's created internally derived from the hidden flag on a parameter, but I can't see where16:25
zanebcwolferh: I doubt it's that easy. http://git.openstack.org/cgit/openstack/heat/tree/heat/engine/stack.py#n1702 <- DELETE_COMPLETE happens before stack_object.Stack.delete()16:25
shardyand if that's the case, I'm not quite clear why it's stored with the environment16:25
jasondshardy: https://github.com/openstack/heat/blob/master/heat/objects/raw_template.py#L9916:25
shardyjasond: aha, thanks, missed that16:26
jasondshardy: np16:26
shardyso, really it's an internal thing, and we shouldn't be exposing that to the user?16:26
shardyjasond: I'm thinking at least we should pop it from the environment returned via environment show, as it can never be passed back in via the API (or, actually, can it?)16:27
shardyheatclient definitely prevents it in environment_format, but it may actually be possible to pass it in via not-heatclient16:27
shardyI imagine that might yield unpredictable results ;)16:27
*** sabeen has quit IRC16:28
jasondshardy: it can't be passed.  i would leave it out of environment show16:28
*** Guest96239 has joined #heat16:28
jasondshardy: at least i don't think it can16:28
*** sabeen has joined #heat16:28
shardyenvironment_format.py inside heat allows it, I think it might be allowed16:28
* shardy tries16:29
cwolferhzaneb, ah, right16:29
* cwolferh now casts a suspicious eye on the ROLLBACK_COMPLETE case16:29
jasondshardy: there's no reason it has to be stored there16:30
shardyjasond: yeah I was thinking the same16:30
shardyI'll pop it from the API response for now, then we can consider if we should migrate it elsewhere16:30
shardyjasond: thanks for the info :)16:31
jasondshardy: sounds good.  anytime16:31
*** bana_k has joined #heat16:31
*** dyasny has quit IRC16:32
*** d0ugal has quit IRC16:33
*** sabeen has quit IRC16:36
*** ricolin has quit IRC16:36
openstackgerritSteven Hardy proposed openstack/heat: Add stack files retrieval API  https://review.openstack.org/32704416:37
*** sorantis has joined #heat16:38
*** dprince has joined #heat16:41
*** rcernin has quit IRC16:42
*** mohankumar_ has quit IRC16:42
*** d0ugal has joined #heat16:49
*** evilt0ne has quit IRC16:51
openstackgerritMerged openstack/heat: Enable convergence by default  https://review.openstack.org/32579816:57
DragoHUZZAH16:58
*** sorantis has quit IRC16:59
*** sabeen has joined #heat17:00
openstackgerritSteven Hardy proposed openstack/heat: Add stack files retrieval API  https://review.openstack.org/32704417:00
*** fragatina has joined #heat17:03
*** fragatin_ has joined #heat17:04
*** thumpba has quit IRC17:04
*** thumpba has joined #heat17:05
*** fragatina has quit IRC17:08
*** derekh has quit IRC17:08
*** racedo has joined #heat17:10
*** sdake has joined #heat17:10
*** vishwanathj has quit IRC17:11
*** vijendar has quit IRC17:15
*** Guest96239 has quit IRC17:16
*** thumpba has quit IRC17:20
*** vijendar has joined #heat17:20
*** thumpba has joined #heat17:21
*** dprince has quit IRC17:22
*** sorantis has joined #heat17:22
*** dprince has joined #heat17:23
*** thumpba has quit IRC17:23
*** thumpba has joined #heat17:25
*** dimtruck is now known as zz_dimtruck17:30
*** zz_dimtruck is now known as dimtruck17:33
*** racedo has quit IRC17:37
*** sabeen has quit IRC17:41
*** bana_k has quit IRC17:43
*** yassine has quit IRC17:45
*** sorantis has quit IRC17:46
openstackgerritRico Lin proposed openstack/heat: DNM Test patch  https://review.openstack.org/32688917:51
*** sdake has quit IRC17:51
*** DevBox has joined #heat17:52
*** bana_k has joined #heat17:54
*** sabeen has joined #heat17:54
*** sabeen has quit IRC18:01
*** sabeen has joined #heat18:01
*** priteau has quit IRC18:05
*** priteau has joined #heat18:05
openstackgerritRabi Mishra proposed openstack/heat: Remove multiple use of CooldownMixin  https://review.openstack.org/29402318:06
*** jamie_h has quit IRC18:07
-openstackstatus- NOTICE: Zuul has been restarted to correct an error condition. Events since 17:30 may have been missed; please 'recheck' your changes if they were uploaded since then, or have "NOT_REGISTERED" errors.18:09
*** fragatin_ has quit IRC18:14
*** dyasny has joined #heat18:16
*** fragatina has joined #heat18:19
openstackgerritSteven Hardy proposed openstack/heat: Keep encrypted_param_names environment internal to heat  https://review.openstack.org/32727518:20
shardyjasond: ^^ FYI18:20
jasondshardy: i'll review it.  thanks18:21
openstackgerritJason Dunsmore proposed openstack/heat: Add default release names for template versions  https://review.openstack.org/31474118:22
jasonddang you gerrit.  my session expired and i lost my update18:28
*** links has joined #heat18:33
*** zeih has joined #heat18:35
*** sdake has joined #heat18:37
*** racedo has joined #heat18:39
Dragojasond: It'd be nice if the session would last longer than 2 hours18:43
jasondDrago: +118:44
*** fragatina has quit IRC18:48
*** Kennan has quit IRC18:52
*** Kennan has joined #heat18:52
*** priteau has quit IRC18:57
Abluso. http://sprunge.us/XBZK. This seems to be why it keeps returning None while evaluating the attribute18:57
Ablur is a dangerous name btw :P I accidently jumped to the return twice before getting there while trying to evaluate variables :P18:58
Ablubut... why is r.action INIT?18:58
Abluthis code is called during validation of the OS::Nova::Server resource18:59
Ablumy own plugin should already be evaluated at that point, right? Otherwise it could not verify the name i am returning from that one, right?19:00
*** jtomasek has joined #heat19:01
Abluhm though: (Pdb) print r.FnGetAtt(attribute, *path_components)19:01
AbluNone19:01
Abluso apparently both if branches would return None...19:01
*** alex_xu has quit IRC19:02
*** alex_xu has joined #heat19:06
*** harlowja has quit IRC19:12
*** harlowja has joined #heat19:15
*** rcernin has joined #heat19:25
*** rajpatel1 has quit IRC19:25
*** links has quit IRC19:31
*** mrpointy has joined #heat19:34
*** rajpatel has joined #heat19:46
*** spzala has quit IRC19:46
*** sdake_ has joined #heat19:47
*** sdake has quit IRC19:48
*** dimtruck is now known as zz_dimtruck19:54
*** vijendar has quit IRC19:56
*** zz_dimtruck is now known as dimtruck19:57
*** cllewellyn_ has joined #heat20:00
*** vijendar has joined #heat20:01
*** EricGonczer_ has quit IRC20:06
*** cllewellyn_ has quit IRC20:12
*** EricGonczer_ has joined #heat20:14
zanebAblu: during validation no resources have been created yet, so get_attr always returns None20:16
zanebAblu: we talked at summit about making it return some kind of placeholder instead, which will be much better. I think folks are working on that20:17
Abluzaneb: how can I pass in an image name over the attribute of a different resource then?20:17
AbluValidation would always fail for that, or am I missing something?20:18
zanebno, I suspect you're right, it's always going to fail validation20:20
AbluHm ok. Well at least we got that sorted out :p20:20
AbluIs there any alternative?20:21
zanebwhen we have the placeholder thing, we'll return a placeholder object that can report its type as "glance.image", and that will make the validation very happy indeed20:21
AbluOr should I file a bug? (If there isn't one already)20:21
zanebAblu: I think you should file a bug. It may be hard to fix, but it definitely seems wrong20:21
AbluOk. I will do it on the following days. I am in bed already20:22
AbluThanks a lot and good night!20:22
zanebAblu: the problem is that you need either an image or a bootable volume20:23
zaneband it's looking at the resolved value to see which one you specified20:23
zanebif both come out as none, it looks like neither20:23
zanebbut it should look at the initial data and see that there is property there, even if it resolves to None during validation20:24
AbluYeah, I debugged so far. But the resolve code was a bit hard to understand to me20:24
AbluWell not the code but the logic20:24
*** spzala has joined #heat20:25
*** racedo has quit IRC20:25
*** spzala has quit IRC20:26
*** spzala has joined #heat20:26
*** thumpba has quit IRC20:27
*** thumpba has joined #heat20:29
*** thumpba has quit IRC20:33
*** thumpba has joined #heat20:43
*** spzala has quit IRC20:45
*** Drago has quit IRC20:45
*** cllewellyn_ has joined #heat20:45
*** dimtruck is now known as zz_dimtruck20:46
*** zeih has quit IRC20:47
*** openstackgerrit has quit IRC20:48
*** openstackgerrit has joined #heat20:49
*** thumpba has quit IRC20:56
*** fragatina has joined #heat21:01
*** sdake_ has quit IRC21:01
*** cwolferh has quit IRC21:02
*** rpothier has quit IRC21:05
*** vijendar has quit IRC21:06
*** chlong has joined #heat21:06
*** cwolferh has joined #heat21:09
*** EricGonczer_ has quit IRC21:18
*** Drago has joined #heat21:19
*** lifeless_ is now known as lifeless21:30
*** DevBox has quit IRC21:40
*** sdake has joined #heat21:49
*** jcoufal has quit IRC21:51
*** rcernin is now known as rcernin|off21:54
*** sdake has quit IRC21:54
*** sdake has joined #heat21:55
openstackgerritOpenStack Proposal Bot proposed openstack/heat: Updated from global requirements  https://review.openstack.org/32736521:57
*** jasond has quit IRC21:58
*** dprince has quit IRC22:01
*** Drago has quit IRC22:03
*** cwolferh has quit IRC22:03
*** Drago has joined #heat22:03
*** Drago has quit IRC22:04
*** Drago has joined #heat22:04
*** cwolferh has joined #heat22:10
*** Drago has quit IRC22:10
*** spotz is now known as spotz_zzz22:13
*** Drago has joined #heat22:17
*** rcernin|off has quit IRC22:21
*** cllewellyn_ has quit IRC22:24
*** zz_dimtruck is now known as dimtruck22:35
*** rajpatel has quit IRC22:46
*** randallburt1 has quit IRC22:54
*** pradk has quit IRC22:54
*** kmARC has joined #heat23:07
*** sabeen has quit IRC23:07
*** kmARC has quit IRC23:08
*** kmARC has joined #heat23:09
*** kmARC has left #heat23:09
*** steveg_afk has quit IRC23:11
*** bnemec has quit IRC23:11
*** bnemec has joined #heat23:19
*** Drago has quit IRC23:21
*** kmARC has joined #heat23:23
*** kmARC has left #heat23:23
*** Drago has joined #heat23:36
*** Drago has quit IRC23:36
*** Drago has joined #heat23:36
*** tochi has joined #heat23:42

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