clarkb | ianw: nodepool dib stack is +A | 00:02 |
---|---|---|
ianw | clarkb: thanks, when the image promotes i'll pull onto nb01 and make sure centos builds | 00:02 |
ianw | ok, 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 |
clarkb | ianw: check docker ps -a and docker image list and compare against docker hub and stuff | 00:04 |
clarkb | (I still haven't found a great workflow for that though) | 00:04 |
ianw | hrm, it still seems to have fungi's change. | 00:05 |
fungi | did you edit the reference container? | 00:06 |
fungi | down/up redeploys from that | 00:06 |
fungi | if you pulled it should have created a new reference though | 00:07 |
ianw | i thought we decided "docker-compose up -d" did pull, but maybe not | 00:07 |
fungi | nope | 00:07 |
fungi | pull is separate | 00:07 |
clarkb | up -d will restart if a newer image is present but it won't get a newer image itself | 00:09 |
ianw | ahh, ok, pulled and digests are the same | 00:09 |
ianw | todo: is to look into why logs are going to syslog AND /var/log/containers | 00:10 |
fungi | there is a logging.conf in there | 00:10 |
ianw | fungi: oohhh, you think the container is logging to /dev/log directly? | 00:11 |
ianw | ok, i have killed the gerritbot ssh connection and it has come back automatically | 00:12 |
ianw | so, that bit, i'm going to call done | 00:12 |
ianw | looks 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/container | 00:13 |
fungi | ianw: it's a theory. i think i saw a syslog in there when i was fiddling that file to get debug logging out of it | 00:13 |
fungi | ahh, yeah, then no idea really | 00:14 |
ianw | hrm, perhaps in rsyslog you need to tell it to stop, like [L] in mod_rewrite | 00:16 |
ianw | yes you do | 00:20 |
fungi | so it was falling through | 00:24 |
*** openstackgerrit has joined #opendev | 00:25 | |
openstackgerrit | Merged openstack/project-config master: Add publish-to-pypi template for sushy-oem-idrac https://review.opendev.org/c/openstack/project-config/+/764100 | 00:25 |
openstackgerrit | Ian Wienand proposed opendev/system-config master: install-docker: stop duplicated rsyslog logs https://review.opendev.org/c/opendev/system-config/+/764102 | 00:25 |
ianw | #status log removed eavesdrop from emergency, gerritbot should be fixed after 763892 & 763927 | 00:29 |
openstackstatus | ianw: finished logging | 00:29 |
openstackgerrit | Ian Wienand proposed opendev/gerritbot master: Update container to use Python 3.8 builder https://review.opendev.org/c/opendev/gerritbot/+/764104 | 00:31 |
*** sboyron has quit IRC | 00:52 | |
*** mlavalle has quit IRC | 01:08 | |
*** tkajinam has quit IRC | 01:31 | |
*** tkajinam has joined #opendev | 01:32 | |
*** hamalq has joined #opendev | 02:45 | |
*** hamalq has quit IRC | 02:49 | |
*** hamalq has joined #opendev | 03:00 | |
*** hamalq has quit IRC | 03:05 | |
ianw | ok, building centos images with the updated dib now | 03:47 |
*** ykarel|away has joined #opendev | 04:13 | |
*** ykarel|away has quit IRC | 04:34 | |
*** raukadah is now known as chandankumar | 04:52 | |
*** ykarel|away has joined #opendev | 05:00 | |
*** ykarel|away is now known as ykarel | 05:07 | |
frickler | mnaser: 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.txt | 05:51 |
frickler | humm, 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 #opendev | 06:07 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/763930 | 06:07 |
openstackgerrit | Merged opendev/system-config master: install-docker: stop duplicated rsyslog logs https://review.opendev.org/c/opendev/system-config/+/764102 | 07:14 |
*** ysandeep|away is now known as ysandeep | 07:15 | |
*** DSpider has joined #opendev | 07:18 | |
*** marios has joined #opendev | 07:20 | |
*** sboyron has joined #opendev | 07:22 | |
*** lpetrut has joined #opendev | 07:22 | |
*** ralonsoh has joined #opendev | 07:25 | |
*** eolivare has joined #opendev | 07:36 | |
*** ykarel_ has joined #opendev | 07:37 | |
*** rpittau|afk is now known as rpittau | 07:37 | |
*** ykarel has quit IRC | 07:40 | |
*** iurygregory is now known as iurygregory|afk | 07:48 | |
*** slaweq has joined #opendev | 07:53 | |
*** hashar has joined #opendev | 08:06 | |
*** ykarel_ is now known as ykarel | 08:34 | |
*** fressi has joined #opendev | 08:41 | |
*** tosky has joined #opendev | 08:50 | |
*** andrewbonney has joined #opendev | 08:56 | |
*** mgoddard has joined #opendev | 09:03 | |
openstackgerrit | Merged openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/763930 | 09:09 |
*** iurygregory|afk is now known as iurygregory | 09:10 | |
*** roman_g has joined #opendev | 09:15 | |
*** sean-k-mooney1 has joined #opendev | 09:16 | |
*** sean-k-mooney has quit IRC | 09:17 | |
*** d34dh0r53 has quit IRC | 09:17 | |
*** d34dh0r53 has joined #opendev | 09:21 | |
openstackgerrit | Daniel Blixt proposed zuul/zuul-jobs master: Use script to populate test file tree fixtures https://review.opendev.org/c/zuul/zuul-jobs/+/764062 | 10:06 |
openstackgerrit | Daniel Blixt proposed zuul/zuul-jobs master: Allow mirror push to delete current branch https://review.opendev.org/c/zuul/zuul-jobs/+/764152 | 10:27 |
*** dtantsur|afk is now known as dtantsur | 10:33 | |
*** roman_g has quit IRC | 10:42 | |
*** hashar has quit IRC | 10:50 | |
*** roman_g has joined #opendev | 11:06 | |
*** mgoddard has quit IRC | 11:12 | |
*** ysandeep is now known as ysandeep|afk | 11:22 | |
*** mgoddard has joined #opendev | 11:23 | |
*** roman_g has quit IRC | 11:59 | |
*** ysandeep|afk is now known as ysandeep | 12:26 | |
*** slaweq_ has joined #opendev | 12:29 | |
*** slaweq has quit IRC | 12:33 | |
*** j4m3s__ has joined #opendev | 13:04 | |
*** ysandeep is now known as ysandeep|brb | 13:08 | |
*** mgoddard has quit IRC | 13:15 | |
*** mgoddard has joined #opendev | 13:17 | |
*** tkajinam has quit IRC | 13:31 | |
*** tkajinam has joined #opendev | 13:32 | |
*** hashar has joined #opendev | 13:38 | |
*** rosmaita has left #opendev | 13:48 | |
*** ysandeep|brb is now known as ysandeep | 13:55 | |
*** sshnaidm is now known as sshnaidm|afk | 13:56 | |
openstackgerrit | Merged openstack/diskimage-builder master: Remove the unused coding style modules https://review.opendev.org/c/openstack/diskimage-builder/+/759791 | 14:02 |
*** auristor has quit IRC | 14:04 | |
*** walshh_ has joined #opendev | 14:06 | |
*** auristor has joined #opendev | 14:08 | |
*** sarmienm has joined #opendev | 14:09 | |
*** sarmienm has left #opendev | 14:10 | |
*** rpittau is now known as rpittau|bbl | 14:29 | |
*** sshnaidm|afk is now known as sshnaidm | 14:34 | |
openstackgerrit | Sorin Sbârnea proposed zuul/zuul-jobs master: Remove ansible-lint path exclusions https://review.opendev.org/c/zuul/zuul-jobs/+/731471 | 14:44 |
zbr | frickler: 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 |
zbr | it 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 slaweq | 14:58 | |
openstackgerrit | Alec 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/+/764054 | 15:03 |
*** hashar has quit IRC | 15:14 | |
*** roman_g has joined #opendev | 15:41 | |
*** rpittau|bbl is now known as rpittau | 15:44 | |
*** sean-k-mooney1 is now known as sean-k-mooney | 15:47 | |
*** lpetrut has quit IRC | 15:52 | |
*** ykarel is now known as ykarel|away | 15:52 | |
*** lpetrut has joined #opendev | 16:01 | |
*** ysandeep is now known as ysandeep|away | 16:12 | |
*** ykarel|away has quit IRC | 16:14 | |
*** chandankumar is now known as raukadah | 16:30 | |
*** d34dh0r53 has quit IRC | 17:00 | |
*** roman_g has quit IRC | 17:01 | |
*** d34dh0r53 has joined #opendev | 17:03 | |
*** marios is now known as marios|out | 17:03 | |
openstackgerrit | Matt McEuen proposed openstack/project-config master: New Project Request: airship/vino https://review.opendev.org/c/openstack/project-config/+/763889 | 17:05 |
openstackgerrit | Matt McEuen proposed openstack/project-config master: New Project Request: airship/sip https://review.opendev.org/c/openstack/project-config/+/763888 | 17:06 |
*** marios|out has quit IRC | 17:15 | |
*** d34dh0r53 has quit IRC | 17:20 | |
openstackgerrit | Matt McEuen proposed openstack/project-config master: New Project Request: airship/vino https://review.opendev.org/c/openstack/project-config/+/763889 | 17:34 |
openstackgerrit | Thierry Carrez proposed openstack/project-config master: Enable py38 tests for openstack/ptgbot https://review.opendev.org/c/openstack/project-config/+/764223 | 17:34 |
openstackgerrit | Merged zuul/zuul-jobs master: Refresh intermediate TLS certs for testing https://review.opendev.org/c/zuul/zuul-jobs/+/764023 | 17:52 |
*** mgoddard has quit IRC | 18:04 | |
*** lpetrut has quit IRC | 18:06 | |
*** eolivare has quit IRC | 18:11 | |
*** sshnaidm has quit IRC | 18:23 | |
*** sshnaidm has joined #opendev | 18:24 | |
*** hashar has joined #opendev | 18:25 | |
*** sshnaidm has quit IRC | 18:28 | |
*** sshnaidm has joined #opendev | 18:28 | |
*** hashar has quit IRC | 18:30 | |
*** rpittau is now known as rpittau|afk | 18:52 | |
*** andrewbonney has quit IRC | 18:53 | |
*** avass has quit IRC | 18:59 | |
*** yourname_ has joined #opendev | 19:00 | |
*** hashar has joined #opendev | 19:03 | |
*** yourname_ is now known as avass | 19:11 | |
*** avass has quit IRC | 19:12 | |
*** yourname_ has joined #opendev | 19:18 | |
*** yourname_ is now known as avass | 19:25 | |
openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Fix typo with container_images siblings logic https://review.opendev.org/c/zuul/zuul-jobs/+/764230 | 19:31 |
*** dtantsur is now known as dtantsur|afk | 19:33 | |
openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Switch to quay.io for container-build jobs https://review.opendev.org/c/zuul/zuul-jobs/+/764237 | 20:02 |
*** sboyron has quit IRC | 20:41 | |
*** ralonsoh has quit IRC | 20:42 | |
ianw | nb01 has gone nuts again and filled up the disk | 20:52 |
fungi | should we consider creating another nb and distributing the load further | 20:55 |
fungi | ? | 20:55 |
ianw | i'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 it | 20:56 |
*** sboyron has joined #opendev | 21:00 | |
*** DSpider has quit IRC | 21:08 | |
*** fressi has quit IRC | 21:16 | |
fungi | yay! https://github.com/go-gitea/gitea/pull/13686 | 21:28 |
fungi | that just merged | 21:28 |
fungi | closes a longstanding issue i opened | 21:28 |
*** hashar has quit IRC | 21:32 | |
fungi | mmm, 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 routes | 22:08 |
fungi | i guess that could work, but relies on them accurately matching all potential git client user agent strings | 22:08 |
fungi | agent.startsWith("git/") || agent.startsWith("JGit/") | 22:09 |
fungi | luca has suggested checking what libgit2 uses | 22:09 |
ianw | yeah, i mean i assume they know that ... but it really seems like a terrible way to do things IMO | 22:09 |
ianw | your 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 fail | 22:10 |
fungi | it risks being rather a bit of whack-a-mole as they discover less popular git clients "winging it" with user agents | 22:10 |
ianw | i guess all sorts of plugins are assuming that those endpoints are owned by pollygerrit | 22:15 |
clarkb | seems like making it repos/x/foo may be the best long term option. Difficulty there is figuring out the transition | 22:18 |
ianw | yeah, 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 IRC | 23:08 | |
*** slaweq has quit IRC | 23:13 | |
*** sboyron has joined #opendev | 23:26 | |
*** dviroel has quit IRC | 23:39 | |
fungi | i couldn't take it any more, and asked the obvious question in a review comment | 23:40 |
ianw | fungi: 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 too | 23:44 |
sean-k-mooney | why dont the use a differnt url like /contib /extention instead of x | 23:44 |
sean-k-mooney | that was the first thing the came to mind when i heard of the issue | 23:45 |
fungi | sean-k-mooney: sure, but then what if someone wants to name their git repository contrib/foo or extension/bar? | 23:45 |
sean-k-mooney | true | 23:45 |
sean-k-mooney | im gkind of suprise it toplevel to be honest | 23:45 |
fungi | my point is that they should just not try to serve git-over-http and the webapp from the same path to begin with | 23:46 |
sean-k-mooney | yep | 23:46 |
ianw | yeah, i mean even making these polygerrit endpoints under _polygerrit/<foo> would have been a much safer initial choice | 23:46 |
sean-k-mooney | kind of like zuul you have /api for the api and the ui is seperate | 23:46 |
ianw | modulo, yes, just giving everything it's own namespace | 23:46 |
fungi | and 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 other | 23:46 |
sean-k-mooney | yep | 23:47 |
sean-k-mooney | and if you really want once they are spercate you can flatten them with a revers proxy | 23:47 |
sean-k-mooney | or apache rerite rules | 23:47 |
fungi | as the developer of the service you can control what namespace(s) one subsystem might use but not the other... choose wisely | 23:48 |
sean-k-mooney | its much hardder to go the other way | 23:48 |
*** tosky has quit IRC | 23:51 | |
ianw | i'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/!