red-hat-storage / ceph-qe-scripts

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

Increase the sync lease to 1200 seconds for 0 shards sync test #506

Closed viduship closed 11 months ago

viduship commented 11 months ago

Passed log: http://magna002.ceph.redhat.com/cephci-jenkins/cephci-run-Y2H8HK/

ckulal commented 11 months ago

Hi @viduship , In a log provided, lease period is still in 120sec, Wait for the sync lease period of 120 seconds'

viduship commented 11 months ago

Hi @viduship , In a log provided, lease period is still in 120sec, Wait for the sync lease period of 120 seconds'

@ckulal yeah, that message was changed later, after the run.

please also note the execution time in the logs for a test which is 46 minutes (including the 1200 seconds sleep time.) http://magna002.ceph.redhat.com/cephci-jenkins/cephci-run-MV1UXH/

I am triggering a new log, that displays the correct message related to the sync lease period

http://magna002.ceph.redhat.com/cephci-jenkins/cephci-run-Y2H8HK/

viduship commented 11 months ago

@ckulal , new passed log with the correct message for sync lease period http://magna002.ceph.redhat.com/cephci-jenkins/cephci-run-Y2H8HK/

ckulal commented 11 months ago

@ckulal , new passed log with the correct message for sync lease period http://magna002.ceph.redhat.com/cephci-jenkins/cephci-run-Y2H8HK/

Hi @viduship , small doubt, here I could see conf dump for rgw_sync_lease_period as 120 {"section":"client.rgw.shared.pri","name":"rgw_sync_lease_period","value":"120","level":"dev","can_update_at_runtime":true,"mask":""},

what does this value represents? It is integrated in all ms suites, but it is not taking effect. the sync happens after 1200 seconds

openshift-ci[bot] commented 11 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 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