Skip to content

Expose version info and install time#31

Open
Janni2006 wants to merge 5 commits intomainfrom
version_metrics
Open

Expose version info and install time#31
Janni2006 wants to merge 5 commits intomainfrom
version_metrics

Conversation

@Janni2006
Copy link
Contributor

The version number and time of installation can now be set in the configuration file. The data is eposed in the node exporter unter the labels "version_info" and "installed_timestamp".

Loaded from configuration file and exposed in prometheus exporter
# Application version
version: 0.1.13
# Installation timestamp (unix epoch). If left blank, it will be set on first run
installed: 1767962682
Copy link
Member

Choose a reason for hiding this comment

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

shouldn´t this be left blank then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As the configuration file isn´t being modified on the displays themselves, the comment was wrong. I planned on doing it but thought that it wouldn´t be the best option. Now 0 is shown if no installation date is provided

@Janni2006
Copy link
Contributor Author

The current software version should be included in the built files (should be specified within the go files) and not through the configuration file to prevent later modification.

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