| *** shardy has quit IRC | 00:41 | |
| *** shardy has joined #openstack-mistral | 00:41 | |
| *** thrash is now known as thrash|g0ne | 01:07 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 05:48 | |
| *** Alexey_Abashkin has joined #openstack-mistral | 05:54 | |
| *** opetrenko_ has quit IRC | 05:54 | |
| *** opetrenko_ has joined #openstack-mistral | 05:55 | |
| *** AlexeyAbashkin has quit IRC | 05:55 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 05:55 | |
| *** jaosorior has quit IRC | 06:58 | |
| *** d0ugal has joined #openstack-mistral | 07:03 | |
| *** Alexey_Abashkin has joined #openstack-mistral | 07:23 | |
| *** jistr is now known as jistr|reloc | 07:25 | |
| *** AlexeyAbashkin has quit IRC | 07:26 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 07:26 | |
| d0ugal | Morning! | 07:58 |
|---|---|---|
| *** shardy has quit IRC | 07:59 | |
| d0ugal | #startmeeting mistral | 08:00 |
| openstack | Meeting started Fri Jun 8 08:00:01 2018 UTC and is due to finish in 60 minutes. The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:00 |
| *** openstack changes topic to " (Meeting topic: mistral)" | 08:00 | |
| openstack | The meeting name has been set to 'mistral' | 08:00 |
| d0ugal | Who is around for the Mistral office hour today? https://etherpad.openstack.org/p/mistral-office-hours | 08:00 |
| d0ugal | rakhmerov, apetrich, bobh, mcdoker181818 ^ | 08:00 |
| d0ugal | As usual, add your IRC nick to line 16 in the etherpad if you want a reminder ping in the future! | 08:01 |
| *** shardy has joined #openstack-mistral | 08:01 | |
| d0ugal | I am going to be proposing the Rocky Milestone 2 releases today. | 08:03 |
| *** jpich has joined #openstack-mistral | 08:05 | |
| *** gkadam has joined #openstack-mistral | 08:06 | |
| *** AlexeyAbashkin has quit IRC | 08:06 | |
| d0ugal | and by today, I mean "now" :) | 08:10 |
| d0ugal | https://review.openstack.org/573610 | 08:10 |
| *** shardy has quit IRC | 08:13 | |
| *** jistr|reloc is now known as jistr | 08:19 | |
| therve | d0ugal: Is it a good time to chat about https://review.openstack.org/572807 ? | 08:27 |
| d0ugal | therve: Sure | 08:27 |
| d0ugal | I'm just doing milestone 2 administrivia | 08:28 |
| therve | d0ugal: https://review.openstack.org/573306 I think is the fix, hopefully that'll be confirmed | 08:28 |
| therve | But it requires a heatclient release, so I was wondering if we can find a workaround | 08:28 |
| *** AlexeyAbashkin has joined #openstack-mistral | 08:28 | |
| d0ugal | Right | 08:29 |
| d0ugal | Do we know where it is called from? | 08:29 |
| therve | Yes, serialization code | 08:29 |
| d0ugal | That is what I suspected. Got a link to relevant line handy? | 08:30 |
| therve | Hum I don't have the full traceback anymore | 08:30 |
| therve | https://github.com/openstack/mistral/blob/master/mistral/serialization.py#L106 was the one in mistral I remember | 08:31 |
| therve | Then it got in jsonutils | 08:31 |
| *** shardy has joined #openstack-mistral | 08:31 | |
| d0ugal | Right | 08:32 |
| d0ugal | therve: I guess a workaround would be for the mistral actions to set loaded=True? | 08:32 |
| therve | d0ugal: Hum maybe. Just the get_stack one though | 08:33 |
| d0ugal | Right | 08:33 |
| d0ugal | It would be a fairly nasty hack, but I don't have any other immediate ideas | 08:33 |
| therve | d0ugal: So there is no way in the workflow to operate on the result of the action before it's serialized? | 08:34 |
| d0ugal | therve: No, I don't think so | 08:34 |
| therve | Right | 08:35 |
| therve | In which case the client release is probably the best way then | 08:35 |
| d0ugal | Yeah | 08:36 |
| d0ugal | I guess my work around in the workflow would be good enough for now? | 08:36 |
| d0ugal | Until the release I mean | 08:37 |
| therve | I guess | 08:37 |
| therve | It doesn't prevent the double request | 08:37 |
| d0ugal | Well, it means we have one slow inefficient request :) | 08:37 |
| d0ugal | but it does mean we store that output in the action execution table in mistral. | 08:38 |
| openstackgerrit | Merged openstack/mistral-extra master: fix tox python3 overrides https://review.openstack.org/572969 | 08:47 |
| openstackgerrit | Merged openstack/mistral-dashboard master: fix tox python3 overrides https://review.openstack.org/572968 | 08:52 |
| *** Alexey_Abashkin has joined #openstack-mistral | 09:00 | |
| *** d0ugal_ has joined #openstack-mistral | 09:00 | |
| *** d0ugal has quit IRC | 09:00 | |
| *** d0ugal_ has quit IRC | 09:00 | |
| *** AlexeyAbashkin has quit IRC | 09:01 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 09:01 | |
| *** d0ugal has joined #openstack-mistral | 09:01 | |
| d0ugal | #endmeeting | 09:01 |
| *** openstack changes topic to " (Meeting topic: test)" | 09:01 | |
| openstack | Meeting ended Fri Jun 8 09:01:10 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-06-08-08.00.html | 09:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-06-08-08.00.txt | 09:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-06-08-08.00.log.html | 09:01 |
| d0ugal | Sorry if I missed anything at the end there. My laptop decided it was a good time to turn off. | 09:01 |
| openstackgerrit | Merged openstack/python-mistralclient master: fix tox python3 overrides https://review.openstack.org/573343 | 09:31 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral stable/queens: Register the collect_timing keystone_authtoken option https://review.openstack.org/573642 | 09:31 |
| *** jaosorior has joined #openstack-mistral | 09:36 | |
| *** thrash|g0ne is now known as thrash | 10:05 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral-dashboard master: Add the restructuredtext check to the flake8 job https://review.openstack.org/573187 | 10:30 |
| openstackgerrit | Dougal Matthews proposed openstack/python-mistralclient master: Add the restructuredtext check to the flake8 job https://review.openstack.org/573191 | 10:32 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral-extra master: Add the restructuredtext check to the flake8 job https://review.openstack.org/573189 | 10:32 |
| *** shardy has quit IRC | 10:46 | |
| *** shardy has joined #openstack-mistral | 10:48 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Add missing query paramater `scope` to the workbook api https://review.openstack.org/555030 | 11:16 |
| *** shardy has quit IRC | 11:19 | |
| *** shardy has joined #openstack-mistral | 11:20 | |
| *** mcdoker1818 has joined #openstack-mistral | 11:52 | |
| *** AlexeyAbashkin has quit IRC | 11:53 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 11:54 | |
| mcdoker1818 | d0ugal: Hi, why didn't you add the test method to Action interface? | 11:56 |
| mcdoker1818 | https://github.com/openstack/mistral/blob/master/mistral/actions/base.py | 11:56 |
| mcdoker1818 | https://github.com/openstack/mistral-lib/blob/master/mistral_lib/actions/base.py | 11:57 |
| *** AlexeyAbashkin has quit IRC | 12:12 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 12:12 | |
| d0ugal | mcdoker1818: because it doesn't do anything and isn't used :) | 12:20 |
| d0ugal | but you can of course add it if you need it for something | 12:20 |
| d0ugal | I wanted to keep it as simple as possible. | 12:20 |
| openstackgerrit | Kim Bao Long proposed openstack/mistral master: Remove NotImplementedError exception https://review.openstack.org/573679 | 12:23 |
| *** AlexeyAbashkin has quit IRC | 12:30 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 12:33 | |
| mcdoker1818 | @d0ugal I want to use this method for stub creation. | 12:34 |
| mcdoker1818 | Every test method of action will return stub data. And we have a task property if it equals true then we execute test method but not run. | 12:34 |
| mcdoker1818 | We have a workflow with 500 tasks. Every execution of this workflow takes long time and HW | 12:34 |
| mcdoker1818 | I think about http servier-stub for openstack or test stub method to make life of a workflow writer easy. | 12:35 |
| mcdoker1818 | How do you test a your workflows? Do you always use a real openstack environment | 12:35 |
| d0ugal | mcdoker1818: yes, we test it in a OpenStack environment | 12:35 |
| d0ugal | mcdoker1818: You can still add a test method to your actions if you want it. | 12:36 |
| d0ugal | mcdoker1818: https://review.openstack.org/#/c/553389/ | 12:36 |
| d0ugal | I experimented recently by unit testing workflows. | 12:37 |
| d0ugal | It works, but it is a little messy. | 12:37 |
| d0ugal | Also, be warned that it uses private Mistral internals - so there is a chance they will change. | 12:37 |
| *** AlexeyAbashkin has quit IRC | 12:41 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 12:41 | |
| mcdoker1818 | https://blueprints.launchpad.net/mistral/+spec/mistral-dry-run | 12:41 |
| mcdoker1818 | :d0ugal I thought about something like that. | 12:46 |
| d0ugal | I know that blueprint, I think that is why some of the actions have test methods but I don't think it was ever used | 12:47 |
| d0ugal | I don't believe in adding code that isn't used yet :) | 12:47 |
| mcdoker1818 | :D | 12:48 |
| *** ninag has joined #openstack-mistral | 12:54 | |
| *** ninag has quit IRC | 12:54 | |
| *** AlexeyAbashkin has quit IRC | 12:56 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 12:57 | |
| *** d0ugal_ has joined #openstack-mistral | 13:08 | |
| *** d0ugal has quit IRC | 13:08 | |
| *** d0ugal_ has quit IRC | 13:08 | |
| *** d0ugal has joined #openstack-mistral | 13:08 | |
| *** d0ugal has quit IRC | 13:08 | |
| *** d0ugal has joined #openstack-mistral | 13:08 | |
| *** jaosorior has quit IRC | 13:29 | |
| *** mcdoker1818 has quit IRC | 13:59 | |
| *** mmethot_ has joined #openstack-mistral | 14:17 | |
| *** shardy has quit IRC | 14:18 | |
| *** mmethot has quit IRC | 14:20 | |
| *** shardy has joined #openstack-mistral | 14:21 | |
| *** AlexeyAbashkin has quit IRC | 14:21 | |
| *** mmethot_ has quit IRC | 14:22 | |
| *** jistr is now known as jistr|mtg | 14:28 | |
| *** mmethot has joined #openstack-mistral | 14:51 | |
| *** shardy has quit IRC | 15:02 | |
| openstackgerrit | Dougal Matthews proposed openstack/python-mistralclient master: Add --debug to get the full traceback https://review.openstack.org/573736 | 15:03 |
| d0ugal | thrash: ^ trying to figure out that nonetype error. | 15:03 |
| thrash | d0ugal: ack | 15:03 |
| thrash | d0ugal: will watch | 15:03 |
| d0ugal | thrash: the check queue is kinda huge, so it might take a while :) | 15:06 |
| d0ugal | Maybe a result for after your PTO | 15:06 |
| thrash | lol | 15:07 |
| *** d0ugal has quit IRC | 15:14 | |
| *** jistr|mtg is now known as jistr | 15:16 | |
| *** thrash is now known as thrash|biab | 15:34 | |
| *** jpich has quit IRC | 16:46 | |
| *** jtomasek has quit IRC | 16:52 | |
| *** mcdoker1818 has joined #openstack-mistral | 17:08 | |
| *** gkadam has quit IRC | 17:11 | |
| *** thrash|biab is now known as thrash | 17:24 | |
| *** mcdoker1818 has quit IRC | 20:52 | |
| *** harlowja has joined #openstack-mistral | 22:30 | |
| openstackgerrit | Merged openstack/mistral stable/queens: Register the collect_timing keystone_authtoken option https://review.openstack.org/573642 | 22:40 |
| openstackgerrit | Merged openstack/mistral-lib master: fix tox python3 overrides https://review.openstack.org/572909 | 22:54 |
| *** mmethot_ has joined #openstack-mistral | 23:23 | |
| *** mmethot has quit IRC | 23:25 | |
| *** mmethot_ has quit IRC | 23:31 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!