Commit Graph

38288 Commits

Author SHA1 Message Date
Josh Durgin
d5e2ca1620 Merge pull request #3277 from ceph/wip-watch-leak
librados: fix a memory leak in watch

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
2015-01-05 12:54:22 -08:00
Sage Weil
ed503776b5 osd/PG: print last_update_applied if != last_update
Signed-off-by: Sage Weil <sage@redhat.com>
2015-01-05 12:53:58 -08:00
Xiaoxi Chen
78d2d310ba Add MOSDRepOp and MOSDRepOpReply
Add the two new message type and change the corresponding code flow as well.

Basically the idea to have MOSDRepOp is to seperate subop(read/write)
out of other subop(pull/push,etc), so that we can cleanup some unused fields in
the message type, then save some encoding/decoding overhead.

The backward compatibility is also remian, if talking with old version OSD who
doesn't support osd_client_subop/subopreply, will fall back to osd_subop/subopreply.

Sage: rename MOSDClientSubOp -> MOSDRepOp

Signed-off-by: Xiaoxi Chen <xiaoxi.chen@intel.com>
Signed-off-by: Sage Weil <sage@redhat.com>
2015-01-05 12:53:58 -08:00
Josh Durgin
f9f64e190e Merge pull request #3206 from ceph/wip-librados-init
librados: init last_objver

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
2015-01-05 12:34:17 -08:00
Ken Dreyer
6ca2eb86be configure: show pkg names when libkeyutils is missing
Prior to this commit, when ./configure can't find libkeyutils, it would
bail out with a terse error message.

Some of the other library checks helpfully print the DEB and RPM package
names in parentheses. Add the DEB and RPM package names to the
libkeyutils check.

Reported-by: Pankaj Garg <Pankaj.Garg@caviumnetworks.com>
Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
2015-01-05 12:19:40 -07:00
Sage Weil
d253badb8d Merge remote-tracking branch 'gh/next' 2015-01-05 11:16:57 -08:00
Sage Weil
3a2e2df79c Merge pull request #3247 from ceph/wip-10422
mon: provide encoded canonical full OSDMap from primary

Reviewed-by: Joao Eduardo Luis <joao@redhat.com>
2015-01-05 11:16:49 -08:00
Sage Weil
5cf84e6393 librados: fix leak of WatchContext on unwatch
The lifecycle matches that of the watch linger_op.  Note that it is NULL
for notify.

Signed-off-by: Sage Weil <sage@redhat.com>
2015-01-05 09:36:43 -08:00
Mykola Golub
b5c24e45a9 qa: add test_tell, which currently is used only for testing 'ceph tell mon version'.
The test requires at least two monitors run.

Signed-off-by: Mykola Golub <mgolub@mirantis.com>
2015-01-05 15:53:54 +02:00
Loic Dachary
f95022e88c Merge pull request #3291 from majianpeng/fix2
test/bufferlist: Remove chmod operation.

Reviewed-by: Loic Dachary <ldachary@redhat.com>
2015-01-05 14:28:50 +01:00
Jianpeng Ma
244777d825 test/bufferlist: For root, don't do permission operation for read_file
case.

For root user, it meet those error:
    test/bufferlist.cc:1880: Failure
    Value of: bl.read_file("testfile", &error)
      Actual: 0
    Expected: -13
    test/bufferlist.cc:1884: Failure
    Value of: bl.length()
      Actual: 8
    Expected: (unsigned)4
    Which is: 4
    test/bufferlist.cc:1886: Failure
    Value of: actual
      Actual: "ABC
    ABC
    "
    Expected: "ABC\n"
    Which is: "ABC

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
2015-01-05 20:51:21 +08:00
Mykola Golub
3b4e021cdc qa: refactor 'ceph -w' magic in test_mon_misc by introducing helper
functions ceph_watch_start and ceph_watch_wait so they can be reused
in other tests.

Signed-off-by: Mykola Golub <mgolub@mirantis.com>
2015-01-05 12:01:15 +02:00
Mykola Golub
11891383ae mon: make ceph tell mon.* version work
Fixes: #6767
Signed-off-by: Mykola Golub <mgolub@mirantis.com>
2015-01-05 12:01:03 +02:00
Haomai Wang
c37abeaf31 Merge pull request #3261 from tchaikov/async-msgr
AsyncMessenger: a trivial refactory and a fix

