Wednesday, 2019-12-04

*** rlandy has quit IRC00:40
*** mattw4 has quit IRC00:53
*** tosky has quit IRC00:56
*** igordc has quit IRC01:44
*** igordc has joined #zuul01:51
*** armstrongs has joined #zuul01:51
*** paladox has quit IRC02:01
*** armstrongs has quit IRC02:01
*** paladox has joined #zuul02:02
*** igordc has quit IRC02:43
*** bhavikdbavishi has joined #zuul03:26
*** bhavikdbavishi1 has joined #zuul03:30
*** bhavikdbavishi has quit IRC03:32
*** bhavikdbavishi1 is now known as bhavikdbavishi03:32
*** yoctozepto has quit IRC03:42
*** pcaruana has joined #zuul06:01
*** raukadah is now known as chkumar|ruck06:12
*** yoctozepto has joined #zuul06:17
*** sshnaidm|afk is now known as sshnaidm06:27
*** bhavikdbavishi has quit IRC06:50
openstackgerritSimon Westphahl proposed zuul/zuul master: Fix exceptions on reconfig with changing window  https://review.opendev.org/69702307:06
*** fdegir has quit IRC07:14
*** fdegir has joined #zuul07:15
*** bhavikdbavishi has joined #zuul07:32
*** jcapitao has joined #zuul07:37
*** yolanda has quit IRC08:18
*** jpena|off is now known as jpena08:18
*** pcaruana has quit IRC08:24
*** johanssone has quit IRC08:24
*** lennyb has quit IRC08:24
*** irclogbot_1 has quit IRC08:24
*** klindgren_ has quit IRC08:24
*** corvus has quit IRC08:24
*** ianw has quit IRC08:24
*** aspiers has quit IRC08:24
*** notnone has quit IRC08:24
*** gothicmindfood has quit IRC08:24
*** bhavikdbavishi has quit IRC08:24
*** dmellado has quit IRC08:24
*** mugsie has quit IRC08:24
*** donnyd has quit IRC08:24
*** shanemcd has quit IRC08:24
*** adam_g has quit IRC08:24
*** jkt has quit IRC08:24
*** mnasiadka has quit IRC08:24
*** webknjaz has quit IRC08:24
*** EmilienM has quit IRC08:24
*** frickler has quit IRC08:24
*** Diabelko has quit IRC08:24
*** fdegir has quit IRC08:24
*** sgw has quit IRC08:24
*** reiterative has quit IRC08:24
*** AJaeger has quit IRC08:24
*** dmsimard has quit IRC08:24
*** swest has quit IRC08:24
*** openstackgerrit has quit IRC08:24
*** SotK has quit IRC08:24
*** jpena has quit IRC08:24
*** evrardjp has quit IRC08:24
*** jcapitao has quit IRC08:24
*** zbr has quit IRC08:24
*** themroc has quit IRC08:24
*** sugaar has quit IRC08:24
*** mgoddard has quit IRC08:24
*** logan- has quit IRC08:24
*** ChrisShort has quit IRC08:24
*** dcastellani has quit IRC08:24
*** zxiiro has quit IRC08:24
*** jlk has quit IRC08:24
*** amotoki has quit IRC08:24
*** fbo has quit IRC08:24
*** clayg has quit IRC08:24
*** ttx has quit IRC08:24
*** smyers has quit IRC08:24
*** persia_ has quit IRC08:24
*** mauved has quit IRC08:24
*** coldtom has quit IRC08:24
*** gouthamr has quit IRC08:24
*** sshnaidm has quit IRC08:24
*** pots has quit IRC08:24
*** daniel2 has quit IRC08:24
*** gundalow has quit IRC08:24
*** wxy-xiyuan has quit IRC08:24
*** fungi has quit IRC08:24
*** stevthedev has quit IRC08:24
*** mmedvede has quit IRC08:24
*** gmann has quit IRC08:24
*** dustinc has quit IRC08:24
*** samccann has quit IRC08:24
*** yoctozepto has quit IRC08:24
*** paladox has quit IRC08:24
*** jangutter has quit IRC08:24
*** decimuscorvinus has quit IRC08:24
*** bstinson has quit IRC08:24
*** panda has quit IRC08:24
*** nhicher has quit IRC08:24
*** tobiash has quit IRC08:24
*** aluria has quit IRC08:24
*** chkumar|ruck has quit IRC08:24
*** tobberydberg has quit IRC08:24
*** Miouge has quit IRC08:24
*** flaper87 has quit IRC08:24
*** timburke has quit IRC08:24
*** tristanC has quit IRC08:24
*** arxcruz has quit IRC08:24
*** jhesketh has quit IRC08:24
*** pabelanger has quit IRC08:24
*** maxamillion has quit IRC08:24
*** Tahvok has quit IRC08:24
*** andreaf has quit IRC08:24
*** johnsom has quit IRC08:24
*** portdirect has quit IRC08:24
*** clarkb has quit IRC08:24
*** ssbarnea has quit IRC08:24
*** tflink has quit IRC08:24
*** tdasilva has quit IRC08:24
*** mgagne has quit IRC08:24
*** dtroyer has quit IRC08:24
*** tributarian has quit IRC08:24
*** guilhermesp has quit IRC08:24
*** iamweswilson has quit IRC08:24
*** kmalloc has quit IRC08:24
*** mordred has quit IRC08:24
*** evgenyl has quit IRC08:24
*** mnaser has quit IRC08:24
*** jtanner has quit IRC08:24
*** Shrews has quit IRC08:24
*** ChanServ has quit IRC08:24
*** jcapitao has joined #zuul08:24
*** bhavikdbavishi has joined #zuul08:24
*** fdegir has joined #zuul08:24
*** yoctozepto has joined #zuul08:24
*** pcaruana has joined #zuul08:24
*** paladox has joined #zuul08:24
*** smyers has joined #zuul08:24
*** jhesketh has joined #zuul08:24
*** sgw has joined #zuul08:24
*** zbr has joined #zuul08:24
*** dmellado has joined #zuul08:24
*** reiterative has joined #zuul08:24
*** jangutter has joined #zuul08:24
*** johanssone has joined #zuul08:24
*** AJaeger has joined #zuul08:24
*** clarkb has joined #zuul08:24
*** mugsie has joined #zuul08:24
*** persia_ has joined #zuul08:24
*** donnyd has joined #zuul08:24
*** decimuscorvinus has joined #zuul08:24
*** themroc has joined #zuul08:24
*** ttx has joined #zuul08:24
*** sugaar has joined #zuul08:24
*** bstinson has joined #zuul08:24
*** Diabelko has joined #zuul08:24
*** frickler has joined #zuul08:24
*** EmilienM has joined #zuul08:24
*** webknjaz has joined #zuul08:24
*** mnasiadka has joined #zuul08:24
*** jkt has joined #zuul08:24
*** adam_g has joined #zuul08:24
*** shanemcd has joined #zuul08:24
*** mauved has joined #zuul08:24
*** mgoddard has joined #zuul08:24
*** logan- has joined #zuul08:24
*** lennyb has joined #zuul08:24
*** dmsimard has joined #zuul08:24
*** panda has joined #zuul08:24
*** swest has joined #zuul08:24
*** ssbarnea has joined #zuul08:24
*** coldtom has joined #zuul08:24
*** gouthamr has joined #zuul08:24
*** tflink has joined #zuul08:24
*** openstackgerrit has joined #zuul08:24
*** Tahvok has joined #zuul08:24
*** nhicher has joined #zuul08:24
*** SotK has joined #zuul08:24
*** sshnaidm has joined #zuul08:24
*** irclogbot_1 has joined #zuul08:24
*** tobiash has joined #zuul08:24
*** klindgren_ has joined #zuul08:24
*** pots has joined #zuul08:24
*** corvus has joined #zuul08:24
*** daniel2 has joined #zuul08:24
*** tdasilva has joined #zuul08:24
*** ChrisShort has joined #zuul08:24
*** zxiiro has joined #zuul08:24
*** dcastellani has joined #zuul08:24
*** mgagne has joined #zuul08:24
*** dtroyer has joined #zuul08:24
*** ianw has joined #zuul08:24
*** gmann has joined #zuul08:24
*** evrardjp has joined #zuul08:24
*** jpena has joined #zuul08:24
*** gundalow has joined #zuul08:24
*** guilhermesp has joined #zuul08:24
*** kmalloc has joined #zuul08:24
*** tributarian has joined #zuul08:24
*** iamweswilson has joined #zuul08:24
*** wxy-xiyuan has joined #zuul08:24
*** clayg has joined #zuul08:24
*** fbo has joined #zuul08:24
*** amotoki has joined #zuul08:24
*** jlk has joined #zuul08:24
*** aluria has joined #zuul08:24
*** chkumar|ruck has joined #zuul08:24
*** fungi has joined #zuul08:24
*** mordred has joined #zuul08:24
*** aspiers has joined #zuul08:24
*** stevthedev has joined #zuul08:24
*** notnone has joined #zuul08:24
*** andreaf has joined #zuul08:24
*** tobberydberg has joined #zuul08:24
*** Miouge has joined #zuul08:24
*** gothicmindfood has joined #zuul08:24
*** mmedvede has joined #zuul08:24
*** flaper87 has joined #zuul08:24
*** timburke has joined #zuul08:24
*** tristanC has joined #zuul08:24
*** arxcruz has joined #zuul08:24
*** dustinc has joined #zuul08:24
*** samccann has joined #zuul08:24
*** ChanServ has joined #zuul08:24
*** evgenyl has joined #zuul08:24
*** mnaser has joined #zuul08:24
*** jtanner has joined #zuul08:24
*** Shrews has joined #zuul08:24
*** pabelanger has joined #zuul08:24
*** maxamillion has joined #zuul08:24
*** portdirect has joined #zuul08:24
*** johnsom has joined #zuul08:24
*** orwell.freenode.net sets mode: +o ChanServ08:24
*** mgoddard has quit IRC08:50
*** mgoddard has joined #zuul08:54
*** yolanda has joined #zuul08:58
*** mhu has joined #zuul08:58
mhucode like amazon!08:59
*** tosky has joined #zuul09:00
openstackgerritMatthieu Huin proposed zuul/zuul master: authentication config: add optional max_token_age, skew  https://review.opendev.org/64240809:19
*** PrinzElvis has joined #zuul09:36
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: DNM: reproducer for log stream termination  https://review.opendev.org/69724409:37
openstackgerritSimon Westphahl proposed zuul/zuul master: Fix job result metric name for node failures  https://review.opendev.org/69724809:46
openstackgerritSimon Westphahl proposed zuul/zuul master: Fix job result metric name for builds to retry  https://review.opendev.org/69724810:02
*** saneax has joined #zuul10:07
*** tosky has quit IRC10:21
*** tosky has joined #zuul10:21
*** mgoddard has quit IRC11:02
*** mgoddard has joined #zuul11:12
*** mgoddard has quit IRC11:12
*** mgoddard has joined #zuul11:13
*** zbr_ has joined #zuul11:13
*** zbr has quit IRC11:17
*** themroc has quit IRC11:51
*** jcapitao is now known as jcapitao|lunch11:59
*** sshnaidm is now known as sshnaidm|afk12:02
*** jpena is now known as jpena|lunch12:07
*** themroc has joined #zuul12:12
*** PrinzElvis has quit IRC12:55
*** Goneri has joined #zuul13:08
*** rlandy has joined #zuul13:11
*** jpena|lunch is now known as jpena13:14
*** jcapitao|lunch is now known as jcapitao13:19
*** rfolco has joined #zuul13:45
*** sshnaidm|afk is now known as sshnaidm14:39
*** bogdando has joined #zuul15:06
bogdandopabelanger: hi, mind please merging https://review.opendev.org/#/c/696337/ ?15:06
*** yolanda__ has joined #zuul15:15
*** bogdando has left #zuul15:17
*** yolanda__ has quit IRC15:18
*** swest has quit IRC15:28
*** Goneri has quit IRC15:29
*** Goneri has joined #zuul15:41
*** sshnaidm is now known as sshnaidm|afk15:57
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers  https://review.opendev.org/69693916:02
*** michael-beaver has joined #zuul16:03
*** jcapitao is now known as jcapitao|afk16:06
*** mattw4 has joined #zuul16:14
*** saneax has quit IRC16:20
*** jcapitao|afk is now known as jcapitao16:21
*** sshnaidm|afk is now known as sshnaidm16:25
*** jangutter has quit IRC16:41
*** chkumar|ruck is now known as raukadah16:43
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers  https://review.opendev.org/69693916:47
*** themroc has quit IRC17:02
*** mhu has quit IRC17:04
*** jpena is now known as jpena|off17:06
*** persia_ is now known as persia17:16
*** jcapitao has quit IRC17:16
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers  https://review.opendev.org/69693918:01
*** lennyb has quit IRC18:05
*** lennyb has joined #zuul18:18
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers  https://review.opendev.org/69693918:21
*** igordc has joined #zuul18:25
*** lennyb has quit IRC18:25
*** themroc has joined #zuul18:42
openstackgerritDavid Shrewsbury proposed zuul/nodepool master: WIP: decouple provider manager from config objects  https://review.opendev.org/69734018:47
SpamapScorvus and pabelanger: Did we have a conversation about supersedent pipelines and ordering? Do I need to put a semaphore on any of my "promote" jobs if they can't run out of order?18:47
fungiSpamapS: do you need to make sure jobs get run for every individual change after merge, or just the most recent one to merge?18:52
fungithere's no "out of order" in supercedent pipelines i can think of18:52
fungiwhen a change merges to your branch, zuul starts running the jobs for it18:53
fungiif a second change merges before those are done, its jobs are queued and wait for the first one to finish18:53
*** tosky has quit IRC18:54
fungiif yet another merges while one ahead of it is queued, it replaces that queued change18:54
fungiso you should only at most ever have two changes queued for the same branch in a supercedent pipeline, an active one and a queued one18:55
fungier, only at most ever have two changes for the same branch in a supercedent pipeline, an active one and a queued one18:55
fungithis is unlike the post pipeline, where it will run jobs for every change which merges to the branch no matter how many merge and no matter how close together in time. for those you would want a semaphore if they can't be allowed to complete out of order18:57
fungi(speaking of promote and post as examples of supercedent and independent pipelines, of course)18:58
tobiashclarkb, mhu: I've commented on https://review.opendev.org/69544618:59
tobiashto summarize: you won't need a semaphore in that case (in contrast to the normal post pipeline) :)19:02
tobiashSpamapS: but beware of file filters, they can have unexpected side effects when using supercedent pipelines19:03
*** mattw4 has quit IRC19:12
*** mattw4 has joined #zuul19:13
tobiashpabelanger, clarkb: I've commented on https://review.opendev.org/69439019:26
clarkbtobiash: thank you19:27
pabelangerSpamapS: there was a bug at one point, IIRC. But we do use a semaphore in zuul.a.c, so only 1 jobs runs between promote and periodic pipelines19:30
openstackgerritMerged zuul/zuul master: Pagure - add support for git.tag.creation event  https://review.opendev.org/67993819:31
*** mattw4 has quit IRC19:32
pabelangerSpamapS: https://review.opendev.org/650440/ was what I was thinking of19:32
*** themroc has quit IRC19:34
SpamapSfungi: just the most recent.19:36
SpamapStobiash: I have explicitly removed all files filters from our gate jobs which produce artifaces, and the promote jobs which consume them. :)19:37
SpamapSartifacts19:37
SpamapSThe main thing I was worried about is just that when we run promote, all we do is start a kubernetes rollout with the new image tags and config values that were generated in gate. So if 3 changes land in rapid succession, I want the 3rd one to be the last one that runs (and ideally, the only one)19:38
fungithe first and third will be, ultimately, because zuul doesn't know not to act on the first one before it gets to the event for the second, and by then it's too late19:40
fungii suppose it could batch up events and defer acting on them, but it doesn't do that now19:41
fungiit just handles each in the order it pops them off the events queue19:41
corvusyeah, if running only #3 is important, i think we can add a delay there19:42
corvusit should be a fairly simple code change19:42
*** mattw4 has joined #zuul19:48
clarkbcorvus: https://review.opendev.org/#/c/675403/ has been updated to address you comment if you want to rereview19:50
SpamapS1 and 3 is OK... best is just 3. :)19:53
SpamapSAnd yeah, a configurable delay and probably a window size would be good, so that you can say "wait either 30s, or 3 changes, before you start"19:54
SpamapSAnyway, not expecting anything just yet, but was chasing some other race condition issues and solving them without semaphores, and then realized maybe I need one. ;)19:54
openstackgerritMerged zuul/zuul master: Pagure - Support for branch creation/deletion  https://review.opendev.org/68511620:29
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers  https://review.opendev.org/69693920:39
*** hashar has joined #zuul20:46
*** michael-beaver has quit IRC20:52
*** rfolco has quit IRC21:06
*** pcaruana has quit IRC21:32
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: add user to shadow file too  https://review.opendev.org/69364722:08
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: create APP_DIR  https://review.opendev.org/69364622:08
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile : install sudo for nodepool-builder  https://review.opendev.org/69470922:08
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: add DEBUG environment flag  https://review.opendev.org/69484522:08
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346422:08
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346422:10
*** tosky has joined #zuul22:10
*** jamesmcarthur has joined #zuul22:17
corvusclarkb, fungi, mordred, tristanC: i think this may be the issue with the podman speculative container job: https://github.com/containers/libpod/issues/431122:18
fungioh, huh22:18
corvusShrews: ^ you'll appreciate that it seems to be a failing of libslirp.22:18
fungiyet again, the container ecosystem fails to consider ipv6?22:19
corvusslirp didn't understand ipv6 the last time i used it.  of course, that was 1994.22:20
corvusalso, someone in that bug suggests socat as a workaround22:20
fungiseems it's maybe been abandoned for 25 years?22:21
Shrewsoof. not surprising tho22:21
fungiand yeah, once again socat ftw22:21
corvusif we do that, it will mean that we would be running *two* socats -- one each to work around a container v6 bug22:21
corvusthey *will* be talking to each other22:21
corvusthis is serial22:21
corvusskopeo <-> socat <-> socat <-> registry22:21
fungiwe might as well just use serial protocol ;)22:21
clarkbat least we know socat works :)22:22
ianwupload via y-modem?22:22
fungiskopeo <-> rs232 <-> success22:22
clarkbha22:22
fungiianw: xmodem or it didn't happen22:22
corvusi really did use ymodem over the weekend22:22
corvus(i made a zigbee device, and the microntroller i used does file transfer over ymodem)22:23
fungii use ymodem on occasion for dealing with firmware transfers over serial lines22:23
fungithat is when i can't get tftp going22:23
ShrewsI used to own a modem. Is that one?22:24
* fungi shakes an acoustic coupler threateningly in Shrews's direction22:24
Shrewsok boomer22:25
fungisqueeee...*boom*22:25
corvusboomer was a great character on bsg22:26
fungii just had a flashback of answering misdialled fax calls and whistling at them to get the pitch to change22:26
*** hashar has quit IRC22:38
*** jamesmcarthur has quit IRC22:59
ianwcorvus: any idea why opendev-buildset-registry has failed @ https://zuul.opendev.org/t/zuul/status? (it just links to https://zuul.opendev.org/t/zuul/opendev-buildset-registry)23:02
corvusclarkb, fungi: ugh, the annoying thing about using socat is going to be iptables23:02
corvusianw: no, maybe the build report will say?23:03
corvusianw: i would expect a log upload failure to cause that, but i would also expect that to be a post_failure23:04
*** jamesmcarthur has joined #zuul23:04
*** jamesmcarthur has quit IRC23:05
*** jamesmcarthur_ has joined #zuul23:05
corvusclarkb, fungi: when we run the buildset registry, we rely on the docker --publish argument to bypass our iptables rules and allow incoming traffic on the port.  if we want the random socat port to be the public port, we're going to need to open that somehow.  but we haven't standardized iptables rules in zuul-jobs, so i don't know that we can do that.23:06
corvusunless we come up with a generic way to do that...rudely add a chain to the start of INPUT or something23:06
fungii have to say, the debian package description for slirp has caused some measure of nostalgia. it begins... "Slirp is a TCP/IP emulator which turns an ordinary shell account into a (C)SLIP/PPP account.  This allows shell users to use all the funky Internet applications like Netscape, Mosaic, CUSeeMe, etc."23:07
ianwcorvus: the weird thing is that i made nodepool-functional-openstack-src dependent on nodepool-functional-openstack-src ... which has failed ... but it still seems to be waiting for it23:09
fungicorvus: yeah, -I with no index defaults to the start of the chain at least23:09
ianwsorry, nodepool-functional-openstack-src dependent on the nodepool-build-image-siblings23:09
corvusianw: hrm, that's lookin like a bug then.23:10
corvusianw: you may need to dequeue it or push up a new ps.23:10
ianwthis is c1305c589f164620bd5cca61924558e9 in the logs ... seeing if anything stands out23:12
corvusthanks, that could help us build a repro case23:12
ianwso it wanted a node for the buildset job: Adding node request <NodeRequest 300-0006403416 <NodeSet ubuntu-bionic [<Node None ('ubuntu-bionic',):ubuntu-bionic>]>> for job opendev-buildset-registry to item23:13
ianwit adds the node request, but then instantly (same microsecond) goes to "is a failing item because ['at least one job failed']" http://paste.openstack.org/show/787130/23:16
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: add user to shadow file too  https://review.opendev.org/69364723:27
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: create APP_DIR  https://review.opendev.org/69364623:27
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile : install sudo for nodepool-builder  https://review.opendev.org/69470923:27
openstackgerritIan Wienand proposed zuul/nodepool master: Dockerfile: add DEBUG environment flag  https://review.opendev.org/69484523:27
openstackgerritIan Wienand proposed zuul/nodepool master: [wip] move openstack testing to use containerised daemon  https://review.opendev.org/69346423:27
clarkbcorvus: can we constrain socat ti a range then just open that?23:28
corvusclarkb: this is for run-buildset-registry, a role meant to be used on potentially any job, so there's no telling what could run there.23:29
*** rlandy is now known as rlandy|bbl23:29
corvuswe could pick a number, and it'd probably be okay.  after all, we have zuul-console running at 19885.  but we're trying to get rid of that :)23:29
corvusi guess we've picked 5000 for the registry23:29
corvusthis argument is falling apart pretty quickly.  :)23:30
ianwcorvus: i wonder if this has to do with opendev-buildset-registry being in state "waiting" ... not queued.  it should be queued to start, then nodepool-build-image-siblings should be waiting for it23:30
corvusso anyway, let's assume we do pick a number...  how do we open that port in a way that works on every zuul system?23:30
clarkbcorvus: we are already doing that in the dockerrole iirc23:31
corvusclarkb: link?23:32
clarkbwhether or not it is generic enough I dont know23:32
* clarkb looks23:32
corvusis the dockerrole install-docker?  why would that need to open a port?23:32
clarkbno where we do ipv6 socat things I think23:33
ianwcorvus: interesting ... of the live queue i just pushed https://zuul.opendev.org/t/zuul/status ... the very first change's nodepool-build-image is "queued" ... the rest are "waiting"23:33
corvusclarkb: i think the only places we do socat are push/pull intermediate registry, and that's always docker -> socat all on localhost23:34
corvusor rather skopeo -> socat23:34
clarkbcorvus: opendev/base-jobs/playbooks/buildset-registry/pre.yaml23:35
corvusclarkb: yeah, that's opendev-specific23:35
clarkbya23:35
corvusso it's not something we could put in zuul-jobs23:36
clarkbbut it is also there because of ipv623:36
fungii agree, what's good enough for opendev isn't necessarily safe for zuul23:36
clarkbif we only needed ipv4 then docker would just make it work aiui23:36
corvusclarkb: i guess you're saying we're already assuming that users may need to open 5000:5001 or have no firewall23:37
clarkbyes23:37
corvusso adding another port to that is reasonable23:37
corvus...we could use 5001 since the proxy is gone...23:37
clarkbreadme could say "ipv4 wil just work because the conatiner ecosystem understands ipv4. If you want ipv6 to work then you need to bootstrap things a bit more becuase the tooling does not understand ipv6 out of the box"23:38
clarkbor whatever and document what the range is23:38
corvuswell, i was also thinking we may want to send all trafic through socat, just because the switch to decide whether or not to use it may be complex23:39
openstackgerritIan Wienand proposed zuul/nodepool master: [dnm] docker job dependency test  https://review.opendev.org/69738623:39
openstackgerritIan Wienand proposed zuul/nodepool master: [dnm] docker dependency tests 2  https://review.opendev.org/69738723:39
openstackgerritIan Wienand proposed zuul/nodepool master: [dnm] docker dependency tests 3  https://review.opendev.org/69738823:39
ianwi feel it definitely ties together the opendev-buildset-registry job across changesets incorrectly23:45
ianwit's like the buildset registry in 697387 is waiting for 697386 ... but they're separate things23:45
corvusit doesn't know they are separate23:46
ianwbut they should be, right?23:47
corvuswe have enough information to optimize that out (in that all of the jobs we're going to run build the same images that are produced by the prior changes).  but we've only told zuul that they "provide docker images" and "require docker images", so it assumes a connection between them.23:48
corvusi think it would be nice to optimize this further, but there are some challenges.23:51
ianwcorvus: hrm, not sure i follow.  why would opendev-buildset-registry in 697387 be waiting for nodepool-build-image from 697386?23:52
corvusbecause 87 depends on 86, and opendev-buildset-registry "requires: docker-image" and nodepool-build-image "provides: docker-image"23:53
*** jamesmcarthur_ has quit IRC23:53
corvusso it's theoretically possible for nodepool-build-image in 86 to produce an image which needs to appear in the registry in 87 in order for nodepool-build-image in 87 to build with it.23:54
corvus*we* know that's not going to happen because it's the same image, but zuul doesn't.23:54
ianwohhh, ok i think the "provides" and "requires" bit i was missing23:55
corvusyeah, in practice it's proving to be a bit too coarse.  it's correct, but not efficient.23:55
*** mattw4 has quit IRC23:56
ianwright now in 697387 opendev-buildset-registry and nodepool-build-image are running concurrently23:56
ianwoh, i guess i didn't do a dependency23:56
corvusi suspect the optimization would look something like specifying in more detail exactly which images are produced/consumed.  but we'd need to make sure that we don't optimize out the currently working case23:57
corvus(or, we could explore making the optimization automatically if the relevant jobs in the buildset are the same)23:57
ianwso, optimisation aside, that it got stuck before is probably related ... i had two jobs that would "provides: docker-image" (the "normal" one, and a new one that created images with sibling installs)23:59

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