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

Capture on link instead of port #1117

Closed
julien-duponchelle opened this issue Mar 14, 2016 · 6 comments
Closed

Capture on link instead of port #1117

julien-duponchelle opened this issue Mar 14, 2016 · 6 comments
Assignees
Milestone

Comments

@julien-duponchelle
Copy link
Contributor

Actually when you start a capture we ask for the interface where the capture should happen. But I'm not sure if it's usefull. Because in theory if GNS3 work the capture should be the same on both side.

My proposal is to remove the choice and behind the scene the network backend can decide where to capture, this mean the api for capture will be on /links in the controller instead of the network adapter.

The current implementation doesn't create issue but I imagine stuff like packet loss configuration will be on the link instead of the port. And in this case it seem logic to have all the packet manipulation in the link API.

@julien-duponchelle julien-duponchelle added this to the 1.5 milestone Mar 14, 2016
@julien-duponchelle
Copy link
Contributor Author

A note on a serial link the capture type should be asked it could be FRAME RELAY, PPP and HDLC

@grossmj
Copy link
Member

grossmj commented Mar 14, 2016

I think I am fine with it excepting if someone see a specific usage I missed.

@Ehlers, anyone?

@ghost
Copy link

ghost commented Mar 14, 2016

I see only one use case for capturing on the port: Debugging if the link is working.
But normally it's sufficient, looking at the interface counters.

So yes, I'm fine with that proposal as well.

@julien-duponchelle
Copy link
Contributor Author

In theory all link should work in GNS3 you should not have to debug GNS3 internals. I think I we need it we can edit the code to force the capture on one side of the link.

@grossmj
Copy link
Member

grossmj commented Mar 23, 2016

Ok, let's do it then.

@grossmj grossmj changed the title Capture on link instead of port? Capture on link instead of port Apr 6, 2016
@julien-duponchelle julien-duponchelle self-assigned this Apr 21, 2016
julien-duponchelle added a commit to GNS3/gns3-server that referenced this issue Apr 21, 2016
@julien-duponchelle
Copy link
Contributor Author

In the same time we need to process most of the capture controller side (write in the captures directory).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants