Wednesday, 2018-06-27

« Tuesday, 2018-06-26 Index Thursday, 2018-06-28 »
*** neatherweb has quit IRC00:05
openstackgerritNakul Dahiwade proposed openstack/heat master: Enhancements to CIDR and IP address constraints.  https://review.openstack.org/57526500:34
*** lifeless has quit IRC00:38
*** ndahiwade has quit IRC00:40
*** lifeless has joined #heat00:45
*** ricolin has joined #heat00:49
openstackgerritZane Bitter proposed openstack/heat master: Purge events that cross-reference with backup stack  https://review.openstack.org/57789100:59
openstackgerritZane Bitter proposed openstack/heat master: DNM: Test purging events  https://review.openstack.org/57743200:59
*** hongbin has joined #heat01:28
*** ramishra has joined #heat01:38
*** mdnadeem has joined #heat01:43
*** jmlowe_ has joined #heat02:09
*** jmlowe has quit IRC02:10
*** ricolin has quit IRC02:40
*** ricolin has joined #heat02:41
openstackgerritZane Bitter proposed openstack/heat master: DNM: Test purging events  https://review.openstack.org/57743202:42
openstackgerritZane Bitter proposed openstack/heat master: Don't re-use resource-properties-data in backup stacks  https://review.openstack.org/57825302:42
openstackgerritZane Bitter proposed openstack/heat master: Rename poorly named function  https://review.openstack.org/57825402:45
openstackgerritZane Bitter proposed openstack/heat master: Fix an exception message  https://review.openstack.org/57825502:45
*** hongbin has quit IRC03:11
*** hongbin has joined #heat03:37
*** ysandeep has joined #heat04:02
*** neatherweb has joined #heat04:03
openstackgerritVu Cong Tuan proposed openstack/heat-dashboard master: Add py36 testenv  https://review.openstack.org/57826604:06
*** AlexeyAbashkin has joined #heat04:20
*** hongbin has quit IRC04:24
*** skramaja has joined #heat04:34
*** AlexeyAbashkin has quit IRC04:49
*** amrith has quit IRC05:15
*** yangyape_ has quit IRC05:35
*** amrith has joined #heat05:47
*** amrith is now known as Guest211005:47
*** Guest35635 has quit IRC06:18
*** ricolin has quit IRC06:20
*** namnh has joined #heat06:28
*** Guest35635 has joined #heat06:29
*** dmellado has joined #heat06:29
*** jcoufal has joined #heat06:32
*** neatherweb has quit IRC06:33
*** yangyapeng has joined #heat06:33
*** AlexeyAbashkin has joined #heat06:35
*** jcoufal_ has joined #heat06:36
*** AlexeyAbashkin has quit IRC06:37
*** ysandeep has quit IRC06:38
*** jcoufal has quit IRC06:39
*** nicolasbock has joined #heat06:40
*** dyasny has quit IRC06:50
*** yangyapeng has quit IRC06:50
*** dyasny has joined #heat06:50
*** yangyapeng has joined #heat06:51
*** yangyapeng has quit IRC06:51
*** yangyapeng has joined #heat06:51
*** hjensas|afk has quit IRC07:02
*** peereb has joined #heat07:03
*** peereb has quit IRC07:04
*** peereb has joined #heat07:04
*** peereb has quit IRC07:05
*** peereb has joined #heat07:06
*** shardy has quit IRC07:28
*** shardy has joined #heat07:31
*** ysandeep has joined #heat07:36
*** rcernin has quit IRC07:51
*** rcernin has joined #heat07:53
*** serlex has joined #heat08:12
*** neatherweb has joined #heat08:22
*** jcoufal_ has quit IRC08:32
openstackgerritXinni Ge proposed openstack/heat-dashboard master: Replace embedded static files with XStatic-*  https://review.openstack.org/55128808:32
*** ricolin has joined #heat08:43
*** rcernin has quit IRC08:47
*** shardy has quit IRC08:48
*** derekh has joined #heat08:52
*** shardy has joined #heat08:52
*** d0ugal has quit IRC08:54
*** gfidente has joined #heat09:03
*** gfidente has quit IRC09:03
*** gfidente has joined #heat09:03
*** neatherweb has quit IRC09:14
*** hjensas|afk has joined #heat09:15
*** yangyapeng has quit IRC09:58
*** yangyapeng has joined #heat09:59
*** mdnadeem has quit IRC10:03
*** yangyapeng has quit IRC10:04
*** jcoufal has joined #heat10:05
*** gkadam__ has quit IRC10:16
*** gkadam__ has joined #heat10:16
*** namnh has quit IRC10:40
*** shardy has quit IRC10:44
*** neatherweb has joined #heat10:47
*** neatherweb has quit IRC11:09
*** neatherweb has joined #heat11:27
openstackgerritRico Lin proposed openstack/heat-specs master: Add multiple OpenStack cloud support  https://review.openstack.org/57836312:01
*** jcoufal has quit IRC12:11
*** shardy has joined #heat12:17
*** crose has joined #heat12:27
ltomasboping zaneb12:32
ltomasbozaneb, it is normal that if we deleted some of the resources (lbaas) the heat stack cannot be removed due to not being able to find those? http://paste.openstack.org/show/724395/12:33
ltomasbozaneb, that usually does not happen for other type of resources (like ports, subnets, instances...)12:34
ltomasbobnemec, ^^12:34
*** crose has quit IRC12:48
*** Guest35635 has quit IRC12:48
ramishraltomasbo: you're probably hitting https://storyboard.openstack.org/#!/story/174783612:50
*** jcoufal has joined #heat12:50
*** Guest35635 has joined #heat12:55
*** edmondsw has joined #heat12:57
*** edmondsw has quit IRC12:57
*** edmondsw has joined #heat12:57
*** shardy has quit IRC13:14
ltomasbothanks ramishra!13:16
*** dyasny has quit IRC13:20
*** dyasny has joined #heat13:22
*** vegarl has quit IRC13:24
*** vegarl has joined #heat13:26
*** priteau has joined #heat13:41
dmelladoltomasbo: you here as well? xD13:50
ltomasbodmellado, :D13:50
*** skramaja has quit IRC13:50
openstackgerritRico Lin proposed openstack/heat master: Add Stack Credential DB  https://review.openstack.org/57839013:54
openstackgerritRico Lin proposed openstack/heat master: Add Stack Credential Object  https://review.openstack.org/57839113:54
openstackgerritRico Lin proposed openstack/heat master: Support check replace stack credential  https://review.openstack.org/57839213:54
openstackgerritRico Lin proposed openstack/heat master: Support remote stack from another OpenStack provider  https://review.openstack.org/57839313:54
openstackgerritRico Lin proposed openstack/heat master: Add document for Multi OpenStack Cloud support  https://review.openstack.org/57839413:54
*** shardy has joined #heat13:59
ricolin#startmeeting heat14:01
openstackMeeting started Wed Jun 27 14:01:13 2018 UTC and is due to finish in 60 minutes.  The chair is ricolin. Information about MeetBot at http://wiki.debian.org/MeetBot.14:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:01
*** openstack changes topic to " (Meeting topic: heat)"14:01
openstackThe meeting name has been set to 'heat'14:01
ramishrahi14:01
ricolinramishra, o/14:01
zanebo/14:02
ricolin#topic adding items to agenda14:02
*** openstack changes topic to "adding items to agenda (Meeting topic: heat)"14:02
ricolin#link https://wiki.openstack.org/wiki/Meetings/HeatAgenda#Agenda_.282018-06-27_1400_UTC.2914:02
*** hjensas|afk has quit IRC14:02
ricolinAny topic you guys like to discuss?14:03
kazsho/14:03
ricolinkazsh, hi14:03
ricolin#topic Multi-Cloud https://review.openstack.org/#/q/topic:bp/multiple-cloud-support+(status:open+OR+status:merged)14:05
*** openstack changes topic to "Multi-Cloud https://review.openstack.org/#/q/topic:bp/multiple-cloud-support+(status:open+OR+status:merged) (Meeting topic: heat)"14:05
ricolinramishra, zaneb I upload the multicloud works earlier14:05
ricolinalso the spec too14:06
ramishraricolin: will take a look14:06
ricolinstill need to add unit test and functional test14:06
ricolinbut would be great if guys can talk a look14:06
ricolinramishra, thx14:06
ricolintricky to parse around credential:)14:07
ramishrazaneb:  I was wondering if we should merge https://review.openstack.org/#/c/480923  or wait for the multicloud thing to be implemened with ssl options in barbican?14:08
zanebadded it to my list14:08
zanebramishra: yeah, that's a good question14:08
zanebI think we need to figure out an interface that works for both14:08
ramishraprobably we can land it, been in review for long time though14:09
ricolindon't see much conflict on both, but I can depend on ssl one since that will be more secure14:09
ramishranot sure if we would complete the multicloud thing this release, though zaneb can surely surprise us:)14:10
zanebramishra: clearly you have not seen my to-do list ;)14:11
ricolinramishra, I would like to see if we can really make it happen:)14:11
ricolinzaneb, and where is it:)14:11
zanebit would be cool if we can do it. that's definitely the thing that is going to generate the most excitement from outside the project14:12
ricolinzaneb, API Broker too IMO14:12
zanebthat's a separate project though14:12
zaneband also something on my to-do list14:13
ricolinzaneb, No, I mean API Broker resource in heat(Not sure if we want to do it, but it;s cool:))14:13
zaneboh, right14:13
zanebyeah, that's also cool and definitely not happening in this release ;)14:13
ricolinI'm going to work on Vitrage resource later once I put tests for multicloud on:)14:14
zanebricolin: btw I just re-reviewed https://review.openstack.org/541558 (that ltomasbo was asking about earlier)14:15
ricolinzaneb, okay will update it right after meeting14:15
zanebricolin: the main thing it needs is a test14:16
ramishrazaneb, ricolin We've similar issue with L7Rule resource. may be that can be fixed too, separate patch would be fine too.14:16
ricolinzaneb, I can't remember why I didn't add unit test, but will recheck again14:16
ramishraI was about to comment on the review and then forgot14:16
ricolinWhat else needs to discuss?14:18
ramishraI've something to discuss..14:18
ramishrahttps://review.openstack.org/#/c/551871/6/heat/common/config.py, I've added a new config option14:19
ramishratherve thinks we should use https://github.com/openstack/heat/blob/master/heat/common/wsgi.py#L211 for it14:20
*** flwang has quit IRC14:20
ramishraIMO, it can be confusing as it's used wrt heat vs swift download limit, but would mean the samething in the context of what we're doing14:21
ramishraWanted to know what others think14:21
zanebI guess the advantage of therve's approach would be that the limit is the same no matter how you use Heat14:22
*** shardy has quit IRC14:22
zanebso you couldn't e.g. end up with something that was working in Swift and then you download the templates locally and try it from there and suddenly it breaks14:23
ramishrayeah, it's for different services and swift does not have any download limit per se14:23
ricolinramishra, so why can't we set unlimit default?14:23
thervericolin: We load the files in memory14:24
zanebricolin: because we don't want to load infinity bytes into Heat's RAM14:24
ramishrazaneb: ok, that probably is a good reason14:24
ramishraI mean trying from locally that works when they are in swift14:24
zanebramishra: yeah, at the very least that's a good reason to set the defaults to the same thing14:25
zaneband arguably they could just use the same config option14:25
ramishraI thought it can be confusing when someone tries to see that error and then look for the config14:25
ramishrait may be intuitive for the devs but not for the users14:26
ramishraI mean operators14:27
ramishraBut I'm ok either way14:27
zanebthe option is not in any group, right?14:28
ricolinI think it's fine to use max_json_body_size since that's the buggest size we gonna accept anyway14:28
ramishrayep14:29
ricolinzaneb, in `Default` I think14:29
zanebI think it's fine to reuse that one then14:29
ricolinwe can also set overwrite conf if we like14:29
ricolinlike when swift_objects_download_limit > max_json_body_size we reset swift_objects_download_limit14:30
zanebricolin: let's not though ;)14:31
ramishraOK, but I still think from user point of view it would be confusing when they see an error  the stack action failed due to that limit when downloading from swift, but anyway I'll change14:31
*** flwang has joined #heat14:32
ricolinramishra, can we get size before we download it?14:32
ramishraricolin: yes, check that patch:)14:32
*** crose has joined #heat14:33
ricolinramishra, then user won't fail during downloading right?:)14:33
zanebtechnically its still the body regardless of whether we're uploading or downloading ;)14:33
ricolinthey fail before it:)14:33
*** felipemonteiro has joined #heat14:33
zanebricolin: ramishra's point is that it'll fail when sending in a very small request, complaining that the request size is too big14:34
*** felipemonteiro_ has joined #heat14:34
zanebI think we'll need to make sure there's different error messages14:35
zanebbut I don't see an obstacle to using the same config option to set the limit14:35
ricolinzaneb, yeah we need good message to show that14:35
*** felipemonteiro_ has quit IRC14:37
ricolinAnything else?:)14:37
*** felipemonteiro_ has joined #heat14:37
ramishraOK we can add another exception type rather than ta config, not use RequestLimitExceeded:)14:38
zaneb+114:38
ricolinramishra, +1 on that!14:38
*** felipemonteiro has quit IRC14:38
ricolinOkay, if nothing to put on, we can close meeting now14:41
ricolinOkay14:43
ricolinplease please please help to review https://review.openstack.org/#/c/578363/ :)14:43
ricolinmulticloud spec14:43
zanebricolin: btw question for you: http://eavesdrop.openstack.org/irclogs/%23heat/%23heat.2018-06-26.log.html#t2018-06-26T13:23:1114:43
ricolinzaneb, you hit that in Master?14:46
ricolinWe did have some period to keep pop those error message out to log during very first introduce policy in code to heat14:47
ricolinin a single milestone14:48
ramishrahttp://logs.openstack.org/53/578253/1/check/heat-functional-orig-mysql-lbaasv2/253251f/logs/screen-h-eng.txt.gz#_Jun_27_03_36_07_82755114:48
zanebricolin: yes, I'm seeing that in the gate14:48
zanebthe policy logs are too chatty, and that's easy enough to fix14:49
ricolin(keep loading logs...)14:49
ricolinzaneb, okay, will find it and take a look14:50
zanebbut the fact that we're also seeing "Fetching data from file:///etc/heat/templates/AWS_RDS_DBInstance.yaml" in there suggests something else is happening, like we're reloading the global environment all the time14:50
ricolinramishra, zaneb so those just information log right14:52
*** hongbin has joined #heat14:52
zanebin that link that ramishra posted it's still doing it 14 minutes after the test started. I'd expect to see the global env loaded only once per worker at startup14:53
ricolinzaneb, might be14:55
ramishraricolin: yeah, it seems to be checking the policy far too often, which probably means something is wrong14:55
zanebthat's my concern, yeah14:57
ricolinAnyway, let's keep trace on that14:59
ramishraricolin: we can probably raise a bug and do the investigation later, meeting time alomost over14:59
ricolinramishra, +114:59
zaneb+115:00
ricolinlet's end meeting now, if nothing to discuss before keep working15:00
ricolin#endmeeting15:01
*** openstack changes topic to "OpenStack Heat Team (logs: http://eavesdrop.openstack.org/irclogs/%23heat/)"15:01
zanebo/15:01
openstackMeeting ended Wed Jun 27 15:01:25 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:01
openstackMinutes:        http://eavesdrop.openstack.org/meetings/heat/2018/heat.2018-06-27-14.01.html15:01
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/heat/2018/heat.2018-06-27-14.01.txt15:01
openstackLog:            http://eavesdrop.openstack.org/meetings/heat/2018/heat.2018-06-27-14.01.log.html15:01
thervezaneb: I remember caching the policy before15:05
therveMaybe it got broken by the policy in code15:05
therveOr maybe I only cached some of it15:08
thervezaneb: It's the resource_types API15:13
therveIt calls Environment.get_types15:13
therveWhich creates a ResourceEnforcer15:13
therveWhich does that15:13
zanebah, ok15:14
zanebtherve: why is it loading templates from the global environment though?15:14
zanebthat seems like overkill for listing types15:15
zanebfor the ResourceEnforcer thing we can just get rid of the log messages (I have a patch for that somewhere I think)15:16
therveLet's see15:17
ricolinzaneb, yeah, those just INFO, so move it or remove it will be fine15:17
zanebricolin: ack, I'll clean up that patch and submit it15:17
thervezaneb: So it must be done by TemplateResource15:18
therveIn TemplateResourceInfo we call get_class which calls taht15:19
therveWe need get_class for support status, version and stuff15:20
therveNow, that should probably be cached somewhere15:20
*** jmlowe_ has quit IRC15:23
*** jcoufal has quit IRC15:25
*** gkadam__ has quit IRC15:26
*** gkadam__ has joined #heat15:26
*** edmondsw has quit IRC15:33
*** yangyapeng has joined #heat15:34
*** peereb has quit IRC15:35
*** yangyapeng has quit IRC15:38
*** pbourke has quit IRC15:48
*** Guest35635 has quit IRC15:49
*** pbourke has joined #heat15:50
*** Guest35635 has joined #heat15:58
*** ramishra has quit IRC16:08
*** edmondsw has joined #heat16:09
*** gfidente has quit IRC16:20
*** serlex has quit IRC16:23
*** dyasny has quit IRC16:24
*** dyasny has joined #heat16:24
*** ysandeep has quit IRC16:29
*** felipemonteiro__ has joined #heat16:36
*** gkadam__ has quit IRC16:37
*** jmlowe has joined #heat16:38
*** jmlowe has quit IRC16:39
*** jmlowe has joined #heat16:39
*** felipemonteiro_ has quit IRC16:40
*** edmondsw has quit IRC16:43
*** edmondsw has joined #heat16:44
*** edmondsw has quit IRC16:44
*** derekh has quit IRC16:51
*** dyasny has quit IRC17:07
*** dyasny has joined #heat17:07
*** felipemonteiro__ has quit IRC17:16
*** felipemonteiro_ has joined #heat17:16
*** yangyapeng has joined #heat17:24
*** yangyapeng has quit IRC17:28
zanebricolin: reviewed your spec17:44
*** ndahiwade has joined #heat17:47
openstackgerritRico Lin proposed openstack/heat master: Check NotFound with pool property in pool member  https://review.openstack.org/54155817:47
ricolinzaneb, ^^^17:48
zanebricolin: you should be asleep17:48
ricolinzaneb, I should!17:49
*** edmondsw has joined #heat18:01
*** edmondsw has quit IRC18:06
*** ndahiwade has quit IRC18:12
*** ndahiwade has joined #heat18:12
*** hjensas|afk has joined #heat18:17
openstackgerritRico Lin proposed openstack/heat-specs master: Add multiple OpenStack cloud support  https://review.openstack.org/57836318:23
openstackgerritNakul Dahiwade proposed openstack/heat master: Re-calculate outputs at the end of a stack-update.  https://review.openstack.org/57693618:23
ricolinzaneb, https://review.openstack.org/#/c/57836318:23
ricolinzaneb, Maybe we can also allow access cloud provider info from local barbican secret?18:24
*** ndahiwade has quit IRC18:32
*** ricolin has quit IRC18:39
*** ndahiwade has joined #heat18:45
*** felipemonteiro_ has quit IRC18:46
*** felipemonteiro_ has joined #heat18:49
*** ndahiwade has quit IRC19:01
*** jmlowe has quit IRC19:42
*** edmondsw has joined #heat19:50
*** edmondsw has quit IRC19:54
*** ayoung has joined #heat19:59
*** AlexeyAbashkin has joined #heat20:02
*** ndahiwade has joined #heat20:05
*** ndahiwade has quit IRC20:10
*** ndahiwade has joined #heat20:25
*** jmlowe has joined #heat20:29
*** AlexeyAbashkin has quit IRC20:45
*** AlexeyAbashkin has joined #heat21:01
*** jtomasek has quit IRC21:03
openstackgerritNakul Dahiwade proposed openstack/heat master: Translation validation of older properties fails  https://review.openstack.org/57856021:10
*** priteau has quit IRC21:10
*** AlexeyAbashkin has quit IRC21:12
openstackgerritNakul Dahiwade proposed openstack/heat master: Enhancements to CIDR and IP address constraints.  https://review.openstack.org/57526521:34
*** jtomasek has joined #heat21:36
*** felipemonteiro_ has quit IRC21:38
*** edmondsw has joined #heat21:38
*** edmondsw has quit IRC21:43
*** rcernin has joined #heat21:50
*** jmlowe has quit IRC21:56
*** neatherweb has quit IRC22:06
*** hongbin has quit IRC22:27
*** jmlowe has joined #heat22:28
*** nicolasbock has quit IRC22:29
*** edmondsw has joined #heat23:27
*** edmondsw has quit IRC23:32
openstackgerritNakul Dahiwade proposed openstack/heat master: delete_trust failure will not block a stack delete  https://review.openstack.org/57665923:42
« Tuesday, 2018-06-26 Index Thursday, 2018-06-28 »

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