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

DAOS-14411 control: Expose dfuse statistics as json #13876

Merged
merged 1 commit into from
Feb 27, 2024
Merged

Conversation

ashleypittman
Copy link
Contributor

  • DAOS-14411 control: Expose dfuse statistics as yaml.

Required-githooks: true

Signed-off-by: Ashley Pittman <[email protected]>
Copy link

github-actions bot commented Feb 26, 2024

Ticket title is 'Improved visibility into dfuse internals.'
Status is 'In Progress'
Labels: 'usability'
https://daosio.atlassian.net/browse/DAOS-14411

@ashleypittman ashleypittman changed the title amd/go stats DAOS-14411 control: Expose dfuse statistics as yaml Feb 26, 2024
@ashleypittman ashleypittman marked this pull request as ready for review February 26, 2024 14:02
@ashleypittman ashleypittman requested review from a team as code owners February 26, 2024 14:02
Copy link
Contributor

@mjmac mjmac left a comment

Choose a reason for hiding this comment

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

Code changes LGTM, but the subject was a little confusing... I think you just need to s/yaml/json/, right?

@ashleypittman ashleypittman changed the title DAOS-14411 control: Expose dfuse statistics as yaml DAOS-14411 control: Expose dfuse statistics as json Feb 26, 2024
@ashleypittman ashleypittman requested a review from a team February 27, 2024 13:49
@ashleypittman ashleypittman merged commit d4c9bda into master Feb 27, 2024
51 checks passed
@ashleypittman ashleypittman deleted the amd/go-stats branch February 27, 2024 13:50
jolivier23 pushed a commit that referenced this pull request Apr 26, 2024
Multiple cherry-picks to add daos fs query feature for fuse
statistics.  Helpful for understanding and tuning DAOS
performance when dfuse is used.

DAOS-13625 dfuse: Merge the info and projection_info structs. (#11881)
DAOS-13658 dfuse: Add filesystem query command. (#12367)
DAOS-12751 control: Add a daos filesystem evict command. (#12331)
DAOS-12751 dfuse: Improve evict command. (#12633)
DAOS-13625 dfuse: Remove dfuse_projection_info entirely. (#12796)
DAOS-13625 dfuse: Replace fs_handle with dfuse_info. (#12894)
DAOS-13625 dfuse: Add core inode_lookup() and inode_decref() functions. (#12573)
DAOS-14411 dfuse: Add per-container statistics. (#12819)
DAOS-14411 control: Expose dfuse statistics as yaml. (#13876)

Features: dfuse

Required-githooks: true

Change-Id: I8ae3cc743697c2434ae0d54b382ee6c585a3b033
Signed-off-by: Ashley Pittman <[email protected]>
Signed-off-by: Jeff Olivier <[email protected]>
jolivier23 added a commit that referenced this pull request Apr 29, 2024
Multiple cherry-picks to add daos fs query feature for fuse
statistics.  Helpful for understanding and tuning DAOS
performance when dfuse is used.

DAOS-13625 dfuse: Merge the info and projection_info structs. (#11881)
DAOS-13658 dfuse: Add filesystem query command. (#12367)
DAOS-12751 control: Add a daos filesystem evict command. (#12331)
DAOS-12751 dfuse: Improve evict command. (#12633)
DAOS-13625 dfuse: Remove dfuse_projection_info entirely. (#12796)
DAOS-13625 dfuse: Replace fs_handle with dfuse_info. (#12894)
DAOS-13625 dfuse: Add core inode_lookup() and inode_decref() functions. (#12573)
DAOS-14411 dfuse: Add per-container statistics. (#12819)
DAOS-14411 control: Expose dfuse statistics as yaml. (#13876)

Changed base branch to google/2.4 for daos_build test

Change-Id: I8ae3cc743697c2434ae0d54b382ee6c585a3b033

Signed-off-by: Ashley Pittman <[email protected]>
Signed-off-by: Jeff Olivier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants