Commit Graph

105366 Commits

Author SHA1 Message Date
Jason Dillaman
703a4ebde4
Merge pull request #31794 from dillaman/wip-42889
librbd: disable zero-copy writes by default

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
2019-12-09 12:47:47 -05:00
Jason Dillaman
bd76f0d10c
Merge pull request #32105 from junxfl/test1
librbd: fix rbd_open_by_id, rbd_open_by_id_read_only 

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
2019-12-09 12:37:37 -05:00
Daniel Gryniewicz
105c6d78aa
Merge pull request #32109 from yehudasa/wip-42868
rgw: list buckets: don't return buckets if limit=0
2019-12-09 12:23:12 -05:00
Sage Weil
2e03356f81 osd/OSDMap: rename old calc_pg_role -> calc_pg_role_broken
Also primary_changed -> primary_changed_broken.

Logged this issue with https://tracker.ceph.com/issues/43213

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 10:56:02 -06:00
Michael Fritch
8c355898f6
ceph-daemon: combine SUDO and ARGS into a single var
- reduce the amount of typing/noise for each CEPH_DAEMON invocation
- ensure the `--image` param is passed to each test invocation
- allow passing additional args to ceph-daemon via CEPH_DAEMON_ARGS

Signed-off-by: Michael Fritch <mfritch@suse.com>
2019-12-09 09:53:31 -07:00
Sage Weil
52fb794f52 osd/OSDMap: remove dead osd_is_valid_op_target
Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 10:53:08 -06:00
Sage Weil
b498d71543 osd/OSDMap: fix get_pg_acting_role()
This needs to take an spgid in order to know what role the osd is
serving.

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 10:47:49 -06:00
Sebastian Wagner
57c3e344ab codeowners: Add ceph2.py to @ceph/orchestrators
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
2019-12-09 17:46:57 +01:00
Lenz Grimmer
22d5bd95e2
Merge pull request #32053 from votdev/left_align_badges
mgr/dashboard: Left align badge datatable columns

Reviewed-by: Laura Paduano <lpaduano@suse.com>
2019-12-09 16:44:56 +00:00
Sage Weil
cfdb569ab2 osd: use spg_t for pending_creates_from_osd
Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 10:44:55 -06:00
Sage Weil
9cc41d2bb4 osd/OSDMap: drop unused get_pg_acting_role()
No callers.

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 10:43:16 -06:00
Lenz Grimmer
d5c54d55eb
Merge pull request #31927 from bk201/wip-43072
mgr/dashboard: hide in-use devices when creating OSDs

Reviewed-by: Laura Paduano <lpaduano@suse.com>
Reviewed-by: Volker Theile <vtheile@suse.com>
2019-12-09 16:42:54 +00:00
Sage Weil
451c5cadf1 osd/OSDMap: fix+simplify is_up_acting_osd_shard
Use the new calc_pg_role() to simplify this function

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 10:42:39 -06:00
Lenz Grimmer
9960f9365a
Merge pull request #32084 from votdev/auth_service_observable
mgr/dashboard: Use Observable in auth.service

