]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
RBD: remove redundant parent check 7174/head
authorxie xingguo <xie.xingguo@zte.com.cn>
Mon, 11 Jan 2016 10:29:01 +0000 (18:29 +0800)
committerxie xingguo <xie.xingguo@zte.com.cn>
Mon, 11 Jan 2016 10:36:02 +0000 (18:36 +0800)
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
src/librbd/internal.cc

index 958be93de5cf8f4e5b39e2d9b456013edea2f1a3..c31da2e9ba2ad55f9866570d13d6a7d9c6b5ca50 100644 (file)
@@ -1950,13 +1950,11 @@ int validate_pool(IoCtx &io_ctx, CephContext *cct) {
     parent_spec parent_spec;
 
     if (ictx->snap_id == CEPH_NOSNAP) {
-      if (!ictx->parent)
-       return -ENOENT;
       parent_spec = ictx->parent_md.spec;
     } else {
       r = ictx->get_parent_spec(ictx->snap_id, &parent_spec);
       if (r < 0) {
-       lderr(ictx->cct) << "Can't find snapshot id" << ictx->snap_id << dendl;
+       lderr(ictx->cct) << "Can't find snapshot id = " << ictx->snap_id << dendl;
        return r;
       }
       if (parent_spec.pool_id == -1)
@@ -1967,7 +1965,7 @@ int validate_pool(IoCtx &io_ctx, CephContext *cct) {
       r = rados.pool_reverse_lookup(parent_spec.pool_id,
                                    parent_pool_name);
       if (r < 0) {
-       lderr(ictx->cct) << "error looking up pool name" << cpp_strerror(r)
+       lderr(ictx->cct) << "error looking up pool name" << cpp_strerror(r)
                         << dendl;
        return r;
       }