Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AI-4451] Add Milvus integration #19055

Merged
merged 31 commits into from
Dec 24, 2024
Merged
Show file tree
Hide file tree
Changes from 25 commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
9c7f4db
Add Milvus skeleton
dkirov-dd Nov 14, 2024
838e614
Add metrics to metadata.csv
dkirov-dd Nov 18, 2024
9bd8157
Add periods in metadata.csv
dkirov-dd Nov 19, 2024
bf86860
Update spec
dkirov-dd Nov 19, 2024
18c16ac
Update config models
dkirov-dd Nov 19, 2024
d3d292c
Update manifest
dkirov-dd Nov 19, 2024
e20c23b
Update check.py
dkirov-dd Nov 19, 2024
9486b21
Add process signature
dkirov-dd Nov 25, 2024
aa71043
Add units in metadata.csv
dkirov-dd Nov 25, 2024
ccea688
Remove process signature
dkirov-dd Nov 25, 2024
e190fd6
Add payload fixture
dkirov-dd Nov 25, 2024
f783bfb
Add tests
dkirov-dd Nov 27, 2024
6da32ed
Fix metadata.csv
dkirov-dd Nov 27, 2024
66d50f8
Add non-Milvus specific metrics
dkirov-dd Nov 27, 2024
8c91841
Update metrics map
dkirov-dd Nov 27, 2024
65b8c37
Expand histogram metrics
dkirov-dd Nov 27, 2024
f414146
Fix metrics mapping and divide test metrics
dkirov-dd Nov 28, 2024
c2c4bf5
Passing unit tests
dkirov-dd Dec 12, 2024
b95c123
Add e2e env and test
dkirov-dd Dec 13, 2024
d28e6bb
Run lint
dkirov-dd Dec 16, 2024
17d2867
Fix CI
dkirov-dd Dec 16, 2024
2e1d3e8
Switch to deprecated classifier tags for CI
dkirov-dd Dec 16, 2024
6ea7d3d
Sync labeler
dkirov-dd Dec 16, 2024
8777306
Remove dashboard and add metric check
dkirov-dd Dec 16, 2024
caf2c33
Implement review suggestions
dkirov-dd Dec 17, 2024
8ee1670
Incorporate documentation suggestions
dkirov-dd Dec 19, 2024
cf5d4e9
Capitalize unix in metadata description
dkirov-dd Dec 19, 2024
ef3ee92
Incorporate documentation suggestions
dkirov-dd Dec 23, 2024
11c337e
Move metrics out of standalone test
dkirov-dd Dec 23, 2024
c628898
Fix unit tests
dkirov-dd Dec 23, 2024
e2e1b5f
Improve README
dkirov-dd Dec 23, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -410,6 +410,10 @@ coverage:
target: 75
flags:
- mesos_master
Milvus:
target: 75
flags:
- milvus
MongoDB:
target: 75
flags:
Expand Down Expand Up @@ -1254,6 +1258,11 @@ flags:
paths:
- mesos_slave/datadog_checks/mesos_slave
- mesos_slave/tests
milvus:
carryforward: true
paths:
- milvus/datadog_checks/milvus
- milvus/tests
mongo:
carryforward: true
paths:
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/config/labeler.yml
Original file line number Diff line number Diff line change
Expand Up @@ -339,6 +339,8 @@ integration/mesos_slave:
- mesos_slave/**/*
integration/metabase:
- metabase/**/*
integration/milvus:
- milvus/**/*
integration/mimecast:
- mimecast/**/*
integration/mongo:
Expand Down
20 changes: 20 additions & 0 deletions .github/workflows/test-all.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2494,6 +2494,26 @@ jobs:
minimum-base-package: ${{ inputs.minimum-base-package }}
pytest-args: ${{ inputs.pytest-args }}
secrets: inherit
j062aeb0:
uses: ./.github/workflows/test-target.yml
with:
job-name: Milvus
target: milvus
platform: linux
runner: '["ubuntu-22.04"]'
repo: "${{ inputs.repo }}"
python-version: "${{ inputs.python-version }}"
standard: ${{ inputs.standard }}
latest: ${{ inputs.latest }}
agent-image: "${{ inputs.agent-image }}"
agent-image-py2: "${{ inputs.agent-image-py2 }}"
agent-image-windows: "${{ inputs.agent-image-windows }}"
agent-image-windows-py2: "${{ inputs.agent-image-windows-py2 }}"
test-py2: ${{ inputs.test-py2 }}
test-py3: ${{ inputs.test-py3 }}
minimum-base-package: ${{ inputs.minimum-base-package }}
pytest-args: ${{ inputs.pytest-args }}
secrets: inherit
j91231ff:
uses: ./.github/workflows/test-target.yml
with:
Expand Down
4 changes: 4 additions & 0 deletions milvus/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# CHANGELOG - Milvus

