Skip to content

feat: move helmchart to this repo#750

Open
RuTHlessBEat200 wants to merge 2 commits into
PatchMon:mainfrom
RuTHlessBEat200:feat_add_helm
Open

feat: move helmchart to this repo#750
RuTHlessBEat200 wants to merge 2 commits into
PatchMon:mainfrom
RuTHlessBEat200:feat_add_helm

Conversation

@RuTHlessBEat200
Copy link
Copy Markdown
Contributor

The Helm chart https://github.com/RuTHlessBEat200/PatchMon-helm has been added to the repo

Please check the workflow
The chart version and server image tag are automatically set to the tag via CI/CD

@RuTHlessBEat200 RuTHlessBEat200 changed the title move helmchart to this repo feat: move helmchart to this repo May 4, 2026
Comment thread helm/patchmon-server/values.yaml Fixed
Comment thread helm/patchmon-server/values.yaml Fixed
@tylerobara
Copy link
Copy Markdown

Are you open to adding gateway api/httproute support?
templates/route.yaml:

{{- if .Values.route.enabled }}
{{- $root := . }}
apiVersion: {{ .Values.route.group }}/{{ .Values.route.version }}
kind: {{ .Values.route.kind }}
metadata:
  name: {{ include "patchmon.fullname" . }}-route
  namespace: {{ .Release.Namespace }}
  {{- with .Values.route.annotations }}
  annotations:
    {{- toYaml . | nindent 4 }}
  {{- end }}
spec:
  {{- with .Values.route.hostnames }}
  hostnames:
    {{- toYaml . | nindent 4 }}
  {{- end }}
  {{- with .Values.route.parentRefs }}
  parentRefs:
    {{- toYaml . | nindent 4 }}
  {{- end }}
  rules:
    - matches:
        - path:
            type: PathPrefix
            value: /
      backendRefs:
        - name: {{ include "patchmon.fullname" $root }}-server
          kind: Service
          namespace: {{ $root.Release.Namespace }}
          port: {{ $root.Values.server.service.port }}
          weight: {{ $root.Values.route.weight | default 100 }}
{{- end }}

values.yaml:

...
route:
  enabled: false
  kind: HTTPRoute
  group: gateway.networking.k8s.io
  version: v1
  annotations: []
  hostnames:
    - patchmon.example.com
  parentRefs:
    - name: gateway
      kind: Gateway
      namespace: kube-system
      sectionName: websecure
...

It could be adjusted to support cases where someone wants to host it from example.com/patchmon

@RuTHlessBEat200
Copy link
Copy Markdown
Contributor Author

@tylerobara I'll add that tomorrow. I originally intended to do it today, but I haven't gotten around to it yet

@RuTHlessBEat200
Copy link
Copy Markdown
Contributor Author

@tylerobara Your suggestion has been added
@9technologygroup I have no idea what's wrong with fmt. When I run make fmt locally, it doesn't report any errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants