Wednesday, 2020-11-25

clarkbianw: nodepool dib stack is +A00:02
ianwclarkb: thanks, when the image promotes i'll pull onto nb01 and make sure centos builds00:02
ianwok, either i didn't pull the container, or the change didn't apply as i'd hoped, or gerritbot is still not handling disconnects :/00:04
clarkbianw: check docker ps -a and docker image list and compare against docker hub and stuff00:04
clarkb(I still haven't found a great workflow for that though)00:04
ianwhrm, it still seems to have fungi's change.00:05
fungidid you edit the reference container?00:06
fungidown/up redeploys from that00:06
fungiif you pulled it should have created a new reference though00:07
ianwi thought we decided "docker-compose up -d" did pull, but maybe not00:07
funginope00:07
fungipull is separate00:07
clarkbup -d will restart if a newer image is present but it won't get a newer image itself00:09
ianwahh, ok, pulled and digests are the same00:09
ianwtodo: is to look into why logs are going to syslog AND /var/log/containers00:10
fungithere is a logging.conf in there00:10
ianwfungi:  oohhh, you think the container is logging to /dev/log directly?00:11
ianwok, i have killed the gerritbot ssh connection and it has come back automatically00:12
ianwso, that bit, i'm going to call done00:12
ianwlooks like the logging conf should just go to stdout; which docker should capture and send to syslog, which rsyslog should capture and redirect to the /var/log/container00:13
fungiianw: it's a theory. i think i saw a syslog in there when i was fiddling that file to get debug logging out of it00:13
fungiahh, yeah, then no idea really00:14
ianwhrm, perhaps in rsyslog you need to tell it to stop, like [L] in mod_rewrite00:16
ianwyes you do00:20
fungiso it was falling through00:24
*** openstackgerrit has joined #opendev00:25
openstackgerritMerged openstack/project-config master: Add publish-to-pypi template for sushy-oem-idrac  https://review.opendev.org/c/openstack/project-config/+/76410000:25
openstackgerritIan Wienand proposed opendev/system-config master: install-docker: stop duplicated rsyslog logs  https://review.opendev.org/c/opendev/system-config/+/76410200:25
ianw#status log removed eavesdrop from emergency, gerritbot should be fixed after 763892 & 76392700:29
openstackstatusianw: finished logging00:29
openstackgerritIan Wienand proposed opendev/gerritbot master: Update container to use Python 3.8 builder  https://review.opendev.org/c/opendev/gerritbot/+/76410400:31
*** sboyron has quit IRC00:52
*** mlavalle has quit IRC01:08
*** tkajinam has quit IRC01:31
*** tkajinam has joined #opendev01:32
*** hamalq has joined #opendev02:45
*** hamalq has quit IRC02:49
*** hamalq has joined #opendev03:00
*** hamalq has quit IRC03:05
ianwok, building centos images with the updated dib now03:47
*** ykarel|away has joined #opendev04:13
*** ykarel|away has quit IRC04:34
*** raukadah is now known as chandankumar04:52
*** ykarel|away has joined #opendev05:00
*** ykarel|away is now known as ykarel05:07
fricklermnaser: do you know about any IO performance issues in vexxhost? I've seen multiple jobs failing with timeouts where the devstack setup already took like 1h instead of the usual 20m, e.g. https://f37eaa10b780bfc64fda-749ac6329d801cba1d5f2575f7d4079d.ssl.cf2.rackcdn.com/763893/2/gate/designate-pdns4/2081081/job-output.txt05:51
fricklerhumm, why did gerrit add me to cc: now instead to reviewers on https://review.opendev.org/c/opendev/gerritbot/+/764104 ? what is cc in that context anyway?05:58
*** whoami-rajat__ has joined #opendev06:07
openstackgerritOpenStack Proposal Bot proposed openstack/project-config master: Normalize projects.yaml  https://review.opendev.org/c/openstack/project-config/+/76393006:07
openstackgerritMerged opendev/system-config master: install-docker: stop duplicated rsyslog logs  https://review.opendev.org/c/opendev/system-config/+/76410207:14
*** ysandeep|away is now known as ysandeep07:15
*** DSpider has joined #opendev07:18
*** marios has joined #opendev07:20
*** sboyron has joined #opendev07:22
*** lpetrut has joined #opendev07:22
*** ralonsoh has joined #opendev07:25
*** eolivare has joined #opendev07:36
*** ykarel_ has joined #opendev07:37
*** rpittau|afk is now known as rpittau07:37
*** ykarel has quit IRC07:40
*** iurygregory is now known as iurygregory|afk07:48
*** slaweq has joined #opendev07:53
*** hashar has joined #opendev08:06
*** ykarel_ is now known as ykarel08:34
*** fressi has joined #opendev08:41
*** tosky has joined #opendev08:50
*** andrewbonney has joined #opendev08:56
*** mgoddard has joined #opendev09:03
openstackgerritMerged openstack/project-config master: Normalize projects.yaml  https://review.opendev.org/c/openstack/project-config/+/76393009:09
*** iurygregory|afk is now known as iurygregory09:10
*** roman_g has joined #opendev09:15
*** sean-k-mooney1 has joined #opendev09:16
*** sean-k-mooney has quit IRC09:17
*** d34dh0r53 has quit IRC09:17
*** d34dh0r53 has joined #opendev09:21
openstackgerritDaniel Blixt proposed zuul/zuul-jobs master: Use script to populate test file tree fixtures  https://review.opendev.org/c/zuul/zuul-jobs/+/76406210:06
openstackgerritDaniel Blixt proposed zuul/zuul-jobs master: Allow mirror push to delete current branch  https://review.opendev.org/c/zuul/zuul-jobs/+/76415210:27
*** dtantsur|afk is now known as dtantsur10:33
*** roman_g has quit IRC10:42
*** hashar has quit IRC10:50
*** roman_g has joined #opendev11:06
*** mgoddard has quit IRC11:12
*** ysandeep is now known as ysandeep|afk11:22
*** mgoddard has joined #opendev11:23
*** roman_g has quit IRC11:59
*** ysandeep|afk is now known as ysandeep12:26
*** slaweq_ has joined #opendev12:29
*** slaweq has quit IRC12:33
*** j4m3s__ has joined #opendev13:04
*** ysandeep is now known as ysandeep|brb13:08
*** mgoddard has quit IRC13:15
*** mgoddard has joined #opendev13:17
*** tkajinam has quit IRC13:31
*** tkajinam has joined #opendev13:32
*** hashar has joined #opendev13:38
*** rosmaita has left #opendev13:48
*** ysandeep|brb is now known as ysandeep13:55
*** sshnaidm is now known as sshnaidm|afk13:56
openstackgerritMerged openstack/diskimage-builder master: Remove the unused coding style modules  https://review.opendev.org/c/openstack/diskimage-builder/+/75979114:02
*** auristor has quit IRC14:04
*** walshh_ has joined #opendev14:06
*** auristor has joined #opendev14:08
*** sarmienm has joined #opendev14:09
*** sarmienm has left #opendev14:10
*** rpittau is now known as rpittau|bbl14:29
*** sshnaidm|afk is now known as sshnaidm14:34
openstackgerritSorin Sb├órnea proposed zuul/zuul-jobs master: Remove ansible-lint path exclusions  https://review.opendev.org/c/zuul/zuul-jobs/+/73147114:44
zbrfrickler: i suspect the cc comes from your configure watches on projects, a change in behavior from old version. look at your user settings.14:52
zbrit may be a guess, but it should be either from configured watches or if someone manually added you to cc instead of review for some reason.14:53
*** slaweq_ is now known as slaweq14:58
openstackgerritAlec Hothan proposed openstack/project-config master: Remove openstack-python and openstack-docs job for x/vmtp project.  https://review.opendev.org/c/openstack/project-config/+/76405415:03
*** hashar has quit IRC15:14
*** roman_g has joined #opendev15:41
*** rpittau|bbl is now known as rpittau15:44
*** sean-k-mooney1 is now known as sean-k-mooney15:47
*** lpetrut has quit IRC15:52
*** ykarel is now known as ykarel|away15:52
*** lpetrut has joined #opendev16:01
*** ysandeep is now known as ysandeep|away16:12
*** ykarel|away has quit IRC16:14
*** chandankumar is now known as raukadah16:30
*** d34dh0r53 has quit IRC17:00
*** roman_g has quit IRC17:01
*** d34dh0r53 has joined #opendev17:03
*** marios is now known as marios|out17:03
openstackgerritMatt McEuen proposed openstack/project-config master: New Project Request: airship/vino  https://review.opendev.org/c/openstack/project-config/+/76388917:05
openstackgerritMatt McEuen proposed openstack/project-config master: New Project Request: airship/sip  https://review.opendev.org/c/openstack/project-config/+/76388817:06
*** marios|out has quit IRC17:15
*** d34dh0r53 has quit IRC17:20
openstackgerritMatt McEuen proposed openstack/project-config master: New Project Request: airship/vino  https://review.opendev.org/c/openstack/project-config/+/76388917:34
openstackgerritThierry Carrez proposed openstack/project-config master: Enable py38 tests for openstack/ptgbot  https://review.opendev.org/c/openstack/project-config/+/76422317:34
openstackgerritMerged zuul/zuul-jobs master: Refresh intermediate TLS certs for testing  https://review.opendev.org/c/zuul/zuul-jobs/+/76402317:52
*** mgoddard has quit IRC18:04
*** lpetrut has quit IRC18:06
*** eolivare has quit IRC18:11
*** sshnaidm has quit IRC18:23
*** sshnaidm has joined #opendev18:24
*** hashar has joined #opendev18:25
*** sshnaidm has quit IRC18:28
*** sshnaidm has joined #opendev18:28
*** hashar has quit IRC18:30
*** rpittau is now known as rpittau|afk18:52
*** andrewbonney has quit IRC18:53
*** avass has quit IRC18:59
*** yourname_ has joined #opendev19:00
*** hashar has joined #opendev19:03
*** yourname_ is now known as avass19:11
*** avass has quit IRC19:12
*** yourname_ has joined #opendev19:18
*** yourname_ is now known as avass19:25
openstackgerritPaul Belanger proposed zuul/zuul-jobs master: Fix typo with container_images siblings logic  https://review.opendev.org/c/zuul/zuul-jobs/+/76423019:31
*** dtantsur is now known as dtantsur|afk19:33
openstackgerritPaul Belanger proposed zuul/zuul-jobs master: Switch to quay.io for container-build jobs  https://review.opendev.org/c/zuul/zuul-jobs/+/76423720:02
*** sboyron has quit IRC20:41
*** ralonsoh has quit IRC20:42
ianwnb01 has gone nuts again and filled up the disk20:52
fungishould we consider creating another nb and distributing the load further20:55
fungi?20:55
ianwi'll try tracing it back and seeing what went wrong.  it had plenty of space and should have been building centos-8-stream when i left it20:56
*** sboyron has joined #opendev21:00
*** DSpider has quit IRC21:08
*** fressi has quit IRC21:16
fungiyay! https://github.com/go-gitea/gitea/pull/1368621:28
fungithat just merged21:28
fungicloses a longstanding issue i opened21:28
*** hashar has quit IRC21:32
fungimmm, https://gerrit-review.googlesource.com/289893 proposes to switch on user agent matches to not serve the pg webapp to git clients trying to access projects whose names might otherwise conflict with the hardcoded url routes22:08
fungii guess that could work, but relies on them accurately matching all potential git client user agent strings22:08
fungiagent.startsWith("git/") || agent.startsWith("JGit/")22:09
fungiluca has suggested checking what libgit2 uses22:09
ianwyeah, i mean i assume they know that ... but it really seems like a terrible way to do things IMO22:09
ianwyour failure case is going to be "dig through the java source to figure out why things aren't being served as you think" which is pretty much the worst way to fail22:10
fungiit risks being rather a bit of whack-a-mole as they discover less popular git clients "winging it" with user agents22:10
ianwi guess all sorts of plugins are assuming that those endpoints are owned by pollygerrit22:15
clarkbseems like making it repos/x/foo may be the best long term option. Difficulty there is figuring out the transition22:18
ianwyeah, that seems about equally as hard as moving it into the hostname too, making it say "git.review.opendev.org"22:22
*** sboyron has quit IRC23:08
*** slaweq has quit IRC23:13
*** sboyron has joined #opendev23:26
*** dviroel has quit IRC23:39
fungii couldn't take it any more, and asked the obvious question in a review comment23:40
ianwfungi: thank you.  my finger has been hovering over the submit button but i couldn't do it :)  it seems like there's some other reasonable comments there now too23:44
sean-k-mooneywhy dont the use a differnt url like /contib /extention instead of x23:44
sean-k-mooneythat was the first thing the came to mind when i heard of the issue23:45
fungisean-k-mooney: sure, but then what if someone wants to name their git repository contrib/foo or extension/bar?23:45
sean-k-mooneytrue23:45
sean-k-mooneyim gkind of suprise it toplevel to be honest23:45
fungimy point is that they should just not try to serve git-over-http and the webapp from the same path to begin with23:46
sean-k-mooneyyep23:46
ianwyeah, i mean even making these polygerrit endpoints under _polygerrit/<foo> would have been a much safer initial choice23:46
sean-k-mooneykind of like zuul you have /api for the api and the ui is seperate23:46
ianwmodulo, yes, just giving everything it's own namespace23:46
fungiand any gymnastics invested in trying to make them share the same url space are going to be far more complicated than just dedicating space to one or the other23:46
sean-k-mooneyyep23:47
sean-k-mooneyand if you really want once they are spercate you can flatten them with a revers proxy23:47
sean-k-mooneyor apache rerite rules23:47
fungias the developer of the service you can control what namespace(s) one subsystem might use but not the other... choose wisely23:48
sean-k-mooneyits much hardder to go the other way23:48
*** tosky has quit IRC23:51
ianwi'm not quite sure i understand the "headless gerrit"23:59

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