{"push_id":1002392447,"size":186,"distinct_size":151,"ref":"refs/heads/master","head":"2786f0bf11c3c2ec0e3e8bd61cd97c6b5a70345d","before":"8c054f741a8102271c3b9ccf6f459210cf40f71e","commits":[{"sha":"db3dc4d3e2a74cfd366cd2ef4d0c9c4581269113","author":{"email":"52aa03431caa830d957d78a0bd19dcb738002b0e@redhat.com","name":"David Zafman"},"message":"osd: build_past_intervals_parallel() add diagnostics before assert\n\nSigned-off-by: David Zafman <dzafman@redhat.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/db3dc4d3e2a74cfd366cd2ef4d0c9c4581269113"},{"sha":"8b777a0c346bc70fd10d07e89368b3785b58f10e","author":{"email":"15e918392590913fa8c5f316bf83f6cdab37c988@redhat.com","name":"Brad Hubbard"},"message":"common: Allow config set with negative value\n\nA recent commit disabled negative values but they are required for variables\nsuch as filestore_merge_threshold.\n\nFixes: #13829\n\nSigned-off-by: Kefu Chai <kchai@redhat.com>\nSigned-off-by: Brad Hubbard <bhubbard@redhat.com>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/8b777a0c346bc70fd10d07e89368b3785b58f10e"},{"sha":"994ac2942d04584d1617e6d4bbd5b880b1ea0448","author":{"email":"15e918392590913fa8c5f316bf83f6cdab37c988@redhat.com","name":"Brad Hubbard"},"message":"qa: Add test for #13829\n\nqa/workunits/cephtool/test.sh: add test for setting negative int options\nsrc/test/daemon_config.cc: remove tests for failed negative values\n\nFixes: #13829\n\nSigned-off-by: Brad Hubbard <bhubbard@redhat.com>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/994ac2942d04584d1617e6d4bbd5b880b1ea0448"},{"sha":"a0b0ae00f25706e702377a915188f7e9a316272f","author":{"email":"52aa03431caa830d957d78a0bd19dcb738002b0e@redhat.com","name":"David Zafman"},"message":"Revert \"osd: do not keep ref of old osdmap in pg\"\n\nThis reverts commit 8978cd6fcf40076dbd944a2938b335d26c62b8d5.\n\nFixes: #14213","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/a0b0ae00f25706e702377a915188f7e9a316272f"},{"sha":"ee39f436815dd64d89f01f0befeb09cff7b9d332","author":{"email":"6dd34506bbd5e58221bbb3e4732d97c91f02277b@redhat.com","name":"Sage Weil"},"message":"Merge pull request #7570 from dzafman/wip-14213\n\nosd: revert \"osd: do not keep ref of old osdmap in pg\"\r\n\r\nReviewed-by: David Zafman <dzafman@redhat.com>\r\nReviewed-by: Sage Weil <sage@redhat.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/ee39f436815dd64d89f01f0befeb09cff7b9d332"},{"sha":"78799a9c92e808a8aa7b51a9450494e59cd13f54","author":{"email":"16ccadb3f2bf0703b4be974f8606e7e98c1cda29@redhat.com","name":"Samuel Just"},"message":"Merge pull request #7405 from ceph/wip-diag-14438\n\nosd: build_past_intervals_parallel() add diagnostics before assert\r\n\r\nReviewed-by: Samuel Just <sjust@redhat.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/78799a9c92e808a8aa7b51a9450494e59cd13f54"},{"sha":"5c09a3e4e483ff7e6f512358a3dab01d37104baf","author":{"email":"f9d8c66a4e898e67db786769b5b347b640e1a9a6@suse.com","name":"Nathan Cutler"},"message":"rpm: drop user_rgw and group_rgw template variables\n\nThese were originally used to template RGW tmpfiles.d, which was eliminated\nby 8e13d89f0f136f5be8ab869190062abfcca0e23d\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/5c09a3e4e483ff7e6f512358a3dab01d37104baf"},{"sha":"a6cc8ea5d96b308f7b5324efcd23b3296c8239e2","author":{"email":"f9d8c66a4e898e67db786769b5b347b640e1a9a6@suse.com","name":"Nathan Cutler"},"message":"rpm: drop systemd_libexec_dir template variable\n\nGeneral cleanup. Actual use of the template variable was discontinued in\n7c9fdf44f2c18659a0bcc03f7b98dafdf9f54448.\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/a6cc8ea5d96b308f7b5324efcd23b3296c8239e2"},{"sha":"fe14a267c890dc67e3f4cc0f3a5cb202d23d108c","author":{"email":"f9d8c66a4e898e67db786769b5b347b640e1a9a6@suse.com","name":"Nathan Cutler"},"message":"ceph.spec.in: declare /usr/share/ceph properly\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/fe14a267c890dc67e3f4cc0f3a5cb202d23d108c"},{"sha":"331e90f450265891c68790bec2f82d39e0d096af","author":{"email":"acb653cc3cb5772fe240cc62baddb37473932bee@sandisk.com","name":"Somnath Roy"},"message":"Use make_shared while creating shared_ptr\n\nmake_shared() will get rid of one extra 'new' call during shared_ptr\ncreation.It is also 20-30% faster than creating shared_ptr directly\nby 'new' call.\n\nSigned-off-by: Somnath Roy <somnath.roy@sandisk.com>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/331e90f450265891c68790bec2f82d39e0d096af"},{"sha":"69291f872e7dd20e88c052cfc220e02d59826c0f","author":{"email":"f9d8c66a4e898e67db786769b5b347b640e1a9a6@suse.com","name":"Nathan Cutler"},"message":"packaging: move ceph_common.sh and ceph-osd-prestart.sh to /usr/lib/ceph\n\nFirst, it makes sense for both ceph_common.sh and ceph-osd-prestart.sh to\nreside in the same directory: make it so.\n\nSecond, /usr/lib exists on both RHEL/Fedora and SLE/openSUSE, whereas\nthe later lacks /usr/libexec. To make this less painful, package\nceph_common.sh and ceph-osd-prestart.sh in /usr/lib/ceph.\n\nThird, allow e.g. FreeBSD to do its own thing by using the $(libexecdir)\nAutoconf variable (but set it to /usr/lib in the spec file).\n\nhttp://tracker.ceph.com/issues/14687 Fixes: #14687\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/69291f872e7dd20e88c052cfc220e02d59826c0f"},{"sha":"68f300dda47936e825f00237ed5263be5961cc60","author":{"email":"5913bbe9383946b538590e008fae0f07f93917e3@zte.com.cn","name":"xie xingguo"},"message":"os/filestore: fix wrong scope of result code for error cases during mkfs\n\nDuring the mkfs process, if we fail to read and sometimes rewrite the\nop_seq, the mkfs process is abnormally terminated but we may\nstill return a positive answer to the caller, which is misleading.\n\nThis patch is provided to solve the above problems by correctly setting\n'ret' correspondingly under these cases, which is the final answer to\nbe passed out for caller.\n\nFixes: #14814\nSigned-off-by: xie xingguo <xie.xingguo@zte.com.cn>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/68f300dda47936e825f00237ed5263be5961cc60"},{"sha":"877eae85ce7630fa05cc16f5f71be40e08607a5c","author":{"email":"5913bbe9383946b538590e008fae0f07f93917e3@zte.com.cn","name":"xie xingguo"},"message":"os/filestore: fix result code overwritten for clone\n\nDuring the clone process, the result code of syscall-fstat which is intend for\ngetting the source size for copying is not checked and will be overwritten\nby the following call, which is of potential risk and thus should be considered\nas unsafe.\n\nThis pr solves the above problem by adding result checking against failure.\nAlso result code is reset to errno for better tracing if the ftruncate syscall\nfailed.\n\nFixes: #14817\nSigned-off-by: xie xingguo <xie.xingguo@zte.com.cn>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/877eae85ce7630fa05cc16f5f71be40e08607a5c"},{"sha":"c7eba2aa3d4605851ec2d1cf9fa04ea6dc1d5ebd","author":{"email":"f9d8c66a4e898e67db786769b5b347b640e1a9a6@suse.com","name":"Nathan Cutler"},"message":"src/yasm-wrapper: ignore parameters starting with ggc-min\n\nWhen the \"with lowmem_builder\" bcond is active, the string \"--param\nggc-min-expand=20 --param ggc-min-heapsize=32768\" is added to\nRPM_OPT_FLAGS. In the course of the build, these parameters get passed on\nto yasm-wrapper, making it unhappy.\n\nhttp://tracker.ceph.com/issues/14811 Fixes: #14811\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/c7eba2aa3d4605851ec2d1cf9fa04ea6dc1d5ebd"},{"sha":"9d25202a1ababf2d6347978966049fe2b88e46dd","author":{"email":"a4b264ee037c625052534d60505bfd0db4776ac4@redhat.com","name":"Kefu Chai"},"message":"Makefile: workaround an automake bug for \"make check\"\n\noverride the automake variable of am__is_gnu_make. it is a workaround\nfor http://debbugs.gnu.org/cgi/bugreport.cgi?bug=18744. this bug was\nfixed in automake 1.15, but automake 1.13 is supported. so we can not\njust require 1.15 using `AM_INIT_AUTOMAKE`. this only happens when we\nhave lots of source files *and* we are using automake v1.14 or lower.\n\nFixes: #14723\nSigned-off-by: Kefu Chai <kchai@redhat.com>\n(cherry picked from commit 19391b71497555c9bcc37b1e063fd4895df605c5)","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/9d25202a1ababf2d6347978966049fe2b88e46dd"},{"sha":"3c5bc07973bc48c8cd8096cfbce15cbc8b39f41a","author":{"email":"a4b264ee037c625052534d60505bfd0db4776ac4@redhat.com","name":"Kefu Chai"},"message":"Makefile: workaround an automake bug for \"make check\"\n\noverride the automake variable of am__is_gnu_make. it is a workaround\nfor http://debbugs.gnu.org/cgi/bugreport.cgi?bug=18744. this bug was\nfixed in automake 1.15, but automake 1.13 is supported. so we can not\njust require 1.15 using `AM_INIT_AUTOMAKE`. this only happens when we\nhave lots of source files *and* we are using automake v1.14 or lower.\n\nFixes: #14723\nSigned-off-by: Kefu Chai <kchai@redhat.com>\n(cherry picked from commit 19391b71497555c9bcc37b1e063fd4895df605c5)","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/3c5bc07973bc48c8cd8096cfbce15cbc8b39f41a"},{"sha":"b2bfd6cc0ba697414a0abca1bb8a1074abb08993","author":{"email":"5913bbe9383946b538590e008fae0f07f93917e3@zte.com.cn","name":"xie xingguo"},"message":"common/TrackedOp: fix inaccurate counting for total slow requests\n\nIn the original design there are two counters in charge of collecting\npotentially problematic requests, namely 'slow' and 'warned'.\nCounter 'slow' is responsible for capturing all the requests which have\nalready hit the \"complain\" limit and shall be marked as \"slow\" while\ncounter 'warned' is responsible for countering those requests which\nhave already hit the \"warning interval\" and thus shall be logged and\nthen outputted.\n\nThe problem here is if 'warned' counter hits the log_threshold,\nwe will quit the entire for loop but there may be residual shard_queues\nwhich may still containing slow requests. As a result, the 'slow' counter\ndoes not reflect the real number of total slow requests in all the\nshard_queues under this case. And no slow requests will be tracked\nespecially when 'log_threshold' is set to zero(Do we do this intentional?\nOr else we shall never allow 'log_threshold' to be zero).\n\nThe solution for the above problem is to keep counting 'slow' requests\nuntil we have finished traversing all the shard_queues, no matter\nwhether we have gathered enough requests for logging or not, and if so,\nwe simply stop counter 'warned' and skip over logging process.\n\nFixes: #14804\nSigned-off-by: xie xingguo <xie.xingguo@zte.com.cn>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/b2bfd6cc0ba697414a0abca1bb8a1074abb08993"},{"sha":"52c04b81a699b0d193cd19263ca5f02fcef46664","author":{"email":"5913bbe9383946b538590e008fae0f07f93917e3@zte.com.cn","name":"xie xingguo"},"message":"common/TrackedOp: avoid implicitly cast for return value\n\nSigned-off-by: xie xingguo <xie.xingguo@zte.com.cn>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/52c04b81a699b0d193cd19263ca5f02fcef46664"},{"sha":"9644df6ddd892d1aad241f8e981d76b49c2182a8","author":{"email":"5913bbe9383946b538590e008fae0f07f93917e3@zte.com.cn","name":"xie xingguo"},"message":"os/filestore: fix result code if sanity check failed during copy_range\n\nAccording to Linux Man Page, the lseek() system call returns -1 and\nsets errno only if it indeed encouters some error.\nThe problem here is if lseek64() returns a positive answer, which\nis not strictly equal to the expected answer however, then errno\nmay not be set and we return success instead, which is wrong.\n\nThis commit solves the above problem by checking against the returned\nvalue of lseek64(), if -1 then we reset result code to errno, otherwise\nwe reset result code to -EINVAL to indicate that it is sanity check not\npassed is the real relevant issue.\n\nOne more thing, as _do_copy_range() may return customized error code,\ncaller shall not redirect result code to errno on failure, which is\nalso shall be considered as wrong action.\n\nFixes: #14827\nSigned-off-by: xie xingguo <xie.xingguo@zte.com.cn>","distinct":true,"url":"https://api.github.com/repos/SUSE/ceph/commits/9644df6ddd892d1aad241f8e981d76b49c2182a8"},{"sha":"1b6faf68fbe272f4ded1b287453e7ab24c37b7da","author":{"email":"f9d8c66a4e898e67db786769b5b347b640e1a9a6@suse.com","name":"Nathan Cutler"},"message":"Refrain from versioning EC testing plugins\n\nSince these plugins are only used in \"make check\", there is no\npoint in versioning them.\n\nhttp://tracker.ceph.com/issues/14756 Fixes: #14756\n\nSigned-off-by: Nathan Cutler <ncutler@suse.com>","distinct":false,"url":"https://api.github.com/repos/SUSE/ceph/commits/1b6faf68fbe272f4ded1b287453e7ab24c37b7da"}]}