summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 15.2.14v15.2.14Jenkins Build Slave User2021-08-052-1/+7
|
* Merge pull request #42599 from ideepika/wip-fix-symlinkYuri Weinstein2021-08-041-1/+0
|\ | | | | | | | | | | octopus: qa/upgrade: drop broken symlink, introduced in octopus only qa Reviewed-by: Yuri Weinstein <yweinste@redhat.com> Reviewed-by: Neha Ojha <nojha@redhat.com>
| * qa/upgrade: drop broken symlinkDeepika Upadhyay2021-08-041-1/+0
| | | | | | | | | | | | | | | | octopus-only: this change was introduced in: c2486c7239f2efff1f87a0c6064ccbf792e90bf0 as a linking unintentional typo while creating symlink, the parent commit was octopus only hence we don't require it in other branches. Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>
* | Merge pull request #42618 from neha-ojha/wip-42421-2-octopusYuri Weinstein2021-08-041-1/+1
|\ \ | | | | | | | | | | | | | | | octopus: qa/*/test_envlibrados_for_rocksdb.sh: fix libarchive dependency Reviewed-by: Yuri Weinstein <yweinste@redhat.com>
| * | qa/*/test_envlibrados_for_rocksdb.sh: fix libarchive dependencyNeha Ojha2021-08-031-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes failures like the following with rhel 8.3 in octopus ``` 2021-08-03T17:32:19.328 INFO:tasks.workunit.client.0.smithi148.stdout:No match for argument: libarchive-3.3.3 2021-08-03T17:32:19.338 INFO:tasks.workunit.client.0.smithi148.stderr:Error: Unable to find a match: libarchive-3.3.3 2021-08-03T17:32:19.376 DEBUG:teuthology.orchestra.run:got remote process result: 1 2021-08-03T17:32:19.377 INFO:tasks.workunit:Stopping ['rados/test_envlibrados_for_rocksdb.sh'] on client.0... ``` Follow-up to https://github.com/ceph/ceph/pull/42421 Signed-off-by: Neha Ojha <nojha@redhat.com>
* | Merge pull request #42586 from rhcs-dashboard/wip-51995-octopusErnesto Puerta2021-08-035-75/+120
|\ \ | |/ |/| | | | | | | | | | | | | | | octopus: mgr/dashboard: cephadm-e2e job script: improvements Reviewed-by: Waad Alkhoury <walkhour@redhat.com> Reviewed-by: Aashish Sharma <aasharma@redhat.com> Reviewed-by: Avan Thakkar <athakkar@redhat.com> Reviewed-by: Ernesto Puerta <epuertat@redhat.com> Reviewed-by: Nizamudeen A <nia@redhat.com> Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>
| * mgr/dashboard: cephadm-e2e script: improvementsAlfonso Martínez2021-08-025-75/+120
|/ | | | | | | | | | | | | | | | Improvements and some adaptations related to the jenkins job. Fixes: https://tracker.ceph.com/issues/51612 Signed-off-by: Alfonso Martínez <almartin@redhat.com> (cherry picked from commit 65b75000b7694cb3cbe617bbec28c513a2522be8) Conflicts: doc/dev/developer_guide/dash-devel.rst - Put changes in HACKING.rst as this file does not exist in the octopus branch. src/pybind/mgr/dashboard/ci/cephadm/bootstrap-cluster.sh - Resolve conflict originated by code that is deleted anyway. Signed-off-by: Alfonso Martínez <almartin@redhat.com>
* Merge pull request #42509 from smithfarm/wip-51837-octopusYuri Weinstein2021-07-301-2/+1
|\ | | | | | | | | | | | | octopus: rpm: drop use of $FIRST_ARG in ceph-immutable-object-cache Reviewed-by: Ilya Dryomov <idryomov@redhat.com> Reviewed-by: Kefu Chai <kchai@redhat.com>
| * rpm: drop use of $FIRST_ARG in ceph-immutable-object-cacheNathan Cutler2021-07-271-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The use of $FIRST_ARG was probably required because the SUSE-specific %service_* rpm macros were playing tricks on the shell positional parameters. This is bad practice and error-prone, so let's assume that no macros should do that anymore and hence it's safe to assume that positional parameters remain unchanged after any rpm macro call. Thanks to Franck Bui for providing the original patch 926433f5d45e557c42f050b43798ba29dc495e02 that this patch is modeled after. NOTE: the use of FIRST_ARG had already been eliminated by 926433f5d45e557c42f050b43798ba29dc495e02 but was re-introduced later by 9466d7098573dafcfede5e9c852374fbbd99f9e7 Fixes: 9466d7098573dafcfede5e9c852374fbbd99f9e7 Fixes: https://tracker.ceph.com/issues/51797 Signed-off-by: Nathan Cutler <ncutler@suse.com> (cherry picked from commit 1cb84a1160ed4108cae30100682b1e3ee7c7721d)
* | Merge pull request #42412 from votdev/wip-51730-octopusYuri Weinstein2021-07-3026-15/+622
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | octopus: mgr/dashboard: Add configurable MOTD or wall notification Reviewed-by: Tatjana Dehler <tdehler@suse.com> Reviewed-by: Laura Paduano <lpaduano@suse.com>
| * | mgr/dashboard: Add configurable MOTD or wall notificationVolker Theile2021-07-2126-15/+622
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/51408 Signed-off-by: Volker Theile <vtheile@suse.com> (cherry picked from commit f7f163e75cf5fb6cd022a8d13c28f5b923e01aed) Conflicts: src/pybind/mgr/dashboard/frontend/src/app/core/navigation/navigation/navigation.component.ts src/pybind/mgr/dashboard/frontend/src/app/shared/api/motd.service.spec.ts src/pybind/mgr/dashboard/frontend/src/app/shared/components/alert-panel/alert-panel.component.html src/pybind/mgr/dashboard/frontend/src/app/shared/components/alert-panel/alert-panel.component.ts src/pybind/mgr/dashboard/frontend/src/app/shared/components/components.module.ts src/pybind/mgr/dashboard/frontend/src/app/shared/pipes/sanitize-html.pipe.spec.ts src/pybind/mgr/dashboard/frontend/src/app/shared/services/motd-notification.service.ts src/pybind/mgr/dashboard/frontend/src/app/shared/services/motd-notification.service.spec.ts src/pybind/mgr/dashboard/module.py src/pybind/mgr/dashboard/plugins/motd.py src/python-common/tox.ini src/python-common/ceph/utils.py
* | | Merge pull request #42484 from MrFreezeex/wip-51841-octopusYuri Weinstein2021-07-301-1/+1
|\ \ \ | | | | | | | | | | | | | | | | octopus: osd: log snaptrim message to dout Reviewed-by: Kefu Chai <kchai@redhat.com>
| * | | osd: log snaptrim message to doutArthur Outhenin-Chalandre2021-07-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This log message is not an error and is done on every tick of the snaptrim process. Replace the derr logging to dout(10) to not log it by default. Fixes: https://tracker.ceph.com/issues/51799 Signed-off-by: Arthur Outhenin-Chalandre <arthur.outhenin-chalandre@cern.ch> (cherry picked from commit e2b2faef11c18df6c2f1f083d32d279e51b63e82)
* | | | Merge PR #42537 into octopusPatrick Donnelly2021-07-297-17/+157
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/42537/head: mon/MDSMonitor: propose if FSMap struct_v is too old mon/MDSMonitor: give a proper error message if FSMap struct_v is too old qa: add tests for fs dump of epoch and trimming qa: add file system support for dumping epoch mon/MDSMonitor: return mon_mds_force_trim_to even if equal to current epoch mon: add debugging for trimming methods mon: fix debug spacing Reviewed-by: Ramana Raja <rraja@redhat.com> Reviewed-by: Neha Ojha <nojha@redhat.com>
| * | | | mon/MDSMonitor: propose if FSMap struct_v is too oldPatrick Donnelly2021-07-294-3/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To flush older versions which may still be an empty MDSMap (for clusters that have never used CephFS), we need to force a proposal so older versions of the struct are trimmed. This is the main fix of this branch. We removed code which processed old encodings of the MDSMap in the mon store via 60bc524. That broke old ceph clusters which never used CephFS (see cited ticket below). This is because the initial epoch is an empty MDSMap (back in Infernalis/Hammer) that is never updated. So, the fix here is to just do proposals periodically until all of the old structs are automatically trimmed by the mons. Fixes: 60bc524827bac072658203e56b1fa3dede9641c5 Fixes: https://tracker.ceph.com/issues/51673 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit 56c3fc802ee8848ba85da4300adcc2ee8bd95416) Conflicts: src/mds/FSMap.cc: adjust for octopus which decodes old MDSMaps src/mon/MDSMonitor.h: trivial conflicts
| * | | | mon/MDSMonitor: give a proper error message if FSMap struct_v is too oldPatrick Donnelly2021-07-291-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/51673 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit 4298f9758138c30e08975a84ced25d81e1fd53b9)
| * | | | qa: add tests for fs dump of epoch and trimmingPatrick Donnelly2021-07-291-0/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit 5ddaa36d17c875a8ab7a6445f5ab0917d9a2dffd) Conflicts: qa/tasks/cephfs/test_admin.py: trivial
| * | | | qa: add file system support for dumping epochPatrick Donnelly2021-07-291-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit ee899d9a44f3ab73c8ebbb82720d89364b9bb600)
| * | | | mon/MDSMonitor: return mon_mds_force_trim_to even if equal to current epochPatrick Donnelly2021-07-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The PaxosService code already excludes the value returned by PaxosService::get_trim_to as the upper bound of the range of epochs to trim. Without this fix, you need to set mon_mds_force_trim_to to one greater than the epoch you want to trim _and_ force the current epoch to be one greater than that; the net result being that you can only force trimming up to 2 epochs behind the current epoch. This change is helpful for resolving issue 51673, but not strictly necessary. Related-to: https://tracker.ceph.com/issues/51673 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit d9dc2f11d56fb4341ba5823f8d17459d10f3b2c1) Conflicts: src/common/options/mon.yaml.in: drop doc change
| * | | | mon: add debugging for trimming methodsPatrick Donnelly2021-07-292-9/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit 9297690e9a4158e1ba3a1bdea924e86b586a390c) Conflicts: src/mon/PaxosService.cc: trivial operator change
| * | | | mon: fix debug spacingPatrick Donnelly2021-07-291-1/+4
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Patrick Donnelly <pdonnell@redhat.com> (cherry picked from commit 147c27c033005f311c7fdce0b80b57f2de47674e) Conflicts: src/mon/PaxosService.cc: add dout msg
* | | | Merge pull request #42374 from ifed01/wip-ifed-bluefs-safer-flush-octYuri Weinstein2021-07-284-29/+123
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | octopus: os/bluestore: Remove possibility of replay log and file inconsistency Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>
| * | | | os/bluestore/bluefs: Add test that detects bluefs inconsistencyAdam Kupczyk2021-07-233-0/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add test that detects possible scenario that will cause BlueFS to have file that contains data that has never been written. This is done by tricking replay log to already accept file metadata (size, allocations), but actual data stored in these allocations is not yet synced to disk. Scenario: 1) write to file h1 on SLOW device 2) flush h1 (and trigger h1 mark to be added to bluefs replay log) 3) write to file h2 4) fsync h2 (forces replay log to be written) The result is: - bluefs log now has stable state of h1 - SLOW device is not yet flushed (no fdatasync()) Test detects this condition and fails. Signed-off-by: Adam Kupczyk <akupczyk@redhat.com> (cherry picked from commit c591a6e14e2c956d268adcaa9aa3e9c8a1fdea2a)
| * | | | os/bluestore/bluefs: Remove possibility of bluefs replay log containing ↵Adam Kupczyk2021-07-203-29/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | files without data It had been possible to have a bluefs replay log to serialize file metadata (size, allocations), but actual data stored in these allocations is not yet synced to disk. This could happen if _flush_range(h1) allocated space for file h1 on device (like SLOW) that will not be used when flushing future replay log. Such thing can happen when we have h2 that wrote to WAL and out replay log is on DB. After fsync(h2) we write to replay log, wait for fdatasync on WAL and DB. There is no waiting on SLOW, but h1 was dirty and has been serialized to replay log. Solution is to delay notifying replay log that it has to include h1 after finishing fdatasync. Fixes: https://tracker.ceph.com/issues/50965 Signed-off-by: Adam Kupczyk <akupczyk@redhat.com> (cherry picked from commit 03ac53f7d4c83e56f664ad371ffe3bc2d40e1837) Conflicts: (trivial - additional future stuff src/os/bluestore/BlueFS.cc
* | | | | Merge pull request #42375 from ifed01/wip-ifed-compact-after-upgrade-octYuri Weinstein2021-07-282-1/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | octopus: os/bluestore: compact db after bulk omap naming upgrade Reviewed-by: Neha Ojha <nojha@redhat.com>
| * | | | | os/bluestore: compact db after bulk omap naming upgrade.Igor Fedotov2021-07-162-1/+12
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Omap naming scheme upgrade introduced recently might perform bulk data removal and hence leave DB in a "degraded" state. Let's compact it. Signed-off-by: Igor Fedotov <ifedotov@suse.com> (cherry picked from commit 0e5c140b79a0b1809a8044696dce1eb7a36b1d41)
* | | | | Merge pull request #42478 from tchaikov/octopus-50456Yuri Weinstein2021-07-287-285/+303
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | octopus: bind on loopback address if no other addresses are available Reviewed-by: Josh Durgin <jdurgin@redhat.com>
| * | | | common/pick_addr: use grading machinery to refactor pick_address()Kefu Chai2021-07-251-38/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | as picking iface on the same NUMA node is not a hard requirement, the grading machinery is a nice fit for this purpose. Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit 329d51c68ec6bf1864aa9430a62d65a93362a1b9)
| * | | | common/pick_address: prefer non-loopback addressesKefu Chai2021-07-254-74/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instead of filtering out loopback ifaces, check for loopback addresses, and prefer non-loopback addresses over loopback addresses. before this change, iface named "lo" is filtered out by default, and "lo" is allowed if `ms_bind_exclude_lo_iface` is false. after this change, iface with address out of 127/8 is prefered. the iface marked down is not considered. the option of "ms_bind_exclude_lo_iface" is removed. the tests are updated accordingly. Fixes: https://tracker.ceph.com/issues/50456 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit a9b9bcd53215a07608a28ac2c8e4a8c8b8e80e66) Conflicts: src/common/options/global.yaml.in src/common/pick_address.cc: trivial resolution
| * | | | common/pick_address: Allow binding on loopback ifaceKefu Chai2021-07-254-20/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in 6147c0917157efd2d35610e759685656a4989abb, "lo" is also skipped when daemon is trying to find an address to bind. but that change reverts the fix of 201b59204374ebdab91bb554b986577a97b19c36, to address the problem. an option named "ms_bind_exclude_lo_iface" is added, it defaults to "true". but it can be changed to false to allow daemon to bind on "lo". Fixes: https://tracker.ceph.com/issues/50012 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit 7f01d36a2ca0576f1ff103ae3fa7c3662e93b722)
| * | | | common/pick_address: document find_ip_in_subnet_list()Kefu Chai2021-07-251-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit b106ec0bbf7fa726062989114f461f2d0a1f93a9)
| * | | | common/pick_address: pass string by referenceKefu Chai2021-07-251-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to silence warnings from clang-tidy. Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit 6d0ed81f796209f27b96811f9140b7fff16a7940)
| * | | | common/pick_addr: refactor pick_address.cc and ipaddr.ccKefu Chai2021-07-254-228/+157
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * do not replicate the same logic in IPv4 and IPv6 paths * use helpers returning bool for filtering the candidate addresses for better readability Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit 52785d5a3607b2f2ee6d41069d18a154b3eb5d45) Conflicts: src/common/ipaddr.cc src/common/pick_address.cc: trivial resolution
| * | | | common/pick_address: use scope_guard for freeifaddrs()Kefu Chai2021-07-251-11/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | for better readability Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit c3c110b5763ac420c4b88f8a545c1c87a71ce59a)
| * | | | common/pick_address: fail if cannot bind with specified network familyKefu Chai2021-07-253-149/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this change partially reverts 9f75dfbf364f5140b3f291e0a2c6769bc3d8cbac we should not proceed against user's will if dual stack is specified but only one network for a network family can be found. the right fix is have better error message and documentation, not to tolerate the failure. Fixes: https://tracker.ceph.com/issues/46845 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit d752acafa0d99c3d7cacfaaaf3ae51770e251aff)
| * | | | pick_address: Warn and continue when you find at least 1 IPv4 or IPv6 addressMatthew Oliver2021-07-255-8/+178
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently if specify a single public or cluster network, yet have both `ms bind ipv4` and `ms bind ipv6` set daemons crash when they can't find both IPs from the same network: unable to find any IPv4 address in networks '2001:db8:11d::/120' interfaces '' And rightly so, of course it can't find an IPv4 network in an IPv6 network. This patch, adds a new helper method, networks_address_family_coverage, that takes the list of networks and returns a bitmap of address families supported. We then check to see if we have enough networks defined and if you don't it'll warn and then continue. Also update the network-config-ref to mention having to define both address family addresses for cluster and or public networks. As well as a warning about `ms bind ipv4` being enabled by default which is easy to miss, there by enabling dual stack when you may only be expect single stack IPv6. Thee is also a drive by to fix a `note` that wan't being displayed due to missing RST syntax. Signed-off-by: Matthew Oliver <moliver@suse.com> Fixes: https://tracker.ceph.com/issues/46845 Fixes: https://tracker.ceph.com/issues/39711 (cherry picked from commit 9f75dfbf364f5140b3f291e0a2c6769bc3d8cbac)
* | | | Merge pull request #42377 from ifed01/wip-ifed-migrate-octYuri Weinstein2021-07-276-14/+2285
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | octopus: ceph-volume: implement bluefs volume migration. Reviewed-by: Kefu Chai <kchai@redhat.com> Reviewed-by: Dimitri Savineau <dsavinea@redhat.com>
| * | | | tests/ceph_volume: add UT for bluefs migration stuffIgor Fedotov2021-07-161-0/+1504
| | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Igor Fedotov <ifedotov@suse.com> (cherry picked from commit f8def0443db59e7df31132953fff708b76417236)
| * | | | ceph-volume: implement bluefs volume migration.Igor Fedotov2021-07-164-1/+756
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a wrapper over ceph-bluestore-tool's bluefs-bdev-migrate command. Primarily intended to introduce LVM tags manipulation which ceph-bluestore-tool is lacking. Signed-off-by: Igor Fedotov <ifedotov@suse.com> (cherry picked from commit 58efeb915198d4fbb40b6fa080312d8bee3141bf) Conflicts: doc/man/8/ceph-volume.rst (old format) src/ceph-volume/ceph_volume/api/lvm.py (renamed get_first_lv method)
| * | | | os/bluestore/bluestore_tool: compare retval stat() with -1Kefu Chai2021-07-161-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | before this change, stat() is always called to check if the file specified by --dev-target exists even if this option is not specified. also, we compare the retval of stat() with ENOENT, while state() returns -1 on error. after this change, stat() is called only if --dev-target is specified, and we compare the retval of stat() with -1 and 0 only, so if --dev-target option is not specified, the tool still hehaves. this change addresses a regression introduced by 94a91f54fe30a4dd113fbc1b02bc3f3d52c82a92 Fixes: https://tracker.ceph.com/issues/50891 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit d4c65a368c9cf35e01604fc3321f867cbe3e4109)
| * | | | tools/ceph-bluestore-tool: be more legible before requesting additional paramsIgor Fedotov2021-07-161-13/+22
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Request DB/WAL size specification when relevant devices are created only. Signed-off-by: Igor Fedotov <ifedotov@suse.com> (cherry picked from commit 94a91f54fe30a4dd113fbc1b02bc3f3d52c82a92)
* | | | Merge pull request #42368 from trociny/wip-51698-octopusYuri Weinstein2021-07-273-0/+92
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | octopus: rgw: allow to set ssl options and ciphers for beast frontend Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | | rgw: set default ssl options for beast frontendMykola Golub2021-07-163-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to 'no_sslv2:no_sslv3:no_tlsv1:no_tlsv1_1' Signed-off-by: Mykola Golub <mgolub@suse.com> (cherry picked from commit fb31c87c2d6c02563d2d2a1e63d5b62bea2c6f91) Conflicts: PendingReleaseNotes (added to 15.2.14 section)
| * | | | rgw: allow to set ssl options and ciphers for beast frontendMykola Golub2021-07-162-0/+80
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two new conf keys are added for "beast" framework: - ssl_options: a colon separated list of ssl context options, documented in boost's ssl::context_base; - ssl_ciphers: a colon separated list of ciphers, documented in openssl's ciphers(1) manual. Example: rgw frontends = beast ... ssl_options=default_workarounds:no_tlsv1:no_tlsv1_1 ssl_ciphers=HIGH:!aNULL:!MD5 Fixes: https://tracker.ceph.com/issues/50932 Signed-off-by: Mykola Golub <mgolub@suse.com> (cherry picked from commit 91abede6357d167063c63eade45421d2f17bb0e7)
* | | | Merge pull request #40383 from singuliere/wip-49745-octopusYuri Weinstein2021-07-271-4/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | octopus: rgw: limit rgw_gc_max_objs to RGW_SHARDS_PRIME_1 Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | | rgw: limit rgw_gc_max_objs to RGW_SHARDS_PRIME_1Rafał Wądołowski2021-03-241-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't allow GC to process more gc ojects than RGW_SHARDS_PRIME_1 Fixes: https://tracker.ceph.com/issues/49321 Signed-off-by: Rafał Wądołowski <rwadolowski@cloudferro.com> (cherry picked from commit 7b163048d93a078e2354665488a27225042d3f1a)
* | | | | Merge pull request #39904 from singuliere/wip-49378-octopusYuri Weinstein2021-07-270-0/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | octopus: cmake: build static libs if they are internal ones Reviewed-by: Kefu Chai <kchai@redhat.com>
| * | | | | cmake: build static libs if they are internal onesKefu Chai2021-03-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | there are chances that user or build script set `BUILD_SHARED_LIBS`, so these convenience libraries (using the autotools' terminology) are built and linked by never get installed. Fixes: https://tracker.ceph.com/issues/38611 Fixes: https://tracker.ceph.com/issues/49080 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit df841b241efd387044d9637b1cf67d198bd1398e) Conflicts: src/blk/CMakeLists.txt introduced in pacific https://github.com/ceph/ceph/pull/34622/files
* | | | | | Merge pull request #42440 from smithfarm/wip-51769-octopusYuri Weinstein2021-07-271-56/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | octopus: rpm: three spec file cleanups Reviewed-by: Kefu Chai <kchai@redhat.com>
| * | | | | | rpm: cleanup: drop useless conditional block in %postun baseNathan Cutler2021-07-211-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "meat" of this conditional was ripped out by 328807f80bb6b5d1aa40631e88d755a194d5d2c2, leaving only an empty shell behind. Signed-off-by: Nathan Cutler <ncutler@suse.com> (cherry picked from commit 3b53003f011cfbe51d3471ab9b6cdb9a24ecd4f7)