Skip to content

Conversation

@tschaefer
Copy link
Owner

No description provided.

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

This PR adds the ability to update existing agents in the system by implementing an UpdateAgent RPC endpoint that allows modification of agent configuration fields such as labels, log sources, metrics settings, and profiles.

Key Changes:

  • Added UpdateAgent RPC method to the AgentService proto definition
  • Implemented UpdateAgent handlers across the gRPC server, controller, and model layers
  • Generated corresponding protobuf code for the new UpdateAgentRequest and UpdateAgentResponse messages

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
api/api.proto Defines the UpdateAgent RPC method and its request/response message structures
api/api.pb.go Generated protobuf code for UpdateAgentRequest and UpdateAgentResponse message types
api/api_grpc.pb.go Generated gRPC client and server code for the UpdateAgent RPC method
internal/grpc/server.go Implements the UpdateAgent gRPC handler with request validation and error handling
internal/controller/agent.go Implements the UpdateAgent business logic including agent retrieval, field updates, and persistence
internal/model/agent.go Implements the UpdateAgent database operation using GORM's Save method

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

@tschaefer tschaefer marked this pull request as ready for review January 6, 2026 09:41
@tschaefer tschaefer merged commit 0a3abca into main Jan 6, 2026
2 checks passed
@tschaefer tschaefer deleted the feature/agent-update branch January 6, 2026 09:41
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