*** ankurrr has joined #murano | 00:30 | |
openstackgerrit | Ankur Rishi proposed a change to stackforge/murano: Add optional fields to packages for supplier info https://review.openstack.org/99465 | 00:31 |
---|---|---|
*** gokrokve has quit IRC | 01:14 | |
*** gokrokve has joined #murano | 01:14 | |
*** gokrokve has quit IRC | 01:19 | |
openstackgerrit | Ankur Rishi proposed a change to stackforge/murano: Add optional fields to packages for supplier info https://review.openstack.org/99465 | 01:29 |
openstackgerrit | Ankur Rishi proposed a change to stackforge/murano: Add optional fields to packages for supplier info https://review.openstack.org/99465 | 01:34 |
ankurrr | ruhe: I had a question about testing migrations: I tried adding a "_check_002" function similar to the "_check_001" function you added in the alembic commit, but I noticed that in running the tests via tox, the neither function gets called. Is there a specific command that needs to be run in order to run those tests? | 01:58 |
*** ankurrr has quit IRC | 01:58 | |
*** sjmc7 has quit IRC | 02:05 | |
*** gokrokve has joined #murano | 02:20 | |
*** gokrokve has quit IRC | 02:24 | |
*** killer_prince has quit IRC | 02:44 | |
*** gokrokve has joined #murano | 02:44 | |
*** killer_prince has joined #murano | 02:52 | |
*** killer_prince is now known as lazy_prince | 02:52 | |
*** gokrokve_ has joined #murano | 03:44 | |
*** gokrokve has quit IRC | 03:48 | |
*** lazy_prince is now known as killer_prince | 03:49 | |
*** killer_prince is now known as lazy_prince | 04:08 | |
*** gokrokve_ has quit IRC | 04:11 | |
*** gokrokve has joined #murano | 04:11 | |
*** gokrokve has quit IRC | 04:16 | |
*** lazy_prince has quit IRC | 04:31 | |
*** gokrokve has joined #murano | 04:43 | |
*** lazy_prince has joined #murano | 04:57 | |
*** killer_prince has joined #murano | 05:27 | |
*** drupalmonkey has joined #murano | 05:32 | |
*** drupalmonkey has quit IRC | 05:44 | |
*** lazy_prince has quit IRC | 06:17 | |
*** killer_prince is now known as lazy_prince | 06:17 | |
*** lazy_prince has quit IRC | 06:19 | |
*** lazy_prince has joined #murano | 06:26 | |
*** chandan_kumar has joined #murano | 06:47 | |
*** openstackgerrit has quit IRC | 07:07 | |
*** stan_lagun has joined #murano | 08:11 | |
*** slagun has quit IRC | 08:14 | |
*** chandan_kumar is now known as chkumar246 | 10:51 | |
*** chkumar246 is now known as chandan_kumar | 10:53 | |
*** chandan_kumar has quit IRC | 10:54 | |
*** chandan_kumar has joined #murano | 10:54 | |
*** sergmelikyan has quit IRC | 11:14 | |
*** asalkeld has quit IRC | 11:15 | |
*** sergmelikyan has joined #murano | 11:23 | |
*** sergmelikyan has quit IRC | 11:23 | |
*** lazy_prince has quit IRC | 11:37 | |
*** lazy_prince has joined #murano | 11:47 | |
*** sergmelikyan has joined #murano | 11:55 | |
*** openstackgerrit has joined #murano | 12:26 | |
openstackgerrit | Serg Melikyan proposed a change to stackforge/murano: Delete Heat stack when environment is deleted https://review.openstack.org/102810 | 12:31 |
*** gokrokve_ has joined #murano | 12:50 | |
*** gokrokve_ has quit IRC | 12:50 | |
*** gokrokve_ has joined #murano | 12:51 | |
*** gokrokv__ has joined #murano | 12:52 | |
*** gokrokve has quit IRC | 12:53 | |
*** gokrokve_ has quit IRC | 12:55 | |
*** drupalmonkey has joined #murano | 12:57 | |
*** lazy_prince has quit IRC | 13:18 | |
openstackgerrit | Serg Melikyan proposed a change to stackforge/murano: Delete Heat stack when environment is deleted https://review.openstack.org/102810 | 13:21 |
ruhe | katyafervent: hi! what's the status of https://review.openstack.org/#/c/98770/ ? | 13:38 |
openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Introduces script for updating horizon config https://review.openstack.org/101578 | 14:05 |
*** sjmc7 has joined #murano | 14:06 | |
katyafervent | I'm planning to create blueprint according to the comments on this review | 14:11 |
katyafervent | and start to implement it soon | 14:11 |
katyafervent | ruhe, is new blueprint needed, right? | 14:11 |
ruhe | katyafervent: new blueprint would be really nice. will you abandon this patch? | 14:12 |
katyafervent | i can keep change id | 14:12 |
katyafervent | but actually it can be abandoned | 14:12 |
sergmelikyan | tox -e venv -- murano-db-manage --config-file=./etc/murano/murano.conf upgrade | 14:20 |
sergmelikyan | ERROR: InvocationError: could not find executable 'murano-db-manage' | 14:20 |
sergmelikyan | ruhe? | 14:20 |
ruhe | sergmelikyan: let me check one more time | 14:22 |
sjmc7 | segmelikyan - run python setup.py develop | 14:23 |
sjmc7 | to install murano-db-manage in venv/bin | 14:23 |
ruhe | sergmelikyan: you probably need to kill .tox/venv | 14:23 |
ruhe | or what sjmc7 said | 14:23 |
sergmelikyan | sjmc7, ruhe thx! | 14:25 |
sergmelikyan | sjmc7, no luck with you solution | 14:25 |
ruhe | and i also need to reply to ankurrr about writing new tests for DB migrations. that's another tricky part i didn't document in commit message and docs | 14:26 |
sjmc7 | yeah, thanks ruhe - my team are very new to python and unit testing, and any help appreciated | 14:26 |
sergmelikyan | ruhe, I cloned clean repo, so no .tox was present before :( | 14:27 |
ruhe | sergmelikyan: ok. i'll check on a clean clone too | 14:30 |
sergmelikyan | ruhe, I am trying one more time with removed .venv | 14:31 |
* sergmelikyan does know that one should not affect another... but... | 14:32 | |
sergmelikyan | ruhe, same thing | 14:33 |
openstackgerrit | Anastasia Kuznetsova proposed a change to stackforge/murano: Add articles about Automated Tests https://review.openstack.org/102500 | 14:35 |
ruhe | sergmelikyan: did a clean clone, ran the same command you posted and got "congratulations :)" from tox | 14:37 |
ruhe | but response "it works on my machine, go away" is not appropriate. so, let's continue debugging :) | 14:37 |
ruhe | can you try "find . -name murano-db-manage" > | 14:39 |
ruhe | s/>/? | 14:39 |
sergmelikyan | nothing | 14:39 |
sergmelikyan | ruhe, I am stupid :( | 14:40 |
sergmelikyan | ruhe, sorry | 14:40 |
ruhe | sergmelikyan: did you find the source of your problem? | 14:42 |
sergmelikyan | ruhe, yes, commit with Alembic was not in source-tree of working directory | 14:43 |
sergmelikyan | I was working on old commit that was not rebased on top of master with alembic | 14:43 |
openstackgerrit | Ekaterina Chernova proposed a change to stackforge/murano-dashboard: Introduces script for updating horizon config https://review.openstack.org/101578 | 14:44 |
sergmelikyan | btw, alembic add system-wide dependency - libpq-dev | 14:46 |
*** killer_prince has joined #murano | 14:46 | |
sjmc7 | yeah, i noticed that sergmelikyan | 14:46 |
sergmelikyan | postgresql-devel in CentOs | 14:46 |
*** killer_prince is now known as lazy_prince | 14:46 | |
katyafervent | https://blueprints.launchpad.net/murano/+spec/divide-upload-package-into-two-steps need approve | 14:51 |
ruhe | katyafervent: first step is to discuss with key people :) | 14:51 |
ruhe | sjmc7: sergmelikyan: Ankur filed a bug about this https://bugs.launchpad.net/murano/+bug/1333985 | 14:52 |
sjmc7 | ah, good man | 14:52 |
katyafervent | let me copy everything fron the review | 14:52 |
katyafervent | *from | 14:52 |
ruhe | sjmc7: sergmelikyan: my suggestion is to document these requirements in developer docs | 14:52 |
ruhe | i should've document those requirements in commit message, but i had them installed long time ago | 14:55 |
sergmelikyan | ruhe, we have list of system-wide requirements - http://murano.readthedocs.org/en/latest/install/prepare.html#system-prerequisites | 14:56 |
sergmelikyan | We can just add this one to the list | 14:56 |
ruhe | sergmelikyan: those requirements are for prod installation | 14:56 |
sergmelikyan | Why then there are in dev docs? | 14:57 |
ruhe | sergmelikyan: it's ok to keep installation guide in the dev docs for now. we can always move them to admin guide | 14:57 |
sergmelikyan | And actually what the difference between prod/dev in terms of system-wide dependencies? | 14:57 |
ruhe | you don't need to have both libmysqlclient and libpq-dev in prod. you'll need only one of them. but for testing you need both of them | 14:58 |
sergmelikyan | I will think where to add new article with list of system-wide dependencies and we can reference them from prod/dev guides. What do you think? | 14:58 |
ruhe | i'd prefer to keep them right in the corresponding guide | 14:59 |
sergmelikyan | Ok, but I am not sure that section I have referenced is only production installation guide | 15:00 |
ruhe | sjmc7: just in case i don't catch Ankur here, please pass him this link http://paste.openstack.org/show/85000/ i tried to explain how to run DB migration tests locally. i'll try to update dev docs later | 15:00 |
sergmelikyan | there is a section called Installing and Running the Development Version in there | 15:00 |
sjmc7 | k | 15:00 |
katyafervent | let's discuss https://blueprints.launchpad.net/murano/+spec/divide-upload-package-into-two-steps | 15:01 |
katyafervent | and approve it :) | 15:01 |
sergmelikyan | :)))) | 15:01 |
ruhe | katyafervent: you need tsufiev and btully to discuss this BP. don't you? | 15:04 |
openstackgerrit | Timur Sufiev proposed a change to stackforge/murano-dashboard: Introduces script for updating horizon config https://review.openstack.org/101578 | 15:06 |
katyafervent | ruhe, we already did | 15:09 |
katyafervent | and Brian's opinion is the same as ours | 15:10 |
katyafervent | do you know which script manages running py27 on murano-dashboard? | 15:11 |
sergmelikyan | katyafervent, I agree with btully | 15:11 |
katyafervent | and why I have different result from local run and from jenkins | 15:11 |
ruhe | katyafervent: ok. can you give more details (in BP specification) about what projects will be affected (client, dashboard, api, docs)? | 15:11 |
*** gokrokv__ has quit IRC | 15:11 | |
ruhe | katyafervent: which command? | 15:12 |
sergmelikyan | katyafervent, https://github.com/stackforge/murano-dashboard/blob/master/tox.ini#L6 | 15:12 |
*** gokrokve has joined #murano | 15:12 | |
ruhe | katyafervent: what's the difference between local and jenkins reports? | 15:13 |
katyafervent | ruhe, done. only murano-dashboard will be changed. BTW do we have separate place where this information should be added? | 15:13 |
ruhe | katyafervent: you can just add that information to the blueprint text | 15:13 |
katyafervent | tox -epy27 succeed locally and failed here http://logs.openstack.org/78/101578/8/check/gate-murano-dashboard-python27/9c7a16d/console.html | 15:14 |
katyafervent | this is something wrong with environment | 15:14 |
katyafervent | I guess run_tests.sh is not used right? | 15:14 |
sjmc7 | katyafervent - no | 15:17 |
sjmc7 | oh.. for dashboard? | 15:18 |
sjmc7 | is there an error in the file/ | 15:19 |
katyafervent | yes, dashboard | 15:19 |
katyafervent | no, it's something with django environment or imports | 15:20 |
katyafervent | /usr/local/jenkins/slave_scripts/run-unittests.sh thats from log. where i can find run-unittests.sh ? | 15:22 |
ruhe | infra uses "tox -e py26" and "tox -e py27" to run unit tests | 15:22 |
katyafervent | same result | 15:23 |
katyafervent | btw, how murano-dashboard is getting installed (before running unit tests?) | 15:25 |
ruhe | katyafervent: i have the same error message when i run unit tests with your patch locally | 15:25 |
ruhe | katyafervent: yes, everything is installed before running the testts | 15:25 |
openstackgerrit | Igor Yozhikov proposed a change to stackforge/murano-deployment: Refactored RabbitMQ.py and scripts https://review.openstack.org/102513 | 15:34 |
katyafervent | reproduced | 15:36 |
katyafervent | why http://tarballs.openstack.org/horizon/horizon-stable-icehouse.tar.gz | 15:36 |
katyafervent | never mind | 15:36 |
tsufiev | ruhe, I agree with https://blueprints.launchpad.net/murano/+spec/divide-upload-package-into-two-steps | 15:37 |
tsufiev | though I think that katyafervent could elaborate it a bit | 15:37 |
*** lazy_prince has quit IRC | 15:38 | |
ruhe | katyafervent: can you speak with tsufiev and discuss where your BP should be elaborated? | 15:38 |
tsufiev | because some time ago we spoke that blueprints should allow less freedom in how they are implemented | 15:38 |
*** killer_prince has joined #murano | 15:39 | |
*** killer_prince is now known as lazy_prince | 15:39 | |
openstackgerrit | Igor Yozhikov proposed a change to stackforge/murano-deployment: Refactored RabbitMQ.py and scripts https://review.openstack.org/102513 | 15:39 |
tsufiev | ruhe, i'll change bp's summary a bit | 15:41 |
*** btully has joined #murano | 15:42 | |
katyafervent | ruhe, ok | 15:43 |
tsufiev | ruhe, katyafervent: i've edited summary for https://blueprints.launchpad.net/murano/+spec/divide-upload-package-into-two-steps Now it clearly defines the implementation | 15:50 |
tsufiev | katyafervent, i've solved the mysterious case of tox refusal | 15:59 |
tsufiev | katyafervent, muranodashboard.test.settings references muranodashboard.settings which was deleted in that commit. But ImportError for muranodashboard.settings is hidden with ImportError for muranodashboardj.test.settings | 16:00 |
tsufiev | need to remember that if python fails to import some module deep below, you'll see only the ImportError for the topmost module in the chain | 16:01 |
tsufiev | so, you need first to generate muranodashboard.setting for tox to succeed | 16:01 |
tsufiev | *muranodashboard.settings | 16:01 |
*** chandan_kumar has quit IRC | 16:02 | |
tsufiev | I suspect that it still exists on your local machine and absent in git - so you didn't reproduce that error | 16:02 |
sjmc7 | ruhe - if someone else makes edits to a change under review, the original author can just fetch it from the url in gerrit, right? | 16:11 |
ruhe | sjmc7: correct | 16:11 |
sjmc7 | gonna help ankur out, but don't want to cause him a git mess to untangle | 16:11 |
openstackgerrit | Steve McLellan proposed a change to stackforge/murano: Add optional fields to packages for supplier info https://review.openstack.org/99465 | 16:14 |
sjmc7 | is ekaterina chernova online? | 16:15 |
ruhe | sjmc7: that's katyafervent | 16:17 |
ruhe | ekaterina = katya in russian :) | 16:17 |
igormarnat | sjmc7, I think she went home for now | 16:18 |
sjmc7 | yeah.. the last name confused me. in that case, katyafervent - can you take a look at the comment and changes i just made to on https://review.openstack.org/#/c/99465/ ? | 16:18 |
igormarnat | =kate | 16:18 |
sjmc7 | sergmelikyan - for your stack deletion change | 16:21 |
sjmc7 | i was going to do the same thing but stan suggested that wasn't the right fix | 16:21 |
sjmc7 | your way seems more natural than relying on a re-deploy | 16:21 |
sjmc7 | but i still think deletion has to happen in the same model as creation - schedule the job, monitor til it's finished | 16:23 |
openstackgerrit | Igor Yozhikov proposed a change to stackforge/murano-deployment: Refactored RabbitMQ.py and scripts https://review.openstack.org/102513 | 16:32 |
*** gokrokve has quit IRC | 16:38 | |
*** gokrokve has joined #murano | 16:38 | |
*** gokrokve has quit IRC | 16:42 | |
sjmc7 | katyafervent & ruhe - for https://review.openstack.org/#/c/99177/, maybe also take a look at the latest changeset for https://review.openstack.org/#/c/99465/ | 17:07 |
sjmc7 | i added tests/fake_v1_package which generates a package in memory; it might be possible to do something similar for that client change | 17:08 |
ruhe | sjmc7: that's a good idea | 17:08 |
sjmc7 | but i agree with ruslan, test fixtures should be as minimal as possible to test whatever they're testing | 17:09 |
sjmc7 | in this case it maybe does make sense to have real files since that's what's under test | 17:09 |
*** chandan_kumar has joined #murano | 17:41 | |
openstackgerrit | Igor Yozhikov proposed a change to stackforge/murano-deployment: Refactored RabbitMQ.py and scripts https://review.openstack.org/102513 | 18:05 |
*** tbh has joined #murano | 18:06 | |
tbh | I am new to murano. I have installed murano using devstack. While deploying an environment, I got an error in murano-engine as "NoClassFound: io.murano.Environment no class found" | 18:08 |
tbh | can anyone help me? | 18:10 |
tbh | I got same error like this https://bugs.launchpad.net/fuel/+bug/1312272 | 18:10 |
tbh | but I want solution in murano not in fule | 18:10 |
tbh | *fule | 18:10 |
sjmc7 | tbh - the built in packages are packaged with the source. from the service directory (/opt/stack or wherever) try: | 18:19 |
sjmc7 | python murano/cmd/manage.py import-package meta/io.murano | 18:19 |
sjmc7 | if your config file is not in /etc/murano/murano.conf you'll need to add the --config-file argument to manage.py | 18:19 |
*** openstackgerrit has quit IRC | 18:20 | |
tbh | sjmc7: you mean "python murano/cmd/manage.py import-package meta/io.murano" ? | 18:21 |
sjmc7 | yep | 18:22 |
ruhe | this code should've trigger the initial import http://git.openstack.org/cgit/stackforge/murano/tree/contrib/devstack/lib/murano#n175 | 18:23 |
ruhe | i wonder why it didn't | 18:23 |
sjmc7 | did you install with devstack, tbh? | 18:24 |
tbh | sjmc7: yes | 18:25 |
sjmc7 | hmm | 18:25 |
sjmc7 | it may be an error in your application file | 18:25 |
sjmc7 | we're currently working on making error reporting better | 18:26 |
sjmc7 | can you put any class definitions in pastebin or somewhere? | 18:26 |
tbh | sjmc7: actually I followed the instructions given here https://github.com/stackforge/murano/tree/master/contrib/devstack | 18:27 |
tbh | currently I dont have error report now | 18:28 |
tbh | sjmc7: application file? | 18:28 |
sjmc7 | what operation is generating the error? | 18:28 |
tbh | sjmc7: when I am deploying an environment | 18:30 |
sjmc7 | with any components in it? | 18:30 |
tbh | sjmc7: telent | 18:32 |
tbh | and tomcat also | 18:32 |
tbh | sjmc7: from https://github.com/murano-project/murano-app-incubator | 18:32 |
sjmc7 | are you able to log in as an administrator to the console? | 18:33 |
tbh | sjmc7: you mean using python-muranoclient? | 18:34 |
sjmc7 | horizon | 18:34 |
sjmc7 | or with python-muranoclient | 18:34 |
sjmc7 | there's a command package-list in python-muranoclient that will list installed packages | 18:35 |
tbh | sjmc7:yeah but it doesn't show anything | 18:35 |
tbh | I tried it before | 18:35 |
sjmc7 | in Horizon, you can see them in murano/manage/package definitions | 18:35 |
tbh | their I can see core-library | 18:35 |
iyozhikov | I made some changes for dashboard ci tests, they became green. I mean jenkins job start test after successful deployment of the components. | 18:36 |
sjmc7 | tbh - you don't see the telnet or tomcat packages? | 18:36 |
tbh | sjmc7: yeah I can see that | 18:37 |
tbh | sjmc7: I tried to deploy using those applications | 18:38 |
sjmc7 | ruhe - are there any problems you guys know about with those packages at the moment? | 18:38 |
*** openstackgerrit has joined #murano | 18:39 | |
ruhe | sergey murashov uses those apps for murano-ci tests. and deployment of these apps works in murano-ci (on ubuntu slaves) | 18:39 |
ruhe | tbh: can you post murano logs somewhere? | 18:40 |
iyozhikov | tbh, have you prepare linux image with murano-agent? | 18:40 |
ruhe | tbh: paste.openstack.org for instance | 18:40 |
tbh | sjmc7: can we have documentation of murano like how it works behind rhe scenes? | 18:40 |
sjmc7 | tbh - yes, we need more documentation | 18:41 |
tbh | ruhe: sry I dont have it now | 18:41 |
tbh | iyozhikov: yeah I installed linux with murano-agent | 18:41 |
*** openstackstatus has joined #murano | 18:41 | |
*** ChanServ sets mode: +v openstackstatus | 18:41 | |
sjmc7 | and it is being worked on | 18:41 |
tbh | I mean in guest vn | 18:41 |
tbh | sjmc7: oh okay | 18:41 |
iyozhikov | tbh, i believe you made it using - http://murano.readthedocs.org/en/latest/image_builders/linux.html#guest-vm-linux-os-preparation | 18:44 |
*** drupalmonkey has quit IRC | 18:46 | |
*** rakhmerov has quit IRC | 18:46 | |
*** chandan_kumar has quit IRC | 18:46 | |
*** btully has quit IRC | 18:46 | |
*** sjmc7 has quit IRC | 18:46 | |
*** sergmelikyan has quit IRC | 18:46 | |
*** tnurlygayanov has quit IRC | 18:46 | |
*** openstackgerrit has quit IRC | 18:46 | |
*** smurashov has quit IRC | 18:46 | |
*** tbh has quit IRC | 18:46 | |
*** openstackgerrit has joined #murano | 18:46 | |
*** chandan_kumar has joined #murano | 18:46 | |
*** btully has joined #murano | 18:46 | |
*** sjmc7 has joined #murano | 18:46 | |
*** drupalmonkey has joined #murano | 18:46 | |
*** sergmelikyan has joined #murano | 18:46 | |
*** tnurlygayanov has joined #murano | 18:46 | |
*** rakhmerov has joined #murano | 18:46 | |
*** smurashov has joined #murano | 18:46 | |
*** peoplemerge has quit IRC | 18:48 | |
*** peoplemerge has joined #murano | 18:48 | |
*** openstackgerrit has quit IRC | 18:51 | |
*** openstackgerrit has joined #murano | 18:52 | |
*** drupalmonkey has quit IRC | 18:56 | |
*** rakhmerov has quit IRC | 18:56 | |
*** drupalmonkey has joined #murano | 19:19 | |
*** rakhmerov has joined #murano | 19:19 | |
*** TravT has joined #murano | 19:47 | |
*** TravT has quit IRC | 20:12 | |
*** TravT has joined #murano | 20:13 | |
*** asalkeld has joined #murano | 20:51 | |
*** drupalmonkey has quit IRC | 21:07 | |
*** chandan_kumar has quit IRC | 22:24 | |
openstackgerrit | Steve McLellan proposed a change to stackforge/python-muranoclient: Allow deletion of multiple environments https://review.openstack.org/102374 | 22:43 |
openstackgerrit | Ankur Rishi proposed a change to stackforge/murano: Add optional fields to packages for supplier info https://review.openstack.org/99465 | 22:53 |
*** asalkeld has left #murano | 23:27 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!