Skip to content

Commit

Permalink
Revert "release: promote apisix v2.x helm chart to default branch (#733
Browse files Browse the repository at this point in the history
…)" (#736)

This reverts commit d75cc71.
  • Loading branch information
nic-6443 authored Mar 20, 2024
1 parent d75cc71 commit a6aa358
Show file tree
Hide file tree
Showing 60 changed files with 4,393 additions and 4,202 deletions.
42 changes: 26 additions & 16 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,40 @@ on:

jobs:

helm_docs:
name: Check the Helm Docs format
runs-on: ubuntu-20.04
steps:
- name: Checkout
uses: actions/checkout@v2
with:
submodules: recursive

- name: Setup Go
uses: actions/setup-go@v3
with:
go-version: '1.19.2'

- name: Run helm-docs
run: |
GOBIN=$PWD GO111MODULE=on go install github.com/norwoodj/helm-docs/cmd/helm-docs@v1.11.0
./helm-docs --chart-search-root=${GITHUB_WORKSPACE}/charts
DIFF=$(git diff ${GITHUB_WORKSPACE}/charts/**/*md)
if [ ! -z "$DIFF" ]; then
echo "Please use helm-docs in your clone, of your fork, of the project, and commit an updated README.md for the chart."
fi
git diff --exit-code
rm -f ./helm-docs
helm:
name: Helm chart
runs-on: ubuntu-20.04
needs: helm_docs
strategy:
matrix:
k8s: ["v1.16.15@sha256:64bac16b83b6adfd04ea3fbcf6c9b5b893277120f2b2cbf9f5fa3e5d4c2260cc", "v1.17.17@sha256:e477ee64df5731aa4ef4deabbafc34e8d9a686b49178f726563598344a3898d5", "v1.18.20@sha256:e3dca5e16116d11363e31639640042a9b1bd2c90f85717a7fc66be34089a8169", "v1.19.16@sha256:81f552397c1e6c1f293f967ecb1344d8857613fb978f963c30e907c32f598467", "v1.20.15@sha256:393bb9096c6c4d723bb17bceb0896407d7db581532d11ea2839c80b28e5d8deb", "v1.21.10@sha256:84709f09756ba4f863769bdcabe5edafc2ada72d3c8c44d6515fc581b66b029c", "v1.22.7@sha256:1dfd72d193bf7da64765fd2f2898f78663b9ba366c2aa74be1fd7498a1873166", "v1.23.4@sha256:0e34f0d0fd448aa2f2819cfd74e99fe5793a6e4938b328f657c8e3f81ee0dfb9"]

steps:

- name: Checkout
uses: actions/checkout@v2
with:
Expand Down Expand Up @@ -69,18 +94,3 @@ jobs:
&& ct install \
--charts charts/apisix'
- name: Setup Go
uses: actions/setup-go@v3
with:
go-version: '1.19.2'

- name: Run helm-docs
run: |
GOBIN=$PWD GO111MODULE=on go install github.com/norwoodj/helm-docs/cmd/helm-docs@v1.11.0
./helm-docs --chart-search-root=${GITHUB_WORKSPACE}/charts
DIFF=$(git diff ${GITHUB_WORKSPACE}/charts/**/*md)
if [ ! -z "$DIFF" ]; then
echo "Please use helm-docs in your clone, of your fork, of the project, and commit a updated README.md for the chart."
fi
git diff --exit-code
rm -f ./helm-docs
5 changes: 0 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,5 @@ Currently, APISIX Ingress Controller automatically manipulates some APISIX resou

| | APISIX | APISIX Ingress | APISIX Dashboard |
| :--------: | :----: | :------------: | :--------------: |
| Chart v2.x | v3.x | v1.x | v3.x |
| Chart v1.x | v3.x | v1.x | v3.x |
| Chart v0.x | v2.x | v1.x | v2.x |

## Changelogs

