Skip to content

Conversation

@nakshatris
Copy link
Collaborator

Moved the required files from reporting module under api, and removed the reporting module.

Please make sure these check boxes are checked before submitting

  • [Y] ** Squashed Commits **
  • [Y] ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.



<dependencies>
<dependency>
Copy link
Collaborator Author

@nakshatris nakshatris May 23, 2025

Choose a reason for hiding this comment

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

@Zakaria-Kofiro : do we need this?
I see it in your PR, but I couldn't find a real need.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed - part of setting up a new central reporting module (CW, S3)

@kevin-mcgoldrick
Copy link
Collaborator

This PR appear to remove the files that send call performance metrics.

  • Cloudwatch (QA)
  • S3 -> Wavefront (PROD, CANADA and PARTNER)

@nakshatris
Copy link
Collaborator Author

After discussion with @kevin-mcgoldrick , I will be keeping the reporting module as-is to avoid risking removing important required files.

@nakshatris nakshatris closed this May 23, 2025
@nakshatris nakshatris deleted the feature/SRE-35204-remove-reporting-module branch June 17, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants