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

OML support #206

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

OML support #206

wants to merge 5 commits into from

Conversation

kohoumas
Copy link

OML is a library ( http://oml.mytestbed.net/projects/oml/wiki/ ) which could be used for measurements gathering, adopted by many EU FIRE / US GENI projects and supported by NICTA, Australia. I have used many times Click together with OML, and I really think that it is a useful complementary tool.

@kohler
Copy link
Owner

kohler commented Jul 30, 2015

Looks pretty good! Now I don't know much about OML. Does --oml actually override the tcpdump-style elements (provide the libpcap interface), or are other elements required?

I do have one request. GNU standards would sort of request that this be enabled by --with-oml, not --enable-oml. Can you make that change?

@@ -993,6 +993,13 @@ dnl
dnl more features
dnl

dnl OML

AC_ARG_ENABLE(oml, [[ --enable-oml enable OML]], :, enable_oml=no)
Copy link
Owner

Choose a reason for hiding this comment

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

Probably wants to be AC_ARG_WITH (see later for examples)

Copy link
Author

Choose a reason for hiding this comment

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

I think this is done

2015-07-30 18:26 GMT+03:00 Eddie Kohler [email protected]:

In configure.in
#206 (comment):

@@ -993,6 +993,13 @@ dnl
dnl more features
dnl

+dnl OML
+
+AC_ARG_ENABLE(oml, [[ --enable-oml enable OML]], :, enable_oml=no)

Probably wants to be AC_ARG_WITH (see later for examples)


Reply to this email directly or view it on GitHub
https://github.com/kohler/click/pull/206/files#r35882953.

@kohoumas
Copy link
Author

The OML library is used to send measurements (throughput, CPU utilization,
etc) to a specific OML server.
The experimenter defines the measurements in its own elements, and then
these elements are able to send measurements to the server, which stores
them in a database.
I think that is sth completely different from the tcpdump-style elements,
that store the packet details in the tcpdump format.

Regarding the change to --with-oml, this is done!

2015-07-30 18:25 GMT+03:00 Eddie Kohler [email protected]:

Looks pretty good! Now I don't know much about OML. Does --oml actually
override the tcpdump-style elements (provide the libpcap interface), or are
other elements required?

I do have one request. GNU standards would sort of request that this be
enabled by --with-oml, not --enable-oml. Can you make that change?


Reply to this email directly or view it on GitHub
#206 (comment).

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.

2 participants