Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds support for running kai inside a Kubernetes cluster using in-cluster configuration. This enables the application to authenticate using the service account mounted in the pod, eliminating the need for external kubeconfig files when deployed in-cluster.
Changes:
- Added a new
-in-clustercommand-line flag to enable in-cluster configuration mode - Implemented
LoadInClusterConfigmethod to configure Kubernetes clients using in-cluster authentication - Added comprehensive test coverage for the new in-cluster configuration functionality
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| cmd/kai/main.go | Added -in-cluster flag and conditional logic to load either in-cluster or kubeconfig-based configuration |
| cluster/manager.go | Implemented LoadInClusterConfig method with client creation, connection testing, and context management |
| cluster/manager_test.go | Added test suite covering edge cases including duplicate context names and default naming behavior |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@basebandit I've opened a new pull request, #110, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@basebandit I've opened a new pull request, #111, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@basebandit I've opened a new pull request, #112, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@basebandit I've opened a new pull request, #113, to work on those changes. Once the pull request is ready, I'll request review from you. |
4a2c802 to
26d7120
Compare
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context.
Type of change
Checklist: