Skip to content

Comments

Release 1.3.0#2168

Merged
SuperQ merged 1 commit intomasterfrom
superq/1.3.0
Nov 18, 2021
Merged

Release 1.3.0#2168
SuperQ merged 1 commit intomasterfrom
superq/1.3.0

Conversation

@SuperQ
Copy link
Member

@SuperQ SuperQ commented Oct 11, 2021

NOTE: In order to support globs in the textfile collector path, filenames exposed by
node_textfile_mtime_seconds now contain the full path name.

Signed-off-by: Ben Kochie superq@gmail.com

Copy link
Member

@roidelapluie roidelapluie left a comment

Choose a reason for hiding this comment

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

I just wonder if we should rename the os collector to os-release

@SuperQ
Copy link
Member Author

SuperQ commented Oct 12, 2021

@roidelapluie I was actually thinking about moving the os collector into the uname collector.

@roidelapluie
Copy link
Member

@roidelapluie I was actually thinking about moving the os collector into the uname collector.

either way is fine, however it is not really uname and it could confuse users. We could merge osrelease and uname in a unique os collector.

@discordianfish
Copy link
Member

discordianfish commented Oct 13, 2021

+1 for merging os and uname into a os collector.

@SuperQ SuperQ force-pushed the superq/1.3.0 branch 3 times, most recently from e8fe4d5 to 0da795a Compare October 19, 2021 10:10
@SuperQ SuperQ force-pushed the superq/1.3.0 branch 4 times, most recently from 0c997d5 to 1fb2dc5 Compare October 27, 2021 12:16
@SuperQ SuperQ marked this pull request as ready for review October 27, 2021 12:31
@SuperQ SuperQ requested a review from discordianfish October 27, 2021 12:31
@discordianfish
Copy link
Member

We should make sure these are fixed before releasing:

@ventifus
Copy link
Contributor

We are good on #2185, we currently have enough tools to work around the issue there.

@bdrung
Copy link
Contributor

bdrung commented Oct 29, 2021

