Skip to content

README: doc metrics details #10

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

Merged
merged 1 commit into from
Jul 15, 2024
Merged

Conversation

synarete
Copy link
Collaborator

@synarete synarete commented Jun 6, 2024

Provide detailed information on exported smbmetrics counters, with minimal 'howto'.

Depends on PR9

@synarete synarete requested a review from obnoxxx June 6, 2024 11:24
Copy link

@obnoxxx obnoxxx left a comment

Choose a reason for hiding this comment

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

looks good in general but one request regarding markdown.

README.md Outdated
@@ -6,8 +6,73 @@ Samba metrics exporter converts 'smbstatus' output into

## Build

```bash
```sh
Copy link

Choose a reason for hiding this comment

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

I think consoleis preferred over bashand shfor shell output. the others are for scripts.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fair enough -- fixed.

Provide detailed information on exported smbmetrics counters, with
minimal 'howto'.

Signed-off-by: Shachar Sharon <[email protected]>
Copy link

@obnoxxx obnoxxx left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@obnoxxx obnoxxx merged commit 8848207 into samba-in-kubernetes:main Jul 15, 2024
7 checks passed
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