What does this MR do and why?

• fix(lint): exclude .opencode and .sessions from Vale scanning

Vale walks the filesystem, not the git index, so it was picking up untracked directories like .opencode/node_modules/, .sessions/, and .dev (used by xtask e2e) causing 39 spurious errors.

Related Issues

Testing

Performance Analysis

• This merge request does not introduce any performance regression. If a performance regression is expected, explain why.

Bohdan Parkhomchuk (2c8e02b1) at 19 Mar 03:36

Problem to Solve

We should replace our ad-hoc Labkit-rs with an official one.

Proposed Solution

1. Set up labkit-rs releases pipeline — configure semantic releases with version increments, git tagging, and changelog so labkit-rs can be properly consumed as a dependency (labkit-rs#21).

2. Integrate labkit-rs into GKG — replace our ad-hoc implementations with the official labkit-rs crate.

3. Smoke test labkit-rs — verify all functionality works end-to-end:

   • Correlation IDs
   • Access logs
   • Structured logging
   • Tracing data
   • Metrics

Bohdan Parkhomchuk (fa6156c5) at 19 Mar 03:36

... and 1 more commit

Summary

• Replace ad-hoc crates/labkit-rs with the official labkit crate from gitlab-org/rust/labkit-rs
• Unified initialization via labkit::Builder (logging + OTel in one call)
• HTTP middleware: CorrelationLayer + GitlabTraceLayer + HttpMetricsLayer replace 4 separate layers
• gRPC middleware: Tower layers (GrpcCorrelationLayer, GrpcTraceLayer, GrpcMetricsLayer) replace manual server_interceptor + GrpcMetrics::record() wrappers in each handler
• Service name derived from --mode flag (gkg-webserver, gkg-indexer, gkg-dispatcher, gkg-healthcheck)
• Delete ~3600 lines of ad-hoc observability code
• Add 5 telemetry integration tests (HTTP metrics, metric attributes, correlation ID echo, correlation ID generation, pipeline observer metrics)

Closes #158

Test plan

• cargo nextest run -p gkg-server — 137 tests pass
• cargo nextest run -p integration-tests — 37 tests pass (including 5 new telemetry tests)
• cargo clippy — no warnings
• cargo fmt --check — clean

@michaelangeloio we don't have tags there yet, releases TBD

We have retries for these, but we use the same tests in labkit-rs so should be ok

In nextest each test is isolated so not really a concern

Summary

• Replace ad-hoc crates/labkit-rs with the official labkit crate from gitlab-org/rust/labkit-rs
• Unified initialization via labkit::Builder (logging + OTel in one call)
• HTTP middleware: CorrelationLayer + GitlabTraceLayer + HttpMetricsLayer replace 4 separate layers
• gRPC middleware: Tower layers (GrpcCorrelationLayer, GrpcTraceLayer, GrpcMetricsLayer) replace manual server_interceptor + GrpcMetrics::record() wrappers in each handler
• Service name derived from --mode flag (gkg-webserver, gkg-indexer, gkg-dispatcher, gkg-healthcheck)
• Delete ~3600 lines of ad-hoc observability code
• Add 5 telemetry integration tests (HTTP metrics, metric attributes, correlation ID echo, correlation ID generation, pipeline observer metrics)

Closes #158

Test plan

• cargo nextest run -p gkg-server — 137 tests pass
• cargo nextest run -p integration-tests — 37 tests pass (including 5 new telemetry tests)
• cargo clippy — no warnings
• cargo fmt --check — clean

Bohdan Parkhomchuk (2c8e02b1) at 19 Mar 01:16

@stejacks-gitlab can we please get a quick review and merge on this bump?

Bohdan Parkhomchuk (cad645f9) at 18 Mar 20:11

... and 16 more commits

Bohdan Parkhomchuk (7c23959d) at 18 Mar 20:07

Summary

• Bump GKG Helm chart version from 0.5.0 to 0.6.0
• Bump GKG image tag from 0.11.0 to 0.12.0

Bohdan Parkhomchuk (857e59a9) at 18 Mar 20:02

Bohdan Parkhomchuk (f2df9d88) at 18 Mar 20:02

... and 1 more commit

• Bump default image tag from 0.10.0 to 0.12.0
• Rename code-push-event handler to code-indexing-task
• Remove code-project-reconciliation handler
• Simplify schedule.tasks default to {}

• Bump default image tag from 0.10.0 to 0.12.0
• Rename code-push-event handler to code-indexing-task
• Remove code-project-reconciliation handler
• Simplify schedule.tasks default to {}

Bohdan Parkhomchuk (857e59a9) at 18 Mar 19:54

Bohdan Parkhomchuk (48f4ec47) at 18 Mar 15:57