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

Document rtrmon a bit #28

Merged
merged 1 commit into from
Oct 25, 2021
Merged

Document rtrmon a bit #28

merged 1 commit into from
Oct 25, 2021

Conversation

ties
Copy link
Collaborator

@ties ties commented Oct 17, 2021

Merge after #22 because this uses the new port.

I realised there was no documentation on how to run rtrmon in the readme, I added some.

@ties ties requested review from job and lspgn October 17, 2021 09:05
@ties ties force-pushed the feature/doc-rtrmon-a-bit branch from c7768c3 to 9e8592b Compare October 17, 2021 09:08
@job
Copy link
Member

job commented Oct 24, 2021

@ties would you mind re-doing this PR? I had to mess with the history... apologies!

  * Add how to run rtrmon in the README
  * Basic explanation of rtrmon metrics in the README
@ties ties force-pushed the feature/doc-rtrmon-a-bit branch from 9e8592b to bd1a7d2 Compare October 25, 2021 17:47
@ties
Copy link
Collaborator Author

ties commented Oct 25, 2021

@job fixed. I don't know another public rtr server (and people may end up using it in production - so it's not a service I would like to run), so I kept this one.

@job job merged commit f7828ab into master Oct 25, 2021
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