Skip to content

Conversation

@synarete
Copy link
Contributor

Samba's commit 9f8d272 ("vfs_ceph_new: use per-share profile macros") enables per-share profile counters for VFS ceph bridge. Enable those by default for each smb-ceph share.

Show available Jenkins commands

Copy link
Contributor

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

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

lgtm.

We don't have any tests specific to metrics yet. In that case I hope we can skip adding this new parameter to share configuration from existing test files.

Samba's commit 9f8d272 ("vfs_ceph_new: use per-share profile macros")
enables per-share profile counters for VFS ceph bridge. Enable those by
default for each smb-ceph share.

Signed-off-by: Shachar Sharon <ssharon@redhat.com>
@synarete synarete force-pushed the ss-mgr-smb-pershare-profile branch from ff8e777 to 450702a Compare July 7, 2025 07:32
@anoopcs9
Copy link
Contributor

anoopcs9 commented Jul 7, 2025

jenkins test make check

Copy link
Contributor

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

Looks OK to me.

@phlogistonjohn
Copy link
Contributor

I am working on doing a teuthology test on three tagged PR, this one included but none of my builds yesterday succeeded. I plan on trying again today.

@synarete synarete merged commit 99def68 into ceph:main Jul 9, 2025
13 checks passed
@phlogistonjohn
Copy link
Contributor

https://pulpito.ceph.com/phlogistonjohn-2025-07-09_19:16:01-orch-wip-phlogistonjohn-testing-1-2025-07-09-1301-distro-default-gibba/

I ran a teuthology test of the smb orch tests and of the two real failures both appeared to be intermittent flakes. No issues related to this change were seen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants