allegroai / clearml-helm-charts

Helm chart repository for the new unified way to deploy ClearML on Kubernetes. ClearML - Auto-Magical CI/CD to streamline your AI workload. Experiment Management, Data Management, Pipeline, Orchestration, Scheduling & Serving in one MLOps/LLMOps solution
https://clear.ml/docs
36 stars 50 forks source link

Issue-275: Honor existingAgentk8sglueSecret in deployment template #276

Closed savitha-qs closed 7 months ago

savitha-qs commented 7 months ago

What this PR does / why we need it:

Addresses Issue 275

Checklist

Which issue(s) this PR fixes:

Fixes #<275>

Special notes for your reviewer:

valeriano-manassero commented 7 months ago

Checks looks good but there's still missing annotations as required in point 5 of 6

savitha-qs commented 7 months ago

@valeriano-manassero I've updated the annotation now. Let me know if it looks good. Thanks!

savitha-qs commented 7 months ago

@valeriano-manassero If this looks good to you, would you mind approving the PR? Or, feel free to merge it. Would appreciate it, thanks!