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

Reduce the number of tests in test_mount_stress.py #54

Closed
Shwetha-Acharya opened this issue Jan 11, 2024 · 0 comments · Fixed by #55
Closed

Reduce the number of tests in test_mount_stress.py #54

Shwetha-Acharya opened this issue Jan 11, 2024 · 0 comments · Fixed by #55

Comments

@Shwetha-Acharya
Copy link
Collaborator

We could reduce the number of tests we run to a single test with 20 clients - I mean if it works with 20 clients, it will probably also work with 5 clients.

Originally posted by @spuiuk in #38 (comment)

Shwetha-Acharya added a commit to Shwetha-Acharya/sit-test-cases that referenced this issue Jan 11, 2024
With this change we are including one stress test with near maximum values
i.e num_clients=20, num_operations=40, file_size=2**25

Other existing tests with lower values are eliminated as the goal
here should be to test with maximum stress. Tests with lower values are
needless when the test with maximum stress passes.

Fixes: samba-in-kubernetes#54
Signed-off-by: Shwetha K Acharya <[email protected]>
spuiuk pushed a commit that referenced this issue Jan 14, 2024
With this change we are including one stress test with near maximum values
i.e num_clients=20, num_operations=40, file_size=2**25

Other existing tests with lower values are eliminated as the goal
here should be to test with maximum stress. Tests with lower values are
needless when the test with maximum stress passes.

Fixes: #54
Signed-off-by: Shwetha K Acharya <[email protected]>
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 a pull request may close this issue.

1 participant