-
Notifications
You must be signed in to change notification settings - Fork 758
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
Initial commit of Ocient integration (ECOINT-49) #2546
Open
ocient-cliimatta
wants to merge
20
commits into
DataDog:master
Choose a base branch
from
Xeograph:main-ocientcheck
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
62dd958
initial commit of Ocient integration
ocient-cliimatta e9cd5d4
Merge pull request #1 from Xeograph/Ocient
ocient-cliimatta 40777c9
Merge branch 'DataDog:master' into master
ewozniak-Ocient b438e40
initial-release
ocient-cliimatta c6989ad
Merge branch 'master' into main-ocientcheck
ocient-cliimatta e8fda71
adding github ci
ocient-cliimatta d80135f
fixed metadata csv
ocient-cliimatta 1888dd0
fix formatting
ocient-cliimatta 6c1f101
fix manifest
ocient-cliimatta 580bcca
fixed linter errors
ocient-cliimatta 3f378a7
resolve ruff checks
ocient-cliimatta 7b83ef1
more ruff fixes
ocient-cliimatta bc6ec1e
fix docker hostname
ocient-cliimatta c56b0da
docker test fixes
ocient-cliimatta 51fcc02
fix e2e tests
ocient-cliimatta 5ca5740
fixing manifest
ocient-cliimatta f8e5fbb
fixing metric check in manifest
ocient-cliimatta 80b36d7
more fixes for manifest and metadata
ocient-cliimatta 906e2d8
fixing metric name in manifest
ocient-cliimatta b07a43b
Fixing suggested wording
ocient-cliimatta File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -246,6 +246,7 @@ | |
/emqx/ @zhongwencool @zmstone @DataDog/ecosystems-review | ||
/anecdote/ @atyulyutaev @DataDog/ecosystems-review | ||
/celerdata/ @yandongxiao [email protected] @DataDog/ecosystems-review | ||
/ocient/ @ocient-cliimatta @Xeograph/service-delivery-engineering | ||
|
||
# Community Partners + Documentation | ||
/1e/*metadata.csv [email protected] @DataDog/documentation @DataDog/ecosystems-review | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
# CHANGELOG - Ocient | ||
|
||
## 1.0.0 / 2024-11-22 | ||
|
||
***Added***: | ||
|
||
* Initial Release | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,66 @@ | ||
# Agent Check: Ocient | ||
|
||
## Overview | ||
|
||
[Ocient][1] Hyperscale Data Warehouse is a data analytics software solutions company that enables all-the-time, compute-intensive analysis of large, complex datasets while optimizing for performance, cost, and energy efficiency. | ||
|
||
With industry-standard interfaces like SQL and JDBC, Ocient makes it easy for organizations to interact with data within its platform. | ||
The Ocient integration enables you to collect metrics related to query performance, disk usage, database tables, and more. | ||
|
||
## Setup | ||
|
||
Follow these instructions to install and configure this check for an Agent running on a host. | ||
|
||
### Installation | ||
|
||
The Ocient check is included in the [Datadog Agent][2] package, so you don't need to install anything else on your Ocient server. | ||
|
||
### Configuration | ||
|
||
To configure this check for an Agent running on a host: | ||
|
||
1. Edit the `ocient.d/conf.yaml` file, in the `conf.d/` folder at the root of your [Agent's configuration directory][3]. For all available configuration options, see the [sample ocient.d/conf.yaml][4]. | ||
|
||
```yaml | ||
instances: | ||
|
||
- use_openmetrics: true # Enables OpenMetrics V2 | ||
|
||
## @param openmetrics_endpoint - string - required | ||
## The URL exposing metrics in the OpenMetrics format. | ||
# | ||
openmetrics_endpoint: http://localhost:<PORT>/metrics | ||
``` | ||
|
||
2. [Restart the Agent][5]. | ||
|
||
### Validation | ||
|
||
[Run the Agent's status subcommand][6] and look for `ocient` under the Checks section. | ||
|
||
## Data Collected | ||
|
||
### Metrics | ||
|
||
See [metadata.csv][7] for a list of metrics provided by this integration. | ||
|
||
### Service Checks | ||
|
||
Ocient does not include any service checks. | ||
|
||
### Events | ||
|
||
Ocient does not include any events. | ||
|
||
## Troubleshooting | ||
|
||
Need help? Contact [Datadog support][8]. | ||
|
||
[1]: https://ocient.com/ | ||
[2]: https://app.datadoghq.com/account/settings/agent/latest | ||
[3]: https://docs.datadoghq.com/agent/guide/agent-configuration-files/#agent-configuration-directory | ||
[4]: https://github.com/DataDog/integrations-extras/blob/master/ocient/datadog_checks/ocient/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-extras/blob/master/ocient/metadata.csv | ||
[8]: https://docs.datadoghq.com/help/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
name: Ocient | ||
files: | ||
- name: ocient.yaml | ||
options: | ||
- template: init_config | ||
options: | ||
- template: init_config/openmetrics | ||
- template: instances | ||
options: | ||
- template: instances/openmetrics | ||
options: | ||
overrides: | ||
openmetrics_endpoint.value.example: http://localhost:9090/metrics/ | ||
openmetrics_endpoint.display_priority: 1 |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please include:
|
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
[] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
__path__ = __import__('pkgutil').extend_path(__path__, __name__) # type: ignore |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
__version__ = '1.0.0' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
from .__about__ import __version__ | ||
from .check import OcientCheck | ||
|
||
__all__ = ['__version__', 'OcientCheck'] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
from datadog_checks.base import OpenMetricsBaseCheckV2 | ||
|
||
from .metrics import METRIC_MAP | ||
|
||
|
||
class OcientCheck(OpenMetricsBaseCheckV2): | ||
__NAMESPACE__ = "ocient" | ||
|
||
DEFAULT_METRIC_LIMIT = 0 | ||
|
||
def __init__(self, name, init_config, instances): | ||
super(OcientCheck, self).__init__(name, init_config, instances) | ||
self.openmetrics_endpoint = self.instance.get("openmetrics_endpoint") | ||
|
||
def get_default_config(self): | ||
default_config = { | ||
"openmetrics_endpoint": self.openmetrics_endpoint, | ||
"metrics": METRIC_MAP, | ||
} | ||
|
||
return default_config |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
# 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 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,128 @@ | ||
# 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 |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add 1-2 sentences about what value this brings to our mutual customers?