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

Initial Snabb Switch Reference Manual #338

Merged
merged 5 commits into from
Dec 19, 2014

Conversation

eugeneia
Copy link
Member

I am admittedly a tiny bit proud to present to you.... a fairly complete Snabb Switch Reference Manual!

By "fairly complete" I mean most parts of the Snabb Switch distribution are documented, and it should already be a useful resource for Snabb Switch hackers, old and new.

What's still missing:

  • I did not document the ipv6.ipv6 app. It looks like a stub/example to me? Uses none of the protocol libraries and as far as I can tell it mostly responds to pings. Not sure.
  • Not all libraries (src/lib) are documented yet. If you think a specific library should be in the documentation, let me know!
  • I probably forgot something: You can either add it yourself or tell me to do it.
  • Proof reading is highly appreciated! Don't attempt to read the whole thing at once please, might be dry. ;)

Merry Christmas!

@eugeneia eugeneia mentioned this pull request Dec 16, 2014

* `duration` - Duration in seconds to run the engine for (as a floating
point number). If this is set you cannot supply `done`.
* `done` - A function to be called repeatedly by the until it
Copy link
Contributor

Choose a reason for hiding this comment

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

"repeatedly by the" .. engine?

@plajjan
Copy link
Contributor

plajjan commented Dec 16, 2014

I read it all at once... massive. Nice work! Answers a lot of stuff I had to find out the hard way. I'll read again but regardless I vote for merge as it's lightyears better than the current situation. Can always fix/add more later on.

BTW, have you considered something like readthedocs.org or gitbook.io for keeping a fairly good-looking document up to date with latest git repo?

@eugeneia
Copy link
Member Author

Thanks for the proof-read, I didn't expect so much valuable feedback so fast! =) As for your suggestions: Yes to all cases! 👍

BTW, have you considered something like readthedocs.org or gitbook.io for keeping a fairly good-looking document up to date with latest git repo?

We could let SnabbBot render the current documentation automatically. I don't consider that very important though, because if the docs change too often (besides stylistic fixes and new additions) that would be a bad sign since they are written as a contract for the user. So including rendered versions with the releases seems good enough to me for now.

Regarding other doc tools: I am actually quite fond of our tool chain. The docs automatically embed into GitHub (albeit, ToC rendering would be nice). They are managed just like any other source file so it should be easy to keep them up to date with the code. And then we heavily use a Ditaa pre-processing step to be able to easily embed graphics using a funny ASCII syntax (its a bit buggy but generally fun to work with).

lukego added a commit that referenced this pull request Dec 19, 2014
Initial Snabb Switch Reference Manual
@lukego lukego merged commit 7f2c230 into snabbco:master Dec 19, 2014
lukego added a commit that referenced this pull request Jan 6, 2015
dpino added a commit to dpino/snabb that referenced this pull request Jun 2, 2016
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