| *** rfolco has joined #zuul | 00:05 | |
| *** rfolco has quit IRC | 00:05 | |
| *** tosky has quit IRC | 00:25 | |
| mnaser | corvus: awesome changes, happy to see ya slowly helping shape this up :D | 00:26 |
|---|---|---|
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Delete dib images when all uploads set to deleting https://review.opendev.org/702062 | 00:58 |
| clarkb | fungi: ^ I've implemented your idea | 00:58 |
| clarkb | Shrews: ^ you should double check that I'm not missing anything obviosu there, but I believe it is safe to delete these images as soon as the uploads are all in a deleting state | 00:58 |
| fungi | cool! i did not yet get a chance to even look into how feasible it would be | 00:58 |
| clarkb | oh I think there is a small bug in that | 01:00 |
| clarkb | new ps shortly | 01:00 |
| fungi | also today i learned about the all() builtin | 01:01 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Delete dib images when all uploads set to deleting https://review.opendev.org/702062 | 01:04 |
| clarkb | I believe ^ fixes the race ps1 had introduced | 01:04 |
| clarkb | basically we have to delete the build record and the build files from the builder that actually built the image | 01:04 |
| clarkb | otherwise we can get out of sync and delete the build record on a different builder before we ever delete the local files | 01:04 |
| clarkb | but ya I think that should make a good improvement to disk usage as long as we have leaky clouds | 01:05 |
| fungi | i spotted something you can clean up in a conditional i think, see inline comment | 01:07 |
| clarkb | fungi: it also returns true if the iterable is empty so the not uploads check can be omitted, but I find this code reads better | 01:07 |
| fungi | ahh, that's fair | 01:07 |
| clarkb | (shows the relationship between the two if blocks better imo) | 01:08 |
| clarkb | fungi: was that your commetn? it doesn't seem to have posted | 01:08 |
| fungi | yeah, my comment on ps1 raced your ps2 upload | 01:09 |
| fungi | just went back and explicitly posted a review for ps1 so the comment will be included | 01:09 |
| clarkb | that whole function could stand to be refactored a bit too I think | 01:10 |
| fungi | but yes, i don't object to it, just wanted to be clear that all([]) is True regardless | 01:11 |
| clarkb | each of the _cleanup* methods earlier do very similar work just on different conditions | 01:11 |
| *** daniel2 has left #zuul | 01:30 | |
| pabelanger | corvus: thanks! I'd most welcome a new release next week, if we restart opendev that is | 01:33 |
| pabelanger | mordred: thanks! left comment about 80gb volume size | 01:34 |
| *** cloudnull has quit IRC | 01:53 | |
| *** saneax has quit IRC | 02:06 | |
| *** mhu has quit IRC | 03:01 | |
| *** saneax has joined #zuul | 03:25 | |
| *** bhavikdbavishi has joined #zuul | 03:51 | |
| *** bhavikdbavishi1 has joined #zuul | 03:56 | |
| *** bhavikdbavishi has quit IRC | 03:58 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:58 | |
| *** sanjayu_ has joined #zuul | 04:10 | |
| *** mgoddard has quit IRC | 04:12 | |
| *** saneax has quit IRC | 04:13 | |
| *** sanjayu__ has joined #zuul | 04:15 | |
| *** sanjayu_ has quit IRC | 04:18 | |
| *** mgoddard has joined #zuul | 04:19 | |
| *** bhavikdbavishi has quit IRC | 04:34 | |
| *** bhavikdbavishi has joined #zuul | 04:35 | |
| *** sanjayu_ has joined #zuul | 04:45 | |
| *** sanjayu__ has quit IRC | 04:48 | |
| *** bhavikdbavishi has quit IRC | 04:52 | |
| *** sanjayu_ has quit IRC | 05:10 | |
| *** sanjayu_ has joined #zuul | 05:10 | |
| *** sanjayu_ has quit IRC | 05:15 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:34 | |
| *** mgagne has quit IRC | 06:04 | |
| *** bhavikdbavishi has joined #zuul | 06:30 | |
| *** amotoki has quit IRC | 06:32 | |
| *** amotoki has joined #zuul | 06:34 | |
| *** bhavikdbavishi has quit IRC | 06:59 | |
| tobiash | mnaser, corvus: re nodepool readiness: it got a ready endpoint recently: https://review.opendev.org/#/c/695001 | 07:44 |
| tobiash | That starts returning 200 as soon as all drivers are running which inkludes they are registered in zk | 07:46 |
| *** pcaruana has joined #zuul | 08:18 | |
| yoctozepto | 22:57:47 <corvus> yoctozepto: maybe next week? [regarding new zuul in openstack infra] | 08:31 |
| yoctozepto | cool, thanks! :-) | 08:31 |
| openstackgerrit | Merged zuul/zuul-jobs master: helm-template: Add role to run 'helm template' https://review.opendev.org/701871 | 08:45 |
| openstackgerrit | Merged zuul/zuul-jobs master: apply-helm-charts: Job to apply Helm charts https://review.opendev.org/701874 | 08:45 |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix typo in helm role https://review.opendev.org/702046 | 08:45 |
| *** bhavikdbavishi has joined #zuul | 09:25 | |
| *** bhavikdbavishi1 has joined #zuul | 09:28 | |
| *** bhavikdbavishi has quit IRC | 09:30 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 09:30 | |
| *** yoctozepto has quit IRC | 10:18 | |
| *** yoctozepto has joined #zuul | 10:19 | |
| *** armstrongs has joined #zuul | 12:30 | |
| *** armstrongs has quit IRC | 12:40 | |
| *** Goneri has quit IRC | 15:12 | |
| *** tosky has joined #zuul | 15:32 | |
| *** pcaruana has quit IRC | 15:40 | |
| *** tosky has quit IRC | 15:44 | |
| *** Goneri has joined #zuul | 15:54 | |
| *** Goneri has quit IRC | 15:58 | |
| corvus | tobiash: ah, i didn't see that, thanks :) | 16:09 |
| *** Goneri has joined #zuul | 16:28 | |
| *** bhavikdbavishi has quit IRC | 16:41 | |
| corvus | mnaser: we run into this problem the second time we sync the zuul app with argo: https://github.com/argoproj/argo-cd/issues/1250 | 16:49 |
| *** Goneri has quit IRC | 17:22 | |
| mnaser | corvus: ah, i'm probably not running into that because i override `application.instanceLabelKey` | 17:31 |
| *** evrardjp has quit IRC | 17:33 | |
| *** evrardjp has joined #zuul | 17:34 | |
| *** pcaruana has joined #zuul | 18:07 | |
| *** decimuscorvinus has quit IRC | 18:54 | |
| *** decimuscorvinus has joined #zuul | 18:55 | |
| *** pcaruana has quit IRC | 19:04 | |
| *** Goneri has joined #zuul | 19:11 | |
| *** irclogbot_3 has quit IRC | 19:18 | |
| *** decimuscorvinus_ has joined #zuul | 19:20 | |
| *** decimuscorvinus has quit IRC | 19:21 | |
| *** openstackstatus has quit IRC | 19:24 | |
| *** panda has quit IRC | 19:24 | |
| *** panda has joined #zuul | 19:24 | |
| *** irclogbot_1 has joined #zuul | 19:26 | |
| *** openstackstatus has joined #zuul | 19:26 | |
| *** ChanServ sets mode: +v openstackstatus | 19:26 | |
| *** armstrongs has joined #zuul | 19:36 | |
| *** armstrongs has quit IRC | 19:45 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-helm master: Allow override of service type for zuul-web https://review.opendev.org/702101 | 20:11 |
| corvus | remote: https://gerrit-review.googlesource.com/c/zuul/ops/+/250112 Add initial bootstrapping instructions and config | 20:11 |
| corvus | produces https://gerrit-zuul.inaugust.com/tenants | 20:12 |
| corvus | paladox, mordred, mnaser: ^ progress | 20:12 |
| paladox | :O | 20:12 |
| corvus | (there's still a bit more work to do before it's functional) | 20:12 |
| corvus | but there is a service up and running, doing nothing. :) | 20:13 |
| paladox | awesome!!! | 20:13 |
| corvus | i have learned many, many things this morning about setting up letsencrypt with gke/gce | 20:13 |
| paladox | corvus congrats and thank you for woringg on that! :) | 20:13 |
| paladox | heh | 20:13 |
| corvus | no prob, it's fun :) | 20:14 |
| corvus | i'll send a status update to the ml next week, and ask about getting a real dns entry | 20:14 |
| paladox | :) | 20:15 |
| paladox | corvus maybe we could get the zuul-status plugin deployed to gerrit-review too? | 20:15 |
| paladox | it'll give a live view of jobs running for that change. | 20:15 |
| paladox | it supports zuul v2 and v3 (though i've only tested v2) | 20:16 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Update sdk to v0.13.0 and install dhall-to-json https://review.opendev.org/702103 | 20:49 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Import dhall-operator https://review.opendev.org/702104 | 20:49 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Import zuul application from dhall-zuul https://review.opendev.org/702105 | 20:49 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new application https://review.opendev.org/702106 | 20:49 |
| tristanC | corvus: clarkb: i refactor the differents things i used for my operator implementation and make them fit within the zuul/zuul-operator repository ^ | 20:52 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new application https://review.opendev.org/702106 | 20:59 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Update sdk to v0.13.0 and install dhall-to-json https://review.opendev.org/702103 | 21:42 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Update sdk to v0.13.0 and install dhall-to-json https://review.opendev.org/702103 | 21:49 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Update sdk to v0.13.0 and install dhall-to-json https://review.opendev.org/702103 | 21:59 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Import dhall-operator https://review.opendev.org/702104 | 22:09 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Import zuul application from dhall-zuul https://review.opendev.org/702105 | 22:12 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new application https://review.opendev.org/702106 | 22:12 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new application https://review.opendev.org/702106 | 22:21 |
| *** ofosos has joined #zuul | 22:37 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!