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

Small Code Changes to Apply or Document #93

Open
VincenzoFerme opened this issue Aug 23, 2016 · 3 comments
Open

Small Code Changes to Apply or Document #93

VincenzoFerme opened this issue Aug 23, 2016 · 3 comments

Comments

@VincenzoFerme
Copy link
Member

VincenzoFerme commented Aug 23, 2016

Resulting data in the tables (refer to the BenchFlow.wfmsTest.1.1 run):

  • network_interface_data has only zeros
  • what is exp_byte_size?
  • where the metrics computed using the network data are stored?
  • faban_run_info: the status is null (you forgot the resultinfo file as described in Description of Faban data #52)
  • the transformation of the responseTimes section of the summary.xml does not work, because the resulting data do not contain the percentiles for example. This is due to the fact you miss to detect the different percentiles identified by nth probably:
<responseTimes unit="microseconds">
            <operation name="isOutYet">
                <avg>589.741</avg>
                <max>3526.234</max>
                <sd>164.058</sd>
                <percentile nth="25" suffix="th">461168601842738.750</percentile>
                <percentile nth="50" suffix="th">461168601842738.750</percentile>
                <percentile nth="75" suffix="th">461168601842738.750</percentile>
                <percentile nth="90" suffix="th" limit="9223372036854776000.000">461168601842738.750</percentile>
                <percentile nth="95" suffix="th">461168601842738.750</percentile>
                <percentile nth="99.9" suffix="th">461168601842738.750</percentile>
                <passed>true</passed>
            </operation>
</responseTimes>
  • Still to check, because empty: container_properties, host_properties, trial_ram, trial_cpu_core, exp_ram, exp_cpu_core, trial_cpu, exp_cpu
  • Fix to container_properties: add labels, NetworkSettings, aufs in log driver is wrong, it is the volumes driver, image should be read from the config block
  • Remove cpu_avg from trial_cpu_core, because we have cpu_mean

Testing:

  • Faban when the execution happens on multiple hosts
@Cerfoglg
Copy link
Contributor

Cerfoglg commented Aug 23, 2016

@VincenzoFerme

  1. I don't know why network_interface_data is only 0s, it takes what's given by Docker, and if not that what nethogs reports.
  2. exp_byte_size contains the metrics computed over the database sizes of the trials of that experiment.
  3. Currently we don't compute metrics over the network data
  4. Status is also written in the summary file, if it's null it's because it's not reported somehow.
  5. I had no idea that was even a possibility, and frankly there is no way to take all possibilities into account when they are this unpredictable. It's either we have these well defined or not, or else it becomes difficult to manage.
  6. Those tables ended up empty because properties failed, and thus ram and cpu scripts failed to find the properties data (cpu core counts, max memory) they needed.

@VincenzoFerme
Copy link
Member Author

@Cerfoglg

  1. Investigate. I verified the raw data, and they seems correct
  2. Ok, I forgot :)
  3. Ok, Openend Add Metrics Computed over the Network Data analysers#95
  4. Where is this described on the faban documentation linked in Description of Faban data #52. I don't find it.
  5. Just check for the right structure for the percentiles. if the entity <percentile> has the <nth> attribute, you also need to take it into account to differentiate when you generate the name of the metric to store the value in the cassandra table. Otherwise you overwrite it.
  6. Yes, fine. This is a note for me, I'm going to check them after a new run

@Cerfoglg
Copy link
Contributor

@VincenzoFerme I believe the last issue listed was cleared

@VincenzoFerme VincenzoFerme self-assigned this Sep 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants