search

banzaicloud / dast-operator

Dynamic Application and API Security Testing
Apache License 2.0
189 stars 28 forks source link

Running `make deploy` returns an error #5

Closed just1689 closed 4 years ago

just1689 commented 4 years ago

Describe the bug Followed the installation steps and running make deploy returns an error.


➜  dast-operator git:(master) ✗ make deploy
cd config/manager && kustomize edit set image controller=reg.captainjustin.space/banzaicloud/dast-operator:latest
kustomize build config/default | kubectl apply -f -
namespace/dast-operator-system unchanged
error: error validating "STDIN": error validating data: [ValidationError(CustomResourceDefinition.spec.validation.openAPIV3Schema.properties.spec.properties.analyzer.properties.service.properties.spec.properties.ports): unknown field "x-kubernetes-list-map-keys" in io.k8s.apiextensions-apiserver.pkg.apis.apiextensions.v1beta1.JSONSchemaProps, ValidationError(CustomResourceDefinition.spec.validation.openAPIV3Schema.properties.spec.properties.analyzer.properties.service.properties.spec.properties.ports): unknown field "x-kubernetes-list-type" in io.k8s.apiextensions-apiserver.pkg.apis.apiextensions.v1beta1.JSONSchemaProps]; if you choose to ignore these errors, turn validation off with --validate=false
Makefile:43: recipe for target 'deploy' failed
make: *** [deploy] Error 1

Steps to reproduce the issue:

Running on Kubernetes 1.15.12


kubectl create namespace cert-manager
helm repo add jetstack https://charts.jetstack.io
helm repo update
kubectl apply --validate=false -f https://github.com/jetstack/cert-manager/releases/download/v0.15.1/cert-manager.crds.yaml
helm install cert-manager jetstack/cert-manager --namespace cert-manager --version v0.15.1
git clone https://github.com/banzaicloud/dast-operator.git
cd dast-operator
make deploy

Expected behavior Should not error out and crash the install

Screenshots

Additional context

pbalogh-sa commented 4 years ago

Thanks for reporting this @just1689 ! We will dig into it.

just1689 commented 4 years ago

Thanks @pbalogh-sa . We're going to be trying this out in a lab cluster. If we find anything to contribute we will be sure to share.

Thanks for the project. Fantastic stuff.

pbalogh-sa commented 4 years ago

@just1689 . In the case of the k8s version 1.15 or below this error happens due to validation. I turned off the validation.

just1689 commented 4 years ago

Super, thank you. Will do