[eoghanriley]

Description

Next implementation of uds monitor pepr

Related Issue

Fixes CLI-285

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Other (security config, docs update, etc)

Checklist before merging

• Test, docs, adr added or updated as needed
• Contributor Guide Steps followed

[greptile-apps]

Greptile Summary

The PR introduces uds core operator monitor for streaming and filtering UDS Core operator and Pepr policy activity.

• Adds the Next-mode command hierarchy and monitoring options.
• Discovers Pepr controller containers and streams their Kubernetes logs.
• Formats operator, policy, mutation, and failure events as terminal text or JSON.
• Adds unit and cluster-integration coverage, test workloads, and a command-placement ADR.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
internal/operator/monitor.go	Implements target discovery, concurrent log streaming and reconnection, event filtering, and output processing; both previously reported lifecycle failures are addressed.
internal/cli/core/operator_monitor.go	Defines the monitor command, validates arguments, and maps CLI flags into operator monitoring options.
internal/operator/formatter.go	Adds structured text rendering for operator and policy events, including mutation patches and repeated-event counts.
internal/operator/monitor_test.go	Covers stream selection, filtering, formatting, error handling, repeated events, and clean-EOF reconnection.
tests/cluster/operator_monitor_test.go	Exercises the command against synthetic admission and watcher pods in a cluster integration environment.

Sequence Diagram

sequenceDiagram
    actor User
    participant CLI as uds core operator monitor
    participant K8s as Kubernetes API
    participant Pods as Pepr controller pods
    participant Processor as Log processor
    User->>CLI: Run monitor with filters
    CLI->>K8s: List pods in pepr-system
    K8s-->>CLI: Admission and watcher pods
    loop Selected containers
        CLI->>Pods: Open log stream
        Pods-->>Processor: Pepr log entries
        Processor-->>User: Filtered text or JSON events
        alt Follow stream ends
            CLI->>CLI: Wait before reconnect
            CLI->>Pods: Reopen log stream
        end
    end

Loading

_{Reviews (2): Last reviewed commit: "fix: greptile feedback" | Re-trigger Greptile}

[chance-coleman]

nice work, happy with this. I'd probably appreciate one more set of eyes on this.

[joelmccoy]

Generally LGTM - great job porting and removing bubblecharm dep. One comment for consideration, i would recommend marking this feature as alpha