| *** gokrokve has quit IRC | 00:03 | |
| *** asalkeld has quit IRC | 01:53 | |
| *** asalkeld has joined #murano | 02:22 | |
| *** chandankumar has joined #murano | 03:55 | |
| *** k4n0 has joined #murano | 05:17 | |
| openstackgerrit | A change was merged to stackforge/murano-dashboard: Implements the License and Requirements tabs https://review.openstack.org/108480 | 05:19 |
|---|---|---|
| openstackgerrit | A change was merged to stackforge/murano-dashboard: Adds some minor formatting to application wizard https://review.openstack.org/108473 | 05:36 |
| *** asalkeld has left #murano | 05:39 | |
| *** sergmelikyan has quit IRC | 06:05 | |
| *** sergmelikyan has joined #murano | 06:05 | |
| *** sergmelikyan has quit IRC | 06:12 | |
| *** sergmelikyan has joined #murano | 06:50 | |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Refactor functional tests https://review.openstack.org/106767 | 08:26 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Fix statuses for environment state https://review.openstack.org/104801 | 08:56 |
| openstackgerrit | Timur Nurlygayanov proposed a change to stackforge/murano-dashboard: Fixed issue with incorrect tearDown method https://review.openstack.org/109206 | 09:33 |
| *** openstackgerrit has quit IRC | 10:01 | |
| *** slagun has joined #murano | 10:41 | |
| *** stan_lagun has quit IRC | 10:45 | |
| *** chandankumar has quit IRC | 12:34 | |
| *** lecalcot has joined #murano | 12:39 | |
| *** k4n0 has quit IRC | 12:50 | |
| *** drupalmonkey has joined #murano | 13:00 | |
| *** openstackgerrit has joined #murano | 13:32 | |
| openstackgerrit | Timur Nurlygayanov proposed a change to stackforge/murano-dashboard: Fixed issue with incorrect tearDown method https://review.openstack.org/109206 | 13:52 |
| *** lecalcot has quit IRC | 13:53 | |
| *** lecalcot_ has joined #murano | 13:58 | |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano: Move and rename functional tests https://review.openstack.org/109987 | 13:59 |
| *** sjmc7 has joined #murano | 14:19 | |
| sjmc7 | hi slagun - is there an example of how i can create an instance of an inbuilt murano class (in my case, Agent/AgentListener) for the purposes of unit tests? | 14:29 |
| slagun | sjmc7: hi | 14:29 |
| slagun | you want to create it from Python or from MuranoPL? | 14:30 |
| sjmc7 | python, preferably | 14:30 |
| slagun | take any unit under dsl/ | 14:30 |
| slagun | they all do it | 14:30 |
| sjmc7 | is there any way without creating a test muranopl class? | 14:31 |
| slagun | you instantiate class by loading appropriate object model | 14:31 |
| sjmc7 | ok. that's the only way to do it? | 14:31 |
| slagun | how would you like create instance of class that is never declared? | 14:32 |
| sjmc7 | in python | 14:32 |
| sjmc7 | AgentListener() | 14:32 |
| slagun | If that class is registered in class loader from Python you don't have to write dummy YAMLs | 14:33 |
| sjmc7 | ok - is there an existing example of instantiating a class directly from python? | 14:34 |
| *** sjmc7_ has joined #murano | 14:35 | |
| sjmc7_ | for context, slagun - i'm testing a change to allow an administrator to disable the use of muranoagent (bp/disable-murano-agent), and i want to check that an exception gets raised versus it just sitting there waiting if you try to use muranoagent in a package | 14:39 |
| *** sjmc7 has quit IRC | 14:39 | |
| slagun | There are 2 ways to do it: 1) You register class as in \murano\engine\system\system_objects.py then you write object Model that has instance of that class (as all unit tests do) 2) You can instantiate it directly as in https://github.com/stackforge/murano/blob/master/murano/dsl/dsl_exception.py#L54 but you need to have correct YAQL context which may be hard | 14:40 |
| sjmc7_ | ok. and i have a class and package loader from the test runner? | 14:41 |
| slagun | yes | 14:41 |
| sjmc7_ | i don't see a package loader defined anywhere in the existing unit tests | 14:43 |
| sjmc7_ | ah, i guess i don't need on | 14:43 |
| sjmc7_ | one | 14:44 |
| sjmc7_ | thanks! | 14:46 |
| slagun | everything is baked up in base classes | 14:50 |
| sjmc7_ | ? | 14:50 |
| slagun | all unit tests inherit DslTestCase | 14:50 |
| slagun | and it has a property class_loader | 14:51 |
| slagun | you don't need package loader | 14:51 |
| sjmc7_ | yep, thanks. still getting a YaqlExecutionException from the test model 'Unable to run new' | 14:51 |
| slagun | you can use class loader to register Python classes *prior* to object model load | 14:51 |
| sjmc7_ | yeah - i'm doing self.class_loader.import_class; model = om.Object... | 14:52 |
| slagun | can you show it (paste.openstack.org)? | 14:52 |
| sjmc7_ | http://paste.openstack.org/show/88711/ | 14:53 |
| slagun | and where did you put your exception? | 14:54 |
| slagun | btw you don't have rootProperty in your class | 14:55 |
| slagun | and you dont have "sys:" declared. Thats why it cannot resolve full class name | 14:56 |
| sjmc7_ | ah, right. the fullname should work? | 14:58 |
| sjmc7_ | ok, seems happier now, thanks | 15:00 |
| slagun | you can just declare sys namespace in YAML | 15:02 |
| sjmc7_ | yep, did that, thanks | 15:04 |
| *** lecalcot_ has quit IRC | 15:04 | |
| *** lecalcot has joined #murano | 15:06 | |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Adds UI for 'is_public' option https://review.openstack.org/109166 | 15:15 |
| openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano: Move and rename functional tests https://review.openstack.org/109987 | 15:37 |
| openstackgerrit | Georgy Okrokvertskhov proposed a change to stackforge/murano: Fix the concurrency issue agent queue creation and VM agent https://review.openstack.org/110030 | 15:46 |
| *** murali has joined #murano | 15:53 | |
| murali | sjmc7_: do i need to restart murano after applying latest git patch updates ? | 15:53 |
| sjmc7_ | yes | 15:54 |
| murali | ok i see | 15:54 |
| murali | sjm7_: what is the command to restart murano | 15:56 |
| sjmc7_ | it depends how you started it | 15:56 |
| murali | i used devstack with murano | 15:56 |
| murali | i treid couple of commands but they are not working | 15:57 |
| sjmc7_ | find the murano-api and murano-engine windows in screen (they'll be towards the end) | 15:57 |
| murali | ok | 15:57 |
| sjmc7_ | they'll be running in the foreground; kill them and restart with the previous command | 15:57 |
| murali | ok let me try | 15:58 |
| openstackgerrit | Steve McLellan proposed a change to stackforge/murano: Allow murano-agent to be disabled https://review.openstack.org/109769 | 16:05 |
| sjmc7_ | ruhe, you there? | 16:13 |
| *** sjmc7_ is now known as sjmc7 | 16:14 | |
| igormarnat | sjmc7, rune got back from the business trip and took day off for today, I'm not sure he'll become online today | 16:15 |
| sjmc7 | k, thanks | 16:15 |
| igormarnat | *ruHe | 16:15 |
| *** bharath has joined #murano | 16:23 | |
| bharath | sergmelikyan, Hi | 16:26 |
| bharath | what does this actually mean https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.linux.Telnet/Resources/DeployTelnet.template#L6 ? | 16:33 |
| bharath | how do we know the variable $appName? | 16:36 |
| *** IgorYozhikov is now known as iyozhikov | 16:45 | |
| *** ankurrr has joined #murano | 16:45 | |
| sjmc7 | bharath - it's passed in by the UI | 16:46 |
| sjmc7 | er, sorry, ignore that | 16:46 |
| bharath | sjmc7, it is random name? | 16:49 |
| iyozhikov | we have connectivity issues with our murano-ci server, it services has been notified with ticket. | 16:49 |
| sjmc7 | katyafervent - do you know the answer to bharath's question? | 16:50 |
| ankurrr | stanlagun: I heard you might be able to answer a question re: MuranoPL. The docs (https://wiki.openstack.org/wiki/Murano/DSL/Blueprint#Block_constructs) suggest that MuranoPL supports while-loops. Is this implemented in MuranoPL? | 16:52 |
| ankurrr | er, I meant slagun (not stanlagun) ^^ | 16:53 |
| slagun | ankurrr: hu! | 16:55 |
| slagun | hi! | 16:55 |
| slagun | yes | 16:56 |
| slagun | https://github.com/stackforge/murano/blob/master/murano/tests/dsl/meta/MacroExamples.yaml#L26 | 16:56 |
| ankurrr | slagun: ok, thanks. let me see if I can compare my while-loop to this example to see why I'm getting a parse error | 16:58 |
| slagun | show me yours, I'll tell you | 16:58 |
| sergmelikyan | bharath, this example is little bit better: https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.PostgreSql/Classes/PostgreSql.yaml#L59 | 17:06 |
| sergmelikyan | https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.PostgreSql/Resources/ConfigurePostgreSqlDatabase.template#L5 | 17:06 |
| sergmelikyan | ^^ values from first link are substituted in second file | 17:07 |
| bharath | sergmelikyan, yeah for telnet, it is not mentioned anywhere , right? | 17:11 |
| *** murali has quit IRC | 17:21 | |
| *** iyozhikov is now known as IgorYozhikov | 17:38 | |
| *** lecalcot has quit IRC | 17:51 | |
| *** lecalcot has joined #murano | 17:51 | |
| *** lecalcot has quit IRC | 17:56 | |
| bharath | tsufiev, sjmc7 can I abandon this https://review.openstack.org/#/c/108778/ ? | 18:24 |
| sjmc7 | if you want to, yes | 18:24 |
| bharath | sjmc7, I mean according to the comments given, I think my changes are wrong and are not required? | 18:26 |
| sjmc7 | tsufiev made the first comment | 18:27 |
| *** Meena has joined #murano | 18:31 | |
| *** lecalcot has joined #murano | 18:32 | |
| bharath | yeah | 18:33 |
| *** bharath has quit IRC | 18:33 | |
| ankurrr | slagun: here's my while loop, but it looks like the parser is failing to detect the end of a block. anything strikes you as incorrect syntax here? http://paste.openstack.org/show/88746/ | 18:47 |
| ankurrr | slagun: maybe my problem is that I'm not correctly instantiating my io.murano.resources.Instance objects | 19:07 |
| sjmc7 | ankurrr, the parentheses are unnecessary/possibly causing problems | 19:21 |
| sjmc7 | around the string concats | 19:21 |
| sjmc7 | the new_instance instantiation doesn't look right either | 19:22 |
| ankurrr | sjmc7: yeah, I changed the new_instance instantion to: $new_instance: new(res:Instance, name => $new_instance_name, flavor => $.flavor, image => $.osImage, keyname => $.keyPair, assignFloatingIp => true) | 19:22 |
| ankurrr | sjmc7: will remove the parens around the string concatenations | 19:23 |
| *** murali has joined #murano | 19:44 | |
| murali | sjmc7: 'module' object has no attribute 'info | 19:44 |
| murali | what is this error mean ? 'module' object has no attribute 'info | 19:44 |
| murali | when i do "Deploy This Component" i see above error | 19:45 |
| murali | is this a bug ? | 19:45 |
| sjmc7 | do you have a stactktrace in the engine log? | 19:45 |
| murali | yes | 19:47 |
| murali | 2014-07-28 14:41:04.554 10772 TRACE murano.common.engine data = self.load_definition(name) 2014-07-28 14:41:04.554 10772 TRACE murano.common.engine File "/opt/stack/murano/murano/engine/package_class_loader.py", line 39, in load_definition 2014-07-28 14:41:04.554 10772 TRACE murano.common.engine logging.info('--------------Tracing the xception------------') 2014-07-28 14:41:04.554 10772 TRACE murano.common.engine AttributeErr | 19:47 |
| murali | here is the complete log | 19:48 |
| murali | 2014-07-28 14:41:04.554 10772 DEBUG muranoclient.common.http [-] curl -i -X GET -H 'X-Auth-Token: 2af40d2ce14d4d079b86d178efa0d8d1' -H 'Content-Type: application/json' -H 'User-Agent: python-muranoclient' http://localhost:8082/v1/catalog/packages?class_name=io.murano.Object&limit=20 log_curl_request /opt/stack/python-muranoclient/muranoclient/common/http.py:158 2014-07-28 14:41:04.554 10772 ERROR murano.common.engine [-] Error during t | 19:48 |
| sjmc7 | can you put it on http://paste.openstack.org/ please? | 19:48 |
| murali | here --- http://paste.openstack.org/show/88763/ | 19:49 |
| sjmc7 | use LOG.info | 19:50 |
| murali | where do i need to use LOG.info ? | 19:52 |
| sjmc7 | where you've used logging.info | 19:52 |
| murali | u mean inside package_class_loader.py? | 19:52 |
| murali | the one i sent you murano-engine log. | 19:54 |
| murali | it has already logging.info | 19:54 |
| sjmc7 | the stacktrace shows you what the error is | 19:55 |
| sjmc7 | you need to use LOG.info | 19:55 |
| murali | ok | 19:55 |
| sjmc7 | the 'logging' module that's imported is not the python one | 19:55 |
| sjmc7 | it's from openstack.common | 19:55 |
| murali | so do i need to import pyhton one | 19:56 |
| sjmc7 | no. you need to use LOG.info | 19:57 |
| murali | ok | 19:57 |
| murali | got you thanks | 19:57 |
| openstackgerrit | Steve McLellan proposed a change to stackforge/murano: Don't hide exception messages during package load https://review.openstack.org/110125 | 20:22 |
| ankurrr | slagun, sjm7: here's the latest. parser chokes on line 49: http://paste.openstack.org/show/88771/ | 20:35 |
| sjmc7 | it looks like the py2.7 unit tests are failing on the postgresql migration checks; i'll send a note to ruslan to see if he can take a look tomorrow | 20:51 |
| *** asalkeld has joined #murano | 20:58 | |
| *** lecalcot has quit IRC | 21:02 | |
| *** lecalcot has joined #murano | 21:10 | |
| slagun | ankurrr: are you still here? | 21:15 |
| ankurrr | slagun: yeah, still here | 21:16 |
| slagun | I've looked over your code | 21:16 |
| slagun | I know why it fails but even you fix it it wouldn't work | 21:16 |
| slagun | the error you get is from PyYAML, not from MuranoPL parser | 21:17 |
| slagun | and the reason is that value on line 49 starts with quote | 21:18 |
| slagun | this is YAML error | 21:18 |
| slagun | this can be fixed using brackets ('vm-' + str($counter)), using format: format('vm-{0}', counter) or using explicit YAML tag: !yaql "'vm-'" + str($counter)" | 21:19 |
| sjmc7 | why does it cause an error? | 21:20 |
| sjmc7 | quotes are normally allowed in yaml | 21:20 |
| slagun | If YAML parser sees value that starts with quotes it considers it to be string and thus it also need to end with quote which is not the case | 21:20 |
| slagun | and anyway this need to be parsed as YAQL expression, not as a string | 21:21 |
| sjmc7 | ah, ok | 21:21 |
| slagun | the main problem with the code is that currently you cannot instantiate MuranoPL classes and deploy them. You also cannot append them to collections | 21:22 |
| slagun | In short you cannot write code that modifies object model from within | 21:22 |
| slagun | btw there will probably never be a method append. I will be just $.instances: $instances + $newInstance | 21:23 |
| openstackgerrit | Steve McLellan proposed a change to stackforge/murano: Default is_public to false https://review.openstack.org/109153 | 21:25 |
| sjmc7 | but even then it won't work this way? | 21:25 |
| ankurrr | slagun: let me see if I understand correctly: I can't create a new instance in my class's deploy() method? I'm fine if I have to "append" with $.instances: $.instances + $new_instance. | 21:27 |
| slagun | it won't. At least until we implement https://blueprints.launchpad.net/murano/+spec/muranopl-object-instantiation | 21:28 |
| slagun | ankurrr: You can think of it as there is no new() method except for several system stateless classes | 21:29 |
| slagun | there is a description why it is so in BP | 21:30 |
| ankurrr | slagun: so, from the BP description, it sounds like I can get the instances created, but they wouldn't be deleted if the environment was deleted. | 21:30 |
| slagun | > Software components usually need some known parents in object model hierarchy (usually Environment). But instances created with new() doesn't have such parent because they are not yet placed inti object model. < | 21:31 |
| slagun | it will not find its environment and fail with exception | 21:31 |
| slagun | And even if it could find it it won't be persisted to object model and you get resource leak | 21:32 |
| sjmc7 | for our purposes, ankurrr, maybe it's best to make it a boolean choice then, whether to have a single node or a cluster | 21:34 |
| slagun | you just need to move that logic to UI | 21:35 |
| slagun | generate object model instances. Like we do in Active Directory | 21:35 |
| *** asalkeld has left #murano | 21:35 | |
| openstackgerrit | Steve McLellan proposed a change to stackforge/murano: Default is_public to false https://review.openstack.org/109153 | 21:36 |
| slagun | instead of passing instanceCount you will pass instance list | 21:36 |
| ankurrr | slagun: ok, I'll take a look at the Active Directory example from the murano-apps-incuabtor repo. Thanks for the info btw. | 21:37 |
| openstackgerrit | Steve McLellan proposed a change to stackforge/murano: Don't hide exception messages during package load https://review.openstack.org/110125 | 21:38 |
| *** lecalcot has quit IRC | 22:00 | |
| *** lecalcot has joined #murano | 22:01 | |
| *** lecalcot has quit IRC | 22:02 | |
| *** lecalcot has joined #murano | 22:07 | |
| *** lecalcot has quit IRC | 22:09 | |
| *** lecalcot has joined #murano | 22:09 | |
| *** lecalcot has quit IRC | 22:10 | |
| *** lecalcot has joined #murano | 22:11 | |
| *** lecalcot has quit IRC | 22:27 | |
| *** lecalcot has joined #murano | 22:28 | |
| *** murali has quit IRC | 22:33 | |
| *** drupalmonkey has quit IRC | 22:35 | |
| *** lecalcot has quit IRC | 22:53 | |
| *** lecalcot has joined #murano | 22:53 | |
| openstackgerrit | Georgy Okrokvertskhov proposed a change to stackforge/murano: Fix the issue with multiple agent call https://review.openstack.org/110156 | 22:56 |
| *** lecalcot has quit IRC | 22:58 | |
| openstackgerrit | Georgy Okrokvertskhov proposed a change to stackforge/murano: Fix the issue with multiple agent call https://review.openstack.org/110156 | 23:06 |
| openstackgerrit | Georgy Okrokvertskhov proposed a change to stackforge/murano: Fix the concurrency issue agent queue creation and VM agent https://review.openstack.org/110030 | 23:10 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!