Skip to content

Conversation

@AncientPatata
Copy link
Contributor

@AncientPatata AncientPatata commented Jul 8, 2025

Motivation

  • The exporter module was previously in the Infra but you had to add it in yourself to the reference deployment to use it.

Description

Added the MongoDB Exporter module to the reference AWS and localhost deployments.

Testing

Tested the different MongoDB deployments with the exporter (localhost,aws) (non-sharded on prem, sharded on prem, atlas). The Atlas MongoDB deployment was however only tested in localhost and not in an AWS deployment.

Impact

Given the relative simplicity of the database there shouldn't be much overhead when monitoring it. Still, moving up to v0.44 for the exporter comes with some performance improvements, fixes to bugs that made the exporter consume too much memory as well as addressing a major security vulnerability.

Additional Information

  • This would also serve as a "how to deploy the mongodb exporter in your custom ArmoniK deployment" since it wasn't documented before.
  • This PR depends on ArmoniK.Infra#250

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have thoroughly tested my modifications and added tests when necessary.
  • Tests pass locally and in the CI.
  • I have assessed the performance impact of my modifications.

@AncientPatata AncientPatata force-pushed the ad/mongodb-exporter branch 2 times, most recently from 1d45415 to c31a8e9 Compare July 8, 2025 15:12
@AncientPatata
Copy link
Contributor Author

Deploy_localhost_AllInOne / coreStream (push) failing is not related to my PR

@AncientPatata AncientPatata force-pushed the ad/mongodb-exporter branch 3 times, most recently from ea1f246 to ee0eeee Compare July 18, 2025 07:47
Copy link
Contributor

@lemaitre-aneo lemaitre-aneo left a comment

Choose a reason for hiding this comment

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

You need to change the infra version to 0.12.4 in versions.tfvars.json

@AncientPatata AncientPatata force-pushed the ad/mongodb-exporter branch 2 times, most recently from 86500c2 to 5ddde79 Compare July 29, 2025 07:00
lemaitre-aneo
lemaitre-aneo previously approved these changes Jul 29, 2025
lemaitre-aneo
lemaitre-aneo previously approved these changes Jul 31, 2025
@sonarqubecloud
Copy link

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