Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Analyze command, automatic ID's, global insecure flag, Microsecond time and some cleanup #17

Merged
merged 4 commits into from
Oct 12, 2024

Conversation

zveinn
Copy link
Contributor

@zveinn zveinn commented Oct 11, 2024

This PR does the following:

  • Implements an Analyze command to grab meaningful statistics and errors from test output files
  • Automatically sets the teste ID if one is missing
  • Makes the insecure flag a global flag so that server and client both respect TLS
  • Changes timers to Microseconds instead of Milliseconds
  • Some minor code cleanup and renaming

automatically generating ids and cleanup

Changing timers to Microseconds and adding the analyze command

removing printling
Copy link

@klauspost klauspost left a comment

Choose a reason for hiding this comment

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

Just some minor style things.

cmd/hperf/analyze.go Show resolved Hide resolved
cmd/hperf/analyze.go Show resolved Hide resolved
cmd/hperf/analyze.go Show resolved Hide resolved
go.mod Outdated Show resolved Hide resolved
@harshavardhana harshavardhana merged commit 0ca547e into minio:main Oct 12, 2024
1 check passed
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.

3 participants