firewalld in RHEL 7.3 contains service definitions for ceph's ports, so
we can simply use "--add-service=ceph-mon" or "--add-service=ceph"
rather than having to remember specific port numbers.
Break out two examples for both MONs and OSDs/MDSs.
Add the "firewall-cmd --reload" step to ensure the --permanent configs
take effect immediately.
Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
we have
2017-02-04T16:15:46.090 INFO:tasks.workunit.client.0.mira032.stdout:error in 22088
2017-02-04T16:15:46.092 INFO:tasks.workunit.client.0.mira032.stderr:bash: line 1: 22092 Alarm clock ceph_test_rados_api_aio 2>&1
2017-02-04T16:15:46.096 INFO:tasks.workunit.client.0.mira032.stderr: 22093 Done | tee ceph_test_rados_api_aio.log
2017-02-04T16:15:46.099 INFO:tasks.workunit.client.0.mira032.stderr: 22094 Done | sed "s/^/ api_aio: /"
2017-02-04T16:15:46.102 INFO:tasks.workunit.client.0.mira032.stderr:+
if a unittest in rados/test.sh fails in teuthology.log, but it would
be desirable to have the failed test name in the line of "error in
22088".
Signed-off-by: Kefu Chai <kchai@redhat.com>
In the case of recovery pull using sub op, and if the prep push on
remote peers fail for some reasons, recovery_info.soid is not set
in prep_push_op_blank. Use it on the primary would lead to an
assertion failure.
Signed-off-by: Zhiqiang Wang <zhiqiang@xsky.com>
the version command is handled by HelpHook, which does not print newline
at the end of the returned JSON. so the output of "status" subcommand
looks like:
=== osd.1 ===
osd.1: running {"version":"Development"}=== osd.2 ===
osd.2: not running.
with this change, a newline is added to the end of version.
also use the "daemon" subcommand to talk to the asok, let the "ceph" cli
to figure out the asok path.
Signed-off-by: Kefu Chai <kchai@redhat.com>
tests: osd-scrub-repair.sh disable scrub backoff in test
Reviewed-by: David Zafman <dzafman@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
there is good chance that the newly scheduled scrub fail to pass the
flip-coin check: the probability of scheduling a new scrub 1/3 every
time we flip the coin. so, instead hardwiring the probability to 1/3,
a new option named "osd_scrub_backoff_ratio" is introduced. so we can
override it when performing test.
Signed-off-by: Kefu Chai <kchai@redhat.com>
Update all calls to Messenger::create()
take 'ms_public_type' or 'ms_cluster_type' instead of 'ms_type'
issue: 971661
Change-Id: Ib234daf393d1838ef596dde35a6d83c1aeecaf10
Signed-off-by: Sarit Zubakov <saritz@mellanox.com>
We use m_work_queue of type ContextWQ in handle_update function but we
do not include common/WorkQueue.h that defines ContextWQ. This results
in dereference of an incomplete type and causes build error in latest
Fedora rawhide (future 26).
Fixes: http://tracker.ceph.com/issues/18862
Signed-off-by: Boris Ranto <branto@redhat.com>
* support xenial
* support "rados" cli debugging
* do not include the ending "'" in the prog name sending to gdb
Signed-off-by: Kefu Chai <kchai@redhat.com>
Use the stat field of PushInfo to do the stat update, which is
more accurate, especially in the case of sparse file. Also,
remove the stat sum update in on_peer_recover, which is done in
on_global_recover.
Signed-off-by: Zhiqiang Wang <zhiqiang@xsky.com>
Some variables have been already free'd by calling
release(), no need to call delete() again on them.
CID 1396200 (#1 of 1): Double free (USE_AFTER_FREE)
double_free: Calling operator delete frees pointer
getxattr_completion which has already been freed.
CID 1396206 (#1 of 1): Double free (USE_AFTER_FREE)
double_free: Calling operator delete frees pointer
getxattr_completion which has already been freed.
CID 1396214 (#1 of 1): Double free (USE_AFTER_FREE)
double_free: Calling operator delete frees pointer
stat_completion which has already been freed.
CID 1396218 (#1 of 1): Double free (USE_AFTER_FREE)
double_free: Calling operator delete frees pointer
stat_completion which has already been freed.
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Fix for:
CID 1395468 (#1 of 1): Self assignment (NO_EFFECT)
self_assign: Assigning bh->last_write to itself has no effect.
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
CID 1395784 (#2-1 of 2): Resource leak (RESOURCE_LEAK)
leaked_storage: Variable state going out of scope
leaks the storage it points to.
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
ID 1398895 (#1 of 1): Resource leak (RESOURCE_LEAK)
leaked_storage: Variable config going out of scope leaks
the storage it points to.
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Fix for:
CID 1398896: Resource leak (RESOURCE_LEAK)
overwrite_var: Overwriting fe in fe = new
RGWLoadGenFrontend(env, config) leaks the storage that fe
points to.
Do not check again and again for framework if there is
already a match, make use of if/else. This should also
fix the RESOURCE_LEAK warning.
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
