Skip to content

Listen on port 80 (the chart's port): surl unreachable since 2026-08-30 - #30

Merged
samerzughul merged 1 commit into
mainfrom
muhannad/listen-on-80
Oct 9, 2026
Merged

samerzughul merged 1 commit into
mainfrom
muhannad/listen-on-80

Conversation

@mmalkhatib

Copy link
Copy Markdown
Contributor

The image is aspnet:8.0, which listens on 8080 by default. The surl Helm chart sets containerPort: 80 and has no way to set the port or add env variables, so every pod from this chart refuses connections.

Impact: Traxis production surl (production/surl, deployed 2026-08-30 11:53 UTC) has been unreachable ever since, and surl.traxis.app returns 503. Checkout's quotation notifications create a short link before sending, so no quotation email or SMS has gone out since then and nothing is written to the quotation's history. Confirmed with a port-forward: pod:8080 answers, pod:80 refuses.

Fix: ENV ASPNETCORE_HTTP_PORTS=80 in the Dockerfile (2 lines; the file's CRLF line endings are kept). After merge, the new image and chart version need a manual cicd-prod.yml run in traxis-surl to reach Traxis production. The immediate production fix is kubectl -n production set env deploy/surl ASPNETCORE_HTTP_PORTS=80.

🤖 Generated with Claude Code

The image is .NET 8, which listens on 8080 unless told otherwise, but the surl chart's
containerPort is 80. Traxis production surl has been unreachable since its 2026-08-30
deploy, which silently stopped all quotation emails and SMS.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mmalkhatib
mmalkhatib force-pushed the muhannad/listen-on-80 branch from 20d415a to a80336e Compare October 9, 2026 20:45
@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: simplify9/coderabbit/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 76f8bf09-8264-4147-b449-08fe96e8fb6b

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@samerzughul
samerzughul merged commit 8f8b8c4 into main Oct 9, 2026
5 checks passed
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