Thursday, 2021-06-10

*** bhagyashris_ has joined #zuul01:12
*** bhagyashris has quit IRC01:20
*** timburke has quit IRC02:00
*** bhavikdbavishi has joined #zuul03:15
*** bhavikdbavishi1 has joined #zuul03:18
*** bhavikdbavishi has quit IRC03:23
*** bhavikdbavishi1 is now known as bhavikdbavishi03:23
*** bhavikdbavishi1 has joined #zuul03:40
*** bhavikdbavishi has quit IRC03:40
*** bhavikdbavishi1 is now known as bhavikdbavishi03:40
*** bhavikdbavishi1 has joined #zuul03:45
*** bhavikdbavishi has quit IRC03:50
*** bhavikdbavishi1 is now known as bhavikdbavishi03:50
*** josefwells has quit IRC04:31
*** timburke has joined #zuul04:47
*** marios has joined #zuul05:00
*** marios is now known as marios|ruck05:01
*** opendevreview has quit IRC05:26
*** opendevreview has joined #zuul05:28
opendevreviewGuillaume Chauvel proposed zuul/zuul master: WIP gitlab quick-start  https://review.opendev.org/c/zuul/zuul/+/79554005:28
*** timburke has quit IRC05:47
*** manoj_katari has joined #zuul06:02
*** bhagyashris_ is now known as bhagyashris06:08
*** bhavikdbavishi has quit IRC06:34
*** rav has joined #zuul06:45
ravIs there an issue on Zuul? My commits do not seem to run zuul suddenly06:46
ravhttps://review.opendev.org/c/x/networking-infoblox/+/79559806:49
ravzuul not running for this06:49
ravok it works..06:53
*** rpittau|afk is now known as rpittau06:54
*** hashar has joined #zuul06:56
*** jpena|off is now known as jpena06:58
*** tosky has joined #zuul07:03
opendevreviewGuillaume Chauvel proposed zuul/zuul master: WIP gitlab status context  https://review.opendev.org/c/zuul/zuul/+/79552507:37
opendevreviewSimon Westphahl proposed zuul/zuul master: Stop jobs on gearman disconnect  https://review.opendev.org/c/zuul/zuul/+/71472209:30
opendevreviewSimon Westphahl proposed zuul/zuul master: Stop jobs on gearman disconnect  https://review.opendev.org/c/zuul/zuul/+/71472209:31
*** jpena is now known as jpena|lunch11:37
*** manoj_katari has quit IRC11:47
*** Shrews has joined #zuul11:53
opendevreviewSimon Westphahl proposed zuul/zuul master: Route streams to different zones via finger gateway  https://review.opendev.org/c/zuul/zuul/+/66496512:10
*** josefwells has joined #zuul12:13
opendevreviewSimon Westphahl proposed zuul/zuul master: Use connection registry in fingergw routing  https://review.opendev.org/c/zuul/zuul/+/79366612:26
opendevreviewSimon Westphahl proposed zuul/zuul master: Close fingergw connection handler on input termination  https://review.opendev.org/c/zuul/zuul/+/79366712:26
opendevreviewSimon Westphahl proposed zuul/zuul master: Support ssl encrypted fingergw  https://review.opendev.org/c/zuul/zuul/+/66495012:26
opendevreviewSimon Westphahl proposed zuul/zuul master: Combine fingergw certificate options  https://review.opendev.org/c/zuul/zuul/+/79367112:26
mhuHi zuul-maint, change 729240 has been stuck with two CR+2 for a while, could it get the final workflow +1?12:30
*** jpena|lunch is now known as jpena12:34
*** yoctozepto is now known as Guest157012:36
*** Guest1570 is now known as yoctozepto12:36
*** manoj_katari has joined #zuul13:14
mordredmhu: done13:24
mhuthanks!13:30
opendevreviewBenjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism  https://review.opendev.org/c/zuul/zuul-jobs/+/79533613:40
opendevreviewBenjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism  https://review.opendev.org/c/zuul/zuul-jobs/+/79533613:41
*** apevec has joined #zuul13:41
opendevreviewBenjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism  https://review.opendev.org/c/zuul/zuul-jobs/+/79533614:06
opendevreviewBenjamin Schanzel proposed zuul/zuul-jobs master: Add a meta log upload role with a failover mechanism  https://review.opendev.org/c/zuul/zuul-jobs/+/79533614:19
opendevreviewFelix Edel proposed zuul/zuul master: Lock/unlock nodes on executor server  https://review.opendev.org/c/zuul/zuul/+/77461014:21
*** manoj_katari has quit IRC14:45
*** timburke has joined #zuul14:47
opendevreviewMerged zuul/zuul master: test zuul-client build-info  https://review.opendev.org/c/zuul/zuul/+/75878315:00
avass[m]mnaser: I'll push a patch to vexxhost/curl in a bit that adds some more jobs.15:52
avass[m]mnaser: is it possible to stack PRs like you can stack changes in gerrit somehow?15:53
*** apevec[m] has joined #zuul15:54
corvusavass: you can Depends-On in the PR message15:56
avass[m]corvus: suppose I could do that15:57
corvusotherwise, i'm not quite sure what happens if you open a second pr which is a superset of commits in the first pr15:57
avass[m]me neither. technically what I want to do is open a PR that targets the PR, but even if that works zuul wouldn't run anything (unprotected branches) :)15:58
*** marios|ruck is now known as marios|out16:02
*** jpena is now known as jpena|off16:17
mnaseravass[m]: maybe you could just take my branch and do a PR on top of it?16:17
mnaserHaving my commit already there16:17
avass[m]yeah I could just rebase against your branch that way16:18
avass[m]I'll probably put my jobs in a separate file for now to avoid merge conflicts :P16:19
*** marios|out has quit IRC16:21
*** rpittau is now known as rpittau|afk16:24
avass[m]mnaser: https://github.com/vexxhost/curl/pull/216:30
*** mhu has quit IRC16:45
mnaseravass[m]: awesome!!!17:23
mnasertrivial review fwiw => https://review.opendev.org/c/zuul/zuul-jobs/+/79532917:23
avass[m]approved it17:25
*** rav has quit IRC17:38
opendevreviewMerged zuul/zuul-jobs master: Bump default Helm version to 2.17.0  https://review.opendev.org/c/zuul/zuul-jobs/+/79532917:44
avass[m]mnaser: off for a bit now, feel free to pick that up17:55
mnaseravass[m]: cool, will do18:55
*** hashar has quit IRC20:00
avass[m]mnaser: I'm not sure I understand what the OVERRIDE_CC and OVERRIDE_CXX variables do. I don't find any reference to them in the repo or know about any tools using that20:00
mnaseravass[m]: i dont know but they are extra jobs that do that /shrug20:01
mnaseri think that's soemthing gcc/g++ picked up20:01
clarkbmight be something autoconf uses?20:01
mnaseror autoconf yeah20:01
avass[m]mnaser: and I have a feeling we may have gotten the matrix expansion wrong. reading the docs it seems like instead of using inheritance there should be jobs setting T=<something> and another job not using those vars and instead setting OVERRIDE_CC/CXX?20:02
mnaseravass[m]: https://docs.travis-ci.com/user/environment-variables/#defining-multiple-variables-per-item20:02
mnaser> If you need to specify several environment variables for each build, put them all on the same line in the env array:20:03
avass[m]mnaser: yeah that's what I mean20:03
mnaserbut20:03
mnaserthat means the travis ci jobs are like20:03
mnaserrunning the same thing20:03
mnaserin several different places?20:03
mnaseravass[m]: https://travis-ci.org/github/curl/curl/builds/772948405 could be help?20:04
mnaserit seems like20:04
avass[m]except jobs have to be unique, to they only run if the packages or something else is different. but it seems like the variables on the first line actually controll what the tests/builds do20:04
mnaserthere is actually jobs with the same env variables20:05
avass[m]mnaser: I'll check that20:05
mnaserso you may be right20:05
mnaserso a job without OVERRIDE_CC doesnt exist20:05
mnaserso id throw them in.. just in case?  to keep things consistent20:06
avass[m]now I'm more confused20:06
mnaseryeah i think the docs make it messy20:07
mnaserthere's not two jobs, its just one20:07
mnaserthey get amended20:07
avass[m]weird.20:07
avass[m]I think we should match the travis-ci list to begin with.20:08
mnaseryes, agreed20:08
mnaseralso,20:08
mnaserhttps://www.google.com/search?q=%22OVERRIDE_CC%22+%22travis%2220:08
mnaserliterally only travis doign this lol20:08
avass[m]except that's also seagate/CURL20:09
avass[m]maybe it's something that used to control something but no one remembered to remove the variables in CI?20:09
fungicruft: <jargon> (back-formation from "crufty") Anything unpleasant that accumulates over time.20:11
avass[m]I found this: https://github.com/curl/curl/pull/4640 which supposedly fixes the ngtcp2 builds, but I also find no reference to those variables there.20:15
mnaseryeah it seems like it my have been cruft20:18
mnaserbut something to bring up later down the line20:18
avass[m]yeah20:18
* avass[m] starts merging jobs20:18
*** zenkuro has quit IRC20:32
avass[m]mnaser: the rest seems straight forward except for the arm/ppc/s390x jobs so I think I could finish this tomorrow20:41
*** apevec has quit IRC21:10
mnaseravass[m]: yay awesome.  i think i can add some aarch64 systems into our nodepool too21:22
fungiwe do have images for that21:23
*** SpamapS has quit IRC21:35
*** spamaps[m] is now known as SpamapS21:36
*** SpamapS has quit IRC21:38
*** SpamapS has joined #zuul21:38
mnaserfwiw happily running 4.5.0 - https://curl.zuul.vexxhost.dev/status22:00
*** Shrews has quit IRC22:04
mordredmnaser: woot!23:15

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!