Thursday, 2020-12-17

*** tosky has quit IRC00:01
openstackgerritIan Wienand proposed zuul/nodepool master: Require dib 3.5.0  https://review.opendev.org/c/zuul/nodepool/+/76743800:24
*** rfolco has joined #zuul00:38
*** wuchunyang has joined #zuul00:55
*** wuchunyang has quit IRC01:15
*** rfolco has quit IRC01:20
*** hamalq_ has quit IRC01:24
*** evrardjp has quit IRC01:24
*** Pilou has quit IRC01:24
*** mnasiadka has quit IRC01:24
*** parallax has quit IRC01:24
*** Open10K8S has quit IRC01:24
*** jbryce has quit IRC01:24
*** leoluk has quit IRC01:24
*** wuchunyang has joined #zuul01:28
*** hamalq_ has joined #zuul01:30
*** evrardjp has joined #zuul01:30
*** Pilou has joined #zuul01:30
*** mnasiadka has joined #zuul01:30
*** parallax has joined #zuul01:30
*** Open10K8S has joined #zuul01:30
*** jbryce has joined #zuul01:30
*** leoluk has joined #zuul01:30
*** wuchunyang has quit IRC01:54
*** sanjayu_ has quit IRC02:02
*** sanjayu_ has joined #zuul02:03
*** hamalq_ has quit IRC03:00
*** bhavikdbavishi has joined #zuul03:08
*** rlandy|bbl is now known as rlandy03:59
*** bhavikdbavishi has quit IRC04:02
*** wuchunyang has joined #zuul04:03
*** mgoddard has quit IRC04:03
*** wuchunyang has quit IRC04:08
*** rlandy has quit IRC04:10
*** bhavikdbavishi has joined #zuul04:17
openstackgerritMerged zuul/nodepool master: Require dib 3.5.0  https://review.opendev.org/c/zuul/nodepool/+/76743804:21
*** wuchunyang has joined #zuul04:47
*** wuchunyang has quit IRC04:52
*** mgoddard has joined #zuul05:05
*** bhavikdbavishi has quit IRC05:25
ianwrestarted builders with ^ to hopefully get good centos-8 builds out05:29
*** evrardjp has quit IRC05:33
*** evrardjp has joined #zuul05:33
*** mgoddard has quit IRC06:00
*** bhavikdbavishi has joined #zuul06:04
*** vishalmanchanda has joined #zuul06:17
*** reiterative has quit IRC06:22
*** reiterative has joined #zuul06:23
*** bhavikdbavishi has quit IRC06:45
*** mgoddard has joined #zuul07:04
*** bhavikdbavishi has joined #zuul07:24
*** mgoddard has quit IRC08:01
*** jcapitao has joined #zuul08:03
*** jpena|off is now known as jpena08:07
*** hashar has joined #zuul08:10
*** bhavikdbavishi1 has joined #zuul08:28
*** bhavikdbavishi has quit IRC08:29
*** bhavikdbavishi1 is now known as bhavikdbavishi08:29
openstackgerritDaniel Pawlik proposed zuul/zuul-operator master: Replace Fedora 31 with Fedora 32 in zuul-operator-functional  https://review.opendev.org/c/zuul/zuul-operator/+/76747308:37
*** mgoddard has joined #zuul09:02
*** tosky has joined #zuul09:12
*** bhavikdbavishi has quit IRC09:22
*** bhavikdbavishi has joined #zuul09:43
lyrHi. My nodepool-launcher is looping build / destroying instances in Openstack since it's unable to connect. How can I reproduce the SSH it's attempting to troubleshoot that ?09:43
*** bhavikdbavishi1 has joined #zuul09:46
*** bhavikdbavishi has quit IRC09:47
*** bhavikdbavishi1 is now known as bhavikdbavishi09:47
*** nils has joined #zuul09:51
*** mgoddard has quit IRC10:01
*** mgoddard has joined #zuul10:05
lyrI tried ssh -o "StrictHostKeyChecking=no" -i /var/lib/nodepool/.ssh/id_rsa zuul-worker@172.16.42.17, seems to be the wrong key10:20
tobiashlyr: nodepool doesn't try to login but just gathers the hostkeys10:20
lyrok, but I should be able to SSH in nonetheless ?10:22
avasslyr: yeah if authorized_keys are set up correctly10:40
avassbut all you need to do to emulator nodepool-launcher you just need to run ssh-keyscan10:41
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Add more installation instructions  https://review.opendev.org/c/zuul/zuul-client/+/76599910:42
*** tosky has quit IRC10:47
*** tosky has joined #zuul10:47
*** rpittau|afk is now known as rpittau10:59
openstackgerritMatthieu Huin proposed zuul/zuul-client master: encrypt: add --public-key argument  https://review.opendev.org/c/zuul/zuul-client/+/76531311:19
*** hashar is now known as hasharLunch11:36
*** jcapitao is now known as jcapitao_lunch11:47
*** rfolco has joined #zuul12:16
*** wuchunyang has joined #zuul12:31
*** jpena is now known as jpena|lunch12:36
*** wuchunyang has quit IRC12:36
*** piotrowskim has joined #zuul12:38
*** rlandy has joined #zuul13:00
avassIs it possible to interpolate the nodepool config from environment like zuul.conf does? I believe I've seen it but can't find it in the docs now13:04
avassoh or maybe secrets could just be part of the environment duh13:08
tobiashavass: afaik it's possible (unless the changes did merge)13:09
avasstobiash: do you mean not possible?13:10
tobiashit should be possible, but I don't find the according changes right now13:10
tobiashhttps://review.opendev.org/c/zuul/zuul/+/57909113:11
tobiashhrm, abandoned13:11
avassthat's zuul/zuul though13:12
avassbut it should work by adding secrets to the pods environment right? unless nodepool removes those for any reason13:12
*** jcapitao_lunch is now known as jcapitao13:12
tobiashhere it is: https://review.opendev.org/c/zuul/nodepool/+/71959913:13
tobiashand zuul should have the same mechanism, just didn't find the change13:14
tobiashand guess what, that change misses a doc change :/13:14
avassah that explains it13:15
tobiashsee https://review.opendev.org/c/zuul/nodepool/+/719599/5/nodepool/tests/fixtures/config_validate/good.yaml for the syntax13:15
avasslooks like the same syntax as configparser13:15
avassoh but without the 's' at the end13:15
tobiashkeep in mind that the vars must be prefixed with NODEPOOL_13:16
avassalright13:16
tobiashzuul supports the same: https://opendev.org/zuul/zuul/src/branch/master/zuul/cmd/__init__.py#L13513:18
avassyep we're already using that13:18
tobiashzuul also has it documented, so just the nodepool docs are missing13:19
*** jpena|lunch is now known as jpena13:31
openstackgerritTristan Cacqueray proposed zuul/nodepool master: config: support for interpollation with a closing `)s` and add doc  https://review.opendev.org/c/zuul/nodepool/+/76755613:34
*** bhavikdbavishi has quit IRC13:36
*** hasharLunch is now known as hashar13:47
*** zenkuro has joined #zuul14:23
*** bhavikdbavishi has joined #zuul14:25
*** bhavikdbavishi1 has joined #zuul14:27
*** bhavikdbavishi has quit IRC14:29
*** bhavikdbavishi1 is now known as bhavikdbavishi14:29
*** SpamapS has joined #zuul15:09
*** SpamapS has quit IRC15:10
*** SpamapS has joined #zuul15:12
*** hashar has quit IRC15:30
*** zenkuro has quit IRC15:31
*** zenkuro has joined #zuul15:31
*** zenkuro has quit IRC15:36
*** bhavikdbavishi has quit IRC15:36
*** zenkuro has joined #zuul15:36
*** bhavikdbavishi has joined #zuul15:38
*** bhavikdbavishi has quit IRC15:50
*** bhavikdbavishi has joined #zuul15:53
*** wuchunyang has joined #zuul16:32
*** wuchunyang has quit IRC16:37
*** hashar has joined #zuul16:38
pabelangerfungi: mordred: do you mind looking at https://review.opendev.org/c/zuul/zuul-jobs/+/767361 that will get us working with tox_siblings16:39
corvuspabelanger: can you catch me up on how using tox facilitates siblings with non-python projects?16:39
pabelangercorvus: sure, the basic idea is, we have container build tooling in zuul. That project isn't python, but we use tox to manage the python dependencies needed to builder the containerfile: https://github.com/ansible/network-ee16:42
mordredcorvus: it's a non-python project that has python depends - so the tox env installs all the python depends into a virtualenv - but the repo in question doesn't itself have python code to install into the virtualenv16:42
* mordred shuts up and lets pabelanger drive :)16:42
pabelangerto avoid the need of  installing a setup.cfg file, for the project name, I've updated the tox role to support passing it in via zuul.project.short_name16:42
corvusoh interesting16:43
pabelangerat the moment, we don't have a good story for zuul job, that needs cross-project python dependencies, outside of using tox16:43
corvuspabelanger: ansible-builder is the thing you want to have CRD for?16:47
corvuspabelanger: tox installs https://github.com/ansible/network-ee/blob/main/requirements.txt as specified in https://github.com/ansible/network-ee/blob/main/tox.ini and that's what's used to build the container image?16:48
pabelangercorvus: yes, ansible-builder right now16:49
corvuspabelanger: there might be others in the future?  like some dependency that ansible-builder has?16:50
pabelangeryes! that is possible16:50
pabelangerbut right now, we don't have that dependency16:50
corvuspabelanger: but to be clear, it's not the stuff in the container right?  it's not https://github.com/ansible/network-ee/blob/main/requirements_combined.txt16:50
pabelangercorvus: right, that is what ansible-builder generates16:50
pabelangerand it uses16:50
corvuspabelanger: 767361 makes sense to me, and sounds reasonable since ultimately ansible-buildir is a python program.  an alternative might be to make an ansible-builder role that either installs ansible-builder from pypi or from the zuul src dir if it's present.  i don't see a reason to do that if you're happy with tox though.16:52
corvuspabelanger: thanks for the catch up :)16:52
*** bhavikdbavishi has quit IRC16:52
pabelangeryes, I think a role is fair too16:53
corvuspabelanger: out of curiosity is quay.io/ansible/python-builder a copy/fork of opendev's?16:54
pabelangerI thought of hacking on some sort of generic virtualenv zuul job, that used ZUUL_SIBLINGS logic to prepare a virtualenv for the user16:54
*** ikhan has quit IRC16:54
*** jcapitao has quit IRC16:54
pabelangercorvus: it is, yes. except centos/ubi based16:54
corvuspabelanger: neat :)16:55
pabelangeryah, people in ansible are really excited by it.  It helps solves the distro package / pip package issue nicely16:55
pabelangerusing bindep.txt / requirements / upper-constraints16:55
*** ikhan has joined #zuul16:58
corvusmhu: 765999 approved but see comment from tobiash (i think re-ordering those is a good idea)17:04
*** piotrowskim has quit IRC17:06
mhucorvus, thanks, looking17:07
*** jpena is now known as jpena|off17:08
mordredpabelanger, corvus: I don't have any bandwith so have a low opinion threshold - but is there any value in considering an opendev/python-base:ubi variant so there doens't have to be a fork? I'm not usually a fan of multi-distro container images for things like zuul - but the argument could certainly be made that it's a valid use case for a base image17:10
corvusmordred: probably worth moving this to #opendev; but if the tripleo/rdo folks might be interested in using it then sounds like it could be a cool collaboration with ansible17:13
mordredcorvus: yah - I was going to ask there- but it didn't seem pabelanger was in there17:13
corvusi dunno if there's an advantage to using it for zuul?17:13
pabelangerlet me join #opendev17:15
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Reorganize the installation documentation, specify versions  https://review.opendev.org/c/zuul/zuul-client/+/76759217:18
mhutobiash, corvus ^17:18
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Zuul Cache role with s3 implementation.  https://review.opendev.org/c/zuul/zuul-jobs/+/76480817:19
corvusmhu: i left a -1 on 765313 (cc tobiash)17:19
openstackgerritTristan Cacqueray proposed zuul/nodepool master: config: support for interpollation with a closing `)s` and add doc  https://review.opendev.org/c/zuul/nodepool/+/76755617:21
openstackgerritMerged zuul/zuul-client master: Add more installation instructions  https://review.opendev.org/c/zuul/zuul-client/+/76599917:25
openstackgerritMerged zuul/zuul-client master: API, CLI: Allow use of tenant-scoped API URLs  https://review.opendev.org/c/zuul/zuul-client/+/76520317:28
openstackgerritMatthieu Huin proposed zuul/zuul-client master: encrypt: add --public-key argument  https://review.opendev.org/c/zuul/zuul-client/+/76531317:37
*** zenkuro has quit IRC17:42
*** zenkuro has joined #zuul17:42
*** hamalq has joined #zuul17:50
*** hamalq_ has joined #zuul17:52
*** hamalq has quit IRC17:56
*** vishalmanchanda has quit IRC18:06
*** rpittau is now known as rpittau|afk18:20
*** hashar is now known as hasharDinner19:03
*** nils has quit IRC19:09
*** AJaeger has joined #zuul19:18
*** AJaeger has quit IRC19:56
*** hasharDinner is now known as hashar20:11
*** rfolco has quit IRC21:09
*** hashar has quit IRC21:14
*** josefwells has joined #zuul21:18
josefwellsHello zuulers.  I'm back again this week.. Trying now to just use the examples tutorial straight up with gerrit and the whole bit21:20
josefwellsmy web interface doesn't seem to be coming up:21:20
josefwellsweb_1           | 2020-12-17 21:19:07,289 WARNING zuul.zk.ZooKeeper: Retrying zookeeper connection21:20
fungijosefwells: https://zuul-ci.org/docs/zuul/discussion/components.html if you're firewalling between the server where zuul-web is running and the zookeeper daemon, make sure the right ports are open21:21
josefwellsI don't see anything crazy in the zookeeper logs, it seems to ahve started21:21
josefwellsthis is all using the docker-compose21:22
fungiahh21:22
fungiso they're running on the same host21:22
josefwellsyes21:22
fungijust in different containers21:22
josefwellsright21:22
fungijosefwells: and you're deploying tip of master, or the most recent versioned release?21:23
josefwellsyes, as of a couple days ago21:23
fungicurrent master (soon to be 4.0.0) requires tls for zookeeper communication, so you might double-check the configuration against this: https://zuul-ci.org/docs/zuul/howtos/zookeeper.html#zk-encrypted-connections21:25
josefwellsok, I'll pull and check the config as well21:25
fungibasically these are the two things i can think of which would produce that error. well or the zookeeper daemon not running at all21:28
fungior other zk connection details being mismatched for some reason21:28
mordredfungi: yah - although I'd still expect the docker-compose to work since we run it in the gate21:31
fungiright21:31
josefwellsI can dump out my zk logs if that would help21:32
fungipoint being we run all of that on every commit, so something's different21:32
josefwellsgerrit came up, so I didn't notice till I got to the point where I was supposed to see stuff on web21:32
josefwellsnot sure how your check ensures that everything is really up21:33
fungicould the "Retrying zookeeper connection" warning be a red herring and the zuul-web container merely races the zk container and so keeps retrying after start until it can connect?21:34
josefwellsLast week I was trying this with a modified setup that used our enterprise github, but I ended up with trouble getting my static external nodes to have working ssh21:34
josefwellsso I figured I would go back to basics21:34
josefwellsnotably last week, I just left off the zoo.cfg and all the cert stuff, because I didn't have it last time I messed with this21:34
josefwellsand I didn't hit these problems with zookeeper21:35
fungiare you sure zuul-web didn't connect to zk? does it continually log that it's retrying to connect?21:37
fungior was it just the once or a few times?21:37
josefwellsfungi: if you mean a transient thing, no, once the network comes up (wait-to-start-gearman finishes) and I get endless zookeeper connection retries21:38
fungigot it21:38
fungibecause yeah these different daemons are hopefully robust against asynchronous starting (we consider it a bug if they're not), but if it never establishes a connection then that suggests connectivity issues or some sort of configuration mismatch (or zookeeper isn't starting at all?)21:39
josefwellsYes, I saw the removal of ansible from some containers and some better checking for databases to be not just up, but functional, etc21:40
josefwellsI had a patch when this first was added to help with athat21:40
*** zenkuro has quit IRC21:51
josefwellssigh.. deleting all containers/volumes/networks and bringing it all back up seems to work22:10
josefwellsI love computers22:10
corvusjosefwells: you turned it off and back on again! :)22:11
funginondeterminism as a service22:12
fungiheisenberg uncertainty computing22:12
corvusjosefwells: but seriously, maybe there was some local state from before a zk-related change that wasn't accounted for22:12
corvusjosefwells: for example, i think the quickstart now runs zk with tls, but didn't before; we have release notes for version upgrades like that, but for a small test installation running latest master it may just be easier to rebuild it22:14
josefwellscorvus: for sure.  I'm still no docker expert, so it was only way in the back of my mind that I thought the volumes or other bits might be interfering here22:16
josefwellsThis whole process is "take it apart, put it back together", I've really built a career on those simple steps.22:17
josefwellssometimes there are extra screws, etc.. clearly overengineered to begin with22:17
* corvus hands josefwells a box of random screws22:18
* josefwells dumps the box in my other box22:18
mordredthe best part about boxes of screws is that no matter how many screws are in the box, it never contains the screw I need22:28
fungii've tapped some die blocks with various diameters and thread pitches so i can sort mine22:30
fungiexcept making the machine screw checker was the fun part, not actually sorting the screws. so instead i still have a giant pile of assorted machine screws and end up just ordering new packs of what i need22:31
fungithough i do at least have an indexed set of baggies with reference screws i can use to work out what i need to order when i have something missing some screws22:32
*** wuchunyang has joined #zuul22:35
*** wuchunyang has quit IRC22:40
*** rlandy has quit IRC23:23
*** Tahvok has quit IRC23:24
*** Tahvok_ has joined #zuul23:24
*** Tahvok_ is now known as Tahvok23:24
*** tosky has quit IRC23:43

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