summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #44649 from cyx1231st/wip-crimson-improve-log-cachemasterYingxin2022-01-2411-270/+392
|\ | | | | | | | | crimson/os/seastore: consolidate seastore_t and seastore_cache logs Reviewed-by: Samuel Just <sjust@redhat.com>
| * crimson/os/seastore/journal: add logs to flush operationYingxin Cheng2022-01-211-0/+4
| | | | | | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * crimson/os/seastore/cache: consolidate logs in levelsYingxin Cheng2022-01-213-78/+159
| | | | | | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * crimson/os/seastore: introduce seastore_t logsYingxin Cheng2022-01-215-51/+104
| | | | | | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * crimson/os/seastore: move transaction-id forward in logYingxin Cheng2022-01-211-4/+4
| | | | | | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * crimson/os/seastore/cache: misc cleanupYingxin Cheng2022-01-212-36/+24
| | | | | | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * crimson/os/seastore: cleanup effort_t and reuse Transaction::io_stat_tYingxin Cheng2022-01-213-64/+60
| | | | | | | | Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
| * crimson/os/seastore: consolidate Cache::init_cached_extent with drop_from_cacheYingxin Cheng2022-01-215-43/+43
| | | | | | | | | | | | | | So Cache::drop_from_cache() can be removed, and make it possible to add related logs within the Cache component. Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
* | Merge pull request #44572 from 5cs/fix-spdk-cmakeKefu Chai2022-01-222-7/+10
|\ \ | | | | | | | | | | | | cmake/modules/BuildSPDK.cmake: link whole-archive Reviewed-by: Kefu Chai <tchaikov@gmail.com>
| * | cmake/modules/BuildSPDK.cmake: link whole-archiveTongliang Deng2022-01-202-7/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We build spdk as static library, linking against them requires the use of `-Wl,--whole-archive` as argument, otherwise we will have error `nvme.c: nvme_probe_internal: *ERROR*: NVMe trtype 256 not available`. This is due to the use of constructor functions in spdk to register NVMe transports. So we need to do so to ensure we call all the constructors. Signed-off-by: Tongliang Deng <dengtongliang@sensetime.com>
* | | Merge pull request #44669 from idryomov/wip-rbd-switch-arguments-fixIlya Dryomov2022-01-2210-10/+10
|\ \ \ | | | | | | | | | | | | | | | | rbd: add missing switch arguments for recognition by get_command_spec() Reviewed-by: Mykola Golub <mgolub@suse.com>
| * | | rbd: add missing switch arguments for recognition by get_command_spec()Ilya Dryomov2022-01-1910-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently this $ rbd --all children img doesn't work, while this $ rbd children --all img or this $ rbd children img --all does. The issue is that -a/--all isn't on the list of known switch arguments. The "rbd children" example may seem contrived but for more complicated commands such as "rbd device map" mixing switches and positional arguments occurs naturally: $ rbd device --device-type nbd --options try-netlink --show-cookie map img Fixes: https://tracker.ceph.com/issues/53935 Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
* | | | Merge pull request #44736 from adamemerson/wip-no-using-namespace-stdKefu Chai2022-01-2229-831/+824
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | src: Remove some 'using namespace std' lines that crept back into headers Reviewed-by: Kefu Chai <tchaikov@gmail.com>
| * | | | neorados: Remove using namespace std::literals from headerAdam C. Emerson2022-01-212-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't actually need it. Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
| * | | | mgr: Remove straggling `using namespace std` from headersAdam C. Emerson2022-01-212-5/+3
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
| * | | | test: Remove straggling `using namespace std` from headersAdam C. Emerson2022-01-216-31/+47
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
| * | | | {osd,crimson}: Remove straggling `using namespace std` from headersAdam C. Emerson2022-01-217-386/+366
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
| * | | | rgw: Remove straggling `using namespace std` from headersAdam C. Emerson2022-01-2112-406/+405
|/ / / / | | | | | | | | | | | | | | | | | | | | A few crept back in. Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
* | | | Merge pull request #44734 from anthonyeleven/anthonyeleven/cephfs-pool-recszdover232022-01-221-14/+17
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | doc/cephfs: Clarify CephFS pool guidance Reviewed-by: Zac Dover <zac.dover@gmail.com>
| * | | | doc/cephfs: Clarify CephFS pool guidanceAnthony D'Atri2022-01-211-14/+17
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>
* | | | | Merge pull request #44509 from rzarzynski/wip-crimson-alienstore-drop-constantKefu Chai2022-01-221-1/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | crimson/alienstore: drop unused constant from AlienStore. Reviewed-by: Samuel Just <sjust@redhat.com> Reviewed-by: Chunmei Liu <chunmei.liu@intel.com> Reviewed-by: Kefu Chai <tchaikov@gmail.com>
| * | | | | crimson/alienstore: drop unused constant from AlienStore.Radoslaw Zarzynski2022-01-101-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
* | | | | | Merge pull request #44646 from neha-ojha/wip-backport-qNeha Ojha2022-01-213-3/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | script: backport updates for quincy Reviewed-by: Josh Durgin <jdurgin@redhat.com> Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>
| * | | | | | script: backport updates for quincyNeha Ojha2022-01-183-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Neha Ojha <nojha@redhat.com>
* | | | | | | Merge pull request #44265 from KellenRenshaw/WIP-replay_allocIgor Fedotov2022-01-211-2/+102
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test/allocator_replay_test: Add replay_alloc option Reviewed-by: Igor Fedotov <igor.fedotov@croit.io>
| * | | | | | | test/allocator_replay_test: Add replay_alloc optionKellen Renshaw2022-01-071-2/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | New replay_alloc option added to ceph_test_alloc_replay binary that permits the loading of a dump produced by "ceph daemon osd.<id> bluestore allocator dump block" and replaying a list of allocation requests against the loaded allocator state an arbitrary number of times (default, 100). Once the allocator dump is loaded, the fragmentation state and free space information are printed. Then the list of allocation requests is replayed against that state. Output consists of the time in ns that the allocator took to return the requested allocation, along with the request. If an allocation request fails or the list of allocation requests is completed, the error info, if any, and the fragementation and free space information is printed. The list of allocations is formatted as a file with one allocation request per line, with space separated values for "want", "unit", "max", and "hint". Values can be any integer format supported by std::scanf()'s %u formatter. The allocation request line format: <want> <unit> [max] [hint] Example request lines: 0x4000 0x4000 0x4000 0x0 0x18000 0x4000 0x18000 0x0 The "want" and "unit" values are required. Optional request values are 0 if not present in the request. Fixes: https://tracker.ceph.com/issues/53571 Signed-off-by: Kellen Renshaw <kellen.renshaw@canonical.com>
* | | | | | | | Merge pull request #44705 from ceph/wip-yuriw-build-scriptIlya Dryomov2022-01-211-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | script/build-integration-branch: add quincy to the list of releases Reviewed-by: Neha Ojha <nojha@redhat.com> Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
| * | | | | | | | qa/tests: add quincy name to teh wip nameYuri Weinstein2022-01-201-1/+1
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Yuri Weinstein <yweinste@redhat.com>
* | | | | | | | Merge pull request #44674 from adamemerson/wip-53941Adam C. Emerson2022-01-214-4/+62
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | rgw: Report empty endpoints as error instead of crashing Reviewed-By: Casey Bodley <cbodley@redhat.com>
| * | | | | | | rgw: Report empty endpoints as error instead of crashingAdam C. Emerson2022-01-204-4/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/53941 Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
* | | | | | | | Merge pull request #44706 from rzarzynski/wip-tests-bl-fix-rebuildNeha Ojha2022-01-211-5/+6
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test/bufferlist: ensure rebuild_aligned_size_and_memory() always rebuilds. Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com> Reviewed-by: Ilya Dryomov <idryomov@redhat.com> Reviewed-by: Neha Ojha <nojha@redhat.com>
| * | | | | | | | test/bufferlist: ensure rebuild_aligned_size_and_memory() always rebuilds.Radoslaw Zarzynski2022-01-211-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before the patch the test case was showing an unreliable behaviour dependent on the underlying memory allocator. It was because the bufferlist rebuild can be skipped, resulting in unchanged number of buffers, if all of them begin at aligned addresses. The commit fixes that by allocating a 4 KiB-aligned buffer and offsetting it by a small constant (42) to ensure the memory added to the bufferlist begins at non-4 KiB address. Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
* | | | | | | | | Merge pull request #44717 from ↵Casey Bodley2022-01-211-2/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | zdover23/wip-doc-2021-01-22-man-page-radosgw-admin-missing-apostrophe doc/man: radosgw-admin.rst: missing apostrophe Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | | | | | | | doc/man: radosgw-admin.rst: missing apostropheZac Dover2022-01-221-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds two missing apostrophes to two different strings. Hat tip to IcePic. Signed-off-by: Zac Dover <zac.dover@gmail.com>
* | | | | | | | | | Merge pull request #44694 from alimaredia/wip-rgw-kmip-teuthology-testAli Maredia2022-01-212-3/+32
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qa: move certificates for kmip task into /etc/ceph Reviewed-by: Al Maredia <amaredia@redhat.com>
| * | | | | | | | | | qa: move certificates for kmip task into /etc/cephAli Maredia2022-01-202-3/+32
| | |_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On rhel/centos the ceph user does not have permission to access these certs which leads to s3-test failures in teuthology. Signed-off-by: Ali Maredia <amaredia@redhat.com>
* | | | | | | | | | Merge pull request #44112 from zaitcev/wip-py_ssize_t_cleanPete Zaitcev2022-01-211-0/+1
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | mgr: Define PY_SSIZE_T_CLEAN ahead of every Python.h
| * | | | | | | | | mgr: Define PY_SSIZE_T_CLEAN ahead of every Python.hPete Zaitcev2021-12-151-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Building on Fedora 35 with Python 3.10 makes vstart to loop forever, throwing the following message: Error EINVAL: SystemError: PY_SSIZE_T_CLEAN macro must be defined for '#' formats I followed the hint in the following document: https://docs.python.org/3/c-api/intro.html It says "recommended" to always define PY_SSIZE_T_CLEAN, but as you can see it is actually required in our case. Fixes: https://tracker.ceph.com/issues/53441 Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
* | | | | | | | | | Merge pull request #44604 from mgfritch/c-v-skip-rbd-devsGuillaume Abrioux2022-01-212-0/+20
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | ceph-volume: show RBD devices as not available
| * | | | | | | | | | ceph-volume: filter RBD devices from the device inventoryMichael Fritch2022-01-202-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid running `blkid` or deploying OSDs on RBD devices by ensuring they do not appear in the `ceph-volume inventory` Fixes: https://tracker.ceph.com/issues/53846 Signed-off-by: Michael Fritch <mfritch@suse.com>
* | | | | | | | | | | Merge pull request #44687 from sebastian-philipp/govSebastian Wagner2022-01-211-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | doc/governance: Adam King Reviewed-by: Adam King <adking@redhat.com> Reviewed-by: Neha Ojha <nojha@redhat.com> Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
| * | | | | | | | | | doc/governance: Adam KingSebastian Wagner2022-01-201-1/+1
| | |_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Sebastian Wagner <sewagner@redhat.com>
* | | | | | | | | | Merge pull request #44042 from myoungwon/wip-add-blk-paddrSamuel Just2022-01-2037-242/+274
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | seastore: add blk_paddr_t Reviewed-by: Samuel Just <sjust@redhat.com>
| * | | | | | | | | | seastore/seastore_types: fix wrong maskingmyoungwon oh2022-01-191-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bit operation due to device_id_t is 8 bit for now Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
| * | | | | | | | | | seastore/random_block_manager: use paddr_t instead of uint64myoungwon oh2022-01-194-17/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reagarding read() and write(), use paddr_t as input parameter instead of uint64 Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
| * | | | | | | | | | seatore/random_block_manager: remove blocks_per_segmentmyoungwon oh2022-01-193-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
| * | | | | | | | | | seastore: changes due to adding blk_paddr_tmyoungwon oh2022-01-194-70/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
| * | | | | | | | | | seastore: add blk_paddr_tmyoungwon oh2022-01-192-29/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
| * | | | | | | | | | seastore: rename segment_off_t to seastore_off_tmyoungwon oh2022-01-1833-120/+121
| | |_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>
* | | | | | | | | | Merge pull request #44685 from rhcs-dashboard/fix-53950-masterErnesto Puerta2022-01-203-5/+4
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgr/dashboard: remove feedback module as always-on Reviewed-by: Alfonso Martínez <almartin@redhat.com> Reviewed-by: Avan Thakkar <athakkar@redhat.com> Reviewed-by: Nizamudeen A <nia@redhat.com>
View Lorry Controller Status