Reviewed-by: Haomai Wang <haomaiwang@gmail.com>
2015-01-05 10:14:40 +08:00
Sage Weil
f5e3b9a610 Merge pull request #3286 from ceph/wip-10351
mount.ceph: avoid spurious error message

Reviewed-by: Sage Weil <sage@redhat.com>
2015-01-03 10:28:21 -08:00
Yan, Zheng
bdd0e3c4bd mount.ceph: avoid spurious error message
/etc/mtab in most modern distributions is a symbol link to
/proc/self/mounts.

Fixes: #10351
Signed-off-by: Yan, Zheng <zyan@redhat.com>
2015-01-03 15:35:00 +08:00
Yan, Zheng
6e67450015 client: close dirfrag when trying to trim an inode
Fixes: #10387
Signed-off-by: Yan, Zheng <zyan@redhat.com>
2015-01-03 11:41:06 +08:00
Loic Dachary
dced90ba87 Merge pull request #3284 from ktdreyer/doc-rados-preflight-os
doc: rm reference to old Ubuntu release

Reviewed-by: Loic Dachary <ldachary@redhat.com>
2015-01-02 18:54:47 +01:00
Ken Dreyer
274b989fec doc: rm reference to old Ubuntu release
Remove the reference to "Ubuntu 12.10" since this is EOL.

Clarify that we only recommend Ubuntu LTS releases.

(Since this information has a tendancy to become stale, perhaps this
whole paragraph should be removed here and we should simply point at the
main OS Recommendations page.)

Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
2015-01-02 10:37:10 -07:00
Loic Dachary
0471be6fe3 debian: create a repository from sources
Create a debian repository suitable for inclusion in
/etc/apt/sources.list, for the current distribution and using the git
clone from the current working directory.

It is meant to quickly provide teuthology with the packages to run a
job.

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-01-02 12:14:39 +01:00
Loic Dachary
020983b0a5 Merge pull request #3223 from objoo/master
mailmap updates

Reviewed-by: Loic Dachary <ldachary@redhat.com>
2015-01-01 22:52:24 +01:00
Kefu Chai
c8f56ab2a2 AsyncMessender: use IOV_MAX instead of IOV_LEN
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
2014-12-31 10:45:00 +08:00
Kefu Chai
d49b694064 AsyncMessenger: s/sended/sent/
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
2014-12-31 10:45:00 +08:00
Kefu Chai
c9da51de48 AsyncMessenger: should retry in case of EINTR
* if EINTR is returned, do_send() should return 0 or `continue`.
* reword the comment a little bit

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
2014-12-31 10:45:00 +08:00
Sage Weil
169e8bd2cf msg: fix factory error message
Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-30 14:08:49 -08:00
Sage Weil
ec1f58a30d unittest_msgr: enable experiemntal async msgr
Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-30 14:08:38 -08:00
Sage Weil
1265603b01 osd: force read bit for watch
We can get an bad reply like this:

     - watch1 received, repop started
     - con resets
     - watch2 received
     - watch2 sees watch in oi
       - replies immediately
       - does side-effects (connects new session/con)
     - watch1 repop completes
       - replies (dropped)
       - does side-effects (connects old session/con)

Two problems: we end up with the wrong session/con connected, and the watch
reply goes out before the watch is actually stable on disk.

Fix this my marking watch ops as reads.  This will make us wait for the
previous watch addition to commit before we process the next one.  Somewhat
overkill, but sufficient.

Fixes: #10441
Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-30 12:27:37 -08:00
Sage Weil
46424419ca osd: drop unused watch_info_t w
Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-30 12:03:18 -08:00
Sage Weil
51f5f12ca4 Merge pull request #3219 from yuyuyu101/wip-10383
Wip 10383: Fix unittest_msgr with AsyncMessenger bug

Reviewed-by: Sage Weil <sage@redhat.com>
2014-12-30 10:46:58 -08:00
Sage Weil
ecdc9fb2ec Merge pull request #3279 from ceph/wip-mon-janitorial-list
src/doc: add monitor janitorial list
2014-12-30 10:44:17 -08:00
Haomai Wang
fd877e0152 Merge pull request #3278 from mslovy/wip-y04
KeyValueStore:: do_transactions: clean up code which is never used

Reviewed-by: Haomai Wang <haomaiwang@gmail.com>
2014-12-30 20:30:34 +08:00
Ning Yao
cef7a82584 KeyValueStore:: do_transactions: clean up code which is never used
clean up ops and bytes which is never used in do_transactions()

