*** kzaitsev_mb has quit IRC | 00:34 | |
*** yamamoto has quit IRC | 02:12 | |
openstackgerrit | xiangxinyong proposed openstack/murano-dashboard: Delete the unsupport page_size in Package Definitions https://review.openstack.org/215143 | 02:17 |
---|---|---|
*** zhurong has joined #murano | 02:24 | |
*** zhurong_ has joined #murano | 02:31 | |
*** zhurong has quit IRC | 02:34 | |
*** yamamoto has joined #murano | 02:58 | |
*** Samos123 has quit IRC | 03:27 | |
*** Samos123 has joined #murano | 03:27 | |
*** xiangxinyong has joined #murano | 03:36 | |
openstackgerrit | zhu.rong proposed openstack/python-muranoclient: Strip json and html from error messages https://review.openstack.org/218714 | 03:46 |
*** prefer has quit IRC | 03:47 | |
*** lawrance_ has joined #murano | 03:50 | |
*** Piet has joined #murano | 03:50 | |
*** Piet has quit IRC | 03:53 | |
*** stanchan has joined #murano | 03:55 | |
*** Piet has joined #murano | 03:56 | |
openstackgerrit | xiangxinyong proposed openstack/python-muranoclient: Fix the reversed incoming parameters of assertEqual https://review.openstack.org/218716 | 03:57 |
*** Piet has quit IRC | 03:59 | |
*** zhurong_ has quit IRC | 04:34 | |
*** xiangxinyong has quit IRC | 04:36 | |
*** zhurong has joined #murano | 04:43 | |
*** sergmelikyan has joined #murano | 04:50 | |
*** jasonsb has joined #murano | 04:50 | |
*** enthurohini has joined #murano | 04:51 | |
*** sergmelikyan has quit IRC | 05:09 | |
*** prefer has joined #murano | 05:12 | |
*** prefer has quit IRC | 05:17 | |
*** xiangxinyong has joined #murano | 05:34 | |
*** prefer has joined #murano | 05:36 | |
openstackgerrit | xiangxinyong proposed openstack/python-muranoclient: Fix the reversed incoming parameters of assertEqual https://review.openstack.org/218716 | 05:48 |
*** prefer has quit IRC | 05:55 | |
*** jasonsb has quit IRC | 06:10 | |
*** sergmelikyan has joined #murano | 06:27 | |
*** sergmelikyan has quit IRC | 06:32 | |
*** mgershen has joined #murano | 07:28 | |
*** junhongl has joined #murano | 07:30 | |
mgershen | Good morning | 07:44 |
*** enthurohini has quit IRC | 07:44 | |
*** samuelBartel has joined #murano | 07:45 | |
*** Nikolay_St has joined #murano | 07:50 | |
*** xiangxinyong has quit IRC | 08:25 | |
*** zhonghua-lee has quit IRC | 08:30 | |
*** ddovbii has joined #murano | 08:42 | |
*** kzaitsev_mb has joined #murano | 08:49 | |
*** kzaitsev_mb has quit IRC | 09:17 | |
*** zhurong has quit IRC | 09:27 | |
*** zhurong has joined #murano | 09:27 | |
*** zhurong has quit IRC | 09:27 | |
*** zhurong has joined #murano | 09:31 | |
*** zhurong has quit IRC | 09:38 | |
slagun | mgershen: morning | 10:04 |
*** lawrance_ has quit IRC | 10:08 | |
*** zhurong has joined #murano | 10:11 | |
*** zhurong has quit IRC | 10:23 | |
*** kzaitsev_mb has joined #murano | 10:27 | |
openstackgerrit | Kirill Zaitsev proposed openstack/murano-dashboard: WIP: apply AUTO_DISCOVER_STATIC_FILES https://review.openstack.org/200764 | 10:58 |
openstackgerrit | Kirill Zaitsev proposed openstack/murano: Run compress command from dashboard devstack plug-in https://review.openstack.org/218807 | 11:01 |
*** enthurohini has joined #murano | 11:04 | |
openstackgerrit | Kirill Zaitsev proposed openstack/murano: Run compress command for dashboard in devstack https://review.openstack.org/218807 | 11:30 |
freerunner | Hi folks! | 11:39 |
freerunner | I would like to raise attention to this patch https://review.openstack.org/#/c/218411/1 | 11:40 |
*** Valli_M has joined #murano | 11:40 | |
freerunner | It will be good, if we merge this ASAP. | 11:40 |
freerunner | This make our CI more stable ;) | 11:40 |
freerunner | @katyafervent: @slagun ^^ | 11:41 |
*** lawrancejing has joined #murano | 11:42 | |
*** zhurong has joined #murano | 11:42 | |
*** kzaitsev_mb has quit IRC | 11:44 | |
*** openstackgerrit has quit IRC | 11:46 | |
*** openstackgerrit has joined #murano | 11:47 | |
*** kzaitsev_mb has joined #murano | 11:56 | |
*** sergmelikyan has joined #murano | 11:58 | |
*** sergmelikyan has quit IRC | 12:02 | |
*** enthurohini has quit IRC | 12:06 | |
openstackgerrit | Merged openstack/murano: Improve functional engine tests cleanup https://review.openstack.org/218411 | 12:14 |
*** raildo-afk is now known as raildo | 12:16 | |
openstackgerrit | Michal Gershenzon proposed openstack/murano: Add support for heat environments https://review.openstack.org/211608 | 12:20 |
*** Valli_M has quit IRC | 12:26 | |
openstackgerrit | Michal Gershenzon proposed openstack/murano: Add support for heat environments https://review.openstack.org/211608 | 12:30 |
mgershen | kzaitsev_ws, are you here? | 12:32 |
openstackgerrit | Nikolay Starodubtsev proposed openstack/murano: Add support for more Cloud Foundry API calls https://review.openstack.org/202011 | 12:32 |
openstackgerrit | Nikolay Starodubtsev proposed openstack/murano: Cloud Foundry Service Broker API initial commit https://review.openstack.org/196820 | 12:32 |
openstackgerrit | Alexey Khivin proposed openstack/murano: BOM symbol removed from Agent-v1.template https://review.openstack.org/218832 | 12:34 |
kzaitsev_mb | mgershen: yep, looking at your patch now =) | 12:35 |
kzaitsev_mb | looks good =) | 12:35 |
mgershen | yay :D | 12:35 |
mgershen | i've changed the order of some lines and removed a comment in PS 10 | 12:36 |
kzaitsev_mb | will want to play with it/deploy a couple of apps. but seems, that it is what we discussed ) | 12:36 |
mgershen | good idea. | 12:36 |
mgershen | I did remove the defaults though.. | 12:36 |
kzaitsev_mb | why? | 12:37 |
mgershen | but they were never really useful, they were taken from the hot template and heat can do it by himself. | 12:37 |
kzaitsev_mb | mgershen: but that wouldn't be nice in UI, unfortunatelly. | 12:38 |
mgershen | I removed them because in the new model the parameters are not passed in the root of the API, but under templateParameters, and I didn't manage to get that working. | 12:38 |
kzaitsev_mb | we would create a field for such parameter and if it is required — would require some input | 12:38 |
kzaitsev_mb | even though it has some meaningfull default in the template itself | 12:39 |
kzaitsev_mb | also. this change looks unrelated | 12:40 |
mgershen | I think we are talking on different defauls | 12:40 |
mgershen | *defaults | 12:40 |
kzaitsev_mb | https://review.openstack.org/#/c/211608/10/murano/packages/hot_package.py line 139 here? | 12:40 |
mgershen | yes | 12:41 |
kzaitsev_mb | this removal looks unrelated | 12:41 |
kzaitsev_mb | it doesn't affect environment feature, does it? | 12:41 |
kzaitsev_mb | if you have environments in the package — this code would not be executes | 12:42 |
kzaitsev_mb | *executed | 12:42 |
*** Valli_M has joined #murano | 12:42 | |
mgershen | the other defaults that are the UI ones were not changed line 381 | 12:43 |
mgershen | it is related because after the change template parameters are passed in the API inside the dict templateParameters and not on the root level. I fond that you can do a contract for such a dict, but I couldn't get it to take defaults. | 12:44 |
kzaitsev_mb | hm, you seem right | 12:46 |
mgershen | before if the user did not send a parameter, murano sent the default from the hot template. not it will send null and the heat engine will take the default parameter from the hot template. | 12:46 |
mgershen | I know it seems strange, but it works, and from what I can see the UI acts the same as before. | 12:47 |
kzaitsev_mb | thats reassuring. I'll check the patch out and test it now then | 12:48 |
mgershen | thank you :) | 12:49 |
kzaitsev_mb | oh, btw. It came to my mind, that currently there is no way to create a package, that supports environments or files other than just zip'ing them in, right? | 12:50 |
mgershen | yes | 12:50 |
kzaitsev_mb | IDK if we file that as a bug or postpone to M as a feature.. | 12:51 |
mgershen | and the feature can be used mostly from API. | 12:51 |
openstackgerrit | Alexey Khivin proposed openstack/murano: BOM symbol removed from Agent-v1.template https://review.openstack.org/218832 | 12:51 |
kzaitsev_mb | should probably discuss at meeting ) | 12:51 |
kzaitsev_mb | yeah, I generally have no problem with that | 12:51 |
kzaitsev_mb | it's just, that we should not forget aoubt the feature. document it at some point in time and perhaps add some client support | 12:52 |
*** junhongl has quit IRC | 12:53 | |
mgershen | I agree. | 12:53 |
kzaitsev_mb | like murano package-create-from-hot -t template -f files -e environment would have been nice to have =) | 12:53 |
mgershen | by the way, do you want me to send you some simple ready packages? | 12:54 |
kzaitsev_mb | mgershen: oh, that would be great! | 12:54 |
mgershen | https://drive.google.com/file/d/0B56NyEcusHFjV2RjaHVpeGh6cUE/view?usp=sharing | 12:55 |
mgershen | https://drive.google.com/open?id=0B56NyEcusHFjMm9JWS04OTQwa2c | 12:55 |
mgershen | and this is an example to an API call to add a package to an environment: http://paste.openstack.org/show/434814/ | 12:58 |
kzaitsev_mb | mgershen: btw. what keys do you feed tar to generate a zip package? =) | 13:01 |
kzaitsev_mb | I use tar czvf but that generates a gzip compressed file | 13:02 |
mgershen | when I moved the files from my devstack to my laptop, I just renamed it, so the right program will open haha ^^" | 13:04 |
mgershen | but this is a nice guide http://www.simplehelp.net/2008/12/15/how-to-create-and-extract-zip-tar-targz-and-tarbz2-files-in-linux/ | 13:07 |
kzaitsev_mb | oh ok | 13:08 |
kzaitsev_mb | I know about bz2 and gzip =) | 13:09 |
kzaitsev_mb | also the article lies! =) | 13:10 |
mgershen | it doesn't work? | 13:10 |
kzaitsev_mb | cvf would not compress anything, just stack files together into 1 file with zero compression and some overhead for padding/boundary info I guess ) | 13:10 |
mgershen | in such cases, my friends say "it's a metaphor" haha | 13:13 |
mgershen | =) | 13:13 |
kzaitsev_mb | have you seen Guardians of the Galaxy? =) | 13:13 |
mgershen | yes | 13:14 |
kzaitsev_mb | http://static.comicvine.com/uploads/scale_medium/11120/111201126/4319830-steve-rogers-i-understood-that-reference.jpg then | 13:14 |
mgershen | XD | 13:14 |
mgershen | and I just saw the honest trailer. do you know the youtube channel called screen junkies https://www.youtube.com/watch?v=dOyJqGtP-wU | 13:16 |
mgershen | (it's addictive) | 13:16 |
kzaitsev_mb | =)) | 13:16 |
openstackgerrit | Stan Lagun proposed openstack/murano-specs: Murano versioning https://review.openstack.org/140402 | 13:23 |
openstackgerrit | zhu.rong proposed openstack/python-muranoclient: Strip json and html from error messages https://review.openstack.org/218714 | 13:26 |
openstackgerrit | Lin Yang proposed openstack/murano: Fix silently overwrite user specified content type https://review.openstack.org/218191 | 13:37 |
*** Valli_M has quit IRC | 13:45 | |
openstackgerrit | zhu.rong proposed openstack/python-muranoclient: Strip json and html from error messages https://review.openstack.org/218714 | 13:51 |
openstackgerrit | zhu.rong proposed openstack/python-muranoclient: Update the git ingore https://review.openstack.org/218863 | 13:54 |
*** dteselkin has quit IRC | 14:03 | |
openstackgerrit | Stan Lagun proposed openstack/murano: Adds version info to ApplicationPackage https://review.openstack.org/214163 | 14:05 |
*** dteselkin has joined #murano | 14:07 | |
*** lawrancejing has quit IRC | 14:21 | |
*** zhurong has quit IRC | 14:22 | |
*** stanchan has quit IRC | 14:24 | |
openstackgerrit | Nikolay Starodubtsev proposed openstack/murano: Log refactoring close to new logging spec https://review.openstack.org/209922 | 14:27 |
*** Nikolay_St has quit IRC | 14:34 | |
openstackgerrit | Merged openstack/murano-dashboard: Allow passing request variables to import forms https://review.openstack.org/217747 | 14:43 |
openstackgerrit | Merged openstack/murano: Simple instance configuration https://review.openstack.org/213708 | 14:43 |
*** Valli_M has joined #murano | 14:49 | |
openstackgerrit | Merged openstack/murano: BOM symbol removed from Agent-v1.template https://review.openstack.org/218832 | 14:51 |
*** samuelBartel has quit IRC | 14:56 | |
*** zhurong has joined #murano | 14:59 | |
openstackgerrit | zhu.rong proposed openstack/python-muranoclient: Strip json and html from error messages https://review.openstack.org/218714 | 15:17 |
*** zhurong has quit IRC | 15:19 | |
openstackgerrit | Merged openstack/murano: Logging API for MuranoPL https://review.openstack.org/211233 | 15:22 |
openstackgerrit | Stan Lagun proposed openstack/murano: Package versioning https://review.openstack.org/216070 | 15:26 |
openstackgerrit | Stan Lagun proposed openstack/murano: Version-aware YAML loading https://review.openstack.org/217457 | 15:26 |
openstackgerrit | Stan Lagun proposed openstack/murano: yaql context versioning https://review.openstack.org/218612 | 15:26 |
openstackgerrit | Stan Lagun proposed openstack/murano: Restores back plugin support https://review.openstack.org/218687 | 15:26 |
openstackgerrit | Merged openstack/murano-dashboard: Delete the unsupport page_size in Package Definitions https://review.openstack.org/215143 | 15:28 |
*** Nikolay_St has joined #murano | 15:39 | |
*** Piet has joined #murano | 15:44 | |
openstackgerrit | Stan Lagun proposed openstack/murano-apps: Fixes availability zone selection https://review.openstack.org/218924 | 15:47 |
*** Valli_M has quit IRC | 15:48 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/python-muranoclient: Fixed issue with cacert parameter https://review.openstack.org/216148 | 15:59 |
*** sergmelikyan has joined #murano | 16:01 | |
openstackgerrit | Stan Lagun proposed openstack/murano: Adds version info to ApplicationPackage https://review.openstack.org/214163 | 16:02 |
openstackgerrit | Stan Lagun proposed openstack/murano: Package versioning https://review.openstack.org/216070 | 16:02 |
openstackgerrit | Stan Lagun proposed openstack/murano: Version-aware YAML loading https://review.openstack.org/217457 | 16:02 |
openstackgerrit | Stan Lagun proposed openstack/murano: Restores back plugin support https://review.openstack.org/218687 | 16:02 |
*** sergmelikyan has quit IRC | 16:23 | |
*** sergmelikyan has joined #murano | 16:26 | |
openstackgerrit | Stan Lagun proposed openstack/murano: Version-aware YAML loading https://review.openstack.org/217457 | 16:29 |
openstackgerrit | Stan Lagun proposed openstack/murano: Package versioning https://review.openstack.org/216070 | 16:33 |
openstackgerrit | Stan Lagun proposed openstack/murano: Version-aware YAML loading https://review.openstack.org/217457 | 16:33 |
*** sergmelikyan has quit IRC | 16:39 | |
openstackgerrit | Michal Gershenzon proposed openstack/murano: Add support for heat environments https://review.openstack.org/211608 | 16:51 |
mgershen | slagun, are you here? | 16:56 |
slagun | mgershen: hi | 16:59 |
mgershen | how are you? | 17:02 |
*** exploreshaifali has joined #murano | 17:04 | |
slagun | I'm good | 17:05 |
slagun | I guess you are concerned with my -1 on environments? | 17:05 |
slagun | sorry for that :) | 17:05 |
*** rhagarty_ has quit IRC | 17:09 | |
*** rhagarty has quit IRC | 17:09 | |
*** rhagarty has joined #murano | 17:14 | |
*** rhagarty_ has joined #murano | 17:14 | |
*** sergmelikyan has joined #murano | 17:17 | |
*** jasonsb has joined #murano | 17:20 | |
*** stanchan has joined #murano | 17:22 | |
mgershen | slagun: yes, but you were right with your -1 :) | 17:25 |
mgershen | I removed the UI, I think it can go in a different PS. what do you think? | 17:26 |
slagun | not sure I get you PS=patch set=the same commit | 17:28 |
slagun | you mean in another commit? | 17:28 |
mgershen | sorry, different review | 17:28 |
mgershen | *I think it can go in a different review. what do you think? | 17:30 |
slagun | you can do it but environments would not be merged until UI commit (which should be dependent commit) gets 2 * +2. In another words you can split it but both parts need to be merged in the same time to avoid possibility that we will reach HCF with environments merged and UI broken | 17:31 |
slagun | but this is my opinion | 17:31 |
mgershen | but the UI is not broken, it is just that the blueprint is incomplete... | 17:32 |
slagun | I'd rather go with fixing UI in the same commit. At least listing file names. This should be really simple like several lines of code | 17:32 |
slagun | It was not broken before you delete it | 17:32 |
*** jasonsb has quit IRC | 17:33 | |
slagun | But since you changed parameter structure and haven't changed UI it becomes broken | 17:33 |
*** jasonsb has joined #murano | 17:33 | |
mgershen | please take a look, I can't see where it is broken | 17:34 |
kzaitsev_mb | slagun: as a matter of fact I think we can remove it from UI completely and add it gradually (1st client than UI) | 17:34 |
kzaitsev_mb | you see | 17:34 |
slagun | oh, you deleted only the part about environment name, not parameters | 17:34 |
mgershen | slagun: yes | 17:34 |
slagun | than its okay | 17:34 |
slagun | then | 17:34 |
kzaitsev_mb | currently there is no way to create the package with env support, other then ziping the archive by hand | 17:35 |
slagun | there is nothing wrong in manual zipping. At least it is not a big issue. But for end user there is no way to get possible field values | 17:36 |
kzaitsev_mb | slagun: that's why I'm saying we could remove the field for now | 17:36 |
kzaitsev_mb | from the UI I mean | 17:36 |
slagun | I agree | 17:36 |
*** jasonsb_ has joined #murano | 17:37 | |
kzaitsev_mb | would probably have to add drop-down and CLI commands in M | 17:37 |
slagun | It should be either removed or done right. Text box is the worst solution here | 17:37 |
mgershen | In PS 11 it is removed | 17:38 |
*** jasonsb has quit IRC | 17:38 | |
slagun | Considering that to add drop-down in UI is a matter of several lines of code I don't see why wait till M | 17:38 |
slagun | client support can be added later because it is less of an issue. PAckage developers are skillful enaugh to know how to use zip | 17:38 |
kzaitsev_mb | slagun: I haven't looked up — on how to get a list of envs ) | 17:39 |
slagun | list of envs = list of files in directory | 17:39 |
slagun | or better some field from each of the envs | 17:39 |
slagun | but to add list of files should be simple | 17:40 |
kzaitsev_mb | can't I create a new env? or specify one by hand? =) | 17:40 |
*** stanchan has quit IRC | 17:40 | |
slagun | no. Envs are packaged in Murano archive | 17:40 |
slagun | you can just select on of them | 17:40 |
kzaitsev_mb | oh, so either no env or one of the pre-defined | 17:41 |
kzaitsev_mb | k | 17:41 |
kzaitsev_mb | that seems easy. Will look into adding them in a separate patch ) | 17:41 |
kzaitsev_mb | mgershen: so current PS is ready for review — without the field, right? | 17:42 |
mgershen | yes | 17:44 |
kzaitsev_mb | k. will look/test it today | 17:45 |
mgershen | _build_hot_resources_dict function inside HotPackage (should be renamed), returns a list of files. it can be used to get the hot envs list. | 17:45 |
kzaitsev_mb | mgershen, slagun: let's add such support in a separate commit, then shall we? | 17:46 |
mgershen | +1 | 17:46 |
slagun | +1 | 17:51 |
*** kzaitsev_mb has quit IRC | 17:51 | |
mgershen | this is an example of how the function can be used (to save you time) http://paste.openstack.org/show/435283/ | 17:56 |
mgershen | kzaitsev_mb, slagun, thank you both | 17:59 |
*** stanchan has joined #murano | 18:02 | |
*** stanchan has quit IRC | 18:07 | |
*** sergmelikyan has quit IRC | 18:09 | |
*** sergmelikyan has joined #murano | 18:20 | |
*** exploreshaifali has quit IRC | 18:28 | |
*** mgershen has quit IRC | 18:37 | |
openstackgerrit | Merged openstack/python-muranoclient: Update the git ingore https://review.openstack.org/218863 | 18:44 |
*** kzaitsev_mb has joined #murano | 19:00 | |
*** sergmelikyan has quit IRC | 19:09 | |
*** exploreshaifali has joined #murano | 19:33 | |
*** sergmelikyan has joined #murano | 20:17 | |
openstackgerrit | Darren Shaw proposed openstack/murano: Update VirtIO download link https://review.openstack.org/219021 | 20:33 |
*** jasonsb_ has quit IRC | 21:01 | |
*** jasonsb has joined #murano | 21:13 | |
*** Berezin has joined #murano | 21:34 | |
*** Berezin has quit IRC | 21:57 | |
*** sergmelikyan has quit IRC | 22:04 | |
*** sergmelikyan has joined #murano | 22:04 | |
*** sergmelikyan has quit IRC | 22:07 | |
*** sergmelikyan has joined #murano | 22:08 | |
*** sergmelikyan has quit IRC | 22:12 | |
*** sergmelikyan has joined #murano | 22:18 | |
*** sergmelikyan has quit IRC | 22:19 | |
*** sergmelikyan has joined #murano | 22:21 | |
*** Piet has quit IRC | 22:22 | |
openstackgerrit | Serg Melikyan proposed openstack/murano: Restores back plugin support https://review.openstack.org/218687 | 22:44 |
*** Piet has joined #murano | 22:52 | |
*** Berezin has joined #murano | 22:53 | |
*** exploreshaifali has quit IRC | 23:04 | |
*** jasonsb has quit IRC | 23:13 | |
*** kzaitsev_mb has quit IRC | 23:19 | |
openstackgerrit | Merged openstack/murano: Updated from global requirements https://review.openstack.org/218519 | 23:23 |
*** kzaitsev_mb has joined #murano | 23:32 | |
*** Berezin has quit IRC | 23:35 | |
*** sergmelikyan has quit IRC | 23:44 | |
*** Berezin has joined #murano | 23:45 | |
openstackgerrit | Merged openstack/murano: Fixed the typo's in multiple files https://review.openstack.org/218674 | 23:52 |
openstackgerrit | Darren Shaw proposed openstack/murano: Update VirtIO download link https://review.openstack.org/219021 | 23:56 |
*** Berezin has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!