Thursday, 2014-09-04

*** sjmc7_ has joined #murano00:06
*** ativelkov has quit IRC00:12
*** sjmc7 has quit IRC00:12
*** ativelkov has joined #murano00:13
*** gokrokve has joined #murano00:24
*** gokrokve_ has joined #murano00:43
*** gokrokve has quit IRC00:43
*** gokrokve_ has quit IRC00:50
*** peoplemerge has quit IRC00:51
*** gokrokve has joined #murano01:00
*** Murali_ has joined #murano01:47
*** Murali_ has quit IRC03:55
*** Murali_ has joined #murano03:56
*** Murali__ has joined #murano04:42
*** Murali_ has quit IRC04:42
*** Murali__ has quit IRC05:27
*** Murali_ has joined #murano05:32
*** killer_prince is now known as lazy_prince05:34
*** lazy_prince is now known as killer_prince06:14
*** Murali_ has quit IRC06:17
*** killer_prince is now known as lazy_prince06:17
*** romainh has joined #murano06:51
openstackgerritEkaterina Chernova proposed a change to stackforge/murano-dashboard: Update horizon settings  https://review.openstack.org/11899007:30
openstackgerritEkaterina Chernova proposed a change to stackforge/murano-dashboard: Fix 'plus' button for creation nested apps  https://review.openstack.org/11899107:30
openstackgerritA change was merged to stackforge/murano-dashboard: Revert "Enhance versionutils.deprecated to work with classes"  https://review.openstack.org/11817507:46
sergmelikyansjmc7_, sorry :( was already away... btw, skyping may be a little faster to catch-up with me. I receive skype messages on my phone08:15
openstackgerritTimur Sufiev proposed a change to stackforge/murano-dashboard: Initial implementation of Actions UI  https://review.openstack.org/9936008:42
*** gokrokve_ has joined #murano09:04
openstackgerritDmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config  https://review.openstack.org/11748809:06
*** gokrokve has quit IRC09:07
*** gokrokve_ has quit IRC09:08
*** gokrokve has joined #murano09:46
*** gokrokve has quit IRC09:52
*** asalkeld has quit IRC10:23
*** k4n0_ has quit IRC10:33
*** k4n0 has joined #murano10:35
*** gokrokve has joined #murano10:46
*** gokrokve has quit IRC10:51
openstackgerritSerg Melikyan proposed a change to stackforge/murano: Fix race condition when two Instances are deployed  https://review.openstack.org/11857011:27
openstackgerritSerg Melikyan proposed a change to stackforge/murano: Remove partial clean-up in Instance.destroy  https://review.openstack.org/11817211:29
*** gokrokve has joined #murano11:46
*** gokrokve has quit IRC11:51
*** ddovbii_ has quit IRC11:51
openstackgerritTimur Sufiev proposed a change to stackforge/murano-dashboard: Fix topology page issue with deps and images  https://review.openstack.org/10406411:54
openstackgerritStan Lagun proposed a change to stackforge/murano: Use Keystone trusts to get fresh token  https://review.openstack.org/11904211:55
openstackgerritDmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config  https://review.openstack.org/11748812:38
*** gokrokve has joined #murano12:46
*** gokrokve has quit IRC12:51
openstackgerritDmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config  https://review.openstack.org/11748813:06
*** k4n0 has quit IRC13:12
openstackgerritStan Lagun proposed a change to stackforge/murano: Use Keystone trusts to get fresh token  https://review.openstack.org/11904213:13
*** Murali has joined #murano13:27
*** lazy_prince is now known as killer_prince13:28
openstackgerritDmitry Teselkin proposed a change to stackforge/murano: Restore auth_uri config option  https://review.openstack.org/11906213:29
*** Murali has quit IRC13:33
*** drupalmonkey has joined #murano13:39
*** DaveJ__ has joined #murano13:45
*** gokrokve has joined #murano13:46
sjmc7_hi sergmelikyan - i have a patch for https://bugs.launchpad.net/murano/+bug/1364446 that we'll probably use for our release13:47
openstackgerritA change was merged to stackforge/murano-dashboard: Update horizon settings  https://review.openstack.org/11899013:47
*** sjmc7_ is now known as sjmc713:47
*** Murali_ has joined #murano13:48
sjmc7our infrastructure folks don't want to enable the extension, but suggested heat shouldn't rely on it (though i don't think that will fly well)13:48
*** gokrokve has quit IRC13:51
*** Murali_ has quit IRC13:52
*** Murali_ has joined #murano13:56
*** killer_prince is now known as lazy_prince14:00
*** drupalmonkey_ has joined #murano14:05
openstackgerritSteve McLellan proposed a change to stackforge/murano: Don't rely on OS::Nova::Server.addresses  https://review.openstack.org/11907214:15
*** drupalmonkey has quit IRC14:17
*** igormarnat has quit IRC14:17
*** drupalmonkey_ is now known as drupalmonkey14:17
openstackgerritA change was merged to stackforge/murano: Move wsgi module to murano/common  https://review.openstack.org/11812214:17
openstackgerritA change was merged to stackforge/murano: Fix issue with Default for Runtime props in HeatSWConfigInstance  https://review.openstack.org/11860214:17
sergmelikyansjmc7, thank you! I was just starting to looking on that bug14:18
sjmc7i just submitted it. i'm not really sure whether we should necessarily fix it upstream, but the patch gives equivalent behavior14:18
sergmelikyanI think output will be same, so I think there is no reason not having it in upstream14:18
sjmc7if anything i guess it could be an upstream heat change, but i'm not sure they'd want to make it14:18
*** igormarnat has joined #murano14:19
openstackgerritA change was merged to stackforge/murano: Fix deployment failure detection  https://review.openstack.org/11858414:19
sergmelikyansjmc7, I have a small concern about that change though14:26
sjmc7ok14:26
sergmelikyanessentially you do networks.values().flatten()14:26
sergmelikyanbut networks is a list (according to specification)14:26
sjmc7networks is a dict14:26
sjmc7{netid: [ip, ip] }\14:26
sergmelikyanhttp://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Nova::Server14:27
sergmelikyanAn ordered list of nics to be added to this server, with information about connected networks, fixed ips, port etc.14:27
sjmc7look at the attributes14:27
sergmelikyannetworks : List14:27
sjmc7not the properties14:27
sergmelikyansjmc7, thx!14:27
sergmelikyanmea culpa14:28
*** romainh has left #murano14:28
sjmc7n.p.14:28
sergmelikyansjmc7, Brain is not yet here, maybe you know14:32
sergmelikyanhttps://bugs.launchpad.net/murano/+bug/1359883 - is this bug still reproduced on your env?14:32
*** gokrokve has joined #murano14:32
sjmc7i haven't seen it, no - it may have been around all the horizon CSS changes14:32
*** DaveJ__ has quit IRC14:36
*** DaveJ__ has joined #murano14:37
sergmelikyanI haven't seen it too, so I guess if Brain will confirm that this issue is not reproducing on your env, we can close this one14:42
sjmc7k. he should be back in an hour14:43
openstackgerritDmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config  https://review.openstack.org/11748814:43
sergmelikyanHuh... gates are  overloaded with amount of changes before J3 release :(14:44
sjmc7yeah, i bet14:44
ruhesergmelikyan: one correction :)14:44
ruheBraian not Brain :)14:44
sjmc7:)14:44
sergmelikyansorry :(14:44
sjmc7it's ok, he likes Brain too14:44
sergmelikyanruhe, thank you14:45
sergmelikyansjmc7, please check this issue https://review.openstack.org/11860214:45
sjmc7ah, yes. it already got merged, but it looks good, thanks :)14:46
sergmelikyanAfter Stan will return from vacation he will add support for Default values in Runtime properties14:46
sjmc7i think the documentation needs updating for Runtime though14:46
sjmc7ah, ok14:46
sjmc7yeah - it would be much nicer not to have a special case14:46
sjmc7thanks!14:46
sergmelikyanChange above is private and should not affect any existing apps relying on this class14:47
sergmelikyansjmc7, special case?14:47
sjmc7yeah. i'll merge it into our tree, i'm about to create a build14:47
sjmc7special case == Runtime not allowing defaults14:47
sergmelikyansjmc7, https://review.openstack.org/118584 - this one is pretty important too. Simple but it was breaking failure detection and appropriate deployment logs for env14:48
sjmc7ah, yes. we noticed that yesterday, hadn't debugged it. thanks14:49
sergmelikyansjmc7, and this one https://review.openstack.org/118172 is a workaround for issue with Instance.yaml described by you, later I will make proper fix that will clean-up SC14:51
sjmc7ok. yeah, we did something similar locally14:51
sergmelikyanAnd last change that I would like to be reviewed by you is this one - https://review.openstack.org/#/c/118570/14:52
sergmelikyanpretty rare, but breaking deployment14:52
sjmc7ah.. we'd seen some intermittent bugs around multiple resources14:53
sergmelikyanFix is also more a workaround than a proper fix, we were planning to revise HeatStack class, but had no time to do it in J timeframe, but it is in roadmap for K14:53
sjmc7isn't there still a race condition if updateTemplate is called twice at the same time?14:55
sergmelikyanYes, but since Murano is built on top of eventlet all is happening in one thread. Race condition can happen only when there is context switch before all updateTemplate call are finished and other thread is called push14:56
sjmc7ok. we'd also had a report of adding a component resulting in redeploy of existing ones which i think ryan filed14:57
sjmc7it might all be related14:57
sergmelikyanWhen deploy is called on Instances at the same time, one Instance calls push before another finished all calls to updateStack. This was happening because context was switched due to calls to Neutron in order to gather appropriate information14:58
sjmc7ah, ok14:59
sjmc7hate concurrency :(14:59
sergmelikyanredeploy of existing apps is a expected behavior though, you need to use Attributes to avoid it15:03
* sergmelikyan searching for a bug from Ryan15:05
sjmc7it doesn't seem like it should be expected behavior15:06
sergmelikyansjmc7, https://bugs.launchpad.net/murano/+bug/1362744 - there a good comment from Stan why it that and how to avoid it15:07
sjmc7ok. but our QA suggested it had created additional VMs15:07
sjmc7i haven't verified it, had bigger fish to fry15:08
sergmelikyan"our QA suggested it had created additional VMs" <-- and this is really strange, never have seen that. But anyway I will appreciate any clue and will at it15:10
sergmelikyan*look at it15:10
sjmc7yeah. we've found a bunch of environmental things outside murano that have been #1 priority15:10
sjmc7were supposed to start proper QA beginning this week but we're not there yet15:11
sergmelikyansjmc7, are there some issues that I can help with?15:12
sjmc7no, i don't think so right now. i'll have people get in touch if they need help. thanks for getting to these bugs so quickly15:13
sjmc7i was hoping not to be spending so much time on infrastructure stuff :(15:13
openstackgerritDmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config  https://review.openstack.org/11748815:26
*** drupalmonkey has quit IRC15:35
*** drupalmonkey has joined #murano15:35
sjmc7sergmelikyan - we may have an authentication issue with heat. i'll get some more details17:02
*** lazy_prince is now known as killer_prince17:06
sergmelikyanbtw, looks like we have working trusts commit. It needs more checking, but at least we have implemented trusts17:09
sjmc7sergmelikyan - it looks like i'm having a network issue17:10
sjmc7in our environment, new users don't have a router created for them17:10
sjmc7everyone needs a router, correct?17:10
sergmelikyansjmc7, as far as I know, I may be wrong. I'll look17:10
sjmc7murano creates a routerinterface, but not router or gateway, it looks like17:11
sjmc7without a router you end up with "'router_id': u'NOT_FOUND'" in the heat template17:12
sergmelikyanUnfortunately yes, router is not created by Murano17:13
sergmelikyanhttps://github.com/stackforge/murano/blob/master/murano/engine/system/net_explorer.py#L6617:13
sergmelikyanI think we can't create Router, because it needs to be plugging in to network by admin17:14
sjmc7ok. i'm still getting an error in heat - "WARNING heat.engine.service [req-5c46467f-4d11-43c8-9333-67824214b021 None] Access denied to resource vertica-vm-117:14
sjmc7"17:14
sjmc7i'll try and find out what's going on. i've been fighting authentication for a couple of days17:14
sergmelikyanhttps://github.com/openstack/heat/blob/master/heat/engine/service.py#L91517:17
sjmc7my mistake17:18
sjmc7logs were confusing, multiple deploys going on17:18
sjmc7the access denied thing is weird though17:18
sjmc7sergmelikyan - i think murano should be able to create a router17:21
sergmelikyansjmc7, i even didn't know that Heat can control access to the particular resources in stack17:25
sjmc7sure it can17:25
sjmc7there're neutron heat resources17:26
*** gokrokve has quit IRC17:26
sjmc7i'm not sure if we'd want to create lots of routers though17:26
sjmc7but it should fail if there is no router, in that case17:27
sjmc7not just put a NOT_FOUND in17:27
sergmelikyanI think we can make a patch that will fail deployment with appropriate message about no router available, and then try to make a patch that creates router for the user.17:28
sjmc7i don't know whether you're meant to have loads of routers17:29
sjmc7i don't know much about neutron17:29
sergmelikyanI am not sure why we didn't make it in first place, but I will find out tomorrow (I will ask ativelkov).17:29
sjmc7ok, thanks. but currently you make it a requirement that users have a router with external connectivity?17:29
sergmelikyanI believe so, otherwise services will not be able to be accessible from outside17:30
sergmelikyanEach router requires an IP address -  thats why you should try to have them as less as possible17:30
sergmelikyanideally no more than one per tenant17:30
sjmc7right17:31
sergmelikyanI remember only this impediment (with IP address) when we were working on Networking, but maybe there something more17:31
sjmc7yeah, i think lots of routers on the same ext-net can cause trouble17:31
sergmelikyanOne of the workarounds may be pre-create murano-router per tenant17:32
sjmc7:(17:32
sergmelikyanTo be sure that only correct one will be used by Murano (in case of many routers)17:33
sjmc7yeah; keep it outside of the heat stack and do it via neutron?17:33
sergmelikyansjmc7, I will try to make patch that creates router as soon as possible (if it's can be done)17:33
sjmc7rather than what we do now to just look up the first one we can find17:33
sergmelikyanI will work on this as first thing tomorrow17:33
sjmc7ok, thanks. if i find anything more then i'll let you know17:34
sergmelikyansjmc7, I think about extending this function https://github.com/stackforge/murano/blob/master/murano/engine/system/net_explorer.py#L6617:34
sjmc7yep17:34
sergmelikyanto create router with specific name (if it is not available) and use it17:35
sjmc7yep17:35
sergmelikyansjmc7, what do you think - should be make possible to use first found router without specific name if no router with specific name is available?17:35
sergmelikyanOr always have specific router for murano?17:36
sjmc7not sure. i'll file a bug and talk to some other people here17:36
sergmelikyanAnd also problem of choosing to which external network to attach will remain17:37
sjmc7yeah17:37
sergmelikyansjmc7, I will appreciate thoughts on this topic from some deployment guys17:37
*** gokrokve has joined #murano17:38
sergmelikyanQuestions from top of my head: 1)Should we attach to first found router not created specificly for murano or not? Should murano create router by itself? Should we use any external network available or only specify in config file?17:38
sjmc7i'll see who i can track down17:38
sergmelikyanAnd another questions is whose credentials use to create router, since is per-tenant17:39
sjmc7yeah, i don't know. there's a high risk of things going wrong17:39
sergmelikyansjmc7, I will try to talk with our guys, and thought from your side will be very usefull, cause it very sensitive questions in terms of cloud deployment17:40
sjmc7you must already have this issue in your deployments17:40
sjmc7is it possible to create floating IPs without?17:40
sergmelikyanI think we have no specific arrangements for routers in MOS, just documentation that says what should be available in order to Murano will able to work17:42
sjmc7yeah - we can just make it a documented pre-requisite17:42
sergmelikyansjmc7, without router? I am not sure. I know very little about networking17:42
sergmelikyansjmc7, since router is per tenant and tenant is setup by admins anyway - this can be managed17:42
sergmelikyanAnyway, I will make a patch that will fail deployment properly of no router is available, that start working on patch that creates router with specific-name (from config) and attach to a network (from config) under credentials (also from config). Later we will discover that this is a wrong way we will just drop last patch17:46
sjmc7ok. sory, in meetings. i'll create a bug in launchpad and attach any info i have17:47
sergmelikyansorry for so many gramar mistakes %) IRC does not give ability to fix them after you have sent message (like Skype)17:47
sjmc7:)  no worries17:47
*** Murali_ has quit IRC18:42
*** Murali__ has joined #murano18:42
*** Murali__ has quit IRC18:44
*** Murali_ has joined #murano18:46
*** Murali_ has quit IRC18:46
*** Murali_ has joined #murano18:46
*** DaveJ__ has quit IRC18:48
*** btully has joined #murano19:14
*** gokrokve has quit IRC21:00
*** gokrokve has joined #murano21:01
*** gokrokve has quit IRC21:04
*** gokrokve has joined #murano21:04
*** gokrokve has quit IRC21:04
*** gokrokve has joined #murano21:05
openstackgerritA change was merged to stackforge/murano-dashboard: Fix 'plus' button for creation nested apps  https://review.openstack.org/11899121:18
*** gokrokve has quit IRC21:21
*** asalkeld has joined #murano21:38
*** drupalmonkey has quit IRC21:55
*** drupalmonkey has joined #murano21:55
*** Murali_ has quit IRC21:58
openstackgerritStan Lagun proposed a change to stackforge/murano: Use Keystone trusts to get fresh token  https://review.openstack.org/11904221:58
*** gokrokve has joined #murano22:04
*** asalkeld has quit IRC22:09
*** sjmc7 has quit IRC22:16
*** gokrokve has quit IRC22:19
*** gokrokve has joined #murano22:19
*** gokrokve has quit IRC22:23
*** asalkeld has joined #murano22:26
*** drupalmonkey has quit IRC22:30
*** gokrokve has joined #murano22:37
*** gokrokve has quit IRC22:38
*** gokrokve has joined #murano22:39
*** bharath has joined #murano23:52
bharathHi23:52
bharathWhen I am adding second application to environment, the UI is displaying multiple times, does anyone faced this bug?23:53
bharathtsufiev,  ^^23:53
openstackgerritA change was merged to stackforge/murano: Restore auth_uri config option  https://review.openstack.org/11906223:58

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