Reviewed-by: Alfonso Martínez <almartin@redhat.com>
2019-12-09 16:34:50 +00:00
Lenz Grimmer
a494757a6e
mgr/dashboard: Improve badge column template. (#32026)
mgr/dashboard: Improve badge column template.

Reviewed-by: Kiefer Chang <kiefer.chang@suse.com>
2019-12-09 16:32:03 +00:00
Sage Weil
f886610e44 Merge PR #31907 into master
* refs/pull/31907/head:
	mgr/dashboard: properly handle a missing rbd-mirror service status
	mgr: cull service daemons when the last instance has been removed
	mgr: ensure new daemons are properly indexed by hostname

Reviewed-by: Sage Weil <sage@redhat.com>
Reviewed-by: Tatjana Dehler <tdehler@suse.com>
Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
2019-12-09 10:30:26 -06:00
Ilya Dryomov
3aae140353 qa: kernel.sh: unlock before rolling back
"rbd snap rollback" expects an unlocked image, but we may get there
locked if object map is enabled (or if lock_on_read is specified in
rbd_default_map_options).

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
2019-12-09 17:29:13 +01:00
Ilya Dryomov
50933b863a qa: krbd_exclusive_option.sh: update for recent kernel changes
Since 5.3:
- a plain "rbd map" acquires the lock, so it's not different from
  "rbd map -o exclusive" in this regard
- if the lock is held by the exclusive peer, I/O is failed right away
  instead of blocking
- lock_timeout option is respected only by "rbd map" and not by I/O

Since 5.5:
- if the mapping is read-only, the lock isn't acquired

Added blacklisting test case, dropped lock_timeout test case.

Fixes: https://tracker.ceph.com/issues/43127
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
2019-12-09 17:29:13 +01:00
Sage Weil
1cf2b90640 osd: use new and improved calc_pg_role()
An acting set might be [0,1,2,1], and we want pgs 1.0s2 and 1.0s4 on
osd.1 to get roles 1 and 3, respectively (instead of 1 and -1).  This
allows the second EC PG on the OSD to have role >= 0.

This probably was unnoticed before, but now it leads to hangs in the
rados/thrash-erasure-code collection because proc_lease() bails out
when is_nonprimary() fails (due to role < 0).

Fixes: https://tracker.ceph.com/issues/43189
Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 09:51:12 -06:00
Sage Weil
b760533591 osd/OSDMap: new calc_pg_role() that takes a pg_shard_t
If you have an acting set like [0,1,2,1] then osd.1 has two possible roles
(1 and 3).  If we simply return the first one we see then we can't
correctly calculate a role for a pg like 1.0s3.  Take pg_shard_t instead.

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 09:50:13 -06:00
Sage Weil
0637c26955 osd/OSDMap: calc_pg_rank -> calc_pg_role
These were identical.

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 09:50:12 -06:00
Sage Weil
5c1b34fc84 osd/PeeringState: debug lines for upacting_features, proc_lease
Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 09:50:12 -06:00
Sage Weil
7ba1dead7d osd/PeeringState: use pg_vector_string for operator<<
This shows NONE instead of a numeric CRUSH_ITEM_NONE.

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 09:50:12 -06:00
Tatjana Dehler
6a988938ac qa/tasks/mgr/dashboard/test_health: update schema
Fixes: https://tracker.ceph.com/issues/43195
Signed-off-by: Tatjana Dehler <tdehler@suse.com>
2019-12-09 16:12:12 +01:00
Sebastian Wagner
2714893288 mgr/rook: Added Mypy static type checking
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
2019-12-09 15:55:41 +01:00
Sage Weil
64b69572b5 Merge PR #32009 into master
* refs/pull/32009/head:
	doc: removes kube-helm installation instructions

Reviewed-by: Sebastian Wagner <swagner@suse.com>
2019-12-09 08:23:44 -06:00
yangjun
3457192c24 librbd: fix rbd_open_by_id, rbd_open_by_id_read_only
These methods incorrectly delete ImageCtx on error, resulting
in double-free heap corruption.

Fixes: https://tracker.ceph.com/issues/43178
Signed-off-by: yangjun <yangjun@cmss.chinamobile.com>
2019-12-09 09:08:22 -05:00
Jos Collin
4c1029bc08
mgr/volumes: check for string values in uid/gid
chown allows strings as per bbbfb44453,
which caused this error. Eventhough uid/gid are input as CephInt, the
qa tests can pass in only string values to _fs_cmd. So try converting
the incoming uid/gid to int in create_subvolume and create_group.
It might be a valid string.

Fixes: https://tracker.ceph.com/issues/43038
Signed-off-by: Jos Collin <jcollin@redhat.com>
2019-12-09 19:25:46 +05:30
Sage Weil
4e75ee0f3a Merge PR #31800 into master
* refs/pull/31800/head:
	doc: Update commands in bootstrap.rst
	Update doc/bootstrap.rst
	Update doc/bootstrap.rst
	Merge branch 'master' into wip-doc-prompts-update
	doc: Update commands in bootstrap.rst

Reviewed-by: Sebastian Wagner <swagner@suse.com>
Reviewed-by: Sage Weil <sage@redhat.com>
2019-12-09 07:17:57 -06:00
Sage Weil
3036d11c60 ceph-daemon: py2: tolerate whitespace before config key name
The py2 ConfigParser doesn't like whitespace before the config option
name.  (The py3 version doesn't care.)  Filter it out before parsing.

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-09 07:15:13 -06:00
Varsha Rao
f6241d1c07 doc/cephfs/disaster-recovery-experts: Add link for scrub and note for scrub_path
Fixes: https://tracker.ceph.com/issues/22204
Signed-off-by: Varsha Rao <varao@redhat.com>
2019-12-09 18:17:41 +05:30
Jan Fajerski
1c8a4b0326
Merge pull request #32106 from taodd/ceph-volume-fix-env
ceph-volume: fix the integer overflow
2019-12-09 11:46:18 +01:00
Lenz Grimmer
713db2994d
mgr/dashboard: add smartctl data as a details tab of a host (#30594)
mgr/dashboard: add smartctl data as a details tab of a host

Reviewed-by: Kiefer Chang <kiefer.chang@suse.com>
Reviewed-by: Laura Paduano <lpaduano@suse.com>
Reviewed-by: Tiago Melo <tmelo@suse.com>
Reviewed-by: Volker Theile <vtheile@suse.com>
2019-12-09 09:42:23 +00:00
Yehuda Sadeh
1bf21326aa qa: radosgw-admin: remove dependency on bunch package
Fixes: https://tracker.ceph.com/issues/43184

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
2019-12-08 19:41:17 -08:00
Sage Weil
105428e8f4 Merge PR #29687 into master
* refs/pull/29687/head:
	test/store_test: test coverage for anti-spillover framework.
	os/bluestore: introduce bluestore_volume_selection_reserved cfg option.
	os/bluestore: beautify RocksDBBlueFSVolumeSelector::dump output
	os/bluestore: introduce 'bluestore bluefs stats' admin socket command
	os/bluestore: implement more intelligent DB volume space managment for BlueFS.
	os/kv: add static method to parse RocksDB options
	os/bluestore: introduce volume selector abstraction to BlueFS

Reviewed-by: Sage Weil <sage@redhat.com>
2019-12-08 15:24:13 -06:00
Sage Weil
d5d4868b9f Merge PR #32108 into master
* refs/pull/32108/head:
	ceph-daemon: remove prepare-host

Reviewed-by: Sebastian Wagner <swagner@suse.com>
2019-12-08 13:26:32 -06:00
Sage Weil
ade391513c ceph-daemon: remove prepare-host
I thought I took this out of the PR but somehow it got merged in... must
have repushed and old branch and not realized.  :/

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-08 11:26:14 -06:00
Sage Weil
e2f8187b4c Merge PR #31959 into master
* refs/pull/31959/head:
	os/bluestore/BlueStore.cc: set priorities for compression stats

Reviewed-by: Ernesto Puerta <epuertat@redhat.com>
Reviewed-by: Igor Fedotov <ifedotov@suse.com>
2019-12-08 10:15:48 -06:00
Sage Weil
1542776e44 Merge PR #31995 into master
* refs/pull/31995/head:
	msg/async: No need lock for func _filter_addrs

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
2019-12-08 10:15:17 -06:00
Sage Weil
9634b20908 Merge PR #32101 into master
* refs/pull/32101/head:
	mgr/orchestrator_cli: make rgw rm take realm+zone
	qa/tasks/mgr/test_orchestrator_cli: fix rgw add test

Reviewed-by: Sebastian Wagner <swagner@suse.com>
2019-12-08 10:14:50 -06:00
Yehuda Sadeh
857fdfb8e0 rgw: list buckets: don't return buckets if limit=0
Fixes: https://tracker.ceph.com/issues/42868

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
2019-12-08 03:08:46 -08:00
Sage Weil
99eb52b68a mgr/orchestrator_cli: make rgw rm take realm+zone
And adjust test accordingly

Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-07 23:41:36 -06:00
taodd
81ff4be4d1 ceph-volume: fix the type mismatch, covert the tries and interval to int from string.
Fixes: https://tracker.ceph.com/issues/43186

Signed-off-by: dongdong tao <dongdong.tao@canonical.com>
2019-12-08 10:26:42 +08:00
Sage Weil
b9dea87ff1 mgr/diskprediction_local: import scipy early to fix self-test deadlock
We are seeing a hang on centos7 (but not ubuntu 18.04) from

        from .predictor import get_diskfailurepredictor_path

in _predict_life_expentancy.

Doing that same predictor import at the top of the file doesn't help--it
hangs right when the diskpredictor_local module is loaded.  Commenting
out the import scipy in predictor.py avoids the hang.

I'm not sure why, but doing the full scipy import here appears to work
around the problem.

Fixes: https://tracker.ceph.com/issues/42764
Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-07 15:34:59 -06:00
Sage Weil
28e9e25e93 Merge PR #32043 into master
* refs/pull/32043/head:
	os/bluestore: default bluestore_block_size 1T -> 100G

Reviewed-by: Igor Fedotov <ifedotov@suse.com>
2019-12-07 14:43:07 -06:00
Sage Weil
d73fe9cbd6 Merge PR #32044 into master
* refs/pull/32044/head:
	mon/OSDMonitor: make memory autotune disable itself if no rocksdb

Reviewed-by: Neha Ojha <nojha@redhat.com>
2019-12-07 14:42:49 -06:00
Sage Weil
552f5109ce Merge PR #31581 into master
* refs/pull/31581/head:
	mon: fix mon_sync_max_payload_size type
	mon: cap keys in mon_sync messages
	mon/MonitorDBStore: improve get_chunk_tx limits
	mon/MonitorDBStore: better size estimation for Transaction

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
2019-12-07 12:35:58 -06:00
Sage Weil
d91cb4270d Merge PR #32048 into master
* refs/pull/32048/head:
	mon/PGMap: use NONE for 'pg ls[-*]' output too

Reviewed-by: Kefu Chai <kchai@redhat.com>
2019-12-07 12:35:20 -06:00
Sage Weil
86950ce9aa Merge PR #32039 into master
* refs/pull/32039/head:
	test: Improve races by using kill_daemons which waits for OSDs terminate
	test: run-standalone.sh: Only run execs in the subdirectories of qa/standalone
	test: Use activate_osd() when restarting OSDs
	test: osd-scrub-snaps.sh: Fix race with osd restart and doing a scrub

Reviewed-by: Neha Ojha <nojha@redhat.com>
2019-12-07 12:28:15 -06:00
Sage Weil
7e1f445422 qa/tasks/mgr/test_orchestrator_cli: fix rgw add test
Signed-off-by: Sage Weil <sage@redhat.com>
2019-12-07 08:14:59 -06:00