Hi @gthomson31, welcome to the community!
This isn’t an n8n-provided helm chart but one created by a fellow community user @Vad1mo (⎈ N8N on Kubernetes with Helm - There is now a Chart for that🎉). Perhaps he can help with this one?
The values themselves seem good to me, in a non-helm deployment you’d also specify database.type for example.