| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 847387: Revert "Pin minikube to 1.25.2" https://review.opendev.org/c/zuul/nodepool/+/847387 | 00:05 | |
| @michael_kelly_anet:matrix.org | corvus: A few more comments added to https://review.opendev.org/c/zuul/zuul-operator/+/853695 | 06:36 |
|---|---|---|
| @michael_kelly_anet:matrix.org | In particular, it seems like maybe just calling this change a breaking change might be the sanest thing to do. Otherwise it needs to do some sort of data migration for ZK and MySQL | 06:37 |
| -@gerrit:opendev.org- Michael Kelly proposed: | 07:39 | |
| - [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] 865945: test: Restructure zuul-operator-functional-k8s layout https://review.opendev.org/c/zuul/zuul-operator/+/865945 | ||
| - [zuul/zuul-operator] 863579: test: Introduce Helm-based functional test https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: k8s: Provide an option to disable PXC operator installer https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 866231: k8s: Clean up cert-manager installer https://review.opendev.org/c/zuul/zuul-operator/+/866231 | ||
| - [zuul/zuul-operator] 863475: k8s: Provide an option to disable cert-manager installation 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] 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 | ||
| - [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 08:49 | |
| - [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | ||
| - [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] 865945: test: Restructure zuul-operator-functional-k8s layout https://review.opendev.org/c/zuul/zuul-operator/+/865945 | ||
| - [zuul/zuul-operator] 863579: test: Introduce Helm-based functional test https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: k8s: Provide an option to disable PXC operator installer https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 866231: k8s: Clean up cert-manager installer https://review.opendev.org/c/zuul/zuul-operator/+/866231 | ||
| - [zuul/zuul-operator] 863475: k8s: Provide an option to disable cert-manager installation 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] 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- Michael Kelly proposed: | 08:56 | |
| - [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | ||
| - [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] 865945: test: Restructure zuul-operator-functional-k8s layout https://review.opendev.org/c/zuul/zuul-operator/+/865945 | ||
| - [zuul/zuul-operator] 863579: test: Introduce Helm-based functional test https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: k8s: Provide an option to disable PXC operator installer https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 866231: k8s: Clean up cert-manager installer https://review.opendev.org/c/zuul/zuul-operator/+/866231 | ||
| - [zuul/zuul-operator] 863475: k8s: Provide an option to disable cert-manager installation 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] 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- Michael Kelly proposed: | 09:03 | |
| - [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | ||
| - [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] 865945: test: Restructure zuul-operator-functional-k8s layout https://review.opendev.org/c/zuul/zuul-operator/+/865945 | ||
| - [zuul/zuul-operator] 863579: test: Introduce Helm-based functional test https://review.opendev.org/c/zuul/zuul-operator/+/863579 | ||
| - [zuul/zuul-operator] 863474: k8s: Provide an option to disable PXC operator installer https://review.opendev.org/c/zuul/zuul-operator/+/863474 | ||
| - [zuul/zuul-operator] 866231: k8s: Clean up cert-manager installer https://review.opendev.org/c/zuul/zuul-operator/+/866231 | ||
| - [zuul/zuul-operator] 863475: k8s: Provide an option to disable cert-manager installation 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] 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- Marvin Becker proposed: [zuul/nodepool] 867971: Add additional pod specs to openshift driver https://review.opendev.org/c/zuul/nodepool/+/867971 | 13:05 | |
| -@gerrit:opendev.org- Marvin Becker proposed wip: [zuul/nodepool] 867971: Add additional pod specs to openshift driver https://review.opendev.org/c/zuul/nodepool/+/867971 | 13:43 | |
| -@gerrit:opendev.org- Marvin Becker proposed wip: [zuul/nodepool] 867971: Add additional pod specs to openshift driver https://review.opendev.org/c/zuul/nodepool/+/867971 | 15:10 | |
| -@gerrit:opendev.org- Marvin Becker proposed wip: [zuul/nodepool] 867971: Add additional pod specs to openshift driver https://review.opendev.org/c/zuul/nodepool/+/867971 | 15:15 | |
| @newbie23:matrix.org | Hi guys, my understanding is that the Zuul merger is in charge of merging changes so that scheduled jobs will be able to test them. | 15:57 |
| What about the final merge (i.e. in a usual configuration, after the gate pipeline) ? | ||
| Considering the GitHub driver, who is actually doing the final merge? GitHub or Zuul? | ||
| I have found a lead in the docs | ||
| https://zuul-ci.org/docs/zuul/latest/drivers/github.html#attr-pipeline.%3Creporter%3E.%3Cgithub%20source%3E.merge | ||
| but I am trying to crosscheck what I think I know :) | ||
| @jim:acmegating.com | newbie23: zuul sends the api call to github to do the merge, it does not push its own ref updates | 15:59 |
| @jim:acmegating.com | and yes, that's the setting that tells it to do that on successful completion of the pipeline | 15:59 |
| @newbie23:matrix.org | Thanks corvus ! | 16:00 |
| @newbie23:matrix.org | That clarifies a few things | 16:00 |
| @newbie23:matrix.org | Since I am at it... The doc says: | 16:02 |
| https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.merge-mode | ||
| "The merge mode which is used by Git for this project. Be sure this matches what the remote system which performs merges (i.e., Gerrit). The requested merge mode will also be used by the GitHub and GitLab drivers when performing merges" | ||
| It is not completely clear to me (aka I did not get it :) ), the "Be sure this matches what the remote system which performs merges (i.e., Gerrit)", what exactly has to match? | ||
| @jim:acmegating.com | newbie23: in the case of github, make sure it's one of the merge modes supported by the github project ('merge', 'cherry-pick', etc) | 16:08 |
| @newbie23:matrix.org | Got it. So provided the specific merge method is allowed, configured within GH, then is the zuul api call that will decide the merge type (e.g. squash merge) | 16:11 |
| @newbie23:matrix.org | is it correct? | 16:12 |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | 16:12 | |
| @jim:acmegating.com | newbie23: yep | 16:12 |
| @newbie23:matrix.org | thank you! | 16:19 |
| @hanson76:matrix.org | > <@jim:acmegating.com> Hanson: your part is done unless someone comes back with a comment, thanks! one of the zuul maintainers will workflow +1 when it's ready to submit | 16:47 |
| I can now confirm that this fix solved our problem and we are now able to run pods win Nodepool under k8s 1.24+ | ||
| @jim:acmegating.com | Hanson: thanks! | 16:51 |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | 17:09 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul-jobs] 867991: push-to-intermediate-registry: Handle container_images https://review.opendev.org/c/zuul/zuul-jobs/+/867991 | 17:55 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | 17:59 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | 18:34 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 867057: WIP: play with noxfile https://review.opendev.org/c/zuul/zuul/+/867057 | 19:10 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 868002: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 20:32 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 868002: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 20:40 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 867057: WIP: play with noxfile https://review.opendev.org/c/zuul/zuul/+/867057 | 21:00 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 868002: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 21:34 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 868002: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 21:39 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | 21:54 | |
| @jpew:matrix.org | Hmm, `build-container-images` seems to require the repository be prefixed by the registry name (e.g. `my-registry/foo/bar`) even though there is `registry` filed to specify the registry. This is unfortunate because it means you can't simply do promotion because you need to do something like: `http://my-registry/v2/foo/bar`).... which you cannot easily construct from the specified variables | 21:58 |
| @jpew:matrix.org | I gues you have to trim the registry name off the front of the repo string? | 21:59 |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 867057: WIP: play with noxfile https://review.opendev.org/c/zuul/zuul/+/867057 | 22:06 | |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 867057: WIP: play with noxfile https://review.opendev.org/c/zuul/zuul/+/867057 | 22:10 | |
| @clarkb:matrix.org | corvus: ^ I got bindep to pass all of its nox jobs so updated your zuul jobs to address the problems I saw with it. Still a bit of work to do to update the zuul-jobs change and add support for reporting inline comments back to zuul and siblings and so on. But hopefully we can see this do something with a less trivial set of jobs | 22:11 |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | 22:38 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 867938: Prefix managed resources with instance name https://review.opendev.org/c/zuul/zuul-operator/+/867938 | 23:22 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!