Wednesday, 2017-06-14

« Tuesday, 2017-06-13 Index Thursday, 2017-06-15 »
*** fragatina has quit IRC00:02
*** boris-42_ has joined #heat00:05
*** gomarivera has quit IRC00:17
*** gomarivera has joined #heat00:18
*** gomarivera has quit IRC00:19
*** gomarivera has joined #heat00:19
*** gomarivera has quit IRC00:29
*** gomarivera has joined #heat00:39
*** ricolin has joined #heat00:45
*** zhurong has joined #heat00:45
openstackgerritMerged openstack/heat master: Get the deployment group resources again after CREATE_COMPLETE  https://review.openstack.org/47172700:54
*** fragatina has joined #heat00:59
*** gomarivera has quit IRC01:00
*** mdnadeem has joined #heat01:03
*** gomarivera has joined #heat01:04
*** fragatina has quit IRC01:05
*** gomarivera has quit IRC01:06
*** gomarivera has joined #heat01:07
*** gomarivera has quit IRC01:07
*** gomarivera has joined #heat01:08
openstackgerrithuangtianhua proposed openstack/heat master: Add 'contains' function  https://review.openstack.org/47159501:13
*** zhurong has quit IRC01:33
*** zhurong has joined #heat01:35
*** namnh has joined #heat01:44
*** gomarivera has quit IRC01:44
*** gomarivera has joined #heat01:44
*** gomarivera has quit IRC01:48
*** fragatina has joined #heat02:01
*** fragatina has quit IRC02:06
*** nwonknu has quit IRC02:12
*** nwonknu has joined #heat02:15
*** nwonknu is now known as Guest1880102:16
*** gomarivera has joined #heat02:17
*** sgran has quit IRC02:19
*** sgran has joined #heat02:26
*** gomarivera has quit IRC02:32
*** gomarivera has joined #heat02:33
*** gomarivera has quit IRC02:45
*** gomarivera has joined #heat02:45
*** gomarivera has quit IRC02:50
*** nicolasbock has joined #heat02:56
*** fragatina has joined #heat03:02
*** fragatina has quit IRC03:06
*** ramishra has joined #heat03:09
*** links has joined #heat03:14
*** dbecker has quit IRC03:18
*** Shunli has joined #heat03:24
*** dbecker has joined #heat03:35
*** dimtruck is now known as zz_dimtruck03:47
*** fragatina has joined #heat04:03
*** fragatina has quit IRC04:07
*** gkadam has joined #heat04:08
*** hoonetorg has quit IRC04:11
*** hoonetorg has joined #heat04:22
*** zhurong has quit IRC04:28
*** EmilienM has quit IRC04:30
*** EmilienM has joined #heat04:35
*** zhurong has joined #heat04:38
*** ramishra has quit IRC04:39
*** mikal has quit IRC04:48
*** mikal has joined #heat04:50
*** prateek has joined #heat04:54
*** skramaja has joined #heat05:12
*** bnemec has quit IRC05:16
*** zhurong has quit IRC05:17
*** zhurong has joined #heat05:20
*** boris-42_ has quit IRC05:34
*** armaan has joined #heat05:43
*** ltomasbo|away is now known as ltomasbo06:01
*** rcernin has joined #heat06:08
thervetiantian, There seems to be something wrong with reno :/06:13
*** LanceHaig has joined #heat06:14
LanceHaigricolin, sorry I missed the meeting yesterday I will read the transcripts06:19
therveLanceHaig, Meeting is today06:20
LanceHaighaha06:21
LanceHaigI need sleep06:21
*** jprovazn has joined #heat06:22
*** Guest77651 has quit IRC06:32
ricolinLanceHaig, you can sleep for 6 hr and join meeting:)06:38
LanceHaig:-)06:38
*** hjensas has quit IRC06:42
*** jaosorior has joined #heat06:49
*** d0ugal has joined #heat07:11
*** ccamacho has joined #heat07:27
*** armaan has quit IRC07:29
*** armaan has joined #heat07:31
*** shardy has joined #heat07:43
*** ghs39ghsggf has joined #heat07:43
*** ghs39ghsggf has left #heat07:43
*** gomarivera has joined #heat07:46
*** hjensas has joined #heat07:46
openstackgerrithuangtianhua proposed openstack/heat master: Add reno for soft policies supporting  https://review.openstack.org/44939307:49
*** gomarivera has quit IRC07:50
*** amoralej|off is now known as amoralej07:52
*** ramishra has joined #heat07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #heat08:00
*** ltomasbo is now known as ltomasbo|away08:10
*** fragatina has joined #heat08:11
*** fragatina has quit IRC08:12
*** fragatina has joined #heat08:13
tiantianheat reno jenkins is broken?08:36
therveYeah the last release introduced a regression08:37
therveI talked to the release guys, ttx is looking into it08:37
thervetiantian, You can remove the reno from the patch and add it later if you want08:37
*** fragatina has quit IRC08:37
therveI think we only build reno when there are new ones08:37
ricolintherve, yep08:38
tiantiantherve, ok, thanks :)08:38
*** armaan has quit IRC08:56
*** dbecker has quit IRC09:04
*** derekh has joined #heat09:06
*** dbecker has joined #heat09:06
*** stokvis has joined #heat09:12
*** stokvis has quit IRC09:19
*** LanceHaig has quit IRC09:19
*** LanceHaig has joined #heat09:19
*** LanceHaig has quit IRC09:19
*** gkadam is now known as gkadam-afk09:20
*** jprovazn has quit IRC09:23
*** LanceHaig has joined #heat09:24
*** Shunli has quit IRC09:32
*** zhurong has quit IRC09:40
*** liusheng has quit IRC09:42
*** liusheng has joined #heat09:42
*** jprovazn has joined #heat09:44
*** mdnadeem has quit IRC09:46
*** ramishra has quit IRC09:49
*** prateek has quit IRC09:49
*** prateek has joined #heat09:50
*** ramishra has joined #heat09:50
*** fragatina has joined #heat09:55
*** ayoung has quit IRC09:56
*** ayoung has joined #heat10:08
*** namnh has quit IRC10:25
*** armaan has joined #heat10:32
*** sai has quit IRC10:41
*** rook has quit IRC10:42
*** jcoufal has joined #heat10:46
*** p01nt3r75 has joined #heat10:49
*** ramishra has quit IRC10:51
*** ramishra has joined #heat10:52
*** ramishra has quit IRC10:57
*** ramishra has joined #heat10:59
*** ltomasbo|away is now known as ltomasbo11:04
*** ioggstream has joined #heat11:06
*** sai has joined #heat11:08
*** rook has joined #heat11:08
*** rook is now known as Guest1875211:08
*** ccard has quit IRC11:09
*** ioggstream has quit IRC11:20
*** jaosorior has quit IRC11:29
*** EricGonczer_ has joined #heat11:36
*** cmyster has joined #heat11:38
*** gkadam-afk is now known as gkadam11:45
*** LanceHaig has quit IRC12:03
openstackgerritBence Romsics proposed openstack/heat master: New resource: Neutron Trunk  https://review.openstack.org/44249612:07
*** LanceHaig has joined #heat12:16
*** chlong has quit IRC12:20
*** ltomasbo is now known as ltomasbo|away12:26
*** prateek has quit IRC12:30
*** dprince has joined #heat12:33
*** ramishra has quit IRC12:35
therveshardy, If you can have a look at https://review.openstack.org/461904 that'd be great12:36
*** ramishra has joined #heat12:37
shardytherve: ack will do12:41
*** amoralej is now known as amoralej|lunch12:42
*** links has quit IRC12:46
*** gmoro has quit IRC12:50
*** catintheroof has joined #heat12:52
*** armaan has quit IRC12:56
*** armaan has joined #heat12:56
*** ramishra has quit IRC13:06
*** yassine has joined #heat13:08
*** gomarivera has joined #heat13:14
*** gomarivera_ has joined #heat13:15
*** gomarivera has quit IRC13:15
*** gkadam has quit IRC13:17
*** openstackgerrit has quit IRC13:18
*** EricGonczer_ has quit IRC13:23
*** jeremyfreudberg has joined #heat13:24
*** amoralej|lunch is now known as amoralej13:29
jeremyfreudberghey heat team, I'm wondering, if we have long-lived stack with a resource type which no longer exists, how is stack deletion handled?13:33
jeremyfreudbergfor example, OS::Nova::FloatingIPAssociation will be removed in Q release, and already right now deleting that resource is broken in Pike due to novaclient change13:33
jeremyfreudberg(Sahara used that resource for M through O releases, we're worried about long-lived Sahara-managed stacks not being able to be deleted)13:34
ricolinjeremyfreudberg, do you hope to keep the ability to kill the origin resource in Nova or just to be able to delete the template resource will be fine?13:35
*** cmyster has quit IRC13:36
jeremyfreudbergricolin, good question, at the very least we want to be able to delete the stack, orphaned resources are not ideal, but we could manage if we have to13:37
*** skramaja has quit IRC13:37
*** gmoro has joined #heat13:37
*** cmyster has joined #heat13:37
*** cmyster has joined #heat13:37
jeremyfreudberg(I was thinking we could change the handle_delete of OS::Nova::FloatingIPAssociation to attempt a delete using Neutron client, not sure if you guys are cool with that)13:37
thervejeremyfreudberg, https://review.openstack.org/#/c/466187/13:37
therveWe had to move quickly on that one because our gate was broken13:38
jeremyfreudbergtherve, awesome13:38
thervejeremyfreudberg, In theory we never want to remove resources. We won't remove OS::Nova::FloatingIPAssociation, we'll make it an alias to nothing if necessary13:39
therveIf a stack can't be deleted, that's a bug we have to fix13:39
therveWe may leave some stuff behind, especially if the underlying APIs are deprecated13:40
jeremyfreudbergtherve, great, my worries are settled13:42
*** zhurong has joined #heat13:42
jeremyfreudbergI will say though, it looks like that patch might run into some issues, it's still using some novaclient calls that are pending removal very soon, see https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L5513:44
*** openstackgerrit has joined #heat13:46
openstackgerritKanagaraj Manickam proposed openstack/heat master: Add tag for server nic  https://review.openstack.org/47374413:46
thervejeremyfreudberg, Well those are in master too13:47
jeremyfreudbergtherve, yes, I'm just saying you'll be needing another followup patch pretty soon13:48
therveThose just got merged, so we have some time13:48
therveAlso, they actually display warnings instead of being in docstrings that we don't read :)13:48
ricolinjeremyfreudberg, yep, you can backport that patch to your env, or use another resource type like OS::Heat::None to at last to remove stack. And as therve said, if that patch can't help to your env, then report a bug:)13:49
jeremyfreudbergricolin, therve, thanks. glad to see everything is under control on your end, and looks like the patch should be fine for Sahara's needs. Thanks!!13:50
ricolintherve, rabi is not around this week?13:56
*** jeremyfreudberg has left #heat13:57
thervericolin, Doesn't look like it13:58
*** chlong has joined #heat14:02
*** prateek has joined #heat14:03
*** bnemec has joined #heat14:05
*** chlong has quit IRC14:06
*** masber has quit IRC14:06
*** masber has joined #heat14:07
*** zz_dimtruck is now known as dimtruck14:11
*** prateek has quit IRC14:12
*** ccamacho is now known as ccamacho|brb14:17
*** links has joined #heat14:20
*** gomarivera_ has quit IRC14:22
openstackgerritZane Bitter proposed openstack/heat master: Fix race in SoftwareDeploymentGroupTest  https://review.openstack.org/47400414:27
*** jaosorior has joined #heat14:28
*** felipemonteiro has joined #heat14:30
*** chlong has joined #heat14:34
*** chlong has quit IRC14:35
*** ramishra has joined #heat14:36
ricolinzaneb, could you help to review https://review.openstack.org/#/c/328613/ when you have time:)14:36
*** links has quit IRC14:36
zanebsure thing14:36
*** chlong has joined #heat14:38
ricolinHi team, meeting in 14mins, feel free to add agenda14:46
ricolinhttps://wiki.openstack.org/wiki/Meetings/HeatAgenda#Agenda_.282017-06-14_1500_UTC.2914:46
*** zhurong has quit IRC14:49
openstackgerritMerged openstack/heat master: Fix nova floatingip resources  https://review.openstack.org/46618714:52
ricolinLanceHaig, kazsh, gaborm, mrwolf, zaneb,  ramishra, therve, shardy, skraynev, prazumovsky, elynn, spzala, tiantian Meeting time! at openstack-meeting-515:01
*** EricGonczer_ has joined #heat15:06
*** fultonj has quit IRC15:09
*** EricGonc_ has joined #heat15:15
*** EricGonczer_ has quit IRC15:17
*** ioggstream has joined #heat15:40
*** fragatina has quit IRC15:56
*** fragatina has joined #heat15:56
zanebtiantian: plugins expect self.t = before while handle_update is being called, but it may be correct to always set it to after once it's done, even if there was an exception16:06
zanebalthough tbh I don't know that we can ever get a consistent picture of a resource's state after cancelling an operation halfway through16:07
zanebthere's no way for the rest of Heat to know what the plugin has and has not done16:08
zaneb(convergence phase 2 would address this, using get_reality)16:08
tiantianzaneb, can we control to accept cancel out of one resource updating?16:09
tiantiannot at the middle point of a resource updating?16:09
zanebthat's actually something that convergence_engine solves. IIRC it will only cancel between calls to handle_update() and/or check_update_complete()16:11
*** p01nt3r75 has quit IRC16:11
zanebbut in legacy we still just kill the thread16:11
zanebso maybe tag the bug fixed-by-convergence and carry on with our lives? ;)16:12
tiantianso this problem wont happen in convergence? I didn't test for it16:12
zanebyeah, I suspect it won't happen in convergence16:12
tiantianI will try later in convergence16:13
tiantianbut you know our product works in legacy, so we still want to fix it in legacy16:14
zanebunderstood16:14
zanebthat may be fixable in legacy, and the code we implemented for convergence might even help, but it would still be hard work16:16
tiantianagree16:17
tiantianhow about to update self.t if we catch cancel exception?16:19
zanebI would at least try that. it will solve some situations16:19
tiantianok, will try tomorrow, thanks for the idea:)16:20
zanebany time :)16:21
zanebramishra: sigh https://review.openstack.org/#/c/474004/2 :/16:23
*** jmlowe has quit IRC16:23
*** masber has quit IRC16:27
*** masber has joined #heat16:28
*** ricolin has quit IRC16:29
*** gomarivera has joined #heat16:36
zanebramishra: on the bright side I think I've found the cause of the timeout race condition16:39
*** annegentle has joined #heat16:42
*** jmlowe has joined #heat16:46
*** rcernin has quit IRC16:49
*** catintheroof has quit IRC17:02
*** catintheroof has joined #heat17:03
openstackgerritZane Bitter proposed openstack/heat master: Fix races in SoftwareDeploymentGroupTest  https://review.openstack.org/47400417:03
*** derekh has quit IRC17:04
*** LanceHaig has quit IRC17:08
*** LanceHaig has joined #heat17:09
*** LanceHaig has quit IRC17:09
*** EricGonc_ has quit IRC17:12
*** yassine has quit IRC17:20
*** armaan has quit IRC17:23
*** shardy has quit IRC17:24
*** jcoufal has quit IRC17:25
*** yassine has joined #heat17:28
*** annegentle has quit IRC17:32
*** annegentle has joined #heat17:33
*** fragatina has quit IRC17:58
*** gmoro has quit IRC17:59
*** nicolasbock has quit IRC18:04
*** nicolasbock has joined #heat18:05
*** amoralej is now known as amoralej|off18:09
*** annegentle has quit IRC18:16
*** annegentle has joined #heat18:20
*** ioggstream has quit IRC18:20
*** ayoung has quit IRC18:30
*** hjensas has quit IRC18:34
*** rcernin has joined #heat19:10
*** armaan has joined #heat19:13
*** nicolasbock has quit IRC19:22
*** jose-phillips has quit IRC19:22
*** hjensas has joined #heat19:22
*** annegentle has quit IRC19:27
*** Guest18752 is now known as rook19:39
*** gmoro has joined #heat19:56
*** jprovazn has quit IRC19:58
*** annegentle has joined #heat20:00
*** ioggstream has joined #heat20:02
*** gmoro has quit IRC20:04
*** dprince has quit IRC20:07
*** jcoufal has joined #heat20:18
thervezaneb, Not all resources are in progress when you do a rolling update :)20:19
zanebugh, dammit20:20
zanebso it was actually relying on the fact that that data was grabbed during the create to show them all as IN_PROGRESS20:21
*** ioggstream has quit IRC20:22
therveYou mean the ridiculous second assertEqual(4)20:24
zanebI still have no idea what the second assertEqual(4) was about :D20:26
therveOh yeah there is a check_input_values call as well, fantastic :)20:27
therveI'm happy because you +2 that patch but I didn't :D20:28
therveOh actually the check_input_values is ok, it only needs the physical_resource_id20:29
zaneblol20:29
zanebI was more interested in the thing it was testing at that point in time20:33
openstackgerritZane Bitter proposed openstack/heat master: Fix races in SoftwareDeploymentGroupTest  https://review.openstack.org/47400420:34
zanebtherve: does that^ look right this time?20:34
thervezaneb, Yep!20:36
therveNot sure we really need created_group_resources20:36
zanebwhew. thanks!20:36
zanebwe need two copies somehow - one where they're all IN_PROGRESS and one where they all have physical_resource_ids20:37
therveI thought the ids were there once in progress20:38
zanebtbh I'm not sure when they get added20:39
zanebfor most resources only when they're CREATE_COMPLETE, but sw deployments must be different20:39
*** armaan has quit IRC20:40
zanebif it's the same time though then we wouldn't have had two failure modes (timeout and NotFound)20:40
therveYeah, the IDs are passed as argument to remove a race...20:40
therve(I did that)20:40
zaneboh, so the failure mode just changed, we didn't have two failure modes at the same time?20:42
thervehttps://bugs.launchpad.net/heat/+bug/153244520:43
openstackLaunchpad bug 1532445 in heat "ParallelDeploymentsTest intermittent failure" [Medium,Fix released] - Assigned to Thomas Herve (therve)20:43
*** armaan has joined #heat20:46
zanebtherve: I had no idea that resource_id_set made a database call20:47
zanebAuthor date 6/07/12 ... oh.20:49
*** dimtruck is now known as zz_dimtruck20:49
*** armaan has quit IRC20:50
therveHeh20:51
zanebat least I didn't +2 it20:51
*** armaan has joined #heat20:53
zanebtherve: in any event, there's a short interval between the DB call that sets the state to CREATE_IN_PROGRESS and the one that sets the resource_id20:54
*** jmlowe has quit IRC21:05
*** ayoung has joined #heat21:12
*** zz_dimtruck is now known as dimtruck21:15
*** ramishra has quit IRC21:19
*** felipemonteiro has quit IRC21:21
*** fragatina has joined #heat21:27
*** hoonetorg has quit IRC21:34
*** gomarivera has quit IRC21:34
*** jmlowe has joined #heat21:40
*** jcoufal_ has joined #heat21:42
*** jcoufal has quit IRC21:43
*** fragatina has quit IRC21:53
*** jcoufal has joined #heat21:55
*** fragatina has joined #heat21:56
*** jcoufal_ has quit IRC21:57
openstackgerritHongbin Lu proposed openstack/heat master: Add Container to Zun resources  https://review.openstack.org/43781022:00
*** fragatina has quit IRC22:04
*** ayoung has quit IRC22:09
zanebyaaaaaay22:11
zanebit passed this time22:11
*** ayoung has joined #heat22:21
*** jcoufal has quit IRC22:26
*** annegentle has quit IRC22:27
*** chlong has quit IRC22:37
*** ayoung has quit IRC22:45
*** bnemec has quit IRC22:59
*** catintheroof has quit IRC23:09
*** armaan has quit IRC23:25
*** armaan has joined #heat23:25
openstackgerritZane Bitter proposed openstack/heat master: Support tenacity exponential backoff retry on resource sync  https://review.openstack.org/32861323:33
« Tuesday, 2017-06-13 Index Thursday, 2017-06-15 »

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