We have made numerous breaking changes to the APISIX Helm chart between v1.x and v2.x. If you wish to continue using or updating the v1.x, please utilize the `apisix/1.x` branch available in the repository.
Binary file added apisix-0.13.2.tgz
Binary file not shown.
2 changes: 1 addition & 1 deletion charts/apisix-dashboard/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.8.0
version: 0.8.2

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
6 changes: 6 additions & 0 deletions charts/apisix-dashboard/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@ _See [helm upgrade](https://helm.sh/docs/helm/helm_upgrade/) for command documen
| config.authentication.secret | string | `"secret"` | Secret for jwt token generation |
| config.authentication.users | list | `[{"password":"admin","username":"admin"}]` | Specifies username and password for login manager api. |
| config.conf.etcd.endpoints | list | `["apisix-etcd:2379"]` | Supports defining multiple etcd host addresses for an etcd cluster |
| config.conf.etcd.mtls | object | `{}` | |
| config.conf.etcd.mtlsExistingSecret | string | `""` | Specifies a secret to be mounted on /etc/etcd for mtls usage |
| config.conf.etcd.password | string | `nil` | Specifies etcd basic auth password if enable etcd auth |
| config.conf.etcd.prefix | string | `"/apisix"` | apisix configurations prefix |
| config.conf.etcd.username | string | `nil` | Specifies etcd basic auth username if enable etcd auth |
Expand All @@ -71,6 +73,8 @@ _See [helm upgrade](https://helm.sh/docs/helm/helm_upgrade/) for command documen
| config.conf.log.accessLog.filePath | string | `"/dev/stdout"` | Error log path |
| config.conf.log.errorLog | object | `{"filePath":"/dev/stderr","level":"warn"}` | Error log level. Supports levels, lower to higher: debug, info, warn, error, panic, fatal |
| config.conf.log.errorLog.filePath | string | `"/dev/stderr"` | Access log path |
| config.conf.plugins | list | `[]` | Overrides plugins in the APISIX Dashboard conf |
| config.schema.configMap | object | `{}` | Overrides APISIX Dashboard schema.json by mounting configMap containing schema.json |
| fullnameOverride | string | `""` | String to fully override apisix-dashboard.fullname template |
| image.pullPolicy | string | `"IfNotPresent"` | Apache APISIX Dashboard image pull policy |
| image.repository | string | `"apache/apisix-dashboard"` | Apache APISIX Dashboard image repository |
Expand All @@ -96,3 +100,5 @@ _See [helm upgrade](https://helm.sh/docs/helm/helm_upgrade/) for command documen
| serviceAccount.create | bool | `true` | Specifies whether a service account should be created |
| serviceAccount.name | string | `""` | The name of the service account to use. If not set and create is true, a name is generated using the fullname template |
| tolerations | list | `[]` | Tolerations for pod assignment |
| topologySpreadConstraints | list | `[]` | Topology Spread Constraints for pod assignment spread across your cluster among failure-domains ref: https://kubernetes.io/docs/concepts/workloads/pods/pod-topology-spread-constraints/#spread-constraints-for-pods |
| updateStrategy | object | `{}` | Update strategy for apisix dashboard deployment |
10 changes: 10 additions & 0 deletions charts/apisix-dashboard/templates/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,10 @@ data:
{{- if .password }}
password: {{ .password }}
{{- end }}
{{- if .mtls }}
mtls:
{{- toYaml .mtls | nindent 10 }}
{{- end }}
{{- end }}
{{- with .log }}
log:
Expand All @@ -61,3 +65,9 @@ data:
password: {{ .password }}
{{- end }}
{{- end }}
{{- with .Values.config.conf.plugins }}
plugins:
{{- range . }}
- {{ . }}
{{- end }}
{{- end }}
26 changes: 26 additions & 0 deletions charts/apisix-dashboard/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,9 @@ spec:
selector:
matchLabels:
{{- include "apisix-dashboard.selectorLabels" . | nindent 6 }}
{{- with .Values.updateStrategy }}
strategy: {{ toYaml . | nindent 4 }}
{{- end }}
template:
metadata:
annotations:
Expand Down Expand Up @@ -74,10 +77,29 @@ spec:
- mountPath: /usr/local/apisix-dashboard/conf/conf.yaml
name: apisix-dashboard-config
subPath: conf.yaml
{{- with .Values.config.schema.configMap }}
- mountPath: /usr/local/apisix-dashboard/conf/schema.json
name: schema-json
subPath: {{ .key }}
{{- end}}
{{- if .Values.config.conf.etcd.mtlsExistingSecret }}
- mountPath: /etc/etcd
name: etcd-config
{{- end}}
volumes:
- configMap:
name: {{ include "apisix-dashboard.fullname" . }}
name: apisix-dashboard-config
{{- with .Values.config.schema.configMap }}
- configMap:
name: {{ .name }}
name: schema-json
{{- end}}
{{- if .Values.config.conf.etcd.mtlsExistingSecret }}
- secret:
secretName: {{ .Values.config.conf.etcd.mtlsExistingSecret }}
name: etcd-config
{{- end}}
{{- with .Values.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
Expand All @@ -90,3 +112,7 @@ spec:
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.topologySpreadConstraints }}
topologySpreadConstraints:
{{- tpl (. | toYaml) $ | nindent 8 }}
{{- end }}
27 changes: 26 additions & 1 deletion charts/apisix-dashboard/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,12 @@ securityContext: {}
# runAsUser: 1000

config:
schema:
# -- Overrides APISIX Dashboard schema.json
# by mounting configMap containing schema.json
configMap: {}
# name: apisix-dashboard-schema
# key: schema.json
conf:
listen:
# -- The address on which the Manager API should listen.
Expand All @@ -89,6 +95,16 @@ config:
username: ~
# -- Specifies etcd basic auth password if enable etcd auth
password: ~

# -- Specifies a secret to be mounted on /etc/etcd for mtls usage
mtlsExistingSecret: ""

# MTLS configuration used for external etcd instances
mtls:
{}
# key_file: /etc/etcd/server-client.key
# cert_file: /etc/etcd/server-client.crt
# ca_file: /etc/etcd/server-ca.crt
log:
# -- Error log level.
# Supports levels, lower to higher: debug, info, warn, error, panic, fatal
Expand All @@ -99,7 +115,8 @@ config:
accessLog:
# -- Error log path
filePath: /dev/stdout

# -- Overrides plugins in the APISIX Dashboard conf
plugins: []
authentication:
# -- Secret for jwt token generation
secret: secret
Expand Down Expand Up @@ -165,10 +182,18 @@ autoscaling:
targetCPUUtilizationPercentage: 80
# targetMemoryUtilizationPercentage: 80

# -- Update strategy for apisix dashboard deployment
updateStrategy: {}
# type: RollingUpdate

# -- Node labels for pod assignment
nodeSelector: {}

# -- Tolerations for pod assignment
tolerations: []

affinity: {}

# -- Topology Spread Constraints for pod assignment spread across your cluster among failure-domains
# ref: https://kubernetes.io/docs/concepts/workloads/pods/pod-topology-spread-constraints/#spread-constraints-for-pods
topologySpreadConstraints: []
4 changes: 2 additions & 2 deletions charts/apisix-ingress-controller/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@ keywords:
- nginx
- crd
type: application
version: 0.11.6
appVersion: 1.6.1
version: 0.14.0
appVersion: 1.8.0
sources:
- https://github.com/apache/apisix-helm-chart

Expand Down
Loading

0 comments on commit a6aa358

Please sign in to comment.