Skip to content

Add support for pushgateway#92

Open
alexmorbo wants to merge 8 commits into
tigerblue77:masterfrom
alexmorbo:master
Open

Add support for pushgateway#92
alexmorbo wants to merge 8 commits into
tigerblue77:masterfrom
alexmorbo:master

Conversation

@alexmorbo
Copy link
Copy Markdown

This Pull Request, introduced functionality to export temperature data to the Pushgateway service. This enhancement is aimed at improving our monitoring capabilities by allowing us to visualize and analyze CPU temperature metrics seamlessly.

Description of Changes:

  1. Feature Addition:

    • Implemented the export of temperature data to Pushgateway.
  2. Environment Variable Configuration:

    • Added a new environment variable PUSH_GATEWAY_URL which must be set to the URL of the Pushgateway service. This enables the application to know where to send the temperature metrics.
  3. Code Integration:

    • Integrated the functionality in the monitoring module, ensuring that the temperature data is sent to the specified Pushgateway service URL.

Steps to Configure:

  1. Ensure that the Pushgateway service is up and running.
  2. Set the PUSH_GATEWAY_URL environment variable with the URL of your Pushgateway service.
    -e PUSH_GATEWAY_URL=http://your-pushgateway-service-url
  3. Run application to apply the new environment settings.

Testing:

  • Verified that the temperature data is successfully exported to and accepted by the Pushgateway service.
  • Ensured that the data can be visualized and analyzed using Prometheus.

Thank you for reviewing this PR. Looking forward to your feedback!

@tigerblue77
Copy link
Copy Markdown
Owner

Hello,
Thanks for your work and pull request but I don't want to add any specific code into this project. I understand it would be nice from some use cases to retrieve values from this container but I'm not good enough in dev to know the best way to meet this need without adding any specific into the main code. Maybe by offering an API that could be requested by monitoring apps ?

@alexmorbo
Copy link
Copy Markdown
Author

@tigerblue77 I finally got around to deploying my fork with Pushgateway.

I updated my fork, added tags, created a Helm chart, and now I can easily deploy the service to Kubernetes.

It looks very convenient.
image

@tigerblue77 tigerblue77 reopened this Jan 6, 2025
@tigerblue77 tigerblue77 force-pushed the master branch 3 times, most recently from e6a2df9 to 0230b48 Compare January 26, 2025 16:30
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.

2 participants