Skip to content

Update ingress configuration in values.yaml.j2 - #107

Merged
pmaslana merged 4 commits into
mainfrom
switch-to-contour
Jul 6, 2026
Merged

Update ingress configuration in values.yaml.j2#107
pmaslana merged 4 commits into
mainfrom
switch-to-contour

Conversation

@pmaslana

@pmaslana pmaslana commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Changing ingress class and redirect annotations affects how external traffic reaches production; misconfiguration could cause routing or TLS redirect issues until the cluster’s Contour setup is validated.

Overview
Ingress for offercodes.chia.net is moved from the NGINX ingress class to Contour, with className: contour set at the top of the ingress block instead of nginx at the bottom.

The TLS redirect annotation is updated from ingress.kubernetes.io/ssl-redirect to ingress.kubernetes.io/force-ssl-redirect, matching Contour’s expected behavior for HTTPS enforcement. cert-manager and host/path settings are unchanged.

Reviewed by Cursor Bugbot for commit 4a3e595. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit edbca8b. Configure here.

Comment thread k8s/values.yaml.j2
@pmaslana
pmaslana merged commit 48bbc50 into main Jul 6, 2026
7 checks passed
@pmaslana
pmaslana deleted the switch-to-contour branch July 6, 2026 16:55
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.

2 participants