Skip to content

fix(level-25): allow adding netpol rule#30

Open
rhotertj wants to merge 1 commit intoManoj-engineer:mainfrom
rhotertj:fix/level-25-relax-validation
Open

fix(level-25): allow adding netpol rule#30
rhotertj wants to merge 1 commit intoManoj-engineer:mainfrom
rhotertj:fix/level-25-relax-validation

Conversation

@rhotertj
Copy link
Copy Markdown

Pull Request

📝 Description

The validation script now allows adding an ingress rule for the frontend pod instead of replacing the one for the "admin-tool".

🎯 Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎮 New level/challenge
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🎨 UI/UX improvement

🔗 Related Issue

Fixes #(issue number)

📍 Affected Levels/Areas

  • World 1 - Basics
  • World 2 - Deployments
  • World 3 - Networking
  • World 4 - Storage
  • World 5 - Security
  • Game Engine
  • Documentation
  • Other: ___________

✅ Testing Checklist

  • I have tested this locally
  • I have run the affected level(s) end-to-end
  • Validation scripts pass correctly
  • Hints are clear and helpful
  • Solution works as intended
  • Debrief explains the concept well
  • No breaking changes to other levels
  • Code follows the project style

🧪 How to Test

📸 Screenshots/Logs (if applicable)

📚 Documentation Updates

  • README.md updated (if needed)
  • Level documentation updated (if applicable)
  • Comments added to complex code
  • CHANGELOG.md updated (if significant change)

✨ Additional Notes

🤔 Questions for Reviewers


Reviewer Checklist

  • Code quality is acceptable
  • Changes align with K8sQuest's learning objectives
  • No security concerns
  • Documentation is adequate
  • Tested locally and works as described

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.

1 participant