*** sjmc7_ has joined #murano | 00:06 | |
*** ativelkov has quit IRC | 00:12 | |
*** sjmc7 has quit IRC | 00:12 | |
*** ativelkov has joined #murano | 00:13 | |
*** gokrokve has joined #murano | 00:24 | |
*** gokrokve_ has joined #murano | 00:43 | |
*** gokrokve has quit IRC | 00:43 | |
*** gokrokve_ has quit IRC | 00:50 | |
*** peoplemerge has quit IRC | 00:51 | |
*** gokrokve has joined #murano | 01:00 | |
*** Murali_ has joined #murano | 01:47 | |
*** Murali_ has quit IRC | 03:55 | |
*** Murali_ has joined #murano | 03:56 | |
*** Murali__ has joined #murano | 04:42 | |
*** Murali_ has quit IRC | 04:42 | |
*** Murali__ has quit IRC | 05:27 | |
*** Murali_ has joined #murano | 05:32 | |
*** killer_prince is now known as lazy_prince | 05:34 | |
*** lazy_prince is now known as killer_prince | 06:14 | |
*** Murali_ has quit IRC | 06:17 | |
*** killer_prince is now known as lazy_prince | 06:17 | |
*** romainh has joined #murano | 06:51 | |
openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Update horizon settings https://review.openstack.org/118990 | 07:30 |
---|---|---|
openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Fix 'plus' button for creation nested apps https://review.openstack.org/118991 | 07:30 |
openstackgerrit | A change was merged to stackforge/murano-dashboard: Revert "Enhance versionutils.deprecated to work with classes" https://review.openstack.org/118175 | 07:46 |
sergmelikyan | sjmc7_, sorry :( was already away... btw, skyping may be a little faster to catch-up with me. I receive skype messages on my phone | 08:15 |
openstackgerrit | Timur Sufiev proposed a change to stackforge/murano-dashboard: Initial implementation of Actions UI https://review.openstack.org/99360 | 08:42 |
*** gokrokve_ has joined #murano | 09:04 | |
openstackgerrit | Dmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config https://review.openstack.org/117488 | 09:06 |
*** gokrokve has quit IRC | 09:07 | |
*** gokrokve_ has quit IRC | 09:08 | |
*** gokrokve has joined #murano | 09:46 | |
*** gokrokve has quit IRC | 09:52 | |
*** asalkeld has quit IRC | 10:23 | |
*** k4n0_ has quit IRC | 10:33 | |
*** k4n0 has joined #murano | 10:35 | |
*** gokrokve has joined #murano | 10:46 | |
*** gokrokve has quit IRC | 10:51 | |
openstackgerrit | Serg Melikyan proposed a change to stackforge/murano: Fix race condition when two Instances are deployed https://review.openstack.org/118570 | 11:27 |
openstackgerrit | Serg Melikyan proposed a change to stackforge/murano: Remove partial clean-up in Instance.destroy https://review.openstack.org/118172 | 11:29 |
*** gokrokve has joined #murano | 11:46 | |
*** gokrokve has quit IRC | 11:51 | |
*** ddovbii_ has quit IRC | 11:51 | |
openstackgerrit | Timur Sufiev proposed a change to stackforge/murano-dashboard: Fix topology page issue with deps and images https://review.openstack.org/104064 | 11:54 |
openstackgerrit | Stan Lagun proposed a change to stackforge/murano: Use Keystone trusts to get fresh token https://review.openstack.org/119042 | 11:55 |
openstackgerrit | Dmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config https://review.openstack.org/117488 | 12:38 |
*** gokrokve has joined #murano | 12:46 | |
*** gokrokve has quit IRC | 12:51 | |
openstackgerrit | Dmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config https://review.openstack.org/117488 | 13:06 |
*** k4n0 has quit IRC | 13:12 | |
openstackgerrit | Stan Lagun proposed a change to stackforge/murano: Use Keystone trusts to get fresh token https://review.openstack.org/119042 | 13:13 |
*** Murali has joined #murano | 13:27 | |
*** lazy_prince is now known as killer_prince | 13:28 | |
openstackgerrit | Dmitry Teselkin proposed a change to stackforge/murano: Restore auth_uri config option https://review.openstack.org/119062 | 13:29 |
*** Murali has quit IRC | 13:33 | |
*** drupalmonkey has joined #murano | 13:39 | |
*** DaveJ__ has joined #murano | 13:45 | |
*** gokrokve has joined #murano | 13:46 | |
sjmc7_ | hi sergmelikyan - i have a patch for https://bugs.launchpad.net/murano/+bug/1364446 that we'll probably use for our release | 13:47 |
openstackgerrit | A change was merged to stackforge/murano-dashboard: Update horizon settings https://review.openstack.org/118990 | 13:47 |
*** sjmc7_ is now known as sjmc7 | 13:47 | |
*** Murali_ has joined #murano | 13:48 | |
sjmc7 | our 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 IRC | 13:51 | |
*** Murali_ has quit IRC | 13:52 | |
*** Murali_ has joined #murano | 13:56 | |
*** killer_prince is now known as lazy_prince | 14:00 | |
*** drupalmonkey_ has joined #murano | 14:05 | |
openstackgerrit | Steve McLellan proposed a change to stackforge/murano: Don't rely on OS::Nova::Server.addresses https://review.openstack.org/119072 | 14:15 |
*** drupalmonkey has quit IRC | 14:17 | |
*** igormarnat has quit IRC | 14:17 | |
*** drupalmonkey_ is now known as drupalmonkey | 14:17 | |
openstackgerrit | A change was merged to stackforge/murano: Move wsgi module to murano/common https://review.openstack.org/118122 | 14:17 |
openstackgerrit | A change was merged to stackforge/murano: Fix issue with Default for Runtime props in HeatSWConfigInstance https://review.openstack.org/118602 | 14:17 |
sergmelikyan | sjmc7, thank you! I was just starting to looking on that bug | 14:18 |
sjmc7 | i just submitted it. i'm not really sure whether we should necessarily fix it upstream, but the patch gives equivalent behavior | 14:18 |
sergmelikyan | I think output will be same, so I think there is no reason not having it in upstream | 14:18 |
sjmc7 | if anything i guess it could be an upstream heat change, but i'm not sure they'd want to make it | 14:18 |
*** igormarnat has joined #murano | 14:19 | |
openstackgerrit | A change was merged to stackforge/murano: Fix deployment failure detection https://review.openstack.org/118584 | 14:19 |
sergmelikyan | sjmc7, I have a small concern about that change though | 14:26 |
sjmc7 | ok | 14:26 |
sergmelikyan | essentially you do networks.values().flatten() | 14:26 |
sergmelikyan | but networks is a list (according to specification) | 14:26 |
sjmc7 | networks is a dict | 14:26 |
sjmc7 | {netid: [ip, ip] }\ | 14:26 |
sergmelikyan | http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Nova::Server | 14:27 |
sergmelikyan | An ordered list of nics to be added to this server, with information about connected networks, fixed ips, port etc. | 14:27 |
sjmc7 | look at the attributes | 14:27 |
sergmelikyan | networks : List | 14:27 |
sjmc7 | not the properties | 14:27 |
sergmelikyan | sjmc7, thx! | 14:27 |
sergmelikyan | mea culpa | 14:28 |
*** romainh has left #murano | 14:28 | |
sjmc7 | n.p. | 14:28 |
sergmelikyan | sjmc7, Brain is not yet here, maybe you know | 14:32 |
sergmelikyan | https://bugs.launchpad.net/murano/+bug/1359883 - is this bug still reproduced on your env? | 14:32 |
*** gokrokve has joined #murano | 14:32 | |
sjmc7 | i haven't seen it, no - it may have been around all the horizon CSS changes | 14:32 |
*** DaveJ__ has quit IRC | 14:36 | |
*** DaveJ__ has joined #murano | 14:37 | |
sergmelikyan | I 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 one | 14:42 |
sjmc7 | k. he should be back in an hour | 14:43 |
openstackgerrit | Dmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config https://review.openstack.org/117488 | 14:43 |
sergmelikyan | Huh... gates are overloaded with amount of changes before J3 release :( | 14:44 |
sjmc7 | yeah, i bet | 14:44 |
ruhe | sergmelikyan: one correction :) | 14:44 |
ruhe | Braian not Brain :) | 14:44 |
sjmc7 | :) | 14:44 |
sergmelikyan | sorry :( | 14:44 |
sjmc7 | it's ok, he likes Brain too | 14:44 |
sergmelikyan | ruhe, thank you | 14:45 |
sergmelikyan | sjmc7, please check this issue https://review.openstack.org/118602 | 14:45 |
sjmc7 | ah, yes. it already got merged, but it looks good, thanks :) | 14:46 |
sergmelikyan | After Stan will return from vacation he will add support for Default values in Runtime properties | 14:46 |
sjmc7 | i think the documentation needs updating for Runtime though | 14:46 |
sjmc7 | ah, ok | 14:46 |
sjmc7 | yeah - it would be much nicer not to have a special case | 14:46 |
sjmc7 | thanks! | 14:46 |
sergmelikyan | Change above is private and should not affect any existing apps relying on this class | 14:47 |
sergmelikyan | sjmc7, special case? | 14:47 |
sjmc7 | yeah. i'll merge it into our tree, i'm about to create a build | 14:47 |
sjmc7 | special case == Runtime not allowing defaults | 14:47 |
sergmelikyan | sjmc7, https://review.openstack.org/118584 - this one is pretty important too. Simple but it was breaking failure detection and appropriate deployment logs for env | 14:48 |
sjmc7 | ah, yes. we noticed that yesterday, hadn't debugged it. thanks | 14:49 |
sergmelikyan | sjmc7, 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 SC | 14:51 |
sjmc7 | ok. yeah, we did something similar locally | 14:51 |
sergmelikyan | And last change that I would like to be reviewed by you is this one - https://review.openstack.org/#/c/118570/ | 14:52 |
sergmelikyan | pretty rare, but breaking deployment | 14:52 |
sjmc7 | ah.. we'd seen some intermittent bugs around multiple resources | 14:53 |
sergmelikyan | Fix 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 K | 14:53 |
sjmc7 | isn't there still a race condition if updateTemplate is called twice at the same time? | 14:55 |
sergmelikyan | Yes, 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 push | 14:56 |
sjmc7 | ok. we'd also had a report of adding a component resulting in redeploy of existing ones which i think ryan filed | 14:57 |
sjmc7 | it might all be related | 14:57 |
sergmelikyan | When 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 information | 14:58 |
sjmc7 | ah, ok | 14:59 |
sjmc7 | hate concurrency :( | 14:59 |
sergmelikyan | redeploy of existing apps is a expected behavior though, you need to use Attributes to avoid it | 15:03 |
* sergmelikyan searching for a bug from Ryan | 15:05 | |
sjmc7 | it doesn't seem like it should be expected behavior | 15:06 |
sergmelikyan | sjmc7, https://bugs.launchpad.net/murano/+bug/1362744 - there a good comment from Stan why it that and how to avoid it | 15:07 |
sjmc7 | ok. but our QA suggested it had created additional VMs | 15:07 |
sjmc7 | i haven't verified it, had bigger fish to fry | 15: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 it | 15:10 |
sergmelikyan | *look at it | 15:10 |
sjmc7 | yeah. we've found a bunch of environmental things outside murano that have been #1 priority | 15:10 |
sjmc7 | were supposed to start proper QA beginning this week but we're not there yet | 15:11 |
sergmelikyan | sjmc7, are there some issues that I can help with? | 15:12 |
sjmc7 | no, 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 quickly | 15:13 |
sjmc7 | i was hoping not to be spending so much time on infrastructure stuff :( | 15:13 |
openstackgerrit | Dmitry Teselkin proposed a change to stackforge/murano-deployment: Update devstack config https://review.openstack.org/117488 | 15:26 |
*** drupalmonkey has quit IRC | 15:35 | |
*** drupalmonkey has joined #murano | 15:35 | |
sjmc7 | sergmelikyan - we may have an authentication issue with heat. i'll get some more details | 17:02 |
*** lazy_prince is now known as killer_prince | 17:06 | |
sergmelikyan | btw, looks like we have working trusts commit. It needs more checking, but at least we have implemented trusts | 17:09 |
sjmc7 | sergmelikyan - it looks like i'm having a network issue | 17:10 |
sjmc7 | in our environment, new users don't have a router created for them | 17:10 |
sjmc7 | everyone needs a router, correct? | 17:10 |
sergmelikyan | sjmc7, as far as I know, I may be wrong. I'll look | 17:10 |
sjmc7 | murano creates a routerinterface, but not router or gateway, it looks like | 17:11 |
sjmc7 | without a router you end up with "'router_id': u'NOT_FOUND'" in the heat template | 17:12 |
sergmelikyan | Unfortunately yes, router is not created by Murano | 17:13 |
sergmelikyan | https://github.com/stackforge/murano/blob/master/murano/engine/system/net_explorer.py#L66 | 17:13 |
sergmelikyan | I think we can't create Router, because it needs to be plugging in to network by admin | 17:14 |
sjmc7 | ok. 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-1 | 17:14 |
sjmc7 | " | 17:14 |
sjmc7 | i'll try and find out what's going on. i've been fighting authentication for a couple of days | 17:14 |
sergmelikyan | https://github.com/openstack/heat/blob/master/heat/engine/service.py#L915 | 17:17 |
sjmc7 | my mistake | 17:18 |
sjmc7 | logs were confusing, multiple deploys going on | 17:18 |
sjmc7 | the access denied thing is weird though | 17:18 |
sjmc7 | sergmelikyan - i think murano should be able to create a router | 17:21 |
sergmelikyan | sjmc7, i even didn't know that Heat can control access to the particular resources in stack | 17:25 |
sjmc7 | sure it can | 17:25 |
sjmc7 | there're neutron heat resources | 17:26 |
*** gokrokve has quit IRC | 17:26 | |
sjmc7 | i'm not sure if we'd want to create lots of routers though | 17:26 |
sjmc7 | but it should fail if there is no router, in that case | 17:27 |
sjmc7 | not just put a NOT_FOUND in | 17:27 |
sergmelikyan | I 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 |
sjmc7 | i don't know whether you're meant to have loads of routers | 17:29 |
sjmc7 | i don't know much about neutron | 17:29 |
sergmelikyan | I am not sure why we didn't make it in first place, but I will find out tomorrow (I will ask ativelkov). | 17:29 |
sjmc7 | ok, thanks. but currently you make it a requirement that users have a router with external connectivity? | 17:29 |
sergmelikyan | I believe so, otherwise services will not be able to be accessible from outside | 17:30 |
sergmelikyan | Each router requires an IP address - thats why you should try to have them as less as possible | 17:30 |
sergmelikyan | ideally no more than one per tenant | 17:30 |
sjmc7 | right | 17:31 |
sergmelikyan | I remember only this impediment (with IP address) when we were working on Networking, but maybe there something more | 17:31 |
sjmc7 | yeah, i think lots of routers on the same ext-net can cause trouble | 17:31 |
sergmelikyan | One of the workarounds may be pre-create murano-router per tenant | 17:32 |
sjmc7 | :( | 17:32 |
sergmelikyan | To be sure that only correct one will be used by Murano (in case of many routers) | 17:33 |
sjmc7 | yeah; keep it outside of the heat stack and do it via neutron? | 17:33 |
sergmelikyan | sjmc7, I will try to make patch that creates router as soon as possible (if it's can be done) | 17:33 |
sjmc7 | rather than what we do now to just look up the first one we can find | 17:33 |
sergmelikyan | I will work on this as first thing tomorrow | 17:33 |
sjmc7 | ok, thanks. if i find anything more then i'll let you know | 17:34 |
sergmelikyan | sjmc7, I think about extending this function https://github.com/stackforge/murano/blob/master/murano/engine/system/net_explorer.py#L66 | 17:34 |
sjmc7 | yep | 17:34 |
sergmelikyan | to create router with specific name (if it is not available) and use it | 17:35 |
sjmc7 | yep | 17:35 |
sergmelikyan | sjmc7, 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 |
sergmelikyan | Or always have specific router for murano? | 17:36 |
sjmc7 | not sure. i'll file a bug and talk to some other people here | 17:36 |
sergmelikyan | And also problem of choosing to which external network to attach will remain | 17:37 |
sjmc7 | yeah | 17:37 |
sergmelikyan | sjmc7, I will appreciate thoughts on this topic from some deployment guys | 17:37 |
*** gokrokve has joined #murano | 17:38 | |
sergmelikyan | Questions 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 |
sjmc7 | i'll see who i can track down | 17:38 |
sergmelikyan | And another questions is whose credentials use to create router, since is per-tenant | 17:39 |
sjmc7 | yeah, i don't know. there's a high risk of things going wrong | 17:39 |
sergmelikyan | sjmc7, 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 deployment | 17:40 |
sjmc7 | you must already have this issue in your deployments | 17:40 |
sjmc7 | is it possible to create floating IPs without? | 17:40 |
sergmelikyan | I 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 work | 17:42 |
sjmc7 | yeah - we can just make it a documented pre-requisite | 17:42 |
sergmelikyan | sjmc7, without router? I am not sure. I know very little about networking | 17:42 |
sergmelikyan | sjmc7, since router is per tenant and tenant is setup by admins anyway - this can be managed | 17:42 |
sergmelikyan | Anyway, 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 patch | 17:46 |
sjmc7 | ok. sory, in meetings. i'll create a bug in launchpad and attach any info i have | 17:47 |
sergmelikyan | sorry for so many gramar mistakes %) IRC does not give ability to fix them after you have sent message (like Skype) | 17:47 |
sjmc7 | :) no worries | 17:47 |
*** Murali_ has quit IRC | 18:42 | |
*** Murali__ has joined #murano | 18:42 | |
*** Murali__ has quit IRC | 18:44 | |
*** Murali_ has joined #murano | 18:46 | |
*** Murali_ has quit IRC | 18:46 | |
*** Murali_ has joined #murano | 18:46 | |
*** DaveJ__ has quit IRC | 18:48 | |
*** btully has joined #murano | 19:14 | |
*** gokrokve has quit IRC | 21:00 | |
*** gokrokve has joined #murano | 21:01 | |
*** gokrokve has quit IRC | 21:04 | |
*** gokrokve has joined #murano | 21:04 | |
*** gokrokve has quit IRC | 21:04 | |
*** gokrokve has joined #murano | 21:05 | |
openstackgerrit | A change was merged to stackforge/murano-dashboard: Fix 'plus' button for creation nested apps https://review.openstack.org/118991 | 21:18 |
*** gokrokve has quit IRC | 21:21 | |
*** asalkeld has joined #murano | 21:38 | |
*** drupalmonkey has quit IRC | 21:55 | |
*** drupalmonkey has joined #murano | 21:55 | |
*** Murali_ has quit IRC | 21:58 | |
openstackgerrit | Stan Lagun proposed a change to stackforge/murano: Use Keystone trusts to get fresh token https://review.openstack.org/119042 | 21:58 |
*** gokrokve has joined #murano | 22:04 | |
*** asalkeld has quit IRC | 22:09 | |
*** sjmc7 has quit IRC | 22:16 | |
*** gokrokve has quit IRC | 22:19 | |
*** gokrokve has joined #murano | 22:19 | |
*** gokrokve has quit IRC | 22:23 | |
*** asalkeld has joined #murano | 22:26 | |
*** drupalmonkey has quit IRC | 22:30 | |
*** gokrokve has joined #murano | 22:37 | |
*** gokrokve has quit IRC | 22:38 | |
*** gokrokve has joined #murano | 22:39 | |
*** bharath has joined #murano | 23:52 | |
bharath | Hi | 23:52 |
bharath | When I am adding second application to environment, the UI is displaying multiple times, does anyone faced this bug? | 23:53 |
bharath | tsufiev, ^^ | 23:53 |
openstackgerrit | A change was merged to stackforge/murano: Restore auth_uri config option https://review.openstack.org/119062 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!