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

rados/singleton-nomsgr/all/11439: fix greps #1076

Closed
wants to merge 1 commit into from
Closed

Conversation

tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Jul 2, 2016

because, in OSD.cc

  • if (!osdmap->have_pg_pool(pgid.pool())) {
  •  dout(10) << **func** << ": skipping PG " << pgid << " because we
    
    don't have pool "
  •      << pgid.pool() << dendl;
    
  •  continue;
    
  • }
  • if (!osdmap->have_pg_pool(pgid.pool())) {
  • derr << **func** << ": could not find map for epoch " << map_epoch
    
  •      << " on pg " << pgid << ", but the pool is not present in the"
    
  •      << "current map, so this is probably a result of bug 10617."
    
  •      << "Skipping the pg for now, you can use ceph-objectstore-tool "
    
  •      << "to clean it up later." << dendl;
    
  • continue;
    
  • } else {

Fixes: http://tracker.ceph.com/issues/16157
Signed-off-by: Kefu Chai kchai@redhat.com

because, in OSD.cc
-    if (!osdmap->have_pg_pool(pgid.pool())) {
-      dout(10) << __func__ << ": skipping PG " << pgid << " because we
       don't have pool "
-	       << pgid.pool() << dendl;
-      continue;
-    }
+	if (!osdmap->have_pg_pool(pgid.pool())) {
+	  derr << __func__ << ": could not find map for epoch " << map_epoch
+	       << " on pg " << pgid << ", but the pool is not present in the"
+	       << "current map, so this is probably a result of bug 10617."
+	       << "Skipping the pg for now, you can use ceph-objectstore-tool "
+	       << "to clean it up later." << dendl;
+	  continue;
+	} else {

Fixes: http://tracker.ceph.com/issues/16157
Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov closed this Jul 4, 2016
@tchaikov tchaikov deleted the wip-16157 branch July 4, 2016 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant