[samba] New chart (#477)

* New chart

* trigger workflow

* Update README.md

* Update common.yaml

* Use default service name

* Add htsp port

* bump

README.md

@@ -1,9 +1,11 @@
 # k8s@Home collection of helm charts
 
-[![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
+[![Discord](https://img.shields.io/badge/discord-chat-7289DA.svg)](https://discord.gg/stmx7vh)
 [![](https://github.com/k8s-at-home/charts/workflows/Release%20Charts/badge.svg?branch=master)](https://github.com/k8s-at-home/charts/actions)
 [![pre-commit](https://img.shields.io/badge/pre--commit-enabled-brightgreen?logo=pre-commit&logoColor=white)](https://github.com/pre-commit/pre-commit)
-[![Artifact HUB](https://img.shields.io/endpoint?url=https://artifacthub.io/badge/repository/k8s-at-home)](https://artifacthub.io/packages/search?repo=k8s-at-home)
+[![Artifact Hub](https://img.shields.io/endpoint?url=https://artifacthub.io/badge/repository/k8s-at-home)](https://artifacthub.io/packages/search?repo=k8s-at-home)
+[![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
+
 ## Usage
 
 [Helm](https://helm.sh) must be installed to use the charts.
@@ -19,7 +21,18 @@ You can then run `helm search repo k8s-at-home` to see the charts.
 
 ## Charts
 
-See [artifact hub](https://artifacthub.io/packages/search?org=k8s-at-home) for a complete list.
+See [Artifact Hub](https://artifacthub.io/packages/search?org=k8s-at-home) or [charts](./charts/) for a complete list.
+
+## Common Library
+
+The majority of the charts in this repository are using a common library we built. The reason we went with this pattern is a lot of applications are pretty similar in their requirements for Helm charts. Using a library makes maintaining much easier and keeps charts consistent in features. If contributing a new chart first try to make it using the library we built.
+
+More information can be found [here](https://github.com/k8s-at-home/charts/tree/master/charts/common).
+
+## Support
+
+- Having problems with a chart or have a feature request? Open a new [issue](https://github.com/k8s-at-home/charts/issues/new/choose).
+- New idea? Discuss [here](https://github.com/k8s-at-home/charts/discussions) or in our [Discord](https://discord.gg/sTMX7Vh) in the `#charts` channel.
 
 ## Contributing
 

charts/samba/.helmignore

@@ -0,0 +1,24 @@
+# Patterns to ignore when building packages.
+# This supports shell glob matching, relative path matching, and
+# negation (prefixed with !). Only one pattern per line.
+.DS_Store
+# Common VCS dirs
+.git/
+.gitignore
+.bzr/
+.bzrignore
+.hg/
+.hgignore
+.svn/
+# Common backup files
+*.swp
+*.bak
+*.tmp
+*~
+# Various IDEs
+.project
+.idea/
+*.tmproj
+.vscode
+# OWNERS file for Kubernetes
+OWNERS

charts/samba/Chart.yaml

@@ -0,0 +1,18 @@
+apiVersion: v2
+appVersion: latest
+description: A simple in-cluster Samba server
+name: samba
+version: 1.0.0
+keywords:
+  - samba
+home: https://github.com/k8s-at-home/charts/tree/master/charts/samba
+icon: https://upload.wikimedia.org/wikipedia/commons/thumb/d/db/Samba_logo_2010.svg/500px-Samba_logo_2010.svg.png
+sources:
+  - https://github.com/dperson/samba
+maintainers:
+  - name: billimek
+    email: jeff@billimek.com
+dependencies:
+  - name: common
+    repository: https://k8s-at-home.com/charts/
+    version: 2.2.0

charts/samba/OWNERS

@@ -0,0 +1,8 @@
+approvers:
+- billimek
+- onedr0p
+- bjw-s
+reviewers:
+- billimek
+- onedr0p
+- bjw-s

charts/samba/README.md

@@ -0,0 +1,67 @@
+# samba
+
+This is a helm chart for a [samba](https://github.com/dperson/samba) server.
+
+**This chart is not maintained by the upstream project and any issues with the chart should be raised [here](https://github.com/k8s-at-home/charts/issues/new/choose)**
+
+## TL;DR;
+
+```shell
+$ helm repo add k8s-at-home https://k8s-at-home.com/charts/
+$ helm install k8s-at-home/samba
+```
+
+## Installing the Chart
+
+To install the chart with the release name `my-release`:
+
+```console
+helm install --name my-release k8s-at-home/samba
+```
+
+## Uninstalling the Chart
+
+To uninstall/delete the `my-release` deployment:
+
+```console
+helm delete my-release --purge
+```
+
+The command removes all the Kubernetes components associated with the chart and deletes the release.
+
+## Configuration
+Read through the charts [values.yaml](https://github.com/k8s-at-home/charts/blob/master/charts/samba/values.yaml)
+file. It has several commented out suggested values.
+Additionally you can take a look at the common library [values.yaml](https://github.com/k8s-at-home/charts/blob/master/charts/common/values.yaml) for more (advanced) configuration options.
+
+Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`. For example,
+```console
+helm install samba \
+  --set env.TZ="America/New_York" \
+    k8s-at-home/samba
+```
+Alternatively, a YAML file that specifies the values for the above parameters can be provided while installing the
+chart. For example,
+```console
+helm install samba k8s-at-home/samba --values values.yaml 
+```
+
+```yaml
+image:
+  tag: ...
+```
+
+---
+**NOTE**
+
+If you get
+```console
+Error: rendered manifests contain a resource that already exists. Unable to continue with install: existing resource conflict: ...`
+```
+it may be because you uninstalled the chart with `skipuninstall` enabled, you need to manually delete the pvc or use `existingClaim`.
+
+---
+
+## Upgrading an existing Release to a new major version
+
+A major chart version change (like 4.0.1 -> 5.0.0) indicates that there is an incompatible breaking change potentially needing manual actions.

charts/samba/templates/NOTES.txt

@@ -0,0 +1 @@
+{{- include "common.notes.defaultNotes" . -}}

charts/samba/templates/common.yaml

@@ -0,0 +1 @@
+{{ include "common.all" . }}

charts/samba/values.yaml

@@ -0,0 +1,43 @@
+# Default values for samba.
+
+image:
+  repository: dperson/samba
+  tag: latest
+  pullPolicy: Always
+
+strategy:
+  type: Recreate
+
+probes:
+  liveness:
+    enabled: false
+  readiness:
+    enabled: false
+
+service:
+  port:
+    name: tcp
+    port: 445
+  additionalPorts:
+    - name: netbios
+      port: 139
+
+## For all available environment variables and configuration options, please see:
+## https://github.com/dperson/samba/blob/master/README.md
+env: {}
+  # TZ: UTC
+  # SHARE1: share1;/share/samba/share1
+  # SHARE2: share2;/share/samba/share2
+
+## Configure the volume mounts for your shared folders here.
+additionalVolumeMounts: []
+# - name: my-hostpath-data
+#   mountPath: /share/samba/share1
+# - name: my-pvc-data
+#   mountPath: /share/samba/share2
+
+## Configure the volumes for your shared folders here.
+additionalVolumes: []
+# - name: my-pvc-data
+#   persistentVolumeClaim:
+#     claimName: my-pvc-name

charts/tvheadend/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: latest
 description: TVheadend - a TV streaming server and digital video recorder
 name: tvheadend
-version: 1.1.0
+version: 1.1.1
 keywords:
   - tv
   - streaming

charts/tvheadend/values.yaml

@@ -18,6 +18,10 @@ env: {}
 service:
   port:
     port: 9981
+# additionalPorts:
+#   port:
+#     port: 9982
+#     name: htsp
 
 persistence:
   config: