Overhaul helm#490
Conversation
* k3s-simplifications * missed some key updates (replace hostnames) and more * rework and final run through * remove service name, it's not used anymore, it's set directly in ingress.yaml * restore original Chart.yaml & Chart.lock * remove DO specific annotations from base chart, fix global so they override * add certificateDomain * add dependency to base metrics-api * adjust/indent values for metrics-api dependency * missing Chart.lock * fix indentation * moved postgres and auth values from secret to deployment.yaml --------- Co-authored-by: Dario A Lencina-Talarico <darioalessandrolencina@gmail.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| @@ -0,0 +1,21 @@ | |||
| piVersion: cert-manager.io/v1 | |||
There was a problem hiding this comment.
Bug: YAML apiVersion field misspelled causing invalid manifest
The Kubernetes apiVersion field is misspelled as piVersion on line 1. This typo (missing the leading a) will cause kubectl apply to reject this manifest as invalid, preventing the Route53 cert-manager Issuer from being created. Certificate provisioning for deployments using AWS Route53 DNS01 challenge will fail.
jboyd01
left a comment
jboyd01
left a comment
There was a problem hiding this comment.
Wow, really big change! Just a few issues when I tried it out, see #531
My install was pretty smooth, a couple issues which took a bit to gain the proper context due to the large chart, but I think this is a good direction for simplification. Only one comment outside the PR - I don't understand the art in notes.txt :)
| | | | | | | |/ _ \| '_ \ / _ \ | '__| / _ \/ _` | || | | | | | ||
| | |_| | |___ | | (_) | |_) || __/ | | | __/ (_| | || | |_| | | ||
| |____/ \____||_|\___/|_.__/ \___| |_| \___|\__,_|_||_|\__, | | ||
| |___/ |
There was a problem hiding this comment.
I'm having a hard time figuring this one out. I'm sure it's simple, but all I see here is "DClobe really" and I can't make sense of it ;)
* tweaks necessary to make overhauled charts work * fix typo on
…d-stale-mic-speaker-list-in-meeting-settings-modal fix: avoid stale mic/speaker list in in-meeting settings modal
2 participants
Note
Introduce a unified Videocall Helm chart with componentized services and add region/infrastructure deployment charts, docs, and cleanup of legacy charts.
helm/videocall/): New chart deployingui,websocket,webtransport,website,metricsClient, andmetricsServerwith component-scoped selectors, services, HPAs, ingress, and TLS certs.README.md,QUICKSTART.md,MIGRATION.md,CHART_SUMMARY.md, and aK3S_DEPLOYMENT_GUIDE.md.helm-videocall-deployment/):nats,prometheus,grafana(with dashboards),postgres,ingress-nginx,external-dns, andcert-managerissuers (Cloudflare, DigitalOcean, Route53).us-east/andsingapore/with values forwebsocket,webtransport, monitoring, and website; plus appengineering-vlog.rustlemania-ui,rustlemania-websocket,rustlemania-webtransport,videocall-website, legacy metrics-api templates, and old postgres chart artifacts).webtransport/websocketvalues.Written by Cursor Bugbot for commit 658412f. This will update automatically on new commits. Configure here.