Skip to content

Add monitor resource to logs upload - #164

Merged
c-mongan merged 1 commit into
mainfrom
cmongan/public-logs-upload-monitor-resource
Jun 6, 2026
Merged

Add monitor resource to logs upload#164
c-mongan merged 1 commit into
mainfrom
cmongan/public-logs-upload-monitor-resource

Conversation

@c-mongan

@c-mongan c-mongan commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • pass the Azure Monitor auth resource to live Logs Ingestion API uploads
  • show the same resource flag in dry-run upload plans
  • cover both paths with focused tests

Verification

  • node --test agentops-cli/test/index.test.js --test-name-pattern "azure-ingest logs-upload|product audit"
  • node agentops-cli/src/index.js product audit --json
  • git diff --check

Live note

  • V2 ingestion resources are deployed in rg-copilot-agentops-dev.
  • The upload command now requests a token for https://monitor.azure.com/.
  • RBAC assignment for Monitoring Metrics Publisher has been added for the current user; live upload retry is still subject to Azure propagation.

@c-mongan
c-mongan merged commit 3fe1eb5 into main Jun 6, 2026
3 checks passed
@c-mongan
c-mongan deleted the cmongan/public-logs-upload-monitor-resource branch June 6, 2026 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant