Adding ability to add 'extraManifests'

[JunaidChaudry]

All Submissions:

• Have you opened an Issue before filing this PR?
• Have you signed our CLA?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

[JunaidChaudry]

Closes #213

[JunaidChaudry]

Can a reviewer kindly review this?

[igor-karpukhin]

cc @slaskawi @irajdeep

[JunaidChaudry]

@slaskawi Can you please review this one as well?

The feature aligns with many other opensource helm charts (e.g. grafana, prometheus, argo-cd, etc.), to support extra k8s manifests using the values files.
This allows the end user to add any additional files needed to support their helm deployment. A nice use case now a days is to mount secrets on the fly by leveraging the Secrets Store CSI Driver..

[slaskawi]

Overall, I'm fine with extraObjects. We're seen this approach in Grafana and Argo, and we see it useful.

I'm not sure about adding annotations on Service Accounts. Could you please provide us examples why it's necessary and extract this into a separate PR?

[github-actions]

This PR is being marked stale because it has been open for 60 days with no activity. Please update the PR or ask for a fresh review.

[slaskawi]

@JunaidChaudry Could you please check my comment and rebase this PR?

[JunaidChaudry]

@slaskawi Apologies for the VERY long delay.

I have rebased with main, and please also check out my comment around serviceaccount annotations, and why I think it should be included.

[github-actions]

This PR is being marked stale because it has been open for 60 days with no activity. Please update the PR or ask for a fresh review.