Review `parca-scrape-target` for listing

[ppasotti (Pietro Pasotti)]

The Parca Scrape Agent Operator is a workloadless charm meant to act as integration point for uncharmed workloads exposing a pprof endpoint so that they can be scraped by the parca-k8s charm.

This charm is in that respect very similar to prometheus-scrape-target.

PR for code review: parca-scrape-target-operator/pull/428

Metadata links

• charmhub.io : Charmhub | Deploy Parca Scrape Target Operator using Charmhub - The Open Operator Collection
• project repository: GitHub - canonical/parca-scrape-target-operator: A charmed operator for scraping Parca targets that do not exist within the Juju model. · GitHub

CI Links

• Code linting or-style checks: parca-scrape-target-operator/tox.ini at main · canonical/parca-scrape-target-operator · GitHub
• Release automation: observability/.github/workflows/charm-pull-request.yaml at main · canonical/observability · GitHub
• Unit test implementation: parca-k8s-operator/tests/unit at main · canonical/parca-scrape-target-operator · GitHub
• Unit test results: https://github.com/canonical/parca-scrape-target-operator/actions/runs/13493720782
• Integration test implementation:parca-scrape-target-operator/tests/integration at main · canonical/parca-scrape-target-operator · GitHub
• Integration test results: https://github.com/canonical/parca-scrape-target-operator/actions/runs/13493720782

Documentation Links

• Usage: Charmed Parca
• Contribution: parca-scrape-target-operator/CONTRIBUTING.md at main · canonical/parca-scrape-target-operator · GitHub
• Licensing: parca-scrape-target-operator/LICENSE at main · canonical/parca-scrape-target-operator · GitHub
• Publisher: Pietro Pasotti (pietro.pasotti@canonical.com) Canonical Observabitaly platform

[tony-meyer]

Thanks @ppasotti for the review request!

@weii-wang can you please help with this review? You can go through and tick off the items on the checklist and post the result in this thread. This prior listing request could serve as an example of how to go about it. Please ping me or anyone from Charm-Tech for any questions.

Note that the parca-agent (subordinate) and parca-k8s-operator charms are also being reviewed at the moment, but I think the reviews can all be done independently.

[weii-wang]

Yes, no problem, I will review this shortly, thanks!

[weii-wang]

@ppasotti I have finished the review. Overall, it looks great to me. I’ve left some minor comments, thanks!

[ppasotti (Pietro Pasotti)]

Thanks! So how does this work now @tony-meyer, do we need to address all comments before the listing can proceed, or is it fine if we create issues for them and process them in our regular roadmap?

[tony-meyer]

Thanks @weii-wang!

@ppasotti

So how does this work now @tony-meyer, do we need to address all comments before the listing can proceed, or is it fine if we create issues for them and process them in our regular roadmap?

None of those comments look like blockers for listing to me. I think it’s fine to go ahead and list and you can address them through your regular processes. @odysseus-k could you organise the listing, please?

[tony-meyer]

Hi @odysseus-k and @adityagoel28 - I think maybe this one was missed? It’s good for listing, but I don’t see it in the store.

[adityagoel28 (Aditya Goel)]

Hi @tony-meyer , I have listed the charm prometheus-scrape-target-k8s. It is visible now.

[mmkay (Mateusz Kulewicz)]

Hey @adityagoel28: you mention prometheus-scrape-target-k8s (which is also important to be listed) but shouldn’t it be parca-scrape-target? I don’t see it (maybe yet) when I search for Charmhub | The Open Operator Collection

[tony-meyer]

I see parca-scrape-target when I search for “parca” now, so I assume this was just a timing issue.