| *** pratikmallya has quit IRC | 00:03 | |
| *** yuanying has quit IRC | 00:04 | |
| *** yuanying has joined #heat | 00:05 | |
| *** hdd has quit IRC | 00:09 | |
| *** alex_xu has quit IRC | 00:19 | |
| *** alex_xu has joined #heat | 00:21 | |
| *** EricGonczer_ has joined #heat | 00:21 | |
| *** psanchez has quit IRC | 00:21 | |
| *** psanchez has joined #heat | 00:23 | |
| *** sdake has quit IRC | 00:25 | |
| *** sridhar_ram has quit IRC | 00:25 | |
| *** gokrokve has joined #heat | 00:28 | |
| *** chlong has joined #heat | 00:28 | |
| *** mkoderer has quit IRC | 00:30 | |
| *** yuanying has quit IRC | 00:32 | |
| *** sridhar_ram has joined #heat | 00:32 | |
| *** yuanying has joined #heat | 00:34 | |
| *** mkoderer has joined #heat | 00:37 | |
| *** gokrokve has quit IRC | 00:43 | |
| *** gokrokve has joined #heat | 00:44 | |
| *** yuanying has quit IRC | 00:44 | |
| *** yuanying has joined #heat | 00:47 | |
| *** gokrokve has quit IRC | 00:49 | |
| *** openstackstatus has quit IRC | 00:50 | |
| *** openstack has joined #heat | 00:53 | |
| *** gokrokve has joined #heat | 00:58 | |
| *** sridhar_ram has joined #heat | 01:01 | |
| openstackgerrit | Merged openstack/heat: Fix mocking in several tests causing excessive run-times https://review.openstack.org/254981 | 01:01 |
|---|---|---|
| *** ricolin has joined #heat | 01:06 | |
| *** sridhar_ram has quit IRC | 01:22 | |
| *** sridhar_ram1 has joined #heat | 01:24 | |
| *** zhenguo has joined #heat | 01:24 | |
| *** sridhar_ram1 has quit IRC | 01:25 | |
| *** Yanyanhu has joined #heat | 01:27 | |
| *** sridhar_ram has joined #heat | 01:37 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Don't check existence of keystone resource for empty value https://review.openstack.org/254672 | 01:41 |
| *** alex_xu has quit IRC | 01:41 | |
| *** alex_xu has joined #heat | 01:43 | |
| *** sridhar_ram has quit IRC | 01:45 | |
| *** jamielennox is now known as jamielennox|away | 01:47 | |
| *** bana_k has quit IRC | 01:49 | |
| *** gokrokve has quit IRC | 01:50 | |
| *** EricGonczer_ has quit IRC | 01:56 | |
| *** manishg has joined #heat | 01:57 | |
| *** elynn__ has joined #heat | 01:58 | |
| *** sridhar_ram has joined #heat | 01:59 | |
| *** sridhar_ram has quit IRC | 02:02 | |
| *** chlong has quit IRC | 02:03 | |
| *** bana_k has joined #heat | 02:03 | |
| *** elynn___ has joined #heat | 02:03 | |
| *** pratikmallya has joined #heat | 02:03 | |
| *** elynn__ has quit IRC | 02:04 | |
| *** dixiaoli has joined #heat | 02:07 | |
| *** vgridnev has quit IRC | 02:12 | |
| *** gokrokve has joined #heat | 02:14 | |
| *** sthillma has quit IRC | 02:20 | |
| openstackgerrit | Ethan Lynn proposed openstack/heat: Fix AWS::EC2::SecurityGroup update problem https://review.openstack.org/254609 | 02:27 |
| elynn___ | Morning ;) | 02:27 |
| *** gokrokve has quit IRC | 02:30 | |
| *** gokrokve has joined #heat | 02:31 | |
| *** elynn___ has quit IRC | 02:31 | |
| *** elynn has joined #heat | 02:32 | |
| *** sthillma has joined #heat | 02:35 | |
| *** gokrokve has quit IRC | 02:38 | |
| *** manishg has quit IRC | 02:43 | |
| *** manishg has joined #heat | 02:48 | |
| *** jamielennox|away is now known as jamielennox | 02:50 | |
| *** sthillma has quit IRC | 02:51 | |
| *** Qiming has joined #heat | 02:56 | |
| *** gokrokve has joined #heat | 02:59 | |
| *** manishg has quit IRC | 03:04 | |
| *** gokrokve has quit IRC | 03:04 | |
| *** bana_k has quit IRC | 03:13 | |
| *** chlong has joined #heat | 03:16 | |
| *** pratikmallya has quit IRC | 03:21 | |
| *** yuanying has quit IRC | 03:23 | |
| *** links has joined #heat | 03:24 | |
| *** tochi has quit IRC | 03:30 | |
| openstackgerrit | huangtianhua proposed openstack/heat: Support neutron QoSBandwidthLimitRule resource plugin https://review.openstack.org/235117 | 03:35 |
| *** alex_xu_ has joined #heat | 03:40 | |
| *** alex_xu has quit IRC | 03:42 | |
| *** chlong has quit IRC | 03:49 | |
| *** chlong has joined #heat | 03:54 | |
| *** pratikmallya has joined #heat | 03:57 | |
| *** tochi has joined #heat | 03:58 | |
| *** elynn has quit IRC | 03:58 | |
| *** links has quit IRC | 04:03 | |
| *** pratikmallya has quit IRC | 04:05 | |
| *** tochi has quit IRC | 04:11 | |
| *** dims has joined #heat | 04:13 | |
| *** lkarm has joined #heat | 04:15 | |
| *** tochi has joined #heat | 04:15 | |
| *** dims has quit IRC | 04:15 | |
| *** dims has joined #heat | 04:18 | |
| *** lkarm has quit IRC | 04:19 | |
| *** gokrokve has joined #heat | 04:22 | |
| *** yuanying has joined #heat | 04:28 | |
| *** dims has quit IRC | 04:29 | |
| *** gokrokve has quit IRC | 04:30 | |
| stevebaker | zaneb: /msg -bip backlog 5 | 04:31 |
| stevebaker | ugh | 04:31 |
| *** bana_k has joined #heat | 04:32 | |
| *** elynn_ has joined #heat | 04:34 | |
| *** KanagarajM has joined #heat | 04:37 | |
| *** links has joined #heat | 04:38 | |
| *** yuanying has quit IRC | 04:46 | |
| *** bana_k has quit IRC | 04:50 | |
| *** sthillma has joined #heat | 04:53 | |
| *** sthillma_ has joined #heat | 04:55 | |
| *** sthillma has quit IRC | 04:58 | |
| *** sthillma_ is now known as sthillma | 04:58 | |
| *** yuanying has joined #heat | 05:00 | |
| *** gokrokve has joined #heat | 05:01 | |
| *** boris-42_ has quit IRC | 05:03 | |
| *** sdake has joined #heat | 05:15 | |
| *** dims has joined #heat | 05:20 | |
| *** Qiming has quit IRC | 05:22 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Adds ignore_errors for template_validate REST API https://review.openstack.org/246870 | 05:24 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Adds ignore_errors for template_validate in engine https://review.openstack.org/246869 | 05:24 |
| *** Qiming has joined #heat | 05:27 | |
| *** Marga_ has quit IRC | 05:28 | |
| *** gokrokve_ has joined #heat | 05:28 | |
| *** gokrokve has quit IRC | 05:28 | |
| *** mohankumar has joined #heat | 05:29 | |
| *** gberginc has quit IRC | 05:32 | |
| *** vijayagurug has joined #heat | 05:33 | |
| *** vgridnev has joined #heat | 05:42 | |
| *** gokrokve_ has quit IRC | 05:51 | |
| *** gokrokve has joined #heat | 05:52 | |
| *** sthillma has quit IRC | 05:56 | |
| *** Marga_ has joined #heat | 06:12 | |
| *** Marga_ has quit IRC | 06:14 | |
| skraynev | morning all | 06:14 |
| *** Marga_ has joined #heat | 06:14 | |
| ricolin | morning | 06:14 |
| *** Marga_ has quit IRC | 06:15 | |
| ricolin | skraynev: and thanks a lot for the nomination：） | 06:15 |
| *** Marga_ has joined #heat | 06:15 | |
| skraynev | ricolin: np. it's community decision ;) and benefit for the good work ;) | 06:16 |
| ricolin | WooHoo! | 06:17 |
| *** hdd has joined #heat | 06:19 | |
| *** logan- has quit IRC | 06:24 | |
| *** alex_xu has joined #heat | 06:25 | |
| *** alex_xu_ has quit IRC | 06:28 | |
| *** gokrokve has quit IRC | 06:32 | |
| *** hdd has quit IRC | 06:32 | |
| *** thetrav has joined #heat | 06:35 | |
| thetrav | heat-manage db_sync heat is giving me the Invalid target type: None error | 06:35 |
| thetrav | connection string exists in /etc/heat/heat.conf and looks correct to me, however I can't tell if that's what heat-manage is barfing on | 06:36 |
| thetrav | it was in default, not database | 06:37 |
| * thetrav forehead slaps | 06:37 | |
| *** chlong has quit IRC | 06:40 | |
| *** lkarm has joined #heat | 06:42 | |
| openstackgerrit | Rui Chen proposed openstack/python-heatclient: Fix Resource.__eq__ mismatch semantics of object equal https://review.openstack.org/251830 | 06:44 |
| *** lkarm has quit IRC | 06:46 | |
| *** gberginc has joined #heat | 06:47 | |
| skraynev | meeting time | 07:00 |
| *** pratikmallya has joined #heat | 07:00 | |
| *** gokrokve has joined #heat | 07:02 | |
| skraynev | shardy, therve, pas-ha, ramishra: ^ | 07:02 |
| *** jtomasek has joined #heat | 07:03 | |
| *** jtomasek has quit IRC | 07:06 | |
| *** shardy has joined #heat | 07:06 | |
| *** gokrokve has quit IRC | 07:07 | |
| *** pas-ha has joined #heat | 07:09 | |
| *** rpothier has joined #heat | 07:13 | |
| *** pas-ha has quit IRC | 07:15 | |
| *** pas-ha has joined #heat | 07:15 | |
| *** tspatzier has joined #heat | 07:16 | |
| *** jaosorior has joined #heat | 07:19 | |
| *** jaosorior has quit IRC | 07:19 | |
| *** jaosorior has joined #heat | 07:20 | |
| *** jtomasek has joined #heat | 07:21 | |
| *** boris-42_ has joined #heat | 07:31 | |
| *** sridhar_ram has joined #heat | 07:43 | |
| *** pratikmallya has quit IRC | 07:45 | |
| *** ahonda has joined #heat | 07:48 | |
| *** thetrav has quit IRC | 07:50 | |
| *** e0ne has joined #heat | 07:51 | |
| shardy | rpothier: sorry, ran out of time - just looking to check if there's a spec re conditionals | 08:00 |
| rpothier | shardy: the spec is https://review.openstack.org/#/c/245042/ | 08:01 |
| *** zeih has joined #heat | 08:01 | |
| cmyster | shardy: good morning | 08:02 |
| shardy | rpothier: interesting, so it's a direct reinterpretation of the CFN interface | 08:02 |
| shardy | thanks for highlighting it | 08:02 |
| cmyster | shardy: when you have a sec I could use your help with something | 08:03 |
| *** gokrokve has joined #heat | 08:03 | |
| skraynev | shardy: on the last meeting I understood, that we have not decision about it. So I think we may send mail and ask negative sides of this staff. (I know only requests from users and currently don't see any real objections) | 08:04 |
| *** jprovazn has joined #heat | 08:05 | |
| skraynev | ricolin: about External resource, do you need more review for patch or what? I have seen two -1 | 08:05 |
| shardy | skraynev: +1 given my complaints about lack of spec progress/consensus a ML thread is a good idea | 08:05 |
| shardy | FWIW the cfn style interface looks pretty clean to me | 08:05 |
| shardy | The previous attempt at this has been stalled for 10 months https://review.openstack.org/#/c/153771/ | 08:06 |
| ricolin | skraynev: I think that also point out shardy's point that we need more reviewer on blueprint | 08:06 |
| shardy | cmyster: sure - need to grab coffee soon but go ahead ;) | 08:07 |
| ricolin | skraynev: what happen in that bp, that bp get in to specs, but can not be approved | 08:07 |
| cmyster | shardy: then grab that coffee first ;) | 08:07 |
| *** sorantis has joined #heat | 08:07 | |
| shardy | ricolin: someone wastes a lot of time implementing it then ends up blocked and abandons the code | 08:08 |
| shardy | it's happened several times unfortunately :( | 08:08 |
| ricolin | skraynev: I think we need more reviewer on that blueprint | 08:08 |
| *** gokrokve has quit IRC | 08:08 | |
| shardy | cmyster: ack, be back in ~10mins ;) | 08:09 |
| ricolin | shardy: yes, and I think this one just need more discuss on implementation approch | 08:09 |
| cmyster | shardy: cheers | 08:09 |
| ricolin | shardy: I think we can implement a minimum approch for external resource | 08:11 |
| skraynev | shardy: just for clarification. I will send mail with question about objections for Support Conditionals. and you plan to write mail about issue with reviewing process for old Specs, right? | 08:11 |
| ricolin | shardy: then extend that approch if needed | 08:15 |
| *** palexster has joined #heat | 08:21 | |
| shardy | ricolin: agree - a good first step would be to only support external_resource_id, and not allow the dance where you remove the id and heat takes control of the resource | 08:23 |
| *** vgridnev has quit IRC | 08:23 | |
| *** bana_k has joined #heat | 08:23 | |
| ricolin | shardy: agree with that | 08:26 |
| therve | shardy, Dang how do you manage to make your patch https://review.openstack.org/#/c/254981/ created 4 days after https://review.openstack.org/#/c/253479/ land before :) | 08:27 |
| ricolin | shardy: I will update it and add more reviewer | 08:27 |
| *** huangtianhua has quit IRC | 08:29 | |
| *** tiantian has joined #heat | 08:30 | |
| shardy | ricolin: oops - sorry therve I didn't see that patch | 08:32 |
| therve | shardy, I more worries how reviewers ignored me for 5 days :p | 08:33 |
| shardy | therve: Yeah, it's a big problem for sure, everyone is struggling to keep up with the volume of reviews | 08:34 |
| shardy | anyway, sorry for missing it | 08:34 |
| shardy | there are actually a bunch of other broken tests, some of which I fixed in my patch | 08:34 |
| shardy | my local tox runtime went down by half after https://review.openstack.org/#/c/254981/ | 08:35 |
| shardy | there are still others which are broken and taking ~20 seconds to run | 08:35 |
| therve | The gate doesn't show that much | 08:36 |
| therve | Probably faster than your machine | 08:36 |
| therve | The timeouts on wait conditions can probably be improved | 08:37 |
| shardy | Yeah, I didn't notice it until I finally got annoyed by how long I was waiting locally ;) | 08:37 |
| shardy | therve: one strange thing is a lot of tests are breaking trying to contact services for custom contraint validation | 08:37 |
| shardy | but the tests still pass, they just take a long time | 08:37 |
| shardy | I've not figured out why yet | 08:38 |
| therve | Not sure I see that locally | 08:38 |
| shardy | revert https://review.openstack.org/#/c/254981/ | 08:38 |
| therve | shardy, Maybe you have a pyc around? | 08:38 |
| *** tiantian has quit IRC | 08:38 | |
| therve | Ah yeah before your patch maybe | 08:39 |
| *** shardy has quit IRC | 08:42 | |
| *** shardy has joined #heat | 08:43 | |
| *** shardy has quit IRC | 08:49 | |
| *** shardy has joined #heat | 08:51 | |
| openstackgerrit | Thomas Herve proposed openstack/heat: Fix time mocking on wait condition timeout tests https://review.openstack.org/255134 | 08:55 |
| *** jaosorior has quit IRC | 08:55 | |
| therve | shardy, https://review.openstack.org/#/c/255134/ another 10s :) | 08:55 |
| *** zeih has quit IRC | 08:58 | |
| *** yassine__ has joined #heat | 09:01 | |
| openstackgerrit | Ethan Lynn proposed openstack/heat: Add OS::Senlin::Cluster resource https://review.openstack.org/247429 | 09:02 |
| *** bana_k has quit IRC | 09:03 | |
| shardy | therve: thanks! :) | 09:03 |
| *** gokrokve has joined #heat | 09:04 | |
| *** openstackstatus has quit IRC | 09:05 | |
| *** openstack has joined #heat | 09:08 | |
| *** gokrokve has quit IRC | 09:09 | |
| *** blues-man has joined #heat | 09:12 | |
| *** e0ne has quit IRC | 09:15 | |
| *** jaosorior has joined #heat | 09:16 | |
| *** zeih has joined #heat | 09:18 | |
| *** skraynev has quit IRC | 09:20 | |
| openstackgerrit | Merged openstack/heat: Add subnetpool property to subnet resource https://review.openstack.org/246188 | 09:21 |
| *** lkarm has joined #heat | 09:25 | |
| *** vijayagurug has joined #heat | 09:28 | |
| *** skraynev has joined #heat | 09:28 | |
| *** sridhar_ram has quit IRC | 09:29 | |
| *** lkarm has quit IRC | 09:29 | |
| *** vgridnev has joined #heat | 09:32 | |
| *** openstackgerrit has quit IRC | 09:32 | |
| *** openstackgerrit has joined #heat | 09:33 | |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Don't call neutron when there is no prop_diff https://review.openstack.org/251762 | 09:36 |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Fix provider_net resource update https://review.openstack.org/252192 | 09:36 |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Make value_specs updatable for firewall resource https://review.openstack.org/252159 | 09:36 |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Fix OS::Neutron:Net handle_update https://review.openstack.org/255158 | 09:36 |
| *** blues-man has quit IRC | 09:48 | |
| *** skraynev has quit IRC | 09:48 | |
| *** skraynev has joined #heat | 09:49 | |
| *** blues-man has joined #heat | 09:53 | |
| *** nijaba has quit IRC | 10:02 | |
| *** openstackgerrit has quit IRC | 10:02 | |
| *** zhenguo has quit IRC | 10:02 | |
| *** openstackgerrit has joined #heat | 10:03 | |
| *** e0ne has joined #heat | 10:10 | |
| *** jamielennox is now known as jamielennox|away | 10:13 | |
| openstackgerrit | Merged openstack/heat: Remove unused LOG variable with logging import https://review.openstack.org/254653 | 10:14 |
| *** sileht has quit IRC | 10:16 | |
| *** sileht has joined #heat | 10:19 | |
| *** sileht has quit IRC | 10:19 | |
| *** sileht has joined #heat | 10:20 | |
| *** jcoufal has joined #heat | 10:25 | |
| *** dims has quit IRC | 10:25 | |
| *** Yanyanhu has quit IRC | 10:26 | |
| openstackgerrit | Merged openstack/heat: Fix time mocking on wait condition timeout tests https://review.openstack.org/255134 | 10:30 |
| openstackgerrit | Rico Lin proposed openstack/heat: Add validate_with_client in BaseCustomConstraint https://review.openstack.org/250583 | 10:32 |
| openstackgerrit | Rico Lin proposed openstack/heat: Use EntityNotFound instead of NotFound from Manila https://review.openstack.org/250623 | 10:38 |
| *** jistr has joined #heat | 10:39 | |
| openstackgerrit | Ethan Lynn proposed openstack/heat: Add OS::Senlin::Cluster resource https://review.openstack.org/247429 | 10:40 |
| *** mohankumar has quit IRC | 10:41 | |
| *** yassine__ has quit IRC | 10:56 | |
| *** dixiaoli has quit IRC | 10:59 | |
| *** ricolin has quit IRC | 10:59 | |
| *** tlashchova has joined #heat | 11:03 | |
| openstackgerrit | Ethan Lynn proposed openstack/heat: Add OS::Senlin::Profile resource https://review.openstack.org/247430 | 11:04 |
| *** yuanying has quit IRC | 11:07 | |
| *** links has quit IRC | 11:26 | |
| skraynev | sirushti: hi. around? | 11:32 |
| *** tochi has quit IRC | 11:34 | |
| openstackgerrit | Ethan Lynn proposed openstack/heat: Refactor neutron constraints https://review.openstack.org/253326 | 11:34 |
| *** dims has joined #heat | 11:37 | |
| *** gokrokve has joined #heat | 11:50 | |
| openstackgerrit | Ethan Lynn proposed openstack/heat: Add neutron provider constraints https://review.openstack.org/253327 | 11:52 |
| *** gokrokve has quit IRC | 11:55 | |
| *** elynn_ has quit IRC | 11:59 | |
| *** yassine__ has joined #heat | 12:00 | |
| *** prazumovsky has joined #heat | 12:01 | |
| *** mohankumar has joined #heat | 12:06 | |
| *** KanagarajM has quit IRC | 12:13 | |
| *** links has joined #heat | 12:16 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Fix min value in ttl range of Designate::Record https://review.openstack.org/255229 | 12:22 |
| *** EricGonczer_ has joined #heat | 12:24 | |
| *** EricGonczer_ has quit IRC | 12:25 | |
| *** sorantis_ has joined #heat | 12:32 | |
| *** sorantis has quit IRC | 12:34 | |
| *** sorantis_ is now known as sorantis | 12:34 | |
| *** jaosorior has quit IRC | 12:42 | |
| *** chlong has joined #heat | 12:42 | |
| *** jaosorior has joined #heat | 12:42 | |
| *** boris-42_ has quit IRC | 12:43 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: Fix _show_resource in Designate::Record https://review.openstack.org/255242 | 12:46 |
| *** shardy has quit IRC | 12:49 | |
| *** shardy has joined #heat | 12:51 | |
| *** rpothier has quit IRC | 12:52 | |
| openstackgerrit | Sergey Kraynev proposed openstack/heat: Fix race condition for WaitCondition with several signals https://review.openstack.org/232124 | 12:58 |
| *** dprince has joined #heat | 13:01 | |
| *** manishg has joined #heat | 13:10 | |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Call _persist_state for UPDATE in 'state_set' https://review.openstack.org/254619 | 13:13 |
| *** manishg has quit IRC | 13:15 | |
| prazumovsky | @KanagarajM, please note my comment in https://review.openstack.org/#/c/254256/ | 13:16 |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: [WIP] Designate::Record get_live_state implementation https://review.openstack.org/255253 | 13:19 |
| openstackgerrit | Sergey Kraynev proposed openstack/heat: Fix race condition for WaitCondition with several signals https://review.openstack.org/232124 | 13:23 |
| *** vijayagurug has left #heat | 13:29 | |
| *** lkarm has joined #heat | 13:31 | |
| *** EricGonczer_ has joined #heat | 13:33 | |
| *** psanchez has quit IRC | 13:37 | |
| *** EricGonc_ has joined #heat | 13:40 | |
| *** arlion has quit IRC | 13:43 | |
| *** EricGonc_ has quit IRC | 13:44 | |
| *** EricGonczer_ has quit IRC | 13:44 | |
| *** arlion has joined #heat | 13:47 | |
| *** arlion has joined #heat | 13:47 | |
| *** gberginc has quit IRC | 13:47 | |
| *** arlion has quit IRC | 13:48 | |
| *** arlion has joined #heat | 13:51 | |
| *** arlion has joined #heat | 13:51 | |
| *** sorantis has quit IRC | 13:53 | |
| openstackgerrit | Rico Lin proposed openstack/heat: Use specified client service name for client usage https://review.openstack.org/250687 | 13:54 |
| *** logan- has joined #heat | 14:01 | |
| *** sdake has quit IRC | 14:02 | |
| *** jonesbr has joined #heat | 14:09 | |
| *** manishg has joined #heat | 14:11 | |
| *** pratikmallya has joined #heat | 14:12 | |
| *** manishg has quit IRC | 14:15 | |
| *** gokrokve has joined #heat | 14:22 | |
| *** pratikmallya has quit IRC | 14:22 | |
| sirushti | skraynev, hey, what's up? | 14:25 |
| *** blomquisg has quit IRC | 14:25 | |
| skraynev | sirushti: hey. I had a question about one my patch | 14:25 |
| skraynev | I tried to get engine_id value inside of resource.py class | 14:25 |
| sirushti | skraynev, sure | 14:25 |
| skraynev | so wanted to ask how will be better to do it | 14:25 |
| skraynev | I implemented some obvious approach | 14:26 |
| skraynev | so comments are welcome | 14:26 |
| sirushti | skraynev, there's a patch up for review? | 14:26 |
| skraynev | sirushti: https://review.openstack.org/#/c/232124/ | 14:28 |
| sirushti | skraynev, sure, I'll try and take a look at it in a bit after the test results for that come back. | 14:30 |
| sirushti | skraynev, wait, it just did | 14:31 |
| *** dyasny has quit IRC | 14:31 | |
| *** spzala has joined #heat | 14:32 | |
| skraynev | sirushti: yes. and It works (because I fixed it), but I am wondering, that it's correct way | 14:34 |
| *** jdob_lt has joined #heat | 14:34 | |
| *** vijendar has joined #heat | 14:38 | |
| openstackgerrit | Peter Razumovsky proposed openstack/heat: [WIP] Neutron::Net get_live_state implementation https://review.openstack.org/255287 | 14:38 |
| ramishra_ | it seems 'gate-grenade-dsvm-heat' job is failing frequently:/ | 14:39 |
| *** blomquisg has joined #heat | 14:40 | |
| *** pradk has joined #heat | 14:42 | |
| *** pradk has quit IRC | 14:43 | |
| *** KanagarajM has joined #heat | 14:44 | |
| ramishra_ | hmm.. 'Server 1686b183-dcd0-454f-84d7-7c8d6aa725d1 failed to reach ACTIVE status' and 'Server dea77eb7-1615-41ab-ab51-5a48dff0bacb failed to build and is in ERROR status'. looks like infra issue. | 14:44 |
| *** pradk has joined #heat | 14:44 | |
| *** pratikmallya has joined #heat | 14:45 | |
| *** neelashah has joined #heat | 14:45 | |
| *** dyasny has joined #heat | 14:46 | |
| *** pratikma_ has joined #heat | 14:46 | |
| *** gokrokve has quit IRC | 14:49 | |
| *** pratikmallya has quit IRC | 14:50 | |
| *** jprovazn has quit IRC | 14:50 | |
| *** prazumovsky has quit IRC | 14:57 | |
| *** jasond has joined #heat | 15:05 | |
| *** EricGonczer_ has joined #heat | 15:08 | |
| *** manishg has joined #heat | 15:11 | |
| openstackgerrit | Rico Lin proposed openstack/heat: Use specified client service name for client usage https://review.openstack.org/250687 | 15:14 |
| *** mohankumar has quit IRC | 15:17 | |
| *** derekh has joined #heat | 15:18 | |
| *** zeih has quit IRC | 15:22 | |
| *** manishg has quit IRC | 15:22 | |
| shardy | therve: Hey, question re bug #1506079 | 15:23 |
| openstack | bug 1506079 in heat "purge_deleted doesn't handle prev_raw_template_id" [Medium,Fix released] https://launchpad.net/bugs/1506079 - Assigned to Thomas Herve (therve) | 15:23 |
| therve | Yep | 15:23 |
| shardy | therve: since that fix, I get an error from heat-manage purge_deleted: | 15:23 |
| shardy | http://paste.openstack.org/show/481326/ | 15:23 |
| shardy | I see the same thing in two different (near master) environments, any idea why? | 15:23 |
| therve | Hum, no | 15:25 |
| shardy | therve: k, something seems up with the query, and reverting the fix solves it, I'll raise a bug and we can work it out there | 15:26 |
| therve | :/ | 15:26 |
| therve | shardy, Is it a case of a query working on sqlite and not mysql? | 15:26 |
| therve | It took some time to make the test pass | 15:27 |
| shardy | therve: could be - I'm running it against mariadb on Fedora and Centos | 15:27 |
| shardy | do we have an integration test for that or only unit tests? | 15:27 |
| therve | Only unit I believe | 15:27 |
| therve | But that query looks good to me... | 15:28 |
| *** gokrokve has joined #heat | 15:30 | |
| shardy | therve: strange, OK I've raised bug #1524387 and we can work it out there | 15:30 |
| openstack | bug 1524387 in heat "heat-manage purge_deleted broken" [Undecided,New] https://launchpad.net/bugs/1524387 | 15:30 |
| *** sorantis has joined #heat | 15:30 | |
| *** inc0 has joined #heat | 15:30 | |
| therve | shardy, Yeah reproduced... | 15:31 |
| therve | Sigh | 15:31 |
| therve | shardy, I don't think I can make a query work for both sqlite and mysql | 15:33 |
| *** dprince has quit IRC | 15:33 | |
| shardy | therve: IMO the important thing is making it work with non-sqlite even if we can only have integration test coverage vs unit tests | 15:34 |
| therve | shardy, That can work if we actually fetch the data before the delete | 15:34 |
| *** gokrokve has quit IRC | 15:35 | |
| *** jprovazn has joined #heat | 15:35 | |
| shardy | therve: Yeah, can we do a simpler but less efficient query rows, save ids, then delete rows sequence? | 15:36 |
| shardy | It's not like this tool is all that performance critical, folks are running it as a cron job | 15:36 |
| therve | shardy, http://paste.openstack.org/show/481328/ maybe | 15:37 |
| therve | It seems to at least not blow up | 15:38 |
| openstackgerrit | Mark Vanderwiel proposed openstack/python-heatclient: Add openstack client resource type list and show https://review.openstack.org/254330 | 15:40 |
| shardy | therve: +1, that works for me | 15:40 |
| openstackgerrit | Mark Vanderwiel proposed openstack/python-heatclient: Add openstack client resource type list and show https://review.openstack.org/254330 | 15:41 |
| *** rpothier has joined #heat | 15:43 | |
| *** blues-man has quit IRC | 15:43 | |
| *** blues-man has joined #heat | 15:43 | |
| therve | OK I'll propose that then | 15:44 |
| *** gokrokve has joined #heat | 15:45 | |
| *** sorantis has quit IRC | 15:45 | |
| *** Qiming has quit IRC | 15:45 | |
| openstackgerrit | Thomas Herve proposed openstack/heat: Fix raw_template purge query https://review.openstack.org/255328 | 15:46 |
| therve | shardy, On a side note, I wonder if we don't trigger backports a bit too quickly | 15:46 |
| therve | It's the 2nd broken backport I see in 2 days | 15:46 |
| therve | Maybe it's just a sad state of our testing infrastructure | 15:47 |
| *** blomquisg has quit IRC | 15:47 | |
| *** blomquisg has joined #heat | 15:47 | |
| *** nijaba has joined #heat | 15:47 | |
| *** nijaba has quit IRC | 15:47 | |
| *** nijaba has joined #heat | 15:47 | |
| shardy | therve: I think it's a test/CI issue - that backport landed nearly a month after the commit to master | 15:48 |
| shardy | and, ultimately, folks want to consume the fixes downstream of the stable branches | 15:49 |
| therve | Fair enough | 15:49 |
| therve | I wonder if we could setup a unit test env in the gate using mysql | 15:49 |
| shardy | therve: possibly - would it be easier to have an integration test, which creates a bunch of stacks, then purges them? | 15:52 |
| *** vijayagurug has joined #heat | 15:53 | |
| therve | shardy, For that particular issue sure | 15:53 |
| *** vijayagurug has left #heat | 15:53 | |
| therve | I guess we'll start here :) | 15:53 |
| shardy | I was thinking having an integration test, then using db credentials to check the DB tables, and stack-show <id> on the soft-deleted-now-gone stacks would be a pretty good test | 15:55 |
| *** sdake has joined #heat | 15:55 | |
| shardy | it's certainly a lot closer to real world usage than sqlite ;) | 15:55 |
| *** gokrokve has quit IRC | 15:58 | |
| *** gokrokve has joined #heat | 15:59 | |
| *** randallburt has joined #heat | 16:02 | |
| *** randallburt has quit IRC | 16:02 | |
| *** randallburt has joined #heat | 16:03 | |
| *** pradk has quit IRC | 16:05 | |
| *** vishwanathj has quit IRC | 16:08 | |
| *** vishwanathj has joined #heat | 16:08 | |
| *** manishg has joined #heat | 16:10 | |
| *** pradk has joined #heat | 16:10 | |
| *** sdake has quit IRC | 16:10 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Fixes the error logic in resource_data_get_all() https://review.openstack.org/255347 | 16:14 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: heat-manage resource_data_list https://review.openstack.org/255348 | 16:14 |
| *** bana_k has joined #heat | 16:17 | |
| *** jcoufal has quit IRC | 16:20 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: heat-manage resource_data_list https://review.openstack.org/255348 | 16:21 |
| *** vgridnev has quit IRC | 16:23 | |
| openstackgerrit | Thomas Herve proposed openstack/heat: Fix raw_template purge query https://review.openstack.org/255328 | 16:24 |
| therve | shardy, added a simple test | 16:24 |
| shardy | therve: great, thanks! :) | 16:26 |
| *** Drago has joined #heat | 16:27 | |
| *** jdob_lt has quit IRC | 16:29 | |
| *** crose has joined #heat | 16:31 | |
| *** jaosorior has quit IRC | 16:32 | |
| *** oudimou has joined #heat | 16:32 | |
| *** crose has quit IRC | 16:33 | |
| *** yassine__ has quit IRC | 16:38 | |
| *** KanagarajM has quit IRC | 16:46 | |
| *** jprovazn has quit IRC | 16:46 | |
| *** KanagarajM has joined #heat | 16:46 | |
| *** jprovazn has joined #heat | 16:48 | |
| *** gokrokve has quit IRC | 16:51 | |
| *** manishg has quit IRC | 16:52 | |
| *** mohankumar has joined #heat | 16:54 | |
| *** sridhar_ram has joined #heat | 16:54 | |
| *** sdake has joined #heat | 16:56 | |
| *** manishg has joined #heat | 16:56 | |
| *** sthillma has joined #heat | 17:00 | |
| *** links has quit IRC | 17:01 | |
| *** sthillma_ has joined #heat | 17:01 | |
| *** sthillma has quit IRC | 17:05 | |
| *** sthillma_ is now known as sthillma | 17:05 | |
| *** jprovazn has quit IRC | 17:06 | |
| *** Marga_ has quit IRC | 17:06 | |
| *** jprovazn has joined #heat | 17:06 | |
| *** dprince has joined #heat | 17:10 | |
| *** psanchez has joined #heat | 17:11 | |
| *** Marga_ has joined #heat | 17:11 | |
| *** Marga_ has quit IRC | 17:12 | |
| openstackgerrit | Merged openstack/heat: Fix min value in ttl range of Designate::Record https://review.openstack.org/255229 | 17:12 |
| *** oudimou has left #heat | 17:14 | |
| *** vgridnev has joined #heat | 17:15 | |
| *** gberginc has joined #heat | 17:16 | |
| *** jistr has quit IRC | 17:19 | |
| *** blomquisg has quit IRC | 17:23 | |
| *** dims has quit IRC | 17:24 | |
| *** oudimou has joined #heat | 17:25 | |
| oudimou | Hi everyone ! | 17:25 |
| oudimou | Is this the right place to ask for help (quick troobleshoot on my template) | 17:26 |
| oudimou | ?` | 17:26 |
| *** blues-man has quit IRC | 17:29 | |
| *** gokrokve has joined #heat | 17:30 | |
| *** blomquisg has joined #heat | 17:36 | |
| *** oudimou has quit IRC | 17:37 | |
| *** manishg has quit IRC | 17:41 | |
| *** kebray has joined #heat | 17:41 | |
| *** blues-man has joined #heat | 17:41 | |
| *** cmyster has quit IRC | 17:42 | |
| *** hdd has joined #heat | 17:43 | |
| *** cmyster has joined #heat | 17:43 | |
| *** cmyster has joined #heat | 17:43 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Add Resource Notifications to Heat https://review.openstack.org/246556 | 17:46 |
| *** gberginc has quit IRC | 17:47 | |
| *** manishg has joined #heat | 17:49 | |
| *** pratikma_ has quit IRC | 17:51 | |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Don't call neutron when there is no prop_diff https://review.openstack.org/251762 | 17:51 |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Fix OS::Neutron:Net handle_update https://review.openstack.org/255158 | 17:51 |
| openstackgerrit | Rabi Mishra proposed openstack/heat: Fix provider_net resource update https://review.openstack.org/252192 | 17:51 |
| *** manishg has quit IRC | 17:54 | |
| openstackgerrit | Mark Vanderwiel proposed openstack/python-heatclient: Add openstack client resource type list and show https://review.openstack.org/254330 | 17:56 |
| *** blues-man has quit IRC | 18:05 | |
| *** sdake has quit IRC | 18:08 | |
| *** blues-man has joined #heat | 18:09 | |
| openstackgerrit | Mark Vanderwiel proposed openstack/python-heatclient: Add openstack client resource type list and show https://review.openstack.org/254330 | 18:11 |
| *** blues-man has quit IRC | 18:14 | |
| *** mohankumar has quit IRC | 18:17 | |
| *** pratikmallya has joined #heat | 18:18 | |
| *** kebray has quit IRC | 18:19 | |
| *** manishg has joined #heat | 18:20 | |
| *** bana_k has quit IRC | 18:24 | |
| *** blues-man has joined #heat | 18:27 | |
| *** larsks has joined #heat | 18:30 | |
| *** bana_k has joined #heat | 18:35 | |
| *** e0ne has quit IRC | 18:41 | |
| *** dsneddon has quit IRC | 18:42 | |
| *** sridhar_ram has quit IRC | 18:42 | |
| *** neelashah has quit IRC | 18:42 | |
| *** sthillma has quit IRC | 18:43 | |
| *** sdake has joined #heat | 18:43 | |
| *** manishg has quit IRC | 18:44 | |
| *** Marga_ has joined #heat | 18:46 | |
| *** dsneddon has joined #heat | 18:46 | |
| *** hdd has quit IRC | 18:47 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: Fixes the error logic in resource_data_get_all() https://review.openstack.org/255347 | 18:47 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: heat-manage resource_data_list https://review.openstack.org/255348 | 18:47 |
| *** Marga__ has joined #heat | 18:49 | |
| *** Marga_ has quit IRC | 18:50 | |
| *** Marga__ has quit IRC | 18:50 | |
| *** Marga_ has joined #heat | 18:51 | |
| *** manishg has joined #heat | 18:57 | |
| *** gokrokve has quit IRC | 18:57 | |
| *** jonesbr has quit IRC | 18:59 | |
| *** gokrokve has joined #heat | 18:59 | |
| *** gokrokve has quit IRC | 18:59 | |
| *** david-ly_ has joined #heat | 19:00 | |
| *** rpothier has quit IRC | 19:01 | |
| *** manishg has quit IRC | 19:02 | |
| *** dprince has quit IRC | 19:03 | |
| *** david-ly_ is now known as david-lyle_ | 19:03 | |
| *** david-lyle has quit IRC | 19:03 | |
| *** jonesbr has joined #heat | 19:04 | |
| *** neelashah has joined #heat | 19:04 | |
| *** gokrokve has joined #heat | 19:04 | |
| *** david-lyle_ is now known as david-lyle | 19:04 | |
| *** gberginc has joined #heat | 19:05 | |
| *** gokrokve has quit IRC | 19:05 | |
| *** gokrokve has joined #heat | 19:06 | |
| *** derekh has quit IRC | 19:10 | |
| *** gokrokve has quit IRC | 19:10 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: WIP Add multi-version supportability in client plugin https://review.openstack.org/255443 | 19:11 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/heat: WIP Adds tag support for glance image resource plugin https://review.openstack.org/255444 | 19:11 |
| *** sridhar_ram has joined #heat | 19:12 | |
| *** e0ne has joined #heat | 19:15 | |
| *** sthillma has joined #heat | 19:16 | |
| *** KanagarajM has quit IRC | 19:37 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Check for None value when determining root_id https://review.openstack.org/255453 | 19:39 |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Check for None value when determining root_id https://review.openstack.org/255453 | 19:40 |
| *** lkarm has quit IRC | 19:40 | |
| *** tspatzier has quit IRC | 19:41 | |
| *** lkarm has joined #heat | 19:44 | |
| *** gokrokve has joined #heat | 19:51 | |
| *** gokrokve has quit IRC | 19:52 | |
| *** gokrokve has joined #heat | 19:52 | |
| *** gokrokve has quit IRC | 19:53 | |
| *** kevinbenton has quit IRC | 19:54 | |
| *** lkarm has quit IRC | 19:56 | |
| *** sthillma has quit IRC | 19:57 | |
| *** lkarm has joined #heat | 19:57 | |
| *** spzala has quit IRC | 19:57 | |
| *** david-lyle has quit IRC | 19:58 | |
| *** lkarm has quit IRC | 19:58 | |
| *** david-lyle has joined #heat | 19:58 | |
| *** dprince has joined #heat | 19:58 | |
| *** blomquisg has quit IRC | 19:59 | |
| *** neelashah has quit IRC | 20:03 | |
| *** sridhar_ram has quit IRC | 20:04 | |
| *** lkarm has joined #heat | 20:05 | |
| openstackgerrit | Bryan Jones proposed openstack/heat: LBaaS v2 Pool Member https://review.openstack.org/241251 | 20:09 |
| openstackgerrit | Bryan Jones proposed openstack/heat: LBaaS v2 Pool https://review.openstack.org/240049 | 20:09 |
| openstackgerrit | Bryan Jones proposed openstack/heat: LBaas v2 HealthMonitor https://review.openstack.org/245287 | 20:09 |
| openstackgerrit | Bryan Jones proposed openstack/heat: Add cmd_resource option to NeutronClient.resolve https://review.openstack.org/241008 | 20:09 |
| openstackgerrit | Bryan Jones proposed openstack/heat: LBaaS v2 Loadbalancer https://review.openstack.org/238612 | 20:09 |
| openstackgerrit | Bryan Jones proposed openstack/heat: LBaaS v2 Listener https://review.openstack.org/239755 | 20:09 |
| *** lkarm has quit IRC | 20:10 | |
| *** Marga_ has quit IRC | 20:11 | |
| *** Marga_ has joined #heat | 20:12 | |
| *** blomquisg has joined #heat | 20:13 | |
| *** kevinbenton_ has joined #heat | 20:13 | |
| *** kevinbenton_ has quit IRC | 20:14 | |
| *** kevinbenton has joined #heat | 20:14 | |
| *** vgridnev has quit IRC | 20:17 | |
| *** manishg has joined #heat | 20:21 | |
| *** gokrokve has joined #heat | 20:24 | |
| *** spzala has joined #heat | 20:24 | |
| *** sthillma has joined #heat | 20:26 | |
| *** gokrokve has quit IRC | 20:27 | |
| *** gokrokve has joined #heat | 20:28 | |
| openstackgerrit | Jason Dunsmore proposed openstack/heat: Do not allow new creates for HIDDEN resources https://review.openstack.org/249460 | 20:30 |
| *** randallburt has quit IRC | 20:31 | |
| openstackgerrit | Mark Vanderwiel proposed openstack/python-heatclient: Add openstack client stack resource metadata show https://review.openstack.org/255483 | 20:32 |
| *** jprovazn has quit IRC | 20:32 | |
| *** randallburt has joined #heat | 20:32 | |
| *** sridhar_ram has joined #heat | 20:34 | |
| *** neelashah has joined #heat | 20:37 | |
| *** randallburt has quit IRC | 20:37 | |
| *** Marga_ has quit IRC | 20:42 | |
| *** gokrokve has quit IRC | 20:43 | |
| *** kebray has joined #heat | 20:43 | |
| *** sthillma has quit IRC | 20:45 | |
| *** dims has joined #heat | 20:49 | |
| *** kebray has quit IRC | 20:55 | |
| *** dims has quit IRC | 20:58 | |
| *** Marga_ has joined #heat | 20:59 | |
| *** hdd has joined #heat | 20:59 | |
| *** dims has joined #heat | 21:00 | |
| *** blues-man has quit IRC | 21:00 | |
| *** dims has quit IRC | 21:01 | |
| *** Marga_ has quit IRC | 21:03 | |
| openstackgerrit | Bryan Jones proposed openstack/python-heatclient: OpenstackClient plugin for stack abandon https://review.openstack.org/246613 | 21:10 |
| openstackgerrit | Bryan Jones proposed openstack/python-heatclient: Add JsonFormat display option https://review.openstack.org/255523 | 21:10 |
| *** tspatzier has joined #heat | 21:12 | |
| *** neelashah has quit IRC | 21:15 | |
| *** Marga_ has joined #heat | 21:15 | |
| *** sthillma has joined #heat | 21:16 | |
| *** dprince has quit IRC | 21:19 | |
| *** Marga_ has quit IRC | 21:23 | |
| *** neelashah has joined #heat | 21:27 | |
| openstackgerrit | Bryan Jones proposed openstack/python-heatclient: Add JsonFormat display option https://review.openstack.org/255523 | 21:30 |
| openstackgerrit | Bryan Jones proposed openstack/python-heatclient: OpenstackClient plugin for stack abandon https://review.openstack.org/246613 | 21:30 |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Check for None value when determining root_id https://review.openstack.org/255453 | 21:32 |
| *** e0ne has quit IRC | 21:32 | |
| *** gberginc has quit IRC | 21:32 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Make root_stack_id determination more robust https://review.openstack.org/255453 | 21:36 |
| *** neelashah1 has joined #heat | 21:38 | |
| *** neelashah has quit IRC | 21:38 | |
| *** bana_k has quit IRC | 21:45 | |
| *** Marga_ has joined #heat | 21:51 | |
| openstackgerrit | Bryan Jones proposed openstack/heat: LBaaS v2 Pool Member https://review.openstack.org/241251 | 21:52 |
| openstackgerrit | Bryan Jones proposed openstack/heat: LBaas v2 HealthMonitor https://review.openstack.org/245287 | 21:52 |
| openstackgerrit | Bryan Jones proposed openstack/heat: Add cmd_resource option to NeutronClient.resolve https://review.openstack.org/241008 | 21:52 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat: Updated from global requirements https://review.openstack.org/255551 | 21:55 |
| *** sridhar_ram has quit IRC | 21:56 | |
| *** Marga_ has quit IRC | 22:00 | |
| openstackgerrit | Mark Vanderwiel proposed openstack/python-heatclient: Add openstack client stack resource metadata show https://review.openstack.org/255483 | 22:00 |
| *** pratikma_ has joined #heat | 22:02 | |
| *** pratikma_ has quit IRC | 22:03 | |
| *** cwolferh has quit IRC | 22:03 | |
| *** pratikma_ has joined #heat | 22:03 | |
| *** pratikmallya has quit IRC | 22:03 | |
| *** aweiteka has quit IRC | 22:03 | |
| *** inc0 has quit IRC | 22:04 | |
| *** sridhar_ram has joined #heat | 22:06 | |
| *** neelashah1 has quit IRC | 22:08 | |
| *** bana_k has joined #heat | 22:11 | |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Make root_stack_id determination more robust https://review.openstack.org/255453 | 22:11 |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Make root_stack_id determination more robust https://review.openstack.org/255453 | 22:11 |
| *** neelashah has joined #heat | 22:13 | |
| *** crose has joined #heat | 22:15 | |
| *** EricGonczer_ has quit IRC | 22:22 | |
| Drago | shardy: Hey, you there? | 22:23 |
| *** crose has quit IRC | 22:24 | |
| *** Marga_ has joined #heat | 22:24 | |
| *** neelashah has quit IRC | 22:25 | |
| shardy | Drago: Hi | 22:25 |
| *** jonesbr has left #heat | 22:26 | |
| Drago | shardy: Hi, what would be the process for removing https://github.com/openstack/heat/blob/master/heat/engine/resources/stack_user.py#L76-L83? | 22:26 |
| shardy | Drago: It's difficult, because we have no way of knowing if operators are maintaining user stacks from before the transition from resource_id to resource_data | 22:28 |
| *** Marga_ has quit IRC | 22:28 | |
| shardy | Drago: is it causing you some issues? | 22:28 |
| Drago | shardy: Yes, we have a problem where a subuser (that can't create or delete keystone users) can create a stack, but not delete it because Heat attempts to delete the stack user (which was never created) because _get_user_id always returns an ID | 22:30 |
| Drago | The stack user is never created since we're using temp_urls here | 22:31 |
| shardy | Drago: Ok, hmm, well it looks like it was introduced in Icehouse, so that's EOL upstream & we can probably justifiably remove it | 22:32 |
| shardy | Drago: the issue is, the stacks created on Icehouse shouldn't be EOL, even if icehouse itself is | 22:33 |
| *** david-lyle has quit IRC | 22:34 | |
| shardy | Drago: I guess we either remove it and replace it with an error in the log, or make it configurable | 22:35 |
| shardy | Drago: are you using the in-tree tempurl implementation, and that never sets user_id? | 22:35 |
| shardy | If so, we could just set the user_id to None for that, then we'd not hit that path | 22:36 |
| Drago | shardy: I understand. The problem seems like it could be fixed if icehouse stacks could be migrated/fixed. It would be a matter of populating 'user_id' with the resource ID in the resource metadata like _get_user_id does now | 22:36 |
| Drago | shardy: Yes, user_id is not set until the first time _get_user_id is called | 22:37 |
| shardy | Drago: Ok, cool, well I'm fine with removing it, but it'd be good to document migration of any icehouse stacks in the release notes | 22:38 |
| Drago | shardy: user_id being none is precisely the reason that path is hit :) | 22:38 |
| shardy | Drago: heh, yeah, sorry getting late here ;) | 22:38 |
| *** david-ly_ has joined #heat | 22:39 | |
| Drago | shardy: Does migration handle populating user_id for icehouse stacks? Or, are you saying that operators would have to do that themselves? | 22:39 |
| openstackgerrit | Pratik Mallya proposed openstack/heat: Add Resource Notifications to Heat https://review.openstack.org/246556 | 22:39 |
| shardy | Drago: well it populates it if you do anything that makes the stack call _get_user_id | 22:40 |
| shardy | but only with that workaround in place ;) | 22:40 |
| shardy | Drago: we don't have a DB data migration which does it | 22:40 |
| shardy | only that code | 22:41 |
| Drago | shardy: right, so should something be added for operators to do that on-demand at the same time this code is removed? | 22:41 |
| shardy | Drago: that would be ideal, yes | 22:42 |
| shardy | then we replace the FIXME part with a LOG.error which points to a bug which explains the operator workaround, and we document it in the release notes | 22:42 |
| *** pratikmallya has joined #heat | 22:43 | |
| shardy | Drago: if you're happy to raise a bug and document such an operator workaround, that would be great & I'm +2 on removing the code | 22:43 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/heat: remove containers on delete https://review.openstack.org/118190 | 22:43 |
| *** tspatzier has quit IRC | 22:44 | |
| Drago | shardy: cool, I'll work on it, thanks! | 22:44 |
| shardy | Drago: great, thanks! :) | 22:44 |
| *** tspatzier has joined #heat | 22:45 | |
| shardy | night all | 22:45 |
| *** shardy has quit IRC | 22:45 | |
| *** david-ly_ is now known as david-lyle | 22:45 | |
| *** tspatzier has quit IRC | 22:45 | |
| *** pratikma_ has quit IRC | 22:46 | |
| *** pratikmallya has quit IRC | 22:47 | |
| *** harlowja has quit IRC | 22:51 | |
| *** harlowja has joined #heat | 22:51 | |
| *** sdake has quit IRC | 22:52 | |
| openstackgerrit | Mark Vanderwiel proposed openstack/python-heatclient: Add openstack client resource type list and show https://review.openstack.org/254330 | 22:52 |
| *** Marga_ has joined #heat | 22:55 | |
| *** yuanying has joined #heat | 22:56 | |
| *** jasond has quit IRC | 22:57 | |
| *** blomquisg has quit IRC | 22:59 | |
| *** boris-42_ has joined #heat | 22:59 | |
| *** Marga_ has quit IRC | 22:59 | |
| *** vijendar has quit IRC | 23:01 | |
| *** vgridnev has joined #heat | 23:03 | |
| *** gokrokve has joined #heat | 23:04 | |
| *** sthillma has quit IRC | 23:05 | |
| *** gokrokve has quit IRC | 23:08 | |
| *** Marga_ has joined #heat | 23:11 | |
| *** blomquisg has joined #heat | 23:12 | |
| *** Marga_ has quit IRC | 23:12 | |
| *** Marga_ has joined #heat | 23:13 | |
| *** Marga_ has quit IRC | 23:13 | |
| *** Marga_ has joined #heat | 23:14 | |
| *** vgridnev has quit IRC | 23:19 | |
| *** sthillma has joined #heat | 23:36 | |
| *** tochi has joined #heat | 23:37 | |
| *** cwolferh has joined #heat | 23:38 | |
| *** pradk has quit IRC | 23:40 | |
| *** ahonda has quit IRC | 23:44 | |
| *** spzala has quit IRC | 23:45 | |
| *** kebray has joined #heat | 23:49 | |
| *** ahonda has joined #heat | 23:49 | |
| *** Qiming has joined #heat | 23:51 | |
| *** kebray has quit IRC | 23:51 | |
| *** manishg has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!