Friday, 2019-07-19

*** igordc has quit IRC00:05
*** sgw has quit IRC00:12
*** igordc has joined #zuul00:24
*** igordc has quit IRC00:30
*** swest has quit IRC01:29
*** swest has joined #zuul01:43
*** bhavikdbavishi has joined #zuul02:26
*** bjackman has joined #zuul02:37
*** bhavikdbavishi1 has joined #zuul03:01
*** bhavikdbavishi has quit IRC03:02
*** bhavikdbavishi1 is now known as bhavikdbavishi03:02
*** sgw has joined #zuul03:41
*** swest has quit IRC04:28
*** pcaruana has joined #zuul04:37
*** bolg has joined #zuul05:16
*** swest has joined #zuul05:18
*** swest has quit IRC05:27
*** swest has joined #zuul05:29
*** swest has quit IRC05:33
*** raukadah is now known as chandankumar05:36
*** homeski has quit IRC05:47
*** swest has joined #zuul05:47
*** mgoddard has quit IRC05:48
*** swest has quit IRC05:48
*** homeski has joined #zuul05:48
*** bolg_ has joined #zuul05:49
*** bolg has quit IRC05:49
*** mgoddard has joined #zuul05:51
*** swest has joined #zuul05:55
*** zbr has quit IRC07:15
*** zbr has joined #zuul07:15
*** tosky has joined #zuul07:24
openstackgerritMerged zuul/zuul-jobs master: Install system dependencies for tox-molecule  https://review.opendev.org/67102907:56
*** bolg_ has quit IRC08:38
*** panda is now known as panda|drappt08:50
*** bolg has joined #zuul09:16
*** yolanda has quit IRC09:30
*** bhavikdbavishi has quit IRC10:06
*** panda|drappt is now known as panda10:14
openstackgerritJan Kubovy proposed zuul/zuul master: Update heuristing of parallel starting builds.  https://review.opendev.org/67170210:36
openstackgerritTobias Henkel proposed zuul/nodepool master: Fix node failures when at volume quota  https://review.opendev.org/67170410:58
openstackgerritTobias Henkel proposed zuul/nodepool master: Fix node failures when at volume quota  https://review.opendev.org/67170410:59
*** panda is now known as panda|lunch11:42
*** bjackman has quit IRC12:31
*** panda|lunch is now known as panda12:36
*** bhavikdbavishi has joined #zuul12:54
*** rlandy has joined #zuul13:09
*** rlandy_ has joined #zuul13:13
*** rlandy_ has quit IRC13:13
*** rlandy_ has joined #zuul13:13
*** rlandy_ has quit IRC13:15
*** rlandy_ has joined #zuul13:15
*** rlandy has quit IRC13:16
*** rlandy_ is now known as rlandy13:17
*** EmilienM is now known as EvilienM13:38
*** jeliu_ has joined #zuul13:50
*** igordc has joined #zuul13:52
*** bolg has quit IRC13:59
*** sgw has quit IRC14:13
*** igordc has quit IRC14:20
*** sgw has joined #zuul14:27
*** mattw4 has joined #zuul14:42
*** rlandy has quit IRC14:43
*** rlandy has joined #zuul14:46
*** tosky_ has joined #zuul14:59
*** tosky has quit IRC15:02
*** tosky_ is now known as tosky15:10
openstackgerritJames E. Blair proposed zuul/project-config master: Configure gate to supercede check  https://review.opendev.org/67175715:12
*** jamesmcarthur has joined #zuul15:18
*** mattw4 has quit IRC15:22
*** mattw4 has joined #zuul15:22
openstackgerritTristan Cacqueray proposed zuul/zuul master: manager: specify report failure in logs  https://review.opendev.org/67176015:26
*** zbr is now known as zbr|out15:30
*** mattw4 has quit IRC15:39
Shrewscorvus: any reason to hold off approving https://review.opendev.org/671757 now?15:45
clarkbI think we should approve it now so that we can confirm it works sooner :)15:46
Shrewsalso, looks like the opendev-promote-javascript-content job failed on https://review.opendev.org/66737115:47
Shrewshttp://paste.openstack.org/show/754591/15:48
*** pcaruana has quit IRC15:49
clarkbShrews: I'm guessing that is fallout from the tenant transition15:49
*** bhavikdbavishi1 has joined #zuul15:53
*** bhavikdbavishi has quit IRC15:54
*** bhavikdbavishi1 is now known as bhavikdbavishi15:54
*** jangutter has quit IRC15:55
openstackgerritMerged zuul/project-config master: Configure gate to supercede check  https://review.opendev.org/67175716:00
*** tosky has quit IRC16:08
*** kmalloc is now known as needscoffee16:11
*** chandankumar is now known as raukadah16:39
*** igordc has joined #zuul17:10
*** jamesmcarthur has quit IRC17:25
*** mattw4 has joined #zuul17:32
*** rlandy is now known as rlandy|brb17:36
*** tosky has joined #zuul17:49
openstackgerritMerged zuul/zuul-jobs master: upload-logs-swift: option for disabling the indexer  https://review.opendev.org/66335517:52
*** rlandy|brb is now known as rlandy17:59
*** jamesmcarthur has joined #zuul18:07
openstackgerritMerged zuul/zuul-jobs master: Add install-devstack test job  https://review.opendev.org/67019518:37
*** gtema has joined #zuul18:38
*** armstrongs has joined #zuul18:52
*** jamesmcarthur has quit IRC18:58
*** jamesmcarthur has joined #zuul19:01
*** jamesmcarthur has quit IRC19:07
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: configure-pypi-mirrors: add initial role  https://review.opendev.org/58116519:14
openstackgerritTristan Cacqueray proposed zuul/nodepool master: static: add host-key-checking toggle  https://review.opendev.org/65367919:26
*** gtema has quit IRC19:30
openstackgerritTristan Cacqueray proposed zuul/nodepool master: static: add host-key-checking toggle  https://review.opendev.org/65367919:34
corvustristanC: thanks!  that raises some questions about the mirror_info variable -- i left comments on https://review.opendev.org/669948 and https://review.opendev.org/58116519:36
corvusclarkb, fungi: can you also take a look at the comments on https://review.opendev.org/669948 and let me know what you think?19:37
corvusactually, come to think of it, i'd also like more eyes on the comments in https://review.opendev.org/581165 am i right that we should put the architecture variables on the calling side, or should they go in the role as tristanC has them?19:38
corvus(my feeling is that since that's what goes into the pip.conf file, then it should probably be on the calling side)19:39
clarkbI'll look.19:40
corvusand yes, this is all totally bikeshedding, but we already painted this shed once, and the paint is flaking off, so we better do better surface prep this time :)19:40
clarkbfwiw I've been pretending to understand how to debug js to undersatnd why log streaming doesn't work in ff anymore. It almost looks like requestAnimationFrame is hanging19:40
clarkbbut I can't tell which call to that bceause the FF performance debugger doesn't know how to give me line numbers in the unobfuscated code19:41
corvusaw, i wish i hadn't read that :(19:41
clarkb(the debugger does though so maybe can figure it out with breakpoints)19:41
clarkbI also note the console says the service worker is disabled19:41
clarkbwhich maybe is the problem if we need a service worker to render and animation frame?19:42
*** jamesmcarthur has joined #zuul19:42
clarkbmight be good for other FF users to test if it works for them. I'm on the beta release channel so it may be a bug in the beta19:42
tristanCcorvus: you're welcome, we actually don't need that role anymore, but i wanted to try the new zuul-tests.d system, and happy to test the mirror_info proposal19:42
clarkbor behavior change that will break us when next release happens19:42
clarkbcorvus: responded on https://review.opendev.org/#/c/669948/619:45
corvustristanC: ok.  i still like the idea -- and since we have to make new mirror roles anyway, let's stick with it -- we can make a bunch of small roles like that, and then have an optional "configure all the mirrors" role that includes them19:45
corvusclarkb: okay, so your vote is keep it independent of "pypi" (ie, you don't need a pypi mirror to have a wheel mirror) and call it "wheel", yeah?19:46
clarkbyes19:46
clarkband I agree with you that the wheel mirror and pypi mirror paths should be set by the top level input and not inferred19:47
corvusyeah, and thinking about what you wrote in that comment, that lets you do things like "{% if arch has mirror %}url{% endif %}19:48
*** jamesmcarthur has quit IRC19:48
clarkblog streaming is not broken in firefox 67.04 but is in 69.0.519:49
clarkber thats 69.0.b519:49
clarkbanyone have a firefox 68 to test?19:49
corvusi'm running 68 and log streaming works19:49
clarkbok so possibly something in the 69 beta then. Looks like I have updates I can apply with a browser restart so will do that and test agaqin19:50
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: configure-pypi-mirrors: add initial role  https://review.opendev.org/58116519:50
clarkbstill broken but different!19:52
clarkbthe black window for the xterm.js terminal is now tiny instead of full window si19:52
clarkbthe black window for the xterm.js terminal is now tiny instead of full window size19:52
clarkbbah19:52
clarkband then there it expands19:52
*** michael-beaver has joined #zuul19:54
clarkbI guess the other thing I can try is updating xterm.js and uses the artifacts from check to test on newer FF19:54
openstackgerritJames E. Blair proposed zuul/nodepool master: Remove opendev-promote-javascript-content  https://review.opendev.org/67185119:54
corvusShrews: thanks for catching that :)19:54
clarkbtristanC: ^ do you know why we cap xterm at <3.14.0?19:57
clarkbI seem to recall they updated and broke something19:57
clarkbgit log to the rescure20:01
clarkbcan't set the theme on newer xterm /me reads docs to see if that was renamed20:01
clarkbhuh issue may have been fixed 9 days ago and is in the 3.14.5 release looks like /me tries this20:04
tristanCclarkb: maybe dropping the cap is enough20:04
clarkbtristanC: ya thats what I'm thinking. Regenerating a lock file as soon as I figure out that process again then will push a change up20:05
clarkbthere is a fix we need too from corvus I think20:05
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: configure-pypi-mirrors: add initial role  https://review.opendev.org/58116520:08
openstackgerritClark Boylan proposed zuul/zuul master: Update xterm to >= 3.14.5  https://review.opendev.org/67185820:09
*** jamesmcarthur has joined #zuul20:20
*** jamesmcarthur has quit IRC20:26
clarkbok thats just broken now20:28
clarkbsomething wrong with a dimensions property that can't be found. QUickly getitng out of my comfort area here :)20:29
clarkbthe good news is that is the dimensions of the renderer which implies maybe they have changed things to fix whatever problem I have20:31
corvustristanC: i've started hacking on some js stuff, and i finally installed an editor plugin for jsx (https://github.com/felipeochoa/rjsx-mode).  it seems to want to use four spaces and have ; at the end of a lot of lines (current formatting is 2 spaces and omitting ;).  is rjsx-mode set to more standard jsx, and should we switch to that?  or is our current format better and i should customize rjsx-mode?20:33
corvusmordred: ^ you may also have opinions20:34
corvus(note, i don't have an opinion, i'd be happy with either, i just don't want to do one thing if we're going to turn around and do the other thing later)20:35
mordredcorvus: I never have opinions20:35
* mordred reads20:35
openstackgerritClark Boylan proposed zuul/zuul master: Update xterm to >= 3.14.5  https://review.opendev.org/67185820:36
clarkbIf I can read js (I can't) then that may do it20:36
mordredcorvus: so ... the ;s are not needed but are allowed - using them seems to be a thing longer-term js people still do by habit20:36
corvusmordred: so you're seeing lack-of-semis in newer jsx?20:38
mordredcorvus: it's apparently quite the style war in javascript land20:38
mordredthere are adamant camps, it seems, on both sides20:38
* corvus backs away slowly20:38
mordredhttps://flaviocopes.com/javascript-automatic-semicolon-insertion/20:38
mordredyeah20:38
corvuseither way is easy enough: (setq js2-strict-missing-semi-warning nil)20:38
mordredI picked up "no semi" when we did the original angular with the standard.js style applied20:39
mordred(also, as a python programmer, I find the semi-less code more pleasant to look at)20:39
corvusokay, on balance i think i'm just going customize rjsx-mode to match what we have :)20:39
mordredBUT - in my weekend personal website hacking, I ahve consistently seen both styles come through20:39
mordredcorvus: kk20:39
corvus(at least, it seems that what we have isn't obviously wrong, and i don't care enough to think about it more than that :)20:40
mordredcorvus: for 4 space v 2 space I am not sure what the current fashion is - I like 2 because the js callbacks seem to cause lots of indentation to happen20:40
mordredcorvus: +100020:40
corvus(setq js2-strict-missing-semi-warning nil)20:40
corvus(setq js-indent-level 2)20:40
corvusand done!20:41
mordredsweet!20:41
mordred(a quick google seems to indicate that 2 space has been becoming the popular choice - so maybe we're on the right side of that one already)20:41
mordredI have to admit - go has actually made me come around to being a fan of hard-tabs20:42
corvusi'd be happy to switch to tabs any day of the week :)20:42
mordred(I know, I lose my python hacker card for saying that) - but largely because hard-tabs allow people to pick an indentation preference and not have to agree on it20:42
clarkbtabs are fewer bytes too for that precious wirespeed20:42
mordredI've very much enjoyed setting ts=2 for my golang files and being able to actually read them20:42
openstackgerritMerged zuul/nodepool master: Remove opendev-promote-javascript-content  https://review.opendev.org/67185120:47
fungiand i'm back20:47
fungitime to see what all this scrollback's about20:47
fungiclarkb: if it helps, not all ff is broken? i just streamed a job console with ff67 no problem20:50
fungiare you sure it's not an extension breaking you?20:50
fungioh, now that i read further, you're on 79 and corvus's 68 works20:51
fungii'm living in the past20:51
fungi(just like jethro tull)20:51
fungis/79/69/20:52
mordredcorvus, tristanC: I did this: https://review.opendev.org/#/c/671243/ which documents the /info endpoints ... and in doing it I looked at what we publish for opendev - which is pretty much all null20:53
clarkbfungi: ya confirmed works on 67 and 68 but not 69 beta 5 or 620:53
mordredI think that means we're not actually using anything from that except for the tenant20:53
mordredthat, then, makes me wonder if we want to do the work towards providing real values and consuming them - or just getting rid of cruft that were ideas we might use but never did20:54
mordred(also - yay for docs pointing out things like that)20:54
fungi"jsx" pronounced like "deus ex"20:54
*** jamesmcarthur has joined #zuul20:55
fungialso i like explicit semicolons, but then again i probably still have stockholm syndrome from writing in c20:56
fungii my c past, i preferred 3-space tabs. i must be a rebel20:57
*** jamesmcarthur has quit IRC21:00
mordredfungi: I have never heard anyone advocate for 3-space tabs. you impress me21:03
fungii have plenty of other terrible opinions but i'll save them for next week21:03
clarkbok I'm quickly out of my depth debugging this jsx dimensions of window thing21:10
clarkbbut hoping to brute force it with the browser debugger21:10
openstackgerritClark Boylan proposed zuul/zuul master: Update xterm to >= 3.14.5  https://review.opendev.org/67185821:16
clarkbdebugger says that should exist21:16
clarkband seems to have values that look right21:16
openstackgerritJames E. Blair proposed zuul/zuul master: Display artifacts in build page  https://review.opendev.org/67186521:21
*** jeliu_ has quit IRC21:25
clarkbok 671858 works in chrome now but not in ff 69b621:29
clarkboh wait it eventually loaded about 30 lines of text in ff21:29
fungithe old code works in chrome too, right?21:29
clarkbfungi: yes21:30
clarkbso this is maybe the slightest of improvements on ff but not what I would call functional21:30
*** EvilienM is now known as EmilienM21:30
clarkbhttp://logs.openstack.org/58/671858/3/check/zuul-build-dashboard/ef1ec96/npm/html if you want to test it yourself21:30
*** jamesmcarthur has joined #zuul21:32
*** jamesmcarthur has quit IRC21:36
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add mirror_info documentation  https://review.opendev.org/66994821:43
*** tosky has quit IRC21:43
corvusclarkb: okay, updated with your suggestions ^21:43
fungiclarkb: yeah, the test is working fine for me on ff67, fwiw21:45
funginice and speedy21:46
fungistreams thousands of lines of backlog in long-running jobs in a matter of a few seconds and then updates with new lines cleanly21:46
fungimakes me wonder if the ff69 beta builds are regressed terribly somehow21:47
corvusclarkb: seems to work as before in ff6821:47
clarkbI'm told there are webrender changes in ff69 (though it is disabled in my build)21:48
clarkbI'm trying to test it with webrender enabled21:48
clarkb(maybe the broke the other thing in the prcess)21:48
clarkbnope no luck21:54
clarkbso now I have it working quickly22:08
clarkbI disabled hardware acceleration22:08
clarkbso it seems related to rendering somewhere22:08
clarkband going to default boring settings seems to work22:08
fungioh fun22:30
fungidisabled hardware accel settings in ff itself, or in your desktop environment/windowing system?22:30
clarkbin ff itself. It isn't enabled by default on linux unless you are on nightly which they enabled for webrender22:31
clarkbso I'm guessing it is something related to the webrender changes after all22:31
clarkbbut in the lower level acceleration tooling22:31
fungisomewhere in about:config?22:32
clarkbyes layers.acceleration.force-enabled22:32
clarkband friends22:32
fungihuh. it's set true for me22:33
fungii must have done that a while back22:33
fungior debian sets it in its builds in unstable22:33
fungiit's showing as in a modified status, so thinking i must have set it myself22:33
openstackgerritJames E. Blair proposed zuul/zuul master: Display artifacts in build page  https://review.opendev.org/67186523:00
*** rlandy has quit IRC23:07
*** mattw4 has quit IRC23:08
*** jamesmcarthur has joined #zuul23:08
*** sgw has quit IRC23:14
*** jamesmcarthur has quit IRC23:22
*** jamesmcarthur has joined #zuul23:23
*** michael-beaver has quit IRC23:23
*** jamesmcarthur has quit IRC23:28
*** igordc has quit IRC23:48
*** jamesmcarthur has joined #zuul23:58

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