Add some operator improvements

slopezz commented 3 years ago

Remove namespace from title. Current grafana-operator versions are already grouping dashboard by namespace, so there is no need to add it to the title because it is done automatically by the operator
Update default namespace on examples Makefile to use the new one with operator-sdk v1 (prometheus-exporter-operator-system)
Re-structure ansible role to make easier the addition on new exporters:
- Every exporter customization is located on its own exporter directory, so all ansible templates are generic (deploymet, service, servicemonitor, grafanadashboard)
- Grafana dashboard management will be easier, because the direct json jinja template is located on its own exporter dir (so without indentiation)
- Every exporter container params (args, command, env, volumeMounts..), if they are present, are located on its own exporter dir
- Every exporter volumes, if they are present, are located on its own exporter dir
Fix error on mysql dashboard with current qps query when having more than one timeseries db because there are more than 1 pod withing defined interval.
Add new manual kustomize to make make deploy/undeploy targets work. On default kustomize (intended for olm) envvar WATCH_NAMESPACE is taken from olm annotation, and when used with make deploy (so no olm), operator fails because envvar WATCH_NAMESPACE is empty, it thinks it is working on cluster scope mode (which is disabled intenationally, operator is using role not cluster-role). So with new manual kustomize, we are overriding WATCH_NAMESPACE to use the default namespace WATCH_NAMESPACE=prometheus-exporter-operator-system
Refactor docs, and add documentation about how to install the operator (with the upgrade to operator-sdk v1) and how to do an operator release
Create new alpha release 0.3.0-alpha.12

3scale-robot commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: slopezz

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/3scale-ops/prometheus-exporter-operator/blob/main/OWNERS)~~ [slopezz] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

raelga commented 3 years ago

/lgtm

roivaz commented 3 years ago

/lgtm

3scale-ops / prometheus-exporter-operator

Add some operator improvements #24