Skip to content

Conversation

anish-mudaraddi
Copy link
Collaborator

@anish-mudaraddi anish-mudaraddi commented Aug 22, 2025

  • add source code for monitoring tools
  • bundle together multiple monitoring scripts into one python package - cloudMonitoring
  • add docker image
  • use pyproject.toml as modern method of setting up a python project

benefits:

  • single dockerized image for running monitoring scripts - can run anywhere - including as cron jobs on our K8s clusters
  • extensible - able to write further monitoring scripts and tie into the package fairly easily

future improvements:

  • Need to decouple scripts from influxdb as the service is likely to be deprecated - replacing with victoria metrics

@anish-mudaraddi anish-mudaraddi marked this pull request as ready for review August 28, 2025 09:06
@anish-mudaraddi anish-mudaraddi force-pushed the slottifier-metrics-collection branch from 61057c4 to 174aa9d Compare August 28, 2025 10:19
- add source code for monitoring tools
- bundle together multiple monitoring scripts into one python package - cloudMonitoring
- add docker image

benefits:
- single dockerized image for running monitoring scripts - can run anywhere - including as cron jobs on our K8s clusters
-  extensible - able to write further monitoring scripts and tie into the package fairly easily

future improvements:

- Need to decouple scripts from influxdb as the service is likely to be deprecated - replacing with victoria metrics
updates to openstack mean that just using list_hypervisors command won't return usage data - this functionality is possible with our query library - so I've updated the code to use it
we'll only ever need one config file so move it out of its own folder

document the way to install and use the package

change the command registry names to be clearer as to what each script does
default fp is hardcoded to `monitoring.conf` and will be searched for in the current directory
@anish-mudaraddi anish-mudaraddi force-pushed the slottifier-metrics-collection branch from ae0502a to 2dbbecd Compare August 29, 2025 12:21
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