| @clarkb:matrix.org | josefwells: it might help to paste the actual tracebacks for more context | 00:04 |
|---|---|---|
| @josefwells:matrix.org | yeah, sorry, I'm trying to isolate better... I'm not sure about this database stuff getting created, or how to trigger it to get created. | 00:08 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 00:31 | |
| - [zuul/zuul] 817626: Remove rpcclient from autohold tests https://review.opendev.org/c/zuul/zuul/+/817626 | ||
| - [zuul/zuul] 819699: Remove rpcclient tests https://review.opendev.org/c/zuul/zuul/+/819699 | ||
| @clarkb:matrix.org | corvus: ^ is that the rebase necessary to deal with the merge conflict? | 00:37 |
| @clarkb:matrix.org | hrm no change numbers are different | 00:37 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 00:38 | |
| - [zuul/zuul] 819700: Remove rpcclient use from github tests https://review.opendev.org/c/zuul/zuul/+/819700 | ||
| - [zuul/zuul] 819701: Remove rpcclient from gitlab tests https://review.opendev.org/c/zuul/zuul/+/819701 | ||
| @clarkb:matrix.org | oh derp those are the chaneg merging. They'll merge up to the conflict | 00:41 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 819702: Remove rpcclient from pagure tests https://review.opendev.org/c/zuul/zuul/+/819702 | 02:10 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 819703: Remove rpcclient from test_reporting https://review.opendev.org/c/zuul/zuul/+/819703 | 02:11 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 02:24 | |
| - [zuul/zuul] 819704: Remove rpcclient from test_v3 https://review.opendev.org/c/zuul/zuul/+/819704 | ||
| - [zuul/zuul] 819705: Remove rpcclient from test_web https://review.opendev.org/c/zuul/zuul/+/819705 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 02:24 | |
| - [zuul/zuul] 819706: Remove rpcclient from test_zuulclient https://review.opendev.org/c/zuul/zuul/+/819706 | ||
| - [zuul/zuul] 819707: Remove rpcclient from test_circular_dependencies https://review.opendev.org/c/zuul/zuul/+/819707 | ||
| - [zuul/zuul] 819708: Remove rpcclient from connection test fixtures https://review.opendev.org/c/zuul/zuul/+/819708 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 14:45 | |
| - [zuul/zuul] 819709: Remove the rpc client and listener https://review.opendev.org/c/zuul/zuul/+/819709 | ||
| - [zuul/zuul] 819710: Remove disused gearman_server usages from tests https://review.opendev.org/c/zuul/zuul/+/819710 | ||
| - [zuul/zuul] 819711: Remove gearman server https://review.opendev.org/c/zuul/zuul/+/819711 | ||
| - [zuul/zuul] 819712: Remove gearman from docs https://review.opendev.org/c/zuul/zuul/+/819712 | ||
| - [zuul/zuul] 825014: Remove "sql connection" backwards compatability for database https://review.opendev.org/c/zuul/zuul/+/825014 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 14:53 | |
| - [zuul/zuul] 823587: Add some ZK debug scripts https://review.opendev.org/c/zuul/zuul/+/823587 | ||
| - [zuul/zuul] 824077: Add a zk-shell debug script https://review.opendev.org/c/zuul/zuul/+/824077 | ||
| @jim:acmegating.com | Clark: there's the rebase ^ | 14:53 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 824482: Add "zuul delete-pipeline-state" command https://review.opendev.org/c/zuul/zuul/+/824482 | 14:54 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 826158: Add upload-logs-ibm role https://review.opendev.org/c/zuul/zuul-jobs/+/826158 | 15:03 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 15:36 | |
| - [zuul/zuul-operator] 812234: Add support for imagePullSecrets https://review.opendev.org/c/zuul/zuul-operator/+/812234 | ||
| - [zuul/zuul-operator] 812240: Fix zuulRegistryImageVersion https://review.opendev.org/c/zuul/zuul-operator/+/812240 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 825685: Revert "Don't run non-docs jobs on docs changes" https://review.opendev.org/c/zuul/zuul/+/825685 | 16:11 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 817640: Add some pipeline processing stats https://review.opendev.org/c/zuul/zuul/+/817640 | 16:17 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 807464: Add metastatic driver https://review.opendev.org/c/zuul/nodepool/+/807464 | 16:33 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 825014: Remove "sql connection" backwards compatability for database https://review.opendev.org/c/zuul/zuul/+/825014 | 16:52 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 823587: Add some ZK debug scripts https://review.opendev.org/c/zuul/zuul/+/823587 | 16:52 | |
| @jim:acmegating.com | zuul-maint: i'm working with OIF folks on press release, etc for zuul v5; we're looking to make the release and announcement feb 1 (one week). | 17:00 |
| @spamaps:spamaps.ems.host | This is kind of weird re-familiarizing myself with diskimage-builder .. a thing I kind of sort of remember being a major author of at one point. ;) | 17:02 |
| @jim:acmegating.com | spamaps: you may find https://review.opendev.org/803123 interesting | 17:07 |
| @spamaps:spamaps.ems.host | That is somewhat interesting. I'd quite like to focus my efforts on Dockerfile rather than elements.. but.. the thing I'm building is essentially docker + pull some images into that docker.. and docker-in-docker can be.. dicey. ;) | 17:08 |
| @spamaps:spamaps.ems.host | And I'm remembering that one of my favorite things about DIB is the aggressive caching. Iterating is relatively fast. | 17:11 |
| @spamaps:spamaps.ems.host | Before I get too far down this road.. has anyone made an element that basically pre-populates a dockerd with images? | 17:22 |
| @spamaps:spamaps.ems.host | I seem to remember wanting exactly that some years ago. | 17:23 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 824077: Add a zk-shell debug script https://review.opendev.org/c/zuul/zuul/+/824077 | 17:28 | |
| @clarkb:matrix.org | > <@spamaps:spamaps.ems.host> Before I get too far down this road.. has anyone made an element that basically pre-populates a dockerd with images? | 17:29 |
| Not that I know of | ||
| @spamaps:spamaps.ems.host | It's a thorny challenge because docker won't run in a chroot. | 17:31 |
| @spamaps:spamaps.ems.host | I recall when I did it before I had to essentially stop dockerd on the host, bind mount in the empty dir from the dib build.. start a new dockerd.. pull.. then save that empty dir in as /var/lib/docker .. for it to work. | 17:32 |
| @spamaps:spamaps.ems.host | Part of me is like "I wonder if we can just run tese with podman" | 17:32 |
| @spamaps:spamaps.ems.host | * Part of me is like "I wonder if we can just run these with podman" | 17:33 |
| @jim:acmegating.com | ++ or if dockerd necessary, consider skopeo at boot | 17:33 |
| @spamaps:spamaps.ems.host | Oh that's new. | 17:34 |
| @spamaps:spamaps.ems.host | well that's just perfect I'm going to try skopeo since it has "docker daemon storage" | 17:35 |
| @clarkb:matrix.org | oh ya skopeo has been useful for working around dockerd weirdness | 17:37 |
| @clarkb:matrix.org | Leaving this here as I don't quite have time to look further right now. But the --change flag to zuul-client autohold doesn't seem to set the ref filter on the hold according to the autohold listing | 17:42 |
| @clarkb:matrix.org | it is set to .* when I say --change 123456 | 17:42 |
| @spamaps:spamaps.ems.host | 🤦♂️can't get skopeo to run on our Ubuntu 18.04 VMs. | 18:06 |
| @spamaps:spamaps.ems.host | not even inside docker :( | 18:06 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 826357: Add QuotaSupport to drivers that don't have it https://review.opendev.org/c/zuul/nodepool/+/826357 | 18:54 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Benjamin Schanzel: [zuul/nodepool] 825129: Fix tenant-quota for providers without label quota infos https://review.opendev.org/c/zuul/nodepool/+/825129 | 18:54 | |
| @tobias.henkel:matrix.org | Spamaps: we have an element that pre populares dockerd, I think I even posted it once as a poc as a change. I can look it up later if you want | 18:58 |
| @tobias.henkel:matrix.org | * Spamaps: we have an element that pre populates dockerd, I think I even posted it once as a poc as a change. I can look it up later if you want | 18:58 |
| @spamaps:spamaps.ems.host | > <@tobias.henkel:matrix.org> Spamaps: we have an element that pre populates dockerd, I think I even posted it once as a poc as a change. I can look it up later if you want | 19:05 |
| yes please | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 825014: Remove "sql connection" backwards compatability for database https://review.opendev.org/c/zuul/zuul/+/825014 | 19:25 | |
| @tobias.henkel:matrix.org | spamaps: https://review.opendev.org/c/openstack/diskimage-builder/+/767706 | 19:34 |
| @tobias.henkel:matrix.org | spamaps: that even works if dib is running inside docker (our nodepool-builders are running inside k8s) | 19:35 |
| @tobias.henkel:matrix.org | spamaps: it essentially runs dockerd inside bwrap and pulls the images then | 19:36 |
| @spamaps:spamaps.ems.host | Oh I'm def trying that out. :) | 19:38 |
| @hanson76:matrix.org | Small request, could this https://review.opendev.org/c/zuul/nodepool/+/824645 change be reviewed and merge soon? | 20:58 |
| We are hit by this problem quite a lot, more or less every third or forth EC2 instance that is created "fail" because of it. | ||
| I've no way to test the change before it's merged, but can provide quick feedback on it when it before next release of Nodepool. | ||
| @clarkb:matrix.org | corvus: I think you can go ahead and approve https://review.opendev.org/c/zuul/zuul/+/819709/ if you like. The delta according to gerrit is just the commit rebase? not substantive changes? Also https://review.opendev.org/c/zuul/zuul/+/819712/ and https://review.opendev.org/c/zuul/zuul/+/825014/ have comments worth addressing | 23:56 |
| @clarkb:matrix.org | I'm happy to try and fix those two changes and rebase the switch to focal test nodes at the same time if you like | 23:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!