| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: drop rustup default run https://review.opendev.org/c/zuul/zuul-jobs/+/812272 | 00:24 |
|---|---|---|
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: rework global install https://review.opendev.org/c/zuul/zuul-jobs/+/812272 | 00:28 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: rework global install https://review.opendev.org/c/zuul/zuul-jobs/+/812272 | 00:32 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: rework global install https://review.opendev.org/c/zuul/zuul-jobs/+/812272 | 00:37 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: rework global install https://review.opendev.org/c/zuul/zuul-jobs/+/812272 | 00:43 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: verify cryptography build on Ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/812273 | 01:00 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: verify cryptography build on Ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/812273 | 02:10 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: verify cryptography build on Ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/812273 | 02:17 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: verify cryptography build on Ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/812273 | 02:21 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: verify cryptography build on Ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/812273 | 02:30 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: verify cryptography build on Ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/812273 | 02:39 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ensure-rust: verify cryptography build on Ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/812273 | 02:44 |
| *** ysandeep|out is now known as ysandeep | 03:19 | |
| opendevreview | Kendall Nelson proposed opendev/system-config master: Setup Letsencrypt for ptgbot site https://review.opendev.org/c/opendev/system-config/+/804791 | 03:25 |
| opendevreview | Merged zuul/zuul-jobs master: Test tox.requires doesn't break sibling installs https://review.opendev.org/c/zuul/zuul-jobs/+/812004 | 03:28 |
| opendevreview | Merged zuul/zuul-jobs master: More exact section matching for tox showconfig https://review.opendev.org/c/zuul/zuul-jobs/+/812160 | 03:36 |
| opendevreview | Kendall Nelson proposed opendev/zone-opendev.org master: Add CNAME for ptgbot.opendev.org https://review.opendev.org/c/opendev/zone-opendev.org/+/804790 | 03:40 |
| *** ysandeep is now known as ysandeep|brb | 04:55 | |
| *** ysandeep|brb is now known as ysandeep | 05:22 | |
| *** ykarel is now known as ykarel|mtg | 06:02 | |
| opendevreview | Ian Wienand proposed opendev/system-config master: gerrit: add its actions.config file https://review.opendev.org/c/opendev/system-config/+/812279 | 06:11 |
| opendevreview | Merged opendev/bindep master: Require packaging https://review.opendev.org/c/opendev/bindep/+/812238 | 06:25 |
| opendevreview | Merged openstack/diskimage-builder master: Move grubenv to EFI dir https://review.opendev.org/c/openstack/diskimage-builder/+/804000 | 06:46 |
| opendevreview | Merged openstack/diskimage-builder master: Support grubby and the Bootloader Spec https://review.opendev.org/c/openstack/diskimage-builder/+/804002 | 06:46 |
| opendevreview | Merged openstack/diskimage-builder master: RHEL/Centos 9 does not have package grub2-efi-x64-modules https://review.opendev.org/c/openstack/diskimage-builder/+/804816 | 06:51 |
| opendevreview | Merged openstack/diskimage-builder master: Add policycoreutils package mappings for RHEL/Centos 9 https://review.opendev.org/c/openstack/diskimage-builder/+/804817 | 06:51 |
| opendevreview | Merged openstack/diskimage-builder master: Add DIB_YUM_REPO_PACKAGE as an alternative to DIB_YUM_REPO_CONF https://review.opendev.org/c/openstack/diskimage-builder/+/804819 | 06:51 |
| *** bhagyashris_ is now known as bhagyashris|rover | 07:08 | |
| *** sshnaidm|off is now known as sshnaidm | 07:26 | |
| *** jpena|off is now known as jpena | 07:31 | |
| *** ykarel|mtg is now known as ykarel | 07:34 | |
| mrkiko | fungi: corvus: hi! I'm testing it right now. Unfortunately, the cursor movemenets are still not very consistent. I press enter on a change but I find myself in another, and the cursor goes continously up on the right angle of the screen, when the "Sync:" gets updated with the nmber of synced changes. Hope I've been clear. | 08:06 |
| mrkiko | thanks a lot for your work guys | 08:06 |
| *** ysandeep is now known as ysandeep|lunch | 08:12 | |
| *** ykarel is now known as ykarel|lunch | 09:09 | |
| *** ysandeep|lunch is now known as ysandeep | 09:17 | |
| *** ykarel|lunch is now known as ykarel | 10:10 | |
| fungi | probably it should be changed to force focus to the first selectable widget | 11:07 |
| *** jpena is now known as jpena|lunch | 11:29 | |
| *** ysandeep is now known as ysandeep|brb | 11:39 | |
| *** dviroel|out is now known as dviroel | 11:43 | |
| *** ysandeep|brb is now known as ysandeep | 12:13 | |
| *** jpena|lunch is now known as jpena | 12:22 | |
| *** ysandeep is now known as ysandeep|brb | 12:29 | |
| *** ysandeep|brb is now known as ysandeep | 12:58 | |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Setting Up Ansible For ptgbot https://review.opendev.org/c/opendev/system-config/+/803190 | 13:28 |
| fungi | diablo_rojo_phone: ianw: ^ i've tried to address the latest error there | 13:29 |
| *** TheJulia is now known as needsleep | 13:49 | |
| *** needsleep is now known as TheJulia | 13:49 | |
| *** avass[m] is now known as AlbinVass[m] | 13:55 | |
| *** dviroel is now known as dviroel|doc_apt | 13:55 | |
| corvus | mrkiko: yes, so far i think the improvements should be once you are in a change and are navigating in the commit message or in the change messages; there are still several types of buttons that haven't been updated yet. | 14:08 |
| fungi | corvus: related, not sure if you saw, i adjusted https://review.opendev.org/812214 based on your earlier feedback but left some questions for you in comments there | 14:14 |
| corvus | fungi: yeah, i think the method is more future-proof | 14:26 |
| opendevreview | Jeremy Stanley proposed ttygroup/gertty master: Make related change entries selectable https://review.opendev.org/c/ttygroup/gertty/+/812214 | 14:31 |
| fungi | corvus: so like that ^ ? | 14:31 |
| fungi | i guess i can also drop the shadow __init__() method there | 14:32 |
| opendevreview | Jeremy Stanley proposed ttygroup/gertty master: Make related change entries selectable https://review.opendev.org/c/ttygroup/gertty/+/812214 | 14:33 |
| corvus | fungi: loverly! i'm running it now; so much better :) | 14:38 |
| opendevreview | Merged ttygroup/gertty master: Make related change entries selectable https://review.opendev.org/c/ttygroup/gertty/+/812214 | 14:39 |
| fungi | awesome | 14:40 |
| *** ykarel is now known as ykarel|away | 14:42 | |
| *** ysandeep is now known as ysandeep|dinner | 16:06 | |
| *** dviroel|doc_apt is now known as dviroel | 16:16 | |
| opendevreview | Kendall Nelson proposed opendev/system-config master: Setup Letsencrypt for ptgbot site https://review.opendev.org/c/opendev/system-config/+/804791 | 16:18 |
| opendevreview | Kendall Nelson proposed opendev/zone-opendev.org master: Add CNAME for ptgbot.opendev.org https://review.opendev.org/c/opendev/zone-opendev.org/+/804790 | 16:22 |
| diablo_rojo | I think I got all of ianw's comments addressed. | 16:24 |
| diablo_rojo | If you can take a look at some point today fungi or clarkb :) | 16:24 |
| diablo_rojo | (for the ptgbot work) | 16:24 |
| fungi | yeah, i no longer see startup errors from ptgbot after my latest update to the ansible change to use the correct config file format | 16:25 |
| fungi | though it doesn't seem to log anything at all now... i suppose silence indicates success there? | 16:26 |
| *** jpena is now known as jpena|off | 16:34 | |
| fungi | diablo_rojo: i left some comments on 804790, let me know if they're not clear | 16:41 |
| fungi | also on 804791 | 16:42 |
| *** ysandeep|dinner is now known as ysandeep|out | 17:25 | |
| fungi | diablo_rojo_phone: since you seem to be otherwise occupied, i'll update those per my comments on them | 17:27 |
| opendevreview | Jeremy Stanley proposed opendev/zone-opendev.org master: Add CNAME for ptgbot.opendev.org https://review.opendev.org/c/opendev/zone-opendev.org/+/804790 | 17:43 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Setup Letsencrypt for ptgbot site https://review.opendev.org/c/opendev/system-config/+/804791 | 17:58 |
| fungi | diablo_rojo_phone: ^ i've updated those two changes with fixes | 18:01 |
| opendevreview | Ade Lee proposed zuul/zuul-jobs master: DNM https://review.opendev.org/c/zuul/zuul-jobs/+/807031 | 20:12 |
| *** diablo_rojo is now known as Guest1814 | 20:22 | |
| fungi | python 3.10.0 is officially released! | 20:42 |
| fungi | time for me to start tracking 3.11 alphas i guess | 20:43 |
| opendevreview | Merged zuul/zuul-jobs master: ensure-tox: Avoid Reinstalling tox venv if present https://review.opendev.org/c/zuul/zuul-jobs/+/735339 | 20:48 |
| opendevreview | Merged zuul/zuul-jobs master: ensure-twine: Avoid Reinstalling twine if present https://review.opendev.org/c/zuul/zuul-jobs/+/735340 | 20:58 |
| ianw | fungi: iirc it writes a .json file that is then picked up by some mostly static html, right? is the apache deployment bits for that separate? | 21:22 |
| fungi | ianw: yes, we likely need to grab the vhost config template from puppet-ptgbot | 21:29 |
| ianw | fungi: are the files in the container though? or are we reverse proxying to it? i'm not sure we worked this bit out | 21:33 |
| opendevreview | Ian Wienand proposed opendev/system-config master: gerrit: add its actions.config file https://review.opendev.org/c/opendev/system-config/+/812279 | 21:36 |
| fungi | ianw: i don't know, i hadn't been paying close attention to it until i looked on friday to see why the jobs for that change were failing. i think the static files themselves would be included in the container image, and then the bot would write its state out to the directory mapped into the container from the host | 21:41 |
| fungi | but maybe the dockerfile for it is incomplete | 21:42 |
| fungi | is there an easy way to inspect container image contents? can they be mounted onto a loop device or something, like a disk image? | 21:42 |
| ianw | in this case i'd just run it and exec bash | 21:43 |
| fungi | i meant could i just download the container image artifact and inspect the contents statelessly without installing docker tools on my workstation, but i suppose i could make a failing dnm followup and set an autohold for that | 21:45 |
| ianw | from what i can see, the source tree has a html/ directory but it's not installed in any way | 21:45 |
| fungi | yeah, that's what apache would normally serve | 21:46 |
| ianw | so the simplest thing would be to clone that into a /var/www directory on eavesdrop01 | 21:47 |
| fungi | instead of adding it to the dockerfile? | 21:47 |
| ianw | and then bind-mount the directory for the .json file into the ptgbot container, and have apache expose that | 21:47 |
| ianw | well it won't help much in the container unless we also run something in there and reverse proxy to it | 21:48 |
| fungi | oh, right | 21:49 |
| ianw | that said, a bunch of static files could probably be served by python's http.server without issue | 21:49 |
| ianw | (which could be run separately in the container, or i guess as a thread of the current daemon) | 21:50 |
| fungi | or we could put the ptgbot site on static, dump the files there, and then just serve the bot's json file from eavesdrop | 21:51 |
| fungi | or proxy to it like we so with the meetings site | 21:52 |
| fungi | though that gets into a bigger redesign of what's already been written | 21:52 |
| ianw | twisted.web.static might be an option | 21:56 |
| ianw | it seems most sane to try and keep it all in the one container. otherwise it's no so much contained as ... i dunno, the opposite of that :) | 21:58 |
| fungi | leaking | 22:01 |
| fungi | i guess the bot startup could copy the static content files from the container into a mapped directory which apache outside the container would serve, but yeah still feels like a layer violation of some sort | 22:02 |
| ianw | i can try modifying ptgbot to copy the html and modify the docker to run a http.server on a port. that seems the most logical thing. apache can reverse proxy to that, and we can set it to basically infinitely cache | 22:04 |
| fungi | i should be able to help or review shortly, my workstation is tied up building python 3.10 to switch my default python3 over to | 22:11 |
| fungi | still need to rebuild my venvs | 22:11 |
| fungi | the openstackclient venv sure takes a while to recreate :/ | 22:12 |
| ianw | ok, i have it installing and serving, i think this can work | 22:55 |
| *** diablo_rojo is now known as Guest1826 | 23:21 | |
| clarkb | catching up on things and will try t oget the meeting agenda out | 23:34 |
| clarkb | notice that wiki's backups are complaining about some sort of inconsistency | 23:34 |
| clarkb | fungi: ianw ^ any idea if anyone has looked at that yet? I probably won't get to it today | 23:35 |
| ianw | i haven't, will put on todo list | 23:35 |
| ianw | clarkb: one thing for agenda i haven't added is scheduling gerrit 3.3 or maybe 3.4 upgrades | 23:35 |
| clarkb | ianw: ++ I'll get that on there | 23:36 |
| clarkb | I have a bunch of the work for 3.3 prepped. I think 3.4 is a much bigger jump and might need more care | 23:36 |
| ianw | yep, i had a quick look, because a fix to the its error messages we have spewing merged; seems like just moving to the master version of that is the easiest way | 23:36 |
| clarkb | master version of ? | 23:39 |
| ianw | sorry, the its-base plugin | 23:39 |
| clarkb | got it | 23:39 |
| clarkb | ok meeting agenda sent | 23:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!