Signed-off-by: Ning Yao <zay11022@gmail.com>
2014-12-30 10:53:21 +00:00
Joao Eduardo Luis
dfd6a382cd src/doc: add monitor janitorial list
Signed-off-by: Joao Eduardo Luis <joao@redhat.com>
2014-12-30 10:10:52 +00:00
Sage Weil
dfee5e5337 Merge pull request #3274 from ceph/wip-10341
tools: ceph-monstore-tool: refactor cli parsing

Reviewed-by: Sage Weil <sage@redhat.com>
2014-12-29 16:28:57 -08:00
Sage Weil
7d75f0cf76 Makefile: include radosgw-admin in base
Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-29 15:55:24 -08:00
Sage Weil
38350a09bd client: fix quota signed/unsigned warning
client/Client.cc: In member function 'bool Client::is_quota_bytes_exceeded(Inode*, uint64_t)':
client/Client.cc:10393:66: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (quota->max_bytes && (rstat->rbytes + new_bytes) > quota->max_bytes)

Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-29 15:47:28 -08:00
Sage Weil
af03583c25 Merge pull request #3275 from liewegas/wip-experimental
mark some features experimental and warn accordingly

Reviewed-by: Yehuda Sadeh <yehuda@redhat.com>
2014-12-29 14:22:17 -08:00
Sage Weil
b7b4534d78 common: log which experimental features are enabled.
This will appear in the logs regardless of whether or not the features
get used/enabled via other config options.

Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-29 14:19:14 -08:00
Sage Weil
55405db537 ms: mark async messenger experimental
Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-29 14:19:14 -08:00
Sage Weil
44ce7cc1de os: rename keyvaluestore-dev -> keyvaluestore; mark experimental
Use a clean name for keyvaluestore (no -dev suffix), but mark as
experimental to ensure users know what they are signing up for.

Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-29 14:19:14 -08:00
Sage Weil
8a559720fb common: add 'enable experimental data corrupting features'
Signed-off-by: Sage Weil <sage@redhat.com>
2014-12-29 14:19:13 -08:00
Sage Weil
d54295e5a8 Merge pull request #3265 from yuyuyu101/coverify-fix
Event: Avoid illegal multiplication for memset

Reviewed-by: Sage Weil <sage@redhat.com>
2014-12-29 10:56:23 -08:00
John Wilkins
9b3c0b6140 Merge pull request #3260 from nilamdyuti/wip-doc-ceph-object-gateway
doc: Updates information for SSL configuration in Ceph Object Gateway installation.

Reviewed-by: John Wilkins <jowilkin@redhat.com>
2014-12-29 10:39:24 -08:00
Joao Eduardo Luis
783956ccb6 tools: ceph-monstore-tool: validate start/stop vals
on commands that use 'start' and 'stop'/'end' values, we should validate
them and inform the user when 'stop' > 'start'.

Signed-off-by: Joao Eduardo Luis <joao@redhat.com>
2014-12-29 17:50:28 +00:00
Joao Eduardo Luis
60e9c3f003 tools: ceph-monstore-tool: output number of paxos versions dumped
Signed-off-by: Joao Eduardo Luis <joao@redhat.com>
2014-12-29 17:50:28 +00:00
Joao Eduardo Luis
b27ca4c024 tools: ceph-monstore-tool: raw map to stdout if outfile not specified
Signed-off-by: Joao Eduardo Luis <joao@redhat.com>
2014-12-29 17:50:28 +00:00
Joao Eduardo Luis
68f3b77ddd tools: ceph-monstore-tool: use subcmd parser helper
reduces the noise in the code

Signed-off-by: Joao Eduardo Luis <joao@redhat.com>
2014-12-29 17:50:28 +00:00
Joao Eduardo Luis
a186ee1229 tools: ceph_monstore_tool: add sub-command parser helper function
Signed-off-by: Joao Eduardo Luis <joao@redhat.com>
2014-12-29 17:50:28 +00:00
Joao Eduardo Luis
fe662c5ae2 tools: ceph-monstore-tool: a cli that works
Fixes: #10341

Signed-off-by: Joao Eduardo Luis <joao@redhat.com>
2014-12-29 17:50:28 +00:00
Haomai Wang
793c32d5f1 Event: Avoid illegal multiplication for memset
Because "file_events" is "FileEvent" type and adding an integral value to
this pointer automatically scales that value by the size.

Signed-off-by: Haomai Wang <haomaiwang@gmail.com>
2014-12-28 21:31:35 +08:00