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

Polish project documentation #542

Merged
merged 22 commits into from
Mar 4, 2022
Merged

Polish project documentation #542

merged 22 commits into from
Mar 4, 2022

Conversation

osinstom
Copy link
Contributor

@osinstom osinstom commented Mar 3, 2022

This PR updates the project documentation, describing the current state of the project.

@osinstom osinstom self-assigned this Mar 3, 2022

### Complete
**BESS-UPF**
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@pudelkoM could you verify the feature list for BESS-UPF?

Copy link
Member

Choose a reason for hiding this comment

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

What comes to mind are:

  • Simple QoS support with slice and session meters
  • Per-flow latency and throughout metrics

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added.

ccascone
ccascone previously approved these changes Mar 4, 2022
Copy link
Contributor

@ccascone ccascone left a comment

Choose a reason for hiding this comment

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

Overall looks good. Please feel free to incorporate my suggestions as you see fit.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
Tomasz Osiński and others added 9 commits March 4, 2022 10:09
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
@osinstom osinstom requested a review from ccascone March 4, 2022 17:45
ccascone
ccascone previously approved these changes Mar 4, 2022
Copy link
Contributor

@ccascone ccascone left a comment

Choose a reason for hiding this comment

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

More minor suggestions. It would be good to have an explanation of the content of each directory, but that go in a future PR. Feel free to merge when ready.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Tomasz Osiński and others added 2 commits March 4, 2022 20:04
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
Co-authored-by: Carmelo Cascone <carmelo@opennetworking.org>
@osinstom osinstom merged commit 3c91d71 into master Mar 4, 2022
@osinstom osinstom deleted the update-readme branch March 4, 2022 19:10
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.

3 participants