Skip to content

Remove vestigial containerPort: 443 from the frontend Deployment #4359

Description

@imnasnainaec

Composed by Devin:

The frontend container declares containerPort: 443 at deploy/helm/thecombine/charts/frontend/templates/deployment-frontend.yaml:69, but nothing inside the container listens on 443:

  • nginx's active config listens only on 80 (nginx/templates/default.conf.template:2).
  • TLS is terminated at the Kubernetes Ingress, not the frontend container (docs/deploy/kubernetes_design/README.md:143-147).
  • The only listen 443 is in nginx/templates/url_moved.conf:17-18, which lacks the .template suffix and isn't processed into an active server block.

containerPort is informational, so this is harmless dead config — a cleanup, not a bug.

Change: remove the - containerPort: 443 line at deploy/helm/thecombine/charts/frontend/templates/deployment-frontend.yaml:69.

Confirm first: no Service/Ingress targets the pod's 443, and nginx/templates/url_moved.conf isn't wired to an active 443 listener.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Size: SSmall sized issuelowpriorityThis issue might not be worked on

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions