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

jewel: rgw: multisite: memory leak on failed lease in RGWDataSyncShardCR #15457

Merged
merged 3 commits into from Aug 26, 2017

Commits on Jun 16, 2017

  1. rgw: dont spawn error_repo until lease is acquired

    if RGWDataSyncShardCR fails to acquire its lease, it doesn't call
    error_repo->finish() to stop the coroutine. wait until the lease
    succeeds before spawning the error_repo
    
    Fixes: http://tracker.ceph.com/issues/19446
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    (cherry picked from commit 1524a5e)
    
    Conflicts:
            src/rgw/rgw_data_sync.cc (different argument list of RGWOmapAppend()
                function in jewel, compared to master)
    cbodley authored and smithfarm committed Jun 16, 2017
    Copy the full SHA
    9b12fe0 View commit details
    Browse the repository at this point in the history
  2. rgw: should delete in_stream_req if conn->get_obj(...) return not zer…

    …o value
    
    Signed-off-by: weiqiaomiao <wei.qiaomiao@zte.com.cn>
    (cherry picked from commit 494fbe9)
    weiqiaomiao authored and smithfarm committed Jun 16, 2017
    Copy the full SHA
    730c88e View commit details
    Browse the repository at this point in the history
  3. rgw: RGWPeriodPusher spawns http thread before cr thread

    Fixes: http://tracker.ceph.com/issues/19834
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    (cherry picked from commit 21a943d)
    cbodley authored and smithfarm committed Jun 16, 2017
    Copy the full SHA
    8cf3f18 View commit details
    Browse the repository at this point in the history