Skip to content

Conversation

@bluepal-prasanthi-moparthi
Copy link
Collaborator

This PR includes the following Monitoring - related endpoints:

  1. GET - /_admin/log/entries
  2. DELETE - /_admin/log/level
  3. GET - /_admin/log/structured
  4. PUT - /_admin/log/structured
  5. GET - /_db/{database-name}/_admin/server/api-calls
  6. GET - /_db/{database-name}/_admin/metrics/v2

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds several monitoring and administrative endpoints to the ArangoDB Go driver, enabling better observability and log management capabilities. The implementation includes comprehensive testing for all new endpoints and proper integration with existing client patterns.

  • Adds 6 new admin/monitoring endpoints for log management, structured logging, API call tracking, and metrics collection
  • Implements proper query parameter handling and validation for complex log filtering options
  • Provides comprehensive test coverage for all new endpoints with version compatibility checks

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
v2/arangodb/client_admin.go Defines interfaces and data structures for new monitoring endpoints
v2/arangodb/client_admin_log_impl.go Implements the actual HTTP client methods for all new endpoints
v2/tests/admin_log_test.go Adds comprehensive test coverage for all new endpoints
Makefile Updates default ArangoDB Docker image to enterprise edition

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@bluepal-prasanthi-moparthi bluepal-prasanthi-moparthi force-pushed the feature/add_mis_endpoints_in_monitoring branch from 3469011 to f2d0664 Compare September 5, 2025 10:42
@jwierzbo jwierzbo merged commit 896083e into master Sep 5, 2025
9 checks passed
@jwierzbo jwierzbo deleted the feature/add_mis_endpoints_in_monitoring branch September 5, 2025 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants