Helm: Add optional annotations for podMonitors and prometheusRules

[sathieu]

My particular use case is adding the following annotation:

argocd.argoproj.io/sync-options: SkipDryRunOnMissingResource=true

Ref: https://argo-cd.readthedocs.io/en/stable/user-guide/sync-options/#skip-dry-run-for-new-custom-resources-types

[fedepaol]

cc @gclawes

@sathieu mind checking the CI failures?

[sathieu]

@fedepaol I don't understand the failures, but I reworked the code. I need that someone unlock the CI.

[sathieu]

@fedepaol I don't understand the CI failure:

Error: An error occurred trying to start process '/usr/bin/bash' with working directory '/home/runner/work/metallb/metallb/metallb'. No such file or directory

[fedepaol]

@fedepaol I don't understand the CI failure:

Error: An error occurred trying to start process '/usr/bin/bash' with working directory '/home/runner/work/metallb/metallb/metallb'. No such file or directory

oh, the use-operator lane was broken for other reasons (waiting for metallb/metallb-operator#206 to be merged) but this may be some other thing, let me fix it.
Your PR doesn't have anything to do with that.

[sathieu]

@fedepaol I've rebased on top of main.

[gclawes]

The diff looks good to me. @fedepaol are we still waiting on a fix for the operator related CI failures?

[fedepaol]

The diff looks good to me. @fedepaol are we still waiting on a fix for the operator related CI failures?

Should be good now, I am trying to keep a fifo approach in open prs (and sorry, I've been busy in the past few days).
Also, @gclawes if a PR looks good to you, just click the rebase button and after CI is green merge it :-)

[sathieu]

thanks for merging 🙏

[fedepaol]

Thanks for the contribution! ❤️