| *** sdake has quit IRC | 00:06 | |
| *** tochi has quit IRC | 00:10 | |
| *** pratikmallya has joined #heat | 00:18 | |
| *** hdd has quit IRC | 00:19 | |
| *** tochi has joined #heat | 00:23 | |
| *** harlowja_ has joined #heat | 00:27 | |
| *** harlowja has quit IRC | 00:30 | |
| *** sdake has joined #heat | 00:32 | |
| openstackgerrit | Di XiaoLi proposed openstack/heat: Add OS::Neutron::RBACPolicy resource https://review.openstack.org/255720 | 00:38 |
|---|---|---|
| *** jaime49 has quit IRC | 00:43 | |
| *** sorantis has quit IRC | 00:48 | |
| *** jaime49 has joined #heat | 00:48 | |
| *** hdd has joined #heat | 00:53 | |
| *** daneyon has joined #heat | 01:01 | |
| *** pratikmallya has quit IRC | 01:03 | |
| *** jaime49 has quit IRC | 01:07 | |
| *** zaneb has quit IRC | 01:14 | |
| *** clayton has quit IRC | 01:17 | |
| *** clayton has joined #heat | 01:18 | |
| *** manishg has joined #heat | 01:23 | |
| *** manishg has quit IRC | 01:24 | |
| *** manishg has joined #heat | 01:24 | |
| *** zaneb has joined #heat | 01:27 | |
| *** clayton has quit IRC | 01:30 | |
| *** Qiming has quit IRC | 01:32 | |
| *** hdd has quit IRC | 01:34 | |
| *** sridhar_ram has quit IRC | 01:36 | |
| *** sdake has quit IRC | 01:40 | |
| *** clayton has joined #heat | 01:51 | |
| *** Yanyanhu has joined #heat | 01:52 | |
| *** sridhar_ram has joined #heat | 02:02 | |
| *** elynn has joined #heat | 02:15 | |
| *** sridhar_ram has quit IRC | 02:17 | |
| *** bana_k has quit IRC | 02:21 | |
| *** Qiming has joined #heat | 02:24 | |
| *** sdake has joined #heat | 02:27 | |
| openstackgerrit | Daniel Gonzalez Nothnagel proposed openstack/heat: Add scenario test for multi-region https://review.openstack.org/248011 | 02:46 |
| *** daneyon has quit IRC | 02:51 | |
| *** dims has quit IRC | 02:59 | |
| openstackgerrit | Merged openstack/heat: Remove stray debug from context.py https://review.openstack.org/263814 | 03:09 |
| *** daneyon has joined #heat | 03:19 | |
| openstackgerrit | Merged openstack/python-heatclient: Deprecated tox -downloadcache option removed https://review.openstack.org/256815 | 03:34 |
| *** ricolin has joined #heat | 03:37 | |
| openstackgerrit | Yatin Kumbhare proposed openstack/heat: Fix for the deprecated library function https://review.openstack.org/263659 | 03:40 |
| *** sabeen1 has joined #heat | 03:41 | |
| *** sabeen2 has joined #heat | 03:44 | |
| *** sabeen1 has quit IRC | 03:46 | |
| *** tochi has quit IRC | 03:49 | |
| *** daneyon has quit IRC | 03:49 | |
| *** links has joined #heat | 03:51 | |
| *** manishg has quit IRC | 03:51 | |
| *** manishg has joined #heat | 03:52 | |
| *** cwolferh has quit IRC | 03:58 | |
| *** dyasny has quit IRC | 04:00 | |
| *** ricolin has quit IRC | 04:05 | |
| *** coolsvap|away is now known as coolsvap | 04:06 | |
| *** tochi has joined #heat | 04:10 | |
| *** PsionTheory has quit IRC | 04:13 | |
| *** kebray has joined #heat | 04:13 | |
| *** kebray has quit IRC | 04:15 | |
| *** kebray has joined #heat | 04:15 | |
| *** manishg has quit IRC | 04:23 | |
| *** cwolferh has joined #heat | 04:29 | |
| *** coolsvap is now known as coolsvap|away | 04:29 | |
| openstackgerrit | Merged openstack/python-heatclient: Replace assertEqual(None, *) with assertIsNone in tests https://review.openstack.org/259020 | 04:32 |
| *** sdake has quit IRC | 04:38 | |
| *** elynn has quit IRC | 04:42 | |
| *** sdake has joined #heat | 04:44 | |
| *** elynn has joined #heat | 04:58 | |
| *** ananta has joined #heat | 05:04 | |
| *** ricolin has joined #heat | 05:12 | |
| *** kebray has quit IRC | 05:13 | |
| *** kebray has joined #heat | 05:18 | |
| *** manishg has joined #heat | 05:21 | |
| *** gberginc has quit IRC | 05:27 | |
| *** blomquisg has quit IRC | 05:31 | |
| *** coolsvap|away is now known as coolsvap | 05:35 | |
| *** jrist has quit IRC | 05:39 | |
| openstackgerrit | Anant Patil proposed openstack/heat: Convergence: Set traversal to None on failure https://review.openstack.org/261208 | 05:41 |
| *** jrist has joined #heat | 05:51 | |
| *** coolsvap is now known as coolsvap|away | 05:55 | |
| openstackgerrit | Crag Wolfe proposed openstack/heat: Do not try to save event resource if too big for db column https://review.openstack.org/259274 | 06:00 |
| *** cwolferh has quit IRC | 06:05 | |
| *** bana_k has joined #heat | 06:10 | |
| *** dixiaoli has joined #heat | 06:10 | |
| *** coolsvap|away is now known as coolsvap | 06:18 | |
| *** ricolin has quit IRC | 06:31 | |
| *** Qiming has quit IRC | 06:34 | |
| *** skraynev_ has joined #heat | 06:41 | |
| *** chlong has quit IRC | 06:50 | |
| skraynev_ | morning all | 06:54 |
| *** sabeen2 has quit IRC | 06:54 | |
| *** KanagarajM has joined #heat | 06:56 | |
| *** Qiming has joined #heat | 06:58 | |
| *** kebray has quit IRC | 06:59 | |
| skraynev_ | meeting time | 07:00 |
| ananta | #join openstack-meeting | 07:00 |
| *** prazumovsky has joined #heat | 07:00 | |
| *** rakesh_hs has joined #heat | 07:04 | |
| *** gberginc has joined #heat | 07:05 | |
| *** ricolin has joined #heat | 07:15 | |
| *** ricolin has quit IRC | 07:16 | |
| *** ricolin has joined #heat | 07:19 | |
| *** e0ne has joined #heat | 07:25 | |
| *** ricolin has quit IRC | 07:28 | |
| *** ricolin has joined #heat | 07:29 | |
| *** bana_k has quit IRC | 07:34 | |
| *** ricolin has quit IRC | 07:37 | |
| *** rwsu has quit IRC | 07:47 | |
| *** prazumovsky has quit IRC | 07:48 | |
| *** jprovazn has joined #heat | 07:50 | |
| *** coolsvap is now known as coolsvap|away | 08:00 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Improve gateway_ip updation for subnet https://review.openstack.org/250136 | 08:02 |
| *** prazumovsky has joined #heat | 08:03 | |
| *** Yanyanhu has quit IRC | 08:05 | |
| *** Yanyanhu has joined #heat | 08:06 | |
| *** coolsvap|away is now known as coolsvap | 08:06 | |
| *** chlong has joined #heat | 08:11 | |
| *** prazumovsky has quit IRC | 08:15 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/heat: TEST: aodh+keystoneauth1 https://review.openstack.org/250530 | 08:16 |
| *** jcoufal has joined #heat | 08:31 | |
| *** shardy has joined #heat | 08:32 | |
| *** ricolin has joined #heat | 08:33 | |
| *** rwsu has joined #heat | 08:40 | |
| *** jprovazn has quit IRC | 08:43 | |
| openstackgerrit | Reedip proposed openstack/heat-cfntools: py26/py33 is no longer supported by Infra's CI https://review.openstack.org/264076 | 08:47 |
| *** sdake has quit IRC | 09:02 | |
| *** jprovazn has joined #heat | 09:07 | |
| openstackgerrit | Thomas Herve proposed openstack/heat: Fix race condition in tests with swift key https://review.openstack.org/263749 | 09:09 |
| *** derekh has joined #heat | 09:16 | |
| *** jistr has joined #heat | 09:17 | |
| *** ccard has joined #heat | 09:18 | |
| *** yassine has joined #heat | 09:19 | |
| *** skraynev_ has quit IRC | 09:22 | |
| *** zeih has joined #heat | 10:00 | |
| *** tlashchova has joined #heat | 10:01 | |
| openstackgerrit | Rakesh H S proposed openstack/heat: Fix validation of server personality property https://review.openstack.org/260362 | 10:15 |
| openstackgerrit | Thomas Herve proposed openstack/heat: Fix race condition in tests with swift key https://review.openstack.org/263749 | 10:19 |
| *** tochi has quit IRC | 10:26 | |
| *** dixiaoli has quit IRC | 10:27 | |
| *** Yanyanhu has quit IRC | 10:29 | |
| *** asalkeld has quit IRC | 10:31 | |
| openstackgerrit | Anant Patil proposed openstack/heat: Convergence: Pick resource from dead engine worker https://review.openstack.org/264115 | 10:40 |
| *** elynn has quit IRC | 10:41 | |
| *** dims has joined #heat | 10:47 | |
| *** yassine has quit IRC | 10:55 | |
| tlashchova | hi all | 10:58 |
| tlashchova | I have a question about heat-templates | 10:58 |
| tlashchova | we have two wordpress templates here https://github.com/openstack/heat-templates/tree/master/hot/F20 | 10:59 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Add resolve_outputs parameter to stacks API https://review.openstack.org/263270 | 11:00 |
| tlashchova | description says that we can use them without heat-cfntools in the image | 11:00 |
| tlashchova | but property user_data_format is missing and it's HEAT_CFNTOOLS by default | 11:01 |
| tlashchova | looks like it must be RAW | 11:01 |
| *** skraynev_ has joined #heat | 11:05 | |
| tlashchova | do we need to fix it? | 11:05 |
| *** zeih has quit IRC | 11:10 | |
| therve | tlashchova, Good question | 11:15 |
| therve | Have you tried it? I wonder if cloudinit doesn't handle it either way | 11:15 |
| therve | Otherwise yeah it should be changed to RAW | 11:15 |
| tlashchova | therve, yes, I have tried. The script is not executed and cloud-init-output.log contains errors | 11:23 |
| tlashchova | with RAW works fine | 11:23 |
| openstackgerrit | Tetiana Lashchova proposed openstack/heat-templates: Change user_data_format to RAW in WordPress templates https://review.openstack.org/264138 | 11:35 |
| *** KanagarajM has quit IRC | 11:44 | |
| shardy | tlashchova: changing it to RAW is fine, but IIRC that did work without it at one point | 11:45 |
| shardy | we're supposed to append the userdata as a cfn-userdata mime part, which should, I think, get run via the part-handler we inject | 11:46 |
| shardy | https://github.com/openstack/heat/blob/master/heat/engine/clients/os/nova.py#L330 | 11:46 |
| shardy | https://github.com/openstack/heat/blob/master/heat/cloudinit/loguserdata.py#L103 | 11:47 |
| *** coolsvap is now known as coolsvap|away | 11:47 | |
| shardy | tlashchova: IIRC the "RAW" format didn't exist when we wrote that example template, but switching to it seems OK | 11:48 |
| shardy | it'd just be good to confirm it's not a regression that it's needed | 11:48 |
| *** zeih has joined #heat | 11:51 | |
| *** ananta has quit IRC | 11:57 | |
| therve | ramishra_, So https://review.openstack.org/#/c/263749/ is passing, hooray | 11:58 |
| therve | Do you think I should file something for the potential issue in the plugin? | 11:59 |
| *** yassine has joined #heat | 11:59 | |
| ramishra_ | therve: should we recheck it a few times to see if we don't see the earlier issue? | 11:59 |
| therve | ramishra_, Meh. Considering it's a once a week occurence, I don't think so. | 12:00 |
| therve | Also the logic sounds fine this time :) | 12:00 |
| ramishra_ | yeah, should not be an issue in merging it. We had several fixes already merged for the same issue;) | 12:02 |
| *** skraynev has quit IRC | 12:04 | |
| *** zeih has quit IRC | 12:21 | |
| *** tlashchova has quit IRC | 12:29 | |
| *** tlashchova has joined #heat | 12:32 | |
| openstackgerrit | Sirushti Murugesan proposed openstack/heat: DNM: Convergence: Opportunistic Upgrade Tests https://review.openstack.org/230292 | 12:51 |
| openstackgerrit | Sirushti Murugesan proposed openstack/heat: DNM: Upgrade tests: Restart Heat Engine with Covergence turned on https://review.openstack.org/235242 | 12:51 |
| *** gberginc has quit IRC | 12:56 | |
| *** rakesh_hs has quit IRC | 13:04 | |
| *** dprince has joined #heat | 13:09 | |
| *** dyasny has joined #heat | 13:31 | |
| *** links has quit IRC | 13:32 | |
| *** ricolin has quit IRC | 13:54 | |
| *** zeih has joined #heat | 13:58 | |
| *** dyasny has quit IRC | 13:59 | |
| *** jonesbr has joined #heat | 14:07 | |
| *** dyasny has joined #heat | 14:10 | |
| *** rpothier has joined #heat | 14:11 | |
| *** dslevin_ has joined #heat | 14:12 | |
| *** skraynev has joined #heat | 14:18 | |
| *** blomquisg has joined #heat | 14:21 | |
| *** links has joined #heat | 14:23 | |
| *** neelashah has joined #heat | 14:25 | |
| *** links has quit IRC | 14:28 | |
| *** skraynev_ has quit IRC | 14:30 | |
| *** jaime49 has joined #heat | 14:34 | |
| *** shadower has quit IRC | 14:36 | |
| *** dyasny_ has joined #heat | 14:39 | |
| *** shadower has joined #heat | 14:40 | |
| *** dyasny has quit IRC | 14:40 | |
| *** pradk has joined #heat | 14:42 | |
| *** jprovazn has quit IRC | 14:47 | |
| openstackgerrit | Merged openstack/heat: Add basic get_live_state implementation https://review.openstack.org/244065 | 14:47 |
| *** inc0 has joined #heat | 14:49 | |
| *** zeih has quit IRC | 14:56 | |
| *** skraynev_ has joined #heat | 14:57 | |
| *** zeih has joined #heat | 15:02 | |
| *** spotz_zzz is now known as spotz | 15:05 | |
| *** dprince has quit IRC | 15:16 | |
| *** dprince has joined #heat | 15:18 | |
| *** elynn has joined #heat | 15:18 | |
| tlashchova | shardy: I have tested wordpess template on master once again without user_data_format property and with RAW. In the first case it fails with the following log: http://paste.openstack.org/show/483142/ | 15:18 |
| *** dprince has quit IRC | 15:20 | |
| *** dprince has joined #heat | 15:20 | |
| shardy | tlashchova: thanks, what sort of image are you using to test? | 15:22 |
| tlashchova | fedora-23 | 15:22 |
| shardy | the python: No such file or directory looks odd | 15:22 |
| *** jaime49 has quit IRC | 15:25 | |
| shardy | tlashchova: does /usr/bin/env python work on the VM when it's running? | 15:25 |
| * shardy wonders about python3 by default in f23 images | 15:26 | |
| *** srwilkers_ has joined #heat | 15:28 | |
| *** blomquisg has quit IRC | 15:30 | |
| openstackgerrit | LiuNanke proposed openstack/heat-cfntools: Replace assertEqual(None, *) with assertIsNone in tests https://review.openstack.org/264229 | 15:31 |
| *** inc0 has quit IRC | 15:31 | |
| *** dprince has quit IRC | 15:32 | |
| *** dprince has joined #heat | 15:32 | |
| *** arlion has left #heat | 15:33 | |
| *** inc0 has joined #heat | 15:34 | |
| *** jprovazn has joined #heat | 15:35 | |
| tlashchova | shardy: no, it doesn't work | 15:35 |
| shardy | tlashchova: Ok, thanks, that must mean /var/lib/cloud/instance/scripts/loguserdata.py doesn't run at all, which probably explains why you're having to add the RAW | 15:37 |
| openstackgerrit | Oleksii Chuprykov proposed openstack/heat: Add OS::Barbican::*Container https://review.openstack.org/223136 | 15:38 |
| *** vijendar has joined #heat | 15:38 | |
| shardy | tlashchova: could you please raise a bug with the details we discussed, including the error you pasted and the exact version of F23 image? | 15:38 |
| shardy | I assume loguserdata not running at all is something we'll want to fix | 15:38 |
| tlashchova | Ok | 15:40 |
| *** zeih has quit IRC | 15:41 | |
| therve | ryansb, Hey, here's what I've been managing to do with the zaqar work in: https://gist.github.com/therve/9c7c762d3eb81b076be4 | 15:43 |
| therve | A bit rough but I think we're getting somewhere | 15:43 |
| *** jprovazn has quit IRC | 15:43 | |
| therve | shardy, Maybe you'd be interested by this ^^^. At least I am by your feedback :) | 15:44 |
| *** jprovazn has joined #heat | 15:44 | |
| *** blomquisg has joined #heat | 15:45 | |
| shardy | therve: nice! So it just blocks on the ws.recv() until a message gets posted? | 15:48 |
| *** mragupat has joined #heat | 15:48 | |
| therve | shardy, Yep! | 15:50 |
| therve | And in this case a message is a heat event | 15:50 |
| *** jaime49 has joined #heat | 15:50 | |
| therve | No polling, at least on the client side :) | 15:51 |
| *** elynn has quit IRC | 15:51 | |
| *** jaime49 has quit IRC | 15:51 | |
| shardy | Nice, I wonder if we can refactor things in os-collect-config, so we can poll for metadata without sleeping | 15:51 |
| *** jaime49 has joined #heat | 15:51 | |
| shardy | is there a way to filter the queues on the zaqar side per-resource? | 15:51 |
| *** jaime49 has joined #heat | 15:52 | |
| shardy | e.g so we could wait for an event for a specific resource, which then reloads the metadata? | 15:52 |
| *** hdd has joined #heat | 15:52 | |
| therve | We already have support for zaqar in software config, right? | 15:52 |
| shardy | therve: Yeah, but it's via a sleep/poll still isn't it? | 15:52 |
| therve | So it would "just" be a matter of changing os-collect-config transport | 15:53 |
| therve | Yeah it uses the regular HTTP API, not websocket | 15:53 |
| shardy | Cool, I'm looking for a way to reduce the latency where we're sleeping but something actually happened | 15:53 |
| therve | That'd be worth a try. I'd be happy to help :) | 15:54 |
| *** hdd has quit IRC | 15:56 | |
| shardy | therve: Ah, we already create a queue per physical_resource_name, so you're right it's just a change to the existing collector, along with perhaps setting the occ poll interval to zero | 15:57 |
| therve | Yeah occ model is to call the collector in a loop, so we'd have to tweak that somehow | 15:57 |
| shardy | therve: I'm not sure how much time it'll gain us, but it'd be great if you have bandwidth to look into it | 15:57 |
| shardy | I know stevebaker has been moving things to swift, but that still has the polling problem | 15:57 |
| *** mragupat has quit IRC | 15:58 | |
| ryansb | therve: will check it out | 15:58 |
| *** mragupat has joined #heat | 15:58 | |
| *** neelashah has quit IRC | 15:59 | |
| therve | shardy, I'll certainly have a look | 15:59 |
| *** Qiming has quit IRC | 15:59 | |
| therve | It may require a non-trivial refactoring of occ | 15:59 |
| ryansb | shardy: zaqar doesn't do filtering within a queue | 15:59 |
| *** neelashah has joined #heat | 16:00 | |
| shardy | ryansb: OK, I realized we create a queue per server resource anyway, so I don't think we need to | 16:00 |
| ryansb | so you'd have to use s specific queue per thing you want distinct notifications for (in this case, physical resource name) | 16:00 |
| shardy | I was thinking we'd need to filter the events, but looks like I was mistaken | 16:00 |
| ryansb | but if you want to wait for a specific event, you'll have to filter client side | 16:00 |
| ryansb | err, specific *type* of event on a resource, I mean | 16:00 |
| therve | Yeah we have a different queue for config metadata | 16:01 |
| shardy | https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/nova/server.py#L587 | 16:01 |
| *** sabeen1 has joined #heat | 16:02 | |
| openstackgerrit | Neetu Jain proposed openstack/python-heatclient: Fix error message on authentication failure https://review.openstack.org/263871 | 16:03 |
| *** sabeen2 has joined #heat | 16:04 | |
| therve | On a side note, using keystone client is often (always?) a painful experience | 16:04 |
| ryansb | pretty sure always | 16:04 |
| therve | I don't know if it's a documentation issue but my brain hurts every time | 16:04 |
| shardy | Yeah, in theory the new auth plugin stuff is easier, but in practice it's just way more stuff to understand | 16:05 |
| *** dyasny__ has joined #heat | 16:06 | |
| *** sabeen1 has quit IRC | 16:06 | |
| *** dyasny_ has quit IRC | 16:07 | |
| *** ddaskal has joined #heat | 16:07 | |
| *** randallburt has joined #heat | 16:08 | |
| openstackgerrit | Neetu Jain proposed openstack/python-heatclient: Specific error for heat template passed as env template https://review.openstack.org/263889 | 16:08 |
| *** vgridnev has joined #heat | 16:13 | |
| *** ddaskal_ has joined #heat | 16:14 | |
| *** sdake has joined #heat | 16:14 | |
| *** ddaskal has quit IRC | 16:17 | |
| *** kebray has joined #heat | 16:17 | |
| *** dyasny__ has quit IRC | 16:19 | |
| *** dyasny__ has joined #heat | 16:20 | |
| *** jasond has joined #heat | 16:25 | |
| *** Drago has joined #heat | 16:32 | |
| *** dyasny_ has joined #heat | 16:33 | |
| *** dyasny__ has quit IRC | 16:34 | |
| *** dslevin_ has quit IRC | 16:34 | |
| *** kebray has quit IRC | 16:38 | |
| *** vishwanathj has quit IRC | 16:42 | |
| openstackgerrit | Jay Dobies proposed openstack/python-heatclient: Added -P support to template-validate https://review.openstack.org/264261 | 16:42 |
| *** dslevin_ has joined #heat | 16:44 | |
| *** dyasny_ has quit IRC | 16:45 | |
| *** dslevin_ has quit IRC | 16:49 | |
| *** blomquisg has quit IRC | 16:56 | |
| *** cwolferh has joined #heat | 17:00 | |
| *** arlion has joined #heat | 17:00 | |
| *** arlion has joined #heat | 17:00 | |
| *** kebray has joined #heat | 17:02 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: Tolerate inconsistent backup stack on delete https://review.openstack.org/264275 | 17:06 |
| *** yassine has quit IRC | 17:06 | |
| *** dyasny has joined #heat | 17:09 | |
| *** blomquisg has joined #heat | 17:10 | |
| *** neelashah has quit IRC | 17:16 | |
| *** kebray has quit IRC | 17:20 | |
| *** jaime49 has quit IRC | 17:24 | |
| *** kebray has joined #heat | 17:26 | |
| *** e0ne has quit IRC | 17:32 | |
| *** sridhar_ram has joined #heat | 17:32 | |
| *** neelashah has joined #heat | 17:35 | |
| *** jistr has quit IRC | 17:45 | |
| *** spotz is now known as spotz_zzz | 17:49 | |
| cwolferh | looking for +1's on https://review.openstack.org/#/c/254351/ :-) | 17:54 |
| ryansb | cwolferh: neato :) | 17:55 |
| cwolferh | ryansb, thanks | 17:55 |
| *** spotz_zzz is now known as spotz | 17:58 | |
| *** derekh has quit IRC | 17:59 | |
| openstackgerrit | Sabeen proposed openstack/heat: Add tests for heat template apis https://review.openstack.org/224763 | 18:01 |
| *** neelashah has quit IRC | 18:08 | |
| *** daneyon has joined #heat | 18:14 | |
| *** daneyon_ has joined #heat | 18:15 | |
| *** ddaskal_ has quit IRC | 18:15 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: Tolerate inconsistent backup stack on delete https://review.openstack.org/264275 | 18:15 |
| *** ddaskal has joined #heat | 18:16 | |
| *** daneyon has quit IRC | 18:19 | |
| *** ddaskal has quit IRC | 18:20 | |
| *** PsionTheory has joined #heat | 18:24 | |
| therve | randallburt, The main issue with purge is not a race, it's bug 1524387 | 18:24 |
| openstack | bug 1524387 in heat "heat-manage purge_deleted broken" [High,In progress] https://launchpad.net/bugs/1524387 - Assigned to Thomas Herve (therve) | 18:24 |
| randallburt | therve: oh, ok. | 18:25 |
| randallburt | therve: which still begs the question of why not just fix the query instead of how the query is called. | 18:25 |
| therve | randallburt, Also by keeping your -1 you encourage the "not moving forward" part :) | 18:25 |
| randallburt | therve: no I'm not. −2 stops the train. not -1 | 18:26 |
| therve | Doesn't stop but slow | 18:26 |
| therve | "just" fixing the query is not as easy | 18:26 |
| *** rdo has quit IRC | 18:26 | |
| *** hdd has joined #heat | 18:26 | |
| therve | Also, the reason I didn't "just" fix the query is that the tests I added keep failing | 18:26 |
| *** pratikmallya has joined #heat | 18:26 | |
| *** rdo has joined #heat | 18:28 | |
| therve | randallburt, Also, you don't use stack tags? | 18:29 |
| therve | Because that's another issue fixed | 18:29 |
| randallburt | therve: sorry, in a conference, 1 sec | 18:32 |
| randallburt | therve: understood. yes, we do, I just haven't heard our ops folks running into these issues. Maybe they have and haven't told me. The concern on our end is that purges can be quite large. | 18:34 |
| randallburt | therve: even if we run more frequently we still keep deleted stacks around for a longer time than most because of policy/support reasons. | 18:35 |
| randallburt | therve: also larger customers using on-demand automation can burst hundreds of short-lived stacks in a day on occasion | 18:37 |
| *** sabeen2 has quit IRC | 18:38 | |
| therve | randallburt, The fact that you keep them a long time shouldn't matter to performance much | 18:38 |
| therve | randallburt, I understand your concerns, the counter argument is that it doesn't work at all in other environments | 18:38 |
| randallburt | therve: it doesn't when you batch delete. It does when you iterate | 18:39 |
| therve | Also I'm somewhat paying the price of actually adding a real tests | 18:39 |
| therve | randallburt, The master version doesn't batch delete AFAIU | 18:39 |
| *** manishg has quit IRC | 18:39 | |
| randallburt | therve: no good deed goes unpunished as they say | 18:39 |
| therve | It doesn't compute the list, but still uses the whole dataset | 18:40 |
| randallburt | therve: well, not batch per se but "delete where" in a single transaction | 18:40 |
| *** ahelkhou has quit IRC | 18:40 | |
| *** manishg has joined #heat | 18:40 | |
| therve | A single transaction per delete | 18:41 |
| *** sabeen1 has joined #heat | 18:41 | |
| therve | If it uses a single transaction for all deletes it may work, but it's not the case | 18:41 |
| therve | Also it would probably create contention | 18:41 |
| randallburt | therve: right, based our tests when I originally patched the purge, computing the where clause rather than iterating through the list was orders of magnitude faster | 18:42 |
| randallburt | therve: I mean deleting 100 stacks in a single delete/where clause vs issuing 100 individual delete stack queries | 18:43 |
| therve | randallburt, My patch doesn't do the latter though | 18:43 |
| randallburt | therve: no? | 18:44 |
| randallburt | therve: looking again. | 18:44 |
| *** sabeen3 has joined #heat | 18:44 | |
| *** e0ne has joined #heat | 18:44 | |
| therve | Ah yeah, I see the previous version. No, what I did is not so bad. | 18:45 |
| *** sabeen1 has quit IRC | 18:46 | |
| openstackgerrit | Sabeen proposed openstack/heat: Add tests for heat template apis https://review.openstack.org/224763 | 18:46 |
| randallburt | therve: oh crap. I've misunderstood the patch. | 18:46 |
| randallburt | therve: sorry about that. | 18:46 |
| randallburt | therve: ok, I'll remove −1 now. | 18:47 |
| therve | No problem. Glad I insisted a bit, thanks :) | 18:47 |
| randallburt | therve: me too :D, if I get a chance this afternoon, I'll test this proper against a larger dataset and +2 | 18:49 |
| *** gberginc has joined #heat | 18:51 | |
| *** jonesbr has quit IRC | 18:54 | |
| *** jprovazn has quit IRC | 18:55 | |
| *** jcoufal has quit IRC | 18:58 | |
| *** Drago has quit IRC | 18:59 | |
| *** jonesbr has joined #heat | 19:00 | |
| *** sthillma has joined #heat | 19:00 | |
| *** neelashah has joined #heat | 19:08 | |
| *** blomquisg has quit IRC | 19:08 | |
| *** tlashchova has quit IRC | 19:10 | |
| *** neelashah1 has joined #heat | 19:13 | |
| *** neelashah has quit IRC | 19:14 | |
| *** ddaskal has joined #heat | 19:17 | |
| *** harlowja_ has quit IRC | 19:18 | |
| *** harlowja has joined #heat | 19:19 | |
| *** ddaskal has quit IRC | 19:23 | |
| *** blomquisg has joined #heat | 19:23 | |
| *** jaime49 has joined #heat | 19:26 | |
| *** ahelkhou has joined #heat | 19:27 | |
| *** Drago has joined #heat | 19:32 | |
| *** bana_k has joined #heat | 19:38 | |
| *** sthillma has quit IRC | 19:42 | |
| *** kebray has quit IRC | 19:43 | |
| *** ddaskal has joined #heat | 19:47 | |
| *** e0ne has quit IRC | 19:47 | |
| *** vgridnev has quit IRC | 19:48 | |
| *** crose has joined #heat | 19:51 | |
| *** e0ne has joined #heat | 20:28 | |
| *** e0ne has quit IRC | 20:29 | |
| *** sthillma has joined #heat | 20:30 | |
| *** e0ne has joined #heat | 20:30 | |
| *** e0ne has quit IRC | 20:33 | |
| *** vgridnev has joined #heat | 20:33 | |
| *** sorantis has joined #heat | 20:34 | |
| *** srwilkers_ has quit IRC | 20:51 | |
| *** vishwanathj has joined #heat | 20:56 | |
| *** FL1SK has quit IRC | 21:00 | |
| openstackgerrit | Drago proposed openstack/heat: Remove unconditional return of user_id https://review.openstack.org/264351 | 21:06 |
| *** mragupat has quit IRC | 21:16 | |
| *** mragupat has joined #heat | 21:16 | |
| *** jasond has quit IRC | 21:27 | |
| *** jasond has joined #heat | 21:28 | |
| *** ddaskal has quit IRC | 21:29 | |
| *** ddaskal has joined #heat | 21:33 | |
| *** kebray has joined #heat | 21:38 | |
| *** dprince has quit IRC | 21:39 | |
| *** blomquisg has quit IRC | 21:41 | |
| *** skraynev_ has quit IRC | 21:52 | |
| *** kebray has quit IRC | 21:54 | |
| *** blomquisg has joined #heat | 21:54 | |
| *** kebray has joined #heat | 21:55 | |
| *** vgridnev has quit IRC | 21:55 | |
| *** vgridnev has joined #heat | 21:56 | |
| *** vgridnev has quit IRC | 21:56 | |
| *** jaime49 has quit IRC | 21:59 | |
| *** vishwana_ has joined #heat | 22:03 | |
| *** randallburt1 has joined #heat | 22:03 | |
| *** randallburt has quit IRC | 22:04 | |
| *** vishwanathj has quit IRC | 22:05 | |
| *** dyasny has quit IRC | 22:05 | |
| *** markvan has quit IRC | 22:05 | |
| *** dsneddon has quit IRC | 22:05 | |
| *** markvan has joined #heat | 22:05 | |
| *** dyasny has joined #heat | 22:07 | |
| *** ddaskal_ has joined #heat | 22:11 | |
| *** ddaskal has quit IRC | 22:13 | |
| *** crose has quit IRC | 22:15 | |
| *** dsneddon has joined #heat | 22:19 | |
| *** dims has quit IRC | 22:22 | |
| openstackgerrit | Steven Hardy proposed openstack/heat: Populate context roles when using stored context https://review.openstack.org/263935 | 22:26 |
| *** shardy has quit IRC | 22:26 | |
| *** inc0 has quit IRC | 22:27 | |
| *** blomquisg has quit IRC | 22:28 | |
| *** neelashah1 has quit IRC | 22:29 | |
| *** jonesbr has left #heat | 22:29 | |
| *** FL1SK has joined #heat | 22:30 | |
| *** pratikmallya has quit IRC | 22:30 | |
| *** jtomasek has quit IRC | 22:30 | |
| *** ddaskal has joined #heat | 22:35 | |
| *** ddaskal_ has quit IRC | 22:37 | |
| *** rpothier has quit IRC | 22:40 | |
| *** hdd has quit IRC | 22:42 | |
| *** jasond has quit IRC | 22:49 | |
| *** dims has joined #heat | 22:53 | |
| *** pratikmallya has joined #heat | 22:54 | |
| *** vijendar has quit IRC | 22:57 | |
| *** mragupat has quit IRC | 22:57 | |
| *** pratikmallya has quit IRC | 22:59 | |
| *** sorantis has quit IRC | 23:01 | |
| *** spotz has quit IRC | 23:04 | |
| *** dims has quit IRC | 23:04 | |
| *** spotz has joined #heat | 23:07 | |
| *** sorantis has joined #heat | 23:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/264419 | 23:14 |
| *** jaime49 has joined #heat | 23:14 | |
| *** jaime49 has quit IRC | 23:16 | |
| *** jaime49 has joined #heat | 23:18 | |
| *** pradk has quit IRC | 23:18 | |
| *** jaime49 has quit IRC | 23:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/264419 | 23:22 |
| *** pratikmallya has joined #heat | 23:22 | |
| *** hdd has joined #heat | 23:24 | |
| *** dims_ has joined #heat | 23:29 | |
| *** hdd_ has joined #heat | 23:29 | |
| *** hdd has quit IRC | 23:29 | |
| *** dims_ has quit IRC | 23:34 | |
| *** sridhar_ram1 has joined #heat | 23:36 | |
| *** dims_ has joined #heat | 23:36 | |
| *** sridhar_ram has quit IRC | 23:37 | |
| *** tochi has joined #heat | 23:40 | |
| *** kebray has quit IRC | 23:41 | |
| *** Qiming has joined #heat | 23:41 | |
| *** kebray has joined #heat | 23:46 | |
| *** spotz is now known as spotz_zzz | 23:50 | |
| *** ddaskal has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!