Repository: honeydipper/honeydipper
Pull request #343 (2020-11-14 20:57:52)

chore: clean up unused charts definition
#### Description
The charts are now managed in [honeydipper-charts](https://github.com/honeydipper/honeydipper-charts) repo.

#### This PR fixes the following issues


wyardley: Can we move it the other direction instead?
Charles546: > Can we move it the other direction instead?

I want the chart version and app version to be disassociated. So, the app binary and the deployment template(chart) should go through different CI piplines and released with different cadence. I am not sure if coupling them into one single repo is the right approach. 

The `.Chart.AppVersion` acts as a default but doesn't limit the usage of the chart with other version of the app binary. This makes it prone to error with outdated default version.  One option is not using `.Chart.AppVersion` at all(removing it from the `Chart.yaml`). So, the `.Values.image.tag` becomes required. And, we don't have to keep updating the `AppVersion` in the chart. 

The whole helm community seems undecided how to handle this either.
https://github.com/helm/helm/issues/8194
wyardley: Looks good. Makes sense about keeping it in the same pipeline, as long as we integrate ct etc., I think also good to have the artifact get pushed to the git release if we can, vs. committed to git.

Re-running the flaky test

Patch:
diff --git a/deployments/helm/honeydipper/.helmignore b/deployments/helm/honeydipper/.helmignore
deleted file mode 100644
index f0c1319..0000000
--- a/deployments/helm/honeydipper/.helmignore
+++ /dev/null
@@ -1,21 +0,0 @@
-# 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
diff --git a/deployments/helm/honeydipper/Chart.yaml b/deployments/helm/honeydipper/Chart.yaml
deleted file mode 100644
index 8eb2d0b..0000000
--- a/deployments/helm/honeydipper/Chart.yaml
+++ /dev/null
@@ -1,6 +0,0 @@
-apiVersion: v1
-appVersion: "latest"
-description: An event driven orchestration system
-name: honeydipper
-version: 0.1.2
-icon: "https://raw.githubusercontent.com/honeydipper/honeydipper/master/logo/log_medium.png"
diff --git a/deployments/helm/honeydipper/README.md b/deployments/helm/honeydipper/README.md
deleted file mode 100644
index 4e09966..0000000
--- a/deployments/helm/honeydipper/README.md
+++ /dev/null
@@ -1,105 +0,0 @@
-# Honeydipper
-
-[Honeydipper](https://github.com/honeydipper/honeydipper) is basically, your swiss army knife for systems engineering and operations. An event-driven, policy-based orchestration system, with a pluggable open architecture.
-
-## Introduction
-
-This chart creates a deployment to run the Honeydipper daemon, and exposes a service pointing to the webhook listener.
-
-## Prerequisites
-
- * Kubernetes 1.10+
-
-## Required Configuration
-
-Honeydipper daemon loads most of the configurations from git repos. To start, a few environment variables are required to bootstrap repo. Below are the settings required.
-
- * daemon.env.REPO, defaults to `https://github.com/honeydipper/honeydipper-config-essentials.git`
-
-Please refer to [values.yaml](./values.yaml) for detailed example.
-
-## Customization
-
-The following options are supported . See [values.yaml](./values.yaml) for more detailed explanation and examples:
-
-| Parameter | Description | Default |
-| --------- | ----------- | ------- |
-| daemon.useSource | Instead of use prebuilt docker image, compile and run from source code | `false` |
-| daemon.useBranch | If using source code, specify the branch/tag to use | `master` |
-| daemon.sourceRepo | If using source code, specify alternative repo to use, should be a fork of official source code | `https://github.com/honeydipper/honeydipper.git` |
-| daemon.gitIdentityFilePath | If cloning source code from private repo using ssh protocol, use this to specify path to the id_rsa file, not including the file name | `/root` |
-| daemon.githubHostKey | If cloning source code from private repo using ssh protocol, use this to specify the github host key to avoid git ssh failure | current github host key |
-| daemon.spec | Additional pod spec settings for daemon deploy, can be anything other than `containers`, `nodeSelector`, `affinity`, `tolerations` or `volumes` | |
-| daemon.replicaCount | The number of replicas of daemons to start | 1 |
-| daemon.image.repository | The docker image for Honeydipper daemon | `honeydipper/honeydipper` |
-| daemon.image.tag | The version tag of the docker image to use | `latest` |
-| daemon.image.pullPolicy | Pull policy for the docker image | `Always` |
-| daemon.args | Honeydipper daemon args, should be a list of Honeydipper daemon services, empty means all | |
-| daemon.resources | The pod resource spec, cpu limit, memory limit, etc., as defined in a pod sped | |
-| daemon.nodeSelector | A map of node selectors for the pod, as defined in a pod sped | |
-| daemon.affinity | A map of affinity settings for the pod, as defined in a pod sped | |
-| daemon.tolerations | A list of toleration settings for the pod, as defined in a pod sped | |
-| daemon.volumes | A map of volumes for the pod, from volume name to its mount point and a `spec` defining the volume | |
-| daemon.env | A map of environment variables from name to its value, the value is either a string or a data structure used for `valueFrom` as defined in a pod spec | |
-| drivers.webhook.service.type | The exposed service type for the webhook | `LoadBalancer` |
-| drivers.webhook.service.port | The exposed service port for the webhook, needs to match the driver configurations set in the configuration repo | 8080 |
-| drivers.webhook.service.nodePort | The exposed service node port for the webhook. If set to 0, Kubernetes will assign a random port. | 0 |
-| drivers.webhook.ingress.enabled | Use ingress controller for the webhook service | `false` |
-| drivers.webhook.ingress.annotations | If using ingress controllers, specify a map of anotations | |
-| drivers.webhook.ingress.path | If using ingress controllers, specify the path mapped to webhook | |
-| drivers.webhook.ingress.hosts | If using ingress controllers, specify a list of host names for the webhook | |
-| drivers.webhook.ingress.tls | If using ingress controllers, specify the secret that defines the key and certs | |
-| drivers.redis.local | Use redis sidecar container for eventbus, this is only useful for testing. In production, multiple replicas of daemon should share the redis server | true |
-
-### Specifying environment
-
-Using command line:
-
-```helm
-helm --set daemon.env.REPO=git@github.com:example/example.git,daemon.env.BOOTSTRAP_PATH=/submodule,daemon.env.BRANCH=test_branch install test incubator/honeydipper
-```
-
-In values file
-
-```yaml
-daemon:
-  env:
-    REPO: git@github.com:example/example.git
-    BOOTSTRAP_PATH: /submodule
-    BRANCH: test_branch
-    SECRET_VARIABLE:
-      secretRef:
-        name: my-secret
-        key: my-key
-```
-
-### Mounting volumes
-
-Using values file
-
-```yaml
-daemon:
-  volumes:
-    my-config:
-      mountPath: /etc/myconfig/file
-      subPath: file
-      spec:
-        configMap:
-          name: my-config
-```
-
-## Testing the Deployment
-
-To perform a sanity test (i.e. ensure Honeydipper daemon is running, and when you curl the webhook url `http(s)://your-service-url/health`, you should get a `200` response code.)
-
- 1. Install the chart
-
-```bash
-helm install -f test-values.yaml --name trial incubator/honeydipper --debug
-```
-
- 2. Run the tests
-
-```bash
-helm test trial
-```
diff --git a/deployments/helm/honeydipper/templates/NOTES.txt b/deployments/helm/honeydipper/templates/NOTES.txt
deleted file mode 100644
index 477862a..0000000
--- a/deployments/helm/honeydipper/templates/NOTES.txt
+++ /dev/null
@@ -1,19 +0,0 @@
-1. Get the application URL by running these commands:
-{{- if .Values.drivers.webhook.ingress.enabled }}
-{{- range .Values.drivers.webhook.ingress.hosts }}
-  http{{ if $.Values.drivers.webhook.ingress.tls }}s{{ end }}://{{ . }}{{ $.Values.drivers.webhook.ingress.path }}
-{{- end }}
-{{- else if contains "NodePort" .Values.drivers.webhook.service.type }}
-  export NODE_PORT=$(kubectl get --namespace {{ .Release.Namespace }} -o jsonpath="{.spec.ports[0].nodePort}" services {{ include "honeydipper.fullname" . }})
-  export NODE_IP=$(kubectl get nodes --namespace {{ .Release.Namespace }} -o jsonpath="{.items[0].status.addresses[0].address}")
-  echo http://$NODE_IP:$NODE_PORT
-{{- else if contains "LoadBalancer" .Values.drivers.webhook.service.type }}
-     NOTE: It may take a few minutes for the LoadBalancer IP to be available.
-           You can watch the status of by running 'kubectl get svc -w {{ include "honeydipper.fullname" . }}'
-  export SERVICE_IP=$(kubectl get svc --namespace {{ .Release.Namespace }} {{ include "honeydipper.fullname" . }} -o jsonpath='{.status.loadBalancer.ingress[0].ip}')
-  echo http://$SERVICE_IP:{{ .Values.drivers.webhook.service.port }}
-{{- else if contains "ClusterIP" .Values.drivers.webhook.service.type }}
-  export POD_NAME=$(kubectl get pods --namespace {{ .Release.Namespace }} -l "app.kubernetes.io/name={{ include "honeydipper.name" . }},app.kubernetes.io/instance={{ .Release.Name }}" -o jsonpath="{.items[0].metadata.name}")
-  echo "Visit http://127.0.0.1:8080 to use your application"
-  kubectl port-forward $POD_NAME 8080:8080
-{{- end }}
diff --git a/deployments/helm/honeydipper/templates/_helpers.tpl b/deployments/helm/honeydipper/templates/_helpers.tpl
deleted file mode 100644
index da40fc7..0000000
--- a/deployments/helm/honeydipper/templates/_helpers.tpl
+++ /dev/null
@@ -1,32 +0,0 @@
-{{/* vim: set filetype=mustache: */}}
-{{/*
-Expand the name of the chart.
-*/}}
-{{- define "honeydipper.name" -}}
-{{- default .Chart.Name .Values.nameOverride | trunc 63 | trimSuffix "-" -}}
-{{- end -}}
-
-{{/*
-Create a default fully qualified app name.
-We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).
-If release name contains chart name it will be used as a full name.
-*/}}
-{{- define "honeydipper.fullname" -}}
-{{- if .Values.fullnameOverride -}}
-{{- .Values.fullnameOverride | trunc 63 | trimSuffix "-" -}}
-{{- else -}}
-{{- $name := default .Chart.Name .Values.nameOverride -}}
-{{- if contains $name .Release.Name -}}
-{{- .Release.Name | trunc 63 | trimSuffix "-" -}}
-{{- else -}}
-{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
-{{- end -}}
-{{- end -}}
-{{- end -}}
-
-{{/*
-Create chart name and version as used by the chart label.
-*/}}
-{{- define "honeydipper.chart" -}}
-{{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" -}}
-{{- end -}}
diff --git a/deployments/helm/honeydipper/templates/daemon-deployment.yaml b/deployments/helm/honeydipper/templates/daemon-deployment.yaml
deleted file mode 100644
index 1b64035..0000000
--- a/deployments/helm/honeydipper/templates/daemon-deployment.yaml
+++ /dev/null
@@ -1,115 +0,0 @@
-apiVersion: apps/v1beta2
-kind: Deployment
-metadata:
-  name: {{ include "honeydipper.fullname" . }}
-  labels:
-    app.kubernetes.io/name: {{ include "honeydipper.name" . }}
-    helm.sh/chart: {{ include "honeydipper.chart" . }}
-    app.kubernetes.io/instance: {{ .Release.Name }}
-    app.kubernetes.io/managed-by: {{ .Release.Service }}
-spec:
-  replicas: {{ .Values.daemon.replicaCount }}
-  selector:
-    matchLabels:
-      app.kubernetes.io/name: {{ include "honeydipper.name" . }}
-      app.kubernetes.io/instance: {{ .Release.Name }}
-  template:
-    metadata:
-      labels:
-        app.kubernetes.io/name: {{ include "honeydipper.name" . }}
-        app.kubernetes.io/instance: {{ .Release.Name }}
-    spec:
-      {{- with .Values.daemon.spec }}
-      {{ toYaml . | indent 6 | trim }}
-      {{- end }}
-      containers:
-        {{- if .Values.drivers.redis.local }}
-        - name: {{ .Chart.Name }}-redis
-          image: "redis:latest"
-        {{- end }}
-        - name: {{ .Chart.Name }}
-          {{- if .Values.daemon.useSource }}
-          image: "golang:alpine"
-          command:
-            - "/bin/sh"
-            - "-c"
-            - |
-              apk add openssh git &&
-              go get -u github.com/golang/dep/cmd/dep &&
-              mkdir -p ~/.ssh &&
-              chmod 700 ~/.ssh &&
-              {{- if .Values.daemon.gitIdentityFilePath }}
-              echo $'Host github.com\n\tIdentityFile\t'{{ .Values.daemon.gitIdentityFilePath }}/id_rsa > ~/.ssh/config &&
-              {{- end }}
-              {{- if .Values.daemon.gitHostKey }}
-              echo 'github.com ssh-rsa {{ .Values.daemon.githubHostKey }}' > ~/.ssh/known_hosts &&
-              {{- end }}
-              mkdir -p /go/src/github.com/honeydipper &&
-              cd /go/src/github.com/honeydipper &&
-              git clone {{ .Values.daemon.sourceRepo }} &&
-              cd honeydipper &&
-              git checkout {{ .Values.daemon.useBranch }} &&
-              dep ensure &&
-              go install ./... &&
-              honeydipper {{ range $k, $v := .Values.daemon.args }}{{ quote $v }}{{ end }} ||
-              sleep 3600
-          {{- else }}
-          image: "{{ .Values.daemon.image.repository }}:{{ .Values.daemon.image.tag }}"
-          {{- end }}
-          imagePullPolicy: {{ .Values.daemon.image.pullPolicy }}
-          {{- with .Values.daemon.args }}
-          args:
-            {{ toYaml . | indent 12 | trim }}
-          {{- end }}
-          {{- with .Values.daemon.env }}
-          env:
-            {{- range $envName, $envValue := . }}
-            - name: {{ $envName }}
-              {{- if kindIs "string" $envValue }}
-              value: {{ $envValue }}
-              {{- else }}
-              valueFrom:
-                {{ toYaml $envValue | indent 16 | trim }}
-              {{- end }}
-            {{- end }}
-          {{- end }}
-          {{- with .Values.daemon.resources }}
-          resources:
-            {{ toYaml .Values.daemon.resources | indent 12 | trim }}
-          {{- end }}
-          {{- with .Values.daemon.volumes }}
-          volumeMounts:
-            {{- range $vName, $vMount := . }}
-            - name: {{ $vName }}
-              {{ omit $vMount "spec" | toYaml | indent 14 | trim }}
-            {{- end }}
-          {{- end }}
-          readinessProbe:
-            tcpSocket:
-              port: {{ .Values.drivers.webhook.service.port }}
-            initialDelaySeconds: 20
-            periodSeconds: 10
-          livenessProbe:
-            tcpSocket:
-              port: {{ .Values.drivers.webhook.service.port }}
-            initialDelaySeconds: 20
-            periodSeconds: 20
-      {{- with .Values.daemon.nodeSelector }}
-      nodeSelector:
-        {{ toYaml . | indent 8 | trim }}
-      {{- end }}
-      {{- with .Values.daemon.affinity }}
-      affinity:
-        {{ toYaml . | indent 8 | trim }}
-      {{- end }}
-      {{- with .Values.daemon.tolerations }}
-      tolerations:
-        {{ toYaml . | indent 8 | trim }}
-      {{- end }}
-      {{- with .Values.daemon.volumes }}
-      volumes:
-        {{- range $vName, $vMount := . }}
-        - name: {{ $vName }}
-          {{ index $vMount "spec" | toYaml | indent 10 | trim }}
-        {{- end }}
-      {{- end }}
diff --git a/deployments/helm/honeydipper/templates/webhook-ingress.yaml b/deployments/helm/honeydipper/templates/webhook-ingress.yaml
deleted file mode 100644
index 1917fd3..0000000
--- a/deployments/helm/honeydipper/templates/webhook-ingress.yaml
+++ /dev/null
@@ -1,38 +0,0 @@
-{{- if .Values.drivers.webhook.ingress.enabled -}}
-{{- $fullName := include "honeydipper.fullname" . -}}
-{{- $ingressPath := .Values.drivers.webhook.ingress.path -}}
-apiVersion: extensions/v1beta1
-kind: Ingress
-metadata:
-  name: {{ $fullName }}
-  labels:
-    app.kubernetes.io/name: {{ include "honeydipper.name" . }}
-    helm.sh/chart: {{ include "honeydipper.chart" . }}
-    app.kubernetes.io/instance: {{ .Release.Name }}
-    app.kubernetes.io/managed-by: {{ .Release.Service }}
-{{- with .Values.drivers.webhook.ingress.annotations }}
-  annotations:
-{{ toYaml . | indent 4 }}
-{{- end }}
-spec:
-{{- if .Values.drivers.webhook.ingress.tls }}
-  tls:
-  {{- range .Values.drivers.webhook.ingress.tls }}
-    - hosts:
-      {{- range .hosts }}
-        - {{ . | quote }}
-      {{- end }}
-      secretName: {{ .secretName }}
-  {{- end }}
-{{- end }}
-  rules:
-  {{- range .Values.drivers.webhook.ingress.hosts }}
-    - host: {{ . | quote }}
-      http:
-        paths:
-          - path: {{ $ingressPath }}
-            backend:
-              serviceName: {{ $fullName }}
-              servicePort: http
-  {{- end }}
-{{- end }}
diff --git a/deployments/helm/honeydipper/templates/webhook-service.yaml b/deployments/helm/honeydipper/templates/webhook-service.yaml
deleted file mode 100644
index c0857b7..0000000
--- a/deployments/helm/honeydipper/templates/webhook-service.yaml
+++ /dev/null
@@ -1,24 +0,0 @@
-{{- if or (has "receiver" .Values.daemon.args) (not .Values.daemon.args) -}}
-apiVersion: v1
-kind: Service
-metadata:
-  name: {{ include "honeydipper.fullname" . }}
-  labels:
-    app.kubernetes.io/name: {{ include "honeydipper.name" . }}
-    helm.sh/chart: {{ include "honeydipper.chart" . }}
-    app.kubernetes.io/instance: {{ .Release.Name }}
-    app.kubernetes.io/managed-by: {{ .Release.Service }}
-spec:
-  type: {{ .Values.drivers.webhook.service.type }}
-  ports:
-    - port: {{ .Values.drivers.webhook.service.port }}
-      targetPort: {{ .Values.drivers.webhook.service.port }}
-      {{- if .Values.drivers.webhook.service.nodePort }}
-      nodePort: {{ .Values.drivers.webhook.service.nodePort }}
-      {{- end }}
-      protocol: TCP
-      name: webhook
-  selector:
-    app.kubernetes.io/name: {{ include "honeydipper.name" . }}
-    app.kubernetes.io/instance: {{ .Release.Name }}
-{{- end -}}
diff --git a/deployments/helm/honeydipper/values.yaml b/deployments/helm/honeydipper/values.yaml
deleted file mode 100644
index 986221d..0000000
--- a/deployments/helm/honeydipper/values.yaml
+++ /dev/null
@@ -1,103 +0,0 @@
-# Default values for honeydipper.
-# This is a YAML-formatted file.
-# Declare variables to be passed into your templates.
-
-# nameOverride is used to override the chart name part of the release name
-nameOverride: ""
-# fullnameOverride is used to override the release name
-fullnameOverride: ""
-
-# daemon is used to specify the settings for the daemon
-daemon:
-  # useSource true means building the daemon using source code instead of prebuilt image
-  useSource: false
-  # useBranch specifies which branch of source code to use
-  useBranch: "master"
-  # sourceRepo specifies the repo to download source code from, could be a clone of original https://github.com/honeydipper/honeydipper.git
-  sourceRepo: "https://github.com/honeydipper/honeydipper.git"
-
-  # gitIdentityFilePath points to the ssh key to use for cloning source code from private repo
-  gitIdentityFilePath: "/root"
-  # githubHostKey is the github.com host key
-  githubHostKey: "AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ=="
-
-  # spec is used to create the container spec in the manifest file
-  spec: {}
-
-  # replicaCount specifies how many replica daemon should be running
-  replicaCount: 1
-
-  # image specifies what image to use to start the daemon
-  image:
-    # repository is the image repository for the daemon container
-    repository: honeydipper/honeydipper
-    # tag is the version of the docker image to use
-    tag: "latest"
-    # pullPolicy specify when to refresh the image
-    pullPolicy: Always
-
-  # args are the arguments passed to the daemon, a list of services to run in the daemon
-  args: []
-
-  # resources specifies the resource limitation for the daemon container
-  resources: {}
-    # We usually recommend not to specify default resources and to leave this as a conscious
-    # choice for the user. This also increases chances charts run on environments with little
-    # resources, such as Minikube. If you do want to specify resources, uncomment the following
-    # lines, adjust them as necessary, and remove the curly braces after 'resources:'.
-    # limits:
-    #  cpu: 100m
-    #  memory: 128Mi
-    # requests:
-    #  cpu: 100m
-    #  memory: 128Mi
-
-  # nodeSelector is used for specifying what nodes to place the daemon containers on
-  nodeSelector: {}
-
-  # tolerations is used in the daemon pod speicification
-  tolerations: []
-
-  # tolerations is used in the daemon pod speicification
-  affinity: {}
-
-  # env is a list of environment variables for the daemon container
-  env:
-    REPO: https://github.com/honeydipper/honeydipper-config-essentials
-
-  # volumes is a list of volumes to be mounted on the daemon container
-  volumes: {}
-
-# drivers containers the settings for the drivers
-drivers:
-  # redis settings for the redis related drivers
-  redis:
-    # local means run a local redis side car container, only useful for testing
-    local: true
-  # webhook settings for the webhook driver
-  webhook:
-    # service specifies how the webhook driver service is exposed
-    service:
-      # type is the type of kubernetes service for the webhook
-      type: LoadBalancer
-      port: 8080
-      nodePort: 0
-
-    # ingress controller settings
-    ingress:
-      # enabled is used for creating an ingress kubernetes resouce
-      enabled: false
-      # annotations are the annotations for the  ingress resource
-      annotations: {}
-        # kubernetes.io/ingress.class: nginx
-        # kubernetes.io/tls-acme: "true"
-      # path specify the path of the ingress
-      path: /
-      # hosts specify the hosts of the ingress
-      hosts:
-        - chart-example.local
-      # tls is used for specifying the ssl cert using secret
-      tls: []
-      #  - secretName: chart-example-tls
-      #    hosts:
-      #      - chart-example.local


Test patch:
diff --git a/deployments/helm/honeydipper/templates/tests/test-webhook-service.yaml b/deployments/helm/honeydipper/templates/tests/test-webhook-service.yaml
deleted file mode 100644
index 925c9fc..0000000
--- a/deployments/helm/honeydipper/templates/tests/test-webhook-service.yaml
+++ /dev/null
@@ -1,16 +0,0 @@
-apiVersion: v1
-kind: Pod
-metadata:
-  name: "{{ .Release.Name }}-webhook-test"
-  annotations:
-    "helm.sh/hook": test-success
-spec:
-  containers:
-  - name: {{ .Release.Name }}-webhook-test
-    image: alpine:3.8
-    command:
-      - "sh"
-      - "-c"
-      - |
-        wget -nv http://{{ include "honeydipper.fullname" . }}.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.drivers.webhook.service.port }}/health
-  restartPolicy: Never
diff --git a/deployments/helm/honeydipper/test-values.yaml b/deployments/helm/honeydipper/test-values.yaml
deleted file mode 100644
index dfcbd83..0000000
--- a/deployments/helm/honeydipper/test-values.yaml
+++ /dev/null
@@ -1,4 +0,0 @@
-daemon:
-  env:
-    REPO: https://github.com/honeydipper/honeydipper
-    BOOTSTRAP_PATH: /deployments/helm/honeydipper/test_fixtures
diff --git a/deployments/helm/honeydipper/test_fixtures/init.yaml b/deployments/helm/honeydipper/test_fixtures/init.yaml
deleted file mode 100644
index 5ec812c..0000000
--- a/deployments/helm/honeydipper/test_fixtures/init.yaml
+++ /dev/null
@@ -1,11 +0,0 @@
----
-repos:
-  - repo: https://github.com/honeydipper/honeydipper-config-essentials
-
-rules:
-  - when:
-      driver: webhook
-      conditions:
-        url: "/health"
-    do:
-      content: noop