Tuesday, 2023-02-14

clarkbfungi: I've approved the first change in that stack which should address the base jobs issue.00:23
ianwclarkb: [2023-02-14T00:22:24.158Z] [sshd-SshDaemon[11a8735d](port=22)-nio2-thread-2] DEBUG com.google.gerrit.sshd.ChannelIdTrackingUnknownChannelReferenceHandler : channelInitialized(ChannelSession[id=0, recipient=-1]-ServerSessionImpl[admin@/]) updated last tracked channel ID null => 000:29
ianwamazing what happens when you actually register the handler :)00:29
clarkbthat was it then00:29
ianwthis also probably "fixes" the problem, in so much as the original "fixed" anything.  because the channel tracker never initialized anything, it's logic of ignoring the channel error if the channel had ever been opened never triggered00:31
ianwi'm going to talk a quick walk, but will reorganise all the changes and reply to the original bug00:33
clarkbianw: I think I understand what you are saying but having a hard time parsing that :)00:33
clarkbthe channel isn't being found because they didn't fully register the tracker whcih elads to the ssh connection issues?00:33
clarkbbut only if it needs to lookup the channel tracker?00:34
clarkbenjoy your walk I can read the bug update :)00:34
ianwyeah, basically "lastTracked" in "if ((lastTracked != null) && (channelId <= lastTracked.intValue()))" is never != null00:36
opendevreviewMerged opendev/system-config master: Farewell limestone  https://review.opendev.org/c/opendev/system-config/+/87342802:23
opendevreviewIan Wienand proposed opendev/system-config master: [wip] gerrit: increase ssh channel debugging  https://review.opendev.org/c/opendev/system-config/+/87321402:41
*** yadnesh|away is now known as yadnesh04:28
*** jpena|off is now known as jpena08:25
*** gibi_ is now known as gibi09:00
*** jpena is now known as jpena|off09:39
*** jpena|off is now known as jpena10:06
*** dviroel|out is now known as dviroel10:58
*** dasm|off is now known as dasm13:10
*** yadnesh is now known as yadnesh|away13:31
*** \join_subline is now known as \join_weakmayors14:36
clarkbno new cert complaints!16:11
fungiinfra-root: config-core: any additional opinions on caching a new version of cirros on our nodes? https://review.opendev.org/87324417:19
clarkblet me double check urls and I can approve it17:20
clarkbwe may be able to remove some old cirros images too17:20
fungisee my comments on the review, yeah17:20
fungialso 873429 and 873430 are more cleanup changes for clouds we no longer use for nodepool17:21
clarkbfungi: ya I think even if 0.3.4 is still used we probably download it once a month and can just let the jobs do that at runtime17:21
fungithat was my thinking too. if opensearch isn't showing a version as used in its retention period, then it's probably not necessary to cache17:22
fungiwhich means we could drop everything except 0.5.2 while adding 0.6.117:22
fungiso 0.3.4, 0.3.5, 0.4.017:23
fungiand 0.5.117:23
fungii'll propose that as a followup change17:24
*** jpena is now known as jpena|off17:24
clarkbfungi: the inap/iweb/packethost cleanups have been approved17:26
clarkbI shold actually try to dedicate a good chunk of today to reviews. I feel like I'm falling way behind on those17:26
opendevreviewJeremy Stanley proposed openstack/project-config master: Stop caching infrequently-used CirrOS images  https://review.opendev.org/c/openstack/project-config/+/87373517:28
opendevreviewMerged openstack/project-config master: Cache Cirros 0.6.1 images  https://review.opendev.org/c/openstack/project-config/+/87324417:34
opendevreviewMerged opendev/system-config master: Finish cleaning up packethost references  https://review.opendev.org/c/opendev/system-config/+/87342918:07
opendevreviewMerged opendev/system-config master: Final cleanup of internap/inap/iweb references  https://review.opendev.org/c/opendev/system-config/+/87343018:07
opendevreviewMerged opendev/system-config master: mailman: set web auto field size to "AutoField"  https://review.opendev.org/c/opendev/system-config/+/87333719:45
clarkbianw: I've approved the docker signed-by key change20:01
ianwthanks i'll keep an eye on deployment20:01
clarkbleft a note in there for some quick version support I dug up. Seems like as far back as xenial may do it even20:01
fungikeep in mind there was a point at which binary pgp files were supported there but asc encoded file support came later20:03
opendevreviewMerged opendev/system-config master: install-docker: switch from deprecated apt-key  https://review.opendev.org/c/opendev/system-config/+/87280821:03
clarkbianw: looks like codesearch deploy failed on ^21:34
clarkbthough apparently I never loaded ssh keys so I can't pull logs up quickly21:34
clarkbcodesearch itself is still running though21:34
clarkbianw: Failed to update apt cache: unknown reason21:36
fungi104.130.172.34 is the new held lists9921:37
clarkbI think I see the issue. Trying to confirm on codesearch then will push a fix21:38
opendevreviewClark Boylan proposed opendev/system-config master: Fix docker.gpg.asc path in sources.list entry  https://review.opendev.org/c/opendev/system-config/+/87375421:41
clarkbianw: fungi ^ I think that corrects it21:41
clarkbI did that by hand on codesearch and `sudo apt-get update` was happy afterwards21:41
fungiusually the .gpg extension means binary format and .asc is ascii-encoded22:01
clarkbmakes sense. In this case it is ascii encoded so .asc is correct we just have to point at it properly22:01
fungii would have expected it to be either docker.gpg or docker.asc, in retrospect, but fixing it to be consistent with the configuration so it works is what matters for now yes22:03
clarkboh you mean .gpg and .asc instead of .gpg and .gpg.asc22:07
fungiyes, .asc is generally interpreted as ascii-armored gpg data in that context22:15
ianwclarkb: thanks!  ... how did all the tests pass with that?23:23
ianwWARNING: The following packages cannot be authenticated!23:25
ianw  containerd.io docker-ce-cli docker-ce23:25
ianwAuthentication warning overridden.23:25
ianw       "allow_unauthenticated": false, is disabled23:30
ianwso why was it allowed?!23:30
clarkbianw: oh because we don't sign our mirrors either and we disable that23:32
clarkbvia /etc/apt/preferences.d or soething23:32
ianwhrm, yeah23:34
ianwi think now we can specify that in the sources line23:36
ianw23.04 is also going to switch to deb822 format, which is going to break ~ everything23:48
clarkbthats for the sources.list files?23:49
ianwsources.list will go with individual files in /etc/apt/sources.list.d, in the more verbose format, aiui23:51

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