This pull request misses to mention the new collector: Lnstat: expose metrics from /proc/net/stat (#1771)

@hodgesds
Copy link
Contributor

hodgesds commented Nov 1, 2021

I'd really like to sneak #2190 in if possible.

@yeganeahmadnejad
Copy link

when it's going to be released ?

@discordianfish
Copy link
Member

@SuperQ Can we include the clocksource change?

@SuperQ
Copy link
Member Author

SuperQ commented Nov 15, 2021

@discordianfish Yup, now that we have the additional ethtool fixes in, I'll sync everything up and ping you for review.

@bdrung
Copy link
Contributor

bdrung commented Nov 15, 2021

Can we also have #2188 in the release?

NOTE: In order to support globs in the textfile collector path, filenames exposed by
      `node_textfile_mtime_seconds` now contain the full path name.

* [CHANGE] Add path label to rapl collector #2146
* [CHANGE] Exclude filesystems under /run/credentials #2157
* [FEATURE] Add lnstat collector for metrics from  /proc/net/stat/ #1771
* [FEATURE] Add darwin powersupply collector #1777
* [FEATURE] Add support for monitoring GPUs on Linux #1998
* [FEATURE] Add Darwin thermal collector #2032
* [FEATURE] Add os release collector #2094
* [FEATURE] Add netdev.address-info collector #2105
* [ENHANCEMENT] Support glob textfile collector directories #1985
* [ENHANCEMENT] ethtool: Expose node_ethtool_info metric #2080
* [ENHANCEMENT] Use include/exclude flags for ethtool filtering #2165
* [ENHANCEMENT] Add flag to disable guest CPU metrics #2123
* [ENHANCEMENT] Add DMI collector #2131
* [ENHANCEMENT] Add threads metrics to processes collector #2164
* [ENHANCMMENT] Reduce timer GC delays in the Linux filesystem collector #2169
* [BUGFIX] ethtool: Sanitize metric names #2093
* [BUGFIX] Fix ethtool collector for multiple interfaces #2126
* [BUGFIX] Fix possible panic on macOS #2133
* [BUGFIX] Collect flag_info and bug_info only for one core #2156

Signed-off-by: Ben Kochie <superq@gmail.com>
@SuperQ
Copy link
Member Author

SuperQ commented Nov 18, 2021

Ok, I think we're ready to go. Everyone please review for final release.

@SuperQ SuperQ requested a review from roidelapluie November 18, 2021 08:44
@bdrung
Copy link
Contributor

bdrung commented Nov 18, 2021

Looks good from my side.

Copy link
Member

@roidelapluie roidelapluie left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Member

@discordianfish discordianfish left a comment

Choose a reason for hiding this comment

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

LGTM :shipit: 🚀

@SuperQ SuperQ merged commit c65f870 into master Nov 18, 2021
@SuperQ SuperQ deleted the superq/1.3.0 branch November 18, 2021 15:11
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
NOTE: In order to support globs in the textfile collector path, filenames exposed by
      `node_textfile_mtime_seconds` now contain the full path name.

* [CHANGE] Add path label to rapl collector prometheus#2146
* [CHANGE] Exclude filesystems under /run/credentials prometheus#2157
* [FEATURE] Add lnstat collector for metrics from  /proc/net/stat/ prometheus#1771
* [FEATURE] Add darwin powersupply collector prometheus#1777
* [FEATURE] Add support for monitoring GPUs on Linux prometheus#1998
* [FEATURE] Add Darwin thermal collector prometheus#2032
* [FEATURE] Add os release collector prometheus#2094
* [FEATURE] Add netdev.address-info collector prometheus#2105
* [ENHANCEMENT] Support glob textfile collector directories prometheus#1985
* [ENHANCEMENT] ethtool: Expose node_ethtool_info metric prometheus#2080
* [ENHANCEMENT] Use include/exclude flags for ethtool filtering prometheus#2165
* [ENHANCEMENT] Add flag to disable guest CPU metrics prometheus#2123
* [ENHANCEMENT] Add DMI collector prometheus#2131
* [ENHANCEMENT] Add threads metrics to processes collector prometheus#2164
* [ENHANCMMENT] Reduce timer GC delays in the Linux filesystem collector prometheus#2169
* [BUGFIX] ethtool: Sanitize metric names prometheus#2093
* [BUGFIX] Fix ethtool collector for multiple interfaces prometheus#2126
* [BUGFIX] Fix possible panic on macOS prometheus#2133
* [BUGFIX] Collect flag_info and bug_info only for one core prometheus#2156

Signed-off-by: Ben Kochie <superq@gmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
NOTE: In order to support globs in the textfile collector path, filenames exposed by
      `node_textfile_mtime_seconds` now contain the full path name.

* [CHANGE] Add path label to rapl collector prometheus#2146
* [CHANGE] Exclude filesystems under /run/credentials prometheus#2157
* [FEATURE] Add lnstat collector for metrics from  /proc/net/stat/ prometheus#1771
* [FEATURE] Add darwin powersupply collector prometheus#1777
* [FEATURE] Add support for monitoring GPUs on Linux prometheus#1998
* [FEATURE] Add Darwin thermal collector prometheus#2032
* [FEATURE] Add os release collector prometheus#2094
* [FEATURE] Add netdev.address-info collector prometheus#2105
* [ENHANCEMENT] Support glob textfile collector directories prometheus#1985
* [ENHANCEMENT] ethtool: Expose node_ethtool_info metric prometheus#2080
* [ENHANCEMENT] Use include/exclude flags for ethtool filtering prometheus#2165
* [ENHANCEMENT] Add flag to disable guest CPU metrics prometheus#2123
* [ENHANCEMENT] Add DMI collector prometheus#2131
* [ENHANCEMENT] Add threads metrics to processes collector prometheus#2164
* [ENHANCMMENT] Reduce timer GC delays in the Linux filesystem collector prometheus#2169
* [BUGFIX] ethtool: Sanitize metric names prometheus#2093
* [BUGFIX] Fix ethtool collector for multiple interfaces prometheus#2126
* [BUGFIX] Fix possible panic on macOS prometheus#2133
* [BUGFIX] Collect flag_info and bug_info only for one core prometheus#2156

Signed-off-by: Ben Kochie <superq@gmail.com>
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.

7 participants