| @iwienand:matrix.org | If you do not need the cluster to pull images from a private registry, you can paste `{"auths":{"fake":{"auth":"aWQ6cGFzcwo="}}}` as the pull secret. | 00:02 |
|---|---|---|
| @iwienand:matrix.org | of course | 00:02 |
| @clarkb:matrix.org | Ya that is along the lines of what I remember reading | 00:05 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 00:56 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:02 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:10 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:18 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:23 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:27 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:39 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:42 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 01:58 | |
| @iwienand:matrix.org | 2022-08-23 02:01:59.354534 | centos-9-stream | You need to enable virtualization in BIOS | 02:03 |
| @iwienand:matrix.org | it seemed to fail on iweb, but passed on inmotion | 02:04 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 02:08 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 02:24 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 02:37 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 02:48 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] ensure-openshift-crc https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 03:02 | |
| @iwienand:matrix.org | $ ~/bin/crc start | 03:31 |
| only 8.321GB of memory found (9.664GB required) | ||
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] Use CRC to run a local OpenShift 4 https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 04:06 | |
| @iwienand:matrix.org | > <@gerrit:opendev.org> Ian Wienand proposed: [zuul/zuul-jobs] 854047: [wip] Use CRC to run a local OpenShift 4 https://review.opendev.org/c/zuul/zuul-jobs/+/854047 | 04:08 |
| i'm giving up on this for now; for the record I've put a bunch in the change description, but I have no idea if this is just tilting at windmils in a non-nested-virt 8gb system. | ||
| @iwienand:matrix.org | i think we should probably find another way to test zuul-registry; and the only other place it is used is in nodepool for tox testing against openshift. perhaps that can do some sort of API mocking | 04:10 |
| @iwienand:matrix.org | > <@iwienand:matrix.org> i think we should probably find another way to test zuul-registry; and the only other place it is used is in nodepool for tox testing against openshift. perhaps that can do some sort of API mocking | 04:33 |
| also for the record filed https://github.com/code-ready/crc/issues/3309 | ||
| @iwienand:matrix.org | https://github.com/code-ready/crc/issues/3309#issuecomment-1223599562 | 06:43 |
| @iwienand:matrix.org | ``` | 06:44 |
| Perhaps a microshift option would be more suited for this, but this is currently on hold due to the rebase they are performing. We will be looking into this soon | ||
| However 8GB would not suffice for running OpenShift 4.x as our default is really the bare minimum without an application running. | ||
| ``` | ||
| @iwienand:matrix.org | corvus: https://review.opendev.org/q/topic:stream-2.7-container is a series to add explicit 2.7 testing to zuul_stream. might be nice to have this running for the ansible 6 bringup too | 06:47 |
| -@gerrit:opendev.org- Benjamin Schanzel proposed on behalf of Tobias Henkel: [zuul/nodepool] 743790: Check for images to upload single threaded https://review.opendev.org/c/zuul/nodepool/+/743790 | 09:07 | |
| -@gerrit:opendev.org- Albin Vass proposed wip: [zuul/zuul] 854171: Fix links for jobs with special characters https://review.opendev.org/c/zuul/zuul/+/854171 | 11:04 | |
| -@gerrit:opendev.org- Albin Vass proposed wip: [zuul/zuul] 854171: Fix links for jobs with special characters https://review.opendev.org/c/zuul/zuul/+/854171 | 11:38 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 854174: Run docker-compose up test setup script with env https://review.opendev.org/c/zuul/zuul/+/854174 | 11:41 | |
| @westphahl:matrix.org | zuul-maint: ^ small fix for test-setup-docker.sh | 11:43 |
| @avass:vassast.org | > <@westphahl:matrix.org> zuul-maint: ^ small fix for test-setup-docker.sh | 11:49 |
| I just ran into that :) | ||
| -@gerrit:opendev.org- Ke Niu proposed: [zuul/zuul] 854161: remove unicode prefix from code https://review.opendev.org/c/zuul/zuul/+/854161 | 12:20 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 854174: Run docker-compose up test setup script with env https://review.opendev.org/c/zuul/zuul/+/854174 | 12:32 | |
| @westphahl:matrix.org | fungi: thanks for the comment. I did not seesorry, missed ianw changed | 12:32 |
| @westphahl:matrix.org | * fungi: thanks for the comment. I did not see ianw changed | 12:33 |
| @westphahl:matrix.org | * fungi: thanks for the comment. I did not see ianw change | 12:33 |
| @westphahl:matrix.org | update 854174 to default to the env cmd so we can pick which solution we want | 12:33 |
| @fungicide:matrix.org | > <@iwienand:matrix.org> $ ~/bin/crc start | 12:34 |
| > only 8.321GB of memory found (9.664GB required) | ||
| we do have test nodes in opendev with more ram, if there's an openshift-specific job which needs it | ||
| @westphahl:matrix.org | * updated 854174 to default to the env cmd so we can pick which solution we want | 12:35 |
| -@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed: [zuul/nodepool] 854181: Run docker-compose up test setup script with env https://review.opendev.org/c/zuul/nodepool/+/854181 | 12:40 | |
| -@gerrit:opendev.org- Albin Vass marked as active: [zuul/zuul] 854171: Fix links for jobs with special characters https://review.opendev.org/c/zuul/zuul/+/854171 | 13:23 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 13:37 | |
| - [zuul/nodepool] 852670: Add support for dynamic tags https://review.opendev.org/c/zuul/nodepool/+/852670 | ||
| - [zuul/nodepool] 853371: AWS add support for multiple instance types https://review.opendev.org/c/zuul/nodepool/+/853371 | ||
| - [zuul/nodepool] 853372: Convert GCE to state machine driver and remove simple https://review.opendev.org/c/zuul/nodepool/+/853372 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 13:41 | |
| - [zuul/nodepool] 852466: Allow specifying diskimage metadata/tags https://review.opendev.org/c/zuul/nodepool/+/852466 | ||
| - [zuul/nodepool] 852670: Add support for dynamic tags https://review.opendev.org/c/zuul/nodepool/+/852670 | ||
| - [zuul/nodepool] 853371: AWS add support for multiple instance types https://review.opendev.org/c/zuul/nodepool/+/853371 | ||
| - [zuul/nodepool] 853372: Convert GCE to state machine driver and remove simple https://review.opendev.org/c/zuul/nodepool/+/853372 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 853552: Add Ansible 6 https://review.opendev.org/c/zuul/zuul/+/853552 | 14:01 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 854046: Remove merge-failure pipeline configuration https://review.opendev.org/c/zuul/zuul/+/854046 | 14:11 | |
| @avass:vassast.org | Simple fix to get the api and links to the job pages working if anyone has time for a quick review: https://review.opendev.org/c/zuul/zuul/+/854171 | 14:16 |
| -@gerrit:opendev.org- Zuul merged on behalf of Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org: [zuul/nodepool] 854181: Run docker-compose up test setup script with env https://review.opendev.org/c/zuul/nodepool/+/854181 | 14:20 | |
| @jim:acmegating.com | Albin Vass: we used to have some character restrictions in job names... have you run into any other problems with '/' in a job name? any issues with zuul-jobs roles? | 14:23 |
| @avass:vassast.org | We've been running jobs with that format for a couple of months now and that's the only issue we've seen so far | 14:24 |
| @avass:vassast.org | Everything else has been working normally | 14:26 |
| @jim:acmegating.com | thanks | 14:31 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 854174: Run docker-compose up test setup script with env https://review.opendev.org/c/zuul/zuul/+/854174 | 14:38 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Albin Vass: [zuul/zuul] 854171: Fix links for jobs with special characters https://review.opendev.org/c/zuul/zuul/+/854171 | 15:50 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 18:07 | |
| - [zuul/nodepool] 852670: Add support for dynamic tags https://review.opendev.org/c/zuul/nodepool/+/852670 | ||
| - [zuul/nodepool] 853371: AWS add support for multiple instance types https://review.opendev.org/c/zuul/nodepool/+/853371 | ||
| - [zuul/nodepool] 853372: Convert GCE to state machine driver and remove simple https://review.opendev.org/c/zuul/nodepool/+/853372 | ||
| @iwienand:matrix.org | > <@fungicide:matrix.org> we do have test nodes in opendev with more ram, if there's an openshift-specific job which needs it | 19:02 |
| in combination with having to override the kvm tests, makes me think that it won't work with non-nested virt anyway | ||
| @fungicide:matrix.org | > <@iwienand:matrix.org> in combination with having to override the kvm tests, makes me think that it won't work with non-nested virt anyway | 19:11 |
| at least one provider where we have larger flavors also supports nested virt, so could still be worth a try | ||
| @jim:acmegating.com | ianw: are you still interested in maintaining the changes to zuul_console with its additional complexity, and also the test environment? i noticed in 853208 you suggested some future work but said you don't want to tackle it now. if you still want all of this and are willing to continue work on it, that's fine. but i think given that the original issue is moot, we do have a brief opportunity to revert it before our next release. | 19:30 |
| @fungicide:matrix.org | or at least back it out temporarily and put it in again with more testing once there's an opportunity to work on that | 19:31 |
| @iwienand:matrix.org | it couldn't be more tested in terms of in-zuul testing | 19:32 |
| @iwienand:matrix.org | i feel like https://review.opendev.org/c/zuul/zuul/+/850269/ https://review.opendev.org/c/zuul/zuul/+/850593/ are generically good changes | 19:32 |
| @iwienand:matrix.org | as is https://review.opendev.org/c/zuul/zuul/+/850718 | 19:33 |
| @fungicide:matrix.org | oh, right now i get what was meant by "and also the test environment" | 19:34 |
| @iwienand:matrix.org | testing against 2.7 should be done anyway (https://review.opendev.org/c/zuul/zuul/+/853208) | 19:34 |
| @iwienand:matrix.org | the remaining issue is that if you are running zuul console AND it can't be communicated to, it will leak the files | 19:35 |
| @fungicide:matrix.org | * oh, right .now i get what was meant by "and also the test environment" | 19:36 |
| @iwienand:matrix.org | that can probably be more explicitly called out in https://review.opendev.org/c/zuul/zuul/+/851942 | 19:36 |
| @fungicide:matrix.org | * oh, right. now i get what was meant by "and also the test environment" | 19:37 |
| @iwienand:matrix.org | the protocol option gives us the opportunity to modify the file-name on disk; a-la https://review.opendev.org/c/zuul/zuul/+/850079 | 19:37 |
| @jim:acmegating.com | ianw: at this point,it is a feature with no user requirement driving it. so i'm trying to gauge your interest in maintaining all of this complexity in the future. if you're still up for pushing it forward and are interested in dealing with further issues and maintenance great. but given that opendev won't be making use of the feature, if you lose interest in it, it may be better to revert some of the complexity before it gets baked into a release. | 19:39 |
| @jim:acmegating.com | also, they're all good changes. :) | 19:40 |
| @iwienand:matrix.org | i haven't lost any interest; i think this is a needed feature of zuul but it seems we continue to have room for improvement on it and i think we have the basis of it | 19:41 |
| @jim:acmegating.com | ianw: okay, stick with the plan works for me :) | 19:42 |
| @iwienand:matrix.org | i'm wondering if the protocol work can now build on some the bits i quickly looked at that was tunneling this console stream over a unix socket | 19:43 |
| @jim:acmegating.com | that would be swell, i'd love to see those changes resurrected | 19:43 |
| @iwienand:matrix.org | for reference https://review.opendev.org/c/zuul/zuul/+/542469/ is what i was looking at | 19:51 |
| -@gerrit:opendev.org- Marvin Becker proposed wip: [zuul/nodepool] 853993: [WIP] Add disable and enable node commands https://review.opendev.org/c/zuul/nodepool/+/853993 | 20:12 | |
| @mbecker12:matrix.org | Hi everyone! In our team, we have a use-case for putting nodes in an idle state while running maintenance tasks on them. I understand that this is likely a rather niche use-case but in our setup of static nodes we were missing the functionality to disable a worker in nodepool so that it doesn't get anything scheduled during maintenance. I proposed some changes above and it would be cool to discuss this to make sure this a reasonable addition and that it behaves as expected also for other types of nodes :) | 20:16 |
| @iwienand:matrix.org | mbecker12: apropos a prior discussion here, do you run zuul_console and have port 19885 open to executors on your static nodes? | 20:32 |
| @mbecker12:matrix.org | as part of a pre playbook to a job, yes we run zuul_console. and looks like the port is open as well | 21:04 |
| @clarkb:matrix.org | mbecker12: I left a comment around trying to make that sort of state change safer for various node types and nodes in different states. Let me know what you think in the review | 21:21 |
| @jim:acmegating.com | Clark: mbecker12 replied. fwiw, i don't think the use case is niche. but also note that another way to address that use case is to update the nodepool config to add/remove nodes from it. that means that changes are more audited. | 21:31 |
| @jim:acmegating.com | that can be a little slow right now of course if a config file change has to go through deployment tooling; but if we implement the nodepool-in-zuul spec i'm proposing, then those changes would be instantaneous. | 21:32 |
| -@gerrit:opendev.org- Ian Wienand proposed: | 21:34 | |
| - [zuul/zuul] 853603: zuul_stream: handle non-string msg value https://review.opendev.org/c/zuul/zuul/+/853603 | ||
| - [zuul/zuul] 854284: [wip] capture callback errors in remote zuul jobs https://review.opendev.org/c/zuul/zuul/+/854284 | ||
| - [zuul/zuul] 854285: zuul_stream: handle failed_when tasks https://review.opendev.org/c/zuul/zuul/+/854285 | ||
| @jim:acmegating.com | mbecker12: what version of zuul are you running? | 21:35 |
| @mbecker12:matrix.org | > update the nodepool config to add/remove nodes from it. | 21:38 |
| we're doing that right now but are not really happy with that solution as it is quite cumbersome, at least in our setup, to have the tooling around it in order to make it work properly | ||
| @mbecker12:matrix.org | * > update the nodepool config to add/remove nodes from it. | 21:38 |
| we're doing that right now but are not really happy with that solution as it is quite cumbersome, at least in our setup, to have the tooling around it in order to make it work properly | ||
| @mbecker12:matrix.org | 6.1.0 | 21:38 |
| @jim:acmegating.com | > <@mbecker12:matrix.org> > update the nodepool config to add/remove nodes from it. | 21:39 |
| > | ||
| > we're doing that right now but are not really happy with that solution as it is quite cumbersome, at least in our setup, to have the tooling around it in order to make it work properly | ||
| cool, then that workflow should get better in the future. that doesn't preclude this change though -- it's nice to be able to do emergency actions quickly. | ||
| -@gerrit:opendev.org- Ian Wienand proposed: | 21:59 | |
| - [zuul/zuul] 854284: [wip] capture callback errors in remote zuul jobs https://review.opendev.org/c/zuul/zuul/+/854284 | ||
| - [zuul/zuul] 854285: zuul_stream: handle failed_when tasks https://review.opendev.org/c/zuul/zuul/+/854285 | ||
| - [zuul/zuul] 853603: zuul_stream: handle non-string msg value https://review.opendev.org/c/zuul/zuul/+/853603 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 849887: Add Nodepool in Zuul spec https://review.opendev.org/c/zuul/zuul/+/849887 | 22:10 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853603: zuul_stream: handle non-string msg value https://review.opendev.org/c/zuul/zuul/+/853603 | 23:40 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!