Alan-Jowett / CoPilot-For-Consensus / 20968334560

82%

Build Type

push

github

Committed by web-flow

Commit Message

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 Monitor OpenTelemetry packages are available at runtime
- Fixes missing dependencies in container deployments to Azure
- Resolves issue #852

The install_adapters.py script was installing adapters without their
optional extras, causing ImportError at runtime for azure-monitor-opentelemetry
packages when deployed to Azure Container Apps.

Services affected: auth, ingestion, parsing, chunking, embedding, orchestrator

* fix: pin azure monitor exporter to available beta

* Fix azure monitor metrics tests

Signed-off-by: GitHub Copilot <copilot@github.com>

* Make adapter installs skip missing azure extras

Signed-off-by: GitHub Copilot <copilot@github.com>

* Fix Azure Monitor metrics test failures

Signed-off-by: GitHub Copilot <copilot@github.com>

* Stabilize Azure Monitor metrics tests with valid defaults

Signed-off-by: GitHub Copilot <copilot@github.com>

* Pin azure monitor exporter beta and tidy metrics tests

Signed-off-by: GitHub Copilot <copilot@github.com>

* Address PR review: pin versions consistently and improve AST parsing

Signed-off-by: GitHub Copilot <copilot@github.com>

* Fix azure-monitor-opentelemetry-exporter compatibility: pin opentelemetry to ~=1.21.0

* Add comprehensive test coverage for install_adapters functions

- Test _extras_from_setup() with various setup.py structures
- Test select_azure_extra() priority and selection logic
- Cover edge cases: missing files, malformed setup.py, unicode content
- Test handling of qualified names and setup function detection
- Verify 'azure' priority over 'azuremonitor' when both present
- All 16 tests passing

* Improve clarity of _extras_from_setup function logic

- Remove duplicate return statement after except block
- Add comprehensive docstring ... (continued)

Run Details

6 of 6 new or added lines in 1 file covered. (100.0%)

15 existing lines in 4 files now uncovered.

6868 of 8356 relevant lines covered (82.19%)

0.92 hits per line