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

chore: update/fixes to support large numbers of volumes and pods #115

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

blaisedias
Copy link
Contributor

  • Add helper function VerifyMayastorResourceUsageIsZero use this wait (with timeout) for resource usage to drop before completing a test case. Useful when large numbers of volumes and pods have been created.
  • Update CreatePVC to check that pvc returned is not nil
  • fix ScanFioPodLogs
    • add timeout value to wait for pods to transition from pending
      • default to 60 for most callers - as before
      • set to 1000 for MonitorE2EFioPod
    • refresh pod when checking status

- Add helper function VerifyMayastorResourceUsageIsZero
  use this wait (with timeout) for resource usage to drop before
  completing a test case. Useful when large numbers of volumes
  and pods have been created.
- Update CreatePVC to check that pvc returned is not nil
- fix ScanFioPodLogs
  - add timeout value to wait for pods to transition from
    pending
    - default to 60 for most callers - as before
    - set to 1000 for MonitorE2EFioPod
  - refresh pod when checking status

Signed-off-by: Blaise Dias <[email protected]>
@rohan2794 rohan2794 requested review from r1jt and removed request for harshshekhar15 January 7, 2025 12:37
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