Thursday, 2014-06-12

*** sjmc7 has quit IRC00:02
SpamapSstevebaker: cores can resurrect abandoned changes IIRC00:03
stevebakeroh, I see a restore button00:04
stevebakerSpamapS: I can replicate the lock issue locally, bisecting now00:04
*** bgorski has quit IRC00:08
SpamapSstevebaker: woot00:08
SpamapSstevebaker: you know, thinking more about it, I think if we succeed in setting user_creds_id = NULL when the user_creds are gone, that will actually fix the gate problem.00:09
SpamapSstevebaker: but the bigger problem is that we are likely interrupting deletes wrong.00:09
stevebakerSpamapS: I'll test that once I'm done bisecting00:09
*** jreypo has quit IRC00:10
stevebakerapparently this is the culprit https://review.openstack.org/#/c/97569/00:10
*** dims__ has quit IRC00:11
*** dims has joined #heat00:11
SpamapSstevebaker: that is.. odd00:14
SpamapSstevebaker: is it possible that just changed the race winner?00:15
*** Qiming has joined #heat00:16
stevebakerSpamapS: seems likely, doing an unconditional authenticate would hit keystone, which would lead to an eventlet yield00:16
stevebakerSpamapS: your patch doesn't help much, but reverting that change does. I suggest we revert urgently00:17
stevebakerzaneb: ^00:18
*** dims has quit IRC00:21
*** bgorski has joined #heat00:22
SpamapSstevebaker: indeed!00:22
SpamapSstevebaker: and I'm not surprised that the None check doesn't help.. it is just a symptom of the failed race.00:23
openstackgerritSteve Baker proposed a change to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions"  https://review.openstack.org/9951700:23
openstackgerritSteve Baker proposed a change to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions"  https://review.openstack.org/9951700:23
SpamapSstevebaker: can you get rid of that bare space too?00:24
openstackgerritSteve Baker proposed a change to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions"  https://review.openstack.org/9951700:24
stevebakerjogo: https://review.openstack.org/#/c/99517/ is the revert which should fix bug 130602900:24
uvirtbotLaunchpad bug 1306029 in heat "tempest.api.orchestration.stacks.test_stacks times out during tearDownClass" [Critical,Triaged] https://launchpad.net/bugs/130602900:24
*** zhiyan_ is now known as zhiyan00:27
SpamapSstevebaker: IMO, it will _hide_ bug 1306029 .. but yes. :)00:27
uvirtbotLaunchpad bug 1306029 in heat "tempest.api.orchestration.stacks.test_stacks times out during tearDownClass" [Critical,Triaged] https://launchpad.net/bugs/130602900:27
*** matsuhashi has joined #heat00:27
*** dims has joined #heat00:27
*** nosnos has joined #heat00:32
stevebakerSpamapS: yeah, it hides the bug, but we know about it now00:37
stevebakerits still Critical00:38
*** randallburt has joined #heat00:42
*** randallburt has quit IRC00:43
*** Qiming has quit IRC00:43
*** randallburt has joined #heat00:43
*** m_22 has joined #heat00:46
*** liusheng has quit IRC00:51
*** liusheng has joined #heat00:52
*** yanyan has joined #heat00:52
zanebstevebaker: what was the test you used to bisect?00:57
stevebakerzaneb: tempest test_stacks00:58
stevebakerzaneb: testr run --until-failure test_stacks00:58
zanebyou just ran it multiple times?00:58
zanebfor each patch, I mean00:58
stevebakerzaneb: the test doesn't actually fail. I guess the check job fails because of the ERROR entries in the heat-engine log01:01
zaneboh01:01
zanebbut there is a bug where it is actually timing out, right?01:01
zanebdo we think that's a different cause?01:01
stevebakerzaneb: actually I lie, I modified test_stack_crud_no_resources to do 4 deletes in succession. Anything less than 3 doesn't trigger it01:04
jogostevebaker: sweet01:05
stevebakerzaneb: Yeah, I'm not sure what the test timeout failure is, but I'd wager 500ml beer its the same problem01:06
*** randallburt has quit IRC01:14
*** gokrokve has joined #heat01:14
cylistevebaker:  so therve's sync with oslo-incubator also obviates this one, right?  https://review.openstack.org/#/c/77472/ (which just seems to be a sync of the config module)01:15
stevebakercyli: I'm guessing so. You should sync with him when he is awake01:15
cylistevebaker:  ok.  all the other active reviews depend on the previous ones (e.g. https://review.openstack.org/#/c/72798/ depends on https://review.openstack.org/#/c/77472/) - if indeed https://review.openstack.org/#/c/77472/ should be abandoned, do I need to do anything to change the dependencies?01:17
cyli(or will it automatically change if I keep the change ID but rebase https://review.openstack.org/#/c/72798 so it doesn't have the parent commit from https://review.openstack.org/#/c/77472/ ?)01:20
stevebakercyli: you should pull down the whole series and do an interactive rebase to remove the abandoned ones01:20
cylistevebaker:  oh, I see, great.  :)  thank you so much for yours and zaneb's patience explaining everything!01:21
zanebcyli: what stevebaker said, or just cherry-pick the ones you need onto master01:21
zanebthe latter might be easier, because there will no doubt be heaps of conflicts01:22
zanebgit fetch https://review.openstack.org/openstack/heat refs/changes/<random_number>/<review_number>/<patchset_number> && git cherry-pick FETCH_HEAD01:23
cylizaneb:  thanks!01:23
zanebexcept don't actually use a random number, get it from the Dowload section on the review ;)01:24
zanebDownload*01:24
cyli:)  ok, thanks!01:25
zanebif you click on 'cherry-pick' it gives you the command you can copy01:25
*** tiantian has joined #heat01:25
cylioh nice01:25
*** yanyan has quit IRC01:27
zaneboh, the random number is the last two digits of the review id01:29
zanebI guess that makes sense for distributing them evenly01:29
*** gokrokve has quit IRC01:30
*** Qiming has joined #heat01:30
*** nati_ueno has quit IRC01:30
openstackgerritYing Li proposed a change to openstack/heat: This is part of the refactoring to fix the  inheritance hierarchy of AutoScaleResourceGroup. In this case, attempting to move some common functionality out of the inheritance hierarchy entirely (perhaps to the heat/scaling module).  https://review.openstack.org/9947001:30
*** gokrokve has joined #heat01:30
*** randallburt has joined #heat01:31
*** Qiming has quit IRC01:32
zanebcyli: btw that ^ commit is missing a headline, which is why pep8 is failing it. https://wiki.openstack.org/wiki/GitCommitMessages is a really helpful page to read01:33
zanebeveryone involved in OpenStack should read that wiki page once a month imo ;D01:34
*** gokrokve has quit IRC01:35
*** m_22 has quit IRC01:35
cylizaneb:  oh ok!  I thought it was just that the commit message line length was too long (as per the pep8 failure).  Thanks for the link!01:36
*** ericgoncz has joined #heat01:38
*** alexheneveld has quit IRC01:39
*** zns has quit IRC01:39
*** DandyPandy has joined #heat01:39
*** ericgoncz_ has joined #heat01:40
*** DandyPandy has quit IRC01:40
*** DandyPandy has joined #heat01:41
*** rwsu has quit IRC01:42
*** ericgoncz has quit IRC01:44
*** ericgoncz_ is now known as ericgoncz01:44
*** bgorski has quit IRC01:44
openstackgerritYing Li proposed a change to openstack/heat: Move calculating new capacity to a function  https://review.openstack.org/9947001:46
*** alexheneveld has joined #heat01:48
*** rpothier has left #heat01:51
*** alexheneveld has quit IRC02:02
*** achampion has joined #heat02:05
*** rpothier has joined #heat02:06
*** aweiteka has quit IRC02:08
elynn_morning all!02:15
openstackgerrithuangtianhua proposed a change to openstack/heat: Add quotas module with driver and resources  https://review.openstack.org/9671602:16
openstackgerrithuangtianhua proposed a change to openstack/heat: Add db APIs to access quota  https://review.openstack.org/9669702:16
*** rpothier_ has joined #heat02:18
*** ericgoncz has quit IRC02:19
openstackgerrithuangtianhua proposed a change to openstack/heat: Add RPC methods for quota actions  https://review.openstack.org/9700102:19
openstackgerrithuangtianhua proposed a change to openstack/heat: Add APIs for quota  https://review.openstack.org/9802602:20
*** dims has quit IRC02:40
*** ramishra has joined #heat02:56
*** gokrokve has joined #heat03:05
*** dims_ has joined #heat03:07
openstackgerritA change was merged to openstack/heat: Revert "Use auth_token_info to initialize heat_keystoneclient sessions"  https://review.openstack.org/9951703:08
*** dims_ has quit IRC03:12
stevebakerthat was quick03:13
SpamapSstevebaker: cause it fixed the gate ;)03:14
SpamapSstevebaker: now to find the race03:15
* SpamapS goes to eat dinner03:15
*** piyush has joined #heat03:15
*** zaneb has quit IRC03:16
*** rpothier has quit IRC03:17
*** piyush has quit IRC03:19
*** cmyster has joined #heat03:25
*** cmyster has joined #heat03:25
cmystermorning03:26
*** sgordon has joined #heat03:29
*** zhiyan is now known as zhiyan_03:30
*** akuznetsov has joined #heat03:30
*** zaneb has joined #heat03:31
*** ramishra has quit IRC03:33
*** arbylee has quit IRC03:37
*** zhiyan_ is now known as zhiyan03:41
*** nosnos has quit IRC03:44
openstackgerrithuangtianhua proposed a change to openstack/heat: Implement NovaFloatingIpAssociation res updatable  https://review.openstack.org/9338903:47
*** ramishra has joined #heat03:49
*** zns has joined #heat03:51
*** achampio1 has joined #heat03:51
zanebSpamapS: still about?03:53
*** achampion has quit IRC03:54
openstackgerritEthan Lynn proposed a change to openstack/heat: Add parameter type validation before stack-create  https://review.openstack.org/9956103:55
elynn_hi stevebaker , could you review this patch again? I just add some tests for it. https://review.openstack.org/#/c/98580/03:56
*** elynn_ has quit IRC03:57
*** elynn has joined #heat03:58
*** zns has quit IRC03:59
*** zns has joined #heat04:00
*** dstufft has joined #heat04:02
dstufftHey, is there any way to have an incrementing value in a hot parameter? specifically the servername so it can be foo1, foo2, foo3, etc.04:03
*** arbylee has joined #heat04:04
stevebakerdstufft: currently no04:05
dstufftok04:05
dstufftthanks04:06
stevebakerdstufft: can you just use the default server names?04:07
dstufftstevebaker: maybe, will have to poke a bit :)04:07
stevebakerdstufft: they attempt to be a reasonable combination of readable and unique04:08
dstufftok04:08
*** akuznetsov has quit IRC04:09
*** elynn has quit IRC04:09
*** elynn has joined #heat04:10
*** matsuhashi has quit IRC04:11
randallburtstevebaker, dstufft there's this in review for ResourceGroup that would allow you to do something like that: https://review.openstack.org/#/c/88636/04:11
*** DandyPandy has quit IRC04:12
openstackgerritZane Bitter proposed a change to openstack/heat: Fix race condition deleting in-progress stack  https://review.openstack.org/9956204:13
*** lazy_prince has joined #heat04:13
dstufftstevebaker: kinda ugly, but it works well enough I think. thanks04:13
dstufftrandallburt: cool!04:13
*** KanagarajM has joined #heat04:15
tiantian<elynn_>: See my comments:)04:15
*** harlowja is now known as harlowja_away04:16
zanebstevebaker: I may have a culprit on the race condition front, see above ^^^04:17
*** andersonvom has joined #heat04:17
*** arbylee has quit IRC04:18
stevebakerzaneb: cool, I'll thrash it locally soon04:19
zanebeeexcellent04:19
*** akuznetsov has joined #heat04:20
stevebakerzaneb: is eventlet.sleep OK to do inside a task?04:22
zanebby task you mean heat.engine.scheduler task?04:22
zanebit's... OK, but rarely what you want04:22
zanebit will allow other requests to run, but not other things in the same request that should be happening in parallel04:23
*** randallburt has quit IRC04:23
stevebakerzaneb: yes, this is for exponential backoff of action retries04:24
zanebyeah, that would be bad04:24
zanebdon't do that ;)04:24
stevebakerany other ways I could sleep?04:25
zanebinstead of "sleep(n)", do "for i in xrange(n): yield"04:25
stevebakerdoes a yield imply 1 second?04:26
zanebyes04:26
zanebwell, sort of04:26
stevebakeryeah, close enough04:27
zanebthe top level thing runs until _all_ of the tasks have yielded, then sleeps for 1s04:27
zanebso it depends how much time other tasks take up04:27
zanebbut for your purposes, close enough04:27
*** dstufft has left #heat04:30
zanebok, half past midnight. time for bed04:36
*** zns has quit IRC04:36
*** zns has joined #heat04:39
*** andersonvom has quit IRC04:45
*** saju_m has joined #heat04:47
*** matsuhashi has joined #heat04:48
*** nosnos has joined #heat04:48
*** gokrokve has quit IRC04:53
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/9650704:54
*** achampion has joined #heat05:08
*** achampio1 has quit IRC05:10
elynnHi tiantian05:16
*** zns has quit IRC05:16
*** zns has joined #heat05:16
elynnI think you misunderstand my logic in this patch https://review.openstack.org/#/c/98580/05:16
*** saju_m has quit IRC05:18
*** daneyon_ has quit IRC05:29
*** sgordon has quit IRC05:36
*** akuznetsov has quit IRC05:40
*** mandira has joined #heat05:45
*** nkhare has joined #heat05:47
mandiraHello all............05:48
mandira<therve>hi05:49
mandira<therve>i want to attach volume to this web server instance in this template ....http://paste.openstack.org/show/83751/.....can you explain me how can i do this ?05:54
openstackgerritEthan Lynn proposed a change to openstack/heat: Add parameter type validation before stack-create  https://review.openstack.org/9956105:59
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex  https://review.openstack.org/8975006:02
sirushtimandira, Have you looked at the AWS::EC2::VolumeAttachment resource?06:08
mandira<sirushti>i have seen that and added AWS::EC2::VolumeAttachment this to my template but got error06:10
*** akuznetsov has joined #heat06:11
sirushtimandira, what's the error?06:12
tiantian<elynn>: hi lauch time was06:14
elynntiantian, I reply your comment. : )06:15
mandira<sirushti>property error availability referance attribute is incorrect06:15
mandira<sirushti>i found that in my "launchconfig" resources ,availability  zone property is not there in aws template format....06:17
sirushtimandira, could you post the template and the exact error on paste.openstack.org and post the corresponding link here?06:21
mandira<sirushti>ok06:22
*** jprovazn has joined #heat06:24
*** akuznetsov has quit IRC06:25
mandira<sirushti>this is my template "http://paste.openstack.org/show/83753/"06:27
mandiraand my error is "Create_Failed: Resource CREATE failed: StackValidationFailed: Property error : DataVolume: AvailabilityZone The Referenced Attribute (LaunchConfig AvailabilityZone) is incorrect."06:27
sirushtimandira, could you try "Fn::GetAZs" : "" instead of what you've entered for your DataVolume resource?06:39
mandira<sirushti>i am trying that06:45
*** jamiehannaford has joined #heat06:46
therveGood morning!06:46
cylitherve: morning!06:46
thervecyli, Hi!06:47
mandira<sirushti> not working.. :( getting error Error: ERROR: Property error : DataVolume: AvailabilityZone Value must be a string06:47
*** akuznetsov has joined #heat06:48
thervemandira, if you want to use volumes with AWS::AutoScaling::LaunchConfiguration you need to use BlockDeviceMappings I think06:49
mandira<therve> i tried BlockDeviceMapping..it throws error that it is yet to be implemented06:50
*** alexknith has joined #heat06:50
therveOh yeah :)06:50
thervemandira, So you can't use that06:50
therveYou need OS::Heat::AutoScalingGroup06:50
alexknithshardy: Hi !06:51
mandira<therve> means insted of AWS::AutoScaling::LaunchConfiguration, i have to use OS::Heat::AutoScalingGroup ?06:53
shardymorning all06:53
*** cody-somerville has quit IRC06:53
thervemandira, And no AWS::AutoScaling::AutoScalingGroup as well06:53
mandira<therve>ok..i am trying that06:56
mandirathanks :)06:56
*** jstrachan has joined #heat06:57
openstackgerritSteven Hardy proposed a change to openstack/heat: Remove unused generate_request_id  https://review.openstack.org/9958307:00
*** julienvey has joined #heat07:01
*** zns_ has joined #heat07:02
*** akuznetsov has quit IRC07:02
*** zns has quit IRC07:03
*** mandira has quit IRC07:04
*** mandira has joined #heat07:05
*** cody-somerville has joined #heat07:05
*** jcoufal has joined #heat07:13
*** akuznetsov has joined #heat07:15
*** mandira has quit IRC07:24
*** e0ne has joined #heat07:24
*** tomek_adamczewsk has joined #heat07:27
*** zns_ has quit IRC07:29
*** Yanyan has joined #heat07:33
openstackgerrithuangtianhua proposed a change to openstack/heat: Add RPC methods for quota actions  https://review.openstack.org/9700107:33
openstackgerrithuangtianhua proposed a change to openstack/heat: Add quotas module with driver and resources  https://review.openstack.org/9671607:33
openstackgerrithuangtianhua proposed a change to openstack/heat: Add quota model and migration  https://review.openstack.org/9669607:33
openstackgerrithuangtianhua proposed a change to openstack/heat: Add db APIs to access quota  https://review.openstack.org/9669707:33
openstackgerrithuangtianhua proposed a change to openstack/heat: Add APIs for quota  https://review.openstack.org/9802607:33
openstackgerrithuangtianhua proposed a change to openstack/heat: Expose stack limits API  https://review.openstack.org/9958907:33
*** jcoufal has quit IRC07:33
*** tomek_adamczewsk has quit IRC07:36
*** alexheneveld has joined #heat07:36
*** cody-somerville has quit IRC07:36
*** tomek_adamczewsk has joined #heat07:36
tiantian<shardy>hi07:42
*** TonyBurn has joined #heat07:43
tiantian<shardy>: I reply your comment about the floatingip updatable patch: https://review.openstack.org/#/c/93389/07:43
*** jcoufal has joined #heat07:43
*** vpanizo_ has joined #heat07:44
*** e0ne has quit IRC07:44
*** e0ne has joined #heat07:45
*** vpanizo has quit IRC07:46
*** tomek_adamczewsk has quit IRC07:47
*** tomek_adamczewsk has joined #heat07:49
*** e0ne has quit IRC07:49
*** cody-somerville has joined #heat07:50
*** jcoufal has quit IRC07:51
pas-hamorning all07:58
*** jcoufal has joined #heat08:00
shardytiantian: I'm still not clear on why doing delete/create in handle_update is better than raising UpdateReplace08:00
*** tomek_adamczewsk has quit IRC08:01
shardytiantian: Is it to avoid the update replacing the dependent server?08:01
shardytiantian: basically, your handle_update *is* replacing the resource, because you delete the underlying resource and change the resource_id08:01
*** jistr has joined #heat08:02
*** tomek_adamczewsk has joined #heat08:04
*** tomek_adamczewsk has quit IRC08:06
*** tomek_adamczewsk has joined #heat08:06
tiantianshardy>: The UpdateReplace flow is to create a new and then to delete the old one, isn't?08:08
tiantian<shardy>: for current code, the 'server' and 'floatingip' are all not updatable, when stack-update with a new value either one of the two, it will go to the UpdateReplace flow, isn't?08:11
*** derekh_ has joined #heat08:11
*** Fayablazer has joined #heat08:12
*** fbo_away is now known as fbo08:14
*** alexheneveld has quit IRC08:14
*** jcoufal has quit IRC08:14
*** Qiming has joined #heat08:15
*** plomakin_ has quit IRC08:16
elynnhi shardy, do you have time to review my patches? https://review.openstack.org/#/c/99561/ https://review.openstack.org/#/c/98580/08:16
*** skraynev has quit IRC08:16
*** tnurlygayanov has quit IRC08:17
shardytiantian: ah, that's what I was missing - you want to reverse the flow so delete->create instead of create->delete via UpdateReplace?08:19
tiantian<shardy>: yes, if the properties are updatable, then it will delete and then create08:21
tiantian<shardy>: in handle_update()08:21
tiantian<shardy>: and if change the server_id, there is no need to delete08:22
tiantian<shardy>: just to create the association, the same as the neutron floatingip do, that patch has already merged:)08:23
pas-hashardy, yes, even when needed to recreate a resource a correct way is to delete then create (as VolumeAttachment for example), as there can be only one "thing", and UpdateReplace first creates two08:23
pas-hasometimes that is08:23
*** tomek_adamczewsk has quit IRC08:24
openstackgerritEthan Lynn proposed a change to openstack/heat: Enable SubnetId update for EC2::Instance  https://review.openstack.org/9804208:24
tiantianright:)08:25
shardytiantian: If the server_id changes, don't you need to deallocate the floating IP from the old server_id?08:25
tiantian<shardy>: yes, I test it08:25
shardypas-ha, tiantian: Thanks for the clarification, I understand the handle_delete path now08:26
* shardy probably just needs more coffee08:26
*** tomek_adamczewsk has joined #heat08:27
tiantian<shardy>: :)08:27
shardytiantian: What I mean is why isn't there a remove_floating_ip in the handle_update, when server_id has changed?08:27
shardyDoes nova automatically deallocate the IP from the existing server when you do add_floating_ip?08:27
tiantian<shardy>: because it will success, yes nova will do this08:28
tiantian<shardy>: I test this situation just in nova08:29
tiantian<shardy>: would you please to review the patch about the quota and absolute limits:)08:34
shardytiantian: I started looking at it but you've not addressed my comment re the quota_update DB API08:34
*** julienvey has quit IRC08:35
tiantian<shardy>: about this, I want to discuss with you:)08:35
shardyI don't really want to -1 it three times for the same reason, so I'll wait for feedback from other heat-core folks to see if they agree with me08:35
*** julienvey has joined #heat08:35
*** tomek_adamczewsk has quit IRC08:36
tiantian<shardy>: the quota_update is to update the limit of a quota_resource, such as stacks08:36
tiantian<shardy>: We can't create a new quota, just to update the limit of the quota_resources which pre-defined08:37
tiantian<shardy>: If we update a quota_resource limit, then we record this new limit of the quota_resource in DB08:37
tiantian<shardy>: It's the implement of the quota_update, nova/cinder/neutron do the same08:38
*** cody-somerville has quit IRC08:39
*** jcoufal has joined #heat08:43
tiantian<shardy>: and for the first time of update a limit of a resource, we need to create a new record, and also there is not quota_create api in nova/neutron/cinder08:46
openstackgerritEthan Lynn proposed a change to openstack/heat: Implement PrivateIpAddress property for EC2::Instance  https://review.openstack.org/9858008:46
shardytiantian: I guess that's my argument, in your current implementation there's no point having quota_create08:47
shardyand the pattern you're adopting, while it may be consistent with nova/cinder/neutron, is completely inconsistent with everything else in our DB API08:47
alexknithshardy: I had another look at the devstack conf for both keystone & heat, I've followed that but to no avail. Same config files keystone+heat on devstack-ubuntu and on centos-rdo but on my centos-rdo the auth between heat and keystone is still broken. Can you have a look at the logs,configs ? Many thanks !! the paste in a sec08:47
shardyalexknith: I'd try re-running packstack with the answer file it generated on first install08:48
shardyI don't really have the time to get into more debugging this morning, sorry08:48
alexknithshardy: ok, no worries08:49
*** dims_ has joined #heat08:49
alexknithshardy: maybe someone else ?08:49
alexknithcmyster: Hi !08:50
cmysterHi alexknith08:50
alexknithcmyster: Hi ! can you help with a bit of debugging ?08:51
cmysterdepends :)08:51
*** cody-somerville has joined #heat08:53
*** e0ne has joined #heat08:53
cmysteralexknith: off to eat in 1008:53
*** dims_ has quit IRC08:54
tiantian<shardy>: the quota implement in nova/cinder/neutron are not the same08:55
*** tomek_adamczewsk has joined #heat08:56
cmysterright08:57
cmysterwell, I'm off to eat alexknith, see you when I get back08:57
pas-hashardy, what's the local workflow when I need to add something to heat.conf.sample? any extra steps?08:58
shardypas-ha: tools/config/generate_sample.sh08:59
pas-haok08:59
pas-hathanks08:59
openstackgerritYaoguo Jiang proposed a change to openstack/heat: Add boolean type parameter process for template resource  https://review.openstack.org/9960309:00
hoangdohello guys, mind if I ask a question. I obtained alarm_url from ScalingPolicy (value: { get_attr : [web_server_scaleup_policy, alarm_url] }). Then what data I should post to the url to trigger alarm? I tried to post some random data and got 404.09:01
*** tspatzier has joined #heat09:01
tiantianOK, hope more people to review the patches09:01
alexknithcmyster: enjoy :)09:08
*** ramishra has quit IRC09:11
*** sorantis has joined #heat09:17
*** mandira has joined #heat09:21
*** zhiyan is now known as zhiyan_09:23
openstackgerrithuangtianhua proposed a change to openstack/heat: Implement NovaFloatingIpAssociation res updatable  https://review.openstack.org/9338909:30
openstackgerritChen Xiao proposed a change to openstack/heat: Fix DB2 column 'id' autoincrement issue  https://review.openstack.org/9207309:35
tiantian<shardy>: I have add the comments for the floating ip updatable patch. review agian:) thanks09:37
*** jprovazn has quit IRC09:38
*** Yanyan has quit IRC09:39
*** julienve_ has joined #heat09:39
shardytiantian: +2, thanks! :)09:40
*** julienvey has quit IRC09:43
*** e0ne_ has joined #heat09:44
*** e0ne has quit IRC09:48
*** jprovazn has joined #heat10:08
*** mandira has quit IRC10:10
*** saju_m has joined #heat10:10
*** tspatzier has quit IRC10:12
*** tspatzier has joined #heat10:13
*** skraynev has joined #heat10:14
*** Qiming has quit IRC10:17
*** tspatzier has quit IRC10:19
*** matsuhashi has quit IRC10:20
*** matsuhashi has joined #heat10:20
*** akuznetsov has quit IRC10:21
*** dims_ has joined #heat10:23
*** matsuhashi has quit IRC10:25
*** CarstenD has joined #heat10:26
CarstenDhello10:26
pas-haCarstenD, hi10:26
*** blues-man has joined #heat10:28
blues-manhello10:28
CarstenDI’m trying to figure out, how to pass a yml file as user-data inside a hot template. the manual command looks something like this „nova boot --user-data ./cloud-config.yaml --image   …“10:28
CarstenDI’ve found a few examples … but I can’t get it to work. it creates the stack without using the user-data10:30
shardyCarstenD: you need user_data_format=RAW to pass the data directly to cloud-init10:31
shardyCarstenD: unfortunately, if you're running Havana, that won't exist as it was added during Icehouse10:32
CarstenDumpf … ok :-)10:33
shardyhttps://github.com/openstack/heat-templates/blob/master/hot/software-config/example-templates/example-cloud-init.yaml#L9910:33
CarstenDthen I’ve found another reason to update ...10:33
shardyCarstenD: wel also added some other resources, e.g OS::Heat::CloudConfig in that example, which you may find useful10:34
*** denis_makogon has joined #heat10:34
CarstenDyes, thanx10:34
*** e0ne_ has quit IRC10:37
*** e0ne has joined #heat10:37
openstackgerritA change was merged to openstack/heat: Use ResourceDefinition when updating Load Balancer  https://review.openstack.org/9600310:39
*** e0ne has quit IRC10:41
*** tomek_adamczewsk has quit IRC10:43
*** tiantian has quit IRC10:43
*** tomek_adamczewsk has joined #heat10:44
*** ericgoncz has joined #heat10:48
*** mkollaro has joined #heat10:52
*** tomek_adamczewsk has quit IRC10:57
*** saju_m has quit IRC10:58
*** tomek_adamczewsk has joined #heat10:58
*** vpanizo_ has quit IRC11:04
*** Qiming has joined #heat11:06
*** saju_m has joined #heat11:10
*** nosnos has quit IRC11:13
*** ericgoncz has quit IRC11:15
*** e0ne has joined #heat11:15
*** e0ne has quit IRC11:17
*** e0ne has joined #heat11:17
*** e0ne has quit IRC11:22
*** shakayumi has joined #heat11:23
*** shakayumi has quit IRC11:23
*** shakayumi has joined #heat11:24
*** nkhare has quit IRC11:26
*** plomakin has joined #heat11:29
*** sorantis has quit IRC11:30
*** julienve_ has quit IRC11:35
*** julienvey has joined #heat11:36
*** blues-man has quit IRC11:37
*** sorantis has joined #heat11:37
*** julienve_ has joined #heat11:38
afaranhaWhat's the deal with Heat Templates? Can I use 'tab' to indenting? And how many white spaces do I need to use?11:38
*** jamiehan_ has joined #heat11:39
CarstenDfor the yaml files, use 2 white spaces to indent one level11:39
CarstenDbest is to use an editor which can display whitespaces, that helps a lot11:40
*** jamiehannaford has quit IRC11:41
*** julienvey has quit IRC11:41
afaranhaI'm using it, but I having "<unicode string>" errors, I tried using 'tab', 1 white space, 2, and 411:41
afaranhaand all of them returns error :/11:42
cmysterabout unicode string ?11:43
cmysterand your editor won't tell you anything about them ?11:43
cmysterlike, IDK ^M at the end of a line or something like that11:44
*** e0ne has joined #heat11:45
*** jamiehan_ is now known as jamiehannaford11:45
afaranhano, I'm using Sublime. I don't see it. Also, when I'm using 4 white spaces the error change to: ERROR: u'u\'"str_replace" syntax should be str_replace:\\n11:46
* cmyster uses elvis but thats just him11:47
afaranhanever heard about it11:49
*** shakayumi has quit IRC11:50
cmysterafaranha: its a VI (not VIM) clone11:51
CarstenDas far as i understand, I could use OS::Heat::CloudConfig to include my cloud init yaml inside a hot file … is it possible to do something like this? http://paste.openstack.org/show/83797/11:52
CarstenDespecially the line 19 to 21 …11:52
*** rpothier_ has quit IRC11:53
CarstenDthe problem is, I need to get a token url from etcd.io for core os. but the token must be the same for each instance inside the ResourceGroup11:54
CarstenDI’m not sure if the command is only executed once for the group or once for each count inside the group11:55
*** achampion has quit IRC12:00
*** aniket has joined #heat12:01
*** aniket has quit IRC12:01
*** aniket has joined #heat12:02
*** cmyster has quit IRC12:02
anikethi all12:03
CarstenDhi aniket12:04
aniket<CarstenD>hello12:04
aniket<CarstenD>for volume scaling,i am using OS::Heat::AutoScalingGroup....do you have any idea which alarm should i use to moniter volume usage?12:08
CarstenDI’m sorry … I’m new to heat … just here to ask questions for my own :-)12:09
afaranhaHow can I supply user_data in heat templates? Accordingly to this guide http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#str-replace , after "str_replace" I need to indent to start "template: |", but accordingly to my Heat I cannot do this, str_replace and template: should be in the same level, and now my heat it's complaining that the str_replace content must be a string: "ERROR: Property error : my_instance: user_data Value mus12:09
aniket<CarstenD>haha..welcome12:09
afaranhaThis is my template file: http://paste.openstack.org/show/83798/12:10
*** KanagarajM has quit IRC12:10
aniket<elynn> hi12:12
*** jamiehannaford has quit IRC12:14
afaranhatherve: Hi12:14
therveHello12:14
*** jamiehannaford has joined #heat12:15
afaranhado you know about heat templates?12:15
therveYeah12:17
*** jdob has joined #heat12:17
afaranhatherve: I'm having some issues with it, could you help?12:18
afaranhaI'm following this template guide: http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#str-replace but it's not working12:18
*** rbuilta has joined #heat12:18
therveYou're missing an indentation before the template key12:19
therveAlso you don't have keys so maybe don't use str_replace12:19
afaranhawhich keys?12:19
afaranhaThis is my template so far: http://paste.openstack.org/show/83798/12:20
therveWell params12:20
afaranhatherve: Do you know why this is not working?12:22
*** dims_ has quit IRC12:22
therveafaranha, Yes because you're missing an indentation before the template key12:22
*** dims_ has joined #heat12:23
*** jamiehan_ has joined #heat12:23
afaranhatherve: if i indent it it will raise another error, let mw show12:23
therveafaranha, Why do you use str_replace if you don't replace anything?12:23
*** jamiehannaford has quit IRC12:24
*** e0ne_ has joined #heat12:24
afaranhatherve: Here is template with the error raised: http://paste.openstack.org/show/83799/12:24
therveafaranha, Yeah you don't have params12:25
therveNot the bestest error message12:25
*** dims_ is now known as dims12:25
*** john-n-seattle has joined #heat12:26
CarstenDhttp://paste.openstack.org/show/83800/12:26
CarstenDmaybe you want something like this?12:27
afaranhaI didn't understand what you mean by params, which param?12:27
afaranhaCarstenD: ah, ok, gonna try that, thanks12:27
CarstenDstr replace is for replacing a string(needle = param) in a haystack (template)12:27
*** john-n-seattle has quit IRC12:28
*** e0ne has quit IRC12:28
CarstenDin line 24 get_attr must be get_param12:28
CarstenDhttp://paste.openstack.org/show/83801/12:29
*** tomek_adamczewsk has quit IRC12:30
afaranhatherve CarstenD: Thank you, it worked fine :D12:30
CarstenDyou’re welcome … just learning heat templates myself12:31
*** tomek_adamczewsk has joined #heat12:33
*** TonyBurn has quit IRC12:37
*** akuznetsov has joined #heat12:38
*** chandan_kumar has joined #heat12:41
*** pafuent has joined #heat12:42
*** chandan_kumar has quit IRC12:44
*** tomek_adamczewsk has quit IRC12:45
*** openstackgerrit has quit IRC12:46
*** tomek_adamczewsk has joined #heat12:47
*** openstackgerrit has joined #heat12:48
openstackgerritThomas Herve proposed a change to openstack/heat: Port to oslo.messaging  https://review.openstack.org/9964812:48
*** sgordon has joined #heat12:49
*** chandankumar has joined #heat12:50
*** julienve_ has quit IRC12:55
*** dims_ has joined #heat12:55
*** julienvey has joined #heat12:56
*** tomek_adamczewsk has quit IRC12:56
*** dims has quit IRC12:56
*** tomek_adamczewsk has joined #heat12:57
*** achampion has joined #heat12:58
aniket<therve> hi..is there any heat version equivalent to AWS::AutoScaling::LaunchConfiguration? any idea?13:01
therveaniket, You mean for OS::Heat::AutoScalingGroup ?13:02
shardyaniket: http://docs.openstack.org/developer/heat/template_guide/index.html13:02
aniket<therve>yes13:02
therveaniket, You shouldn't need one13:03
*** jamiehannaford has joined #heat13:03
shardyaniket: You can just reference a nested stack, like in this example for ResourceGroup13:04
shardyhttps://github.com/hardys/demo_templates/blob/master/juno_summit_intro_to_heat/example3_server_with_volume_group/server_with_volume_group.yaml13:04
shardyI think you need "resource" instead of "resource_def" for AutoScalingGroup13:05
shardyalso see https://github.com/openstack/heat-templates/blob/master/hot/autoscaling.yaml13:06
*** jamiehan_ has quit IRC13:06
aniket<shardy> ok...thanks..i am going through that13:08
*** tomek_adamczewsk has quit IRC13:08
*** tomek_adamczewsk has joined #heat13:11
*** vijendar has joined #heat13:14
*** sorantis has quit IRC13:17
*** mkollaro has quit IRC13:22
*** sgordon has quit IRC13:23
*** ericgoncz has joined #heat13:26
aniket<shardy>are you there?13:27
*** blomquisg has joined #heat13:28
aniket<shardy>http://paste.openstack.org/show/83805/ this is my template...as per http://docs.openstack.org/developer/heat/template_guide/index.html,in OS::Heat::AutoScalingGroup,cooldown property is optional..but this template is throwing error13:29
*** tomek_adamczewsk has quit IRC13:30
therveaniket, Which error?13:30
aniket<therve>cooldown:Unknown Property13:31
*** jergerber has joined #heat13:32
therveaniket, isn't it "Cooldown: Unknown property" ?13:32
aniket<therve>yes13:33
therveaniket, Case matters13:33
aniket<therve>yup..got it :P13:33
shardyaniket: please refer to the autoscaling.yaml example I linked, then you can see how to use these resources13:33
shardy"resource": { "Ref" : "EC2Instance" } won't work, as resource needs to be a nested stack definition, not a resource ID13:34
*** andersonvom has joined #heat13:34
*** julienve_ has joined #heat13:36
*** julienvey has quit IRC13:38
*** pas-ha has quit IRC13:40
*** mkollaro has joined #heat13:40
*** pas-ha has joined #heat13:42
*** andersonvom has quit IRC13:42
*** sileht has quit IRC13:43
aniket<shardy>https://github.com/openstack/heat-templates/blob/master/hot/autoscaling.yaml in this..type: lb_server.yaml..can you please give some information on this..?13:44
aniket<shardy>should i need to write a template with ld_server.yaml ?13:45
aniket*like13:45
shardyaniket: yes, it's specifying another heat template so you can scale nested stacks13:46
shardyaniket: You can just specify e.g OS::Nova::Server if you want to scale instances13:46
*** Tross has quit IRC13:48
*** DandyPandy has joined #heat13:48
*** afaranha has quit IRC13:49
aniket<shardy> i want to scale volume..for that if i refer https://github.com/openstack/heat-templates/blob/master/hot/F18/NovaInstanceWithCinderVolume_Native.yaml this template as resource,will it be working?13:54
*** beekneemech is now known as bnemec13:54
shardyaniket: provided that template works for you, yes it should13:55
*** zz_gondoi is now known as gondoi13:57
aniket<shardy> so,this https://github.com/openstack/heat-templates/blob/master/hot/F18/NovaInstanceWithCinderVolume_Native.yaml template should be in working condition on my openstack,to launch my volume scaling stack?13:57
*** sgordon has joined #heat13:58
*** jistr has quit IRC14:00
*** CoilDomain has joined #heat14:01
CoilDomainHow would you end up passing an ip address from one resource to another in a template?14:01
*** sileht has joined #heat14:02
CoilDomainI wanted to input that into a set of commands for the next resource in line14:02
*** jistr has joined #heat14:02
*** randallburt has joined #heat14:05
*** randallburt has quit IRC14:05
*** randallburt has joined #heat14:05
therveHum there is an issue with the test runner, it doesn't load all the tests14:06
*** ericgoncz_ has joined #heat14:06
*** ericgoncz has quit IRC14:08
*** ericgoncz_ is now known as ericgoncz14:08
therveGood lord, it doesn't fail anymore when there is an import error14:08
*** asalkeld has quit IRC14:12
*** sabeen has joined #heat14:15
*** arbylee has joined #heat14:18
*** tomek_adamczewsk has joined #heat14:20
*** arbylee has quit IRC14:21
*** arbylee has joined #heat14:22
*** sjmc7 has joined #heat14:25
*** jcoufal has quit IRC14:27
*** CarstenD has quit IRC14:34
*** radez_g0n3 is now known as radez14:34
*** piyush has joined #heat14:36
*** piyush has quit IRC14:36
*** piyush has joined #heat14:37
*** lazy_prince has quit IRC14:37
*** alexheneveld has joined #heat14:38
*** kgriffs|afk is now known as kgriffs14:40
*** mkerrin has quit IRC14:44
*** Meeral has joined #heat14:45
*** Meeral has quit IRC14:49
*** tomek_adamczewsk has quit IRC14:50
*** tomek_adamczewsk has joined #heat14:50
*** Qiming has quit IRC14:54
*** jistr has quit IRC14:56
*** jistr has joined #heat14:56
*** tomek_adamczewsk has quit IRC14:57
*** Fayablazer has quit IRC14:58
*** tomek_adamczewsk has joined #heat14:59
*** alexheneveld has quit IRC15:02
*** jprovazn is now known as jprovazn_afk15:04
*** Fayablazer has joined #heat15:06
*** Michalik- has quit IRC15:11
*** akuznetsov has quit IRC15:11
*** tomek_adamczewsk has quit IRC15:14
*** andreaf has quit IRC15:16
*** daneyon has joined #heat15:18
*** akuznetsov has joined #heat15:19
*** denis_makogon has quit IRC15:19
*** blues-man has joined #heat15:21
*** bandarji has joined #heat15:24
*** bandarji has quit IRC15:24
*** alexpilotti has joined #heat15:26
*** kgriffs is now known as kgriffs|afk15:30
*** kgriffs|afk is now known as kgriffs15:33
*** jamiehannaford has quit IRC15:36
*** jamiehannaford has joined #heat15:37
*** jamiehannaford has quit IRC15:38
*** jamiehannaford has joined #heat15:40
*** saju_m has quit IRC15:41
*** Michalik- has joined #heat15:43
*** samstav has joined #heat15:44
*** zns has joined #heat15:49
*** daneyon has quit IRC15:50
*** daneyon has joined #heat15:51
*** robsparker has joined #heat15:52
*** blues-man has quit IRC15:53
*** gokrokve has joined #heat15:54
*** blues-man has joined #heat15:54
*** daneyon_ has joined #heat15:55
*** daneyon has quit IRC15:55
*** bandarji has joined #heat15:59
*** e0ne_ has quit IRC15:59
*** e0ne has joined #heat16:00
*** akuznetsov has quit IRC16:03
*** mestery has quit IRC16:03
*** mestery has joined #heat16:04
*** e0ne has quit IRC16:04
*** alexheneveld has joined #heat16:07
*** tango|2 has joined #heat16:09
*** jistr has quit IRC16:19
*** alexknith has quit IRC16:22
*** blues-man has quit IRC16:28
*** bandarji has quit IRC16:32
*** alexheneveld has quit IRC16:33
*** gokrokve has quit IRC16:33
*** akuznetsov has joined #heat16:34
*** openstackgerrit has quit IRC16:38
*** bandarji has joined #heat16:38
*** Fayablazer has quit IRC16:45
*** Fayablazer has joined #heat16:52
*** mkerrin has joined #heat16:53
*** harlowja_away is now known as harlowja16:54
*** erecio has quit IRC16:58
*** skraynev has quit IRC16:58
*** jcoufal has joined #heat16:58
*** skraynev has joined #heat16:59
*** erecio has joined #heat16:59
*** rwsu has joined #heat17:00
*** radez is now known as radez_g0n317:00
*** alexheneveld has joined #heat17:01
*** samstav_ has joined #heat17:03
*** randallburt has quit IRC17:03
*** zns_ has joined #heat17:04
*** erecio has quit IRC17:06
*** nati_ueno has joined #heat17:06
*** zns has quit IRC17:07
*** samstav_ has quit IRC17:07
*** samstav has quit IRC17:08
*** samstav has joined #heat17:08
*** erecio has joined #heat17:09
*** zns_ has quit IRC17:09
*** zns has joined #heat17:10
*** samstav_ has joined #heat17:10
*** samstav has quit IRC17:10
SpamapSzaneb: 99562 looks pretty interesting17:13
zanebSpamapS: I'm just about to post a fixed version too17:14
SpamapSzaneb: I kind of don't want you distracted from update-failure-recovery .. because the more I'm getting into the initial POC forays into convergence, the more I'm seeing that I basically have to build on your template-snippet-saving to do it right.17:14
*** derekh_ has quit IRC17:14
zanebtoo late, I've been distracted from that for like 2 weeks :(17:15
*** tspatzier has joined #heat17:18
*** tspatzier has quit IRC17:21
*** gokrokve has joined #heat17:22
*** saju_m has joined #heat17:26
*** DandyPandy has quit IRC17:28
*** jstrachan has quit IRC17:29
*** samstav_ has quit IRC17:29
*** rwsu has quit IRC17:30
*** ericgoncz has quit IRC17:31
*** rwsu has joined #heat17:32
*** jprovazn_afk is now known as jprovazn17:41
*** mkollaro has quit IRC17:47
*** andersonvom has joined #heat17:49
*** zaneb has quit IRC17:52
*** alexheneveld has quit IRC17:53
*** mkollaro has joined #heat17:54
*** daneyon_ has quit IRC17:55
*** andersonvom_ has joined #heat17:59
*** mspreitz has joined #heat17:59
*** edmund has joined #heat18:00
*** andersonvom has quit IRC18:02
*** ericgoncz has joined #heat18:04
*** alexheneveld has joined #heat18:06
*** zaneb has joined #heat18:06
*** piyush has quit IRC18:07
*** jdob has quit IRC18:12
*** jdob has joined #heat18:13
*** openstackgerrit has joined #heat18:13
openstackgerritBartosz Górski proposed a change to openstack/heat: Adds multi-cloud to OS::Heat::Stack resource  https://review.openstack.org/9853318:14
openstackgerritBartosz Górski proposed a change to openstack/heat: Adds multi-region support for stack resource  https://review.openstack.org/5331318:14
*** ericgoncz_ has joined #heat18:14
*** DandyPandy has joined #heat18:17
*** jcoufal has quit IRC18:17
*** ericgoncz has quit IRC18:18
*** ericgoncz_ is now known as ericgoncz18:18
*** DandyPandy has quit IRC18:18
*** DandyPandy has joined #heat18:19
*** e0ne has joined #heat18:21
*** jprovazn is now known as jprovazn_afk18:22
*** zns has quit IRC18:23
*** mspreitz has quit IRC18:25
*** jdob has quit IRC18:25
*** jdob has joined #heat18:25
*** piyush has joined #heat18:25
*** randallburt has joined #heat18:26
openstackgerritZane Bitter proposed a change to openstack/heat: Fix race condition deleting in-progress stack  https://review.openstack.org/9956218:27
*** piyush1 has joined #heat18:27
zanebSpamapS, stevebaker: I'm reasonably confident that ^ should fix our race18:27
*** kgriffs is now known as kgriffs|afk18:28
*** samstav has joined #heat18:28
*** samstav has quit IRC18:29
*** samstav has joined #heat18:30
*** piyush has quit IRC18:30
*** zns has joined #heat18:30
*** DandyPandy has quit IRC18:32
*** DandyPandy has joined #heat18:32
*** piyush1 has quit IRC18:33
*** piyush has joined #heat18:33
*** erecio has quit IRC18:34
*** erecio has joined #heat18:34
*** DandyPandy_ has joined #heat18:35
*** bandarji has quit IRC18:35
*** DandyPandy_ has joined #heat18:37
*** DandyPandy has quit IRC18:37
*** DandyPandy_ has quit IRC18:37
*** DandyPandy has joined #heat18:38
*** DandyPandy has quit IRC18:40
*** Fayablazer has quit IRC18:41
*** rbuilta has quit IRC18:43
openstackgerritTon Ngo proposed a change to openstack/heat-specs: Stack breakpoint specification  https://review.openstack.org/9973418:46
*** DandyPandy has joined #heat18:48
*** DandyPandy_ has joined #heat18:51
*** DandyPandy_ has joined #heat18:52
*** erecio has quit IRC18:53
*** randallburt has quit IRC18:54
*** DandyPan_ has joined #heat18:54
*** randallburt has joined #heat18:54
*** DandyPandy has quit IRC18:55
*** randallburt has quit IRC18:55
*** DandyPandy_ has quit IRC18:56
*** randallburt has joined #heat18:57
*** kgriffs|afk is now known as kgriffs18:58
*** samstav_ has joined #heat19:00
*** samstav has quit IRC19:02
*** hoangdo has quit IRC19:02
*** funzo has quit IRC19:02
*** erecio has joined #heat19:03
*** funzo has joined #heat19:03
*** hoangdo has joined #heat19:03
*** tango|3 has joined #heat19:05
*** radez_g0n3 is now known as radez19:05
*** Tross has joined #heat19:07
*** tango|2 has quit IRC19:07
*** kgriffs is now known as kgriffs|afk19:09
*** kgriffs|afk is now known as kgriffs19:10
*** Tross has left #heat19:13
*** e0ne has quit IRC19:13
*** e0ne has joined #heat19:14
*** sabeen1 has joined #heat19:16
*** e0ne has quit IRC19:16
*** sabeen has quit IRC19:17
*** sabeen has joined #heat19:19
*** sabeen1 has quit IRC19:19
*** sabeen has quit IRC19:20
*** sabeen has joined #heat19:20
*** alexheneveld has quit IRC19:33
*** saju_m has quit IRC19:34
*** saju_m has joined #heat19:34
*** bandarji has joined #heat19:37
*** aweiteka has joined #heat19:41
*** saju_m has quit IRC19:46
*** nati_ueno has quit IRC19:47
*** saju_m has joined #heat19:47
*** e0ne has joined #heat19:48
*** nati_ueno has joined #heat19:56
*** akuznetsov has quit IRC19:56
*** e0ne has quit IRC19:59
*** e0ne has joined #heat20:00
*** e0ne has quit IRC20:02
*** arbylee has quit IRC20:03
*** arbylee has joined #heat20:06
*** zns has quit IRC20:07
*** radez is now known as radez_g0n320:07
*** zns has joined #heat20:07
*** julienve_ has quit IRC20:08
*** ericgoncz has quit IRC20:08
*** tomek_adamczewsk has joined #heat20:14
*** DandyPan_ has quit IRC20:17
*** morganfainberg has quit IRC20:18
*** morganfainberg has joined #heat20:18
*** noTHD has joined #heat20:24
*** nati_ueno has quit IRC20:28
*** TravT has joined #heat20:29
*** radez_g0n3 is now known as radez20:33
*** alexheneveld has joined #heat20:34
*** openstackgerrit has quit IRC20:35
*** openstackgerrit has joined #heat20:36
*** jprovazn_afk has quit IRC20:37
*** julienvey has joined #heat20:38
*** julienvey has quit IRC20:40
*** julienvey has joined #heat20:40
SpamapSOk this has sat a long time20:40
SpamapSis nobody doing reviews on python-heatclient? https://review.openstack.org/#/c/93834/20:40
*** nati_ueno has joined #heat20:44
*** julienvey has quit IRC20:45
*** dims_ has quit IRC20:47
stevebakermorning20:49
openstackgerritDavid J Hu proposed a change to openstack/python-heatclient: This is a Keystone v3 integration POC.  https://review.openstack.org/9272820:49
*** erecio has quit IRC20:53
*** jamiehannaford has quit IRC20:53
*** achampion has quit IRC20:56
*** e0ne has joined #heat20:57
*** asalkeld has joined #heat20:58
*** jdob has quit IRC20:59
openstackgerritA change was merged to openstack/heat: Handle missing user_creds in stack.delete  https://review.openstack.org/9950321:00
*** randallburt has quit IRC21:01
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/9650721:02
*** gokrokve_ has joined #heat21:02
*** alexheneveld has quit IRC21:03
*** pafuent has left #heat21:04
SpamapSstevebaker: so have you tried reproducing the race with https://review.openstack.org/#/c/99562/ and unreverting the revert we did yesterday?21:04
*** gokrokve has quit IRC21:05
stevebakerSpamapS: I'll try it now. I was having too much fun with my resiliant-heat-with-transient-failures patch21:05
*** nati_ueno has quit IRC21:07
cylistevebaker: zaneb:  just as an update, synced with therve yesterday - he has a lot of that work done already, going to try to help with any remaining pieces :)21:08
cylistevebaker: zaneb: wrt to oslo.messaging blueprint21:09
zanebcyli: cool, thanks for the update :)21:09
stevebakercyli: cool, thanks. has he just been doing config fixes, or oslo.messaging too?21:09
cylistevebaker:  oslo.messaging21:09
cylistevebaker:  it looks like he has a lot of the updates from those patches already, but less so the tests.21:10
stevebakerok21:10
*** vpanizo has joined #heat21:14
*** bgorski has joined #heat21:15
*** DandyPandy has joined #heat21:15
*** julienvey has joined #heat21:15
*** DandyPandy has quit IRC21:16
*** DandyPandy has joined #heat21:16
*** randallburt has joined #heat21:20
*** piyush has quit IRC21:21
*** e0ne has quit IRC21:26
*** e0ne has joined #heat21:26
*** petertoft has joined #heat21:28
*** TravT has quit IRC21:31
openstackgerritSteve Baker proposed a change to openstack/heat: Fix race condition deleting in-progress stack  https://review.openstack.org/9956221:31
openstackgerritSteve Baker proposed a change to openstack/heat: Use auth_token_info to initialize heat_keystoneclient sessions  https://review.openstack.org/9976921:31
*** e0ne has quit IRC21:31
stevebakerSpamapS, zaneb, I still get a racey looking error http://paste.openstack.org/show/83866/21:32
zanebdoh21:32
stevebakersee my review comment https://review.openstack.org/#/c/99562/21:32
SpamapSraces are like that21:38
SpamapSyou find all the easy ones when you go looking21:38
zanebstevebaker: fwiw I based patch set 2 on the patch *before* the revert specifically so you wouldn't have to revert the revert on top of it :P21:38
stevebakerzaneb: oh, I didn't notice that. no wonder I was getting cherry-pick weirdness21:39
*** julienvey has quit IRC21:40
zanebstevebaker: why the -1, btw?21:43
stevebakerbecause it didn't fix the race?21:43
zanebbut it fixes *a* race21:44
zanebthough clearly not the one we are looking for :(21:45
zaneband will get that 'lock already deleted' crap out of our logs21:45
*** jamiehannaford has joined #heat21:46
*** arbylee has quit IRC21:47
stevebakerzaneb: ok, since its an improvement I'll approve21:48
*** kgriffs is now known as kgriffs|afk21:49
stevebakera lock testing tempest api test would be pretty trivial, I wonder why we don't have one21:49
zanebstevebaker: thanks. no hurry to approve, but definitely no reason to not commit it eventually imo21:50
*** radez is now known as radez_g0n321:50
*** ccrouch has quit IRC21:51
*** sgordon has quit IRC21:53
*** mcwoods has quit IRC21:55
*** mcwoods has joined #heat21:56
*** dims_ has joined #heat21:57
*** aniket has quit IRC21:58
*** arbylee has joined #heat22:03
*** nati_ueno has joined #heat22:03
*** ccrouch has joined #heat22:04
*** ccrouch has quit IRC22:04
*** julienvey has joined #heat22:04
*** piyush has joined #heat22:11
*** ccrouch has joined #heat22:12
*** jergerber has quit IRC22:12
openstackgerritVijendar Komalla proposed a change to openstack/heat: Encrypt heat template parameters  https://review.openstack.org/9790022:13
*** piyush has quit IRC22:13
*** vijendar has quit IRC22:14
*** alexheneveld has joined #heat22:15
*** gondoi is now known as zz_gondoi22:17
*** julienvey has quit IRC22:18
*** blomquisg has quit IRC22:18
*** andreaf has joined #heat22:20
*** petertoft has quit IRC22:26
SpamapSHrm.. do we expect templates that do things like    {get_attr: [ Res1, {get_param: some_parameter_name} ]} to validate?22:29
SpamapSseems like validation should only stick to schemas and function names.. not anything that can't be resolved statically.22:30
*** gokrokve_ has quit IRC22:32
*** sabeen has quit IRC22:32
openstackgerritTon Ngo proposed a change to openstack/heat-specs: Stack breakpoint specification  https://review.openstack.org/9973422:33
*** bandarji has quit IRC22:35
stevebakerSpamapS: it would be nice if that would work22:36
*** andreaf has quit IRC22:36
*** arbylee has quit IRC22:36
SpamapSstevebaker: I think we just need to think carefully about validation as a subset of parsing.22:37
*** achampion has joined #heat22:41
*** jogo has left #heat22:42
*** gokrokve has joined #heat22:46
*** sjmc7 has quit IRC22:50
*** randallburt has quit IRC22:52
*** alexheneveld has quit IRC23:00
*** dims_ has quit IRC23:01
*** bandarji has joined #heat23:03
*** bandarji1 has joined #heat23:09
*** bandarji has quit IRC23:09
*** alexheneveld has joined #heat23:12
thervecyli, Hi! I posted a first patch FWIW23:19
therveFixing the tests now23:19
cylitherve:  oh wow, you are awake!23:20
cylitherve:  thanks!23:20
therveThat is true23:20
cylitherve:  heh, ok, so I guess I should stop trying to do that too :D23:21
cylitherve:  oh wait, did you mean adding tests?23:23
*** andersonvom_ has quit IRC23:23
thervecyli, No making the current ones pass23:23
cylitherve:  oh ok23:24
cylitherve:  can I assign this to you then?  https://blueprints.launchpad.net/heat/+spec/oslo-messaging23:25
therveSpamapS, We don't want i18n in debug, we actually just removed some23:25
cylitherve:  (since you've basically done it now :))23:25
thervecyli, Sure!23:25
cyli(all I've done so far is ask a bunch of questions about how to get started :))23:26
thervecyli, You'll see in my next patch that I'm actually mocking private APIs, so suggestions welcome23:26
cylitherve:  ok :)23:26
*** randallburt has joined #heat23:28
stevebakerwe may have a fix for most heat-slow failures https://review.openstack.org/#/c/99694/23:34
*** jamiehannaford has quit IRC23:35
openstackgerritRandall Burt proposed a change to openstack/heat: Refactor OS::Nova::Server delete handling  https://review.openstack.org/9980023:36
*** gokrokve has quit IRC23:39
thervestevebaker, Using external DNS servers feels strange23:40
stevebakertherve: I think the test already did23:40
therveStill :)23:41
*** edmund has quit IRC23:42
openstackgerritThomas Herve proposed a change to openstack/heat: Port to oslo.messaging  https://review.openstack.org/9964823:44
*** ccrouch has quit IRC23:44
*** Qiming has joined #heat23:45
*** andersonvom has joined #heat23:51
stevebakertherve: btw, I'm thinking if there is an API call failure in a custom constraint then the validation should pass (with a logged warning)23:52
thervestevebaker, Hum really? What kind of failure?23:53
stevebakertherve: transient network failures, or nova-api having kittens23:53
thervestevebaker, How you used HP cloud? :)23:53
therveThat feels a bit weird though23:54
stevebakerbetter than failing the stack though23:54
stevebakerlonger term there could be some retry logic23:54
therveBut if it fails at validation wouldn't it fail just after at creation?23:55
stevebakertherve: I'm working on making the actions resilient to transient failures, so it can retry until success. But that is all for nothing if validation fails23:57
thervestevebaker, Presumably it should use the same mechanism23:58
stevebakertherve: yes, that has just occured to me23:58

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