Tuesday, 2021-04-06

*** dpawlik5 has quit IRC00:10
*** dpawlik7 has joined #zuul00:11
*** hamalq has quit IRC00:28
*** dpawlik7 has quit IRC00:57
*** dpawlik5 has joined #zuul00:57
*** dpawlik5 has quit IRC01:23
*** dpawlik2 has joined #zuul01:23
*** dpawlik2 has quit IRC01:33
*** dpawlik3 has joined #zuul01:36
*** evrardjp has quit IRC02:33
*** evrardjp has joined #zuul02:33
*** dpawlik9 has joined #zuul02:35
*** dpawlik3 has quit IRC02:36
*** bhavikdbavishi has joined #zuul03:30
*** bhavikdbavishi1 has joined #zuul03:33
*** bhavikdbavishi has quit IRC03:35
*** bhavikdbavishi1 is now known as bhavikdbavishi03:35
*** ykarel has joined #zuul03:51
*** dpawlik9 has quit IRC04:35
*** dpawlik6 has joined #zuul04:35
*** bhavikdbavishi has quit IRC04:52
*** dpawlik7 has joined #zuul05:34
*** dpawlik6 has quit IRC05:35
openstackgerritFelix Edel proposed zuul/zuul master: Execute builds via ZooKeeper  https://review.opendev.org/c/zuul/zuul/+/77090206:10
*** saneax has joined #zuul06:14
*** sanjayu_ has joined #zuul06:22
*** saneax has quit IRC06:24
*** jcapitao has joined #zuul06:34
*** dpawlik5 has joined #zuul06:42
*** dpawlik7 has quit IRC06:42
*** dpawlik5 has quit IRC06:52
*** dpawlik07 has joined #zuul06:55
*** dpawlik07 has quit IRC06:57
*** sshnaidm is now known as sshnaidm|afk06:58
*** dpawlik5 has joined #zuul06:58
*** dpawlik5 has quit IRC07:12
*** dpawlik5 has joined #zuul07:13
*** dpawlik5 is now known as dpawlik07:14
*** sanjayu_ has quit IRC07:23
*** sanjayu_ has joined #zuul07:24
*** dpawlik has quit IRC07:28
*** dpawlik6 has joined #zuul07:28
*** avass has quit IRC07:29
*** avass has joined #zuul07:30
*** avass has quit IRC07:32
*** avass has joined #zuul07:34
*** tosky has joined #zuul07:38
*** dpawlik6 has quit IRC07:43
*** jpena|off is now known as jpena07:51
avassLooks like tox-py27 isn't working for zuul-jobs because of a new openstacksdk version: https://zuul.opendev.org/t/zuul/build/1d2b154d0d2f4c40b8235f482179e98b08:25
avassoh checking test-requirements py<=3.5 should use an older version so maybe that's because of a new tox version?08:45
*** ykarel is now known as ykarel|lunch08:50
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Use openstacksdk 0.45.0 for python2.7  https://review.opendev.org/c/zuul/zuul-jobs/+/78489408:52
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Use openstacksdk 0.45.0 for python2.7  https://review.opendev.org/c/zuul/zuul-jobs/+/78489408:54
openstackgerritMerged zuul/zuul-jobs master: ensure-podman: Use official podman repos for ubuntu  https://review.opendev.org/c/zuul/zuul-jobs/+/76517709:17
*** holser has joined #zuul09:30
openstackgerritSorin Sbârnea proposed zuul/zuul master: Document local testing  https://review.opendev.org/c/zuul/zuul/+/76646009:41
*** jcapitao_ has joined #zuul09:49
*** jcapitao has quit IRC09:52
*** jcapitao_ has quit IRC09:57
*** sshnaidm|afk has quit IRC10:09
*** ykarel|lunch is now known as ykarel10:11
*** sshnaidm has joined #zuul10:18
*** fdegir4 is now known as fdegir10:22
*** ykarel_ has joined #zuul10:32
*** ykarel has quit IRC10:34
*** ykarel_ is now known as ykarel11:13
*** jpena is now known as jpena|lunch11:31
*** rlandy has joined #zuul11:50
*** bhavikdbavishi has joined #zuul12:11
pabelangeravass: thanks for merging podman change12:24
*** jcapitao has joined #zuul12:25
*** jpena|lunch is now known as jpena12:27
openstackgerritMerged zuul/zuul-jobs master: Document algorithm var for remove-build-sshkey  https://review.opendev.org/c/zuul/zuul-jobs/+/78398812:50
*** ykarel_ has joined #zuul12:53
*** ykarel has quit IRC12:56
*** ykarel_ is now known as ykarel13:36
*** bhavikdbavishi has quit IRC13:36
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Update operator-framwork to v1.4.2  https://review.opendev.org/c/zuul/zuul-operator/+/78445713:40
openstackgerritMerged zuul/zuul-jobs master: ensure-zookeeper: add use_tmpfs parameter  https://review.opendev.org/c/zuul/zuul-jobs/+/78398213:48
*** avass has quit IRC13:48
*** avass has joined #zuul13:50
avasscorvus: this is ready to merge right? https://review.opendev.org/c/zuul/zuul-jobs/+/78200413:50
corvusavass: yep!13:50
avasszuul-jobs test-requirements.txt has been a bit flaky today. sometimes tox-py27 gets different versions of openstacksdk (version >0.45.0 doesn't support python2.7) so hopefully that'll merge13:52
zbravass: py27 are failing everywhere for various reasons since last week. Most of my patches that had a py27 job failed.13:54
zbralso zuul-quick-start fails to start too: Status code was -1 and not [200]: Connection failure: [Errno 104] Connection reset by peer13:54
zbrhttps://zuul.opendev.org/t/zuul/builds?job_name=zuul-quick-start&project=zuul/zuul13:54
corvusavass: that's a little confusing -- sdk 46 was released in april; we've been pinned to <48 which was released in june13:55
avasscorvus: I compared two jobs that succeeded vs failed and the succeeding one got 0.45.0 and anything >0.45.0 says it's only compatible with >=python3.513:57
avassI could be missing something13:57
avassfor example: https://zuul.opendev.org/t/zuul/build/4ff471fa35b24d149f76c2cae1fae9bf/log/tox/py27-1.log#1313:58
avassvs: https://zuul.opendev.org/t/zuul/build/c28e3d1672d04dbdad7924ebba93fe83/log/tox/py27-1.log#1313:59
avassand it errors on 0.47.0 not being compatible with python2.713:59
avassit's pulling the package from two different mirrors however14:00
corvusavass: i agree, sdk >45 requires py3.  perhaps some other package has further constrained sdk to <46 for the past 9 months and that constraint was just removed.14:02
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Update operator-framwork to v1.4.2  https://review.opendev.org/c/zuul/zuul-operator/+/78445714:09
avassI couldn't figure out how to constrain the package version with a python_version range however. I'd expect it to be something like this but pip doesn't like it: https://review.opendev.org/c/zuul/zuul-jobs/+/78489414:11
corvusfungi, clarkb, mordred: ^?14:13
mordredlooking14:14
mordredI don't know - that looks reasonable to me - maybe clarkb or fungi know14:17
mordredcould also just go for simplicity and pin to <0.46 for python_version <= 3.514:18
avassthat fails with "Double requirement given: ..." instead14:19
avassbecause of the <3 as well14:19
mordredcorvus, avass : see fungi's comment in #opendev ... this could actually be a mirror issue14:19
mordred(the "why did this start failing all of a sudden")14:19
avassclarkb: I checked with daniel and the problem we saw was actually a problem where if you stacked two changes on top of eachother and the second change deletes a playbook along with the job using it zuul still tries to run that job and errors because it can't find the playbook. I'm not sure if that's related to your issue14:25
avassI think we had something similar a while ago related to branches but I can't remember what the situation looked like then but it gave a similar error from what I remember.14:27
fungicorvus: mordred: to be clear, i haven't looked into it yet. the symptoms sound similar to when fastly has directed clients to the bandersnatch mirror pypi uses as a fallback for when warehouse isn't reachable, and someone said there was a pypi outage over the weekend14:27
fungiwe've also seen some fastly endpoints continue directing clients to the fallback instead of warehouse, so it ends up depending on where in the world you are as to which you get14:28
avassfungi: yeah I should have added a "could" to my message14:28
mordred"yay"14:28
fungialso the pypi admins were saying they were going to replace that bandersnatch-built fallback with something which would provide python-requires metadata in the indices, but i have no idea if they ever got around to that14:29
corvusfungi: yeah, seems very likely it's somehow related;  though it's also true that we don't have a py 2.x cap, and so must have somehow been getting that some other way.  but maybe whatever other way that is is still in effect and when the mirror is normal again we won't need avass's patch?14:29
corvus(and, moreover, that avass's patch may not be working (or would not work in this case) because of the mirror issue?)14:30
corvusso i guess we can't really evaluate that patch until the mirror situation stabilizes14:31
avassThis is a bit harder to debug however: https://zuul.opendev.org/t/zuul/build/add02ab2cb5247ad9a9caa8a8d1c0ae014:40
corvusavass: post failure, no logs :(14:41
corvushopefully that's just a one-off failure?14:41
avassI guessing it is14:41
clarkbavass: in this case it is running a job with a second repo included but that second repo doesn't have a change to update to. We expect it to update to latest $branch but that doesn't seem to happen because the code revision for lastest $branch is laraedy there due to a fast forward emrge14:47
clarkbavass: mordred  I don't know why that metadata on the requirement doesn't work14:48
fungiright, has to be fast-forwardable, so that both the previous and new branch states are implicitly present14:48
clarkbmaybe >=3 needs to be before <= 3.6 and it is order dependnet?14:48
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Update operator-framwork to v1.4.2  https://review.opendev.org/c/zuul/zuul-operator/+/78445714:49
avasstried that and pip just doesn't seem to like multiple version constraints for python_version, "...;python_version<'3.6;python_version>='3'" doesn't work either :/14:50
clarkbhttps://www.python.org/dev/peps/pep-0508/ has the full grammar14:52
mordredavass: oh - wait ...14:52
clarkbI think you can use and14:52
mordredyeah14:52
clarkbpython_version>=3 and python_version<3.6 or similar14:52
mordredclarkb beat me to it14:53
avassthat works14:55
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Use openstacksdk 0.45.0 for python2.7  https://review.opendev.org/c/zuul/zuul-jobs/+/78489414:55
*** y2kenny has joined #zuul14:58
*** bhavikdbavishi has joined #zuul15:37
avassthere could be problems with log uploads: https://zuul.opendev.org/t/zuul/buildset/da4c8da2559c4840ac877d4b9519064e15:37
corvusavass: yeah, fungi and clarkb are looking at it in #opendev15:40
fungishould hopefully be resolved in a few minutes15:41
fungilooks like one of our storage donors is returning errors when we try to authenticate in one region15:41
-openstackstatus- NOTICE: POST_FAILURE results between 14:00 and 15:50 UTC can be safely rechecked, and were due to authentication problems in one of our storage donor regions15:55
*** ykarel has quit IRC16:11
*** hamalq has joined #zuul16:13
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Update operator-framwork to v1.4.2  https://review.opendev.org/c/zuul/zuul-operator/+/78445716:25
clarkbmore zuul users https://superuser.openstack.org/articles/ntt-and-kddi-challenges-for-sustainable-infrastructure-transformation/16:31
fungia synopsis at https://zuul-ci.org/users.html would be good16:38
*** jpena is now known as jpena|off16:58
*** jcapitao has quit IRC17:00
*** jcapitao has joined #zuul17:02
*** jcapitao has quit IRC17:05
y2kennywhat do you guys use for get gerrit event feed into IRC?17:05
clarkby2kenny: https://opendev.org/opendev/gerritbot there are opendevorg/gerritbot docker images too iirc17:06
y2kennyclarkb: thanks.  I will check it out17:06
clarkbit listens to ssh stream events and then based on a yaml config decides which events to send to irc channels17:07
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Remove command args override and update operator-framework to v1.4.2  https://review.opendev.org/c/zuul/zuul-operator/+/78418117:25
*** bhavikdbavishi1 has joined #zuul17:25
tristanCavass: corvus: fyi, if i understand correctly, the kubernetes network issue i mentioned previously is happening with the old operator-framework version used by the zuul-operator. I squahed the sdk bump with the args override fix in 784181 which should now pass the ci17:27
*** bhavikdbavishi has quit IRC17:27
*** bhavikdbavishi1 is now known as bhavikdbavishi17:27
*** sanjayu_ has quit IRC17:35
*** saneax has joined #zuul17:50
*** saneax has quit IRC17:55
*** y2kenny has quit IRC17:59
funginow that we're well past the holiday weekend, i can see increased cache memory occupied on opendev's scheduler, but active memory use remains steady and is not increasing: http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=64792&rra_id=all18:35
fungisemaphore leak fix seems to be holding up18:36
*** bhavikdbavishi has quit IRC18:42
*** bhavikdbavishi has joined #zuul18:43
*** bhavikdbavishi has quit IRC18:53
*** y2kenny has joined #zuul18:56
openstackgerritJeremy Stanley proposed zuul/zuul-website master: User story from KDDI, NEC, and NTT  https://review.opendev.org/c/zuul/zuul-website/+/78503319:04
y2kennycorvus: For https://review.opendev.org/c/zuul/zuul/+/783986, are there anything else needed before it can go in?19:05
avasstobiash: I've been able to hack the docker daemon a bit to allow a "--image" flag that it should pull on startup. It doesn't look like it should be too hard to use the docker packages to create a cli to do that without the daemon so I'll continue looking at that whenever I have time :)19:10
avasstristanC: nice19:15
*** hamalq has quit IRC19:44
*** hamalq has joined #zuul19:45
*** josefwells has joined #zuul19:47
openstackgerritJames E. Blair proposed zuul/zuul-operator master: WIP: Use kopf operator framework  https://review.opendev.org/c/zuul/zuul-operator/+/78503920:02
corvustristanC: +3;  also ^20:07
corvusi'm sure that's not going to pass tests on the first try :)20:12
tristanCcorvus: great, that looks good to me. but you removed the tests20:13
corvustristanC: oh then maybe they will pass!  ;)20:13
corvussorry that was not intentional20:13
corvusi really thought i kept them, but i think i got turned around in a rebase or something; i'll fix20:14
tristanCcorvus: if i remember correctly, one of the operatorsdk's benefit is that it manages resource ownership, how does your implementation sets the owner references?20:19
corvustristanC: kopf's "adopt()" method does that; in the utils file, when we send all the objects to k8s, we adopt them first, so everything created by the operator is owned by the original zuul crd20:20
corvuser zuul cr not crd :)20:20
corvushttps://review.opendev.org/c/zuul/zuul-operator/+/785039/1/zuul_operator/utils.py#4520:21
tristanCcorvus: i see, thanks20:24
openstackgerritMerged zuul/zuul-operator master: Remove command args override and update operator-framework to v1.4.2  https://review.opendev.org/c/zuul/zuul-operator/+/78418120:35
openstackgerritJames E. Blair proposed zuul/zuul-operator master: WIP: Use kopf operator framework  https://review.opendev.org/c/zuul/zuul-operator/+/78503920:46
openstackgerritJames E. Blair proposed zuul/zuul-operator master: Bump API version to v1alpha2  https://review.opendev.org/c/zuul/zuul-operator/+/78504720:59
corvusi think i might continue work with followup changes and squash them, just to avoid having the initial change get bigger right now20:59
corvusmnaser: https://gerrit.googlesource.com/zuul/ops/+/refs/heads/master/k8s has tls zk; you may be able to crib from that to prepare for the upgrade.  and remember 3.19.x supports tls zk already so you can do that first.21:34
corvusclarkb, fungi: before you page out all the knowledge about semaphores we obtained last week -- can you take a look at https://review.opendev.org/784523 and see if you think that would have helped?21:41
*** avass has quit IRC21:44
openstackgerritJames E. Blair proposed zuul/zuul master: Add a checkpoint release note  https://review.opendev.org/c/zuul/zuul/+/78505421:58
corvusi think we're about to wrap up the "event processing in zk" phase of the sos work; i'd like to go ahead and merge those outstanding changes, restart opendev on them, and release a checkpoint (presumably 4.2.0) so we have something to fall back on for the next phase (as we did with 4.1.0 recently)21:59
clarkbcorvus: looking22:10
corvustristanC, mordred: it sort of looks like the job failed due to pxc taking too long -- https://zuul.opendev.org/t/zuul/build/ea03794c933a41c880c51792d7371f8e/log/docker/k8s_operator_zuul-operator-f74b88756-x2mzl_default_f80b73a4-08c9-4f3f-81df-57b47edfc0d9_0.txt22:12
corvuswe only give it 4 minutes; i think we'll just need to bump the timeout, as booting a 3-node pxc cluster can take quite a while22:13
clarkbcorvus: from the commit message it makes sense. I'll dig into the code too though and leave a proper review22:13
clarkbcorvus: note we should coordinate with openstack and airship relaeses this week if we restart opendev22:13
clarkbas they are both trying to get big release artifacts out the door22:14
corvusclarkb: thanks for looking;  and noted; i think we can restart either wednesday or friday and try to minimize disruption there.22:15
fungicorvus: on a leak-ish note, not sure if you saw my comments over the weekend but we're continuing to see stuck node requests where launchers take the lock and never report completion back to the scheduler. i'm at a loss as to how to debug further... thread dump maybe?22:16
corvusfungi: yeah, that sounds promising22:17
fungii'll try to grab one before the next time i restart it22:18
fungithanks22:18
openstackgerritJames E. Blair proposed zuul/zuul-operator master: WIP: Use kopf operator framework  https://review.opendev.org/c/zuul/zuul-operator/+/78503922:19
*** rlandy is now known as rlandy|bbl22:20
*** holser has quit IRC22:23
tristanCcorvus: there seems to be a connection error for cert manager: https://0c87aaef96fbe9fbbe9d-b9bca1a4764ecffb2935d8726176eb4d.ssl.cf2.rackcdn.com/785039/2/check/zuul-operator-functional-k8s/ea03794/docker/k8s_operator_zuul-operator-f74b88756-x2mzl_default_f80b73a4-08c9-4f3f-81df-57b47edfc0d9_0.txt22:28
*** y2kenny has quit IRC22:29
clarkbcorvus: I'm trying to convince myself that union is the right operation at https://review.opendev.org/c/zuul/zuul/+/784523/1/zuul/zk/semaphore.py line 178. Should it be intersection?22:39
clarkbintersection will give you the holders that are still holding from the first set without jobs that have stopped or started which the comments in the method imply is what you want?22:39
corvustristanC: oh yeah, i think that's a transient error; afaict there's no way to determine that it's ready except to try and retry :(  there's a cert-manager bug report about it and they say the same.  i tried to minimize that, but it's timing dependent; we may be able to just sleep a couple more seconds after it looks like it's ready and reduce the chances we hit it further.  but you can see the retry in22:43
corvusaction there -- the next log line is a few seconds later and it starts the handler all agoin22:43
clarkbleft that question in a review too if it is easier to look at it in that context22:43
corvusclarkb: looking now22:43
openstackgerritJames E. Blair proposed zuul/zuul master: Periodically clean up leaked semaphores  https://review.opendev.org/c/zuul/zuul/+/78452322:47
corvusclarkb: yep, i got that wrong, thanks :)22:47
clarkbI knew I took group theory for a reason22:50
fungii had logic in two separate departments at school and still missed that22:53
corvusme going to csc310: https://1.bp.blogspot.com/-AZNV9I1nBF4/UB6SvKtIR6I/AAAAAAAAAWY/fk8_oWh9JCI/s640/midvale+school+for+the+gifted.jpg22:55
clarkbit was interseting to get the same math subjects from two departments. In CS land it was all practical application of those topics to things relevant to CS. Parsing, algorithm analysis etc. In the math department it was write all the proofs23:01
fungiyeah, i took logic in both math and philosophy departments just to compare them23:02
fungiit was interesting. a lot of it was the same, just using different notation23:02
clarkband in physics its was "this math is really difficult and I don't want you to have to take differential equiations so we're going to spend a week teaching you just enough to figure out capacitance"23:03
fungii recall introduction to astrophysics, the professor only cared that your answers were the right order of magnitude23:07
corvusfungi: i have it on good authority that "space is big"23:09
corvusread that in a guidebook somewhere23:09
openstackgerritJames E. Blair proposed zuul/zuul-operator master: WIP: Use kopf operator framework  https://review.opendev.org/c/zuul/zuul-operator/+/78503923:11

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