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

[FEATURE]: Integrated DataDog Api in Dora Metrics #145

Conversation

maelaubert56
Copy link
Member

[FEATURE]: Integrated DataDog Api in Dora Metrics

📄 Description

In this PR, we introduced a new MonitoringApi that retrieves MonitoringEvents using the DataDog API.
We replaced the previous mock event data with this API.
The database has been updated to include additional configurations, such as DataDog API keys, URL, and monitoring ID.
A new function has been added to DateUtils for timestamp conversion.
Tests in DevOps Auditor have been updated, and a new test file has been added to the core for MonitoringUtils.

✨ Commits

  • [FEATURE]: add MonitoringApi to fetch and convert DataDog events, update ApplicationModel and types for DataDog configuration
  • [FEATURE]: enable JSON module resolution in TypeScript and add tests for MonitoringUtils conversion of DataDog events
  • [FEATURE]: integrate MonitoringApi to fetch monitoring events and update DoraMetrics processing
  • [FEATURE]: update DoraMetricsUtils tests to use mockMonitoringEvents and adjust score calculations

Maël AUBERT added 4 commits March 12, 2025 10:53
Copy link
Member

@helabenkhalfallah helabenkhalfallah left a comment

Choose a reason for hiding this comment

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

Perfect!

@helabenkhalfallah helabenkhalfallah merged commit 692b859 into main Mar 12, 2025
1 check passed
@helabenkhalfallah helabenkhalfallah deleted the feature/ISSUE-115/integrate-datadog-api-in-devops-auditor branch March 12, 2025 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants