Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pacific: mgr/volumes: A few mgr volumes pacific backports #45205

Merged
merged 5 commits into from Mar 22, 2022

Conversation

kotreshhr
Copy link
Contributor


  1. Backport Tracker: https://tracker.ceph.com/issues/54332
    Parent Tracker: https://tracker.ceph.com/issues/54121
    Parent PR: mgr/volumes: Inherit file quota attr to clone #44873

  1. Backport Tracker: https://tracker.ceph.com/issues/54335
    Parent Tracker: https://tracker.ceph.com/issues/54099
    Parent PR: mgr/volumes: Fix subvolumegroup ls #44854

  1. Backport Tracker: https://tracker.ceph.com/issues/54256
    Parent Tracker: https://tracker.ceph.com/issues/54066
    Parent PR: mgr/volumes: Fix clone uid/gid mismatch #44970

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@kotreshhr kotreshhr requested a review from a team March 1, 2022 06:17
@github-actions github-actions bot added cephfs Ceph File System pybind tests labels Mar 1, 2022
@github-actions github-actions bot added this to the pacific milestone Mar 1, 2022
@kotreshhr kotreshhr requested review from vshankar and ajarr March 1, 2022 06:17
@kotreshhr
Copy link
Contributor Author

jenkins test make check

@yuriw yuriw modified the milestones: pacific, octopus Mar 3, 2022
Copy link
Contributor

@ajarr ajarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR looks good. This needs testing.

@ajarr
Copy link
Contributor

ajarr commented Mar 14, 2022

@yuriw this PR is meant for pacific. Why was this PR added to octopus milestone instead of pacific?

@kotreshhr
Copy link
Contributor Author

jenkins test make check

@yuriw yuriw modified the milestones: octopus, pacific Mar 15, 2022
Fixes: https://tracker.ceph.com/issues/54121
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit 96c7963)
The file quota attribute 'ceph.quota.max_files'
is not inherited to the cloned subvolume. This
patch fixes the same.

Fixes: https://tracker.ceph.com/issues/54121
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit 72671c8)
Fixes: https://tracker.ceph.com/issues/54099
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit 95a83ef)
The subvolumegroup ls listed '_deleting' directory which is
internal to 'mgr/volumes' and should not be listed as a
subvolumegroup. This patch fixes the same by filtering it.

Fixes: https://tracker.ceph.com/issues/54099
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit a125b0e)
This is the regression caused by commit 18b85c5.
The 'set_attrs' function sets the uid/gid of the
group to the subvolume if uid/gid is not passed.
The attrs of the clone should match the source
snapshot. Hence, don't use the 'set_attrs'
function to set only the quota attrs for the
clone.

Fixes: https://tracker.ceph.com/issues/54066
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit b3c9e6b)
@yuriw yuriw merged commit 104cbff into ceph:pacific Mar 22, 2022
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants