feat(nextcloud): add GatewayAPI httpRoute ingress successor

[DrummyFloyd]

Description of the change

Add httpRoute kind, because is the new sucesso of ingress and is now GA

Benefits

better control , and new standard

Checklist

• I have read the CONTRIBUTING.md doc.
• DCO has been signed off on the commit.
• Chart version bumped in Chart.yaml according to semver.
• (optional) Parameters are documented in the README.md

[wrenix]

The format of httpRoute values.yaml is different when using helm create <chartname. we should be align with it.
So please rename http to httpRoute.

also drop filters, matches and additionalRules for just rules.

here the templates

{{- if .Values.httpRoute.enabled -}}
{{- $fullName := include "nextcloud.fullname" . -}}
{{- $svcPort := .Values.service.port -}}
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: {{ $fullName }}
  labels:
    {{- include "nextcloud.labels" . | nindent 4 }}
  {{- with .Values.httpRoute.annotations }}
  annotations:
    {{- toYaml . | nindent 4 }}
  {{- end }}
spec:
  parentRefs:
    {{- with .Values.httpRoute.parentRefs }}
      {{- toYaml . | nindent 4 }}
    {{- end }}
  {{- with .Values.httpRoute.hostnames }}
  hostnames:
    {{- toYaml . | nindent 4 }}
  {{- end }}
  rules:
    {{- range .Values.httpRoute.rules }}
    {{- with .matches }}
    - matches:
      {{- toYaml . | nindent 8 }}
    {{- end }}
    {{- with .filters }}
      filters:
      {{- toYaml . | nindent 8 }}
    {{- end }}
      backendRefs:
        - name: {{ $fullName }}
          port: {{ $svcPort }}
          weight: 1
    {{- end }}
{{- end }}

[DrummyFloyd]

The format of httpRoute values.yaml is different when using helm create <chartname. we should be align with it. So please rename http to httpRoute.

also drop filters, matches and additionalRules for just rules.

here the templates

{{- if .Values.httpRoute.enabled -}}
{{- $fullName := include "nextcloud.fullname" . -}}
{{- $svcPort := .Values.service.port -}}
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: {{ $fullName }}
  labels:
    {{- include "nextcloud.labels" . | nindent 4 }}
  {{- with .Values.httpRoute.annotations }}
  annotations:
    {{- toYaml . | nindent 4 }}
  {{- end }}
spec:
  parentRefs:
    {{- with .Values.httpRoute.parentRefs }}
      {{- toYaml . | nindent 4 }}
    {{- end }}
  {{- with .Values.httpRoute.hostnames }}
  hostnames:
    {{- toYaml . | nindent 4 }}
  {{- end }}
  rules:
    {{- range .Values.httpRoute.rules }}
    {{- with .matches }}
    - matches:
      {{- toYaml . | nindent 8 }}
    {{- end }}
    {{- with .filters }}
      filters:
      {{- toYaml . | nindent 8 }}
    {{- end }}
      backendRefs:
        - name: {{ $fullName }}
          port: {{ $svcPort }}
          weight: 1
    {{- end }}
{{- end }}

should be good, now, but can you check on your side, because i have a strange behaviour .. on my end (must be related to my helm (CLI) latest update

Error: template: nextcloud/templates/_helpers.tpl:14:14: executing "nextcloud.fullname" at <.Values.fullnameOverride>: nil pointer evaluating interface {}.fullnameOverride

[wrenix]

Sorry for the long delay. I have not find any time on the weekend to test your code.

[DrummyFloyd]

Sorry for the long delay. I have not find any time on the weekend to test your code.

don't worry ^^
meanwhile i use a plain manifest for my need =)

[wrenix]

I updated my gatewayapi setup with treafik, but it does not work.
I found the problem, see below.

PS: do you like to rebase your code for the next merge?

[DrummyFloyd]

thank for the fix, and the rebase!