| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 863570: [wip] use dummy package for ensure-pip https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | 00:32 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 00:36 | |
| - [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] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [zuul/zuul-operator] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| - [zuul/zuul-operator] 863571: web: Enable custom metadata for Service resources https://review.opendev.org/c/zuul/zuul-operator/+/863571 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 863570: [wip] use dummy package for ensure-pip https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | 00:48 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 01:10 | |
| - [zuul/zuul-operator] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| - [zuul/zuul-operator] 863572: bug: Properly parameterize zookeeper-client-tls everywhere https://review.opendev.org/c/zuul/zuul-operator/+/863572 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 01:14 | |
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [zuul/zuul-operator] 863571: web: Enable custom metadata for Service resources https://review.opendev.org/c/zuul/zuul-operator/+/863571 | ||
| - [zuul/zuul-operator] 863572: bug: Properly parameterize zookeeper-client-tls everywhere https://review.opendev.org/c/zuul/zuul-operator/+/863572 | ||
| - [zuul/zuul-operator] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 863570: [wip] use dummy package for ensure-pip https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | 01:50 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 863570: [wip] use dummy package for ensure-pip https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | 01:51 | |
| -@gerrit:opendev.org- Ian Wienand proposed: | 02:07 | |
| - [zuul/zuul-jobs] 863570: ensure-pip: use dummy package for testing https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | ||
| - [zuul/zuul-jobs] 854933: linter: Use capitals for names https://review.opendev.org/c/zuul/zuul-jobs/+/854933 | ||
| - [zuul/zuul-jobs] 861559: Fix ansible-lint name[template] https://review.opendev.org/c/zuul/zuul-jobs/+/861559 | ||
| - [zuul/zuul-jobs] 861560: Add names to include tasks https://review.opendev.org/c/zuul/zuul-jobs/+/861560 | ||
| - [zuul/zuul-jobs] 861562: Standarise block/when ordering https://review.opendev.org/c/zuul/zuul-jobs/+/861562 | ||
| - [zuul/zuul-jobs] 861563: Update to ansible-lint 6.8.2 https://review.opendev.org/c/zuul/zuul-jobs/+/861563 | ||
| - [zuul/zuul-jobs] 861588: Revert sphinx pin https://review.opendev.org/c/zuul/zuul-jobs/+/861588 | ||
| @michael_kelly_anet:matrix.org | corvus: Easier to converse in this fashion then via ping-ponging messages in reviews regarding https://review.opendev.org/c/zuul/zuul-operator/+/861488 | 02:56 |
|---|---|---|
| You can't really differentiate between PXC (or cert-manager) installed independently vs PXC being installed by zuul-operator: CRDs installed by PXC are a global resource which may change between versions. Making zuul-operator manage PXC upgrades means you're basically turning zuul-operator into RedHat Operator Lifecycle Manager, which is probably not desirable for most use cases. | ||
| @michael_kelly_anet:matrix.org | zuul-operator is also not *uninstalling* PXC or cert-manager when they're no longer in use, which leaves someone who just wants to use the operator in a relatively canned format with having to hunt down and scrub a bunch of extra state that's not entirely obvious. ie: you have to go back and understand everything that it has installed by scraping through the operator source code, which is not exactly user friendly. | 03:00 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 863575: [wip] skopeo/podman jammy roles https://review.opendev.org/c/zuul/zuul-jobs/+/863575 | 03:03 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 03:11 | |
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 03:18 | |
| - [zuul/zuul-operator] 863191: helm: Add pxc-operator as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/863191 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 03:26 | |
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 03:29 | |
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: | 03:31 | |
| - [zuul/zuul-jobs] 863575: ensure-podman/skopeo : update for Jammy https://review.opendev.org/c/zuul/zuul-jobs/+/863575 | ||
| - [zuul/zuul-jobs] 863570: ensure-pip: use dummy package for testing https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | ||
| - [zuul/zuul-jobs] 854933: linter: Use capitals for names https://review.opendev.org/c/zuul/zuul-jobs/+/854933 | ||
| - [zuul/zuul-jobs] 861559: Fix ansible-lint name[template] https://review.opendev.org/c/zuul/zuul-jobs/+/861559 | ||
| - [zuul/zuul-jobs] 861560: Add names to include tasks https://review.opendev.org/c/zuul/zuul-jobs/+/861560 | ||
| - [zuul/zuul-jobs] 861562: Standarise block/when ordering https://review.opendev.org/c/zuul/zuul-jobs/+/861562 | ||
| - [zuul/zuul-jobs] 861563: Update to ansible-lint 6.8.2 https://review.opendev.org/c/zuul/zuul-jobs/+/861563 | ||
| - [zuul/zuul-jobs] 863578: zuul-tests: autogenerate some more roles https://review.opendev.org/c/zuul/zuul-jobs/+/863578 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 03:48 | |
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| - [zuul/zuul-operator] 863579: test: Convert functional test to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 04:16 | |
| - [zuul/zuul-operator] 863579: test: Convert functional test to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: | 04:16 | |
| - [zuul/zuul-jobs] 863575: ensure-podman/skopeo : update for Jammy https://review.opendev.org/c/zuul/zuul-jobs/+/863575 | ||
| - [zuul/zuul-jobs] 863570: ensure-pip: use dummy package for testing https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | ||
| - [zuul/zuul-jobs] 854933: linter: Use capitals for names https://review.opendev.org/c/zuul/zuul-jobs/+/854933 | ||
| - [zuul/zuul-jobs] 861559: Fix ansible-lint name[template] https://review.opendev.org/c/zuul/zuul-jobs/+/861559 | ||
| - [zuul/zuul-jobs] 861560: Add names to include tasks https://review.opendev.org/c/zuul/zuul-jobs/+/861560 | ||
| - [zuul/zuul-jobs] 861562: Standarise block/when ordering https://review.opendev.org/c/zuul/zuul-jobs/+/861562 | ||
| - [zuul/zuul-jobs] 861563: Update to ansible-lint 6.8.2 https://review.opendev.org/c/zuul/zuul-jobs/+/861563 | ||
| - [zuul/zuul-jobs] 863578: zuul-tests: autogenerate some more roles https://review.opendev.org/c/zuul/zuul-jobs/+/863578 | ||
| - [zuul/zuul-jobs] 863582: Make test-registry-buildset-registery-k8s-<docker|crio> non-voting https://review.opendev.org/c/zuul/zuul-jobs/+/863582 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 04:40 | |
| - [zuul/zuul-operator] 863579: test: Convert functional test to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: | 05:23 | |
| - [zuul/zuul-jobs] 863582: Make test-registry-buildset-registery-k8s-<docker|crio> non-voting https://review.opendev.org/c/zuul/zuul-jobs/+/863582 | ||
| - [zuul/zuul-jobs] 863575: ensure-podman/skopeo : update for Jammy https://review.opendev.org/c/zuul/zuul-jobs/+/863575 | ||
| - [zuul/zuul-jobs] 863570: ensure-pip: use dummy package for testing https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | ||
| - [zuul/zuul-jobs] 854933: linter: Use capitals for names https://review.opendev.org/c/zuul/zuul-jobs/+/854933 | ||
| - [zuul/zuul-jobs] 861559: Fix ansible-lint name[template] https://review.opendev.org/c/zuul/zuul-jobs/+/861559 | ||
| - [zuul/zuul-jobs] 861560: Add names to include tasks https://review.opendev.org/c/zuul/zuul-jobs/+/861560 | ||
| - [zuul/zuul-jobs] 861562: Standarise block/when ordering https://review.opendev.org/c/zuul/zuul-jobs/+/861562 | ||
| - [zuul/zuul-jobs] 861563: Update to ansible-lint 6.8.2 https://review.opendev.org/c/zuul/zuul-jobs/+/861563 | ||
| - [zuul/zuul-jobs] 863578: zuul-tests: autogenerate some more roles https://review.opendev.org/c/zuul/zuul-jobs/+/863578 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 05:29 | |
| - [zuul/zuul-operator] 861488: helm: Add a basic helm chart for zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/861488 | ||
| - [zuul/zuul-operator] 862390: helm: Add cert-manager as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/862390 | ||
| - [zuul/zuul-operator] 863191: helm: Add pxc-operator as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/863191 | ||
| - [zuul/zuul-operator] 863579: test: Convert functional test to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| - [zuul/zuul-operator] 863586: helm: Remove unnecessary CRD access from clusterrole https://review.opendev.org/c/zuul/zuul-operator/+/863586 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 06:34 | |
| - [zuul/zuul-operator] 861488: helm: Add a basic helm chart for zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/861488 | ||
| - [zuul/zuul-operator] 862390: helm: Add cert-manager as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/862390 | ||
| - [zuul/zuul-operator] 863191: helm: Add pxc-operator as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/863191 | ||
| - [zuul/zuul-operator] 863579: test: Convert functional test to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863586: helm: Remove unnecessary CRD access from clusterrole https://review.opendev.org/c/zuul/zuul-operator/+/863586 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| -@gerrit:opendev.org- Ian Wienand proposed: | 06:53 | |
| - [zuul/zuul-jobs] 863582: Make test-registry-buildset-registery-k8s-<docker|crio> non-voting https://review.opendev.org/c/zuul/zuul-jobs/+/863582 | ||
| - [zuul/zuul-jobs] 863575: ensure-podman/skopeo : update for Jammy https://review.opendev.org/c/zuul/zuul-jobs/+/863575 | ||
| - [zuul/zuul-jobs] 863570: ensure-pip: use dummy package for testing https://review.opendev.org/c/zuul/zuul-jobs/+/863570 | ||
| - [zuul/zuul-jobs] 854933: linter: Use capitals for names https://review.opendev.org/c/zuul/zuul-jobs/+/854933 | ||
| - [zuul/zuul-jobs] 861559: Fix ansible-lint name[template] https://review.opendev.org/c/zuul/zuul-jobs/+/861559 | ||
| - [zuul/zuul-jobs] 861560: Add names to include tasks https://review.opendev.org/c/zuul/zuul-jobs/+/861560 | ||
| - [zuul/zuul-jobs] 861562: Standarise block/when ordering https://review.opendev.org/c/zuul/zuul-jobs/+/861562 | ||
| - [zuul/zuul-jobs] 861563: Update to ansible-lint 6.8.2 https://review.opendev.org/c/zuul/zuul-jobs/+/861563 | ||
| - [zuul/zuul-jobs] 863578: zuul-tests: autogenerate some more roles https://review.opendev.org/c/zuul/zuul-jobs/+/863578 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 07:52 | |
| - [zuul/zuul-operator] 863579: test: Convert functional test to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863586: helm: Remove unnecessary CRD access from clusterrole https://review.opendev.org/c/zuul/zuul-operator/+/863586 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| - [zuul/zuul-operator] 863594: helm: Add pxc-operator as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/863594 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 862978: Add playbook semaphores https://review.opendev.org/c/zuul/zuul/+/862978 | 13:48 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 16:31 | |
| - [zuul/zuul-operator] 861488: helm: Add a basic helm chart for zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/861488 | ||
| - [zuul/zuul-operator] 862390: helm: Add cert-manager as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/862390 | ||
| - [zuul/zuul-operator] 863594: helm: Add pxc-operator as optional dependency https://review.opendev.org/c/zuul/zuul-operator/+/863594 | ||
| - [zuul/zuul-operator] 863579: test: Convert functional test to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: pxc: Don't install PXC operator from Zuul operator https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 863475: cert-manager: Don't install cert-manager via zuul-operator https://review.opendev.org/c/zuul/zuul-operator/+/863475 | ||
| - [zuul/zuul-operator] 863586: helm: Remove unnecessary CRD access from clusterrole https://review.opendev.org/c/zuul/zuul-operator/+/863586 | ||
| - [zuul/zuul-operator] 863476: k8s: Enable administrator to limit the watched namespace scope https://review.opendev.org/c/zuul/zuul-operator/+/863476 | ||
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| @michael_kelly_anet:matrix.org | Can I get a second review and maybe a Workflow bump for https://review.opendev.org/c/zuul/zuul-operator/+/853592/15 ? | 17:01 |
| @noonedeadpunk:matrix.org | Hey folks. I have weird thing here, also regarding job vars/extra-vars. | 17:22 |
| @noonedeadpunk:matrix.org | so assume we have these jobs: https://paste.openstack.org/show/bFna1vX86EaLcvD3SpZp/ | 17:23 |
| @noonedeadpunk:matrix.org | but for both of them `tested_service_groups` will be `cinder_api` | 17:23 |
| @noonedeadpunk:matrix.org | Since there's statement that "all vars are host vars at the end" I was quite surprised some time before, I guess according to prescedance https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html#ansible-variable-precedence | 17:25 |
| @noonedeadpunk:matrix.org | play_vars indeed have more priority then host_vars | 17:25 |
| @noonedeadpunk:matrix.org | But it's soooooo confusing that you have no option to override that in zuul.... | 17:26 |
| @clarkb:matrix.org | can't you use a default for that? | 17:28 |
| @clarkb:matrix.org | I suspect you can | 17:28 |
| @clarkb:matrix.org | but I've not tested it | 17:28 |
| @noonedeadpunk:matrix.org | What you mean under default? Like role default? | 17:28 |
| @clarkb:matrix.org | I don't think that is confusing if ansible says your playbook var takes precedence | 17:28 |
| @clarkb:matrix.org | zuul is just running ansible | 17:28 |
| @noonedeadpunk:matrix.org | Yes, but zuul says that extra-vars have prescedence over them | 17:29 |
| @noonedeadpunk:matrix.org | And ansible also uses extra-vars terminology as smth with top prescedence | 17:29 |
| @noonedeadpunk:matrix.org | And extra-vars worked as extra vars <5.0 or smth.... | 17:29 |
| @clarkb:matrix.org | yes, there was a security bug that forced things to change. Unfortunate, but we weren't left with much choice and approximate it now by resolving the precedence in zuul itself | 17:30 |
| @clarkb:matrix.org | but once ansible is running zuul can't do that anymore os the ansible precedence takes over | 17:30 |
| @clarkb:matrix.org | as far as dfeault I mean set the playbook var with a default, then if the job doesn't set it then playbook var default will be used. If they job does set it then you get the job version | 17:30 |
| @noonedeadpunk:matrix.org | Yeah, I mean, I can move all to zuul jobs in this case, but I might not in other cases when content is being fetched from galaxy or other places... | 17:31 |
| @clarkb:matrix.org | another option would be to always set it at the job level and use the job var and not a playbook var | 17:31 |
| @noonedeadpunk:matrix.org | oh, ok | 17:31 |
| @noonedeadpunk:matrix.org | yeah, might be default filter is good enough here | 17:31 |
| @clarkb:matrix.org | * as far as dfeault I mean set the playbook var with a default, then if the job doesn't set it then playbook var default will be used. If the job does set it then you get the job version | 17:31 |
| @clarkb:matrix.org | note I landed a change to clarify the extra vars zuul behavior after your last bit of feedback | 17:34 |
| @noonedeadpunk:matrix.org | oh | 17:34 |
| @clarkb:matrix.org | and it doesn't say that it has higher precedence than playbook vars | 17:34 |
| @noonedeadpunk:matrix.org | is it merged ?:D | 17:34 |
| @clarkb:matrix.org | only job host and group. Yes it merged | 17:34 |
| @noonedeadpunk:matrix.org | Ah yes | 17:35 |
| @noonedeadpunk:matrix.org | I was looking at older docs eventually | 17:35 |
| @noonedeadpunk:matrix.org | Thanks Clark and sorry - I just expect things from ansible and when digging after couple of hours against the wall I'm checking source/prescedence thing again become a bit unfair | 17:36 |
| @noonedeadpunk:matrix.org | (and opionated) | 17:37 |
| -@gerrit:opendev.org- Michael Kelly proposed: | 17:53 | |
| - [zuul/zuul-operator] 863477: k8s: Allow use of a default image version besides latest https://review.opendev.org/c/zuul/zuul-operator/+/863477 | ||
| - [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] 863439: doc: Re-write install doc to use helm chart https://review.opendev.org/c/zuul/zuul-operator/+/863439 | ||
| @y2kenny:matrix.org | Are there any documentation on how autohold works? I was searching the doc and I think the closest I can find is autohold for zuul client, max_hold_expiration and default_hold_expiration. Is there auto hold setting on a per job basis? | 19:40 |
| @clarkb:matrix.org | Kenny Ho: the zuul client autohold command should document all the valid filter criteria. That does include job and ref and so on | 19:44 |
| @clarkb:matrix.org | corvus: question on https://review.opendev.org/c/zuul/zuul/+/863104 but feel free to approve that change if that isn't practical or if you want to do it in a followup | 19:46 |
| @clarkb:matrix.org | and a question on the child of that change too | 19:50 |
| @clarkb:matrix.org | corvus: and a few questions on the last change in that stack | 21:40 |
| @jim:acmegating.com | Clark: replied, thanks :) | 23:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!