*** rrecio has quit IRC | 00:06 | |
*** cheneydc has joined #openstack-mistral | 00:59 | |
*** toddjohn has joined #openstack-mistral | 01:53 | |
*** toddjohn has quit IRC | 01:58 | |
*** gyee has quit IRC | 02:59 | |
rakhmerov | szaher: hi | 04:18 |
---|---|---|
*** vishwanathj has joined #openstack-mistral | 04:27 | |
*** vishwanathj has quit IRC | 04:36 | |
kong | rakhmerov: hi, have you done the refactoring job? | 04:46 |
rakhmerov | kong: hi! | 04:47 |
rakhmerov | yes | 04:47 |
* kong wants to make sure before get off work | 04:47 | |
kong | rakhmerov: ok, cool | 04:47 |
rakhmerov | there's still some places but we'll be fixing them step by step | 04:47 |
kong | i am planing to continue the event trigger bp | 04:47 |
kong | rakhmerov: all right | 04:47 |
rakhmerov | yep, I was going to ask you about it ) | 04:47 |
kong | hah | 04:47 |
rakhmerov | how it is going | 04:47 |
rakhmerov | :) | 04:47 |
kong | good, thanks | 04:48 |
kong | how abou you | 04:48 |
kong | I was moving house those days, just finished | 04:48 |
kong | rakhmerov: I have to go catch the bug, will be here in the evening | 04:49 |
kong | s/bug/bus | 04:49 |
rakhmerov | ok | 04:49 |
rakhmerov | ddeja: hi, are your patches ready for review? | 04:50 |
rakhmerov | I mean RPC stuff | 04:50 |
openstackgerrit | hardik proposed openstack/mistral: Release note for murano actions support https://review.openstack.org/329763 | 05:09 |
openstackgerrit | Merged openstack/mistral: Add magnum certificates and mservices actions https://review.openstack.org/329042 | 05:09 |
openstackgerrit | Merged openstack/mistral: Rename the to_string method to to_json to clarify it's purpose https://review.openstack.org/327889 | 05:28 |
openstackgerrit | Merged openstack/python-mistralclient: Add osprofiler option to trace operations https://review.openstack.org/328045 | 05:29 |
*** openstackgerrit has quit IRC | 06:18 | |
*** openstackgerrit has joined #openstack-mistral | 06:18 | |
*** toddjohn has joined #openstack-mistral | 06:31 | |
*** toddjohn has quit IRC | 06:36 | |
mflobo | kong, should I add a new relese note here https://github.com/openstack/mistral/tree/master/releasenotes/notes ? | 06:37 |
hparekh | mflobo, Hi you can use reno tool. make sure you are in root mistral (e.g. /opt/stack/mistral) and use reno new <note name> | 06:42 |
hparekh | mflobo, It will create note at at releasenotes/notes directory | 06:43 |
openstackgerrit | hardik proposed openstack/mistral: Added doc string for enforce method https://review.openstack.org/329776 | 06:46 |
*** toddjohn has joined #openstack-mistral | 06:59 | |
*** toddjohn has quit IRC | 07:04 | |
*** FL1SK has quit IRC | 07:09 | |
openstackgerrit | hardik proposed openstack/mistral: Added doc string for enforce method https://review.openstack.org/329776 | 07:12 |
*** mgershen has joined #openstack-mistral | 07:24 | |
*** shardy has joined #openstack-mistral | 07:24 | |
*** openstackgerrit has quit IRC | 07:33 | |
*** openstackgerrit has joined #openstack-mistral | 07:33 | |
*** shardy has quit IRC | 07:43 | |
ddeja | rakhmerov: Hi - they are working, but I added note to config that using kombu is experimetnal - I'm writing tests for it | 08:04 |
ddeja | and It would be good for someone else to test it before merging ;) | 08:04 |
*** FL1SK has joined #openstack-mistral | 08:06 | |
*** shardy has joined #openstack-mistral | 08:10 | |
ddeja | In general - they are ready for review. And I didn't change a lot - rebasing was unexpectedly easy | 08:12 |
rakhmerov | ddeja: ok, I see | 08:13 |
rakhmerov | ddeja: did you validate the logic and everything? | 08:13 |
rakhmerov | I mean are you ok with the implementation that Nikolay provided? | 08:14 |
rakhmerov | I'm asking because I remember that there was a lot to discuss on that | 08:14 |
rakhmerov | and another question: are you planning to work on Pika impl soon? | 08:15 |
mflobo | hparekh, I'm not using reno tool (I don't know it BTW), so I'll create the YAML file and send the review | 08:15 |
ddeja | rakhmerov: generally I'm ok. I may found something while writing tests, but right now I don't see a lot of place for improvements | 08:17 |
ddeja | about pika, I was thinking to add it in separate patch when this one is merged | 08:18 |
ddeja | but I can add it to current patchset if that would be better | 08:18 |
rakhmerov | ddeja: ok | 08:20 |
rakhmerov | yes, I didn't mean to have it in these patches ) | 08:20 |
rakhmerov | ddeja: no, it's definitely a separate patch | 08:20 |
rakhmerov | mflobo: hi, please use reno | 08:21 |
rakhmerov | it also creates a unique id for a release note | 08:21 |
rakhmerov | reno new <note name> | 08:21 |
mflobo | rakhmerov, is there RPM for CentOS? or I must clone the repo | 08:22 |
*** dmk0202 has joined #openstack-mistral | 08:22 | |
rakhmerov | RPM for what? | 08:22 |
rakhmerov | just clone the repo, sure | 08:22 |
*** toddjohn has joined #openstack-mistral | 08:32 | |
openstackgerrit | Xavier Hardy proposed openstack/mistral: Fix API inconsistencies with GET /v2/workflows https://review.openstack.org/321016 | 09:14 |
*** mgershen has quit IRC | 09:16 | |
*** toddjohn has quit IRC | 09:16 | |
*** mgershen has joined #openstack-mistral | 09:16 | |
*** mgershen has quit IRC | 09:21 | |
kong | mflobo: please take a look at http://docs.openstack.org/developer/reno/design.html | 09:39 |
dmellado | mflobo: you can just install it using pip | 09:42 |
kong | mflobo: in one word, pip install reno, go to mistral dir, run 'reno new magnum-actions-support' | 09:43 |
kong | you can see a new file created under releasenotes/notes/ | 09:43 |
kong | modify that file. you can see other files as reference | 09:44 |
mflobo | kong, dmellado, thanks for the info, I'll do this today | 09:44 |
kong | mflobo: thanks for that | 09:44 |
*** mgershen has joined #openstack-mistral | 09:45 | |
kong | rakhmerov: the new code looks awesome! | 09:46 |
kong | good job, man | 09:46 |
dmellado | kong: now that I got you here I was pinging hparekh the other day about the tempest plugin for mistral | 09:48 |
dmellado | IMHO the tests, even if bundled with mistral | 09:48 |
dmellado | should follow the setup steps | 09:49 |
dmellado | and have a skip_checks in their base class | 09:49 |
dmellado | was there any reason for not having this? | 09:49 |
kong | dmellado: what's your meaning by 'skip_checks'? what kind of tests do you want to skip? | 09:50 |
dmellado | kong: as per another projects' tempest tests | 09:50 |
dmellado | there's a part on the setup steps | 09:51 |
dmellado | that skips the tests if the service isn't availble | 09:51 |
kong | yeah | 09:51 |
dmellado | in this case, mistral xD | 09:51 |
dmellado | so as per RDO | 09:53 |
kong | do you mean, we should add skip_checks in test_openstack_actions.py? | 09:54 |
kong | I didn't see other locations we should skip tests | 09:54 |
kong | could you please provide me more detail? | 09:55 |
openstackgerrit | Merged openstack/mistral: Added doc string for enforce method https://review.openstack.org/329776 | 09:57 |
dmellado | kong: sorry, got pulled | 10:11 |
kong | :-) | 10:11 |
rakhmerov | kong: thanks :) | 10:12 |
dmellado | kong: I was thinking about something like having that here maybe https://github.com/openstack/mistral/blob/master/mistral_tempest_tests/services/base.py | 10:13 |
dmellado | as the tests class inherits from there | 10:13 |
d0ugal | So, I am still learning YAQL, what is wrong with this? <% $.status or 'SUCCESS' %> | 10:13 |
kong | dmellado: I just want to try to understand what problems you want to fix | 10:14 |
dmellado | kong: in RDO, we split the package mistral between mistral and mistral-tests | 10:14 |
dmellado | but when we install the *test package and not mistral due to dependencies | 10:15 |
dmellado | it fails the tests as it tries to run them but the service isn't available | 10:15 |
dmellado | a simple skip if CONF.services_availble.mistral | 10:15 |
dmellado | should fix that | 10:15 |
kong | dmellado: hmm...I may know your problem | 10:17 |
dmellado | kong: we can always skip that but I think having mistral implement a skip decorator would be cleaner | 10:18 |
kong | dmellado: I wonder if mistral is the only project we see that problem? | 10:20 |
kong | how do you deal with others? | 10:20 |
dmellado | kong: every other project implements that skip decorator | 10:21 |
dmellado | so so far, mistral is the only one that I'm having to deal with in this way ;) | 10:21 |
* kong is looking at Murano | 10:21 | |
dmellado | kong: let me show you an example | 10:21 |
kong | dmellado: appreciate | 10:22 |
kong | I saw that here https://github.com/openstack/murano/blob/master/murano_tempest_tests/tests/api/application_catalog/base.py#L72 | 10:22 |
dmellado | kong: i.e. ceilometer | 10:22 |
dmellado | https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/tempest/api/base.py#L50-L53 | 10:22 |
dmellado | kong: yep, exactly ;) | 10:23 |
kong | dmellado: ok, that should be easy to fix | 10:23 |
dmellado | kong: I was checking the simple patch myself but it seems there's some call around I'm not familiar with | 10:23 |
kong | dmellado: did you raise a ticket for that? | 10:23 |
dmellado | kong: nope, but I'll do if you want it | 10:23 |
kong | dmellado: yes, please | 10:24 |
kong | leave the bug there, and I (or others) will do that | 10:24 |
kong | dmellado: thanks | 10:24 |
kong | feel free to assign it to me if it's convenient for you | 10:25 |
dmellado | kong: https://bugs.launchpad.net/mistral/+bug/1592748 | 10:28 |
openstack | Launchpad bug 1592748 in Mistral "Mistral tempest tests doesn't implement a skip method" [Undecided,New] | 10:28 |
dmellado | will do, thanks! | 10:28 |
dmellado | kong: I can't assign it to you in launchpad, as I'm 'not related to the project, or so it says' xD | 10:28 |
dmellado | so please pick it up ;) | 10:28 |
kong | dmellado: done | 10:29 |
dmellado | thanks! | 10:29 |
dmellado | kong: feel free to reach me when in doubts about this too ;) | 10:29 |
openstackgerrit | Xavier Hardy proposed openstack/mistral: API inconsistencies with GET /v2/workflows https://review.openstack.org/321016 | 10:58 |
openstackgerrit | Xavier Hardy proposed openstack/mistral: API inconsistencies with GET /v2/workflows https://review.openstack.org/321016 | 11:03 |
*** dprince has joined #openstack-mistral | 11:14 | |
*** hparekh has quit IRC | 11:16 | |
*** hparekh has joined #openstack-mistral | 11:19 | |
*** hparekh_ has joined #openstack-mistral | 11:19 | |
*** hparekh_ has quit IRC | 11:20 | |
szaher | Hi guys | 11:28 |
szaher | I am trying to run the get first image action from glance actions and it's not working and i am getting this error Invalid input [name=glance.images_get, class=NoneType, unexpected=[u'image_id']] | 11:28 |
kong | something wrong when getting glance action input | 11:36 |
szaher | kong: Yea I know | 11:36 |
kong | szaher: what's the output of 'mistral action-list | grep glance' in your env? | 11:36 |
kong | there is no input param | 11:36 |
szaher | there is no input for glance.get_images | 11:36 |
szaher | | glance.images_get | True | | None | <none> | 2016-06-14 11:25:28 | None | | 11:37 |
kong | I saw that. maybe there is lack of testing towards openstack service integrations | 11:37 |
kong | szaher: please fire a bug | 11:38 |
szaher | this is liberty release ! | 11:38 |
kong | i am in master | 11:39 |
szaher | any idea how can I solve this ? | 11:40 |
kong | szaher: take a look at sync_db.py in tools/ | 11:42 |
openstackgerrit | Merged openstack/mistral: Release note for murano actions support https://review.openstack.org/329763 | 11:42 |
kong | that's the place we populate action db | 11:43 |
openstackgerrit | Marcos Fermín Lobo proposed openstack/mistral: Release noe for magnum actions support https://review.openstack.org/329894 | 12:02 |
mflobo | rakhmerov, kong, dmellado ^ | 12:02 |
openstackgerrit | Lingxian Kong proposed openstack/mistral: Add configuration for Mistral tempest testing https://review.openstack.org/329895 | 12:02 |
szaher | kong: ./sync_db.sh --config-file /opt/stack/service/mistral/etc/mistral/mistral.conf | 12:03 |
szaher | venv create: /opt/stack/venv/mistral/mistral/.tox/venv | 12:03 |
szaher | it stuck here ! | 12:03 |
dmellado | mflobo: ack, I'll have a look later ;) | 12:03 |
kong | mflobo: please take a look at https://github.com/openstack/mistral/blob/master/releasenotes/notes/new-service-actions-support-47279bd649732632.yaml, I believe adding a new service in Mistral will have some effect to upgrading. | 12:07 |
mflobo | kong, ok, I'll change the patchset, thanks | 12:08 |
kong | when upgrading Mistral, operators need to run 'python tools/sync_db.py' somehow to re-populate the db | 12:08 |
* kong needs some sleep... | 12:09 | |
szaher | kong: I tried with sync_db and still glance has no inputs ! | 12:19 |
mflobo | kong, should put the same "prelude" https://github.com/openstack/mistral/blob/master/releasenotes/notes/new-service-actions-support-47279bd649732632.yaml in mi patch? | 12:25 |
ddeja | szaher: if you are facing bug regarding some action in liberty, probably the only way would be to write our own custom action http://docs.openstack.org/developer/mistral/developer/creating_custom_action.html | 12:27 |
ddeja | szaher: here you can look at my plugin for talking with nova, you could do similar to talk with glance https://github.com/gryf/mistral-evacuate/blob/master/filter_vm_action.py | 12:28 |
openstackgerrit | Marcos Fermín Lobo proposed openstack/mistral: Release noe for magnum actions support https://review.openstack.org/329894 | 12:29 |
szaher | ddeja: there is a problem here. when I tried to install mistral liberty version the installed packages versions were exceeding the upper constraints for liberty !! | 12:56 |
szaher | that might be a problem | 12:56 |
szaher | I tried to add upper.txt and install the max available version for liberty it didn't work ! | 12:57 |
ddeja | szaher: but you do install mistral in liberty version, right? | 12:58 |
*** rbrady has joined #openstack-mistral | 12:59 | |
*** dprince has quit IRC | 13:02 | |
szaher | yes | 13:03 |
szaher | szaher: and I was checking against upper constraints liberty | 13:03 |
mflobo | rakhmerov, I guess this blueprint can be set as finished https://blueprints.launchpad.net/mistral/+spec/add-policy-file | 13:09 |
szaher | ddeja: after installing with upper constraints actions are not created and I got this error message http://paste.openstack.org/show/516250/ | 13:14 |
*** bobh has joined #openstack-mistral | 13:22 | |
d0ugal | Why would you ever want to run this without a transaction? https://github.com/openstack/mistral/blob/master/mistral/services/actions.py#L39 | 13:31 |
ddeja | szaher: hm, thats strange that you have to apply upper-constriants... how did you install mistral? | 13:42 |
szaher | git clone mistral_url | 13:43 |
szaher | cd mistral | 13:43 |
szaher | git checkout origin/stable/liberty | 13:43 |
szaher | pip install -e . | 13:43 |
szaher | ddeja: that's how I install mistral | 13:45 |
*** dprince has joined #openstack-mistral | 13:59 | |
ddeja | szaher: On my PC 'git checkout origin/stable/liberty' didn't really checkout me to liberty branch, but rather created new branch (so I am still on master), but this may be some misconfiguration on my side. | 14:02 |
ddeja | szaher: It looks like installation is ok, so the only thing that I can advise is to write your own action that would connect to glance and do what you want to do | 14:03 |
ddeja | szaher: BTW. I was able to succesfully launch mitaka mistral to work with kilo openstack, so maybe you can try to use newer version of mistral. Therefore we could fix bugs that hit you | 14:04 |
ddeja | becouse for Liberty version it is unlikely to fix anything rather than security issue (AFAIC) | 14:05 |
*** dprince has quit IRC | 14:25 | |
*** catintheroof has joined #openstack-mistral | 14:32 | |
szaher | ddeja: Thanks! I will try mitaka version if not I will try to write my own workflows | 14:53 |
szaher | *actions | 14:53 |
*** tonytan4ever has joined #openstack-mistral | 14:58 | |
*** rrecio has joined #openstack-mistral | 14:58 | |
*** toddjohn has joined #openstack-mistral | 15:04 | |
*** catintheroof has quit IRC | 15:08 | |
*** toddjohn has quit IRC | 15:09 | |
*** rrecio has quit IRC | 15:11 | |
*** rrecio has joined #openstack-mistral | 15:15 | |
*** tonytan4ever has quit IRC | 15:24 | |
*** shardy has quit IRC | 15:35 | |
szaher | ddeja: mistral-db-manage is not working mitaka !!!. even sync_db.py ! | 15:45 |
szaher | I don't know what is wrong am I doing | 15:45 |
ddeja | szaher: you have to provide logs/traceback. Without it I'm helpless | 15:46 |
szaher | ddeja: http://paste.openstack.org/show/516304/ | 15:46 |
ddeja | szaher: that's a bug on our side... I thought that bugfix was merged to mitaka branch. To fix this, go to file mistral/engine/rpc.py and delete lines 38-79. It should fix this | 15:50 |
szaher | ddeja: thanks | 15:51 |
*** dmk0202 has quit IRC | 15:52 | |
szaher | ddeja: I think this patch needs to be backported ! | 15:54 |
ddeja | szaher: It should be. As I said, I thought it is already done | 15:55 |
szaher | ddeja: /opt/stack/venv/mistral/mistral/.tox/venv/local/lib/python2.7/site-packages/sqlalchemy/sql/default_comparator.py:153: SAWarning: The IN-predicate on "action_definitions_v2.id" was invoked with an empty sequence. This results in a contradiction, which nonetheless can be expensive to evaluate. Consider alternative strategies for improved performance. | 15:55 |
ddeja | szaher: it's only a Warning | 15:56 |
*** tonytan4ever has joined #openstack-mistral | 15:56 | |
szaher | actions table is empty | 15:56 |
szaher | the script takes long time and still waiting ... !! | 15:56 |
ddeja | szaher: it may take some time. If it's empty after it finishes then there is a problem | 15:57 |
*** toddjohn has joined #openstack-mistral | 15:59 | |
szaher | ddeja: for how long I should leave this script running ? | 16:02 |
*** toddjohn has quit IRC | 16:04 | |
ddeja | szaher: I would say that no longer than 10 minutes... did you use mistral-db-manage or sync-db script? | 16:04 |
szaher | db-manage then sync-db | 16:05 |
ddeja | szaher: if it take too long, try sync_db.py script | 16:05 |
szaher | ddeja: http://paste.openstack.org/show/516311/ any idea why this is happening ? | 16:05 |
ddeja | it's the same, but gives more output, so it's esier to debug | 16:05 |
ddeja | szaher: you got such traceback while running mistral-db-manage? | 16:06 |
szaher | No | 16:06 |
ddeja | szaher: so when? | 16:07 |
*** krotscheck is now known as krotscheck_dcm | 16:09 | |
szaher | mistral-server --conf.. | 16:09 |
szaher | glance actions has no input as well ! | 16:09 |
szaher | the same problem exists | 16:09 |
ddeja | szaher: OK, but you have used mistral action-list right? So it means that mistral server is working and when you try to launch it second time, which probably coused the tracebacks | 16:11 |
ddeja | could you remind me what exaclty action you wanted to use? | 16:11 |
szaher | the simple get first image | 16:12 |
szaher | get_image_id [task_ex_id=c5421350-256a-45c9-868e-563a341b736a] -> Failed to run action [action_ex_id=9039a9ae-b502-4643-9a84-a5d1b5b08b55, action_cls='<class 'mistral.actions.action_factory.GlanceAction'>', attributes='{u'client_method_name': u'images.list'}', params='{}'] | | 16:12 |
szaher | | | GlanceAction.images.list failed: <class 'keystoneauth1.exceptions.auth.AuthorizationFailure'>: Cannot authorize API client. | 16:12 |
*** tonytan4ever has quit IRC | 16:14 | |
ddeja | szaher: so for now it looks like you need to write your own action | 16:14 |
ddeja | szaher: and we have to fix a bug in mistral | 16:15 |
szaher | ddeja: do you think this is a problem with the action itself ? | 16:15 |
ddeja | szaher: I don't know, but if there is no input parameters for actions, that should have some, there is a bug, probably on our side | 16:15 |
ddeja | szaher: oh, but in this case there is some problem with autentication | 16:16 |
ddeja | images.list should just work... | 16:16 |
szaher | it was working in liberty version ! | 16:17 |
ddeja | szaher: please check with configuration instruction https://github.com/openstack/mistral#configuring-mistral if everything is set | 16:18 |
ddeja | reinstall may overwrite some files | 16:18 |
*** tonytan4ever has joined #openstack-mistral | 16:19 | |
szaher | ddeja: I am install mistral everytime in a new virtualenv | 16:19 |
szaher | anyway I can't find any solid documentation and I am not sure about the project stability | 16:19 |
ddeja | szaher: OK, but after upgrading from liberty to mitaka it may overwrite etc/mistral.conf and lost the keystone authtoken properties | 16:22 |
ddeja | so mistral couldn't authenticate | 16:22 |
szaher | ddeja: by the way I am keeping the configuration file in a different place | 16:24 |
szaher | It's not in a place where oslo config generator will be able to reach | 16:25 |
ddeja | szaher: oh, OK. | 16:25 |
*** tonytan4ever has quit IRC | 16:26 | |
szaher | ddeja: Thanks for your help it's really appreciated but I fed up with mistral ! | 16:26 |
ddeja | szaher: I need to drop unfortunately. I'll be reachable tommorow from about 7 UTC. | 16:27 |
ddeja | szaher: thank you for your feedback | 16:27 |
szaher | ddeja: have a good night | 16:27 |
ddeja | szaher: thanks, bye | 16:27 |
*** dprince has joined #openstack-mistral | 16:31 | |
*** tonytan4ever has joined #openstack-mistral | 17:17 | |
*** tonytan4ever has quit IRC | 18:04 | |
*** tonytan4ever has joined #openstack-mistral | 18:16 | |
*** tonytan4ever has quit IRC | 18:19 | |
*** toddjohn has joined #openstack-mistral | 18:41 | |
*** toddjohn has quit IRC | 18:46 | |
*** tonytan4ever has joined #openstack-mistral | 19:09 | |
*** tonytan4ever has quit IRC | 19:14 | |
*** dprince has quit IRC | 19:21 | |
*** openstackgerrit has quit IRC | 19:33 | |
*** openstackgerrit has joined #openstack-mistral | 19:33 | |
*** krotscheck_dcm is now known as krotscheck | 20:00 | |
*** toure has joined #openstack-mistral | 20:06 | |
*** dprince has joined #openstack-mistral | 20:07 | |
*** toure is now known as toure|biab | 20:19 | |
*** toddjohn has joined #openstack-mistral | 20:24 | |
*** toddjohn has quit IRC | 21:03 | |
*** toddjohn has joined #openstack-mistral | 21:04 | |
*** toddjohn has quit IRC | 21:09 | |
*** dprince has quit IRC | 21:13 | |
*** gyee has joined #openstack-mistral | 21:42 | |
*** bobh has quit IRC | 21:58 | |
*** openstackgerrit has quit IRC | 22:02 | |
*** catintheroof has joined #openstack-mistral | 22:04 | |
*** openstackgerrit has joined #openstack-mistral | 22:06 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral: Fix postgresql test failure https://review.openstack.org/330250 | 22:28 |
*** catintheroof has quit IRC | 22:41 | |
*** toddjohn has joined #openstack-mistral | 22:43 | |
*** toddjohn has quit IRC | 22:47 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!