commonLabels and commonAnnotations

[famfop]

Edit:
This PR fixes setting labels and annotations on resources. Using bitnami/common's commonLabels and commonAnnotations, "global" labels and annotations may be set which will be added to all resources.

On certain resources, i.e. for cron, ingress, persistence, seederJob, serviceAccount and openproject "custom annotations may be set, which will be added together with the commonAnnotations.

I tested it with helm template test-release . -f customvalues.yaml where my customvalues.yaml looks like:

commonLabels:
  environment: staging
  testlabel: asdf
commonAnnotations:
  testAnnotation: hello
cron:
  annotations:
    customAnnotation: crontest
ingress:
  annotations:
    customAnnotation: ingresstest
persistence:
  annotations:
    customAnnotation: persistencetest
seederJob:
  annotations:
    customAnnotation: seederjobtest
serviceAccount:
  annotations:
    customAnnotation: serviceaccounttest
openproject:
  annotations:
    customAnnotation: openprojecttest
memcached:
  commonLabels:
    environment: staging
    testlabel: asdf
  commonAnnotations:
    testAnnotation: hellomemcached
postgresql:
  commonLabels:
    environment: staging
    testlabel: asdf
  commonAnnotations:
    testAnnotation: hellopostgres

Note that memcached and postgresql need the labels and annotations to be passed again as they are not being added automatically (thus doubling config).

---

Original PR message:
also, add commonLabels as empty dicts in values.yaml "for visibility".

This PR tries to fix the issue stated in issue #178

EDIT: Note, all "matchLabels" used in selectors were NOT checked, I only tested for setting custom labels which did work with this dummy customlabels.yaml

commonLabels:
  environment: staging
  testlabel: asdf
memcached:
  commonLabels:
    environment: staging
    testlabel: asdf
postgresql:
  commonLabels:
    environment: staging
    testlabel: asdf

[changeset-bot]

🦋 Changeset detected

Latest commit: a3f6bf0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@openproject/helm-charts	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[xeruf]

would love to see that merged, we have a similar issue with velero needing labels!

[oliverguenther]

Sorry this got lost for so long, but I've finally found time to include this. I extended it with a spec to ensure coverage and allow additional labels per web/worker deployment.