Skip to content

Commit 84a3296

Browse files
committed
fixes from Garys PR comments
1 parent 023648e commit 84a3296

File tree

6 files changed

+10
-10
lines changed

6 files changed

+10
-10
lines changed
File renamed without changes.

Charts/fastcs/.schema.config.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ useHelmDocs: false
1717
noAdditionalProperties: false
1818

1919
schemaRoot:
20-
title: FastCS Helm chart schema
21-
description: Schema to allow validation of values passed to FastCS Helm chart
20+
title: FastCS Helm chart
21+
description: Helm chart for deploying a FastCS application
2222
# No additional properties in schema root for tighter protection:
2323
additionalProperties: false
2424

Charts/fastcs/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
apiVersion: v2
22
name: fastcs
3-
description: A Helm chart for Kubernetes
3+
description: A Helm chart for FastCS Applications
44

55
# A chart can be either an 'application' or a 'library' chart.
66
#

Charts/fastcs/values.schema.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"$schema": "https://json-schema.org/draft/2020-12/schema",
3-
"title": "FastCS Helm chart schema",
4-
"description": "Schema to allow validation of values passed to FastCS Helm chart",
3+
"title": "FastCS Helm chart",
4+
"description": "Helm chart for deploying a FastCS application",
55
"type": "object",
66
"properties": {
77
"affinity": {

Charts/fastcs/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ initCommand:
3535
- /bin/bash
3636
- -c
3737
- |
38-
echo "running as account"; id
38+
echo "Running as account"; id
3939
if [ -d /dest/venv ]; then
4040
echo "Virtual environment already exists, skipping copy"
4141
else

Dockerfile

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
88
graphviz \
99
&& rm -rf /var/lib/apt/lists/*
1010

11-
# Install helm for the dev container. This is the recommended
11+
# Install helm for the dev container. This is the recommended
1212
# approach per the docs: https://helm.sh/docs/intro/install
13-
RUN curl -fsSL -o get_helm.sh https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3; \
14-
chmod 700 get_helm.sh; \
15-
./get_helm.sh; \
13+
RUN curl -fsSL -o get_helm.sh https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3 && \
14+
chmod 700 get_helm.sh && \
15+
./get_helm.sh && \
1616
rm get_helm.sh
1717
RUN helm plugin install https://github.com/losisin/helm-values-schema-json.git
1818

0 commit comments

Comments
 (0)