[jaeger-operator] Chart missing idiomatic features and alignment with operator manifests

stevehipwell commented 2 years ago

Requirement - what kind of business use case are you trying to solve?

I need the jaeger-operator Helm chart to be able to match the capabilities described by the operator manifests and to be able to be customised with idiomatic Helm patterns.

Problem - what in Jaeger blocks you from solving the requirement?

The following areas are some of the areas in the jaeger-operator Helm chart that need improving.

Update Helm chart API to v2 for Helm v3 (v1 is EOL)
Update CRD to match the operator manifest
Add missing pod probes
Add standard chart customisation points
Add ServiceMonitor (as defined in Operator manifests)
Fix watch namspace logic

Proposal - what do you suggest to solve the problem or improve the existing situation?

I'm happy to open a PR to make these changes, but I've seen a number of open PRs covering some of these areas that haven't been merged so I'd like to know that if I did this work it would be reviewed and merged if correct?

Any open questions to address

n/a

batazor commented 2 years ago

@stevehipwell Yeah, that would be great.

stevehipwell commented 2 years ago

I'll try and get a PR in this week.

jaegertracing / helm-charts