From 5a3baf1bd852d6c5d0de10a33839658789edc4eb Mon Sep 17 00:00:00 2001 From: Jason Dillaman Date: Fri, 29 Sep 2017 15:11:38 -0400 Subject: [PATCH] librbd: snapshots should be created/removed against data pool Fixes: http://tracker.ceph.com/issues/21567 Signed-off-by: Jason Dillaman --- src/librbd/operation/SnapshotCreateRequest.cc | 4 ++-- src/librbd/operation/SnapshotRemoveRequest.cc | 2 +- src/test/librbd/operation/test_mock_SnapshotCreateRequest.cc | 4 ++-- src/test/librbd/operation/test_mock_SnapshotRemoveRequest.cc | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/librbd/operation/SnapshotCreateRequest.cc b/src/librbd/operation/SnapshotCreateRequest.cc index d332d870c15d..63ddd8848c4d 100644 --- a/src/librbd/operation/SnapshotCreateRequest.cc +++ b/src/librbd/operation/SnapshotCreateRequest.cc @@ -129,7 +129,7 @@ void SnapshotCreateRequest::send_allocate_snap_id() { librados::AioCompletion *rados_completion = create_rados_callback< SnapshotCreateRequest, &SnapshotCreateRequest::handle_allocate_snap_id>(this); - image_ctx.md_ctx.aio_selfmanaged_snap_create(&m_snap_id, rados_completion); + image_ctx.data_ctx.aio_selfmanaged_snap_create(&m_snap_id, rados_completion); rados_completion->release(); } @@ -255,7 +255,7 @@ void SnapshotCreateRequest::send_release_snap_id() { librados::AioCompletion *rados_completion = create_rados_callback< SnapshotCreateRequest, &SnapshotCreateRequest::handle_release_snap_id>(this); - image_ctx.md_ctx.aio_selfmanaged_snap_remove(m_snap_id, rados_completion); + image_ctx.data_ctx.aio_selfmanaged_snap_remove(m_snap_id, rados_completion); rados_completion->release(); } diff --git a/src/librbd/operation/SnapshotRemoveRequest.cc b/src/librbd/operation/SnapshotRemoveRequest.cc index 4daf9fdfb312..967de95b15ca 100644 --- a/src/librbd/operation/SnapshotRemoveRequest.cc +++ b/src/librbd/operation/SnapshotRemoveRequest.cc @@ -205,7 +205,7 @@ void SnapshotRemoveRequest::send_release_snap_id() { librados::AioCompletion *rados_completion = this->create_callback_completion(); - image_ctx.md_ctx.aio_selfmanaged_snap_remove(m_snap_id, rados_completion); + image_ctx.data_ctx.aio_selfmanaged_snap_remove(m_snap_id, rados_completion); rados_completion->release(); } diff --git a/src/test/librbd/operation/test_mock_SnapshotCreateRequest.cc b/src/test/librbd/operation/test_mock_SnapshotCreateRequest.cc index 845c36fde88b..7124df5b367b 100644 --- a/src/test/librbd/operation/test_mock_SnapshotCreateRequest.cc +++ b/src/test/librbd/operation/test_mock_SnapshotCreateRequest.cc @@ -43,7 +43,7 @@ public: } void expect_allocate_snap_id(MockImageCtx &mock_image_ctx, int r) { - auto &expect = EXPECT_CALL(get_mock_io_ctx(mock_image_ctx.md_ctx), + auto &expect = EXPECT_CALL(get_mock_io_ctx(mock_image_ctx.data_ctx), selfmanaged_snap_create(_)); if (r < 0 && r != -ESTALE) { expect.WillOnce(Return(r)); @@ -53,7 +53,7 @@ public: } void expect_release_snap_id(MockImageCtx &mock_image_ctx, int r) { - auto &expect = EXPECT_CALL(get_mock_io_ctx(mock_image_ctx.md_ctx), + auto &expect = EXPECT_CALL(get_mock_io_ctx(mock_image_ctx.data_ctx), selfmanaged_snap_remove(_)); if (r < 0) { expect.WillOnce(Return(r)); diff --git a/src/test/librbd/operation/test_mock_SnapshotRemoveRequest.cc b/src/test/librbd/operation/test_mock_SnapshotRemoveRequest.cc index 3f5fe186865a..d16e0a4d2dc5 100644 --- a/src/test/librbd/operation/test_mock_SnapshotRemoveRequest.cc +++ b/src/test/librbd/operation/test_mock_SnapshotRemoveRequest.cc @@ -110,7 +110,7 @@ public: } void expect_release_snap_id(MockImageCtx &mock_image_ctx) { - EXPECT_CALL(get_mock_io_ctx(mock_image_ctx.md_ctx), + EXPECT_CALL(get_mock_io_ctx(mock_image_ctx.data_ctx), selfmanaged_snap_remove(_)) .WillOnce(DoDefault()); } -- 2.47.3