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.

Sign up for GitHub

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

Jump to bottom

[Perf, Stress] Command-line test name should be matched case-insensitive #16850

Closed
mikeharder opened this issue Nov 10, 2020 · 0 comments · Fixed by #16965
Closed

[Perf, Stress] Command-line test name should be matched case-insensitive #16850

mikeharder opened this issue Nov 10, 2020 · 0 comments · Fixed by #16965
Assignees
@mikeharder
Labels
Central-EngSys This issue is owned by the Engineering System team. pillar-reliability The issue is related to reliability, one of our core engineering pillars. (includes stress testing)

Comments

@mikeharder
Copy link
Member

Currently, the command-line argument for the test name is treated case-sensitive and must be lowercase. This should be matched case-insensitive if possible.
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 10, 2020
@mikeharder mikeharder self-assigned this Nov 10, 2020
@mikeharder mikeharder added the pillar-reliability The issue is related to reliability, one of our core engineering pillars. (includes stress testing) label Nov 10, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 10, 2020
@mikeharder mikeharder added Central-EngSys This issue is owned by the Engineering System team. needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. and removed pillar-reliability The issue is related to reliability, one of our core engineering pillars. (includes stress testing) labels Nov 10, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 10, 2020
@mikeharder mikeharder added the pillar-reliability The issue is related to reliability, one of our core engineering pillars. (includes stress testing) label Nov 10, 2020
mikeharder added a commit to mikeharder/azure-sdk-for-net that referenced this issue Nov 11, 2020
@mikeharder
Case-insensitive command-line parsing
6dffeab 
- Fixes Azure#16850
@mikeharder mikeharder changed the title [Stress] Command-line test name should be matched case-insensitive [Perf, Stress] Command-line test name should be matched case-insensitive Nov 11, 2020
@mikeharder mikeharder mentioned this issue Nov 13, 2020
Merged
mikeharder added a commit that referenced this issue Nov 17, 2020
@mikeharder
[Perf, Stress] Case-insensitive command-line parsing (#16965)
8153513 
- Perf and Stress frameworks should use case-insensitive command-line parsing
- Refactor common code into PerfStressUtilities shared source
- Add PerfOptions.StatusInterval to align with StressOptions
- Fixes #16850
annelo-msft pushed a commit to annelo-msft/azure-sdk-for-net that referenced this issue Feb 17, 2021
@mikeharder @annelo-msft
[Perf, Stress] Case-insensitive command-line parsing (Azure#16965)
4d524b5 
- Perf and Stress frameworks should use case-insensitive command-line parsing
- Refactor common code into PerfStressUtilities shared source
- Add PerfOptions.StatusInterval to align with StressOptions
- Fixes Azure#16850
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mikeharder mikeharder

Labels
Central-EngSys This issue is owned by the Engineering System team. pillar-reliability The issue is related to reliability, one of our core engineering pillars. (includes stress testing)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Perf, Stress] Case-insensitive command-line parsing mikeharder/azure-sdk-for-net
1 participant
@mikeharder