Skip to content

Remove unused variable readOnly from filesystem_linux.go#1173

Merged
SuperQ merged 1 commit intoprometheus:masterfrom
jeromefroe:jeromefroe/remove-unused-variable
Nov 30, 2018
Merged

Remove unused variable readOnly from filesystem_linux.go#1173
SuperQ merged 1 commit intoprometheus:masterfrom
jeromefroe:jeromefroe/remove-unused-variable

Conversation

@jeromefroe
Copy link
Contributor

The pull request #1002 changed the logic used on Linux servers to determine if a filesystem is
read-only. As a result of this change, the variable readOnly is now unused and can be removed.

Signed-off-by: Jerome Froelich jeromefroelich@hotmail.com

The pull request #1002 changed the logic used on Linux servers to determine if a filesystem is
read-only. As a result of this change, the variable `readOnly` is now unused and can be removed.

Signed-off-by: Jerome Froelich <jeromefroelich@hotmail.com>
Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Good catch, thanks!

@SuperQ SuperQ merged commit 0cb0c4d into prometheus:master Nov 30, 2018
@jeromefroe jeromefroe deleted the jeromefroe/remove-unused-variable branch November 30, 2018 14:00
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
…#1173)

The pull request prometheus#1002 changed the logic used on Linux servers to determine if a filesystem is
read-only. As a result of this change, the variable `readOnly` is now unused and can be removed.

Signed-off-by: Jerome Froelich <jeromefroelich@hotmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
…#1173)

The pull request prometheus#1002 changed the logic used on Linux servers to determine if a filesystem is
read-only. As a result of this change, the variable `readOnly` is now unused and can be removed.

Signed-off-by: Jerome Froelich <jeromefroelich@hotmail.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.

2 participants