Tuesday, 2015-03-03

Dragojamielennox: oh, forgot to ask. How would I rescope the token?00:05
jamielennoxDrago: you can auth.V3Token(old_auth.get_token())00:06
jamielennoxauth_url=old_auth.get_endpoint(auth.AUTH_INTERFACE)00:06
*** hdd has quit IRC00:08
Dragojamielennox: ah, excellent00:08
*** Qiming has joined #heat00:09
openstackgerritSteve Baker proposed openstack/heat-templates: Boot config for installing software-config agent  https://review.openstack.org/11928200:13
openstackgerritSteve Baker proposed openstack/heat-templates: Boot config environment for test images  https://review.openstack.org/15817200:13
openstackgerritSteve Baker proposed openstack/heat-templates: Elements for building a heat test image  https://review.openstack.org/15967000:13
*** alexheneveld has quit IRC00:14
openstackgerritOpenStack Proposal Bot proposed openstack/heat: Updated from global requirements  https://review.openstack.org/16056600:14
*** chlong has quit IRC00:15
*** Qiming has quit IRC00:16
*** alexheneveld has joined #heat00:18
*** dims has quit IRC00:27
*** EricGonczer_ has joined #heat00:37
*** hdd has joined #heat00:53
*** Marga_ has quit IRC00:58
hddIs it possible for heat to ensure the status of stack resources? For example, if a stack resource is deleted, will heat be aware of it?01:00
stevebakerhdd: currently no, but the convergence effort will eventuall provide this01:02
hddthanks stevebaker01:03
*** Tango|2 has quit IRC01:03
stevebakerhdd: although if you delete the stack, heat won't mind that you've already deleted the resource01:03
*** Qiming has joined #heat01:05
*** dimsum__ has joined #heat01:08
openstackgerritshihanzhang proposed openstack/heat: Add config option to override url for versions  https://review.openstack.org/16026701:11
*** huangtianhua has joined #heat01:22
openstackgerritAngus Salkeld proposed openstack/heat: Cleanup the update policy tests  https://review.openstack.org/15854801:33
openstackgerritAngus Salkeld proposed openstack/heat: StackResource convert operations to use RPC  https://review.openstack.org/11633601:33
openstackgerritAngus Salkeld proposed openstack/heat: Remove the template mapping from the env before creating child stacks  https://review.openstack.org/15654701:33
openstackgerritAngus Salkeld proposed openstack/heat: Don't pass the parent_resource object into Stack()  https://review.openstack.org/15654601:33
openstackgerritAngus Salkeld proposed openstack/heat: Correct the check_update_completion logic in instance_group  https://review.openstack.org/16059501:33
*** achanda has quit IRC01:38
*** Yanyanhu has joined #heat01:45
*** sarob has joined #heat01:50
*** rwsu is now known as rwsu-afk02:06
*** LiJiansheng has joined #heat02:09
*** erkules_ has joined #heat02:11
*** erkules has quit IRC02:14
openstackgerritSteve Baker proposed openstack/python-heatclient: Implement deployment-create  https://review.openstack.org/16024002:19
*** GonZoPT has quit IRC02:20
*** alexheneveld has quit IRC02:29
*** alexheneveld has joined #heat02:30
*** alextricity has quit IRC02:30
*** elynn has joined #heat02:31
elynnmorning all:)02:34
*** hdd has quit IRC02:44
openstackgerritshihanzhang proposed openstack/heat: Add config option to override url for versions  https://review.openstack.org/16026702:50
openstackgerritMerged openstack/heat: Patch validate methods with mock instead of mox  https://review.openstack.org/16037002:51
openstackgerritMerged openstack/heat: Move software service logic to own module  https://review.openstack.org/16019102:55
openstackgerritMerged openstack/heat: Imported Translations from Transifex  https://review.openstack.org/16008202:55
openstackgerritMerged openstack/heat: Utility for database migration.  https://review.openstack.org/15944303:00
openstackgerritQiming Teng proposed openstack/heat: Utility function for int param checking  https://review.openstack.org/16061503:04
*** GonZo2K has joined #heat03:05
*** MVenesio has joined #heat03:06
*** EricGonczer_ has quit IRC03:26
*** Yanyanhu has quit IRC03:31
*** Yanyanhu has joined #heat03:32
*** dimsum__ has quit IRC03:45
*** MVenesio has quit IRC03:47
*** MVenesio has joined #heat03:47
*** GonZo2K has quit IRC03:48
*** jpeeler has quit IRC04:01
cmystermorning04:01
asalkeldhi04:03
*** hdd has joined #heat04:05
*** ygoto has joined #heat04:07
*** Marga_ has joined #heat04:08
*** thedodd has joined #heat04:08
*** harlowja_ is now known as harlowja_away04:13
cmystercoffee++04:14
*** KanagarajM has joined #heat04:24
KanagarajMgood morning all04:25
*** AnxiousGarlic has joined #heat04:28
*** AnxiousGarlic has left #heat04:29
*** MVenesio has quit IRC04:30
*** hdd has quit IRC04:34
*** coolsvap_ is now known as coolsvap04:37
*** EricGonczer_ has joined #heat04:40
*** EricGonczer_ has quit IRC04:45
*** dimsum__ has joined #heat04:46
openstackgerritQiming Teng proposed openstack/heat: Utility function for int param checking  https://review.openstack.org/16061504:51
*** ananta has joined #heat04:51
*** dimsum__ has quit IRC04:51
anantagood morning/afternoon04:52
*** nkhare has joined #heat04:55
cmysterindeed04:56
anantasure04:57
openstackgerritMerged openstack/heat: Updated from global requirements  https://review.openstack.org/16056604:59
*** signed8bit has joined #heat05:00
*** hdd has joined #heat05:02
cmysterhmmmm05:04
cmysterso heat wants oslo.log 0.4.0 and up yet other want it=0.3.005:04
cmysterothers05:04
*** david-lyle_afk has joined #heat05:06
cmysterahhh, here we go I had 0.3.0 'cause devstack was OK with having a bit older version (I ran stack.sh a few hours ago)05:07
*** hdd has quit IRC05:08
*** hdd has joined #heat05:09
*** huangtianhua has quit IRC05:16
*** dmn has joined #heat05:18
*** akuznetsov has joined #heat05:19
*** cmyster has quit IRC05:26
*** Marga_ has quit IRC05:28
*** KanagarajM_ has joined #heat05:28
*** KanagarajM has quit IRC05:32
*** thedodd has quit IRC05:34
openstackgerritRabi Mishra proposed openstack/heat-templates: WIP heat-config-docker-compose  https://review.openstack.org/16064205:37
*** thedodd has joined #heat05:37
*** signed8bit has quit IRC05:42
ramishrastevebaker: Hi05:42
*** signed8bit has joined #heat05:43
ramishrastevebaker: Pl. have a look when you have time. https://review.openstack.org/16064205:43
*** david-lyle_afk has quit IRC05:44
openstackgerritAngus Salkeld proposed openstack/heat: Cleanup the update policy tests  https://review.openstack.org/15854805:47
openstackgerritAngus Salkeld proposed openstack/heat: Correct the check_update_completion logic in instance_group  https://review.openstack.org/16059505:47
openstackgerritAngus Salkeld proposed openstack/heat: StackResource convert operations to use RPC  https://review.openstack.org/11633605:47
openstackgerritAngus Salkeld proposed openstack/heat: Remove the template mapping from the env before creating child stacks  https://review.openstack.org/15654705:47
openstackgerritAngus Salkeld proposed openstack/heat: Don't pass the parent_resource object into Stack()  https://review.openstack.org/15654605:47
*** vijayagurug has joined #heat05:50
*** vijayagurug has quit IRC05:54
*** Marga_ has joined #heat06:01
openstackgerritOpenStack Proposal Bot proposed openstack/heat: Imported Translations from Transifex  https://review.openstack.org/16064706:03
*** dmn has quit IRC06:04
*** cmyster has joined #heat06:04
*** achanda has joined #heat06:14
*** huangtianhua has joined #heat06:17
*** dmn has joined #heat06:20
*** huangtianhua has quit IRC06:23
*** hdd has quit IRC06:25
*** huangtianhua has joined #heat06:30
*** signed8bit has quit IRC06:30
*** thedodd has quit IRC06:31
*** rakesh_hs has joined #heat06:37
*** david-lyle_afk has joined #heat06:44
*** ishant has joined #heat06:49
*** vijayagurug has joined #heat06:55
*** reed has quit IRC06:58
*** inc0 has joined #heat07:03
inc0good morning07:03
*** cmyster has quit IRC07:03
*** dulek has joined #heat07:04
inc0quick question. I'm thinking about rolling upgrades of openstack. Let's assume we use tripleO and heat underneath. So, we want to make an upgrade of host A, which is represented as nested-stack-A. Before we even start to play with it, we should ensure that all VMs are migrated out of it and so on. Can we do that in heat? I mean to have pre-update hook and add custom logic to it for example?07:06
*** jtomasek has joined #heat07:07
inc0breakpoints might be useful here, but it would require some tinkering anyway07:08
openstackgerrithuangtianhua proposed openstack/heat: Apply neutron subnet constraint  https://review.openstack.org/13587507:09
Qimingin0, interesting use case, but not sure if it is relevant to Heat07:12
*** cmyster has joined #heat07:12
inc0Qiming, I'm quite certain it is. Especially if we take tripleO into account07:12
inc0let's assume we have 100 compute nodes with juno on it07:12
Qimingwhat you want seems more like a workflow to me, and it involves quite some invocations to admin APIs that is beyond Heat's scope07:12
inc0I was rather thinking about implementing some sort of hooks07:13
inc0while I don't want to make heat explicitly call nova apis and so on07:13
*** tspatzier has joined #heat07:13
Qimingsay, if you have 90% of the work done in your hooks, are you still so sure it is relevant to Heat?07:13
inc0We still need to have these hooks07:14
*** dmn has quit IRC07:14
inc0inside template07:14
inc0"don't upgrade resource X before this script ends"07:14
Qiminginc0, that sounds more like a workflow than a template based orchestration07:15
inc0Qiming, agree, but we need something to call this workflow and hold on resource upgrage until workflow end07:16
inc0s07:16
Qimingmaybe ansible or mistral is more suited for this task? I don't have an answer07:16
Qimingmaybe the Mistral resource type can help straighten the flow, at least to a certain degree?07:18
Qiminginc0, this one: https://review.openstack.org/#/c/147645/07:20
*** openstackgerrit has quit IRC07:22
*** openstackgerrit has joined #heat07:22
shardyinc0: I think software config resources provide some of the answer here, but also there is an element of external workflow too07:23
shardyfor tripleo, perhaps the abstraction layer might end up being Tuskar, which could plug in to e.g Mistral or some other workflow implementation, to drive each discrete phase of the upgrade07:23
inc0shardy, Qiming maybe some sort of fake resource and dependency?07:25
asalkeldskraynev: have you started with this yet: https://blueprints.launchpad.net/heat/+spec/convergence-push-data07:26
asalkeldskraynev: I can grab that if you haven't started ^07:26
shardyinc0: "fake resource"?07:26
inc0I just think this is relevant use-case for tripleO07:26
skraynevasalkeld: good morning07:26
openstackgerrithuangtianhua proposed openstack/heat: Apply neutron router constraint  https://review.openstack.org/13588007:26
asalkeldhi skraynev07:26
shardyinc0: as you say, breakpoints help here, when you want to control the flow of updates outside of heat07:26
skraynevasalkeld: I plan do it today.07:26
inc0shardy, resource, which handle-upgrade will call Mistral workflow, and Update-complete will trigger after this workflow finishes07:26
asalkeldskraynev: ok07:27
asalkeldI'll post what i have done07:27
shardyinc0: you're right, it is, I'm aware some folks are interested and/or looking at what you describe07:27
skraynevif it is ok, I will continue, otherwise you can grab :)07:27
asalkeldskraynev: it's yours don't worry, only wanting help if you are busy07:28
inc0I'll refinte this thought and maybe post a bp about it07:28
inc0I just wanted to make sure that we don't have this capability at this moment07:28
shardyinc0: I think the thing is, "handle update" isn't a good fit for the heat declarative model, but you're right, perhaps there could be a relationship between mistral resources in the parent template and the update of nested templates07:28
skraynevasalkeld: no. I currently ready for it :) finish or hold other not so important things07:29
shardytbh I'd assumed we'd want the updates to remain triggerd by heat, but with the steps controllable via breakpoints07:29
shardymorning all btw ;)07:29
shadower'lo07:29
asalkeldskraynev: ok - I'll look at some other bp's07:30
shadowerasalkeld: re #3 in https://review.openstack.org/#/c/146123/, what would you suggest instead? I resource_registlry is a mapping of types to templates, but breakpoints aren't.07:30
shadowernot sure how to make it more consistent07:30
cmystero/07:30
skraynevasalkeld: thx :) if I meet some issue, I will ping you.07:31
asalkeldhi shadower07:31
shadowerhey :-)07:31
asalkeldmaybe a dict instead of list07:31
asalkeldor use resource_registry07:32
asalkeldi was just trying to look at it from a user's perspective and it looked a bit odd07:32
cmysterre breakpoints, whats the review's link?07:32
shadowercmyster: https://review.openstack.org/#/c/146123/07:32
cmysterthanks07:33
inc0shardy, my thoughts exactly - I'd love to see undercloud heat template upgrade perform no-downtime release upgrade of overcloud07:33
cmysterhmmm07:33
cmysterno downtime is tricky07:33
inc0cmyster, tell me about it07:34
shardyinc0: Yeah, me too - I think we need containers to make that simpler though, image replacement upgrades are really expensive (particularly for compute nodes)07:34
shadowerasalkeld: if we used dicts, the nested resources would be represented by nesting the dictionaries?07:34
asalkeldshadower: yeah07:34
cmysterinc0: say you have a single resource and that needs an upgrade? move to new, grace kill the old ?07:34
inc0shardy, containers or not, we still need to have hook like that - I wouldn't risk upgrading nova-compute with VMs still working on it07:34
asalkeldshadower:  (looks better in yaml than json)07:34
shadowertrue07:34
asalkeldshadower: i don07:35
asalkeldshadower: i don't have a smart option for your right now, but just want something better07:35
inc0cmyster, something like that...with for example live-migrate everything out of compute07:35
shadowerrigth fair enouh, I'll think about it07:35
cmysterinc0: good luck yo07:36
shadowerasalkeld: not sure about the "use resource_registry" suggestion. How would that work07:36
shardyinc0: I think there are three approaches 1. In-place update (traditional e.g package based update), 2. Image replacement update (e.g rolling upgrades via rebuild), 3. Container based upgrade (ideally not touching any VMs, without restart or interruption)07:36
shardythere are pro's and cons to each of them07:36
asalkeldshardy: hold on, i'll make an etherpad for us to bikeshed07:36
shadowerasalkeld: something like passing a "breakpoint: true" or whatnot instead of the nemplate name as a registry value?07:36
asalkelds/shardy/shadowder07:36
asalkeldshadower: yip07:37
shardyasalkeld: good plan, I was going to suggest a ML thread but either works07:37
shardyinc0: thanks for bringing this up, I know quite a few folks are interested in this07:37
asalkeldhttps://etherpad.openstack.org/p/heat-breakpoints07:37
*** darsh29_ has joined #heat07:37
inc0shardy, in-place will fail due to 3rd party libs version - unless we use for example virtualenv, which is just another option of separation07:37
shardyinc0: "3rd party libs version" ?07:38
shardyIt's not very cloudy, but folks have been doing in-place package based upgrades for years07:38
*** akuznetsov has quit IRC07:38
inc0we want to do in-place upgrade of nova-compute which uses oslo.whatever in version >=0.10.0 and neutron requires <=0.9.007:39
inc0shardy, but not no-downtime upgrades really...it is possible only if you upgrade all the stuff at once07:39
inc0still, its not very good way to do it07:40
inc0not the perfect anyway07:40
cmysternothing is,07:41
cmysteralso you want to consider what to do if an update fails07:41
shardyinc0: There is no perfect way, I'm leaning towards containers as the most optimal given current options though07:41
cmysterwhich aproach will revert that changes to something that works07:41
cmyster+1 on those07:41
cmysterjust for downgrading if needed07:41
shardyparticularly if we can have the hypervisor on the host so VMs survive container upgrades07:41
inc0shardy, agree - but image-replace isn't that bad eighter if we prepare host correctly07:41
asalkeldshadower: some examples there now07:41
shadowerasalkeld: thanks, that makes it clearer07:42
inc0I mean, migrate everything out of node in question07:42
cmysterinc0: prepare host correctly might be seen incorrect to some user...07:42
*** elynn has quit IRC07:42
inc0cmyster, hence make a hook in template07:42
shardyinc0: I guess, it's just a big overhead with stateful nodes07:43
inc0whole conversation started from my question -> can we apply a hook "prepare stuff before upgrading it"07:43
* cmyster needs more data he guesses07:43
shardyinc0: basically, yes, via a SoftwareConfig resource which is marked only to run on UPDATE (the action in the Deployment)07:43
*** jprovazn has joined #heat07:43
shardyso, e.g you could have a ResourceGroup of nested stacks, which contain a server, and a update-only SoftwareConfig (amongst other stuff)07:44
inc0I'll have to try that..or some magic around mistral resources07:44
shardyor maybe it'd need to be delete-only for image replacement07:44
shardywe'd have to see how that looks for rebuild07:44
shardybut, basically I think some variation of that pattern would work07:45
inc0yeah...and we need to make sure to *not* perform step2 if step1 fails07:45
shardybut the problem is, the actors in the heat template are the infrastructure operators, not necessarily the operators of the deployed cloud07:45
shardyso a SoftwareConfig resource isn't necessarily the best place to e.g do an evacuate of a compute node07:45
inc0it does seem a bit hacky to me...07:46
inc0I would rather do some dependency magic with mistral resorces07:46
* asalkeld heads off to make supper07:46
shardyyeah, thus my comment about Tuskar - really you need a layer above heat, to drive the upgrade07:46
inc0or even add some explicite definition inside template07:46
shadowerasalkeld: thanks. Of the two I prefer the first one. I'll see if I can figure out something else (but that sounds pretty good to me) and address the other stuff in the review07:46
shardycurrently, that layer of control doesn't really exist in TripleO07:46
shardyinc0: Shall I start a ML thread on this topic?  I know some folks have been looking at it, so wider discussion of what the solution might look like would be good07:47
*** Marga_ has quit IRC07:48
* shardy goes to get coffee07:49
inc0shardy, please do07:49
inc0seems like good thing to work on in L07:49
*** Marga_ has joined #heat07:58
inc0shardy, I'll start gather ideas and info for bp07:59
*** ifarkas has joined #heat08:01
*** vijayagurug has quit IRC08:05
*** erkules_ is now known as erkules08:05
*** Marga_ has quit IRC08:06
*** jcoufal has joined #heat08:07
*** alexheneveld has quit IRC08:12
*** lsmola has joined #heat08:17
*** dimsum__ has joined #heat08:24
*** akuznetsov has joined #heat08:27
*** dimsum__ has quit IRC08:29
asalkeldKanagarajM_: hi you about?08:31
KanagarajM_asalkeld: hi08:31
asalkeldhttps://blueprints.launchpad.net/heat/+spec/keystone-resources08:31
asalkeldwhat is the state of that?08:31
asalkeldKanagarajM_: when would you get time to post a patch?08:32
KanagarajM_asalkeld: I'm working on that and should be summiting the patch before tomorrow08:32
asalkeldKanagarajM_: ok, thanks for the update08:32
asalkeld(I have a release meeting and need to know the status of bps)08:32
KanagarajM_asalkeld: sure. no prb.08:33
*** darsh29_ has quit IRC08:37
*** pitr-ch has joined #heat08:37
*** dulek has quit IRC08:38
*** dulek has joined #heat08:39
*** liuh has joined #heat08:43
openstackgerritAnant Patil proposed openstack/heat: Add extra data to Stack table for Convergence.  https://review.openstack.org/15521108:45
*** inc0 has quit IRC08:46
*** jistr has joined #heat08:46
*** ygoto has quit IRC08:46
*** dulek_ has joined #heat08:50
*** inc0 has joined #heat08:52
*** dulek has quit IRC08:53
openstackgerritShaoHe Feng proposed openstack/heat: First objects - stack and raw template  https://review.openstack.org/14684108:53
*** achanda has quit IRC08:55
*** inc0_ has joined #heat08:57
asalkeldananta: do you have to do filter(a=x).filter(b=y) or can you do filter(a=x, b=y)08:59
*** dulek_ has quit IRC09:00
asalkeld(not a big deal, just wondering)09:00
*** inc0 has quit IRC09:00
anantasorry, in which context?09:00
anantaoh09:01
anantai think the earlier one09:01
anantafilter(a=x).filter(b=y)09:01
*** dulek has joined #heat09:03
anantai don't know if there are any performance issue wrt this09:04
*** dulek has quit IRC09:04
anantafilter(a=x, b=y) seems like AND condition of SQL query09:04
anantaand filter(a=x).filter(b=x) seems like making 2 queries :) I am not sure though09:04
*** akuznetsov has quit IRC09:05
*** lifeless has quit IRC09:05
*** lifeless has joined #heat09:06
anantaasalkeld: do you know of any implications of using them?09:06
asalkeldsorry ananta, was in the releasemgr meeting09:08
asalkeldananta: haven't a clue, just seemed a little odd09:09
asalkeldALL: in the release manager meeting we (ttx and I) agreed to "FPF for everything but *convergence* BPs, keystone-resources needs to put something up by Thursday"09:10
therveI'm pretty sure the 2 expressions do the same09:11
asalkeldKanagarajM_: ^09:11
asalkeldhi therve09:11
therveHi!09:11
asalkeldtherve: so no real performance change?09:11
anantaI think it is just for readability09:11
asalkeldthat's was all i was wondering09:11
QimingFPF?09:11
anantatherve: agree with you09:11
therveasalkeld, Well one is just one less function call :)09:11
asalkeldtherve: that is just building the query, not actually doing the query?09:12
therveasalkeld, Correct09:12
asalkeldok, cool09:12
therveIt's modifying a Query object in place09:12
asalkeldQiming: https://wiki.openstack.org/wiki/Kilo_Release_Schedule09:12
asalkeldFeatureProposalFreeze09:13
Qiminggot it09:13
asalkeldmeaning you have to have posted your feature and the blueprint needs to be "in review"09:13
asalkeldsince convergence is disabled, we can keep working on it09:14
KanagarajM_asalkeld: sure, i could summit the patch for keystone-resource by this time.09:14
asalkeldKanagarajM_: cool09:14
asalkeldKanagarajM_: doesn't have to be "perfect"09:14
asalkeldjust something "up"09:14
KanagarajM_asalkeld: sure.09:14
asalkeldKanagarajM_: send me an email when it's up, and i'll put the bp into the right state09:15
KanagarajM_asalkeld: sure. I will send mail.09:16
asalkeldthx09:16
*** serg_melikyan has joined #heat09:22
*** huangtianhua has quit IRC09:24
*** liuh has quit IRC09:25
*** alexheneveld has joined #heat09:28
anantaasalkeld: http://docs.sqlalchemy.org/en/rel_0_9/orm/query.html#sqlalchemy.orm.query.Query.update09:30
anantait just forms the update bulk query to be issued as a single SQL command09:31
asalkeldok, thanks for checking it out09:34
skraynevshardy: ping09:37
*** che-arne has joined #heat09:38
skraynevshardy: about this one https://review.openstack.org/#/c/159978/1/requirements.txt09:38
shardyskraynev: hi09:38
skraynevI have added it in global requirements09:38
shardyskraynev: Ok, why is it needed?09:38
skraynevit was related with issue for tempest tests on CentOS09:38
skraynevwait a second I give a link09:38
skraynevhttps://bugs.launchpad.net/heat/+bug/140575209:39
openstackLaunchpad bug 1405752 in heat "Hanging heat-api after validate request" [High,Triaged] - Assigned to Sergey Kraynev (skraynev)09:39
skraynevIf we use older version in CentosOS and start tempest tests it may hang heat api09:40
shardyskraynev: is 2.1.0 definitely the version which fixed it?09:40
shardyvs some version between 1.2.1 and 2.1.0?09:40
skraynevI tested it step by step09:40
skraynevso only in 2.1.0 it works correct09:40
shardyOk, it'd be great if we knew which patch fixed it09:41
shardyit's unfortunate that we lose this context (bug reference) with bot-proposed patches :(09:42
openstackgerritMerged openstack/heat: Cleanup the update policy tests  https://review.openstack.org/15854809:42
skraynevI may add it manually09:42
skraynevif it make sense09:42
skraynevI am just not sure how bot react on it09:42
shardyskraynev: Ok, maybe just a comment explaining the change and referencing the bug is enough09:44
shardyskraynev: then I'll remove my -109:44
skraynevok09:44
shardythanks for the explanation :)09:44
skraynevshardy: np :)09:44
*** Yanyanhu has quit IRC09:45
*** nkhare has quit IRC09:48
*** coolsvap is now known as coolsvap_09:52
*** serg_melikyan has quit IRC09:53
*** nkhare has joined #heat09:55
*** serg_melikyan has joined #heat09:57
*** LiJiansheng has quit IRC10:11
*** LiJiansheng has joined #heat10:11
*** Qiming has quit IRC10:14
*** inc0_ has quit IRC10:29
*** inc0 has joined #heat10:31
*** nkhare has quit IRC10:32
*** nkhare has joined #heat10:32
asalkeldshardy: how are to set the milestone for bug 104575210:36
openstackbug 1045752 in unity (Ubuntu) "Dash - Clicking on the main left hand side preview image in a Dash Preview should close the Preview and return the user to the results view" [High,Fix released] https://launchpad.net/bugs/104575210:36
asalkeldoops10:36
asalkeldshardy: how are to set the milestone for bug 140575210:36
openstackbug 1405752 in heat "Hanging heat-api after validate request" [High,Triaged] https://launchpad.net/bugs/1405752 - Assigned to Sergey Kraynev (skraynev)10:36
asalkeld(stable/juno)10:36
asalkeldi see, it won't have a patch10:37
asalkeldshrug10:37
shardyasalkeld: If it only affects stable, I'd maybe mark the main heat bug invalid, and target to series for juno, referencing the requirements sync skraynev and I were just discussing?10:42
skraynevam here. what's up?10:42
asalkeldhow do you target to juno10:43
shardyasalkeld: I've just done it10:43
shardy"Target to Series"10:43
asalkeldo10:43
shardyI guess the master bug can be fix committed, or invalid?10:44
asalkeldinvalid10:44
shardyI'm not quite sure when the master requirements bump for requests happened10:44
asalkeldme neither10:44
asalkeldbtw, thanks shardy10:44
*** Qiming has joined #heat10:44
openstackgerritMerged openstack/heat: Imported Translations from Transifex  https://review.openstack.org/16064710:45
shardyasalkeld: np, reminds me of my PTL stint, click, click, click ;)10:45
asalkeldphew, yeah10:45
shardyI think I read they're dropping the granular tracking of features for L?10:46
shardythat should make things easier IMO10:46
*** tochi has quit IRC10:47
*** inc0 has quit IRC10:48
KanagarajM_hi10:57
KanagarajM_is there any library used in openstack to find diff of python dict?10:58
*** inc0 has joined #heat10:58
shardyKanagarajM_: resource.Resource.update_template_diff does that11:00
shardyIIRC when I wrote that, I couldn't see an obvious existing solution11:01
KanagarajM_shardy: ok. thanks. let me check it.11:01
openstackgerritPeter Razumovsky proposed openstack/heat: Fix mocking requests in test_engine_service  https://review.openstack.org/16073711:08
*** inc0 has quit IRC11:13
openstackgerritQiming Teng proposed openstack/heat: Make Heat ASG always do rolling_updates  https://review.openstack.org/13794511:15
openstackgerritPeter Razumovsky proposed openstack/heat: Fix mocking requests in test_engine_service  https://review.openstack.org/16073711:16
*** dimsum__ has joined #heat11:16
*** coolsvap_ is now known as coolsvap11:17
*** serg_melikyan has quit IRC11:25
*** ishant has quit IRC11:26
*** kairat_kushaev has quit IRC11:26
*** tellesnobrega has quit IRC11:27
*** tellesnobrega has joined #heat11:33
*** inc0 has joined #heat11:44
*** inc0 has quit IRC11:45
*** cdent has joined #heat11:49
openstackgerritShaoHe Feng proposed openstack/heat: First objects - stack and raw template  https://review.openstack.org/14684111:50
openstackgerritMerged openstack/python-heatclient: Fix SessionClient error when endpoint=None  https://review.openstack.org/16043111:51
*** kairat_kushaev has joined #heat11:52
Qimingreview.openstack.org down?11:56
matttQiming: loads for me11:56
Qimingmattt, thanks11:57
openstackgerritAnant Patil proposed openstack/heat: Add extra data to Stack table for Convergence.  https://review.openstack.org/15521112:03
*** KanagarajM_ has quit IRC12:04
*** EmilienM|afk is now known as EmilienM12:05
*** prazumovsky has joined #heat12:06
openstackgerritYoshimi Tominaga proposed openstack/heat: Add the documentation of how to not set the gateway IP  https://review.openstack.org/15909012:06
*** serg_melikyan has joined #heat12:12
*** serg_melikyan has quit IRC12:17
*** sgordon has quit IRC12:17
*** sgordon has joined #heat12:17
*** sgordon has quit IRC12:18
*** sgordon has joined #heat12:18
*** MVenesio has joined #heat12:26
*** nkhare has quit IRC12:26
Qimingskraynev, there?12:32
*** ParsectiX has joined #heat12:35
skraynevQiming: ping, I was at dinner ;)12:37
Qiminghttps://review.openstack.org/#/c/153192/12:38
Qimingskraynev, curious why gate hates it, any idea?12:38
skraynevAWS lbs ?12:38
Qimingyes12:38
*** asalkeld has quit IRC12:39
skraynevIMO, because it takes a lot of time and can not work faster on infra nodes12:40
Qimingskraynev, I doubt that12:40
skraynevI heard that loading one vm with fedora takes 10 minutes...12:41
skraynevanother idea was related with issues during installation ha on lb12:41
skraynevha-proxy12:41
Qimingthen it means the build_timeout either is ineffective or is not long enough?12:41
Qimingif you check the log, the timestamps of the error message are not showing such a long delay12:42
*** serg_melikyan has joined #heat12:43
*** Yanyanhu has joined #heat12:43
skraynevI thought about short delay for waitcondition12:44
skraynevbut I didi not find yet where it may be configured12:44
*** ananta has quit IRC12:44
Qimingthe latest log shows (if I'm understanding correctly) test_aws_loadbalancer started at 2015-03-03 11:14:45.68512:45
Qimingand the failure is printed at 2015-03-03 11:25:22.93012:45
skraynevcorrect12:45
skraynev10 minutes12:45
*** dimsum__ is now known as dims12:46
skraynevI set build_timeout to 180012:47
skraynevi.e. 30 minutes12:47
*** yassine has joined #heat12:47
Qimingit took 637.226158 seconds to fail12:47
skraynevbut it fails after 10 minutes12:47
skraynevyeah12:47
skraynevlooks like some internal waitcondition constraint12:47
skraynevi.e. we wait answer 10 minutes12:48
skraynevIMO it may be related with one of two possible things:12:48
skraynev1. time is really not enough (node is very slow)12:48
skraynev2. some issues during installation LB are not related with time (maybe network )12:49
Qiming10 min is the wait condition timeout12:50
skraynevI wanted to investigate it, but now work on convergence patch. so if you have any ideas, I welcome for them :) and you can play with patch o review :)12:50
skraynevQiming: yeah12:50
Qiminghow am I supposed to test it locally?12:51
skraynevand as I understand it's not mentioned in heat config12:51
QimingI mean just run this scenario w/o tox -eintegration as a whole12:51
skraynevQiming: right, but also need to download in glance image for LB12:52
skraynevit uses fedora 21 with specific name12:52
skraynevFedora-Cloud-Base-20141203-21.x86_6412:52
skraynevhttps://download.fedoraproject.org/pub/fedora/linux/releases/21/Cloud/Images/x86_64/Fedora-Cloud-Base-20141203-21.x86_64.qcow212:52
skraynevalso I recommend to start only this test ;) otherwise you spent a lot of time on other tests12:59
Qimingthat's my question, ;)13:00
Qiminghow to start only this test?13:00
*** Yanyanhu has quit IRC13:03
skraynevoh... :)13:04
skraynevI misunderstand you :)13:04
skraynevwait a second13:04
skraynevhttps://wiki.openstack.org/wiki/Testr#Debugging_.28pdb.29_Tests13:05
skraynevi.e.13:05
*** serg_melikyan has quit IRC13:06
skraynevinstall all requirements in local env13:06
skraynevtestr list-tests test_aws_loadbalancer > lb_test13:07
skraynevpython -m testtools.run discover --load-list lb_list13:07
Qimingokay, will try it13:08
skraynevAlso I think it's possible to do via tox -eintegration -- test_aws_loadbalancer13:08
skraynevbut I can not say that it 100 % works, because uses first way :)13:08
Qimingokay13:09
skraynevalso first way allows to debug with pdb13:09
*** ifarkas has quit IRC13:09
Qimingit runs very fast and passed13:09
Qimingmust be wrong13:09
QimingI tried this: tox -epy26 -- heat_integrationtests.scenario.test_aws_loadbalancer13:10
*** ifarkas has joined #heat13:12
*** dims has quit IRC13:14
*** dims has joined #heat13:15
*** EricGonczer_ has joined #heat13:15
openstackgerritQiming Teng proposed openstack/heat: Make Heat ASG always do rolling_updates  https://review.openstack.org/13794513:15
*** EricGonczer_ has quit IRC13:16
*** jprovazn has quit IRC13:17
*** sarob has quit IRC13:20
*** sarob has joined #heat13:21
*** tlashchova has quit IRC13:24
*** sarob has quit IRC13:26
*** EricGonczer_ has joined #heat13:27
*** sorantis has joined #heat13:28
*** serg_melikyan has joined #heat13:29
skraynevQiming: not sure, that it will work correct13:29
Qimingwill look into this when my plate is not so full13:30
skraynev" it runs very fast and passed " - didi you have heat_integration.conf file with all required options?13:30
skraynevok13:30
*** alexheneveld has quit IRC13:34
*** tlashchova has joined #heat13:36
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed response in properties  https://review.openstack.org/12715013:37
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed exception  https://review.openstack.org/16079213:37
*** hdd has joined #heat13:38
*** LiJiansheng has quit IRC13:40
*** EricGonczer_ has quit IRC13:46
*** jprovazn has joined #heat13:49
openstackgerritMerged openstack/heat: Fix mocking requests in test_engine_service  https://review.openstack.org/16073713:49
*** akuznetsov has joined #heat13:55
*** radez_g0n3 is now known as radez13:57
*** jmckind has joined #heat14:06
*** sarob has joined #heat14:09
*** hdd has quit IRC14:12
*** spzala has joined #heat14:25
*** chlong has joined #heat14:27
*** nkhare has joined #heat14:29
*** Marga_ has joined #heat14:35
*** Marga_ has quit IRC14:37
*** sarob_ has joined #heat14:37
*** Marga_ has joined #heat14:38
*** sarob has quit IRC14:39
*** coolsvap is now known as coolsvap_14:43
*** MVenesio_ has joined #heat14:44
*** MVenesio has quit IRC14:47
openstackgerritThomas Herve proposed openstack/heat: Make Heat ASG always do rolling_updates  https://review.openstack.org/13794514:47
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed exception  https://review.openstack.org/16079214:48
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed response in properties  https://review.openstack.org/12715014:48
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed response in outputs  https://review.openstack.org/16081914:48
*** Drago has quit IRC14:48
*** serg_melikyan has quit IRC14:54
*** hdd has joined #heat14:54
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed exception  https://review.openstack.org/16079214:54
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed response in properties  https://review.openstack.org/12715014:54
openstackgerritPeter Razumovsky proposed openstack/heat: Improve StackValidationFailed response in outputs  https://review.openstack.org/16081914:54
*** serg_melikyan has joined #heat14:59
*** crose has joined #heat15:00
*** che-arne has quit IRC15:00
*** yassine has quit IRC15:03
*** MVenesio has joined #heat15:05
*** nkhare has quit IRC15:06
*** Drago has joined #heat15:07
*** MVenesio_ has quit IRC15:08
*** Drago has quit IRC15:08
*** Drago has joined #heat15:08
*** vijendar has joined #heat15:08
*** KarolynChambers has quit IRC15:09
*** KarolynChambers has joined #heat15:10
*** sorantis has quit IRC15:11
*** aweiteka has quit IRC15:11
*** thedodd has joined #heat15:12
*** aweiteka has joined #heat15:13
*** sarob_ is now known as sarob15:13
*** Marga_ has quit IRC15:13
*** alexheneveld has joined #heat15:14
*** Qiming has quit IRC15:16
*** Qiming has joined #heat15:17
*** ParsectiX has quit IRC15:22
openstackgerritRabi Mishra proposed openstack/heat-templates: WIP heat-config-docker-compose  https://review.openstack.org/16064215:22
*** Tross has quit IRC15:23
*** sorantis has joined #heat15:24
*** alexpilotti has joined #heat15:25
*** serg_melikyan has quit IRC15:27
*** prazumovsky has quit IRC15:28
*** yassine has joined #heat15:30
*** serg_melikyan has joined #heat15:30
*** aweiteka has quit IRC15:36
*** alextricity has joined #heat15:40
openstackgerritRakesh H S proposed openstack/heat: Convergence sync_point DB table  https://review.openstack.org/15573215:42
*** rpothier has joined #heat15:46
*** EricGonczer_ has joined #heat15:47
*** pm90_ has joined #heat15:49
*** rakesh_hs has quit IRC15:52
*** EricGonc_ has joined #heat15:56
*** tspatzier has quit IRC15:58
*** spzala has quit IRC15:58
*** spzala has joined #heat15:59
*** EricGonc_ has quit IRC16:00
*** EricGonc_ has joined #heat16:00
*** EricGonczer_ has quit IRC16:00
*** Tross has joined #heat16:02
*** ekarlso has quit IRC16:03
*** SpamapS has quit IRC16:03
*** SpamapS has joined #heat16:04
*** Drago has quit IRC16:07
*** Drago has joined #heat16:07
*** ekarlso has joined #heat16:09
openstackgerritTetiana Lashchova proposed openstack/heat-specs: Implement datastore_version custom constraint  https://review.openstack.org/16085916:09
*** pitr-ch has quit IRC16:10
openstackgerritGrzegorz Grasza (xek) proposed openstack/heat: First objects - stack and raw template  https://review.openstack.org/14684116:13
*** ifarkas has quit IRC16:13
*** Qiming has quit IRC16:14
openstackgerritTetiana Lashchova proposed openstack/heat-specs: Fix link in cinder-custom-constraints  https://review.openstack.org/16086116:14
*** tspatzier has joined #heat16:30
*** alexpilotti has quit IRC16:31
*** che-arne has joined #heat16:32
openstackgerritGrzegorz Grasza (xek) proposed openstack/heat: First objects - stack and raw template  https://review.openstack.org/14684116:32
*** aweiteka has joined #heat16:33
*** xek has joined #heat16:34
*** otoolee has quit IRC16:36
*** david-lyle_afk has quit IRC16:37
*** reed has joined #heat16:38
*** otoolee has joined #heat16:44
*** cdent has quit IRC16:46
*** jistr has quit IRC16:50
*** thedodd has quit IRC16:51
*** jcoufal has quit IRC16:54
*** serg_melikyan has quit IRC16:57
*** serg_melikyan has joined #heat16:58
alextricitystevebaker: Do you know if there is a python hook for SoftwareConfig?17:02
shardyalextricity: can you explain what you mean?  SoftwareDeployments trigger python "hook scripts" for various CM tools, where the config is encapsulated by a SoftwareConfig resource17:03
alextricityshardy: Thanks for your response. I'm trying to use a python script instead of shell script as part of my software config. Can I still use my inputs and the outputs in my python script?17:04
shardyalextricity: yes I believe so, let me see if I can find an example17:05
shardyhttps://review.openstack.org/#/c/134749/5/hot/software-config/example-templates/example-deployments.yaml17:05
shardythere's an example, you can just specify the python inline with the appropriate shebang17:06
*** alexpilotti has joined #heat17:11
alextricityshardy nice17:12
alextricityThank you17:15
shardynp17:15
*** chlong has quit IRC17:17
*** akuznetsov has quit IRC17:23
*** jpeeler has joined #heat17:26
*** sorantis has quit IRC17:27
*** sarob has quit IRC17:33
*** Drago has quit IRC17:40
*** Drago has joined #heat17:40
*** jprovazn has quit IRC17:42
*** tellesnobrega has quit IRC17:43
openstackgerritOpenStack Proposal Bot proposed openstack/python-heatclient: Updated from global requirements  https://review.openstack.org/15779517:47
*** sarob has joined #heat17:49
*** htruta has quit IRC17:50
*** htruta has joined #heat18:00
openstackgerritSteven Hardy proposed openstack/heat: Convert AutoScalingGroup adjustments to handle_signal  https://review.openstack.org/14349618:00
*** shardy has quit IRC18:01
*** tspatzier has quit IRC18:03
*** tellesnobrega has joined #heat18:04
*** serg_melikyan has quit IRC18:06
*** Marga_ has joined #heat18:07
*** Marga_ has quit IRC18:07
*** Marga_ has joined #heat18:08
*** harlowja_away is now known as harlowja_18:12
*** alexpilotti_ has joined #heat18:16
*** alexpilotti has quit IRC18:16
*** alexpilotti_ is now known as alexpilotti18:16
*** Tango has joined #heat18:17
*** alexheneveld has quit IRC18:21
*** achanda has joined #heat18:23
*** serg_melikyan has joined #heat18:27
*** radez is now known as radez_g0n318:27
*** che-arne has quit IRC18:36
*** dnalezyt has joined #heat18:36
*** pitr-ch has joined #heat18:37
dnalezytI'm looking for a bit of direction on trouble shooting a heat issue with stack-delete.18:38
dnalezytI've got a number of stacks that are stuck in DELETE_IN_PROGRESS that simply refuse to transition to DELETE_COMPLETE or DELETE_FAILED.18:39
dnalezytAfter some digging through log files I found that the issue seems to be with detaching a secondary volume in the stacks.18:39
dnalezytin heat-engine.log I see this same entry repeated:18:40
dnalezyt2015-03-03 11:31:34.070 2804 DEBUG heat.engine.resources.volume [-] Removing attachment 20e17153-09ce-4f64-b9c0-00af7815cdd3 from Instance 77459b56-6dea-47fb-a2ee-0715a91b5168 - volume still in use __call__ /usr/lib/python2.6/site-packages/heat/engine/resources/volume.py:284 2015-03-03 11:31:34.070 2804 DEBUG heat.engine.scheduler [-] Task VolumeDetachTask(20e17153-09ce-4f64-b9c0-00af7815cdd3 -/> 77459b56-6dea-47fb-a2ee-0715a91b5168) s18:40
dnalezytI tried to use the nova api to detach the volume to help things along, but after a few minutes the volume simply reverted to attached an in-use status.18:41
dnalezytI can't seem to find a corresponding error as to why the volume is not being detached in the cinder logs...18:42
dnalezytwhere should I be looking?18:42
larsksdnalezyt: maybe the nova-compute log?18:48
dnalezytgood point, I sometimes forget how interconnected the services are, especially since I was smart enough to use the nova li to try and detach it, just not smart enough to look in those logs.18:50
*** achanda has quit IRC18:57
*** achanda has joined #heat18:57
*** GonZo2K has joined #heat18:59
*** Tross has quit IRC18:59
*** yassine has quit IRC19:01
*** Drago has quit IRC19:05
*** Drago has joined #heat19:05
*** inc0 has joined #heat19:07
dnalezytfinally traced it ddown tot he compute.api on the node hosting the instance.  Looks like a libvirt error, trying to understand it now.  Thanks larsks19:07
larsksGlad to help!19:07
inc0hi guys, any chances you could give me few pointers about that? https://jenkins06.openstack.org/job/gate-heat-python27/3031/console19:07
inc0I can't find reason for it:/19:07
*** jtomasek has quit IRC19:08
*** pitr-ch has quit IRC19:16
*** radez_g0n3 is now known as radez19:20
*** serg_melikyan has quit IRC19:22
*** jdob has quit IRC19:23
*** jdob has joined #heat19:24
*** sdake_ has joined #heat19:34
*** inc0 has quit IRC19:36
*** lsmola has quit IRC19:36
*** alexheneveld has joined #heat19:41
*** MasterPiece has joined #heat19:45
*** lsmola has joined #heat19:49
*** Tross has joined #heat19:51
*** pm90_ has quit IRC19:52
*** dnalezyt has quit IRC19:52
*** Marga_ has quit IRC19:54
*** Marga_ has joined #heat19:54
*** pm90_ has joined #heat20:06
*** pm90_ has quit IRC20:06
*** pm90_ has joined #heat20:07
*** liusheng has quit IRC20:10
*** liusheng has joined #heat20:11
*** Marga_ has quit IRC20:15
*** Marga_ has joined #heat20:16
*** alexheneveld has quit IRC20:17
*** sdake_ has quit IRC20:19
*** yassine has joined #heat20:25
*** radez is now known as radez_g0n320:26
*** MasterPiece has quit IRC20:26
*** kebray has joined #heat20:29
*** achanda has quit IRC20:38
*** kebray has quit IRC20:41
openstackgerritSteve Baker proposed openstack/python-heatclient: Make ; parsing optional in format_parameters  https://review.openstack.org/16023820:43
openstackgerritSteve Baker proposed openstack/python-heatclient: Implement deployment-output-show  https://review.openstack.org/16023920:43
openstackgerritSteve Baker proposed openstack/python-heatclient: Implement deployment-create  https://review.openstack.org/16024020:43
*** yassine has quit IRC20:43
*** achanda has joined #heat20:46
*** alexheneveld has joined #heat20:49
*** asalkeld has joined #heat20:50
*** che-arne has joined #heat20:53
*** kebray has joined #heat21:00
*** crose has quit IRC21:00
*** aweiteka has quit IRC21:03
*** jpeeler has quit IRC21:03
*** radez_g0n3 is now known as radez21:06
*** alexheneveld has quit IRC21:10
openstackgerritMerged openstack/heat-templates: Start of docker hook unit tests  https://review.openstack.org/13614521:13
*** sdake_ has joined #heat21:17
*** aweiteka has joined #heat21:18
*** thedodd has joined #heat21:20
*** kebray has quit IRC21:20
openstackgerritSteve Baker proposed openstack/heat-templates: Boot config for installing software-config agent  https://review.openstack.org/11928221:22
openstackgerritSteve Baker proposed openstack/heat-templates: Boot config environment for test images  https://review.openstack.org/15817221:22
openstackgerritSteve Baker proposed openstack/heat-templates: Elements for building a heat test image  https://review.openstack.org/15967021:22
stevebakerasalkeld, cores: I really need ^ to land so I can work on functional test images21:23
asalkeldstevebaker: ok - just in the project meeting21:24
stevebakerok21:24
*** sdake_ has quit IRC21:25
*** pm90_ has quit IRC21:28
*** pm90_ has joined #heat21:32
*** pm90_ has quit IRC21:32
openstackgerritMerged openstack/heat: Remove the template mapping from the env before creating child stacks  https://review.openstack.org/15654721:32
*** pm90_ has joined #heat21:33
*** blomquisg has quit IRC21:36
*** jpeeler has joined #heat21:38
*** jpeeler has quit IRC21:38
*** jpeeler has joined #heat21:38
*** EmilienM has quit IRC21:44
*** EmilienM has joined #heat21:44
openstackgerritMerged openstack/heat: Don't pass the parent_resource object into Stack()  https://review.openstack.org/15654621:44
*** sdake_ has joined #heat21:47
*** achanda has quit IRC22:01
*** achanda has joined #heat22:07
*** MVenesio has quit IRC22:08
*** EricGonc_ has quit IRC22:21
*** KarolynChambers has quit IRC22:23
*** serg_melikyan has joined #heat22:23
*** rpothier has quit IRC22:23
*** KarolynChambers has joined #heat22:23
openstackgerritWinnie Tsang proposed openstack/python-heatclient: Reuse existing registry and/or existing template  https://review.openstack.org/15469222:24
*** serg_melikyan has quit IRC22:27
*** vijendar has quit IRC22:30
*** sarob has quit IRC22:31
*** Marga_ has quit IRC22:35
*** sdake_ has quit IRC22:37
*** Tango has quit IRC22:43
*** openstackgerrit has quit IRC22:51
*** openstackgerrit has joined #heat22:52
*** pm90_ has quit IRC22:53
*** alexheneveld has joined #heat22:56
asalkeldinteresting: http://goless.readthedocs.org/en/latest/23:01
*** jmckind has quit IRC23:01
*** jpeeler has quit IRC23:02
*** pm90_ has joined #heat23:06
asalkeldstevebaker: just waiting for verified23:09
stevebakerasalkeld: ok23:10
*** sarob has joined #heat23:12
*** alexpilotti has quit IRC23:19
*** blomquisg has joined #heat23:22
*** alexheneveld has quit IRC23:32
*** chlong has joined #heat23:32
*** aweiteka has quit IRC23:32
ramishrastevebaker: Hi23:34
stevebakerramishra:  hi23:34
ramishrastevebaker: saw your comment for https://review.openstack.org/#/c/160642/223:35
stevebakeryep23:35
ramishrastevebaker: fortunately docker-compose kill cleans up all container for the porject..23:36
ramishrastevebaker: deployment_id is being used as the project name23:36
ramishrastevebaker: so it'll clean up all services in the project, based on the existing docker-compse.yml23:37
ramishrastevebaker:  in that folder(project_name)23:37
stevebakerramishra: ok, so all that is needed is some way to call cleanup_containers for deployments which are no longer in the deployment list23:42
ramishrastevebaker: yeah.. yeah..for every update it'll check if the docker-compose.yml exists and then calls kill to cleanup..23:44
*** tochi has joined #heat23:44
stevebakerramishra: I'd rather not kill and recreate the same containers on every update run, that is a bit brutal23:45
ramishrastevebaker: it does a complete cleanup.. that's how it works in docker-compose at the moment.. As there may be changes to the images, some changes to the container.. this is an issue with docker itlsef23:46
ramishrastevebaker: anyway, docker compose would recreate the changes to the services in the new docker-compose.yml.. the only reason we're doing to cleanup is to remove the services removed from the new docker-compse.yml23:49
*** achanda has quit IRC23:49
stevebakerramishra: yes, making sure the deployment always runs with the DELETE action is one way of deleting containers, but the user may forget to specify DELETE action on their deployment resource so it should really handle a deployment entry just disappearing too23:49
ramishrastevebaker:  I mean it recreates the services by default is an issue with 'docker-compose'23:50
ramishrastevebaker: irrepsective of their existence and no change to them23:50
stevebakerramishra: also you could consider implementing some of this as an orc script instead of a hook, like the kubelet one. This gives you access to the entire deployments list so you can compare the list with the compose files and kill the ones which have gone, see https://review.openstack.org/#/c/140887/7/hot/software-config/elements/heat-config-kubelet/os-refresh-config/configure.d/50-heat-config-kubelet23:51
stevebakerramishra: I think best practice would be to include the version in the image name (rather than just latest). This means containers would get redeployed when they change the version. If the user really wants to redeploy their unversioned 'latest' image they can also change something else in the config or deployment resource to trigger redeploy23:54
*** achanda has joined #heat23:56
ramishrastevebaker: I'm more worried out looking at the changes in the services/container properties.. if anything like a port changes, we've to recreate the container.. there is no other way in docker.23:57
*** Qiming has joined #heat23:58
stevebakerramishra: yes, and *any* change to a config or deployment resource results in a completely different c['id'], so you can rely on this behaviour to trigger redeploys23:58

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