Skip to content

feat: initial implementation#2

Merged
phisco merged 2 commits into
mainfrom
initial-implementation
Jan 27, 2026
Merged

feat: initial implementation#2
phisco merged 2 commits into
mainfrom
initial-implementation

Conversation

@phisco
Copy link
Copy Markdown
Collaborator

@phisco phisco commented Jan 22, 2026

Description of your changes

From crossplane/crossplane#7025.

Fixes #

I have:

Copy link
Copy Markdown

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 introduces a reference implementation of a Pipeline Inspector sidecar for Crossplane that captures function pipeline execution data (requests and responses) and logs them to stdout.

Changes:

  • Implements a gRPC server that listens on a Unix socket and logs pipeline inspection events
  • Supports both JSON and human-readable text output formats
  • Includes comprehensive test coverage for the inspector functionality

Reviewed changes

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

Show a summary per file
File Description
main.go Entry point with CLI parsing, gRPC server setup, and graceful shutdown handling
server/server.go Core Inspector implementation with JSON and text formatters
server/server_test.go Comprehensive unit tests for Inspector functionality
go.mod Module definition with dependencies
go.sum Dependency checksums
Dockerfile Multi-stage build using distroless base image
README.md Documentation with usage examples and configuration options

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

Comment thread server/server.go
Comment thread main.go Outdated
Comment thread README.md
Comment thread server/server.go
Signed-off-by: Philippe Scorsolini <p.scorsolini@gmail.com>
@phisco phisco force-pushed the initial-implementation branch from bfb5b77 to daf3fe8 Compare January 22, 2026 13:47
Signed-off-by: Philippe Scorsolini <p.scorsolini@gmail.com>
@phisco phisco marked this pull request as ready for review January 27, 2026 10:40
@phisco phisco merged commit ace0ed1 into main Jan 27, 2026
4 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