Use local chart directory for dev-env and e2e tests (#5165)

This commit is contained in:
Manuel Alejandro de Brito Fontes 2020-02-25 09:35:44 -03:00 committed by GitHub
parent 35264d6e8f
commit e3e16941e5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
7 changed files with 14 additions and 13 deletions

View file

@ -99,10 +99,7 @@ kind load docker-image --name="${KIND_CLUSTER_NAME}" "${DEV_IMAGE}"
echo "[dev-env] deploying NGINX Ingress controller..." echo "[dev-env] deploying NGINX Ingress controller..."
kubectl create namespace ingress-nginx &> /dev/null || true kubectl create namespace ingress-nginx &> /dev/null || true
helm repo add stable https://kubernetes-charts.storage.googleapis.com &> /dev/null || true cat << EOF | helm template ingress-nginx ${DIR}/../charts/ingress-nginx --namespace=ingress-nginx --values - | kubectl apply -n ingress-nginx -f -
helm repo update &> /dev/null || true
cat << EOF | helm template nginx-ingress stable/nginx-ingress --namespace=ingress-nginx --values - | kubectl apply -n ingress-nginx -f -
controller: controller:
image: image:
repository: ${REGISTRY}/nginx-ingress-controller repository: ${REGISTRY}/nginx-ingress-controller

View file

@ -1,5 +1,5 @@
apiVersion: v1 apiVersion: v1
name: nginx-ingress name: ingress-nginx
version: 1.32.0 version: 1.32.0
appVersion: 0.30.0 appVersion: 0.30.0
home: https://github.com/kubernetes/ingress-nginx home: https://github.com/kubernetes/ingress-nginx

View file

@ -16,7 +16,7 @@ $ helm install ingress-nginx
## Introduction ## Introduction
This chart bootstraps an nginx-ingress deployment on a [Kubernetes](http://kubernetes.io) cluster using the [Helm](https://helm.sh) package manager. This chart bootstraps an ingress-nginx deployment on a [Kubernetes](http://kubernetes.io) cluster using the [Helm](https://helm.sh) package manager.
## Prerequisites ## Prerequisites
@ -263,7 +263,7 @@ $ helm install ingress-nginx --name my-release \
You can add Prometheus annotations to the metrics service using `controller.metrics.service.annotations`. Alternatively, if you use the Prometheus Operator, you can enable ServiceMonitor creation using `controller.metrics.serviceMonitor.enabled`. You can add Prometheus annotations to the metrics service using `controller.metrics.service.annotations`. Alternatively, if you use the Prometheus Operator, you can enable ServiceMonitor creation using `controller.metrics.serviceMonitor.enabled`.
## nginx-ingress nginx\_status page/stats server ## ingress-nginx nginx\_status page/stats server
Previous versions of this chart had a `controller.stats.*` configuration block, which is now obsolete due to the following changes in nginx ingress controller: Previous versions of this chart had a `controller.stats.*` configuration block, which is now obsolete due to the following changes in nginx ingress controller:
* in [0.16.1](https://github.com/kubernetes/ingress-nginx/blob/master/Changelog.md#0161), the vts (virtual host traffic status) dashboard was removed * in [0.16.1](https://github.com/kubernetes/ingress-nginx/blob/master/Changelog.md#0161), the vts (virtual host traffic status) dashboard was removed

View file

@ -1,5 +1,4 @@
e2e.test e2e.test
ginkgo ginkgo
kubectl kubectl
/cloud-generic/ charts
/cluster-wide/

View file

@ -23,4 +23,6 @@ COPY namespace-overlays /namespace-overlays
COPY wait-for-nginx.sh / COPY wait-for-nginx.sh /
COPY e2e.test / COPY e2e.test /
COPY charts /charts
CMD [ "/e2e.sh" ] CMD [ "/e2e.sh" ]

View file

@ -15,13 +15,14 @@ endif
.PHONY: container .PHONY: container
container: container:
ifeq ($(USE_DOCKER), true) ifeq ($(USE_DOCKER), true)
@${DIR}/../../build/run-in-docker.sh make e2e-test-binary @$(DIR)/../../build/run-in-docker.sh make e2e-test-binary
else else
@make -C ${DIR}/../../ e2e-test-binary @make -C $(DIR)/../../ e2e-test-binary
endif endif
cp ../e2e/e2e.test . cp $(DIR)/../e2e/e2e.test .
cp ../e2e/wait-for-nginx.sh . cp $(DIR)/../e2e/wait-for-nginx.sh .
cp -R $(DIR)/../../charts .
docker buildx build \ docker buildx build \
--load \ --load \

View file

@ -186,6 +186,8 @@ controller:
# e2e tests do not require information about ingress status # e2e tests do not require information about ingress status
update-status: "false" update-status: "false"
terminationGracePeriodSeconds: 1 terminationGracePeriodSeconds: 1
admissionWebhooks:
enabled: false
defaultBackend: defaultBackend:
enabled: false enabled: false