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

Add dtrack.vuln-analysis.result.processed topic #1166

Merged
merged 2 commits into from
Apr 24, 2024
Merged

Add dtrack.vuln-analysis.result.processed topic #1166

merged 2 commits into from
Apr 24, 2024

Conversation

nscuro
Copy link
Member

@nscuro nscuro commented Mar 29, 2024

Adds the dtrack.vuln-analysis.result.processed topic.

As introduced in DependencyTrack/hyades-apiserver#637

Also removes Kafka Streams related environment variables for API server from docker-compose.yml

This also updates the Grafana dashboard to capture the new processor's duration metrics:

image

@nscuro nscuro added the enhancement New feature or request label Mar 29, 2024
@nscuro nscuro added this to the 0.5.0 milestone Mar 29, 2024
@nscuro nscuro added the blocked label Mar 29, 2024
VinodAnandan
VinodAnandan previously approved these changes Mar 31, 2024
Copy link
Collaborator

@VinodAnandan VinodAnandan left a comment

Choose a reason for hiding this comment

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

LGTM

nscuro added 2 commits April 13, 2024 19:21
As introduced in DependencyTrack/hyades-apiserver#637

Also removes Kafka Streams related environment variables for API server from `docker-compose.yml`

Signed-off-by: nscuro <[email protected]>
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

nscuro added a commit that referenced this pull request Apr 13, 2024
Leverages histograms to calculate p99, p95, p90, p75, and p50 percentiles of event processing durations.

Based on #1166, metrics for the new processors are already included.

Depends on DependencyTrack/hyades-apiserver#652

Signed-off-by: nscuro <[email protected]>
@nscuro nscuro removed the blocked label Apr 24, 2024
@nscuro
Copy link
Member Author

nscuro commented Apr 24, 2024

Unblocked since API server PR was merged.

Copy link
Collaborator

@VinodAnandan VinodAnandan left a comment

Choose a reason for hiding this comment

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

LGTM

@nscuro nscuro merged commit e517e93 into main Apr 24, 2024
11 checks passed
@nscuro nscuro deleted the issue-907-4 branch April 24, 2024 10:34
nscuro added a commit that referenced this pull request Apr 26, 2024
Leverages histograms to calculate p99, p95, p90, p75, and p50 percentiles of event processing durations.

Based on #1166, metrics for the new processors are already included.

Depends on DependencyTrack/hyades-apiserver#652

Signed-off-by: nscuro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants