Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Commit

Permalink
Fix bug where service name and namespace name were transposed (#71)
Browse files Browse the repository at this point in the history
also bump version of helm chart
  • Loading branch information
andyspiers committed Jun 8, 2020
1 parent faeb1b5 commit fea84a1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion deploy/helm/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ apiVersion: v1
appVersion: "1.0"
description: A Helm chart for Kubernetes
name: k-rail
version: 0.1.0
version: 0.1.1
14 changes: 7 additions & 7 deletions deploy/helm/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,15 +32,15 @@ webhooks:
- name: k-rail.cruise-automation.github.com
clientConfig:
service:
namespace: k-rail
name: {{ .Release.Namespace }}
namespace: {{ .Release.Namespace }}
name: k-rail
path: "/"
caBundle: {{ b64enc $ca.Cert }}
rules:
- operations: ["CREATE","UPDATE","CONNECT"]
apiGroups: ["*"]
apiVersions: ["*"]
resources:
resources:
- pods
- deployments
- replicationcontrollers
Expand All @@ -54,14 +54,14 @@ webhooks:
- persistentvolumes
failurePolicy: {{ print .Values.failurePolicy }}
{{- if .Values.reinvocationPolicy }}
reinvocationPolicy: {{ print .Values.reinvocationPolicy }}
reinvocationPolicy: {{ print .Values.reinvocationPolicy }}
{{- end }}
sideEffects: None
namespaceSelector:
matchExpressions:
- key: name
operator: NotIn
values:
values:
- {{ .Release.Namespace }}
---
apiVersion: apps/v1
Expand All @@ -80,7 +80,7 @@ spec:
metadata:
name: k-rail
labels:
name: k-rail
name: k-rail
annotations:
checksum/config: {{ print .Values.config | sha256sum }}
checksum/exemptions: {{ print .Values.exemptions | sha256sum }}
Expand Down Expand Up @@ -218,4 +218,4 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: k-rail
namespace: {{ .Release.Namespace }}
namespace: {{ .Release.Namespace }}

0 comments on commit fea84a1

Please sign in to comment.