| *** catintheroof has quit IRC | 00:19 | |
| *** zhurong has joined #openstack-mistral | 00:55 | |
| *** tung_doan has joined #openstack-mistral | 01:21 | |
| *** catintheroof has joined #openstack-mistral | 02:16 | |
| openstackgerrit | Merged openstack/mistral master: Remove the highlighting choice 'HTTP' https://review.openstack.org/453542 | 02:25 |
|---|---|---|
| openstackgerrit | Merged openstack/mistral master: Use the plain syntax highlighting in the webapi example https://review.openstack.org/453543 | 02:25 |
| openstackgerrit | Merged openstack/mistral master: Remove the empty using_yaql gude https://review.openstack.org/454057 | 02:25 |
| *** catintheroof has quit IRC | 02:34 | |
| *** catintheroof has joined #openstack-mistral | 02:35 | |
| *** catintheroof has quit IRC | 02:40 | |
| openstackgerrit | Merged openstack/mistral master: Use the Mistral syntax highlighting on the dsl v2 page https://review.openstack.org/454058 | 03:23 |
| *** chlong has quit IRC | 03:47 | |
| openstackgerrit | Toure Dunnon proposed openstack/mistral master: [WIP] Workflow Error Analysis https://review.openstack.org/455447 | 04:06 |
| rakhmerov | kong: hi | 04:22 |
| rakhmerov | can you pls reivew https://review.openstack.org/#/c/455327 ? | 04:22 |
| rakhmerov | it's quite important | 04:22 |
| rakhmerov | and super small | 04:23 |
| rakhmerov | d0ugal: or you | 04:23 |
| kong | rakhmerov: i'm looking | 05:01 |
| rakhmerov | thanks | 05:16 |
| openstackgerrit | Merged openstack/mistral master: Limit the number of finished executions. https://review.openstack.org/446680 | 05:24 |
| *** jaosorior_away is now known as jaosorior | 05:55 | |
| *** jtomasek_ has quit IRC | 06:31 | |
| *** kong has quit IRC | 07:39 | |
| *** jpich has joined #openstack-mistral | 07:50 | |
| *** shardy has joined #openstack-mistral | 07:53 | |
| *** Kevin_Zheng has quit IRC | 08:23 | |
| *** sharatss has joined #openstack-mistral | 08:42 | |
| xavierhardy | rakhmerov: is there anything in the pipes to make the access control available to other authentication plugins than keystone ? | 08:58 |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** sharatss has quit IRC | 09:07 | |
| *** warface has joined #openstack-mistral | 09:07 | |
| xavierhardy | rakhmerov? | 09:37 |
| *** openstackgerrit has joined #openstack-mistral | 09:49 | |
| openstackgerrit | Kevin Pouget proposed openstack/python-mistralclient master: Add CLI for event trigger operations https://review.openstack.org/454669 | 09:49 |
| *** mgershen has joined #openstack-mistral | 10:03 | |
| openstackgerrit | Merged openstack/mistral master: Use Jinja2 sandbox environment https://review.openstack.org/455327 | 10:04 |
| *** mgershen1 has quit IRC | 10:05 | |
| *** zhurong has quit IRC | 10:10 | |
| *** zhurong has joined #openstack-mistral | 10:11 | |
| *** mgershen1 has joined #openstack-mistral | 10:18 | |
| *** mgershen has quit IRC | 10:20 | |
| *** kong has joined #openstack-mistral | 10:23 | |
| *** jkilpatr has quit IRC | 10:43 | |
| rakhmerov | xavierhardy: hi, no specific plans. We may do something more advanced for Keycloak in the near future | 10:48 |
| *** fultonj has joined #openstack-mistral | 10:58 | |
| *** jkilpatr has joined #openstack-mistral | 10:59 | |
| *** livelace has joined #openstack-mistral | 11:00 | |
| *** shardy has quit IRC | 11:07 | |
| *** shardy has joined #openstack-mistral | 11:07 | |
| *** shardy is now known as shardy_lunch | 11:09 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: Change workflow lang schema to support advanced publishing https://review.openstack.org/455178 | 11:15 |
| *** thrash|g0ne is now known as thrash | 11:16 | |
| openstackgerrit | Kevin Pouget proposed openstack/python-mistralclient master: Add CLI for event trigger operations https://review.openstack.org/454669 | 11:19 |
| *** mgershen has joined #openstack-mistral | 11:48 | |
| *** mgershen1 has quit IRC | 11:50 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: Change workflow lang schema to support advanced publishing https://review.openstack.org/455178 | 11:59 |
| *** brunograz has joined #openstack-mistral | 11:59 | |
| *** mgershen1 has joined #openstack-mistral | 12:04 | |
| *** mgershen has quit IRC | 12:07 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: Change workflow lang schema to support advanced publishing https://review.openstack.org/455178 | 12:07 |
| *** shardy_lunch is now known as shardy | 12:10 | |
| *** dprince has joined #openstack-mistral | 12:17 | |
| *** breton has quit IRC | 12:17 | |
| *** breton has joined #openstack-mistral | 12:18 | |
| openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Enable WSGI under Apache in devstack https://review.openstack.org/454710 | 12:25 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: Change workflow lang schema to support advanced publishing https://review.openstack.org/455178 | 12:42 |
| rakhmerov | d0ugal, thrash: hi, by any chance do you know what happened with our docs gate? | 12:50 |
| rakhmerov | e.g. http://logs.openstack.org/78/455178/8/check/gate-mistral-docs-ubuntu-xenial/62e042b/console.html | 12:51 |
| d0ugal | has it gone? | 12:51 |
| rakhmerov | yeah | 12:51 |
| rakhmerov | failing | 12:51 |
| d0ugal | looking | 12:51 |
| rakhmerov | seems like for all our patches | 12:51 |
| thrash | rakhmerov: I haven't touched it, but I'll look. :) | 12:51 |
| rakhmerov | yeah, I know. I guess something changed in the env | 12:51 |
| d0ugal | rakhmerov: it switched to Python 3 | 12:51 |
| rakhmerov | woow | 12:51 |
| d0ugal | and it has been broken in Python 3 for some time | 12:52 |
| rakhmerov | how did you know? | 12:52 |
| d0ugal | rakhmerov: http://logs.openstack.org/78/455178/8/check/gate-mistral-docs-ubuntu-xenial/62e042b/console.html#_2017-04-11_12_48_02_102524 | 12:52 |
| rakhmerov | true | 12:52 |
| d0ugal | rakhmerov: in python 3 reduce was moved from being global to functools.reduce | 12:52 |
| d0ugal | so the error makes sense | 12:52 |
| d0ugal | Why has it switched to Python 3? I'm not sure, but we can force it back easily :) | 12:52 |
| d0ugal | Patch incoming. | 12:53 |
| rakhmerov | ok :) | 12:53 |
| rakhmerov | you're fast | 12:53 |
| rakhmerov | let's get it back for now | 12:53 |
| d0ugal | oh. https://github.com/openstack/mistral/blob/master/tox.ini#L55 | 12:53 |
| d0ugal | weird. | 12:53 |
| d0ugal | it already specifies python 2 | 12:53 |
| rakhmerov | hm.. | 12:53 |
| rakhmerov | interesting | 12:53 |
| d0ugal | so maybe the way the docs are built has changed | 12:54 |
| *** zhurong has quit IRC | 12:54 | |
| rakhmerov | that's what I thought | 12:54 |
| rakhmerov | we didn't change anything | 12:54 |
| rakhmerov | on our side | 12:54 |
| rakhmerov | again, some dependency issue | 12:55 |
| d0ugal | rakhmerov: so they don't use our tox file. | 12:56 |
| d0ugal | I'm not sure how we fix this :) | 12:57 |
| rakhmerov | they always haven't? or has it changed lately? | 12:57 |
| d0ugal | I don't know. | 12:57 |
| rakhmerov | ok | 12:57 |
| thrash | d0ugal: this should affect everyone then. | 12:57 |
| rakhmerov | yes | 12:57 |
| d0ugal | thrash: only if their docs build isn't Python 3 compatible | 12:57 |
| thrash | d0ugal: true | 12:58 |
| d0ugal | thrash: I think the API docs generation is the bit that fails | 12:58 |
| rakhmerov | yeah | 12:58 |
| d0ugal | oh, actually, it is shpinxcontrib-pecanwsme that fails. | 12:59 |
| d0ugal | I am not sure what that does for us. | 12:59 |
| d0ugal | I see three options | 12:59 |
| d0ugal | 1) find a way to switch it back to python 2 | 13:00 |
| d0ugal | 2) remove sphinxcontrib-pecanwsme | 13:00 |
| d0ugal | 3) make it work on Python 3 | 13:00 |
| d0ugal | It looks like somebody already tried #3 https://github.com/dreamhost/sphinxcontrib-pecanwsme/pull/18 | 13:00 |
| rakhmerov | I'd assume #1 would be the easiest | 13:02 |
| rakhmerov | if we find where to do it | 13:02 |
| thrash | rakhmerov: looking... | 13:04 |
| rakhmerov | ok | 13:04 |
| *** bobh has joined #openstack-mistral | 13:05 | |
| d0ugal | thrash: I asked in -infra, seems other people have noticed too | 13:06 |
| thrash | d0ugal: ack | 13:06 |
| d0ugal | rakhmerov: any idea what that sphinx extension adds to the docs? | 13:07 |
| thrash | d0ugal: supposedly docs an api created with pecan. | 13:08 |
| d0ugal | thrash: I guessed that bit, but I can't actually see it anywhere | 13:08 |
| thrash | d0ugal: I'm guessing https://docs.openstack.org/developer/mistral/developer/webapi/v2.html | 13:08 |
| thrash | :) | 13:08 |
| rakhmerov | it parses controller classes docstrings and puts them into docs | 13:08 |
| rakhmerov | yes | 13:08 |
| rakhmerov | not docstrings, sorry | 13:09 |
| rakhmerov | wait a sec.. | 13:09 |
| d0ugal | ahh, it adds the rest-controller directive | 13:10 |
| rakhmerov | https://github.com/openstack/mistral/blob/master/mistral/api/controllers/v2/resources.py#L40-L49 | 13:11 |
| rakhmerov | I guess it add this | 13:11 |
| d0ugal | http://codesearch.openstack.org/?q=rest-controller&i=nope&files=.*%5C.rst&repos=mistral | 13:11 |
| d0ugal | rakhmerov: right, it uses that. | 13:11 |
| rakhmerov | it might be an obsolete way of doing this though.. but it is what it is | 13:12 |
| d0ugal | yeah | 13:13 |
| d0ugal | I wish pecan was obsolete :P | 13:14 |
| thrash | not our only problem with python 3 | 13:14 |
| d0ugal | thrash: no? | 13:14 |
| thrash | commented this out: https://github.com/openstack/mistral/blob/master/doc/source/conf.py#L34 | 13:14 |
| thrash | and got: File "/home/bcrochet/workspace/tripleo/mistral/.tox/docs/lib64/python3.5/re.py", line 163, in match | 13:14 |
| thrash | return _compile(pattern, flags).match(string) | 13:14 |
| thrash | TypeError: cannot use a string pattern on a bytes-like object | 13:14 |
| d0ugal | thrash: I done that too :) | 13:14 |
| thrash | which looks like a sphinx prob | 13:15 |
| thrash | or something in the docs | 13:15 |
| d0ugal | yeah, that is a weird one | 13:17 |
| d0ugal | Does anyone find the API docs useful? | 13:18 |
| d0ugal | I think we should just remove them. | 13:18 |
| d0ugal | They just fill the search with results that are not what I want in my experience | 13:18 |
| d0ugal | We even include our tests. https://docs.openstack.org/developer/mistral/api/mistral.tests.unit.actions.html | 13:19 |
| rakhmerov | :) | 13:22 |
| rakhmerov | well, d0ugal, I think we need API docs but may be generated in a different way | 13:22 |
| d0ugal | Why do we need them? | 13:22 |
| rakhmerov | how would I learn about API then? | 13:23 |
| rakhmerov | if we remove them | 13:23 |
| rakhmerov | I guess all projects have them | 13:23 |
| d0ugal | the Python API or the REST API? | 13:23 |
| rakhmerov | REST API at least | 13:23 |
| d0ugal | sure, we need to document the Mistral API | 13:23 |
| rakhmerov | what do you mean? | 13:23 |
| d0ugal | but we have tons of pages in the docs that cover Python code, code that should be considered private, we are not a library | 13:24 |
| rakhmerov | not sure I understand | 13:24 |
| rakhmerov | why code? | 13:24 |
| d0ugal | https://docs.openstack.org/developer/mistral/py-modindex.html | 13:24 |
| rakhmerov | (still confused) | 13:24 |
| d0ugal | That part of the documentation is useless | 13:24 |
| d0ugal | IMO | 13:24 |
| d0ugal | Who is it for? | 13:24 |
| rakhmerov | yeah, we can remove this I guess | 13:25 |
| d0ugal | it is also here: https://docs.openstack.org/developer/mistral/api/modules.html | 13:25 |
| rakhmerov | for me API docs are REST API mainly | 13:25 |
| d0ugal | Sure | 13:25 |
| rakhmerov | I was just confused a little bit | 13:25 |
| rakhmerov | ok | 13:25 |
| d0ugal | Sorry, I switched topics quickly :) | 13:26 |
| rakhmerov | d0ugal: yes, I agree on that | 13:26 |
| rakhmerov | github provides better code navigation capabilities than this stuff | 13:26 |
| d0ugal | the python-generated-docs just dominate the search results. | 13:26 |
| d0ugal | https://docs.openstack.org/developer/mistral/search.html?q=on-success | 13:26 |
| rakhmerov | :) | 13:27 |
| rakhmerov | let's remove them | 13:27 |
| d0ugal | \o/ | 13:27 |
| thrash | which mean we can also remove all the cruft to support it. :D | 13:33 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Change workflow lang schema to support advanced publishing https://review.openstack.org/455178 | 13:37 |
| rakhmerov | yes | 13:38 |
| *** mgershen has joined #openstack-mistral | 13:38 | |
| *** mgershen1 has quit IRC | 13:39 | |
| *** jtomasek_ has joined #openstack-mistral | 13:42 | |
| *** jkilpatr has quit IRC | 13:46 | |
| d0ugal | thrash: I am trying to do this but I can't figure out where it comes from lol | 13:49 |
| *** jkilpatr has joined #openstack-mistral | 13:49 | |
| rakhmerov | d0ugal, thrash: seems like pep8 gate is broken too (not because we did something): http://logs.openstack.org/78/455178/9/check/gate-mistral-pep8-ubuntu-xenial/2061b25/console.html | 13:50 |
| d0ugal | rakhmerov: that is also python 3 related | 13:50 |
| rakhmerov | yes, I guess so | 13:50 |
| d0ugal | rakhmerov: I see that error locally all the time | 13:51 |
| rakhmerov | d0ugal: it also fails with the same error locally | 13:51 |
| rakhmerov | yep ) | 13:51 |
| d0ugal | ah cool, you are Python 3 by default too? :) | 13:51 |
| d0ugal | I have been for a while and it means I suffer through strange issues others don't! | 13:51 |
| thrash | guess I'm too much of a wuss | 13:51 |
| rakhmerov | d0ugal: yes | 13:52 |
| rakhmerov | d0ugal: but we're supposed to get rid of Py 2 soon AFAIK | 13:52 |
| *** mgershen1 has joined #openstack-mistral | 13:52 | |
| d0ugal | really? | 13:52 |
| rakhmerov | yes | 13:53 |
| *** mgershen has quit IRC | 13:53 | |
| rakhmerov | I've heard this many times (including summits) | 13:53 |
| d0ugal | heh | 13:53 |
| rakhmerov | d0ugal: I finally finished my first patch for advanced publishing https://review.openstack.org/#/c/455178/ | 13:54 |
| rakhmerov | sorry for the size, but I really didn't know how to make it smaller | 13:54 |
| rakhmerov | it's just how our specs now work | 13:54 |
| rakhmerov | I'll try to simplify it moving forward, and may be not at once | 13:55 |
| rakhmerov | except these two gates it seems to be passing main gates | 13:55 |
| rakhmerov | btw, coverage gate is also broken with a weird error | 13:55 |
| rakhmerov | d0ugal: I'm out for today | 13:56 |
| d0ugal | coverage gate has always been broken :) | 13:57 |
| d0ugal | rakhmerov: cool, thanks - I'll take a look | 13:57 |
| *** jaosorior is now known as jaosorior_away | 14:06 | |
| d0ugal | where is this used? https://github.com/openstack/mistral/tree/master/api-ref/source | 14:07 |
| d0ugal | ah, so that is an actual REST API only docs | 14:08 |
| d0ugal | man this is confusing. | 14:08 |
| d0ugal | thrash: any idea where the api docs are generated? | 14:16 |
| *** chlong has joined #openstack-mistral | 14:21 | |
| *** mgershen has joined #openstack-mistral | 14:24 | |
| *** mgershen1 has quit IRC | 14:25 | |
| thrash | d0ugal: i don't | 14:26 |
| *** jkilpatr has quit IRC | 14:26 | |
| d0ugal | it really confuses me | 14:26 |
| thrash | d0ugal: I haven't looked at it in detail | 14:28 |
| d0ugal | thrash: something creates the rst files in doc/source/api/ | 14:28 |
| d0ugal | but I don't know what | 14:28 |
| d0ugal | thrash: woah, pbr generates them! | 14:38 |
| *** jkilpatr has joined #openstack-mistral | 14:40 | |
| *** mgershen has quit IRC | 14:41 | |
| *** mgershen has joined #openstack-mistral | 14:41 | |
| d0ugal | thrash, rakhmerov - the release has been rolled back, so it should be fine if we recheck now. | 14:54 |
| *** mgershen1 has joined #openstack-mistral | 14:55 | |
| *** mgershen has quit IRC | 14:56 | |
| thrash | d0ugal: no way... | 15:06 |
| therve | openstackstatus is not here for some reasons | 15:08 |
| d0ugal | huh, weird | 15:08 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Disable pbrs auto python-api generation https://review.openstack.org/455744 | 15:08 |
| *** kong has quit IRC | 15:09 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Set the basepython for the venv tox environment https://review.openstack.org/455745 | 15:12 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Disable pbrs auto python-api generation https://review.openstack.org/455744 | 15:19 |
| *** warface has quit IRC | 15:27 | |
| *** chlong_ has joined #openstack-mistral | 15:29 | |
| *** chlong has quit IRC | 15:32 | |
| *** chlong has joined #openstack-mistral | 15:44 | |
| *** livelace has quit IRC | 15:50 | |
| *** jistr is now known as jistr|mtg | 15:59 | |
| *** mgershen has joined #openstack-mistral | 16:10 | |
| *** mgershen1 has quit IRC | 16:12 | |
| *** jpich has quit IRC | 16:17 | |
| *** d0ugal has quit IRC | 16:18 | |
| *** mgershen1 has joined #openstack-mistral | 16:25 | |
| *** mgershen has quit IRC | 16:27 | |
| *** jistr|mtg is now known as jistr | 16:37 | |
| *** mgershen has joined #openstack-mistral | 16:42 | |
| *** mgershen1 has quit IRC | 16:43 | |
| *** mgershen1 has joined #openstack-mistral | 16:57 | |
| *** mgershen has quit IRC | 16:58 | |
| *** mgershen has joined #openstack-mistral | 17:12 | |
| *** mgershen1 has quit IRC | 17:15 | |
| *** mgershen1 has joined #openstack-mistral | 17:27 | |
| *** mgershen has quit IRC | 17:28 | |
| jenner_ | guys, is anyone running mistral in docker (openshift)? | 17:54 |
| jenner_ | I see there's an example script that creates a mistral.conf upon container startup, but wouldn't it be nice if we could fall back to environment vars if there's no config file? | 17:58 |
| *** mgershen has joined #openstack-mistral | 18:28 | |
| *** mgershen1 has quit IRC | 18:29 | |
| *** mgershen1 has joined #openstack-mistral | 18:43 | |
| *** mgershen has quit IRC | 18:45 | |
| *** bobh has quit IRC | 18:59 | |
| *** mgershen has joined #openstack-mistral | 18:59 | |
| *** mgershen1 has quit IRC | 19:01 | |
| *** mgershen1 has joined #openstack-mistral | 19:13 | |
| *** mgershen has quit IRC | 19:16 | |
| *** toure is now known as toure|biab | 19:24 | |
| *** toure|biab is now known as toure | 19:24 | |
| *** mgershen has joined #openstack-mistral | 19:31 | |
| *** mgershen1 has quit IRC | 19:32 | |
| *** mgershen1 has joined #openstack-mistral | 19:37 | |
| *** mgershen has quit IRC | 19:39 | |
| *** mgershen has joined #openstack-mistral | 19:45 | |
| *** jkilpatr_ has joined #openstack-mistral | 19:45 | |
| *** aspiers has quit IRC | 19:45 | |
| *** mgershen1 has quit IRC | 19:46 | |
| *** aspiers has joined #openstack-mistral | 19:47 | |
| *** jkilpatr has quit IRC | 19:47 | |
| *** shardy has quit IRC | 19:50 | |
| *** mgershen1 has joined #openstack-mistral | 19:56 | |
| *** mgershen has quit IRC | 19:57 | |
| *** bobh has joined #openstack-mistral | 20:00 | |
| *** bobh has quit IRC | 20:05 | |
| *** mgershen1 has quit IRC | 20:08 | |
| *** mgershen has joined #openstack-mistral | 20:08 | |
| *** mgershen1 has joined #openstack-mistral | 20:21 | |
| *** mgershen has quit IRC | 20:23 | |
| *** dprince has quit IRC | 20:27 | |
| *** bobh has joined #openstack-mistral | 20:33 | |
| *** mgershen has joined #openstack-mistral | 20:38 | |
| *** mgershen1 has quit IRC | 20:39 | |
| *** dmellado has quit IRC | 20:48 | |
| *** jtomasek_ has quit IRC | 20:51 | |
| *** mgershen1 has joined #openstack-mistral | 20:51 | |
| *** mgershen has quit IRC | 20:53 | |
| *** mgershen has joined #openstack-mistral | 21:09 | |
| *** chlong has quit IRC | 21:09 | |
| *** chlong_ has quit IRC | 21:09 | |
| *** mgershen1 has quit IRC | 21:11 | |
| *** thrash is now known as thrash|g0ne | 21:15 | |
| *** catintheroof has joined #openstack-mistral | 21:19 | |
| *** catintheroof has quit IRC | 21:21 | |
| *** catintheroof has joined #openstack-mistral | 21:21 | |
| *** kong has joined #openstack-mistral | 21:30 | |
| *** jkilpatr_ has quit IRC | 21:30 | |
| *** jkilpatr_ has joined #openstack-mistral | 22:08 | |
| *** bobh has quit IRC | 22:18 | |
| *** catintheroof has quit IRC | 23:29 | |
| *** catintheroof has joined #openstack-mistral | 23:30 | |
| *** catintheroof has quit IRC | 23:30 | |
| *** jamielennox is now known as jamielennox|away | 23:48 | |
| *** jamielennox|away is now known as jamielennox | 23:51 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!