| Commit message (Collapse) | Author | Files | Lines |
|---|
|
Fixes: https://tracker.ceph.com/issues/69191
https://tracker.ceph.com/issues/69605
Signed-off-by: Dnyaneshwari Talwekar <dtalweka@redhat.com>
|
|
Correct the English in a note in doc/cephadm/host-management.rst.
This commit was made in response to
https://github.com/ceph/ceph/pull/50340, which corrected some of this
note in Quincy.
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
In doc/cephfs/disaster-recovery-experts.rst, incorporate Anthony's
suggestions in
https://github.com/ceph/ceph/pull/61462#discussion_r1923917812
and
https://github.com/ceph/ceph/pull/61462#discussion_r1923920724
and reword the sentences in the section "Using an alternate metadata
pool for recovery" to be in the imperative mood, which better suits the
ordered list format that was introduced in
https://github.com/ceph/ceph/pull/61493.
Follows https://github.com/ceph/ceph/pull/61493.
https://tracker.ceph.com/issues/69557
Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
For the "profile {name}" syntax, pool and namespace restrictions are
independent of each other (i.e. specifying namespace doesn't also
require specifying pool like is currently suggested). A cap can look
like "profile rbd namespace=myns", signifying that the RBD profile is
to be allowed in myns namespace of any pool.
For the "allow {access-spec}" syntax, pool restriction is optional.
A cap can look like "allow r namespace=myns", "allow w object_prefix
myprefix" or "allow rw namespace=myns object_prefix myprefix", for
example.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
|
|
Signed-off-by: Aviv Caro <133020857+caroav@users.noreply.github.com>
Update nvmeof-requirements.rst
Signed-off-by: Aviv Caro <133020857+caroav@users.noreply.github.com>
Update nvmeof-initiator-linux.rst
Signed-off-by: Aviv Caro <133020857+caroav@users.noreply.github.com>
Update nvmeof-initiator-esx.rst
Signed-off-by: Aviv Caro <133020857+caroav@users.noreply.github.com>
Update nvmeof-target-configure.rst
Signed-off-by: Aviv Caro <133020857+caroav@users.noreply.github.com>
doc/rbd: fix broken .rst
Fix .rst errors introduced in https://github.com/ceph/ceph/pull/61477.
This commit will be squashed.
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
Use the RAII wrappers for the dump Formatters,
instead of manually opening and closing sections.
Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
|
|
Put the procedure in the section called "Using an alternate metadata
pool for recovery" into an ordered list, so that it is in a proper
procedure format.
This commit is meant only to break the procedure into steps. The English
language in each of these steps could be improved, but that improvement
will be done after this formatting has been merged and backported.
Follows https://github.com/ceph/ceph/pull/61462.
https://tracker.ceph.com/issues/69557
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
release builds were using the wrong container repo name because of
confused variable naming and inadequate separation. Keep the hostname,
org name, and repo name in separate variables, and assemble the full
path with a version when tagging is done.
Signed-off-by: Dan Mick <dan.mick@redhat.com>
|
|
Image::aio_close() must clear ctx before initiating close. Otherwise
the provided callback may see a non-NULL ctx and attempt to close the
image again from Image destructor, leading to an invalid memory access
as ImageCtx and ImageState are both freed immediately after the image
is closed (i.e. before AioCompletion is completed and the callback is
executed).
The same adjustment is made to Image::close() just for consistency.
Fixes: https://tracker.ceph.com/issues/69619
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
|
|
Fixes: https://tracker.ceph.com/issues/65743
Co-authored-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: junxiang Mu <1948535941@qq.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
|
|
Fixes: https://tracker.ceph.com/issues/69607
Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
|
|
Edit the seventh and final section of
doc/cephfs/disaster-recovery-experts.rst in preparation for adding
deeper explanations of the contexts in which one should use the various
commands listed on that page.
The section edited in this commit is
* Using an alternate metadata pool for recovery
A future commit might beneficially put this section into the format of
an ordered list. If so, such a commit should only reformat the
content and should not make any changes to the English. It's enough to
verify content or format. Let's not overload our editorial faculties by
forcing ourselves to walk and chew gum at the same time.
Follows https://github.com/ceph/ceph/pull/61442
https://tracker.ceph.com/issues/69557
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
Signed-off-by: Ali Masarwa <amasarwa@redhat.com>
|
|
This commit consolidates all necessary changes to address unit tests
related to the introduction of Python type annotations.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This check is already performed in `arg_validators.py`.
By the way, this check is only valid when the user passes a VG/LV with
the `<vg>/<lv>` format.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.batch`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.listing`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
- Rename the parameter `arg` to `osd` for better readability.
- Simplify the check for numeric OSD identifiers using `isdigit()`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.common`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.prepare`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.trigger`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.main`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.deactivate`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.create`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit adds the Python type annotations to `devices.lvm.activate`.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
This commit addresses some python type annotations errors
in `objectstore` code.
Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
|
|
Edit the fifth and sixth sections of
doc/cephfs/disaster-recovery-experts.rst in preparation for adding
deeper explanations of the contexts in which one should use the various
commands listed on that page.
The sections edited in this commit are
- MDS Map Reset
- Recovery From Mission Metadata Objects
Follows https://github.com/ceph/ceph/pull/61427
https://tracker.ceph.com/issues/69557
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
With commit#8c025045332a8005c6e82308fc17a33d38058734, changes
were made to call put_info while erasing bucket attrs. But
DBStore wasn't updating attrs as part of put_info operation.
This commit addresses the same.
Signed-off-by: Soumya Koduri <skoduri@redhat.com>
|
|
Properly wrap a poorly-formatted paragraph that looks just awful in an
80-column viewport and change MDS to "MDS daemons" where the latter
makes the sentence a lot clearer.
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
These changes improve the robustness and reliability of Lua script execution in the RADOS Gateway by enforcing configurable limit on execution time.
- Enhanced `lua_state_guard` to support configurable runtime limits for Lua scripts.
- Updated rgw.yaml.in to include new configuration options for Lua runtime limits.
- Added tests in `test_rgw_lua.cc` to verify Lua script execution with different runtime constraints.
- Updated Lua scripting documentation to reflect the new runtime limit configuration.
Signed-off-by: Oshry Avraham <oshryabiz@gmail.com>
|
|
Edit the third and fourth sections of
doc/cephfs/disaster-recovery-experts.rst in preparation for adding
deeper explanations of the contexts in which one should use the various
commands listed on that page.
Follows https://github.com/ceph/ceph/pull/61426
https://tracker.ceph.com/issues/69557
Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
Even though we opt in right away after upgrading, which minimizes the chance
of this warning coming up, the cluster log still detects it sometimes and
fails the test. So, we can ignore it since it's expected for the warning
to come up for a few seconds in some cases.
Signed-off-by: Laura Flores <lflores@ibm.com>
|
|
1. For the telemetry upgrade tests, we want to
make sure we enable the perf channel before we
upgrade to test that out.
2. We also want to be sure to set up ident information.
in the workunits used before the upgrade.
3. We can remove the "opt out" commands in the before-upgrade
workunits since we want to stay opted in during the upgrade.
We only opt out after all the checks are complete.
4. We mistakenly test "ceph telemetry show" twice when one
of then should be "ceph telemetry show-all".
Signed-off-by: Laura Flores <lflores@ibm.com>
|
|
Rephrase the balancer-related warning issued to people upgrading to
Ceph 19.2.0.
Follows https://github.com/ceph/ceph/pull/61429
Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com>
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
Since main is now pre-Tentacle, we should replace the quincy upgrade
path with squid.
I have kept the quincy workunits since they are still
referenced in the overall upgrade suite. I will remove these
in a separate PR since changing the overall upgrade tests reaches
beyond the scope of just telemetry.
Signed-off-by: Laura Flores <lflores@ibm.com>
|
|
This comment was mistakenly carried over in
https://github.com/ljflores/ceph/commit/6d25c053127f6aaaf942e674dca30ffd42658b9d.
Signed-off-by: Laura Flores <lflores@ibm.com>
|
|
This commit changes the workunit that tests the upgrade path from
reef to x.
The logic to verify collections is now much more robust. Previously,
the test relied on developers to make it aware of any new collections
it needed to verify. Now, I have added logic to check that the # of
known collections matches the # of reported collections. If it does not,
the test errors out, which will force developers to add any new
collections to the upgrade test along with adding it to the telemetry
code.
The test also verifies all collections instead of focusing only on new
collections that were added after the base collections.
Signed-off-by: Laura Flores <lflores@ibm.com>
|
|
Add the API `process_expired_objects` which is used to process and
delete the expired objects (swift or temporarily restored from cloud)
to the Zipper interface.
Signed-off-by: Soumya Koduri <skoduri@redhat.com>
|
|
This change is to update 19.2.0 Squid release documentation with a warning for users regarding potential placement group balancer issues.
Signed-off-by: Laimis Juzeliunas <laimis.juzeliunas@oxylabs.io>
|
|
Remove a link to a tracker for a planned-but-never-developed tool that
would have facilitated the importation or exportation of a RADOS pool to
a file. This is a part of a project to improve the CephFS Disaster
Recovery documentation, which was requested by Eugen Block. See the
tracker issue below for more on this project.
Follows https://github.com/ceph/ceph/pull/61411
See https://github.com/ceph/ceph/pull/61411#discussion_r1918525109
https://tracker.ceph.com/issues/69557
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
Fixes https://tracker.ceph.com/issues/69554
Signed-off-by: Gil Bregman <gbregman@il.ibm.com>
|
|
since it is only a suggestion for debian.
this is a regression from: https://github.com/ceph/ceph/pull/60666
Fixes: https://tracker.ceph.com/issues/69473
Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
|
|
Edit the first two sections of doc/cephfs/disaster-recovery-experts.rst
in preparation for adding deeper explanations of the contexts in which
one should use the various commands listed on that page.
https://tracker.ceph.com/issues/69557
Signed-off-by: Zac Dover <zac.dover@proton.me>
|
|
Typo in the text for the "Unmanaged" option in Create Service.
Fixes: https://tracker.ceph.com/issues/69550
Signed-off-by: N Balachandran <nibalach@redhat.com>
|
|
Signed-off-by: abhishek-kane <abhishek.kane@ibm.com>
|
|
NVMeoF gateway version is independent of ceph version
so 'ceph version' shows wrong nvmeof version in output
(i.e. instead of gateway version, it shows Ceph version).
Hence, remove nvmeof in 'ceph versions' output.
To check for gateway version, use 'gw info' command.
Signed-off-by: Vallari Agrawal <vallari.agrawal@ibm.com>
|
|
10008/tcp is not opened
Fixes: https://tracker.ceph.com/issues/69541
Signed-off-by: Shweta Bhosale <Shweta.Bhosale1@ibm.com>
|
|
In handle_replication_status_header() when the COMPLETED status is
being set to object's attr, it needs to be reflected to the current
live attrs for the corresponding request so it can be reflected in
the first try than the second.
Fixes: https://tracker.ceph.com/issues/69549
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
|
|
bucket_source_sync_checkpoint() requires bucket id to lookup for
full-sync object and the dest pipe doesn't have it. by replcaning
the bucket key in the pipe with the bucket info we will ensure the
bucket id is set.
Fixes: https://tracker.ceph.com/issues/69548
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
|
|
Signed-off-by: Ali Masarwa <amasarwa@redhat.com>
|
