search

canonical / blackbox-exporter-k8s-operator

https://charmhub.io/blackbox-exporter-k8s
Apache License 2.0
0 stars 3 forks source link

Remove catalogue relation #3

Closed simskij closed 1 year ago

simskij commented 1 year ago

https://github.com/canonical/blackbox-exporter-k8s-operator/blob/b7680927df57356b8472bb6cc7ff5e98ba8937f8/metadata.yaml#L43-L44

As the blackbox-exporter doesn't seem to provide any UI (or ingress for that matter), it doesn't really make sense to list it in the catalogue.

lucabello commented 1 year ago

It does provide a (though basic) UI to access the configuration and the logs of a specific executed probe though; though that's maybe not the most relevant for the end-user (?), so I'd understand if we want to remove it

simskij commented 1 year ago

It does provide a (though basic) UI to access the configuration and the logs of a specific executed probe though; though that's maybe not the most relevant for the end-user (?), so I'd understand if we want to remove it

We need to decide either/or. If we list it in the catalogue, it needs to be ingressed. :) Currently it's listed, but not accessible.

lucabello commented 1 year ago

I'm fine with either :) Adding ingress seems trivial enough, so we probably should