Alan-Jowett / CoPilot-For-Consensus / 20562236203

78%

Build Type

push

github

Committed by web-flow

Commit Message

Standardize startup requeue via StartupRequeue; fix publisher API usage in chunking/orchestrator (#630)

* fix: Grafana JWT SSO configuration - use JWKS endpoint instead of key file

Changed Grafana JWT configuration from using a file-based public key
(GF_AUTH_JWT_KEY_FILE) to using the JWKS endpoint (GF_AUTH_JWT_JWK_SET_URL).
This approach:

1. Uses Grafana's native JWKS endpoint support which is more reliable
2. Fetches keys dynamically from the auth service JWKS endpoint
3. Caches keys for 1 hour to reduce load on auth service
4. Uses HTTPS (via gateway) as required by Grafana JWT auth in v12.3.0

Previously, Grafana JWT auth failed with 'no keys found' error because
the static key file approach wasn't properly loading keys. The JWKS
endpoint approach is standard OIDC practice and works with the
/.well-known/jwks.json endpoint already provided by auth service.

Fixes the 'key set for jwt verification is not configured' error and
enables Grafana SSO with OAuth/OIDC authentication flow.

* fix: correct publisher.publish() API call in chunking startup requeue

The RabbitMQPublisher.publish() method expects (exchange, routing_key, event) parameters, not (event_type, data, routing_key, exchange). This was causing 'unexpected keyword argument event_type' errors during startup requeue.

Fixed by building a complete event dict with event_type, timestamp, and data fields, then passing it as the 'event' parameter.

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* fix: correct publisher.publish() API call in orchestrator startup requeue

Same issue as chunking service - the orchestrator was using the old API signature (event_type, data, routing_key, exchange) instead of the correct (exchange, routing_key, event) signature.

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* refactor: use StartupRequeue for orchestrator requeue + helper

Added StartupRequeue.publish_event() to centralize requeue event construction/publishing.
Refactored orchest... (continued)

Run Details

3 of 6 new or added lines in 2 files covered. (50.0%)

2 existing lines in 1 file now uncovered.

5930 of 7783 relevant lines covered (76.19%)

0.85 hits per line