@jjbeckman:matrix.org | > <@fungicide:matrix.org> it looks like the prepare-workspace-git variant is for using git command-line operations instead of the synchronize module, and prepare-workspace-openshift is for overriding kubectl detection in order to force rsync to be used | 02:13 |
---|---|---|
> i don't think you'll find any. we don't have kubernetes clusters in opendev | ||
I see. Thank you. | ||
Just as you mentioned, I was not able to find such an example. | ||
> there may be a kubernetes-specific equivalent of the prepare-workspace role, i'll check the zuul-jobs documentation | ||
> looking at the descriptions for the several prepare-workspace* roles at https://zuul-ci.org/docs/zuul-jobs/general-roles.html#role-prepare-workspace i get the impression that ansible's synchronize module is supposed to magically use kubectl? | ||
I wasn't able to find k8s specific documentation regarding `prepare-workspace`, but it appears to me that it doesn't support kubectl | ||
> General Purpose Roles — Zuul-Jobs documentation | ||
> » Roles » General Purpose Roles View page source General Purpose Roles add-authorized-keys Install SSH public key(s) on all hosts This role is intended to be run at the end of a failed job for which the build node set will be held with zuul’s | ||
> in which case the generic prepare-workspace role could "just work" | ||
I guess this may work, but since I'm using kubectl, which doesn't require SSH, I feel this is an overkill just to copy source code over. | ||
> but i have pretty much zero experience with kubernetes, so hopefully someone who does can confirm | ||
> it looks like the prepare-workspace-git variant is for using git command-line operations instead of the synchronize module, and prepare-workspace-openshift is for overriding kubectl detection in order to force rsync to be used | ||
`prepare-workspace-git` would require somehow copying the GitHub App Private key over to the nodes, which feels tricky. Will check `prepare-workspace-openshift`, thanks, | ||
@jjbeckman:matrix.org | > <@tristanc_:matrix.org> jjbeckman: the project source must be copied explicitly using a base job pre-run. When the connection is kubectl, you might want to use the `prepare-workspace-openshift` role | 02:15 |
Thank you for the suggestions. I've checked out `prepare-workspace-openshift` (https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/prepare-workspace-openshift), and found out that it uses the `oc` command instead of `kubectl`, so it won't work for my environment. However, I've decided to implement something similar to be `kubectl` based, and I feel this is the way I should proceed. | ||
@jim:acmegating.com | jjbeckman: the zuul container images include oc | 02:17 |
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 878178: Vendor persistent recursive watch Kazoo support https://review.opendev.org/c/zuul/nodepool/+/878178 | 06:17 | |
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 06:20 | |
- [zuul/nodepool] 878179: Use a persistent recursive watch for caches https://review.opendev.org/c/zuul/nodepool/+/878179 | ||
- [zuul/nodepool] 877431: Use image cache when launching nodes https://review.opendev.org/c/zuul/nodepool/+/877431 | ||
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 877432: Use node cache in node deleter https://review.opendev.org/c/zuul/nodepool/+/877432 | 06:31 | |
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 877565: Log the reason we decline a request https://review.opendev.org/c/zuul/nodepool/+/877565 | 06:32 | |
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-client] 879528: cli: Further refactoring https://review.opendev.org/c/zuul/zuul-client/+/879528 | 07:31 | |
@mhuin:matrix.org | oh, dark mode got enabled on opendev gerrit? | 08:38 |
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 880032: Expose max. attempts to the job as a Zuul variable https://review.opendev.org/c/zuul/zuul/+/880032 | 09:30 | |
@apevec:matrix.org | dark is the new white | 09:56 |
@clarkb:matrix.org | > <@mhuin:matrix.org> oh, dark mode got enabled on opendev gerrit? | 13:36 |
It has been an option since we upgraded to 3.2 but it was experimental and required explicit opt in via your user settings page. With the upgrade to 3.7 last week it now defaults to matching your browser settings | ||
@mhuin:matrix.org | > <@clarkb:matrix.org> It has been an option since we upgraded to 3.2 but it was experimental and required explicit opt in via your user settings page. With the upgrade to 3.7 last week it now defaults to matching your browser settings | 13:37 |
Right, soon after I checked software factory's gerrit and saw the option was opt-in. Now I am just in dark mode everywhere | ||
@mhuin:matrix.org | so that's pretty nice that dark mode could be enabled in zuul soon as well | 13:37 |
@jim:acmegating.com | getting pretty close :) | 13:44 |
@clarkb:matrix.org | corvus: last week b.schanzel indicated that https://review.opendev.org/c/zuul/zuul/+/872226 would aid in local running of zuul tests due to debian on arm not having mysql pacakges. Did you want to go ahead and send that one in? I had to rebase it which lost your +2 | 18:24 |
@jim:acmegating.com | Clark: ++ | 18:41 |
@clarkb:matrix.org | hrm one test case out of however many hit `pymysql.err.OperationalError: (1130, "localhost' is not allowed to connect to this MariaDB server")` | 21:43 |
@clarkb:matrix.org | which is one because a bunch of other tests are usingthe same mariadb server | 21:43 |
@clarkb:matrix.org | * which is odd because a bunch of other tests are usingthe same mariadb server | 21:45 |
@jim:acmegating.com | Clark: maybe there's a different default connection limit? | 22:17 |
@jim:acmegating.com | Clark: and maybe we should collect the mariadb journal | 22:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!