Tangential thought: we've set up Jenkins on other repos to do some basic tests. The plan is to do the same for the ceph.git, but the test of tests is much more extensive (build tests, possibly teuthology tests).
I wonder if an easy first step would be a trivial test that just verifies that everything has a Signed-off-by. That would get jenkins into the pipeline and we can build from there... [Background: every commit to ceph needs a Signed-off-by: Name <email> line to acknowledge/consent to the open source software license; see https://github.com/ceph/ceph/blob/master/SubmittingPatches ] sage On Thu, 7 Nov 2013, Yehuda Sadeh wrote: > Hey, > > I wanted to backport that fix to dumpling, but just noticed that the > commits were missing a signed-off-by line. Can I add it? > > Thanks, > Yehuda > > > ---------- Forwarded message ---------- > From: Xiangyu (Ray) Lv <[email protected]> > Date: Wed, Oct 30, 2013 at 7:24 PM > Subject: [ceph] Put obj sometimes got 405 (#794) > To: ceph/ceph <[email protected]> > Cc: Yehuda Sadeh <[email protected]> > > > Fixes: > > Bug #6672 (http://tracker.ceph.com/issues/6672) > > Original discussion at #783 > Branch reviewed by @yehudasa. > > ________________________________ > > You can merge this Pull Request by running > > git pull https://github.com/xylv/ceph put-obj-sometimes-got-405 > > Or view, comment on, or merge it at: > > https://github.com/ceph/ceph/pull/794 > > Commit Summary > > [rgw] Set initialized to true after populating table in hex_to_num() > Wrap hex_to_num table into class HexTable > > File Changes > > M src/rgw/rgw_common.cc (20) > > Patch Links: > > https://github.com/ceph/ceph/pull/794.patch > https://github.com/ceph/ceph/pull/794.diff > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
