Skip to content

Add the codejam winners to the k8s cluster#605

Closed
ChrisLovering wants to merge 1 commit intomainfrom
cj-winners
Closed

Add the codejam winners to the k8s cluster#605
ChrisLovering wants to merge 1 commit intomainfrom
cj-winners

Conversation

@ChrisLovering
Copy link
Member

These will be deleted once Johannes next takes his meds

@ChrisLovering ChrisLovering requested a review from jb3 as a code owner September 28, 2025 19:09
@python-discord-policy-bot python-discord-policy-bot bot requested a review from a team September 28, 2025 19:09
@github-actions github-actions bot added the group: kubernetes Issues and pull requests related to the Kubernetes setup label Sep 28, 2025
@github-actions
Copy link

OctoDNS Plan for cj-winners

pythondiscord.com.

cloudflare

Operation Name Type TTL Value Source
Delete gardenias CNAME 300 linode-lb.box.pydis.wtf.
Delete monsteras CNAME 300 linode-lb.box.pydis.wtf.
Delete tulips CNAME 300 linode-lb.box.pydis.wtf.

Summary: Creates=0, Updates=0, Deletes=3, Existing=35, Meta=False

@jb3 jb3 requested a review from Copilot September 28, 2025 19:11
These will be deleted once Johannes next takes his meds
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds Kubernetes configuration for deploying the "monsteras" codejam winner application to the k8s cluster. The PR creates a complete deployment setup with service exposure and ingress routing.

  • Creates Kubernetes deployment, service, and ingress resources for the monsteras application
  • Configures TLS authentication and HTTPS routing through pythondiscord.com domain
  • Sets up security contexts and container configurations for the application

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
kubernetes/namespaces/codejam-winners/monsteras/service.yaml Defines service to expose the monsteras application on port 80
kubernetes/namespaces/codejam-winners/monsteras/ingress.yaml Configures ingress with TLS authentication and domain routing
kubernetes/namespaces/codejam-winners/monsteras/deployment.yaml Sets up deployment with container specifications and security contexts

image: ghcr.io/python-discord/cj-monsteras
imagePullPolicy: Always
ports:
- containerPort: 80
Copy link

Copilot AI Sep 28, 2025

Choose a reason for hiding this comment

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

Port mismatch: the container exposes port 80 but the service targets port 8080. Either change containerPort to 8080 to match the service's targetPort, or update the service's targetPort to 80.

Suggested change
- containerPort: 80
- containerPort: 8080

Copilot uses AI. Check for mistakes.
@ChrisLovering
Copy link
Member Author

@ChrisLovering this is a reminder to take these services down when you are home.

5 similar comments
@jchristgit
Copy link
Member

@ChrisLovering this is a reminder to take these services down when you are home.

@Sockmower
Copy link

@ChrisLovering this is a reminder to take these services down when you are home.

@Dennisschu
Copy link
Contributor

@ChrisLovering this is a reminder to take these services down when you are home.

@fisher60
Copy link
Member

@ChrisLovering this is a reminder to take these services down when you are home.

@jb3
Copy link
Member

jb3 commented Feb 17, 2026

@ChrisLovering this is a reminder to take these services down when you are home.

@ChrisLovering
Copy link
Member Author

Everyone, thank you for your support on this matter. I can now confirm these services are offline 🫡

@github-project-automation github-project-automation bot moved this from Up next to Done in Infrastructure Feb 17, 2026
@ChrisLovering ChrisLovering deleted the cj-winners branch February 17, 2026 19:34
@GDWR
Copy link
Member

GDWR commented Feb 17, 2026

@ChrisLovering this is a reminder to take these services down when you are home.

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

Labels

group: kubernetes Issues and pull requests related to the Kubernetes setup

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants

Comments