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

Meshconfig support for PuNDIT #13

Open
batistaj opened this issue Dec 15, 2016 · 8 comments
Open

Meshconfig support for PuNDIT #13

batistaj opened this issue Dec 15, 2016 · 8 comments
Assignees

Comments

@batistaj
Copy link

It would be helpful if the Meshconfig GUI user were able to enter some PuNDIT related information. Something like the following, where there would be a checkbox to enable several PuNDIT parameters and corresponding fields for the user to fill in.

__ PuNDIT
OWAMP duration (ms) __________
Packets per test __________
PuNDIT central hostname __________

Would it be possible to provide something like this?

Thanks,
Jorge

@soichih
Copy link
Contributor

soichih commented Dec 15, 2016

@batistaj Should I add this under OWAMP test specification? Also.. once it's added to owamp, how should I expose this on the meshconfig itself? Can you provide some keys for each fields?

Also, if you could give me some user-friendly descriptions on each fields, I can add that to the UI also.

@batistaj
Copy link
Author

Hi Soichi,

I think that is reasonable.

Thanks,
Jorge

@soichih
Copy link
Contributor

soichih commented Dec 21, 2016

@arlake228 I am updating the owamp testspec editor for MCA, but I need a bit more information on PuNDIT parameters. Will it be documented on the mesh config doc anytime soon?

@soichih
Copy link
Contributor

soichih commented Jan 30, 2017

@batistaj @arlake228 I still need more information before I can work on this item. I am guessing that, those PuNDIT related parameters needs to be officially defined in the mesh config doc?

@arlake228
Copy link
Collaborator

The PunDIT parameters are supported with a special field that was just recently added to master of the meshconfig code. You can add a custom option to the <test> definition using the reference field as follows:

<test>
  description       PunDIT Packet Loss Tests
  group             pundit_hosts
  test_spec         owamp_test
  <reference>
      name pundit_server
      value 10.0.0.1
   </reference>
</test>

The name and value can take any valid string. I don't know that @batistaj and co even know that's there yet, so not sure they have defined the exact fields they need yet, but they will need to define the exact name/value pairs they want for pundit.

@soichih
Copy link
Contributor

soichih commented Apr 3, 2017

@arlake228 I still don't see any PunDIT related information here (http://docs.perfsonar.net/release_candidates/4.0rc3/config_mesh.html) but will this be added to v4? Is there anything that I should add to MCA at this point?

@apertome
Copy link
Contributor

Removing this from the 4.1b1 timeframe, need to focus on the basic functionality before we do custom/specialized things like this. Also might warrant a wider discussion on how to support extra/custom parameters in general.

@ShawnMcKee
Copy link

This item is not needed for PunDIT but we may want to keep it as a more generic feature request.

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

No branches or pull requests

5 participants