| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 00:39 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 00:43 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 00:59 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 01:23 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 02:01 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 02:58 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 03:55 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 04:03 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 04:08 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 05:17 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 06:05 | |
| - [zuul/zuul-operator] 863571: web: Enable custom metadata for Service resources https://review.opendev.org/c/zuul/zuul-operator/+/863571 | ||
| - [zuul/zuul-operator] 861279: bug: Select scheduler pod based on instance name on update https://review.opendev.org/c/zuul/zuul-operator/+/861279 | ||
| - [zuul/zuul-operator] 863572: bug: Properly parameterize zookeeper-client-tls everywhere https://review.opendev.org/c/zuul/zuul-operator/+/863572 | ||
| - [zuul/zuul-operator] 866295: k8s: Remove unused ClusterRole from rbac-admin https://review.opendev.org/c/zuul/zuul-operator/+/866295 | ||
| - [zuul/zuul-operator] 866296: helm: Support clusteradmin role binding https://review.opendev.org/c/zuul/zuul-operator/+/866296 | ||
| - [zuul/zuul-operator] 866297: k8s: Added deploy cluster admin template https://review.opendev.org/c/zuul/zuul-operator/+/866297 | ||
| - [zuul/zuul-operator] 866406: k8s: Inject rbac.yaml into operator.yaml https://review.opendev.org/c/zuul/zuul-operator/+/866406 | ||
| - [zuul/zuul-operator] 866407: k8s: Provide tools and checker for deploy templates https://review.opendev.org/c/zuul/zuul-operator/+/866407 | ||
| - [zuul/zuul-operator] 863439: doc: Rework install doc to cover both template and helm install https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 06:58 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 08:07 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 08:15 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 866863: Store pause and resume events on the build and report them https://review.opendev.org/c/zuul/zuul/+/866863 | 08:49 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 09:09 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 09:24 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 867063: [wip] zuul-jobs-test-registry-buildset-registry-k8s-microk8s https://review.opendev.org/c/zuul/zuul-jobs/+/867063 | 09:35 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/nodepool] 867046: Correct race in static node reregistration https://review.opendev.org/c/zuul/nodepool/+/867046 | 12:53 | |
| -@gerrit:opendev.org- Ade Lee proposed: [zuul/zuul-jobs] 866881: Add ubuntu to enable-fips role https://review.opendev.org/c/zuul/zuul-jobs/+/866881 | 14:17 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/nodepool] 862630: Add Python 3.11 testing https://review.opendev.org/c/zuul/nodepool/+/862630 | 17:15 | |
| @clarkb:matrix.org | I've been doing some general container stuff for opendev and as part of that looked at zuul image sizing to see if there were any clear wins. There aren't off the top of my head, but there were some interesting things I learned. We include the zuul web static content which is about 50MB in all of the zuul images due to how zuul gets installed. I don't think this would be easy to remove, but is a possible win? pydantic is not small about 50MB as well, I have no idea what depends on it. libicudata is also not small. Looks like lxml type thing smay depend on it though | 18:47 |
|---|---|---|
| @clarkb:matrix.org | I also confirmed that the update to how the base image deals with wheels didn't somehow expand the size of things. The zuul images rm that content in the same step as installing it so we don't care it around longer than necessary | 18:48 |
| @michael_kelly_anet:matrix.org | How large are the Zuul images? I've noticed that they can be quite slow to pull | 18:57 |
| @clarkb:matrix.org | the executor images are largest because they have the ansible installs at about 1.4GB | 18:57 |
| @michael_kelly_anet:matrix.org | Oof. | 18:57 |
| @michael_kelly_anet:matrix.org | That's hefty. | 18:57 |
| @clarkb:matrix.org | the other images are 660MB or so | 18:58 |
| @clarkb:matrix.org | looks like kubectl and oc might be the same file at 115MB so deduping that with a symlink might get us a bit of savings | 19:00 |
| @clarkb:matrix.org | could also hardlink maybe? notsure how portable that is if someone is running this on windows or something | 19:01 |
| @clarkb:matrix.org | one ansible 6 installation is ~640MB | 19:01 |
| @clarkb:matrix.org | together the oc/kubectl binaries and ansible install account for the delta between the other images and the executor | 19:02 |
| @michael_kelly_anet:matrix.org | Not sure how docker is on Windows, but on MacOS it's actually running on top of a shim VM so there's nothing MacOS about it | 19:02 |
| @clarkb:matrix.org | the fortinet ansible collection dir is 114MB | 19:03 |
| @clarkb:matrix.org | hrm for some reason we're ending up with .pyc files in there | 19:04 |
| @clarkb:matrix.org | I wonder if we need to run the ansible install without pyc generation | 19:04 |
| -@gerrit:opendev.org- Clark Boylan proposed: | 19:24 | |
| - [zuul/zuul] 867136: Dedup the oc and kubectl commands in the docker images https://review.opendev.org/c/zuul/zuul/+/867136 | ||
| - [zuul/zuul] 867137: Update openshift client install version https://review.opendev.org/c/zuul/zuul/+/867137 | ||
| @clarkb:matrix.org | someone that uses kubernetes and/or openshift may want to double check those. But I think that will save 115MB of image space | 19:24 |
| @jim:acmegating.com | Clark: i think a noop depends-on change to zuul-operator would exercise it | 19:35 |
| @clarkb:matrix.org | I'll get one up for that | 19:36 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-operator] 867139: DNM trying to test changes to oc/kubectl in the executor image https://review.opendev.org/c/zuul/zuul-operator/+/867139 | 19:41 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Dr. Jens Harbott: [zuul/nodepool] 863812: Add username to detailed node list output https://review.opendev.org/c/zuul/nodepool/+/863812 | 20:26 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 865923: Avoid acquiring pipeline locks in manager postConfig https://review.opendev.org/c/zuul/zuul/+/865923 | 21:09 | |
| @clarkb:matrix.org | The fortinet collection is part of the default ansible packaging. Looking in both the sdist and wheel for ansible there are no pyc files for fortinet | 21:47 |
| @clarkb:matrix.org | something is generating them either as part of the install or after install I guess. /me tries installing to check | 21:48 |
| @clarkb:matrix.org | just doing a pip install dumps them into the site-packages | 21:49 |
| @clarkb:matrix.org | doesn't look like pip honors PYTHONDONTWRITEBYTECODE | 21:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!