| *** gokrokve has joined #murano | 01:00 | |
| *** Murali has quit IRC | 01:27 | |
| *** gokrokve has quit IRC | 01:44 | |
| *** gokrokve has joined #murano | 02:14 | |
| *** gokrokve has quit IRC | 02:19 | |
| *** Murali has joined #murano | 02:43 | |
| *** gokrokve has joined #murano | 03:25 | |
| *** drupalmonkey has quit IRC | 03:28 | |
| *** gokrokve has quit IRC | 03:29 | |
| *** Murali__ has joined #murano | 03:48 | |
| *** Murali has quit IRC | 03:52 | |
| *** Murali__ has quit IRC | 04:45 | |
| *** Murali has joined #murano | 04:45 | |
| *** chandankumar has joined #murano | 04:51 | |
| *** gokrokve has joined #murano | 05:10 | |
| *** gokrokve has quit IRC | 05:36 | |
| *** gokrokve has joined #murano | 05:36 | |
| *** chandankumar has quit IRC | 05:40 | |
| *** gokrokve has quit IRC | 05:41 | |
| *** Murali__ has joined #murano | 05:45 | |
| *** Murali has quit IRC | 05:48 | |
| *** Murali__ has quit IRC | 05:56 | |
| *** chandankumar has joined #murano | 06:02 | |
| *** sjmc7_ has quit IRC | 06:12 | |
| *** k4n0 has joined #murano | 06:30 | |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Fix statuses for environment state https://review.openstack.org/104801 | 07:01 |
|---|---|---|
| *** romainh has joined #murano | 07:31 | |
| openstackgerrit | A change was merged to stackforge/murano-dashboard: Update Image selection in functional test https://review.openstack.org/112785 | 07:40 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Fix statuses for environment state https://review.openstack.org/104801 | 07:43 |
| openstackgerrit | A change was merged to stackforge/murano-dashboard: Fix statuses for environment state https://review.openstack.org/104801 | 08:30 |
| openstackgerrit | A change was merged to stackforge/murano: Default is_public to false https://review.openstack.org/109153 | 08:56 |
| *** DaveJ__ has joined #murano | 09:02 | |
| openstackgerrit | A change was merged to stackforge/murano: Allow murano-agent to be disabled https://review.openstack.org/109769 | 09:49 |
| openstackgerrit | A change was merged to stackforge/murano: Reduce number of API requests during deploy https://review.openstack.org/113676 | 09:50 |
| openstackgerrit | Ruslan Kamaldinov proposed a change to stackforge/murano: [WIP] test db rename deployment -> task https://review.openstack.org/114183 | 09:52 |
| openstackgerrit | A change was merged to stackforge/murano: Fix the concurrency issue agent queue creation and VM agent https://review.openstack.org/110030 | 09:55 |
| openstackgerrit | Serg Melikyan proposed a change to stackforge/python-muranoclient: Bump hacking to 0.9.x series https://review.openstack.org/109260 | 10:14 |
| *** chandankumar has quit IRC | 10:19 | |
| *** chandankumar has joined #murano | 10:31 | |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano: Add openstack libs to config checker https://review.openstack.org/114212 | 11:52 |
| *** stanlagun has joined #murano | 11:59 | |
| openstackgerrit | Timur Nurlygayanov proposed a change to stackforge/murano-dashboard: Fixed issue with is_public=False default value https://review.openstack.org/114215 | 12:05 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano: Add oslo messaging to config checker https://review.openstack.org/114212 | 12:08 |
| openstackgerrit | Serg Melikyan proposed a change to stackforge/murano-dashboard: Support environments being deleted of with delete failure https://review.openstack.org/109511 | 12:15 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano: Add oslo messaging to config checker https://review.openstack.org/114212 | 12:20 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano: Add openstack libs to config checker https://review.openstack.org/114212 | 12:29 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano: Add openstack libs to config checker https://review.openstack.org/114212 | 12:35 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Use wizard to make package uploading clear https://review.openstack.org/110894 | 12:50 |
| openstackgerrit | Serg Melikyan proposed a change to stackforge/murano-dashboard: Support environments being deleted of with delete failure https://review.openstack.org/109511 | 13:02 |
| openstackgerrit | Dmitry Teselkin proposed a change to stackforge/murano: Update Image Builder documentation https://review.openstack.org/114244 | 13:27 |
| openstackgerrit | Serg Melikyan proposed a change to stackforge/murano: Fixes silent deletion of environments https://review.openstack.org/109508 | 13:44 |
| *** romainh has left #murano | 13:49 | |
| *** Murali has joined #murano | 13:53 | |
| *** Murali has quit IRC | 13:58 | |
| *** gokrokve has joined #murano | 13:59 | |
| *** drupalmonkey has joined #murano | 13:59 | |
| *** 17SAAUYGN has joined #murano | 13:59 | |
| openstackgerrit | A change was merged to stackforge/murano-dashboard: Fixed issue with is_public=False default value https://review.openstack.org/114215 | 14:04 |
| ruhe | katyafervent: why would pep8 fails on import order on https://review.openstack.org/#/c/114212/ on my laptop? | 14:08 |
| *** gokrokve has quit IRC | 14:14 | |
| *** gokrokve has joined #murano | 14:14 | |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Use wizard to make package uploading clear https://review.openstack.org/110894 | 14:20 |
| ruhe | sergmelikyan: oi vei. we did a bad thing today http://git.openstack.org/cgit/stackforge/murano/tree/murano/tests/dsl | 14:21 |
| ruhe | dsl should be under murano/tests/unit | 14:22 |
| sergmelikyan | yep, fixed in one of the commits alongside with other fixes in tests | 14:24 |
| sergmelikyan | Why this thing passed the gate? | 14:24 |
| sergmelikyan | ruhe, https://review.openstack.org/109508 | 14:25 |
| ruhe | sergmelikyan: why wouldn't it? it just skipped those | 14:25 |
| sergmelikyan | Why? I thought that on gate job we re-running all unit-tests | 14:26 |
| sergmelikyan | *are rerunning | 14:26 |
| ruhe | we're running all the tests under murano/tests/unit | 14:26 |
| sergmelikyan | and this tests simply were not found... | 14:27 |
| sergmelikyan | ruhe, btw, can you help me with our dvsm job? | 14:27 |
| sergmelikyan | Looks like patch with proper env deletion broke tests | 14:28 |
| sergmelikyan | ruhe, http://logs.openstack.org/08/109508/3/check/gate-murano-devstack-dsvm/94e7808/console.html | 14:28 |
| sergmelikyan | Looks like issue is related to the fact that envs are no more deleted on the spot and task is sent to murano-engine | 14:29 |
| ruhe | one moment | 14:29 |
| sergmelikyan | Do we have working murano-engine in this job? | 14:29 |
| ruhe | sergmelikyan: check out http://logs.openstack.org/08/109508/3/check/gate-murano-devstack-dsvm/94e7808/logs/ | 14:30 |
| sergmelikyan | I came up with simple patch http://paste.openstack.org/show/95048/ but will it work depends on how we running tests | 14:30 |
| sergmelikyan | ruhe, I have already seen this log | 14:30 |
| sergmelikyan | Please, read again my messages above :) | 14:30 |
| sergmelikyan | https://github.com/stackforge/murano/blob/master/murano/tests/functional/api/base.py#L48 | 14:32 |
| sergmelikyan | Fixed by the patch above | 14:32 |
| sergmelikyan | I think this should do the job, what do you think? | 14:32 |
| sergmelikyan | smurashov, can you help me to test my change? | 14:34 |
| sergmelikyan | Is it possible to run this tests on local machine? | 14:34 |
| ruhe | sergmelikyan: sorry, i'm occupied atm | 14:35 |
| sergmelikyan | ruhe, sure, sorry | 14:35 |
| openstackgerrit | A change was merged to stackforge/murano: Add openstack libs to config checker https://review.openstack.org/114212 | 14:40 |
| *** sjmc7 has joined #murano | 14:41 | |
| *** k4n0 has quit IRC | 14:45 | |
| *** gokrokve_ has joined #murano | 15:27 | |
| *** gokrokve has quit IRC | 15:31 | |
| *** gokrokve_ has quit IRC | 15:31 | |
| btully | katyafervent: are you around? | 15:55 |
| katyafervent | yes | 15:55 |
| katyafervent | hi) | 15:56 |
| btully | hi there! :) | 15:56 |
| btully | i had a question for you about the environment state fix you committed. looks like it got merged before i had a chance to do a code review | 15:56 |
| btully | https://review.openstack.org/#/c/104801/ | 15:56 |
| btully | https://bugs.launchpad.net/murano/+bug/1335309 | 15:57 |
| btully | is the bug | 15:57 |
| btully | the question i have is that when you create an environment it creates it, and while the progress bar no longer appears (nice!) the text shown to the user is "Configuring" | 15:58 |
| btully | the same is true when you add a component to an environment | 15:58 |
| btully | the API seems to return "pending" as a status | 15:58 |
| btully | whereas "configuring" makes me think that I need to wait for the UI to tell me it's "ready" for my next step | 15:59 |
| btully | does pending actually = configuring? and if so, does the user ever get informed that it is "ready"? | 16:01 |
| btully | in other words, from a UX perspective, it's easy to assume that you can't deploy the environment because the status says "Configuring" | 16:02 |
| btully | tnurlygayanov: i'd love to hear your opinion on this as well since you +1 the code review | 16:04 |
| *** bharath has joined #murano | 16:13 | |
| katyafervent | btully, it was on review for 2 weeks :) | 16:17 |
| katyafervent | yes, pending is configuring in UI | 16:18 |
| btully | yeah, sorry about that. i haven't had much time for code reviews, sadly :( | 16:18 |
| katyafervent | so are suggesting to rename configuring to something else? | 16:19 |
| btully | if the UI says configuring, can you deploy the environment? or does the user need to wait for it to say "ready"? | 16:19 |
| katyafervent | configuring means that you started work with this environment | 16:19 |
| sjmc7 | the "-ing" is the problem - it strongly suggested you should wait for something to finish | 16:19 |
| katyafervent | the session is open | 16:19 |
| katyafervent | you can go around and add more applications to this environment | 16:19 |
| btully | i.e., are there cases where a user needs to wait for the environment or service to be configured before deploying? | 16:20 |
| katyafervent | after you are are done, deploy should be pressed | 16:20 |
| katyafervent | you guys are better in naming. so you are welcome to suggest new names for statuses | 16:21 |
| btully | for STATUS_DISPLAY_CHOICES the last entry is ('', 'Ready to configure'), | 16:21 |
| btully | assuming that the API will return an empty string for status? | 16:21 |
| katyafervent | they are located here https://github.com/stackforge/murano-dashboard/blob/master/muranodashboard/environments/consts.py | 16:21 |
| btully | so i guess in the API, does 'pending' mean 'ready to configure' or 'ready to deploy' or both depending on the situation? | 16:22 |
| btully | yup | 16:22 |
| btully | that's where i'm looking | 16:22 |
| katyafervent | I guess this is never happen and this status will never be shown | 16:22 |
| btully | (STATUS_ID_PENDING, 'Configuring'), | 16:22 |
| btully | does the API ever return 'ready'? | 16:23 |
| katyafervent | so is it should be just Configure? | 16:23 |
| btully | i'm not sure configure is appropriate either | 16:24 |
| katyafervent | yes | 16:24 |
| katyafervent | preparation? | 16:25 |
| btully | when you create a new env without any services, i suppose a good status would be 'ready to configure' or more specifically 'ready for components' | 16:25 |
| katyafervent | yes means sessions will have status ready after the deployment is finished | 16:25 |
| sjmc7 | how about 'Not deployed'? | 16:26 |
| btully | then when you add a component it says 'configuring' as well which is misleading | 16:26 |
| katyafervent | Deployment preparation ? | 16:26 |
| sjmc7 | this is one of those things like errors i'd really like to have a set of screens showing the flow, it'd make it easier to discuss them | 16:26 |
| *** gokrokve has joined #murano | 16:27 | |
| btully | i'll take some screenshots now if you'd like | 16:28 |
| sjmc7 | it doesn't need to be now necessarily. if there are any word renames that make sense for 'configuring' then we can discuss them | 16:28 |
| katyafervent | well, the api has no middle status. ready to configure' - should be assigned somewhere in dashboard when no services | 16:28 |
| sjmc7 | to the user though, is there any difference between those two? | 16:31 |
| sjmc7 | it feels like "you can deploy", "deploying", "deployed" are the states. and maybe "empty" | 16:31 |
| sjmc7 | so i guess it does make sense | 16:31 |
| billy_ | How should I resolve a failing Fuel Murano Health Check due to NeutronClientException, Security Group {GUID} in use for Murano 0.5 on Icehouse? The following Fuel health check is failing, with a default deployment via Fuel: "Check that user can deploy Telnet service in Murano environment". | 16:32 |
| billy_ | Looks like deploy succeeds in Murano, but Security Group is in use, so heat reports it cannot be destroyed. | 16:33 |
| katyafervent | sjmc7, so we should create a blueprint for renaming and discuss there | 16:34 |
| btully | thanks katyafervent. sjmc7 and I just got pulled into a meeting for about 30 minutes | 16:36 |
| *** chandankumar has quit IRC | 16:46 | |
| btully | i agree katyafervent, if possible for the environment list table, it would be great if there was another status for an environment that was pending but with no services yet added | 16:47 |
| btully | 'Ready for services' or 'ready to configure' | 16:48 |
| btully | whereas if an env has a service added, maybe status would be 'ready to deploy' | 16:49 |
| btully | note also that when you add a service/component, that table status also shows "configuring" so i'm not sure if there is a separate status through the API for components or is that using the same status as the environment? | 16:50 |
| *** chandankumar has joined #murano | 17:03 | |
| btully | katyafervent: sorry to bother you again | 17:05 |
| btully | how exactly did you remove the progress bar? | 17:05 |
| sjmc7 | btully - in the constants, the boolean values | 17:06 |
| btully | right, but i don't see how that removes the animated gif progress bar | 17:07 |
| btully | https://www.dropbox.com/s/d64pq8ebkmnbiwl/environment-simple-ready.png | 17:07 |
| btully | i think we'll need to do the same for the 'ready' state | 17:07 |
| sjmc7 | the boolean values are supposed to indicate whether something is 'happening' | 17:08 |
| btully | are you referring to STATUS_CHOICES = ( | 17:09 |
| btully | (None, True), | 17:09 |
| btully | (STATUS_ID_PENDING, True), | 17:09 |
| btully | (STATUS_ID_DEPLOYING, None), | 17:09 |
| btully | (STATUS_ID_NEW, True) | 17:09 |
| btully | ? | 17:09 |
| btully | reload style | 17:09 |
| sjmc7 | yeah, think that's it | 17:09 |
| btully | we should prob wrap a comment around that to indicate that's what it means | 17:10 |
| sjmc7 | sure. is it not working? sometimes horizon doesn't refresh running under manage.py | 17:10 |
| stanlagun | about statuses | 17:10 |
| stanlagun | https://review.openstack.org/#/c/109508/ | 17:10 |
| btully | look at the screenshot above | 17:11 |
| stanlagun | it changes how they calculated on API and adds additional statuses | 17:11 |
| btully | the env has been deployed and status is ready, but the progress bar still shows | 17:11 |
| btully | interesting so adding True for STATUS_ID_READY hides the progress bar | 17:16 |
| btully | so i guess True = hide the progress bar | 17:16 |
| btully | but it's not exactly boolean unless in python None === False | 17:17 |
| btully | (STATUS_ID_DEPLOYING, None), | 17:17 |
| btully | stanlagun: that looks great. we were just talking about how environment deletion needs to be improved from a UX perspective and this is a great improvement! | 17:20 |
| stanlagun | https://review.openstack.org/#/c/109511/ | 17:21 |
| stanlagun | thats dashboard's side of those changes | 17:21 |
| stanlagun | just need to fix several small issues with those commits. I thing sergmelikyan already doing that | 17:22 |
| btully | awesome | 17:22 |
| btully | :D | 17:22 |
| btully | :D | 17:22 |
| *** chandankumar has quit IRC | 18:22 | |
| sjmc7 | sergmelikyan, thanks for working on those patches, appreciate it | 18:32 |
| sjmc7 | hi stanlagun.. is there a way in muranopl to conditionally put a key:value pair into a heat template? as an example, a heat bug yesterday requires a workaround to add a depends_on: router_interface to FloatingIP resources, but only if heat is creaeting the router interface - see comment on https://bugs.launchpad.net/murano/+bug/1356721 | 18:34 |
| stanlagun | Heat template is just a dictionary that can be modified in MPL. And there is If macro for conditional execution | 18:35 |
| stanlagun | If: condition Then: $template.depends_on: value | 18:36 |
| sjmc7 | ok, so do it afterwards $template.resources.$fip_name.depends_on: $routerIntefance | 18:36 |
| sjmc7 | ok | 18:36 |
| openstackgerrit | Georgy Okrokvertskhov proposed a change to stackforge/murano: Adds REST API endpoint for action execution https://review.openstack.org/99449 | 18:39 |
| bharath | Hey I just saw the assignee of blueprint is not me https://blueprints.launchpad.net/murano/+spec/show-is-public-to-package-definitions | 18:44 |
| bharath | this is my account https://launchpad.net/~bharath-ves | 18:45 |
| bharath | :) | 18:45 |
| bharath | sjmc7, is there any chance to change that if you dont mind? | 18:47 |
| sjmc7 | it is you | 18:48 |
| sjmc7 | oh, i see | 18:49 |
| sjmc7 | there's another one of you! | 18:49 |
| sjmc7 | that's awkward. done | 18:49 |
| bharath | sjmc7, I just saw that | 18:49 |
| bharath | sjmc7, thankyou :) | 18:49 |
| *** bharath has quit IRC | 19:20 | |
| openstackgerrit | Brian Tully proposed a change to stackforge/murano-dashboard: Fix extra column in services table during deploy https://review.openstack.org/113417 | 20:03 |
| *** gokrokve has quit IRC | 20:05 | |
| *** gokrokve has joined #murano | 20:06 | |
| *** gokrokve has quit IRC | 20:08 | |
| *** gokrokve has joined #murano | 20:08 | |
| *** openstackgerrit has quit IRC | 21:46 | |
| *** openstackgerrit has joined #murano | 21:47 | |
| 17SAAUYGN | is there any default username/password if i launch murano app using ubuntu image ? | 21:58 |
| 17SAAUYGN | i tried with default one not working for murano apps | 21:58 |
| 17SAAUYGN | katyafervent : can u please help me | 21:59 |
| sjmc7 | usernames/passwords come from the image itself. typically ubuntu only allows ssh access | 22:01 |
| *** gokrokve has quit IRC | 22:28 | |
| *** gokrokve has joined #murano | 22:40 | |
| *** 17SAAUYGN has quit IRC | 22:50 | |
| *** Murali_ has joined #murano | 23:00 | |
| *** drupalmonkey has quit IRC | 23:11 | |
| *** DaveJ__ has quit IRC | 23:18 | |
| *** Murali_ has quit IRC | 23:44 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
