| openstackgerrit | Merged openstack/murano: Imported Translations from Zanata https://review.openstack.org/268585 | 00:03 |
|---|---|---|
| openstackgerrit | Merged openstack/murano: Updated from global requirements https://review.openstack.org/268464 | 00:03 |
| openstackgerrit | Merged openstack/murano-dashboard: Updated from global requirements https://review.openstack.org/268466 | 00:03 |
| *** yamamoto has quit IRC | 00:27 | |
| *** Marga_ has joined #murano | 00:43 | |
| *** Marga_ has quit IRC | 00:51 | |
| *** Marga_ has joined #murano | 00:52 | |
| *** yamamoto_ has joined #murano | 01:42 | |
| *** zhurong has joined #murano | 01:50 | |
| *** sergmelikyan has quit IRC | 01:55 | |
| openstackgerrit | Hidekazu Nakamura proposed openstack/murano: Add multiple engine workers https://review.openstack.org/268840 | 02:25 |
| *** hidekazu has joined #murano | 02:27 | |
| *** Marga_ has quit IRC | 03:23 | |
| *** Liuqing has joined #murano | 03:54 | |
| *** ashishb has joined #murano | 04:52 | |
| *** ashishb has quit IRC | 04:56 | |
| openstackgerrit | zhurong proposed openstack/murano-dashboard: Add package details link in Package Definitions https://review.openstack.org/268872 | 05:36 |
| *** jasonsb has joined #murano | 05:50 | |
| openstackgerrit | Hidekazu Nakamura proposed openstack/murano: [docs] Update to use openstack CLI instead of glance(v1) https://review.openstack.org/259375 | 06:06 |
| openstackgerrit | zhurong proposed openstack/murano-dashboard: [test]please ingore https://review.openstack.org/268879 | 06:09 |
| *** ashishb has joined #murano | 06:26 | |
| *** jasonsb has quit IRC | 06:45 | |
| openstackgerrit | melissaml proposed openstack/murano: Wrong usage of "an" in the mesages: https://review.openstack.org/266129 | 06:57 |
| *** katyafervent is now known as katyafervent_awa | 08:21 | |
| *** katyafervent_awa is now known as katyafervent | 08:22 | |
| openstackgerrit | melissaml proposed openstack/murano: Wrong usage of "an" in the docstring: https://review.openstack.org/266129 | 08:37 |
| *** samuelBartel has joined #murano | 08:37 | |
| *** naresht has joined #murano | 08:41 | |
| *** tiagogomes has joined #murano | 08:45 | |
| *** klkumar has joined #murano | 09:00 | |
| openstackgerrit | Hidekazu Nakamura proposed openstack/murano: [docs] Update to use openstack CLI instead of glance(v1) https://review.openstack.org/259375 | 09:03 |
| *** gpaz has joined #murano | 09:18 | |
| openstackgerrit | Merged openstack/murano: Broken owner parameter for getAttr/setAttr was fixed https://review.openstack.org/268304 | 09:19 |
| naresht | Hi all. I am using neutron 2015.1.0 release in my MOS setup. I want to update to 2015.1.2 release. How to upgrade from neutron 2015.1.0 to neutron 2015.1.2. Could you tell me how to apply a patch/update..? | 09:33 |
| *** Nikolay_St has joined #murano | 09:59 | |
| openstackgerrit | Merged openstack/python-muranoclient: Repair package-update command https://review.openstack.org/262517 | 10:00 |
| openstackgerrit | Ravi Shekhar Jethani proposed openstack/murano: Python3: Replace dict.iterkeys with six.iterkeys https://review.openstack.org/268964 | 10:11 |
| openstackgerrit | Henar Muñoz proposed openstack/murano: Install murano-agent by cloud-init https://review.openstack.org/234267 | 10:14 |
| openstackgerrit | Henar Muñoz proposed openstack/murano: Install murano-agent by cloud-init https://review.openstack.org/234267 | 10:20 |
| openstackgerrit | Abhishek Kekane proposed openstack/murano: Python3: Replace dict.iterkeys with six.iterkeys https://review.openstack.org/268964 | 10:27 |
| naresht | Hi all. I am getting this error when I tried to install OpenVPN on a MOS instance http://paste.openstack.org/show/484117/ . Could you help me to how to resolve this error ? | 10:32 |
| *** akulshin has joined #murano | 10:32 | |
| *** kzaitsev_mb has joined #murano | 10:33 | |
| kzaitsev_mb | naresht: the trace is a trace of murano-agent? | 10:35 |
| kzaitsev_mb | OpenVPN is a murano app? | 10:35 |
| kzaitsev_mb | pls describe your setup in more detail, as it's currently not clear what exactly you're ding and what goes wrong =) | 10:36 |
| *** tlashchova has joined #murano | 10:37 | |
| openstackgerrit | Ekaterina Chernova proposed openstack/murano-dashboard: [DynamicUI] Apply user regex for the password field https://review.openstack.org/268203 | 10:37 |
| naresht | I created a instance under MOS setup. On that instance I am installing Openvpn server (using this https://help.ubuntu.com/community/OpenVPN) to communicate with Public cloud nodes(Openvpn clients) ? | 10:41 |
| kzaitsev_mb | and how is that connected to murano and your trace? | 10:42 |
| openstackgerrit | Ravi Shekhar Jethani proposed openstack/murano: Python3: Replace dict.itervalues with six.itervalues https://review.openstack.org/268986 | 11:05 |
| *** akulshin has quit IRC | 11:15 | |
| *** zhurong has quit IRC | 11:22 | |
| *** akulshin has joined #murano | 11:23 | |
| openstackgerrit | Kirill Zaitsev proposed openstack/murano: Allow package cache to persist on disc https://review.openstack.org/259999 | 11:26 |
| *** kzaitsev_mb has quit IRC | 11:36 | |
| akulshin | hi everyone! | 11:40 |
| akulshin | i am trying to use switch YAQL expression in UI, and i do not get it... | 11:41 |
| akulshin | availabilityZone: switch($.appConfiguration.osImage, $.startsWith("Red Hat") => 'rhel', true => 'nova') | 11:41 |
| akulshin | but it fails and when i report inside workflow i get Availability zone is switch($.appConfiguration.osImage, $.startsWith("Red Hat") => 'rhel', true => 'nova') | 11:42 |
| akulshin | seems YAQL expression does not work... | 11:42 |
| akulshin | what i am doing wrong? | 11:42 |
| *** yamamoto_ has quit IRC | 11:51 | |
| akulshin | could anybody help with YAQL expressions? | 11:54 |
| *** kzaitsev_mb has joined #murano | 11:56 | |
| akulshin | YAQL? Anybody? | 11:57 |
| katyafervent | Hi akulshin | 12:10 |
| katyafervent | what is your purpose to use switch expression? | 12:11 |
| akulshin | I need to switch availability zone based on image name | 12:11 |
| akulshin | hi katyafervent ;) | 12:12 |
| katyafervent | yaql expressions on the Applications section are intend to be passed to the engine and it compose a json from the user input | 12:12 |
| katyafervent | so not expressions are passed to engine, but the result json) | 12:13 |
| katyafervent | if you want to control something in UI, you have to modify fields section | 12:13 |
| akulshin | so i need to add default value with switch for field availabilityzone and assign it to instance parameter? | 12:14 |
| *** raildo-afk is now known as raildo | 12:15 | |
| katyafervent | oh, and you want to remove availabilityZone: from user import and set it automatically, right? | 12:15 |
| akulshin | yes, but if it will be form foeld, there are hidden propert, so there are no problem | 12:15 |
| akulshin | my expression is correct? | 12:16 |
| katyafervent | let's think what is better. | 12:16 |
| akulshin | is there more then one option? | 12:16 |
| katyafervent | probably it's better to use yaql expresion in that case indeed | 12:17 |
| akulshin | but what is other option(s)? | 12:18 |
| katyafervent | http://yaqluator.com/ | 12:18 |
| katyafervent | you can try your expression here | 12:18 |
| katyafervent | the other option is to set value in Django itself. But it seems to me, murano-dashboard code modification is needed (dynamic UI doesnt have opportunity to support conditions) | 12:20 |
| akulshin | no, this is not the option... | 12:21 |
| slagun | akulshin: hi! | 12:21 |
| akulshin | hi! | 12:22 |
| katyafervent | akulshin, slagun can help you with yaql expression :) | 12:22 |
| akulshin | yes, thank you | 12:22 |
| slagun | akulshin: can you say again which expression doesn't work? | 12:23 |
| akulshin | slagun. i continue to try my availability zone switch | 12:23 |
| akulshin | availabilityZone: switch($.appConfiguration.osImage, $.startsWith("Red Hat") => 'rhel', true => 'nova') | 12:23 |
| akulshin | when i try to deploy, workflow reports switch($.appConfiguration.osImage, $.startsWith("Red Hat") => 'rhel', true => 'nova') as availability zone valye for the instance | 12:24 |
| akulshin | i suppose switch does not executed... | 12:24 |
| slagun | that's because of double quotes | 12:24 |
| slagun | only single quote mark is available in yaql 0.2 | 12:24 |
| akulshin | with single quotes it does to pass after form submission | 12:25 |
| akulshin | katyafervent said that i need to use YAQL only in fields section.... | 12:25 |
| slagun | what is the exception you get with single quotes? | 12:26 |
| slagun | I guess I know | 12:27 |
| slagun | startsWith was defined in murano-engine and not available for UI forms. Now it is in yaql itself, but that was not the case in kilo | 12:28 |
| openstackgerrit | Ekaterina Chernova proposed openstack/murano-dashboard: [DynamicUI] Apply user regex for the password field https://review.openstack.org/268203 | 12:28 |
| akulshin | hm... | 12:30 |
| akulshin | it is not showing anything meaningfull in horizon... | 12:31 |
| akulshin | trying to get to controller nodes | 12:32 |
| slagun | akulshin: you can try switch($.appConfiguration.osImage, 'Red Hat' in $ => 'rhel', true => 'nova') | 12:33 |
| akulshin | waht if i will get one more zone? | 12:33 |
| slagun | switch($.appConfiguration.osImage, $.startsWith("Red Hat") => 'rhel', true => 'nova') | 12:34 |
| slagun | not that one | 12:34 |
| slagun | switch($.appConfiguration.osImage, 'Red Hat' in $ => 'rhel', 'something else' in $ => 'azone2', true => 'nova') | 12:34 |
| *** sergmelikyan has joined #murano | 12:35 | |
| slagun | I know that "in" is not the same as "startsWith" but that what we had in yaql 0.2. You can make a patch to murano-dashboard or yaql to have startsWith. Another option is to say $[0] = R and $[1] = e and $[2] = d ... | 12:36 |
| slagun | but it will cause an exception for short strings | 12:36 |
| akulshin | in is fine, in my case it is a matter of rules. testing this one | 12:37 |
| *** yamamoto has joined #murano | 12:38 | |
| akulshin | well, it is not failing which is good ;) | 12:38 |
| akulshin | can you remind links to sources? i have moved to new mac and cannot find ones you sent me at friday... | 12:39 |
| slagun | > for 1.0 we are working on it. For 0.2 which is used in kilo the only source is source codes. https://github.com/openstack/murano/blob/stable/kilo/murano/engine/system/yaql_functions.py https://github.com/openstack/murano/blob/stable/kilo/murano/dsl/yaql_functions.py and https://github.com/openstack/yaql/tree/stable/0.2/yaql/functions | 12:41 |
| akulshin | thanks! | 12:41 |
| slagun | but first 2 links are for application code only | 12:41 |
| slagun | for UI form it is only the built-in yaql functions + several function introduced by the dashboard | 12:41 |
| akulshin | you mean for workflow? | 12:41 |
| slagun | yes. For MuranoPL classes | 12:42 |
| *** ashishb has quit IRC | 12:44 | |
| akulshin | do you know where i can check what is introduced by dashboard? | 12:44 |
| *** jesuspg has joined #murano | 12:44 | |
| *** sergmelikyan has quit IRC | 12:45 | |
| jesuspg | Hi, I have the next problem with sqlite3 | 12:45 |
| jesuspg | sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) near "DROP": syntax error [SQL: u'ALTER TABLE environment DROP COLUMN networking'] | 12:45 |
| slagun | akulshin: https://github.com/openstack/murano-dashboard/blob/master/muranodashboard/dynamic_ui/yaql_functions.py | 12:45 |
| slagun | akulshin: https://github.com/openstack/murano-dashboard/blob/stable/kilo/muranodashboard/dynamic_ui/yaql_functions.py for Kilo | 12:46 |
| *** sergmelikyan has joined #murano | 12:46 | |
| slagun | jesuspg: hi! That's because SQLite is not supported by database migrations | 12:46 |
| akulshin | slagun : thanks! | 12:46 |
| jesuspg | yes, I know, but there is a migration in alembic with a drop column operation | 12:47 |
| slagun | jesuspg: there is no dtop column in SQL. All modifications to a table structure is ALTER TABLE in SQL. And SQLite doesn't support it | 12:48 |
| jesuspg | then, with this migration file, IMHO murano lost the compatibility with sqlite | 12:48 |
| slagun | yes, it does. Though it works on SQLite if you create database manually | 12:49 |
| akulshin | slagun: suddenly this does not work... | 12:56 |
| akulshin | 2016-01-18 12:54:51 — [LinuxInstance] Image is 6d028d9e-9618-4d70-95a3-ee64081a599c | 12:56 |
| akulshin | 2016-01-18 12:54:51 — [LinuxInstance] Creating Virtual Machine named New Virtual Machine | 12:56 |
| akulshin | 2016-01-18 12:54:51 — [LinuxInstance] Availability zone is nova | 12:56 |
| akulshin | Image is GUID... | 12:56 |
| akulshin | or do I miss some metadata?.. | 12:56 |
| slagun | oh | 13:14 |
| akulshin | but that output is from workflow... | 13:15 |
| slagun | I guess $.appConfiguration.osImage returns image ID | 13:15 |
| akulshin | i guess too, seeking for a way to evaluate this theory ;) | 13:15 |
| slagun | Those expressions in UI is what workflow gets as an input. So if you see GUID on workflow side then it must be the same here | 13:17 |
| akulshin | yes, the same. UI have image guid and send it to workflow | 13:18 |
| *** jesuspg has quit IRC | 13:21 | |
| *** zhurong has joined #murano | 13:22 | |
| akulshin | i believe it is not possible to change availability zone using UI based on image... right? | 13:23 |
| slagun | akulshin: it's bad. Establishing a direct mapping from image ID to availability zone is the only option that comes to my mind (besides patching murano) | 13:23 |
| slagun | well if you don't know all the image IDs in advance and don't want to patch murano, then yes | 13:24 |
| *** jesuspg has joined #murano | 13:25 | |
| akulshin | is it possible to get acces to parameters of custom murano library from ui? | 13:25 |
| akulshin | or make something in workflow? | 13:27 |
| akulshin | one my thought was to modify heat stack in workflow to modify this field, and then call instance.deploy() | 13:31 |
| akulshin | slagun: how do you think? | 13:31 |
| *** kzaitsev_mb has quit IRC | 13:33 | |
| slagun | akulshin: there is no place you can do this | 13:34 |
| slagun | and how will you know the image name in workflow? | 13:34 |
| *** ducttape_ has joined #murano | 13:36 | |
| akulshin | dawn, you right, the same problem... | 13:39 |
| akulshin | but i am using my own library to store openstack installation -depended constants, and theoretically the maping could be there... | 13:40 |
| slagun | if you can map from image ID to azone why can't you put it in UI? | 13:41 |
| *** sergmelikyan has quit IRC | 13:42 | |
| akulshin | becuase the library is a single point of management... | 13:44 |
| *** kzaitsev_mb has joined #murano | 13:46 | |
| slagun | akulshin: well, you can try. You can create Instance (or LinuxMuranoInstance) class inheritor that would make azone property to be mutable and set its value upon deploy() based on some additional knowledge | 13:46 |
| akulshin | yes, i can do this, but what next? i still will not be able to launch the actual instance inside different AZ... | 13:48 |
| slagun | akulshin: why? You can use class configs to put the mapping ID->azone to config file | 13:51 |
| akulshin | but class configs apply only to default parameter values and does not allow any logic. it is simple dictionary, isn't it? | 13:53 |
| *** Liuqing has quit IRC | 13:54 | |
| akulshin | i change availability zone default to rhel for everybode, but then i would need to apply some logic to switch some VMs back to nova | 13:55 |
| *** zhurong has quit IRC | 13:55 | |
| *** sergmelikyan has joined #murano | 13:59 | |
| openstackgerrit | Ravi Shekhar Jethani proposed openstack/murano: Python3: Use urllib from six.moves https://review.openstack.org/269065 | 14:05 |
| *** ducttape_ has quit IRC | 14:12 | |
| *** yamamoto has quit IRC | 14:25 | |
| *** jesuspg has quit IRC | 14:27 | |
| openstackgerrit | ting wang proposed openstack/murano-agent: Clean up removed hacking rule from [flake8] ignore lists https://review.openstack.org/267396 | 14:27 |
| akulshin | slagun: it's pity we cannot make it work using murano... we will patch horizon and add autochoosing functions there... | 14:28 |
| akulshin | thank you for trying to help | 14:28 |
| slagun | akulshin: you can patch dashboard to return image name rather than ID | 14:29 |
| slagun | akulshin: but it is possible to have 2 images with the same name | 14:29 |
| slagun | akulshin: actually it used to return name but we changed it at some point | 14:29 |
| *** ducttape_ has joined #murano | 14:32 | |
| openstackgerrit | Merged openstack/murano: Python3: Replace dict.iterkeys with six.iterkeys https://review.openstack.org/268964 | 14:32 |
| openstackgerrit | Merged openstack/murano: Python3: Replace dict.itervalues with six.itervalues https://review.openstack.org/268986 | 14:33 |
| openstackgerrit | Merged openstack/murano: Python3: Keep compatibility for urllib.urlencode https://review.openstack.org/268816 | 14:34 |
| akulshin | slagun: we found another way! nova-scheduler haves AggregateImagePropertiesIsolation - and this will solve the issue. | 14:34 |
| akulshin | http://docs.openstack.org/kilo/config-reference/content/section_compute-scheduler.html | 14:35 |
| akulshin | seems much more elegant and fool-proof... | 14:36 |
| *** yamamoto has joined #murano | 14:42 | |
| akulshin | but anyway, it was interesting experience in YAQL ;) | 14:46 |
| slagun | akulshin: also a good use case for us | 14:46 |
| akulshin | it is not rare cas e | 14:47 |
| akulshin | sorry... it is not rare case to change something for Instance in workflow... | 14:47 |
| slagun | we are currently designing a feature called "automatic object model generation" where workflow parameters and form content is also controlled by MuranoPL code (up to the point when there won't be UI.yamls at all) | 14:48 |
| akulshin | and it would be nice to have the opportunity to change Instance.everything in workflow before Instance.deploy() called | 14:49 |
| slagun | thinking how to address this use case within that feature | 14:49 |
| slagun | akulshin: it is not a problem to patch core library so that it would be possible to change Instance properties. The problem is how to know the image name etc. | 14:51 |
| slagun | and how to do it right in general | 14:51 |
| akulshin | it is not so easy to patch it, just change usage is not helping | 14:52 |
| slagun | change usage is not enough because you cannot change property values from outside. So you also need to add changeSomething method to the Instance class | 14:52 |
| akulshin | well, if there was object image with properties from glance, then it would help i think. now we get only string, and it is not enough | 14:56 |
| *** sergmelikyan has quit IRC | 14:56 | |
| akulshin | or ability co query nova for additional parameters by UID | 14:57 |
| slagun | well, you can write a plugin for that | 14:57 |
| akulshin | yes, if i would be a programmer... but someday i could make a deep drive into that... | 14:59 |
| *** Nikolay_St has quit IRC | 15:01 | |
| *** ducttape_ has quit IRC | 15:03 | |
| openstackgerrit | ting wang proposed openstack/murano-agent: Clean flake8 ignore https://review.openstack.org/267396 | 15:13 |
| openstackgerrit | Ekaterina Chernova proposed openstack/murano-apps: Fixing typos in README.rst for Kubernetes apps https://review.openstack.org/260959 | 15:29 |
| *** enthurohini has joined #murano | 15:36 | |
| *** klkumar has quit IRC | 15:45 | |
| *** gpaz has quit IRC | 15:49 | |
| *** ducttape_ has joined #murano | 15:50 | |
| openstackgerrit | Stan Lagun proposed openstack/murano: Fixes attribute store for MuranoClassReference types https://review.openstack.org/269154 | 15:51 |
| *** ducttape_ has quit IRC | 15:59 | |
| openstackgerrit | Wang Bo proposed openstack/murano: Using six.moves to import modules only in python2 https://review.openstack.org/269159 | 16:01 |
| *** ducttape_ has joined #murano | 16:02 | |
| openstackgerrit | Wang Bo proposed openstack/murano: Fix python 2 and 3 compatibility issue with six https://review.openstack.org/269159 | 16:09 |
| openstackgerrit | Ekaterina Chernova proposed openstack/murano-apps: [Tomcat][Tests] Add check for a 'deployed' variable https://review.openstack.org/269167 | 16:12 |
| openstackgerrit | Merged openstack/murano: [Murano Docs] [CLI] Manage environments https://review.openstack.org/266462 | 16:25 |
| *** Nikolay_St has joined #murano | 16:35 | |
| openstackgerrit | Wang Bo proposed openstack/murano: Python3:Replace iter.next() with next(iter) https://review.openstack.org/269182 | 16:46 |
| *** madhuri has joined #murano | 16:47 | |
| -openstackstatus- NOTICE: Gerrit is restarting quickly as a workaround for performance degradation | 16:48 | |
| openstackgerrit | Darla Ahlert proposed openstack/murano: [Murano Docs]/[docs] Replace all instances of "tenant" with "project" https://review.openstack.org/264221 | 16:52 |
| *** vahidh has joined #murano | 16:53 | |
| enthurohini | hello kzaitsev_mb | 17:03 |
| *** vahidh has quit IRC | 17:24 | |
| *** ducttape_ has quit IRC | 17:24 | |
| *** samuelBartel has quit IRC | 17:35 | |
| *** kzaitsev_mb has quit IRC | 17:37 | |
| *** vahidh has joined #murano | 17:38 | |
| *** ducttape_ has joined #murano | 17:39 | |
| *** yamamoto has quit IRC | 17:40 | |
| *** ducttape_ has quit IRC | 17:53 | |
| *** vahidh has quit IRC | 18:00 | |
| *** vahidh has joined #murano | 18:00 | |
| *** vahidh has quit IRC | 18:00 | |
| *** samuelBartel has joined #murano | 18:01 | |
| *** vahidh has joined #murano | 18:23 | |
| *** jesuspg has joined #murano | 18:46 | |
| *** jesuspg has quit IRC | 18:55 | |
| *** Marga_ has joined #murano | 18:56 | |
| *** Marga_ has quit IRC | 18:57 | |
| *** ashishb has joined #murano | 18:57 | |
| *** Marga_ has joined #murano | 18:57 | |
| madhuri | Hi I have created an app and I am getting error "NoMatchingFunctionException: No function "#operator_>" matches supplied arguments" | 19:04 |
| madhuri | I can't see in errors what is the error in my app | 19:04 |
| madhuri | Can anyone help? | 19:04 |
| madhuri | kzaitsev_ws: yt? | 19:05 |
| *** kzaitsev_mb has joined #murano | 19:08 | |
| *** ducttape_ has joined #murano | 19:09 | |
| kzaitsev_mb | hi folks | 19:09 |
| kzaitsev_mb | enthurohini: still here? | 19:09 |
| madhuri | Hi kzaitsev_mb | 19:09 |
| enthurohini | yes | 19:10 |
| madhuri | Ok enthurohini first :) | 19:10 |
| kzaitsev_mb | madhuri: which line do you get error from? =) | 19:10 |
| kzaitsev_mb | and what version of murano do you use | 19:10 |
| madhuri | Bit outdated master | 19:11 |
| kzaitsev_mb | enthurohini: had any questions? =) | 19:11 |
| madhuri | That is the problem, I can't find line no in error logs | 19:11 |
| enthurohini | kzaitsev_mb, i was saying that i am having problem in testcases. :P thatswhy its taking time. i will try once again from myside, and wil ask you tomorrow | 19:12 |
| kzaitsev_mb | madhuri: can you grep it for NoMatchingFunctionException ? | 19:12 |
| madhuri | The murano code? | 19:12 |
| kzaitsev_mb | madhuri: or you can upload the class you wrote somewhere and show it to me | 19:12 |
| kzaitsev_mb | the logs | 19:12 |
| madhuri | Ok let me do that | 19:12 |
| kzaitsev_mb | anyway I'll ask you to show me your MuranoPL class at some point )) | 19:13 |
| kzaitsev_mb | so pls upload it to some paste site =) | 19:13 |
| enthurohini | madhuri, hii you can paste at http://paste.openstack.org/ | 19:13 |
| kzaitsev_mb | enthurohini: ok =) cool. Sorry still haven't tested your commit =( had a pretty tight schedule last week. | 19:14 |
| madhuri | kzaitsev_mb: just the classes may not be enough. So I need to upload the whole package and plugin | 19:14 |
| kzaitsev_mb | would do it tomorrow I guess ) | 19:14 |
| kzaitsev_mb | madhuri: let's start with the classes ) | 19:14 |
| madhuri | I can upload a WIP | 19:15 |
| *** ducttape_ has quit IRC | 19:15 | |
| enthurohini | kzaitsev_mb, ok. np. :) actually that commit is suffering from merge conflict. | 19:15 |
| madhuri | But in very bad shape :D | 19:15 |
| *** akulshin has quit IRC | 19:17 | |
| madhuri | kzaitsev_mb: gimme 10 minutes | 19:18 |
| enthurohini | kzaitsev_mb, and as far as i know, two +2 are required for meging the code. and the commit has only two +1 then how merge process is started by jenkins? https://review.openstack.org/266018 | 19:19 |
| kzaitsev_mb | will be back in 10-15 mins | 19:19 |
| kzaitsev_mb | afk now | 19:19 |
| enthurohini | ok | 19:19 |
| enthurohini | :) | 19:19 |
| *** kzaitsev_mb has quit IRC | 19:26 | |
| *** Marga_ has quit IRC | 19:31 | |
| openstackgerrit | Madhuri Kumari proposed openstack/murano: [WIP] Adding Magnum app to deploy Kubernetes Cluster https://review.openstack.org/269250 | 19:40 |
| madhuri | kzaitsev_mb kzaitsev_ws ^^ | 19:41 |
| madhuri | enthurohini: Two +2 and one workflow+1 is required to merge a code | 19:44 |
| enthurohini | madhuri, yes :) | 19:44 |
| madhuri | enthurohini: Your patch has merge conflict so Jenkins failed. | 19:44 |
| madhuri | You need to rebase it | 19:44 |
| enthurohini | yeah, i got it. but why it happened is the question. | 19:46 |
| madhuri | enthurohini: Merge conflicts happens when some patches gets merged in after you upload your patch | 19:47 |
| madhuri | enthurohini: And your patch is doing some change that has been removed from master branch or so | 19:47 |
| madhuri | enthurohini: Have you tried rebasing it? | 19:48 |
| enthurohini | ohk, got it means i dont have current version code. thatswhy it is telling for rebase. | 19:49 |
| enthurohini | not yet. waiting for kirill's comments then will do | 19:49 |
| madhuri | Yes | 19:49 |
| madhuri | Ok good luck (y) | 19:49 |
| madhuri | enthurohini: Are you from India? | 19:50 |
| enthurohini | thanks madhuri. you have developed an application for? | 19:50 |
| madhuri | Magnum | 19:50 |
| enthurohini | madhri, yes i am from indore, INDIA. and you? | 19:50 |
| madhuri | This is my first time I am developing Murano app without much knowledge of Murano :D | 19:51 |
| *** kzaitsev_mb has joined #murano | 19:54 | |
| kzaitsev_mb | enthurohini: the patch is in merge conflict state | 19:55 |
| kzaitsev_mb | you need to fix it | 19:55 |
| kzaitsev_mb | [#murano] enthurohini: this means, that a change by other commit, that has been merged conflicts with your change. you need to checkout the latest master, apply the patch (with a) | 19:55 |
| kzaitsev_mb | with a cherry-pick for example | 19:55 |
| enthurohini | ok, kzaitsev_mb, will fix it | 19:56 |
| kzaitsev_mb | oh I see, madhuri answered most of that =) | 19:57 |
| madhuri | Yeah while waiting for you I tried to help her :) | 19:57 |
| kzaitsev_mb | sorry =) non-openstack part of my life also requires some of my attentions sometimes =) | 19:58 |
| madhuri | Np at all. Pleasure was all mine :) | 19:58 |
| madhuri | kzaitsev_mb: Did you see my app? | 19:59 |
| enthurohini | kzaitsev_mb, she is advising me in personal chat too. she is so helping. great to have you madhuri | 20:00 |
| madhuri | Thanks enthurohini :) | 20:00 |
| kzaitsev_mb | madhuri: you have: $.string().notNull().check($ > 0) | 20:00 |
| kzaitsev_mb | so you're essentially saying that the property should be a string, that is not-null and > 0 | 20:01 |
| madhuri | Oops | 20:01 |
| madhuri | :D | 20:01 |
| kzaitsev_mb | >0 doesn't make sense for yaql strings =) | 20:01 |
| madhuri | Yeah absolutely | 20:01 |
| madhuri | My bad | 20:01 |
| madhuri | Where did you see it in logs? | 20:02 |
| kzaitsev_ws | i didn't checj the logs =) I started with classes file right away ) | 20:02 |
| kzaitsev_mb | madhuri: the would most likely be somewhere in murano-engine's logs | 20:03 |
| kzaitsev_mb | not api's | 20:03 |
| madhuri | Yes I see engine logs only | 20:03 |
| madhuri | But the error log didn't print the line no | 20:03 |
| kzaitsev_mb | yep, we can file a bug for that | 20:04 |
| madhuri | kzaitsev_mb: Any more suggestions is welcome :) | 20:04 |
| madhuri | You can post on my patch | 20:04 |
| kzaitsev_mb | that error logs are not clear when the error happens during contract evaluation | 20:04 |
| kzaitsev_mb | madhuri: do you want this in or did you post it as example? | 20:05 |
| madhuri | No no | 20:05 |
| madhuri | I will be doing modification | 20:05 |
| madhuri | Will post soon | 20:05 |
| madhuri | But yes I need some suggestion | 20:05 |
| kzaitsev_mb | madhuri: anyway — do you want this commit to be merged and maintained by upstream at some point? | 20:06 |
| madhuri | Yes | 20:06 |
| madhuri | I will register a bp explaining it | 20:06 |
| madhuri | But you have already got it | 20:07 |
| kzaitsev_mb | ok, may I ask you to attend our tommorrows IRC meeting then? | 20:07 |
| madhuri | Yes sure | 20:07 |
| madhuri | At what time? | 20:07 |
| kzaitsev_mb | We do not have any policy about plugins as of yet =) | 20:07 |
| madhuri | Ok I had meeting with Serg regarding this | 20:07 |
| kzaitsev_mb | https://wiki.openstack.org/wiki/Meetings/MuranoAgenda you can also add https://wiki.openstack.org/wiki/Meetings/MuranoAgenda to our agenda =) | 20:08 |
| kzaitsev_mb | I've just cleaned it a bit =) | 20:08 |
| madhuri | Sure I will join. Right after Magnum meeting | 20:09 |
| madhuri | :) | 20:09 |
| kzaitsev_mb | ok, I'll have a look at your commit. if you're going to make some adjustements — please add -WIP or add WIP to the commit message 1st line ) | 20:10 |
| kzaitsev_mb | or well don't =) | 20:10 |
| kzaitsev_mb | if you want people to review it seriously ) | 20:10 |
| kzaitsev_mb | but it can get merged ) | 20:10 |
| kzaitsev_mb | so it's usually polite to add WIP somewhere if you intend to change something. | 20:11 |
| kzaitsev_mb | also you should definitelly remove extra whitespaces — gerrit highlights them red and no one likes them ) | 20:12 |
| kzaitsev_mb | I'll go through the code tomorrow ) | 20:12 |
| kzaitsev_mb | madhuri: and yes, our meeting is right next to 'containers' and in the very same channel =))) | 20:12 |
| madhuri | Yeah sure kzaitsev_mb I will do that | 20:14 |
| *** jesuspg has joined #murano | 20:17 | |
| *** akulshin has joined #murano | 20:18 | |
| *** akulshin has quit IRC | 20:23 | |
| *** jesuspg has quit IRC | 20:26 | |
| *** jasonsb has joined #murano | 20:29 | |
| *** jasonsb has quit IRC | 20:31 | |
| *** jasonsb has joined #murano | 20:31 | |
| *** vgridnev has joined #murano | 20:36 | |
| madhuri | kzaitsev_mb: yt? | 20:40 |
| kzaitsev_mb | yep | 20:41 |
| madhuri | I need some suggestion | 20:43 |
| madhuri | Do you few minutes now? | 20:43 |
| kzaitsev_mb | yep | 20:43 |
| kzaitsev_mb | I'm going to release a new version of python-muranoclient in a few moments | 20:43 |
| kzaitsev_mb | so I have plenty of time =) | 20:43 |
| madhuri | In Magnum to create a Kuberenetes cluster, we create a baymodel and then create a bay using the baymodel | 20:44 |
| madhuri | https://github.com/openstack/magnum/blob/master/doc/source/dev/dev-quickstart.rst#building-a-kubernetes-bay | 20:45 |
| madhuri | So in Murano, I have create a Magnum bay an app https://review.openstack.org/#/c/269250/1/contrib/plugins/magnum_plugin/magnum-app/io.murano.apps.MagnumApp/Classes/MagnumBayApp.yaml | 20:46 |
| madhuri | And baymodel a component of it https://review.openstack.org/#/c/269250/1/contrib/plugins/magnum_plugin/magnum-app/io.murano.apps.MagnumApp/Classes/MagnumBaymodel.yaml | 20:47 |
| *** ashishb has quit IRC | 20:47 | |
| madhuri | So is this correct? | 20:47 |
| kzaitsev_mb | looks correct so far | 20:49 |
| kzaitsev_mb | I might need to read through some docs, but what you say sounds reasonable =) | 20:49 |
| madhuri | But I am not able to pass the MagnumBaymodel attributes in MagnumBayApp class | 20:49 |
| slagun | just don't use !yaql "not bool($.getAttr(deployed))" | 20:49 |
| slagun | If: not $.getAttr(deployed, false) | 20:50 |
| madhuri | slagun: Is it for me? | 20:50 |
| slagun | yes | 20:50 |
| madhuri | Thanks slagun | 20:50 |
| slagun | single function with 100500 parameters seems hard to maintain | 20:54 |
| slagun | at least you could pass parameters by name | 20:54 |
| slagun | or to pass a dictionary instead | 20:55 |
| enthurohini | kzaitsev_mb, madhuri, to cherry-pic, first i need to checkout master branch since i have created a topic branch `bp/openstack-client-plugin-support` then `git fetch origin` to have latest content of master branch. After that `git cherry-pic <hash of commit>` then git review. right? or i am missing smth? | 20:56 |
| madhuri | Are you talking about createBaymodel method? | 20:56 |
| slagun | madhuri: yes :) | 20:56 |
| madhuri | I am not sure how we use dic in yaql | 20:56 |
| kzaitsev_mb | enthurohini: right, but you can also use advanced features of git review for that | 20:56 |
| madhuri | Will look for it | 20:57 |
| madhuri | slagun: :) | 20:57 |
| kzaitsev_mb | enthurohini: oh wait, you missed checking out on latest master | 20:57 |
| kzaitsev_mb | enthurohini: 1) git fetch | 20:57 |
| kzaitsev_mb | 2) git checkout master | 20:58 |
| enthurohini | yes | 20:58 |
| *** vgridnev has quit IRC | 20:58 | |
| kzaitsev_mb | 3) git reset --hard origin/master | 20:58 |
| slagun | madhuri: https://paste.ubuntu.com/14570130/ | 20:58 |
| kzaitsev_mb | now you're on the latert master | 20:58 |
| kzaitsev_mb | now you can cherry-pick the commit with | 20:59 |
| madhuri | great thanks slagun :) | 20:59 |
| kzaitsev_mb | 4.1) git cherry-pick <hash> | 20:59 |
| slagun | madhuri: forgot to add spaces: https://paste.ubuntu.com/14570137/ | 20:59 |
| kzaitsev_mb | or 4.2) git review -x 266018 | 20:59 |
| kzaitsev_mb | enthurohini: that would basically do the same, but you don't need to remeber the long hash. just the gerrit change number from the url | 21:00 |
| enthurohini | kzaitsev_mb, what -x will do? | 21:00 |
| kzaitsev_mb | enthurohini: it means 'cherry-pick change 266018 on top of where I am' | 21:01 |
| enthurohini | kzaitsev_mb, at the place of hash i can use gerrit change no.? | 21:01 |
| *** raildo is now known as raildo-afk | 21:01 | |
| kzaitsev_mb | then you'll need to fix errors, marked with ==== <<<< >>>> 'git add' all the files you edited and type in 'git cherry-pick --continue' | 21:02 |
| kzaitsev_mb | enthurohini: no. I meant on the 'latest master' | 21:02 |
| kzaitsev_mb | enthurohini: do you know how git works? (serious question here =)) | 21:04 |
| enthurohini | kzaitsev_mb, yes, i know the basics only, no deep knowledge | 21:04 |
| kzaitsev_mb | enthurohini: cool. pls remind me to explain how it works some day =) just not today, ok? =) | 21:05 |
| enthurohini | kzaitsev_mb, sure. :P | 21:05 |
| *** ducttape_ has joined #murano | 21:11 | |
| *** ducttape_ has quit IRC | 21:15 | |
| *** Marga_ has joined #murano | 21:20 | |
| *** vgridnev has joined #murano | 21:25 | |
| *** Marga_ has quit IRC | 21:25 | |
| *** Marga_ has joined #murano | 21:29 | |
| *** Marga_ has quit IRC | 21:47 | |
| *** vgridnev has quit IRC | 21:53 | |
| openstackgerrit | Madhuri Kumari proposed openstack/murano: [WIP] Adding Magnum app to deploy Kubernetes Cluster https://review.openstack.org/269250 | 21:54 |
| *** madhuri has quit IRC | 21:56 | |
| *** kzaitsev1mb has joined #murano | 22:02 | |
| *** kzaitsev_mb has quit IRC | 22:02 | |
| openstackgerrit | Rohini Choudhary proposed openstack/python-muranoclient: Initial commit for openstack-client support in python-muranoclient https://review.openstack.org/266018 | 22:40 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/murano: Updated from global requirements https://review.openstack.org/269325 | 22:41 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/murano-agent: Updated from global requirements https://review.openstack.org/269326 | 22:42 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-muranoclient: Updated from global requirements https://review.openstack.org/269347 | 22:45 |
| *** jasonsb has quit IRC | 23:00 | |
| *** jasonsb has joined #murano | 23:01 | |
| *** ducttape_ has joined #murano | 23:06 | |
| openstackgerrit | Merged openstack/murano-agent: Updated from global requirements https://review.openstack.org/269326 | 23:22 |
| *** enthurohini has quit IRC | 23:24 | |
| *** ativelkov has quit IRC | 23:31 | |
| *** ducttape_ has quit IRC | 23:32 | |
| *** tsufiev has quit IRC | 23:32 | |
| *** dteselkin has quit IRC | 23:32 | |
| *** IgorYozhikov has quit IRC | 23:32 | |
| *** rakhmerov has quit IRC | 23:32 | |
| *** katyafervent has quit IRC | 23:33 | |
| *** ativelkov has joined #murano | 23:35 | |
| *** IgorYozhikov has joined #murano | 23:36 | |
| *** rakhmerov has joined #murano | 23:37 | |
| *** katyafervent has joined #murano | 23:37 | |
| *** dteselkin has joined #murano | 23:38 | |
| *** tsufiev has joined #murano | 23:39 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!