Ceph

Hammer Source File : src/common/obj_bencher.cc

 if (i % 20 == 0) {
      if (i > 0)
        cur_time.localtime(cout) << "min lat: " << data.min_latency
             << " max lat: " << data.max_latency
             << " avg lat: " << data.avg_latency << std::endl;
      //I'm naughty and don't reset the fill
      bencher->out(cout, cur_time) << setfill(' ')
           << setw(5) << "sec" 
           << setw(8) << "Cur ops" 
           << setw(10) << "started" 
           << setw(10) << "finished" 
           << setw(10) << "avg MB/s" 
           << setw(10) << "cur MB/s" 
           << setw(10) << "last lat" 
           << setw(10) << "avg lat" << std::endl;
    }

last lat and avg lat here has setw(10) : which is causing the issue.

But in Jewel it was changed to :

  bencher->out(cout, cur_time) << setfill(' ')
          << setw(5) << "sec" 
          << setw(8) << "Cur ops" 
          << setw(10) << "started" 
          << setw(10) << "finished" 
          << setw(10) << "avg MB/s" 
          << setw(10) << "cur MB/s" 
          << setw(12) << "last lat(s)" 
          << setw(12) << "avg lat(s)" << std::endl;
    }

last lat and *avg lat * is changed to setw(12) : I hope this will solve the issue.

<< setw(12) << "last lat(s)"
<< setw(12) << "avg lat(s)" << std::endl;

Verified in Jewel , as expected with code change, it works perfectly fine.

So this is a request to backport https://github.com/ceph/ceph/commit/669b9329 ?

Nathan Cutler wrote:

So this is a request to backport https://github.com/ceph/ceph/commit/669b9329 ?

Thanks Nathan for looking into this one. Yes this is the one.

Isn't this just a duplicate of http://tracker.ceph.com/issues/12248 ?

Can we close it to avoid (more) confusion please?

@Brad No, this is not a duplicate. It's a backport tracker issue. We track backports separately from the original issues.

@Brad My apologies! You were right.