Skip to content

Fix PowerMax Integration test failures#340

Merged
Sahiba-Gupta merged 1 commit into
mainfrom
pvt/int_test_failure
Oct 15, 2024
Merged

Fix PowerMax Integration test failures#340
Sahiba-Gupta merged 1 commit into
mainfrom
pvt/int_test_failure

Conversation

@Sahiba-Gupta

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

Copy link
Copy Markdown
Contributor

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

| GitHub Issue # |
https://github.com/dell/csm/issues/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>

Comment thread test/integration/features/integration.feature
@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

@abhi16394 abhi16394 left a comment

Copy link
Copy Markdown
Contributor

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