Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

server ingress fails to install/upgrade #66

Closed
kcorupe opened this issue Oct 31, 2019 · 4 comments · Fixed by #70
Closed

server ingress fails to install/upgrade #66

kcorupe opened this issue Oct 31, 2019 · 4 comments · Fixed by #70

Comments

@kcorupe
Copy link
Contributor

kcorupe commented Oct 31, 2019

Latest chart 4.5 fails to create ingress

helm upgrade --install --namespace aqua csp aqua-helm/server/ -f aqua-helm/values.yaml
UPGRADE FAILED
Error: unable to recognize "": no matches for kind "Ingress" in version "apps/v1"
Error: UPGRADE FAILED: unable to recognize "": no matches for kind "Ingress" in version "apps/v1"

Values.yaml (ingress section)

web:
  ingress:
    annotations:
      kubernetes.io/ingress.class: nginx
    enabled: true
    hosts:
    - aquasec.domain.com
    tls:
    - secretName: aquasec-tls
    - hosts:
      - aquasec.domain.com
@bernieatwar
Copy link

Any work around found for this issue yet?

@kcorupe
Copy link
Contributor Author

kcorupe commented Nov 19, 2019

I disabled ingress in aqua chat and created it manually after. Still waiting on fix from them.

@shayfisher
Copy link

shayfisher commented Nov 24, 2019 •
edited
Loading

They should use v1beta1 as Ingress isn't yet GA - even in k8s version 1.16.x.
You could edit a local copy of the template to reflect the change e.g revert commit:658ba37
On the server/templates/web-ingress.yaml file
It should look like this
apiVersion: extensions/v1beta1

pbrit added a commit to pbrit/aqua-helm that referenced this issue Jan 15, 2020
@pbrit
Fix apiVersion for Ingress
ba195a7 
  Fixes: aquasecurity#66
  More info: https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/
@pbrit pbrit mentioned this issue Jan 15, 2020
Merged
@pbrit
Copy link
Contributor

pbrit commented Jan 15, 2020

FYI: #70 fixes the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix apiVersion for Ingress pbrit/aqua-helm
4 participants
@pbrit @kcorupe @bernieatwar @shayfisher