*** igordc has quit IRC | 00:05 | |
*** sgw has quit IRC | 00:12 | |
*** igordc has joined #zuul | 00:24 | |
*** igordc has quit IRC | 00:30 | |
*** swest has quit IRC | 01:29 | |
*** swest has joined #zuul | 01:43 | |
*** bhavikdbavishi has joined #zuul | 02:26 | |
*** bjackman has joined #zuul | 02:37 | |
*** bhavikdbavishi1 has joined #zuul | 03:01 | |
*** bhavikdbavishi has quit IRC | 03:02 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 03:02 | |
*** sgw has joined #zuul | 03:41 | |
*** swest has quit IRC | 04:28 | |
*** pcaruana has joined #zuul | 04:37 | |
*** bolg has joined #zuul | 05:16 | |
*** swest has joined #zuul | 05:18 | |
*** swest has quit IRC | 05:27 | |
*** swest has joined #zuul | 05:29 | |
*** swest has quit IRC | 05:33 | |
*** raukadah is now known as chandankumar | 05:36 | |
*** homeski has quit IRC | 05:47 | |
*** swest has joined #zuul | 05:47 | |
*** mgoddard has quit IRC | 05:48 | |
*** swest has quit IRC | 05:48 | |
*** homeski has joined #zuul | 05:48 | |
*** bolg_ has joined #zuul | 05:49 | |
*** bolg has quit IRC | 05:49 | |
*** mgoddard has joined #zuul | 05:51 | |
*** swest has joined #zuul | 05:55 | |
*** zbr has quit IRC | 07:15 | |
*** zbr has joined #zuul | 07:15 | |
*** tosky has joined #zuul | 07:24 | |
openstackgerrit | Merged zuul/zuul-jobs master: Install system dependencies for tox-molecule https://review.opendev.org/671029 | 07:56 |
---|---|---|
*** bolg_ has quit IRC | 08:38 | |
*** panda is now known as panda|drappt | 08:50 | |
*** bolg has joined #zuul | 09:16 | |
*** yolanda has quit IRC | 09:30 | |
*** bhavikdbavishi has quit IRC | 10:06 | |
*** panda|drappt is now known as panda | 10:14 | |
openstackgerrit | Jan Kubovy proposed zuul/zuul master: Update heuristing of parallel starting builds. https://review.opendev.org/671702 | 10:36 |
openstackgerrit | Tobias Henkel proposed zuul/nodepool master: Fix node failures when at volume quota https://review.opendev.org/671704 | 10:58 |
openstackgerrit | Tobias Henkel proposed zuul/nodepool master: Fix node failures when at volume quota https://review.opendev.org/671704 | 10:59 |
*** panda is now known as panda|lunch | 11:42 | |
*** bjackman has quit IRC | 12:31 | |
*** panda|lunch is now known as panda | 12:36 | |
*** bhavikdbavishi has joined #zuul | 12:54 | |
*** rlandy has joined #zuul | 13:09 | |
*** rlandy_ has joined #zuul | 13:13 | |
*** rlandy_ has quit IRC | 13:13 | |
*** rlandy_ has joined #zuul | 13:13 | |
*** rlandy_ has quit IRC | 13:15 | |
*** rlandy_ has joined #zuul | 13:15 | |
*** rlandy has quit IRC | 13:16 | |
*** rlandy_ is now known as rlandy | 13:17 | |
*** EmilienM is now known as EvilienM | 13:38 | |
*** jeliu_ has joined #zuul | 13:50 | |
*** igordc has joined #zuul | 13:52 | |
*** bolg has quit IRC | 13:59 | |
*** sgw has quit IRC | 14:13 | |
*** igordc has quit IRC | 14:20 | |
*** sgw has joined #zuul | 14:27 | |
*** mattw4 has joined #zuul | 14:42 | |
*** rlandy has quit IRC | 14:43 | |
*** rlandy has joined #zuul | 14:46 | |
*** tosky_ has joined #zuul | 14:59 | |
*** tosky has quit IRC | 15:02 | |
*** tosky_ is now known as tosky | 15:10 | |
openstackgerrit | James E. Blair proposed zuul/project-config master: Configure gate to supercede check https://review.opendev.org/671757 | 15:12 |
*** jamesmcarthur has joined #zuul | 15:18 | |
*** mattw4 has quit IRC | 15:22 | |
*** mattw4 has joined #zuul | 15:22 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: manager: specify report failure in logs https://review.opendev.org/671760 | 15:26 |
*** zbr is now known as zbr|out | 15:30 | |
*** mattw4 has quit IRC | 15:39 | |
Shrews | corvus: any reason to hold off approving https://review.opendev.org/671757 now? | 15:45 |
clarkb | I think we should approve it now so that we can confirm it works sooner :) | 15:46 |
Shrews | also, looks like the opendev-promote-javascript-content job failed on https://review.opendev.org/667371 | 15:47 |
Shrews | http://paste.openstack.org/show/754591/ | 15:48 |
*** pcaruana has quit IRC | 15:49 | |
clarkb | Shrews: I'm guessing that is fallout from the tenant transition | 15:49 |
*** bhavikdbavishi1 has joined #zuul | 15:53 | |
*** bhavikdbavishi has quit IRC | 15:54 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 15:54 | |
*** jangutter has quit IRC | 15:55 | |
openstackgerrit | Merged zuul/project-config master: Configure gate to supercede check https://review.opendev.org/671757 | 16:00 |
*** tosky has quit IRC | 16:08 | |
*** kmalloc is now known as needscoffee | 16:11 | |
*** chandankumar is now known as raukadah | 16:39 | |
*** igordc has joined #zuul | 17:10 | |
*** jamesmcarthur has quit IRC | 17:25 | |
*** mattw4 has joined #zuul | 17:32 | |
*** rlandy is now known as rlandy|brb | 17:36 | |
*** tosky has joined #zuul | 17:49 | |
openstackgerrit | Merged zuul/zuul-jobs master: upload-logs-swift: option for disabling the indexer https://review.opendev.org/663355 | 17:52 |
*** rlandy|brb is now known as rlandy | 17:59 | |
*** jamesmcarthur has joined #zuul | 18:07 | |
openstackgerrit | Merged zuul/zuul-jobs master: Add install-devstack test job https://review.opendev.org/670195 | 18:37 |
*** gtema has joined #zuul | 18:38 | |
*** armstrongs has joined #zuul | 18:52 | |
*** jamesmcarthur has quit IRC | 18:58 | |
*** jamesmcarthur has joined #zuul | 19:01 | |
*** jamesmcarthur has quit IRC | 19:07 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: configure-pypi-mirrors: add initial role https://review.opendev.org/581165 | 19:14 |
openstackgerrit | Tristan Cacqueray proposed zuul/nodepool master: static: add host-key-checking toggle https://review.opendev.org/653679 | 19:26 |
*** gtema has quit IRC | 19:30 | |
openstackgerrit | Tristan Cacqueray proposed zuul/nodepool master: static: add host-key-checking toggle https://review.opendev.org/653679 | 19:34 |
corvus | tristanC: thanks! that raises some questions about the mirror_info variable -- i left comments on https://review.opendev.org/669948 and https://review.opendev.org/581165 | 19:36 |
corvus | clarkb, 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 |
corvus | actually, 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 |
clarkb | I'll look. | 19:40 |
corvus | and 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 |
clarkb | fwiw 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 hanging | 19:40 |
clarkb | but 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 code | 19:41 |
corvus | aw, i wish i hadn't read that :( | 19:41 |
clarkb | (the debugger does though so maybe can figure it out with breakpoints) | 19:41 |
clarkb | I also note the console says the service worker is disabled | 19:41 |
clarkb | which maybe is the problem if we need a service worker to render and animation frame? | 19:42 |
*** jamesmcarthur has joined #zuul | 19:42 | |
clarkb | might 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 beta | 19:42 |
tristanC | corvus: 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 proposal | 19:42 |
clarkb | or behavior change that will break us when next release happens | 19:42 |
clarkb | corvus: responded on https://review.opendev.org/#/c/669948/6 | 19:45 |
corvus | tristanC: 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 them | 19:45 |
corvus | clarkb: 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 |
clarkb | yes | 19:46 |
clarkb | and I agree with you that the wheel mirror and pypi mirror paths should be set by the top level input and not inferred | 19:47 |
corvus | yeah, 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 IRC | 19:48 | |
clarkb | log streaming is not broken in firefox 67.04 but is in 69.0.5 | 19:49 |
clarkb | er thats 69.0.b5 | 19:49 |
clarkb | anyone have a firefox 68 to test? | 19:49 |
corvus | i'm running 68 and log streaming works | 19:49 |
clarkb | ok 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 agaqin | 19:50 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: configure-pypi-mirrors: add initial role https://review.opendev.org/581165 | 19:50 |
clarkb | still broken but different! | 19:52 |
clarkb | the black window for the xterm.js terminal is now tiny instead of full window si | 19:52 |
clarkb | the black window for the xterm.js terminal is now tiny instead of full window size | 19:52 |
clarkb | bah | 19:52 |
clarkb | and then there it expands | 19:52 |
*** michael-beaver has joined #zuul | 19:54 | |
clarkb | I guess the other thing I can try is updating xterm.js and uses the artifacts from check to test on newer FF | 19:54 |
openstackgerrit | James E. Blair proposed zuul/nodepool master: Remove opendev-promote-javascript-content https://review.opendev.org/671851 | 19:54 |
corvus | Shrews: thanks for catching that :) | 19:54 |
clarkb | tristanC: ^ do you know why we cap xterm at <3.14.0? | 19:57 |
clarkb | I seem to recall they updated and broke something | 19:57 |
clarkb | git log to the rescure | 20:01 |
clarkb | can't set the theme on newer xterm /me reads docs to see if that was renamed | 20:01 |
clarkb | huh issue may have been fixed 9 days ago and is in the 3.14.5 release looks like /me tries this | 20:04 |
tristanC | clarkb: maybe dropping the cap is enough | 20:04 |
clarkb | tristanC: ya thats what I'm thinking. Regenerating a lock file as soon as I figure out that process again then will push a change up | 20:05 |
clarkb | there is a fix we need too from corvus I think | 20:05 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: configure-pypi-mirrors: add initial role https://review.opendev.org/581165 | 20:08 |
openstackgerrit | Clark Boylan proposed zuul/zuul master: Update xterm to >= 3.14.5 https://review.opendev.org/671858 | 20:09 |
*** jamesmcarthur has joined #zuul | 20:20 | |
*** jamesmcarthur has quit IRC | 20:26 | |
clarkb | ok thats just broken now | 20:28 |
clarkb | something wrong with a dimensions property that can't be found. QUickly getitng out of my comfort area here :) | 20:29 |
clarkb | the good news is that is the dimensions of the renderer which implies maybe they have changed things to fix whatever problem I have | 20:31 |
corvus | tristanC: 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 |
corvus | mordred: ^ you may also have opinions | 20: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 |
mordred | corvus: I never have opinions | 20:35 |
* mordred reads | 20:35 | |
openstackgerrit | Clark Boylan proposed zuul/zuul master: Update xterm to >= 3.14.5 https://review.opendev.org/671858 | 20:36 |
clarkb | If I can read js (I can't) then that may do it | 20:36 |
mordred | corvus: so ... the ;s are not needed but are allowed - using them seems to be a thing longer-term js people still do by habit | 20:36 |
corvus | mordred: so you're seeing lack-of-semis in newer jsx? | 20:38 |
mordred | corvus: it's apparently quite the style war in javascript land | 20:38 |
mordred | there are adamant camps, it seems, on both sides | 20:38 |
* corvus backs away slowly | 20:38 | |
mordred | https://flaviocopes.com/javascript-automatic-semicolon-insertion/ | 20:38 |
mordred | yeah | 20:38 |
corvus | either way is easy enough: (setq js2-strict-missing-semi-warning nil) | 20:38 |
mordred | I picked up "no semi" when we did the original angular with the standard.js style applied | 20:39 |
mordred | (also, as a python programmer, I find the semi-less code more pleasant to look at) | 20:39 |
corvus | okay, on balance i think i'm just going customize rjsx-mode to match what we have :) | 20:39 |
mordred | BUT - in my weekend personal website hacking, I ahve consistently seen both styles come through | 20:39 |
mordred | corvus: kk | 20: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 |
mordred | corvus: 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 happen | 20:40 |
mordred | corvus: +1000 | 20:40 |
corvus | (setq js2-strict-missing-semi-warning nil) | 20:40 |
corvus | (setq js-indent-level 2) | 20:40 |
corvus | and done! | 20:41 |
mordred | sweet! | 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 |
mordred | I have to admit - go has actually made me come around to being a fan of hard-tabs | 20:42 |
corvus | i'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 it | 20:42 |
clarkb | tabs are fewer bytes too for that precious wirespeed | 20:42 |
mordred | I've very much enjoyed setting ts=2 for my golang files and being able to actually read them | 20:42 |
openstackgerrit | Merged zuul/nodepool master: Remove opendev-promote-javascript-content https://review.opendev.org/671851 | 20:47 |
fungi | and i'm back | 20:47 |
fungi | time to see what all this scrollback's about | 20:47 |
fungi | clarkb: if it helps, not all ff is broken? i just streamed a job console with ff67 no problem | 20:50 |
fungi | are you sure it's not an extension breaking you? | 20:50 |
fungi | oh, now that i read further, you're on 79 and corvus's 68 works | 20:51 |
fungi | i'm living in the past | 20:51 |
fungi | (just like jethro tull) | 20:51 |
fungi | s/79/69/ | 20:52 |
mordred | corvus, 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 null | 20:53 |
clarkb | fungi: ya confirmed works on 67 and 68 but not 69 beta 5 or 6 | 20:53 |
mordred | I think that means we're not actually using anything from that except for the tenant | 20:53 |
mordred | that, 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 did | 20:54 |
mordred | (also - yay for docs pointing out things like that) | 20:54 |
fungi | "jsx" pronounced like "deus ex" | 20:54 |
*** jamesmcarthur has joined #zuul | 20:55 | |
fungi | also i like explicit semicolons, but then again i probably still have stockholm syndrome from writing in c | 20:56 |
fungi | i my c past, i preferred 3-space tabs. i must be a rebel | 20:57 |
*** jamesmcarthur has quit IRC | 21:00 | |
mordred | fungi: I have never heard anyone advocate for 3-space tabs. you impress me | 21:03 |
fungi | i have plenty of other terrible opinions but i'll save them for next week | 21:03 |
clarkb | ok I'm quickly out of my depth debugging this jsx dimensions of window thing | 21:10 |
clarkb | but hoping to brute force it with the browser debugger | 21:10 |
openstackgerrit | Clark Boylan proposed zuul/zuul master: Update xterm to >= 3.14.5 https://review.opendev.org/671858 | 21:16 |
clarkb | debugger says that should exist | 21:16 |
clarkb | and seems to have values that look right | 21:16 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Display artifacts in build page https://review.opendev.org/671865 | 21:21 |
*** jeliu_ has quit IRC | 21:25 | |
clarkb | ok 671858 works in chrome now but not in ff 69b6 | 21:29 |
clarkb | oh wait it eventually loaded about 30 lines of text in ff | 21:29 |
fungi | the old code works in chrome too, right? | 21:29 |
clarkb | fungi: yes | 21:30 |
clarkb | so this is maybe the slightest of improvements on ff but not what I would call functional | 21:30 |
*** EvilienM is now known as EmilienM | 21:30 | |
clarkb | http://logs.openstack.org/58/671858/3/check/zuul-build-dashboard/ef1ec96/npm/html if you want to test it yourself | 21:30 |
*** jamesmcarthur has joined #zuul | 21:32 | |
*** jamesmcarthur has quit IRC | 21:36 | |
openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add mirror_info documentation https://review.opendev.org/669948 | 21:43 |
*** tosky has quit IRC | 21:43 | |
corvus | clarkb: okay, updated with your suggestions ^ | 21:43 |
fungi | clarkb: yeah, the test is working fine for me on ff67, fwiw | 21:45 |
fungi | nice and speedy | 21:46 |
fungi | streams thousands of lines of backlog in long-running jobs in a matter of a few seconds and then updates with new lines cleanly | 21:46 |
fungi | makes me wonder if the ff69 beta builds are regressed terribly somehow | 21:47 |
corvus | clarkb: seems to work as before in ff68 | 21:47 |
clarkb | I'm told there are webrender changes in ff69 (though it is disabled in my build) | 21:48 |
clarkb | I'm trying to test it with webrender enabled | 21:48 |
clarkb | (maybe the broke the other thing in the prcess) | 21:48 |
clarkb | nope no luck | 21:54 |
clarkb | so now I have it working quickly | 22:08 |
clarkb | I disabled hardware acceleration | 22:08 |
clarkb | so it seems related to rendering somewhere | 22:08 |
clarkb | and going to default boring settings seems to work | 22:08 |
fungi | oh fun | 22:30 |
fungi | disabled hardware accel settings in ff itself, or in your desktop environment/windowing system? | 22:30 |
clarkb | in ff itself. It isn't enabled by default on linux unless you are on nightly which they enabled for webrender | 22:31 |
clarkb | so I'm guessing it is something related to the webrender changes after all | 22:31 |
clarkb | but in the lower level acceleration tooling | 22:31 |
fungi | somewhere in about:config? | 22:32 |
clarkb | yes layers.acceleration.force-enabled | 22:32 |
clarkb | and friends | 22:32 |
fungi | huh. it's set true for me | 22:33 |
fungi | i must have done that a while back | 22:33 |
fungi | or debian sets it in its builds in unstable | 22:33 |
fungi | it's showing as in a modified status, so thinking i must have set it myself | 22:33 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Display artifacts in build page https://review.opendev.org/671865 | 23:00 |
*** rlandy has quit IRC | 23:07 | |
*** mattw4 has quit IRC | 23:08 | |
*** jamesmcarthur has joined #zuul | 23:08 | |
*** sgw has quit IRC | 23:14 | |
*** jamesmcarthur has quit IRC | 23:22 | |
*** jamesmcarthur has joined #zuul | 23:23 | |
*** michael-beaver has quit IRC | 23:23 | |
*** jamesmcarthur has quit IRC | 23:28 | |
*** igordc has quit IRC | 23:48 | |
*** jamesmcarthur has joined #zuul | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!