Remove the code that generate the metrics for the new layout projects

[camilamacedo86]

Feature Request

Is your feature request related to a problem? Please describe.
Currently, we generate the metrics in the run.go(for the main). See: https://github.com/operator-framework/operator-sdk/blob/master/pkg/helm/run.go#L166

However, for the new layout, the metrics are customized. See that we scaffold a service monitor resource in ./config/prometheus/monitor.yaml.

Describe the solution you'd like

• Remove the roles added for it in internal/plugins/helm/v1/scaffolds/templates/role.go
• Replace the metrics impl from the image used by the new layout to be done as: joelanford/helm-operator@a9bcf84
• The main.go added in the [ansible/helm] remove operator-sdk binary from ansible/helm base images #3331 also needs to be updated

[camilamacedo86]

Following what I can think about that need to get done here;

• remove add metrics impl from ansible and helm
• remove the pkg/kube-metrics
• remove what is not longer used from pkg/k8sutil
• check if we have nothing about it to remove from the docs as well

[estroz]

@camilamacedo86 I believe this is done. Please re-open if not.

[camilamacedo86]

/reopen

HI @Ezric, still missing to remove the funcs that were used just in these features in https://github.com/operator-framework/operator-sdk/blob/master/internal/util/k8sutil/k8sutil.go

[openshift-ci-robot]

@camilamacedo86: Reopened this issue.

Details

In response to this:

/reopen

It is not. See that we still need to remove the funcs that were used just in these features in https://github.com/operator-framework/operator-sdk/blob/master/internal/util/k8sutil/k8sutil.go

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.