| *** amoralej|off is now known as amoralej | 06:16 | |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: [DNM] test fedora mirrors https://review.opendev.org/c/openstack/diskimage-builder/+/884581 | 10:03 |
|---|---|---|
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: fedora: don't use CI mirrors https://review.opendev.org/c/openstack/diskimage-builder/+/883798 | 10:03 |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: fedora: don't use CI mirrors https://review.opendev.org/c/openstack/diskimage-builder/+/883798 | 10:53 |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: Extend the checksum files generation procedure https://review.opendev.org/c/openstack/diskimage-builder/+/881298 | 11:00 |
| *** amoralej is now known as amoralej|lunch | 12:19 | |
| fungi | just a reminder that today is a national holiday in the usa, so i don't expect to be around much but hopefully that also makes for a quiet day with so many other contributors also doing the same | 12:44 |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: fedora: don't use CI mirrors https://review.opendev.org/c/openstack/diskimage-builder/+/883798 | 12:56 |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: Extend the checksum files generation procedure https://review.opendev.org/c/openstack/diskimage-builder/+/881298 | 12:58 |
| *** amoralej|lunch is now known as amoralej | 13:13 | |
| *** dhill is now known as Guest1665 | 14:00 | |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: fedora: don't use CI mirrors https://review.opendev.org/c/openstack/diskimage-builder/+/883798 | 14:40 |
| opendevreview | Merged opendev/system-config master: Clean up SB accounts for former admins https://review.opendev.org/c/opendev/system-config/+/489711 | 15:14 |
| *** dasm is now known as Guest1685 | 15:22 | |
| *** amoralej is now known as amoralej|off | 16:07 | |
| *** dasm_ is now known as dasm | 16:23 | |
| *** dasm is now known as Guest1690 | 16:24 | |
| NeilHanlon | also on holiday but had a thought - has anyone ever suggested adding a link in gerrit on the Change view to the related zuul 'run'? i.e., adding a link next to the `gitea` link which points to zuul.opendev.org/t/PROJECT/status/change/CHANGE,PATCHSET ? | 16:58 |
| clarkb | NeilHanlon: a long long time ago we had embedded zuul status but it made the zuul api unhappy. Just having a link back would probably be fine and a good idea | 17:09 |
| clarkb | I think the current zuul summary plugin could potentially be updated to include that | 17:10 |
| NeilHanlon | 👍that sounds like a good place for it | 17:10 |
| * NeilHanlon scribbles down some notes | 17:10 | |
| NeilHanlon | i've not worked with gerrit before but this seems like a good first project heh | 17:11 |
| clarkb | ya the problem was all of us end up with tons of gerrit change tabs open and even if you only query the status api for the currently open tab there may be 100 people doing that at once and it required loading the whole zuul status json blob at the time (I don't think it does anymore) definitely something that we had ideas around but ran out of steam when it ran into performance | 17:12 |
| clarkb | issues. But that was long enough ago that revisinting would be a good idea | 17:12 |
| NeilHanlon | oh, yeah I can see that being a problem. perhaps also a change on the zuul side to detect if the tab is 'active' and pause the websocket streaming | 17:18 |
| NeilHanlon | i misread your message, too, you were just talking about gerrit talking to zuul, not the zuul change page itself | 17:19 |
| clarkb | ya it would show you the running progress of your jobs in gerrit. that was the theory anyway and it worked if only a small number of people had changes open | 17:19 |
| clarkb | the problems were almost all on the zuul side and I think a good chunk of them have been resolved | 17:20 |
| *** Guest1690 is now known as dasm | 18:01 | |
| fungi | the problem era was a | 18:45 |
| fungi | lso before the zuul api added change scoping for status data | 18:45 |
| fungi | and also before zuul-web was split into a separate service, so overloading the status api was impacting the scheduler process itself | 18:46 |
| fungi | there are a lot of reasons why it would probably be perfectly fine now if sensibly designed | 18:46 |
| opendevreview | Maksim Malchuk proposed zuul/zuul-jobs master: [DNM] test zuul/nodepool-builder:siblings not found https://review.opendev.org/c/zuul/zuul-jobs/+/884642 | 19:59 |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: fedora: don't use CI mirrors https://review.opendev.org/c/openstack/diskimage-builder/+/883798 | 19:59 |
| opendevreview | Maksim Malchuk proposed zuul/zuul-jobs master: [DNM] test zuul/nodepool-builder:siblings not found https://review.opendev.org/c/zuul/zuul-jobs/+/884642 | 20:52 |
| opendevreview | Steve Baker proposed openstack/diskimage-builder master: Change DIB_BOOTLOADER_DEFAULT_CMDLINE empty string behaviour https://review.opendev.org/c/openstack/diskimage-builder/+/884644 | 21:32 |
| opendevreview | Maksim Malchuk proposed openstack/diskimage-builder master: fedora: don't use CI mirrors https://review.opendev.org/c/openstack/diskimage-builder/+/883798 | 21:33 |
| opendevreview | Steve Baker proposed openstack/diskimage-builder master: Change DIB_BOOTLOADER_DEFAULT_CMDLINE empty string behaviour https://review.opendev.org/c/openstack/diskimage-builder/+/884644 | 22:01 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!