<!-- towncrier release notes start -->

60 changes: 60 additions & 0 deletions milvus/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,60 @@
# Agent Check: Milvus

## Overview

This check monitors [Milvus][1] through the Datadog Agent.

Include a high level overview of what this integration does:
- What does your product do (in 1-2 sentences)?
- What value will customers get from this integration, and why is it valuable to them?
- What specific data will your integration monitor, and what's the value of that data?
dkirov-dd marked this conversation as resolved.
Show resolved Hide resolved

## Setup

Follow the instructions below to install and configure this check for an Agent running on a host. For containerized environments, see the [Autodiscovery Integration Templates][3] for guidance on applying these instructions.

### Installation

The Milvus check is included in the [Datadog Agent][2] package.
No additional installation is needed on your server.

### Configuration

1. Edit the `milvus.d/conf.yaml` file, in the `conf.d/` folder at the root of your Agent's configuration directory to start collecting your milvus performance data. See the [sample milvus.d/conf.yaml][4] for all available configuration options.
dkirov-dd marked this conversation as resolved.
Show resolved Hide resolved

2. [Restart the Agent][5].

### Validation

[Run the Agent's status subcommand][6] and look for `milvus` under the Checks section.

## Data Collected

### Metrics

See [metadata.csv][7] for a list of metrics provided by this integration.

### Events

The Milvus integration does not include any events.

### Service Checks

The Milvus integration does not include any service checks.

See [service_checks.json][8] for a list of service checks provided by this integration.

## Troubleshooting

Need help? Contact [Datadog support][9].


[1]: **LINK_TO_INTEGRATION_SITE**
dkirov-dd marked this conversation as resolved.
Show resolved Hide resolved
[2]: https://app.datadoghq.com/account/settings/agent/latest
[3]: https://docs.datadoghq.com/agent/kubernetes/integrations/
[4]: https://github.com/DataDog/integrations-core/blob/master/milvus/datadog_checks/milvus/data/conf.yaml.example
[5]: https://docs.datadoghq.com/agent/guide/agent-commands/#start-stop-and-restart-the-agent
[6]: https://docs.datadoghq.com/agent/guide/agent-commands/#agent-status-and-information
[7]: https://github.com/DataDog/integrations-core/blob/master/milvus/metadata.csv
[8]: https://github.com/DataDog/integrations-core/blob/master/milvus/assets/service_checks.json
[9]: https://docs.datadoghq.com/help/
15 changes: 15 additions & 0 deletions milvus/assets/configuration/spec.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
name: Milvus
files:
- name: milvus.yaml
options:
- template: init_config
options:
- template: init_config/openmetrics
- template: instances
options:
- template: instances/openmetrics
overrides:
openmetrics_endpoint.value.example: http://localhost:9091/metrics
openmetrics_endpoint.description: |
Endpoint exposing Milvus' Prometheus metrics. For more information, refer to
https://milvus.io/docs/monitor.md#Monitor-metrics-with-Prometheus.
1 change: 1 addition & 0 deletions milvus/assets/service_checks.json
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
[]
1 change: 1 addition & 0 deletions milvus/changelog.d/19055.added
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Initial Release
4 changes: 4 additions & 0 deletions milvus/datadog_checks/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# (C) Datadog, Inc. 2024-present
# All rights reserved
# Licensed under a 3-clause BSD style license (see LICENSE)
__path__ = __import__('pkgutil').extend_path(__path__, __name__) # type: ignore
4 changes: 4 additions & 0 deletions milvus/datadog_checks/milvus/__about__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# (C) Datadog, Inc. 2024-present
# All rights reserved
# Licensed under a 3-clause BSD style license (see LICENSE)
__version__ = '0.0.1'
7 changes: 7 additions & 0 deletions milvus/datadog_checks/milvus/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
# (C) Datadog, Inc. 2024-present
# All rights reserved
# Licensed under a 3-clause BSD style license (see LICENSE)
from .__about__ import __version__
from .check import MilvusCheck

__all__ = ['__version__', 'MilvusCheck']
21 changes: 21 additions & 0 deletions milvus/datadog_checks/milvus/check.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
# (C) Datadog, Inc. 2024-present
# All rights reserved
# Licensed under a 3-clause BSD style license (see LICENSE)

from datadog_checks.base import OpenMetricsBaseCheckV2
from datadog_checks.milvus.metrics import METRIC_MAP, RENAME_LABELS_MAP


class MilvusCheck(OpenMetricsBaseCheckV2):
__NAMESPACE__ = 'milvus'

DEFAULT_METRIC_LIMIT = 0

def __init__(self, name, init_config, instances=None):
super(MilvusCheck, self).__init__(name, init_config, instances)

def get_default_config(self):
return {
'metrics': [METRIC_MAP],
'rename_labels': RENAME_LABELS_MAP,
}
24 changes: 24 additions & 0 deletions milvus/datadog_checks/milvus/config_models/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
# (C) Datadog, Inc. 2024-present
# All rights reserved
# Licensed under a 3-clause BSD style license (see LICENSE)

# This file is autogenerated.
# To change this file you should edit assets/configuration/spec.yaml and then run the following commands:
# ddev -x validate config -s <INTEGRATION_NAME>
# ddev -x validate models -s <INTEGRATION_NAME>

from .instance import InstanceConfig
from .shared import SharedConfig


class ConfigMixin:
_config_model_instance: InstanceConfig
_config_model_shared: SharedConfig

@property
def config(self) -> InstanceConfig:
return self._config_model_instance

@property
def shared_config(self) -> SharedConfig:
return self._config_model_shared
132 changes: 132 additions & 0 deletions milvus/datadog_checks/milvus/config_models/defaults.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,132 @@
# (C) Datadog, Inc. 2024-present
# All rights reserved
# Licensed under a 3-clause BSD style license (see LICENSE)

# This file is autogenerated.
# To change this file you should edit assets/configuration/spec.yaml and then run the following commands:
# ddev -x validate config -s <INTEGRATION_NAME>
# ddev -x validate models -s <INTEGRATION_NAME>


def shared_skip_proxy():
return False


def shared_timeout():
return 10


def instance_allow_redirects():
return True


def instance_auth_type():
return 'basic'


def instance_cache_metric_wildcards():
return True


def instance_cache_shared_labels():
return True


def instance_collect_counters_with_distributions():
return False


def instance_collect_histogram_buckets():
return True


def instance_disable_generic_tags():
return False


def instance_empty_default_hostname():
return False


def instance_enable_health_service_check():
return True


def instance_histogram_buckets_as_distributions():
return False


def instance_ignore_connection_errors():
return False


def instance_kerberos_auth():
return 'disabled'


def instance_kerberos_delegate():
return False


def instance_kerberos_force_initiate():
return False


def instance_log_requests():
return False


def instance_min_collection_interval():
return 15


def instance_non_cumulative_histogram_buckets():
return False


def instance_persist_connections():
return False


def instance_request_size():
return 16


def instance_skip_proxy():
return False


def instance_tag_by_endpoint():
return True


def instance_telemetry():
return False


def instance_timeout():
return 10


def instance_tls_ignore_warning():
return False


def instance_tls_use_host_header():
return False


def instance_tls_verify():
return True


def instance_use_latest_spec():
return False


def instance_use_legacy_auth_encoding():
return True


def instance_use_process_start_time():
return False
Loading
Loading