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

Add distributed lock manager singlestat files #21

Merged
merged 1 commit into from
Apr 20, 2017

Conversation

joehandzik
Copy link
Contributor

Signed-Off-By: Joe Handzik joseph.t.handzik@hpe.com

Signed-Off-By: Joe Handzik <joseph.t.handzik@hpe.com>
@joehandzik
Copy link
Contributor Author

@roclark Please take a look when you have a moment. The highlights:

  1. Applied some really light heuristics to clean up the OST name (as I type that, should we change that label to "OST" rather than "OSS"?)
  2. Created the directoryDepth concept, to deal with the fact that some of the ldlm elements are an extra directory level away.

Suggestions are always appreciated, feel free to merge it yourself if you're ok with the changes.

@roclark
Copy link
Contributor

roclark commented Apr 20, 2017

Everything looks good to me.

On your point about changing OSS to OST, after another look, yes, I agree with you. We will also want to change MGS -> MGT, MDS -> MDT.

@roclark roclark closed this Apr 20, 2017
@roclark roclark reopened this Apr 20, 2017
@roclark roclark merged commit c68d569 into master Apr 20, 2017
@joehandzik joehandzik deleted the wip-ldlm-single-stats branch April 24, 2017 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants