| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 897664: wip: Send job parent data + artifacts via build request https://review.opendev.org/c/zuul/zuul/+/897664 | 05:25 | |
| @mhuin:matrix.org | Hello, back to nodepool and statsd metrics, I'm reading two apparently conflicting infos in the doc: in installation.rst, it is said that openstacksdk will emit API statistics prefixed with openstack.api (which is the default behavior for openstacksdk) - and in operation.rst, the prefix is said to be "nodepool.task" (which I don't see appearing in the code but I might have been looking at the wrong places). So which one is the right prefix? | 07:38 |
|---|---|---|
| @mhuin:matrix.org | A quick look at https://graphite.opendev.org seems to suggest "nodepool.task" is the right one, but it might also be that the cloud config sets the prefix explicitly | 07:41 |
| @fungicide:matrix.org | > <@mhuin:matrix.org> Hello, back to nodepool and statsd metrics, I'm reading two apparently conflicting infos in the doc: in installation.rst, it is said that openstacksdk will emit API statistics prefixed with openstack.api (which is the default behavior for openstacksdk) - and in operation.rst, the prefix is said to be "nodepool.task" (which I don't see appearing in the code but I might have been looking at the wrong places). So which one is the right prefix? | 12:15 |
| looks like you're right, the docs could use fixing if you're up for that. here's where opendev sets it, for example: https://opendev.org/opendev/system-config/src/branch/master/playbooks/templates/clouds/nodepool_clouds.yaml.j2#L26-L28 | ||
| @mhuin:matrix.org | thanks fungi! Sure, will do | 12:16 |
| @fungicide:matrix.org | keep in mind that parts of what is now openstacksdk used to live inside nodepool itself, so i wouldn't be surprised if some assumptions date back to when that really was a default in nodepool | 12:21 |
| @mfeder:matrix.org | > <@mhuin:matrix.org> My team might implement just that or prometheus alerting integration, I can let you know what we come up with | 12:29 |
| mhu: I'm grappling with a similar dilemma: whether to utilize and process Zuul's MQTT messages or opt for Prometheus alerting to report to the Matrix chat. Could you provide details on the conclusion of your Slack chat reporting from Zuul? | ||
| @mhuin:matrix.org | well, we didn't go much further than the drawing board actually... other priorities came up | 12:31 |
| @mhuin:matrix.org | might be easier to look into prometheus integration. There are statsd metrics related to node failures that are emitted by the scheduler and nodepool | 12:32 |
| @mhuin:matrix.org | so you can ingest them in prometheus with a statsd exporter, you'd need a metrics mapping first | 12:33 |
| @mhuin:matrix.org | luckily I got you -almost- covered: https://softwarefactory-project.io/r/c/software-factory/sf-operator/+/29482 you'll find a script there that scrapes the doc to generate a mapping config | 12:34 |
| @mhuin:matrix.org | then IIUC AlertManager has a slack plugin that only requires a Slack API web hook to push alerts to a channel - much easier than managing a proper bot | 12:35 |
| @mhuin:matrix.org | how about you? how did you intend to tackle this? | 12:35 |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 897664: wip: Send job parent data + artifacts via build request https://review.opendev.org/c/zuul/zuul/+/897664 | 12:44 | |
| @flaper87:matrix.org | 👋 Hi folks! Is there a way to tell Zuul to not stop the check pipeline on PR approvals? Right now, when a PR is approved, the check pipeline is interrupted in favor of whatever that approval triggers and I'd like to change our configs so that it will still finish the check pipeline (assuming it's possible) | 13:02 |
| @jim:acmegating.com | flaper87: remove `supercedes: check` from gate | 13:05 |
| @jim:acmegating.com | flaper87: https://zuul-ci.org/docs/zuul/latest/config/pipeline.html#attr-pipeline.supercedes | 13:07 |
| @mhuin:matrix.org | oh, sorry, didn't see you meant to publish to matrix ... well I'm sure someone somewhere has an AlertManager plugin supporting matrix or irc | 13:11 |
| @flaper87:matrix.org | ah, that's where it is! Thanks 🙏 | 13:14 |
| -@gerrit:opendev.org- Flavio Percoco Premoli proposed: [zuul/zuul] 883837: Generate a per-event timestamp for the ES reporter https://review.opendev.org/c/zuul/zuul/+/883837 | 13:43 | |
| -@gerrit:opendev.org- Flavio Percoco Premoli proposed: [zuul/nodepool] 875847: Allow to pass ImagePullSecrets to Kubernetes pods https://review.opendev.org/c/zuul/nodepool/+/875847 | 13:44 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: [WIP] K8s-based providers: emit API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 13:52 | |
| @flaper87:matrix.org | Is there a way to tell Zuul to add all projects its got access to to the `untrusted-projects` list? Projects come/go often enough in our GH org and it's a bit annoying to have to update the `untrusted-projects` list. I could come up with some way to automatically update this list but maybe there's a better way to do it with Zuul | 14:09 |
| -@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 | 14:28 | |
| @mhuin:matrix.org | doc patch is live https://review.opendev.org/c/zuul/nodepool/+/897840 | 14:29 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: [WIP] K8s-based providers: emit API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 14:46 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: [WIP] K8s-based providers: emit API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 15:23 | |
| @fungicide:matrix.org | > <@flaper87:matrix.org> Is there a way to tell Zuul to add all projects its got access to to the `untrusted-projects` list? Projects come/go often enough in our GH org and it's a bit annoying to have to update the `untrusted-projects` list. I could come up with some way to automatically update this list but maybe there's a better way to do it with Zuul | 16:01 |
| there are a few ways, one is to use a regex in place of a project name | ||
| @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 | 16:02 |
| @fungicide:matrix.org | flaper87: https://zuul-ci.org/docs/zuul/latest/tenants.html covers the `scheduler.tenant_config_script` possibility. i think i was mistaken about regular expressions for project names in the tenant config, you can use them for project entries in project configs (like for adding jobs to pipelines for multiple projects at once) | 16:12 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 897859: WIP: Add image debug build option https://review.opendev.org/c/zuul/zuul/+/897859 | 16:44 | |
| @mordred:inaugust.com | flaper87: I feel like either mnaser or mhu at some point in the past did an automation for that | 17:37 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 897859: WIP: Add image debug build option https://review.opendev.org/c/zuul/zuul/+/897859 | 17:39 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/nodepool] 897835: [WIP] K8s-based providers: emit API metrics with statsd https://review.opendev.org/c/zuul/nodepool/+/897835 | 18:27 | |
| @mhuin:matrix.org | If I did, I must have done it in an altered state, I have no recollection of it, sorry - so probably not me | 18:35 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 897859: WIP: Add image debug build option https://review.opendev.org/c/zuul/zuul/+/897859 | 20:02 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 897859: Add image debug build option https://review.opendev.org/c/zuul/zuul/+/897859 | 20:21 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 897859: Add image debug build option https://review.opendev.org/c/zuul/zuul/+/897859 | 21:32 | |
| @fungicide:matrix.org | i do lots of things in an altered state, but pretty sure that one wasn't me | 21:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!