Recached

Alan-Jowett / CoPilot-For-Consensus
82%

DEFAULT BRANCH: main
Repo Added
Token h869T5AwtIERs6BALB5g9YI6mXWb0J2ss regen
Build 1849 Last
Files 129
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH main
branch: SELECT
CHANGE BRANCH
x
Sync Branches

coverage: 82.133% (-0.06%) from 82.192%
20976629984

push

github

web-flow 
fix(infra): Remove secrets from environment variables, use secret_provider architecture (#855)

* fix: support both appinsights and azure secret names in metrics schema

The Bicep deployment creates Key Vault secrets with these names:
- appinsights-instrumentation-key
- appinsights-connection-string

But the schema was looking for:
- azure_connection_string
- azure_monitor_instrumentation_key

Updated the schema to try both naming conventions for backward compatibility with local dev and Bicep deployments.

* Revert "fix: support both appinsights and azure secret names in metrics schema"

This reverts commit aa51f2710.

* fix: align Azure Monitor secret names with schema expectations

Update Bicep deployment to create Key Vault secrets with names matching
the metrics adapter schema:
- appinsights-instrumentation-key → azure_monitor_instrumentation_key
- appinsights-connection-string → azure_connection_string

These names are expected by the copilot_config secret provider adapter
which reads them from Key Vault using the secret names defined in
docs/schemas/configs/adapters/drivers/metrics/metrics_azure_monitor.json

BREAKING: Deployments using the old secret names will need to be
redeployed to use the new naming convention.

* fix(infra): Remove secrets from environment variables, use secret_provider

Architectural fix: Secrets should be loaded via secret_provider from Key Vault
using schema secret_name field, not injected as environment variables with
@Microsoft.KeyVault(...) references.

Changes:
- Removed APPINSIGHTS_INSTRUMENTATIONKEY and APPLICATIONINSIGHTS_CONNECTION_STRING
  environment variables from all 8 microservices in containerapps.bicep
- These secrets are already in Key Vault (azure_monitor_instrumentation_key,
  azure_connection_string) and will be loaded automatically by copilot_config
  Phase 3 using the secret_provider with managed identity
- Enhanced validate_bicep_config.py to detect architectural vi... (continued)

0 of 8 new or added lines in 1 file covered. (0.0%)

2 existing lines in 1 file now uncovered.

6868 of 8362 relevant lines covered (82.13%)

0.91 hits per line

Relevant lines Covered
8362 RELEVANT LINES 6868 COVERED LINES
0.91 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
20976629984 main fix(infra): Remove secrets from environment variables, use secret_provider architecture (#855) * fix: support both appinsights and azure secret names in metrics schema The Bicep deployment creates Key Vault secrets with these names: - appinsight... push web-flow github
82.13
20968334560 main fix: include azure extras when installing adapters for cloud deployments (#853) * fix: include azure extras when installing adapters for cloud deployments - Updated install_adapters.py to install adapters with [azure] extras - Ensures Azure Moni... push web-flow github
82.19
20944441011 main Standardize Azure naming and add comprehensive adapter testing (#850) * Standardize Azure naming and add comprehensive adapter testing - Standardize all Azure service names to underscore format: - azure_monitor (was azuremonitor) - azure_ser... push web-flow github
80.95
20943792685 main UI: Sort email threads by thread start date with server-side global sorting (#847) * Initial plan * Backend: Always enrich reports with thread metadata including first_message_date Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply... push web-flow github
80.67
20940157007 main Enforce schema validation in unit tests via reusable fixtures (#830) * Initial plan * Add schema-compliant test fixtures and refactor chunking tests - Created tests/fixtures module with document_fixtures.py and event_fixtures.py - Added helpers... push web-flow github
80.66
20938733652 main fix: update adapter names in Azure Dockerfiles (copilot_reporting → copilot_error_reporting, copilot_events → copilot_message_bus) (#849) * Initial plan * fix: replace non-existent adapter names in Azure Dockerfiles - Replace copilot_reporting ... push web-flow github
80.66
20937037234 main fix: strict event validation for archive ingestion and parsing (#845) * fix: strict event validation for archive ingestion and parsing - startup_requeue: Add missing event envelope fields (event_id, version) * Generated event_id as UUID for ea... push web-flow github
80.66
20934787988 main Fix: Use generic adapter discriminants (LOG_TYPE, METRICS_TYPE) instead of service-specific variants (#844) * Fix: Use generic adapter discriminants (LOG_TYPE, METRICS_TYPE) instead of service-specific variants Root Cause: Services were failing ... push web-flow github
80.65
20927455143 main Improve azure_monitor_logger.py test coverage from 49% to 98% (#836) * Initial plan * Add comprehensive tests for azure_monitor_logger - coverage improved to 98% Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com> * Add... push web-flow github
80.66
20926002962 main Achieve 100% test coverage for GitHub, Google, and Microsoft auth providers (#838) * Initial plan * Add comprehensive tests for GitHub, Google, and Microsoft identity providers Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.git... push web-flow github
79.81
See All Builds (1010)

Badge your Repo: CoPilot-For-Consensus

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh