change default settings & update notes

[acelinkio]

What type of PR is this?
cleanup

Which issue does this PR fix:
no issue

What does this PR do / Why do we need it:
Updates the default values.

• image.tag should not be set by default.
• service.type was NodePort. Best practice is to use ClusterIP. NodePort can be accessed externally and does not follow the principle of least privilege as a default.
• ingress.className should not be set by default. By unsetting this value, Kubernetes will use the default class specified in the cluster.
• image.repository used the amd64 specific image, switched to using generic image repository location

with some small edits to comments

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[rchincha]

@acelinkio just curious what necessitated this PR? in your particular case?

Pls also fix the ci failures.

[acelinkio]

Looked for references from people in Homelab community via https://kubesearch.dev . everyone opted to not using this helm chart.

I attempted to use this chart but quickly realized I was better off just creating these manifests manually.

[rchincha]

@acelinkio pls fix the ci failures. We will review and discuss this shortly.