| *** achanda_ has joined #heat | 00:00 | |
| *** Marga_ has quit IRC | 00:03 | |
| *** achanda has quit IRC | 00:04 | |
| *** achanda_ has quit IRC | 00:05 | |
| *** radez is now known as radez_g0n3 | 00:07 | |
| openstackgerrit | Angus Salkeld proposed openstack/heat: Assert that we can update TemplateResources with a new file path https://review.openstack.org/180847 | 00:12 |
|---|---|---|
| *** KanagarajM has joined #heat | 00:19 | |
| *** jamielennox|away is now known as jamielennox | 00:29 | |
| *** Drago has quit IRC | 00:37 | |
| *** alanf-mc has quit IRC | 00:46 | |
| *** jtomasek has quit IRC | 00:48 | |
| openstackgerrit | Angus Salkeld proposed openstack/heat: Assert that we can update TemplateResources with a new file path https://review.openstack.org/180847 | 01:04 |
| *** Qiming has joined #heat | 01:04 | |
| *** mkoderer has quit IRC | 01:05 | |
| *** mkoderer has joined #heat | 01:14 | |
| *** sambol has joined #heat | 01:14 | |
| *** Qiming_ has joined #heat | 01:15 | |
| *** Qiming has quit IRC | 01:18 | |
| *** huangtianhua has joined #heat | 01:23 | |
| *** erkules_ has joined #heat | 01:37 | |
| *** Qiming_ is now known as Qiming | 01:38 | |
| *** huangtianhua has quit IRC | 01:40 | |
| *** erkules has quit IRC | 01:40 | |
| KanagarajM | good morning all ! | 01:42 |
| asalkeld | hi KanagarajM | 01:43 |
| KanagarajM | asalkeld: hi, for the https://review.openstack.org/173710, to remove the duplicate validatio of template, | 01:45 |
| asalkeld | ok | 01:45 |
| KanagarajM | asalkeld: I am thinking of going in two phases 1. introduce hasing as like now 2. and once dogpile caching backend available, use it | 01:46 |
| asalkeld | KanagarajM: ok, sounds fine - also chat to kairat_kushaev about what he is going to do | 01:46 |
| asalkeld | so it all "fits" together | 01:47 |
| KanagarajM | asalkeld: sure. | 01:47 |
| *** kebray has joined #heat | 01:55 | |
| KanagarajM | asalkeld: i have updated specs for designate & Monasaca resource plugins, could you kindly review them https://review.openstack.org/174262 & https://review.openstack.org/174275 | 01:55 |
| asalkeld | sure thing | 01:55 |
| *** kebray has quit IRC | 01:55 | |
| asalkeld | KanagarajM: is it worth defining the attributes? | 01:56 |
| asalkeld | can they be consistent with the ceilo alarms | 01:56 |
| asalkeld | i.e. can the monesca alarms work with our policy? | 01:57 |
| asalkeld | i see the notifcation has a webhook | 01:57 |
| asalkeld | so it looks like it | 01:57 |
| *** xuhaiwei has quit IRC | 01:57 | |
| KanagarajM | asalkeld: i tired to make it consitent, but monasca has alaram and notification as seprate thing in their API design | 01:58 |
| KanagarajM | so i tried to keep the spec in consistent with monsaca | 01:59 |
| asalkeld | KanagarajM: that's fine, i meant more "lets make sure we can use this with our autoscaling policy" | 01:59 |
| asalkeld | maybe a work item to write an example template? | 02:00 |
| *** dims_ has quit IRC | 02:00 | |
| *** dims has joined #heat | 02:00 | |
| KanagarajM | sure. I will add it for example template. | 02:00 |
| *** dims has quit IRC | 02:01 | |
| *** huangtianhua has joined #heat | 02:01 | |
| KanagarajM | as webhook is the only one used by heat autoscalling, do we need to support for email and other too? | 02:01 |
| KanagarajM | in the monasca notification type. | 02:02 |
| asalkeld | KanagarajM: i think just the webhook | 02:03 |
| Qiming | KanagarajM, regarding my objections to model the Notification resource type, please check this: http://lists.openstack.org/pipermail/openstack-dev/2014-January/025889.html | 02:03 |
| asalkeld | Qiming: that "notifcation" is more of a registration to receive the notifcaiton isn't it? | 02:04 |
| asalkeld | like a filter | 02:05 |
| Qiming | yes, I think so | 02:05 |
| *** Raj1 has joined #heat | 02:05 | |
| Qiming | asalkeld, I am really seeing it an inseparable property of the alarm | 02:05 |
| *** Raj2 has joined #heat | 02:05 | |
| asalkeld | Qiming: as long as it returns a uuid that can be tracked, imo it's fine | 02:06 |
| asalkeld | and it is a "noun" | 02:07 |
| Qiming | asalkeld, sure thing, there is nothing wrong for Heat to create it and track it | 02:07 |
| asalkeld | just poorly named | 02:07 |
| Qiming | my concern is about the users | 02:07 |
| Qiming | do they really want to know it, need to know it | 02:07 |
| KanagarajM | Qiming: i assumed that heat resources should be matching with correspsonding service api model. as it does not created any confustion to the user when they use monasca from either API/CLI and heat templates | 02:07 |
| asalkeld | Qiming: if you are familar with monasca and it is missing, then that is unexpected | 02:07 |
| Qiming | and what are the additional values we are providing if we provide an additional resource type to learn | 02:08 |
| asalkeld | Qiming: it's just modelling the new services | 02:08 |
| Qiming | KanagarajM, on the contrarary, I don't think users are interested in MonascaAlarm, GonnociAlarm ... BlablaAlarm | 02:09 |
| asalkeld | unless there is something "broken " with the api, we should expose the services full power | 02:09 |
| *** Raj1 has quit IRC | 02:09 | |
| asalkeld | Qiming: we *could* make an abstraction "Alarm" | 02:09 |
| asalkeld | that uses what ever is available | 02:09 |
| *** david-lyle has joined #heat | 02:10 | |
| asalkeld | but we should also expose the native api | 02:10 |
| Qiming | asalkeld, that would be really really great | 02:10 |
| KanagarajM | i think we could achieve this by using environment in heat, if i try to assume correctly | 02:10 |
| asalkeld | KanagarajM: maybe, but it need discovery to figure out what is availble | 02:11 |
| Qiming | asalkeld, Heat make uses of APIs of existing services, but I don't think we really need to expose the details to users, unless there are benefits to them | 02:11 |
| ramishra | morning all | 02:12 |
| KanagarajM | asalkeld: yes, i agress | 02:12 |
| asalkeld | Qiming: heat is just a yaml to rest converter, but I also agree that we can make new resources that help users that have resonably straight forward needs to abstract some of the complexity - and make more portable templates | 02:14 |
| Qiming | asalkeld, exactly my point | 02:14 |
| Qiming | could not agree more | 02:14 |
| KanagarajM | asalkeld, Qiming: only for consistent point of view, I tried to keep Notification as seprate resource. which gives same user experience to user when they try to used monasaca across API, CLI and template. | 02:14 |
| asalkeld | the problem with abstractions is they are the lowest common demoninator i.e no advanced features from anything | 02:14 |
| asalkeld | some people want full power of the underlying api | 02:15 |
| asalkeld | so we really need both IMHO | 02:15 |
| Qiming | asalkeld, it is really a tough tradeoff to make | 02:15 |
| asalkeld | Qiming: seems like you need a spec for generic alarming | 02:15 |
| asalkeld | autodiscoverable etc... | 02:16 |
| Qiming | asalkeld, we are trying to do so in Senlin | 02:16 |
| asalkeld | we need that for nova-net/neutron | 02:16 |
| Qiming | an alarm with a 'driver' property, if users really need it | 02:16 |
| asalkeld | i see, looking forward to the "deep dive" | 02:17 |
| KanagarajM | for autodiscover, there is a spec https://blueprints.launchpad.net/heat/+spec/keystone-based-resource-availability. | 02:17 |
| *** Yanyanhu has joined #heat | 02:18 | |
| KanagarajM | i believe it might help. | 02:18 |
| asalkeld | this is going to be a more and more important issue as the "big tent" grows, with overlapping features and serivces that are installed inconsistently in different clouds | 02:19 |
| asalkeld | good bye to portablity | 02:20 |
| *** kebray has joined #heat | 02:20 | |
| KanagarajM | yes. seems to be. | 02:20 |
| Qiming | sad thing for users | 02:21 |
| asalkeld | KanagarajM: you *could* make a extra Alarm that was totally compatible with the existing ones? | 02:22 |
| asalkeld | so people just need to patch their environment | 02:22 |
| asalkeld | KanagarajM: Qiming does that seem like a reasonable compromise? | 02:23 |
| Qiming | that would be cool | 02:23 |
| KanagarajM | to understand correctly, provide an additional monasca Alarm resource, which matches extactly the celiometer alarm. | 02:24 |
| asalkeld | yeah | 02:24 |
| asalkeld | not sure how to name that | 02:24 |
| KanagarajM | ok. sure. that would achieve all of our needs. thanks. | 02:25 |
| asalkeld | np | 02:25 |
| * asalkeld the mediator ;) | 02:25 | |
| asalkeld | lunch time | 02:26 |
| *** achanda has joined #heat | 02:29 | |
| *** zzxwill has joined #heat | 02:29 | |
| zzxwill | How to enable heat just generate DEBUG/ERROR/CRITICAL log? | 02:30 |
| asalkeld | zzxwill: it will by default log info and above | 02:31 |
| asalkeld | but go to /etc/heat/heat.conf | 02:31 |
| asalkeld | and set debug=true | 02:31 |
| zzxwill | I've set it to true, but there are still "INFO" log and logs with level, like "[05/07/2015 09:51:08 EDT]sqlalchemy.engine.base.EngineINFO : SELECT raw_template.created_at AS raw_template_created_at" | 02:33 |
| zzxwill | asalkeld: Is it possible that there are any other files control the level of the heat log? | 02:35 |
| asalkeld | only if you have log_config set | 02:37 |
| zzxwill | Where can I set log_config? I'd like to check the setting. | 02:38 |
| asalkeld | zzxwill: https://github.com/openstack/oslo.log/blob/master/oslo_log/_options.py | 02:38 |
| *** andersonvom has joined #heat | 02:38 | |
| asalkeld | that explains all the logging options ^ | 02:38 |
| * asalkeld now eating lunch ... | 02:38 | |
| *** sambol has quit IRC | 02:39 | |
| *** andersonvom has quit IRC | 02:43 | |
| *** Raj2 has quit IRC | 02:43 | |
| *** sambol has joined #heat | 02:54 | |
| zzxwill | I didn't update that _options.py. | 02:55 |
| Qiming | https://github.com/openstack/oslo.log/blob/master/oslo_log/_options.py#L87 | 02:55 |
| Qiming | sqlalchemy log level was default to WARN | 02:55 |
| *** dims has joined #heat | 03:01 | |
| *** dims has quit IRC | 03:06 | |
| *** KanagarajM_ has joined #heat | 03:15 | |
| ramishra | stevebaker, asalkeld Hi! 'gate-heat-templates-dsvm' does not seem to have heat enabled, https://review.openstack.org/#/c/181286/ | 03:18 |
| *** KanagarajM has quit IRC | 03:18 | |
| asalkeld | ramishra: heat is not on by default anymore | 03:20 |
| ramishra | asalkeld: yeah. not good though:) all tests were failing and it took me sometime to realise it:( | 03:22 |
| asalkeld | https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/heat.yaml#L2-L27 | 03:22 |
| asalkeld | that needs updating | 03:22 |
| *** achanda has quit IRC | 03:22 | |
| ramishra | asalkeld: done that. thats the review https://review.openstack.org/#/c/181286/ | 03:22 |
| asalkeld | export DEVSTACK_GATE_HEAT=1 | 03:23 |
| asalkeld | ? | 03:23 |
| asalkeld | either works | 03:23 |
| *** baikai has joined #heat | 03:24 | |
| *** baikai has quit IRC | 03:32 | |
| *** andersonvom has joined #heat | 03:39 | |
| *** KanagarajM_ has quit IRC | 03:42 | |
| *** andersonvom has quit IRC | 03:44 | |
| *** EricGonczer_ has joined #heat | 03:44 | |
| sirushti | ramishra, shouldn't the other services also be enabled since we'd also validate CustomConstraints too on validate? no? | 03:49 |
| *** sambol has quit IRC | 03:53 | |
| stevebaker | ramishra: yeah, that would help. It is no longer enabled by default | 03:54 |
| *** Marga_ has joined #heat | 03:58 | |
| *** Marga_ has quit IRC | 04:00 | |
| *** Marga_ has joined #heat | 04:00 | |
| *** ashishjain has quit IRC | 04:01 | |
| *** EricGonczer_ has quit IRC | 04:03 | |
| *** kebray has quit IRC | 04:04 | |
| *** andersonvom has joined #heat | 04:06 | |
| openstackgerrit | Merged openstack/heat-specs: Search Resource Type https://review.openstack.org/168662 | 04:10 |
| *** pal has joined #heat | 04:11 | |
| *** sdake has quit IRC | 04:13 | |
| *** achanda has joined #heat | 04:22 | |
| *** achanda has quit IRC | 04:28 | |
| *** KanagarajM has joined #heat | 04:32 | |
| KanagarajM | asalkeld: on monasca spec, for adding the new alaram, which is compatiable with existing alrams, shall i name it as OS::Monasca::AlarmI , I for improved | 04:42 |
| *** achanda has joined #heat | 04:42 | |
| KanagarajM | asalkeld: in which, we could allow user directly feed only web hook, | 04:43 |
| *** Yanyanhu has quit IRC | 04:45 | |
| *** Yanyanhu has joined #heat | 04:45 | |
| asalkeld | KanagarajM: i honestly don't know | 04:50 |
| asalkeld | people love to bike shed names:0 | 04:50 |
| asalkeld | brb picking kids up | 04:50 |
| KanagarajM | asalkeld: ok. i will keep with that name and any better names come as part of review, i will update it. | 04:53 |
| *** aweiteka has quit IRC | 04:53 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat-specs: Monasca resource plugin for Alarm and Notification https://review.openstack.org/174262 | 04:56 |
| KanagarajM | Qiming: could you kindly review ^ once, whether it sastify our needs as discussed earlier | 04:57 |
| *** gokrokve has joined #heat | 04:58 | |
| *** vijayagurug has joined #heat | 05:01 | |
| KanagarajM | elynn: hi, are you here? | 05:08 |
| *** gokrokve has quit IRC | 05:09 | |
| *** gokrokve has joined #heat | 05:10 | |
| *** gokrokve has quit IRC | 05:13 | |
| *** gokrokve has joined #heat | 05:13 | |
| *** gokrokve has quit IRC | 05:18 | |
| *** huangtianhua has quit IRC | 05:19 | |
| *** baikai has joined #heat | 05:20 | |
| *** Qiming has quit IRC | 05:21 | |
| *** baikai_ has joined #heat | 05:22 | |
| *** Qiming has joined #heat | 05:22 | |
| *** baikai_ has quit IRC | 05:22 | |
| baikai | Hi | 05:24 |
| *** baikai has quit IRC | 05:27 | |
| *** sabeen1 has quit IRC | 05:36 | |
| *** gokrokve has joined #heat | 05:40 | |
| *** gokrokve has quit IRC | 05:42 | |
| *** gokrokve has joined #heat | 05:42 | |
| sirushti | asalkeld, stevebaker, just a heads up that I've posted a project-config patch for non-voting grenade jobs -> https://review.openstack.org/#/c/178352/ | 05:44 |
| *** inc0 has joined #heat | 05:48 | |
| *** pal has quit IRC | 05:48 | |
| *** pal has joined #heat | 05:48 | |
| elynn | Hi KanagarajM | 05:58 |
| elynn | Leave for lunch before. | 05:58 |
| *** Yanyan has joined #heat | 06:19 | |
| KanagarajM | elynn: hi | 06:19 |
| elynn | Hi kairat_kushaev | 06:21 |
| elynn | Hi KanagarajM | 06:22 |
| *** Yanyanhu has quit IRC | 06:22 | |
| elynn | about your patch? | 06:22 |
| KanagarajM | elynn: yes :) | 06:22 |
| elynn | In handle_update() I didn't get the logic, is it just leave it None and then keystone will not update it? | 06:23 |
| KanagarajM | elynn: if i remember correctly, during kilo, I got review comments to incorporate this way, to behave in the similar mode during create and update | 06:25 |
| KanagarajM | elynn, when the name is not provided, fall back to the physical resource name. | 06:25 |
| *** huangtianhua has joined #heat | 06:27 | |
| elynn | In what case the name will not be provided? | 06:27 |
| elynn | since it's required | 06:27 |
| KanagarajM | when the first part zaneb provided the sepc, zaneb wanted to go with name as not required property and in this case go with default physical resource name. and this is useful in case of any automated test environment. | 06:29 |
| elynn | I recheck your code, find out that name is not required, my bad. | 06:30 |
| elynn | I have another question | 06:30 |
| elynn | If I create a service with name='foo' | 06:31 |
| zzxwill | @Qiming: Got it. Thanks: | 06:31 |
| elynn | the update it ,but leave the name='foo' | 06:31 |
| elynn | will this name be changed? | 06:31 |
| KanagarajM | no, it won't change | 06:32 |
| elynn | Your code is name = (prop_diff.get(self.NAME) or self.physical_resource_name()) | 06:32 |
| elynn | self.physical_resource_name() will return 'foo'? | 06:32 |
| KanagarajM | yes | 06:33 |
| elynn | https://github.com/openstack/heat/blob/master/heat/engine/resource.py#L854-L865 | 06:34 |
| KanagarajM | let me check | 06:34 |
| elynn | Should you override the physical_resource_name() function in your code? | 06:34 |
| elynn | like https://github.com/openstack/heat/blob/master/contrib/heat_zaqar/heat_zaqar/resources/queue.py#L67-L68 | 06:35 |
| KanagarajM | i don't think it will return foo. it will be different, | 06:35 |
| elynn | You finally get my point. | 06:36 |
| KanagarajM | thanks for pointing out. i think, if physical_resource_name() is implemented to return the Name, it might fail during create | 06:37 |
| *** achanda has quit IRC | 06:38 | |
| KanagarajM | instead, we could go with name = (prop_diff.get(self.NAME) or self.properties.get(self.NAME) or self.physical_resource_name()) | 06:38 |
| elynn | hmm, maybe just change your code logic is enough. | 06:38 |
| KanagarajM | sure, i wanted to confirm with you. thanks :) | 06:38 |
| elynn | Why not just leave the name = None if not update it. | 06:38 |
| elynn | https://github.com/openstack/heat/blob/master/contrib/heat_zaqar/heat_zaqar/resources/queue.py#L67-L68 | 06:38 |
| elynn | sorry , wrong link | 06:39 |
| elynn | https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/v3/services.py#L61-L69 | 06:39 |
| KanagarajM | it might itroduce inconsistency with other keystone resources. | 06:39 |
| elynn | There are two situation here. | 06:40 |
| elynn | 1. name didn't change when update, then we don't need to change name property. | 06:41 |
| elynn | 2. name change from 'foo' to None, then we should use self.physical_resource_name() for it. | 06:42 |
| elynn | Is that right? | 06:42 |
| KanagarajM | 2nd case is taken care in the handle_update, i believe | 06:42 |
| elynn | Hmm, seems name = (prop_diff.get(self.NAME) or self.properties.get(self.NAME) or self.physical_resource_name()) can work. | 06:43 |
| KanagarajM | 1st case, yes, only update those properties which are changed, and leave the remaining one as its . | 06:43 |
| kairat_kushaev | elynn: Hi | 06:44 |
| *** Marga_ has quit IRC | 06:44 | |
| KanagarajM | elynn: i will update by considering these cases in place. thanks | 06:45 |
| elynn | I also notice that there's a 'enabled' option for services, could you also add it? | 06:47 |
| *** Qiming_ has joined #heat | 06:48 | |
| *** Qiming has quit IRC | 06:48 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Removes redundant validation of template (1) https://review.openstack.org/173710 | 06:48 |
| KanagarajM | elynn: though enabled is good property to add, it is not documented by keystone. so while the spec was being reviewed, skernav suggested to remove it | 06:48 |
| elynn | ok, i'm fine with it. | 06:49 |
| *** nkhare has joined #heat | 06:49 | |
| KanagarajM | elynn: ok | 06:49 |
| *** pitr-ch has quit IRC | 06:53 | |
| *** pitr-ch has joined #heat | 06:56 | |
| KanagarajM | kairat_kushaev: hi | 06:58 |
| kairat_kushaev | KanagarajM: hi | 06:58 |
| KanagarajM | kairat_kushaev, I'm working on the issue of avoiding the redudant validation of same template https://review.openstack.org/173710, and it might need the dogpile cache backend | 07:00 |
| KanagarajM | to store and retrive the check of validation is done or not. | 07:00 |
| KanagarajM | so i would like get your help on incorporating it. | 07:01 |
| *** jrist has quit IRC | 07:02 | |
| *** jrist has joined #heat | 07:03 | |
| kairat_kushaev | KanagarajM: Hmm, interesting. Yep, I am going to start work on this next week. | 07:04 |
| kairat_kushaev | KanagarajM: But just interesting why do you need to store these checks in cache? | 07:04 |
| KanagarajM | kairat_kushaev: ok. in case of instance_group case, the same template is being used and it might be processed across different engines | 07:05 |
| kairat_kushaev | KanagarajM: When I created blueprint I was going to cache some values to avoid redundant API request | 07:05 |
| kairat_kushaev | KanagarajM: ah, got it. So will it be appropriate if I am starting this work next week? | 07:07 |
| KanagarajM | kairat_kushaev: ok. I was assuming that same cache backend could be used for this issue. | 07:07 |
| kairat_kushaev | KanagarajM: yep | 07:08 |
| KanagarajM | kairat_kushaev: sure. no problem. i can wait for. | 07:09 |
| openstackgerrit | Merged openstack/heat: cleanup of hacking.rst heat doc https://review.openstack.org/177922 | 07:19 |
| openstackgerrit | Sergey Kraynev proposed openstack/python-heatclient: Add functional tests from Tempest https://review.openstack.org/180217 | 07:19 |
| openstackgerrit | Sergey Kraynev proposed openstack/python-heatclient: Add OS_TEST_PATH to testr https://review.openstack.org/180216 | 07:19 |
| openstackgerrit | Sergey Kraynev proposed openstack/python-heatclient: Add first pass at post_test_hook for functional https://review.openstack.org/180539 | 07:19 |
| openstackgerrit | Sergey Kraynev proposed openstack/python-heatclient: Move unittests to subdir unit https://review.openstack.org/180215 | 07:19 |
| skraynev | good morning | 07:20 |
| *** BManojlovic has joined #heat | 07:20 | |
| openstackgerrit | Merged openstack/heat-specs: Cinder volume encryption support https://review.openstack.org/168675 | 07:27 |
| *** huangtianhua has quit IRC | 07:31 | |
| *** ifarkas has joined #heat | 07:31 | |
| *** huangtianhua has joined #heat | 07:33 | |
| *** jamielennox is now known as jamielennox|away | 07:36 | |
| *** andersonvom has quit IRC | 07:38 | |
| *** alexheneveld has joined #heat | 08:05 | |
| *** tobe has joined #heat | 08:05 | |
| *** Qiming_ is now known as Qiming | 08:09 | |
| *** chlong has quit IRC | 08:20 | |
| *** Tango|2 has quit IRC | 08:20 | |
| *** tobe has quit IRC | 08:31 | |
| *** andersonvom has joined #heat | 08:35 | |
| *** sergmelikyan has quit IRC | 08:35 | |
| *** derekh has joined #heat | 08:37 | |
| *** andersonvom has quit IRC | 08:40 | |
| *** tspatzier has joined #heat | 08:40 | |
| *** tochi has quit IRC | 08:49 | |
| *** BManojlovic has quit IRC | 08:53 | |
| openstackgerrit | Merged openstack/heat: changed heat dev docs to comply with conventions https://review.openstack.org/177516 | 08:54 |
| *** cdent has joined #heat | 08:56 | |
| sirushti | elynn, around? | 08:58 |
| elynn | Hi sirushti | 08:58 |
| sirushti | so re:180903, is there a reason you don't want the constraint? | 08:58 |
| *** BManojlovic has joined #heat | 09:00 | |
| elynn | We can leave that for discussion open for others. | 09:01 |
| elynn | Since many project doesn't constraint name in db. | 09:01 |
| elynn | If in v2 API design that we decide to use stack-id for API instead of name, we will need to remove that constraint. | 09:03 |
| elynn | Maybe in the future we can support create stacks with same name. | 09:04 |
| sirushti | elynn, Ok, if it's a pattern we want to follow to be consistent with the other projects, we should atleast use a DB transaction for it? | 09:06 |
| elynn | Could you be more explicit? | 09:08 |
| elynn | Like old patch do? | 09:08 |
| sirushti | elynn, yeah, similar to that, basically I just want the get+update to be atomic | 09:09 |
| sirushti | since the problem is when two engines read the database at the same time before either of them write to the database, they have the context that there is no stack in the database and hence write to the database with duplicate stack names | 09:11 |
| elynn | I think only add constraint on tenant+stack_name can totally remove this problem. | 09:13 |
| elynn | Seems DB transaction may not help, see comments from old patch. | 09:14 |
| sirushti | elynn, sure, that definitely is simpler :) I was only suggesting alternatives since you didn't want to do that in the first place. | 09:15 |
| sirushti | elynn, transactions will work actually, it's just not a very clean solution for the problem | 09:17 |
| elynn | Yes , I'm not a fan of it. My patch only intend to reduce the possibility to create stacks for same name. | 09:17 |
| elynn | Maybe we can discuss that in maillist, if others agree that solution. | 09:21 |
| sirushti | elynn, I suppose other reviewers will eventually just weigh in their opinions | 09:22 |
| *** nkhare has quit IRC | 09:24 | |
| *** asalkeld has quit IRC | 09:34 | |
| *** andersonvom has joined #heat | 09:35 | |
| openstackgerrit | Merged openstack/heat: Assert that we can update TemplateResources with a new file path https://review.openstack.org/180847 | 09:36 |
| *** andersonvom has quit IRC | 09:40 | |
| *** Yanyan has quit IRC | 09:41 | |
| *** alexpilotti has joined #heat | 09:49 | |
| *** coolsvap|afk is now known as coolsvap | 10:03 | |
| *** LiJiansheng has joined #heat | 10:31 | |
| *** zzxwill has quit IRC | 10:31 | |
| *** hdd has joined #heat | 10:34 | |
| *** LiJiansheng has quit IRC | 10:45 | |
| *** hdd has quit IRC | 10:45 | |
| *** Qiming has quit IRC | 10:49 | |
| mattt | stevebaker: will https://review.openstack.org/#/c/179671/ get backported to kilo ? | 10:51 |
| *** huangtianhua has quit IRC | 10:53 | |
| *** dims has joined #heat | 11:13 | |
| *** pal has quit IRC | 11:19 | |
| *** zzxwill has joined #heat | 11:28 | |
| *** dims has quit IRC | 11:28 | |
| *** dims has joined #heat | 11:29 | |
| *** smccully has quit IRC | 11:36 | |
| *** alexheneveld has quit IRC | 11:39 | |
| *** hdd has joined #heat | 11:40 | |
| *** aweiteka has joined #heat | 11:45 | |
| *** Qiming has joined #heat | 11:54 | |
| *** EricGonczer_ has joined #heat | 11:55 | |
| *** hdd has quit IRC | 12:01 | |
| *** rpothier has joined #heat | 12:02 | |
| *** sdake has joined #heat | 12:07 | |
| *** sdake_ has joined #heat | 12:10 | |
| *** derekh is now known as derekh_afk | 12:11 | |
| *** sdake has quit IRC | 12:14 | |
| *** MVenesio has joined #heat | 12:14 | |
| *** KanagarajM has quit IRC | 12:15 | |
| *** alexheneveld has joined #heat | 12:31 | |
| *** vijayagurug has left #heat | 12:31 | |
| *** EricGonczer_ has quit IRC | 12:33 | |
| *** jaime49 has joined #heat | 12:36 | |
| *** Qiming_ has joined #heat | 12:42 | |
| *** sdake has joined #heat | 12:44 | |
| *** Qiming has quit IRC | 12:46 | |
| *** chlong has joined #heat | 12:47 | |
| *** sdake_ has quit IRC | 12:47 | |
| *** inc0 has quit IRC | 12:48 | |
| *** jrist has quit IRC | 12:56 | |
| *** jrist has joined #heat | 12:57 | |
| *** derekh_afk is now known as derekh | 13:00 | |
| *** hdd has joined #heat | 13:01 | |
| *** hdd has quit IRC | 13:03 | |
| *** dyasny has joined #heat | 13:10 | |
| *** cdent has quit IRC | 13:12 | |
| *** EmilienM|afk is now known as EmilienM | 13:15 | |
| *** dims has quit IRC | 13:19 | |
| *** dims has joined #heat | 13:20 | |
| *** jaime49 has quit IRC | 13:27 | |
| *** jaime49 has joined #heat | 13:28 | |
| *** jaime491 has joined #heat | 13:29 | |
| *** jaime491 has quit IRC | 13:29 | |
| *** jaime491 has joined #heat | 13:30 | |
| *** jaime49 has quit IRC | 13:30 | |
| *** jaime49 has joined #heat | 13:31 | |
| *** sabeen1 has joined #heat | 13:32 | |
| *** crose has joined #heat | 13:34 | |
| *** jaime491 has quit IRC | 13:36 | |
| *** coolsvap is now known as coolsvap|afk | 13:37 | |
| *** rpothier has quit IRC | 13:41 | |
| *** Raj1 has joined #heat | 13:43 | |
| *** andersonvom has joined #heat | 13:44 | |
| *** cdent has joined #heat | 13:46 | |
| *** coolsvap has joined #heat | 13:47 | |
| *** andersonvom has quit IRC | 13:48 | |
| *** jaime49 has quit IRC | 13:50 | |
| *** MVenesio has quit IRC | 13:50 | |
| *** jaime49 has joined #heat | 13:50 | |
| *** openstackgerrit has quit IRC | 13:51 | |
| *** dims is now known as dimsum__ | 13:51 | |
| *** openstackgerrit has joined #heat | 13:51 | |
| *** signed8bit has joined #heat | 13:52 | |
| *** jpeeler has joined #heat | 13:53 | |
| *** Raj1 has quit IRC | 13:59 | |
| *** smccully has joined #heat | 13:59 | |
| *** smccully has quit IRC | 14:01 | |
| *** smccully has joined #heat | 14:03 | |
| *** htruta has joined #heat | 14:04 | |
| *** vijayagurug has joined #heat | 14:05 | |
| *** smccully has quit IRC | 14:05 | |
| *** smccully has joined #heat | 14:05 | |
| *** vijayagurug has left #heat | 14:06 | |
| *** smccully has quit IRC | 14:07 | |
| *** smccully has joined #heat | 14:07 | |
| *** Raj1 has joined #heat | 14:07 | |
| *** bnemec has joined #heat | 14:13 | |
| *** radez_g0n3 is now known as radez | 14:14 | |
| *** tspatzier has quit IRC | 14:15 | |
| *** Raj1 has quit IRC | 14:19 | |
| *** gokrokve_ has joined #heat | 14:21 | |
| *** gokrokve has quit IRC | 14:25 | |
| *** shakamunyi has joined #heat | 14:25 | |
| *** hdd has joined #heat | 14:25 | |
| *** gokrokve_ has quit IRC | 14:31 | |
| *** gokrokve has joined #heat | 14:31 | |
| *** bnemec is now known as beekneemech | 14:32 | |
| *** pm90_ has joined #heat | 14:36 | |
| *** sambol has joined #heat | 14:38 | |
| *** Marga_ has joined #heat | 14:38 | |
| *** Marga_ has quit IRC | 14:39 | |
| *** Marga_ has joined #heat | 14:39 | |
| *** vijayagurug has joined #heat | 14:40 | |
| *** vijayagurug has left #heat | 14:41 | |
| *** andersonvom has joined #heat | 14:43 | |
| *** jaime49 has quit IRC | 14:44 | |
| *** packet has joined #heat | 14:45 | |
| *** Tango has joined #heat | 14:48 | |
| *** jasond has joined #heat | 14:52 | |
| *** kebray has joined #heat | 14:54 | |
| *** jasond has quit IRC | 14:58 | |
| *** jasond has joined #heat | 14:58 | |
| *** jasond`` has quit IRC | 15:02 | |
| *** randallburt has joined #heat | 15:03 | |
| *** Raj1 has joined #heat | 15:03 | |
| *** blues-man has joined #heat | 15:06 | |
| *** Raj1 has quit IRC | 15:06 | |
| *** Raj1 has joined #heat | 15:06 | |
| *** jaime49 has joined #heat | 15:08 | |
| *** BManojlovic has quit IRC | 15:09 | |
| *** jaime49 has quit IRC | 15:09 | |
| *** jaime49 has joined #heat | 15:09 | |
| *** andersonvom has quit IRC | 15:11 | |
| *** Marga_ has quit IRC | 15:13 | |
| *** sergmelikyan has joined #heat | 15:13 | |
| *** jasond has quit IRC | 15:14 | |
| *** jasond has joined #heat | 15:18 | |
| openstackgerrit | Merged openstack/heat: Decouple volumes of aws and os https://review.openstack.org/180388 | 15:21 |
| *** shakamunyi has quit IRC | 15:25 | |
| *** Marga_ has joined #heat | 15:28 | |
| *** Drago has joined #heat | 15:30 | |
| *** Drago has quit IRC | 15:30 | |
| *** Drago has joined #heat | 15:30 | |
| *** andersonvom has joined #heat | 15:32 | |
| *** Marga_ has quit IRC | 15:37 | |
| *** andersonvom has quit IRC | 15:38 | |
| *** MasterPiece has quit IRC | 15:38 | |
| *** Qiming_ has quit IRC | 15:39 | |
| *** jaypipes is now known as leakypipes | 15:42 | |
| *** rpothier has joined #heat | 15:44 | |
| openstackgerrit | Merged openstack/heat: fix for retrieving leaf nodes in convergence stack delete https://review.openstack.org/180074 | 15:46 |
| *** EricGonczer_ has joined #heat | 15:46 | |
| *** EricGonczer_ has quit IRC | 15:51 | |
| *** EricGonczer_ has joined #heat | 15:55 | |
| *** openstackstatus has quit IRC | 15:56 | |
| *** openstackstatus has joined #heat | 15:56 | |
| *** ChanServ sets mode: +v openstackstatus | 15:56 | |
| *** hdd has quit IRC | 15:59 | |
| *** randallburt has quit IRC | 16:00 | |
| *** jasond has quit IRC | 16:05 | |
| *** Marga_ has joined #heat | 16:07 | |
| *** jasond has joined #heat | 16:08 | |
| *** shakamunyi has joined #heat | 16:16 | |
| *** Marga_ has quit IRC | 16:21 | |
| *** Marga_ has joined #heat | 16:22 | |
| *** jasond has quit IRC | 16:25 | |
| *** jaime49 has quit IRC | 16:26 | |
| *** Marga_ has quit IRC | 16:30 | |
| *** jasond has joined #heat | 16:30 | |
| *** Raj1 has quit IRC | 16:33 | |
| *** shakamunyi has quit IRC | 16:33 | |
| *** Drago has quit IRC | 16:35 | |
| *** randallburt has joined #heat | 16:38 | |
| *** Marga_ has joined #heat | 16:39 | |
| *** gokrokve has quit IRC | 16:40 | |
| *** radez is now known as radez_g0n3 | 16:41 | |
| *** Raj1 has joined #heat | 16:42 | |
| *** Raj1 has quit IRC | 16:42 | |
| *** Raj1 has joined #heat | 16:42 | |
| *** radez_g0n3 is now known as radez | 16:42 | |
| *** Marga_ has quit IRC | 16:43 | |
| *** harlowja has quit IRC | 16:44 | |
| *** harlowja has joined #heat | 16:44 | |
| *** packet has quit IRC | 16:46 | |
| *** shakamunyi has joined #heat | 16:48 | |
| *** achanda has joined #heat | 16:50 | |
| *** derekh has quit IRC | 16:52 | |
| *** Tango|2 has joined #heat | 16:53 | |
| *** Raj1 has quit IRC | 16:56 | |
| *** Tango has quit IRC | 16:56 | |
| *** Drago has joined #heat | 16:56 | |
| *** Drago has quit IRC | 16:57 | |
| *** Drago has joined #heat | 16:57 | |
| *** crose has quit IRC | 16:57 | |
| *** gokrokve has joined #heat | 16:57 | |
| *** gokrokve has quit IRC | 16:58 | |
| *** gokrokve has joined #heat | 16:58 | |
| *** Marga_ has joined #heat | 17:03 | |
| *** Marga_ has quit IRC | 17:03 | |
| *** Marga_ has joined #heat | 17:03 | |
| *** shakamunyi has quit IRC | 17:07 | |
| *** blues-man has quit IRC | 17:10 | |
| *** shakamunyi has joined #heat | 17:18 | |
| *** shakamunyi has quit IRC | 17:23 | |
| *** Tango has joined #heat | 17:23 | |
| *** Tango|2 has quit IRC | 17:24 | |
| *** Raj1 has joined #heat | 17:26 | |
| *** shakamunyi has joined #heat | 17:35 | |
| *** Raj1 has quit IRC | 17:35 | |
| *** jaime49 has joined #heat | 17:43 | |
| *** jaime49 has quit IRC | 17:43 | |
| *** jaime49 has joined #heat | 17:44 | |
| *** jasond has quit IRC | 17:44 | |
| *** jasond has joined #heat | 17:44 | |
| *** jpeeler has quit IRC | 17:46 | |
| *** daneyon has joined #heat | 17:48 | |
| *** jaime49 has quit IRC | 17:49 | |
| *** gokrokve has quit IRC | 17:58 | |
| *** alexpilotti has quit IRC | 18:01 | |
| *** Raj1 has joined #heat | 18:01 | |
| *** pm90_ has quit IRC | 18:02 | |
| *** alexpilotti has joined #heat | 18:02 | |
| *** allegra-s has joined #heat | 18:02 | |
| *** andersonvom has joined #heat | 18:04 | |
| *** Raj1 has quit IRC | 18:05 | |
| *** alexpilotti has quit IRC | 18:07 | |
| *** andersonvom has quit IRC | 18:11 | |
| *** jpeeler has joined #heat | 18:11 | |
| *** randallburt has quit IRC | 18:13 | |
| *** coolsvap has quit IRC | 18:15 | |
| *** allegra-s has quit IRC | 18:22 | |
| *** rwsu_ has quit IRC | 18:24 | |
| *** Marga_ has quit IRC | 18:30 | |
| *** Marga_ has joined #heat | 18:30 | |
| *** Marga_ has quit IRC | 18:35 | |
| *** Marga_ has joined #heat | 18:35 | |
| *** Marga_ has quit IRC | 18:37 | |
| *** EricGonczer_ has quit IRC | 18:40 | |
| *** EricGonczer_ has joined #heat | 18:41 | |
| *** serg_melikyan has joined #heat | 18:48 | |
| *** allegra-s has joined #heat | 18:50 | |
| *** alanf-mc has joined #heat | 18:51 | |
| *** alanf-mc has quit IRC | 18:51 | |
| *** zzxwill has quit IRC | 18:52 | |
| *** EricGonczer_ has quit IRC | 18:54 | |
| *** serg_melikyan has quit IRC | 19:09 | |
| *** Marga_ has joined #heat | 19:09 | |
| *** vijayagurug has joined #heat | 19:24 | |
| *** serg_melikyan has joined #heat | 19:26 | |
| *** EricGonczer_ has joined #heat | 19:27 | |
| *** serg_melikyan has quit IRC | 19:27 | |
| *** serg_melikyan has joined #heat | 19:27 | |
| *** alexheneveld has quit IRC | 19:29 | |
| *** EricGonc_ has joined #heat | 19:30 | |
| *** EricGonczer_ has quit IRC | 19:31 | |
| *** EricGonc_ has quit IRC | 19:35 | |
| *** EricGonczer_ has joined #heat | 19:37 | |
| *** Raj3 has joined #heat | 19:43 | |
| *** blomquisg has quit IRC | 19:45 | |
| *** randallburt has joined #heat | 19:47 | |
| *** pm90_ has joined #heat | 19:48 | |
| *** jaime49 has joined #heat | 19:50 | |
| *** jaime49 has quit IRC | 19:50 | |
| *** jaime49 has joined #heat | 19:51 | |
| *** pm90_ has quit IRC | 19:54 | |
| *** EricGonczer_ has quit IRC | 19:55 | |
| *** pm90_ has joined #heat | 19:55 | |
| *** EricGonczer_ has joined #heat | 19:56 | |
| *** packet has joined #heat | 20:06 | |
| *** serg_melikyan has quit IRC | 20:07 | |
| *** alanf-mc has joined #heat | 20:10 | |
| *** aweiteka_ has joined #heat | 20:11 | |
| *** aweiteka has quit IRC | 20:13 | |
| *** erkules_ is now known as erkules | 20:15 | |
| *** erkules has quit IRC | 20:15 | |
| *** erkules has joined #heat | 20:15 | |
| *** shakamunyi has quit IRC | 20:17 | |
| *** signed8b_ has joined #heat | 20:19 | |
| *** signed8bit has quit IRC | 20:20 | |
| *** sdake_ has joined #heat | 20:22 | |
| *** radez is now known as radez_g0n3 | 20:24 | |
| *** sdake has quit IRC | 20:26 | |
| *** randallburt has quit IRC | 20:37 | |
| *** Raj3 has quit IRC | 20:39 | |
| *** randallburt has joined #heat | 20:46 | |
| larsks | Is it normal for a stack-update process to also log DELETE messages? That happened to me, but as far as I can tell the stack-update worked correctly and nothing was deleted... | 20:46 |
| *** shakamunyi has joined #heat | 20:48 | |
| *** htruta has quit IRC | 20:51 | |
| zaneb | larsks: are you sure none of your resources were actually replaced during the update? | 20:55 |
| larsks | I didn't think so (at least, not the os::nova::servers that I was logged into at the time). I will have to run through it again and pay closer attention...logs are too cluttered right now because of an *actual* delete operation. | 20:57 |
| *** vijayagurug has quit IRC | 20:58 | |
| *** signed8bit has joined #heat | 21:02 | |
| *** shakamunyi has quit IRC | 21:03 | |
| *** jasond` has joined #heat | 21:04 | |
| *** signed8b_ has quit IRC | 21:06 | |
| *** aweiteka_ has quit IRC | 21:14 | |
| openstackgerrit | Ryan Brown proposed openstack/heat: Add a periodic task to clean up engine threads https://review.openstack.org/181559 | 21:14 |
| *** blomquisg has joined #heat | 21:16 | |
| *** sambol has quit IRC | 21:18 | |
| *** pm90_ has quit IRC | 21:21 | |
| *** shakamunyi has joined #heat | 21:22 | |
| *** sdake_ has quit IRC | 21:26 | |
| *** serg_melikyan has joined #heat | 21:27 | |
| *** signed8bit has quit IRC | 21:29 | |
| *** rwsu has joined #heat | 21:30 | |
| *** rwsu has quit IRC | 21:30 | |
| *** rwsu has joined #heat | 21:31 | |
| *** randallburt has quit IRC | 21:34 | |
| *** rwsu has quit IRC | 21:39 | |
| *** Drago has quit IRC | 21:40 | |
| *** EricGonczer_ has quit IRC | 21:43 | |
| *** andersonvom has joined #heat | 21:43 | |
| *** randallburt has joined #heat | 21:47 | |
| *** alexheneveld has joined #heat | 21:50 | |
| *** jasond` has quit IRC | 21:51 | |
| *** jaime49 has left #heat | 21:52 | |
| *** kebray_ has joined #heat | 21:53 | |
| *** kebray has quit IRC | 21:55 | |
| *** rwsu has joined #heat | 21:58 | |
| *** jasond` has joined #heat | 22:03 | |
| *** jpeeler has quit IRC | 22:03 | |
| *** pm90_ has joined #heat | 22:04 | |
| *** shakamunyi has quit IRC | 22:04 | |
| *** barra204 has joined #heat | 22:05 | |
| *** EmilienM is now known as EmilienM|afk | 22:08 | |
| *** jasond` has quit IRC | 22:09 | |
| *** alexpilotti has joined #heat | 22:14 | |
| *** beekneemech has quit IRC | 22:18 | |
| *** dimsum__ has quit IRC | 22:21 | |
| *** cdent has quit IRC | 22:23 | |
| *** jasond` has joined #heat | 22:25 | |
| *** jasond` has quit IRC | 22:26 | |
| *** serg_melikyan has quit IRC | 22:28 | |
| *** kebray_ has quit IRC | 22:36 | |
| *** allegra-s has quit IRC | 22:43 | |
| *** kebray has joined #heat | 22:48 | |
| *** pm90_ has quit IRC | 22:49 | |
| *** alexheneveld has quit IRC | 22:55 | |
| *** rwsu has quit IRC | 22:56 | |
| *** serg_melikyan has joined #heat | 23:00 | |
| *** serg_melikyan has quit IRC | 23:05 | |
| *** rpothier has quit IRC | 23:06 | |
| *** randallburt has quit IRC | 23:08 | |
| *** packet has quit IRC | 23:15 | |
| *** daneyon_ has joined #heat | 23:16 | |
| *** daneyon has quit IRC | 23:19 | |
| *** pm90_ has joined #heat | 23:19 | |
| *** rwsu has joined #heat | 23:22 | |
| *** barra204 has quit IRC | 23:22 | |
| *** alexheneveld has joined #heat | 23:23 | |
| *** pm90_ has quit IRC | 23:43 | |
| *** pm90_ has joined #heat | 23:44 | |
| *** alexpilotti_ has joined #heat | 23:45 | |
| *** alexpilotti has quit IRC | 23:47 | |
| *** alexpilotti_ is now known as alexpilotti | 23:47 | |
| *** pm90_ has quit IRC | 23:48 | |
| *** andersonvom has quit IRC | 23:54 | |
| *** rwsu has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!