red-hat-storage / ceph-qe-scripts

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

Enhancement to the sync_status module #546

Closed ckulal closed 8 months ago

ckulal commented 8 months ago

Enhancement to the sync_status module: Include logic to check sync status from remote site Enhancement to the user_create module: Include check of sync status post creating user in case of multi-site setup Enhancement to the bucket_create module: Include check of sync status post creating bucket in case of multi-site setup

Logs: http://magna002.ceph.redhat.com/ceph-qe-logs/Chaithra/sync_status_enhance/

ckulal commented 8 months ago

@ckulal , I would suggest to test with the existing tests where we are doing a remote site sync (bucket stats/object download) etc.

Hi @viduship , This PR contains inclusion of sync status check from remote site post creating user and buckets. currently we are checking sync status only from existing site, so as per this PR only addition is sync check from remote site only in case of user and bucket creation.

ckulal commented 8 months ago

Hi @viduship , With the enhancement provided in this PR.

I tried execution of config: test_Mbuckets_with_Nobjects_0_shards_sync_test_haproxy.yaml by commenting "time.sleep(1200)", https://github.com/red-hat-storage/ceph-qe-scripts/blob/master/rgw/v2/tests/s3_swift/reusable.py#L2117

i.e without waiting extra 20 minutes for sync lease period.

Pass log: http://magna002.ceph.redhat.com/ceph-qe-logs/Chaithra/sync_status_enhance/retry/without-sync-lease-wait

Hence with the changes given in this PR, In future we can try to avoid blindly sleep in some scenarios. that might require some testing effor. I will create backlog for the same

openshift-ci[bot] commented 8 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: anrao19, ckulal, hmaheswa, mkasturi18, viduship

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