canonical / prometheus-scrape-config-k8s-operator

https://charmhub.io/prometheus-scrape-config-k8s
Apache License 2.0
1 stars 1 forks source link

Update INTEGRATING.md #48

Closed sed-i closed 4 months ago

sed-i commented 5 months ago

Issue

Deployment diagram is incorrect.

Solution

Fix it.

Fixes: #42

sed-i commented 4 months ago

Thanks @dstathis!

lxd model should have no scrape-config charm at all (cross-model scrape is not supported)

I wonder if in some cases, prometheus could resolve the metric endpoints that scrape-config is collecting for prometheus:

so maybe we don't include the machine model

Afaik, scrape-config is currently used in VM models.

scrape-target should probably live in the COS model

That would certainly work, but I think we do not want to document that use case?