[artifactory-ha] Add Annotations to Primary Artifactory Service and Resources to Nginx Deployment

[yellowplane13]

Add Annotations to primary artifactory service
Add resources to setup container in the nginx deployment yaml

PR Checklist

• Title of the PR starts with chart name (e.g. [artifactory])

What this PR does / why we need it:
Add Annotations to Primary Artifactory Service: This change adds annotations to the primary Artifactory service.

Add resources block to setup container in nginx deployment yaml: This change gives the ability to change limits and requests for the setup init container within the nginx deployment.

[github-actions]

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

[yellowplane13]

I have read the CLA Document and I hereby sign the CLA

[se-kai]

maybe "{{ toYaml .Values.initContainers.resources | indent 10 }}" better? this is init container

[yellowplane13]

Yes, you're right. Just updated it, thanks for pointing it out

[yellowplane13]

Hi @se-kai , could you please review the PR again?
Tagging @oumkale and @chukka as well. As soon as this is merged into a release, we can use this as our upstream chart instead of downloading the entire thing into our repo. Thank you 🙏

[se-kai]

this is already fixed and avialble in current main branch
https://github.com/jfrog/charts/blob/master/stable/artifactory-ha/templates/nginx-deployment.yaml#L79
I used it already and it works fine for me.

[yellowplane13]

Hi @se-kai , thank you for the update. Appreciate having resources block added to nginx-deployment.

We also need annotations added to artifactory-primary-service.yaml's metadata. It is not originally added and we can't use this chart as an upstream until that's added unfortunately. Can that be taken into account for the next release? Link for reference : https://github.com/jfrog/charts/blob/master/stable/artifactory-ha/templates/artifactory-primary-service.yaml#L6
I can rebase and update this PR with just the artifactory-primary-service.yaml changes required if you need.