Skip to content

Conversation

@The0mikkel
Copy link
Member

No description provided.

@The0mikkel The0mikkel self-assigned this Dec 20, 2025
Copilot AI review requested due to automatic review settings December 20, 2025 17:54
Copy link
Contributor

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

This PR renames Kubernetes namespace references from the legacy kubectf prefix to the new ctfpilot prefix across all Kubernetes template files. This is part of a broader rebranding effort to align infrastructure naming with the CTF Pilot project.

Key Changes:

  • Updated namespace fields from kubectf-challenges to ctfpilot-challenges in shared challenge templates
  • Updated namespace fields from kubectf-challenges-instanced to ctfpilot-challenges-instanced in instanced challenge templates
  • Applied changes consistently across both web and TCP challenge deployment types

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
template/shared-web-k8s.yml Updated namespace from kubectf-challenges to ctfpilot-challenges in Deployment, Service, and Ingress resources for shared web challenges
template/shared-tcp-k8s.yml Updated namespace from kubectf-challenges to ctfpilot-challenges in Deployment, Service, and IngressRouteTCP resources for shared TCP challenges, including middleware namespace reference
template/instanced-web-k8s.yml Updated namespace from kubectf-challenges-instanced to ctfpilot-challenges-instanced in Deployment, Service, and Ingress resources for instanced web challenges
template/instanced-tcp-k8s.yml Updated namespace from kubectf-challenges-instanced to ctfpilot-challenges-instanced in Deployment, Service, and IngressRouteTCP resources for instanced TCP challenges

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@The0mikkel The0mikkel merged commit c7cac9c into main Dec 20, 2025
7 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