jfrog / JFrog-Cloud-Installers

Template to deploy Artifactory Enterprise cluster.
Apache License 2.0
78 stars 138 forks source link

[ansible/artifactory_nginx] refactoring of artifactory_nginx role #393

Closed EmptyByte closed 3 months ago

EmptyByte commented 3 months ago

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

What this PR does / why we need it: Refactor the artifactory_nginx ansible role and removes the requirement of a sub-sub-role. It is now more modular. By default it does not configure nginx repositories unless explicitly requested with: artifactory_nginx_use_official_repos: true

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # N/A

Special notes for your reviewer: N/A

github-actions[bot] commented 3 months ago

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

EmptyByte commented 3 months ago

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

vasukinjfrog commented 3 months ago

Review in Progress