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

Fix PowerMax Integration test failures #340

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Conversation

Sahiba-Gupta
Copy link
Contributor

@Sahiba-Gupta Sahiba-Gupta commented Oct 15, 2024

Description
This change will solve the integration test failures for first 16 test cases.

| GitHub Issue # |
dell/csm#1519

Checklist:

  • Have you run format,vet & lint checks against your submission?
  • Have you made sure that the code compiles?
  • Did you run the unit & integration tests successfully?
  • Have you maintained at least 90% code coverage?
  • Have you commented your code, particularly in hard-to-understand areas
  • Have you done corresponding changes to the documentation
  • Did you run tests in a real Kubernetes cluster?
  • Backward compatibility is not broken

I have tested this change locally . Attaching logs and xml file.
success_logs.txt
success_logs2.txt

<testsuites name="CSI Powermax Int Tests" tests="7" skipped="0" failures="0" errors="0" time="341.480639987">
<testsuite name="Powermax OS CSI interface" tests="7" skipped="0" failures="0" errors="0" time="341.479148273">
<testcase name="Create and delete basic thick large volume" status="passed" time="39.710621473"/>
<testcase name="Create and delete basic volume using Gold Service Level" status="passed" time="39.078588608"/>
<testcase name="Create and delete basic volume using various volume sizes #1" status="passed" time="37.327177476"/>
<testcase name="Create and delete basic volume using various volume sizes #2" status="passed" time="40.012844277"/>
<testcase name="Create and delete basic volume using various volume sizes #3" status="passed" time="39.51262478"/>
<testcase name="Create and delete basic volume using various volume sizes #4" status="passed" time="111.142088921"/>
<testcase name="Create volume, try to delete it with incorrect CSI VolumeID, then delete it." status="passed" time="34.669009988"/>
</testsuite>
</testsuites>

<testsuites name="CSI Powermax Int Tests" tests="9" skipped="0" failures="0" errors="0" time="479.201986255">
<testsuite name="Powermax OS CSI interface" tests="9" skipped="0" failures="0" errors="0" time="479.200364151">
<testcase name="Idempotent create and delete basic volume" status="passed" time="40.745355822"/>
<testcase name="Create and delete mount volume" status="passed" time="35.548136072"/>
<testcase name="Create, publish, unpublish and delete basic volume" status="passed" time="46.74114773"/>
<testcase name="Create publish, node stage, node publish, node unpublish, node unstage, unpublish, delete basic volume" status="passed" time="92.03266463"/>
<testcase name="Idempotent create publish, node publish, node unpublish, unpublish, delete basic volume" status="passed" time="142.922456382"/>
<testcase name="Create and delete basic volume to maximum capacity" status="passed" time="38.422573338"/>
<testcase name="Create and delete basic volume beyond maximum capacity" status="passed" time="1.727726434"/>
<testcase name="Create and delete basic volume of minimum capacity" status="passed" time="40.473970108"/>
<testcase name="Create and delete basic volume below minimum capacity" status="passed" time="40.557421112"/>
</testsuite>
</testsuites>

@Sahiba-Gupta Sahiba-Gupta requested a review from tdawe October 15, 2024 17:11
@Sahiba-Gupta Sahiba-Gupta merged commit 5679a6f into main Oct 15, 2024
5 checks passed
Copy link
Contributor

@abhi16394 abhi16394 left a comment

Choose a reason for hiding this comment

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

LGTM

@Sahiba-Gupta Sahiba-Gupta deleted the pvt/int_test_failure branch October 15, 2024 17:23
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.

4 participants