| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 897874: WIP: Refactor bundle in sql connection https://review.opendev.org/c/zuul/zuul/+/897874 | 00:03 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/nodepool] 897928: dnm: Log all AWS API list operation times https://review.opendev.org/c/zuul/nodepool/+/897928 | 08:33 | |
| @flaper87:matrix.org | > <@fungicide:matrix.org> though also the tenant config can be replaced by a generator script and zuul will run that and use its output as the config | 08:38 |
|---|---|---|
| Oh wow, didn't know a regex was allowed there. that would totally do it for us: `org/*` | ||
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: K8s-based providers: emit CoreV1 API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 08:55 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: K8s-based providers: emit CoreV1 API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 09:09 | |
| -@gerrit:opendev.org- Flavio Percoco Premoli proposed: [zuul/zuul] 897931: Mention that untrusted-projects' names can be regexes https://review.opendev.org/c/zuul/zuul/+/897931 | 09:18 | |
| @flaper87:matrix.org | > <@flaper87:matrix.org> Oh wow, didn't know a regex was allowed there. that would totally do it for us: `org/*` | 09:18 |
| regex is working just fine! Wish I knew this before, hope this helps: https://review.opendev.org/c/zuul/zuul/+/897931 | ||
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897840: Doc: clarify statsd metrics reported by openstacksdk https://review.opendev.org/c/zuul/nodepool/+/897840 | 09:32 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: K8s-based providers: emit CoreV1 API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 09:44 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: K8s-based providers: emit CoreV1 API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 11:36 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 897942: AWS driver: Check for volume existence before calculating quota https://review.opendev.org/c/zuul/nodepool/+/897942 | 11:50 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 897942: AWS driver: Check for volume existence before calculating quota https://review.opendev.org/c/zuul/nodepool/+/897942 | 11:52 | |
| @fungicide:matrix.org | > <@flaper87:matrix.org> regex is working just fine! Wish I knew this before, hope this helps: https://review.opendev.org/c/zuul/zuul/+/897931 | 12:29 |
| that's awesome news, thanks for updating the docs! i thought i'd seen it done before, but couldn't find any examples when i went looking so naively assumed it wasn't implemented | ||
| @flaper87:matrix.org | > <@fungicide:matrix.org> that's awesome news, thanks for updating the docs! i thought i'd seen it done before, but couldn't find any examples when i went looking so naively assumed it wasn't implemented | 13:02 |
| Actually, I need to revisit this. I saw it working this morning but it suddenly stopped working. Not sure if this morning was all cached. 🤔 | ||
| @flaper87:matrix.org | > <@fungicide:matrix.org> that's awesome news, thanks for updating the docs! i thought i'd seen it done before, but couldn't find any examples when i went looking so naively assumed it wasn't implemented | 13:02 |
| * Actually, I need to revisit this. I saw it working this morning but it suddenly stopped working. Not sure if this morning was all cached. 🤔 Let me get back to you on this | ||
| @flaper87:matrix.org | > <@flaper87:matrix.org> Actually, I need to revisit this. I saw it working this morning but it suddenly stopped working. Not sure if this morning was all cached. 🤔 Let me get back to you on this | 13:12 |
| Looking at the code, it doesn't seem like GH's source allows for regex. I guess it worked in the AM because all projects were cached somewhere in ZK | ||
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: K8s-based providers: emit CoreV1 API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 13:41 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 897942: AWS driver: Check for volume existence before calculating quota https://review.opendev.org/c/zuul/nodepool/+/897942 | 14:32 | |
| @flaper87:matrix.org | Ok, so, looks like my only options are: | 14:53 |
| - Keep an explicit list of projects | ||
| - Use the tenant_config_script | ||
| @fungicide:matrix.org | flaper87: yeah, or you could try to add regex support, but i have a feeling that would mean querying the remote system for all repository names matching some pattern (or for the names of every repository it has and filtering locally), and it's quite likely that's impractical/impossible | 15:37 |
| @fungicide:matrix.org | gerrit, fwiw, does have a command for listing projects that match a regex. github on the other hand may not | 15:38 |
| @fungicide:matrix.org | `tenant_config_script` has the benefit of being able to encapsulate any weird query logic you want to stick in it, at least, so for github that might be calling the api to request a list of projects in a specific org with a certain characteristic/setting on them | 15:46 |
| @jim:acmegating.com | (or automate the generation of the tenant yaml, if that happens to be easier in your deployment) | 15:52 |
| @fungicide:matrix.org | good point | 16:04 |
| @fungicide:matrix.org | like it could be deployed by ansible from a jinja2 template or whatever | 16:04 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 895518: Load config errors less often https://review.opendev.org/c/zuul/zuul/+/895518 | 16:15 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 895742: Add tenant_status web endpoint https://review.opendev.org/c/zuul/zuul/+/895742 | 16:29 | |
| @mhuin:matrix.org | Hello zuul-maint, patch https://review.opendev.org/c/zuul/nodepool/+/897835 adds "API metrics parity" between statsd metrics issued by openstacksdk and the kubernetes client used by the kubernetes and openshift* drivers - although scoped to CoreV1 API, but could be extended to RBAC as well. I thought what openstacksdk offers out of the box is pretty useful so I was hoping to extend this to every relevant driver eventually | 16:31 |
| @mhuin:matrix.org | my initial intention was to add the feature directly in the python kubernetes client library, but the code is mostly auto-generated from [OpenAPITools's client generator](https://github.com/OpenAPITools/openapi-generator) | 16:33 |
| @mhuin:matrix.org | making it not impossible, but probably difficult to add such a feature upstream, hence doing it in Nodepool instead | 16:34 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 897874: WIP: Refactor bundle in sql connection https://review.opendev.org/c/zuul/zuul/+/897874 | 23:34 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!