Skip to content

Conversation

shanduur
Copy link
Member

@shanduur shanduur commented Sep 30, 2025

Run the tests manually:

Unit tests:

  • Image Factory chart: helm unittest ./deploy/helm/image-factory1
  • Registry chart: helm unittest ./deploy/helm/registry1

Smoke E2E:

  • All-in-One chart: chainsaw test2

Signed-off-by: Mateusz Urbanek [email protected]

Footnotes

  1. https://github.com/helm-unittest/helm-unittest 2

  2. https://kyverno.github.io/chainsaw/latest/

@shanduur shanduur force-pushed the chart branch 3 times, most recently from 1c9ad24 to 4332cb8 Compare September 30, 2025 09:08
@shanduur shanduur marked this pull request as ready for review September 30, 2025 09:08
@talos-bot talos-bot moved this to In Review in Planning Sep 30, 2025
@shanduur shanduur changed the title feat: add helm chart for IF feat: add helm chart for image factory Sep 30, 2025
@shanduur shanduur force-pushed the chart branch 7 times, most recently from dce1c41 to 7e8cf28 Compare September 30, 2025 14:01
{{- $_ := set .Values.ingress.pxe.annotations "kubernetes.io/ingress.class" .Values.ingress.pxe.className}}
{{- end }}
{{- end }}
{{- if semverCompare ">=1.19-0" .Capabilities.KubeVersion.GitVersion -}}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we support such old versions of k8s?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm always using this template for Ingresses, no need to support it, I just didn't removed that part of code. 😄

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

Added helm chart and unit tests for templates.

Signed-off-by: Mateusz Urbanek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

3 participants