Saturday, 2018-08-25

*** ianychoi has quit IRC00:08
*** tristanC has joined #zuul00:34
*** eumel8 has joined #zuul05:15
*** ianychoi has joined #zuul06:41
*** bhavikdbavishi has joined #zuul08:09
*** bhavikdbavishi has quit IRC08:16
*** bhavikdbavishi has joined #zuul08:30
bhavikdbavishi.08:36
bhavikdbavishi@here , how do I validate zuul.d/{jobs,projects,pipeline}.yaml in zuul v3 ? In zuul v2 `zuul-server -l /etc/zuul/layout.yaml -t`.08:38
ssbarneais possible to change zuul output to use ANSIBLE_STDOUT_CALLBACK=debug instead of the default one, which is messing newlines on result.stdout?08:54
mordredssbarnea: zuul actually has its own stdout callback plugin - http://git.zuul-ci.org/cgit/zuul/tree/zuul/ansible/callback/zuul_stream.py09:11
mordredssbarnea: which is what allows us to do live-streaming of command/shell task output09:12
mordredssbarnea: if we're breaking result.stdout from what you're expecting, it sounds like a bug we should fix09:12
mordredbhavikdbavishi: in zuul v3 zuul will validate proposed changes to all of those for you automatically09:13
ssbarneamordred:  ohh, this makes it more complicated to fix in this case. are you sure you know what I am referring to? printing multiline stdout/stderr as json is not really user friendly, thus is why the09:14
ssbarneadebug" ansible module is printing these not as json.09:14
ssbarneain my opinion we should make output easy to read for humans,... so we may want the same behaviour in zuul callback, unless we have some technical issues that prevent doing so.09:15
mordredssbarnea: I'm _not_ sure - I'm also ina weird jetlagged mental state, so it's possible I'm hallucinating this whole conversation :)09:15
mordredssbarnea: yes - so - zuul_stream produces the plain-text stream that is used for the websocket and finger-connection stream09:16
ssbarneai don't mind trying to have a look at the callback and try to do somethign about it, but i first want to know if there is somethign that may prevent me from making the change.09:16
mordredin order to make it a consumable as a plain-text stream,it has to make choices as to what to show and what not to - at the same time, http://git.zuul-ci.org/cgit/zuul/tree/zuul/ansible/callback/zuul_json.py writes out the json file that has *all* of the info - but currently nothing is really consuming that json file - it's just there09:18
ssbarneamordred: ok, the json one has nothing to do with console, it seems that http://git.zuul-ci.org/cgit/zuul/tree/zuul/ansible/callback/zuul_stream.py is the one I would have to improve. i will first file a bug.09:21
ssbarneaand assign it to me09:21
mordred++09:22
mordredssbarnea: also - there is half-finished work to change how we get the stdout stream from the remote ansible in the first place: https://review.openstack.org/#/q/topic:zuul-stream-rework09:23
mordredthe intent there is to get rid of the need for the zuul_console process to be spawned and to restructure it in a way that is potentially upstreamable09:24
mordredthe mechanics of that don't have much to do with the display of the stream itself - but it's worth being aware of as you start to look at that code area09:25
ssbarneamordred: thanks, these do really help getting an idea about how it works and in which direction is going.09:26
*** bhavikdbavishi has quit IRC10:42
*** bhavikdbavishi has joined #zuul12:13
bhavikdbavishissbarnea: ok, great12:55
*** bhavikdbavishi has quit IRC13:21
*** tristanC has quit IRC14:49
*** bhavikdbavishi has joined #zuul16:24
*** tristanC has joined #zuul17:55
*** bhavikdbavishi has quit IRC18:34
SpamapShrm, shouldn't zuul-web be serving up the static page?19:59
SpamapSah looks like pbrx build-images doesn't install the static content20:24
SpamapSmordred: ^ guessing that you meant for the _setup_hook to run in pbrx yeah?20:25

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!