-
Notifications
You must be signed in to change notification settings - Fork 136
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
Service chaining example #60
Comments
Hi @twood02, I would like to take a stab at this. Is examples/test_flow_dir good place to add this feature? I.e., in addition to its current behaviour, I feel the second approach would be simpler, but when users want to check if their flow rules got actually applied, they would need packet steering capability as well. So, perhaps the first option is more practical. |
Hi @knodir, as you have noticed it would be useful for the test flow dir to have this functionality as we could test the added rules and also it wouldn't create a whole new NF which is cleaner to maintain. We're always happy to see people contributing so If you have any questions feel free to ping me |
Thanks for the quick response, @koolzz. I'll get started with adding this functionality to |
relates to issue sdnfv#60
@knodir Hello, just wanted to follow up on this and see if this is still in progress or if you need any help/feedback |
Oh, hi @catherinemeadows! It seems I moved on to the other parts of the code and never completed git commit 9025f45. I'll get back to this sometime this weekend. Let me know if you need it more urgently. |
@knodir Sounds great! No rush, just wanted to check in. Let us know if you have any questions. |
Hi @knodir are you currently working on this? If not I would like to volunteer to take this up. |
Hi @bdevierno1, Yes, please go ahead to take over this. I tried to complete this couple weeks ago but the environment I had during my 9025f45 commit development is not there anymore. So, I don't have a proper setup to test this feature anymore. Feel free to use parts of my half-baked changes in 9025f45. Thanks for chiming in! |
@stevensse21 is going to try this. Step 1:
Use the flow_director API to change the service chain rule. |
This issue describes a simple bug/improvement for the OpenNetVM code, and is a great way for YOU to get involved with improving this open source project! If you want to fix it, just create a pull request with the fix as detailed in the Contributing document. If you have questions, post here!
By default, OpenNetVM creates a default service chain with the action "send to NF with service ID 1". However, when the manager starts it prints "Default service chain: send to sdn NF". We should revise this comment to say "send to service ID 1"
https://github.com/sdnfv/openNetVM/blob/develop/onvm/onvm_mgr/onvm_init.c#L249
We also should create a new NF that demonstrates setting up different default service chains, or service chains for specific flows. This NF would show how to use the Flow Director API. The NF could be very simple -- it might just start up, create some new rules with the flow director, and then exit (the rules will still apply to new traffic that arrives).
The text was updated successfully, but these errors were encountered: