CDCgov / trusted-intermediary

Bringing together healthcare providers by reducing the connection burden.
Apache License 2.0
11 stars 5 forks source link

Remove dynamic memory alert since it is unhelpful #1607

Closed somesylvie closed 3 days ago

somesylvie commented 6 days ago

Description

We added dynamic memory alerts as an experiment, in addition to static memory alerts. The dynamic ones are creating false positive alerts, so we're going to delete them and just keep the static ones

Issue

https://github.com/CDCgov/trusted-intermediary/issues/1606

github-actions[bot] commented 6 days ago

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

**🎫 Ticket compliance analysis 🔶** **[1606](https://github.com/CDCgov/trusted-intermediary/issues/1606) - Partially compliant** Fully compliant requirements: - Delete dynamic memory alert from TI Not compliant requirements: - Delete dynamic memory alert from ingestion service
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected
github-actions[bot] commented 6 days ago

PR Code Suggestions ✨

No code suggestions found for the PR.

sonarcloud[bot] commented 6 days ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud