red-hat-storage / ceph-qe-scripts

Various QE scripts written by QE for Ceph Testing
MIT License
10 stars 29 forks source link

Test Checksum algorithms #636

Open TejasC88 opened 2 weeks ago

TejasC88 commented 2 weeks ago

Polarion ID : CEPH-83591699 Feature : Additional S3 checksums Pass log : http://magna002.ceph.redhat.com/cephci-jenkins/tchandra/test_checksum.txt

openshift-ci[bot] commented 2 weeks ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: TejasC88

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/red-hat-storage/ceph-qe-scripts/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
TejasC88 commented 2 weeks ago

Adding DNM flag , since I will be updating the same script with further changes

TejasC88 commented 1 week ago

Removed the DNM flag. Added support for CEPH-83591679 as well. Known BZ will be fixed in 8.1 : https://bugzilla.redhat.com/show_bug.cgi?id=2310424 Fail log : http://magna002.ceph.redhat.com/cephci-jenkins/tchandra/test_checksum_api.txt

Log for existing multipart test case which will be affected by changes to reusables.py http://magna002.ceph.redhat.com/cephci-jenkins/tchandra/test_aws_multipart.txt

TejasC88 commented 4 days ago

The BZ for checksum of multipart objects is fixed now , so adding the DNM label until script is modified for the expected behaviour

TejasC88 commented 3 days ago

BZ https://bugzilla.redhat.com/show_bug.cgi?id=2310424 is verified now , so modified scripts according to the expected behaviour , attaching pass logs here : http://magna002.ceph.redhat.com/cephci-jenkins/tchandra/test_checksum.txt http://magna002.ceph.redhat.com/cephci-jenkins/tchandra/test_checksum_api.txt

removing DNM flag