FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

e2e: only test disk0 in the Darwin end-to-end test by rtreffer · Pull Request #3825 · prometheus/node_exporter · GitHub

e2e: only test disk0 in the Darwin end-to-end test - #3825

Open
rtreffer wants to merge 1 commit into
prometheus:masterfrom
rtreffer:e2e-darwin-disk0
Open

rtreffer wants to merge 1 commit into
prometheus:masterfrom
rtreffer:e2e-darwin-disk0

Conversation

Copy link
Copy Markdown
Contributor

The Darwin end-to-end test scrapes the real macos-latest runner and diffs the result against collector/fixtures/e2e-output-darwin.txt. We no longer see disk4 on the nodes and that breaks the e2e tests.

This PR removes the usage of disk4 and the related fixtures:

-node_disk_read_errors_total{device="disk4"} 0
-node_disk_read_retries_total{device="disk4"} 0
-node_disk_write_errors_total{device="disk4"} 0
-node_disk_write_retries_total{device="disk4"} 0

This PR also adds a dump of the attached interfaces and disks so we can more easily discover issues in the future.

The Darwin end-to-end test scrapes the real macos-latest runner and diffs
the result against collector/fixtures/e2e-output-darwin.txt. We no longer
see disk4 on the nodes and that breaks the e2e tests.

This PR removes the usage of disk4 and the related fixtures:

  -node_disk_read_errors_total{device="disk4"} 0
  -node_disk_read_retries_total{device="disk4"} 0
  -node_disk_write_errors_total{device="disk4"} 0
  -node_disk_write_retries_total{device="disk4"} 0

This PR also adds a dump of the attached interfaces and disks
so we can more easily discover issues in the future.

Signed-off-by: René Treffer <treffer@measite.de>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL