DPDK CI discussions
 help / color / mirror / Atom feed
From: "Tu, Lijuan" <lijuan.tu@intel.com>
To: Jeremy Plsek <jplsek@iol.unh.edu>, "ci@dpdk.org" <ci@dpdk.org>
Cc: "dpdklab@iol.unh.edu" <dpdklab@iol.unh.edu>
Subject: Re: [dpdk-ci] GA Baseline Performance Testing
Date: Sat, 29 Sep 2018 02:21:35 +0000	[thread overview]
Message-ID: <8CE3E05A3F976642AAB0F4675D0AD20EEE65A4@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <CA+xUZB4XgFSRuV2qGW48sj__vJLFgaTzfA1T8COyxQ+xhzpDdA@mail.gmail.com>

Hi Jeremy,

Agree with that each individual delta result has a tag (such as dpdk version), but I have some comments on the DTS part.
'--update-expected v18.05.1' and `--delete-expected v17.11.4` are not make sense, as DTS is a test suite, it carries out the test cases based on the DPDK and give out results, but it doesn't care about DPDK version.
Could you consider my idea:
	- Each GA release expected results should be saved. 
	- run every single test with the stored expected result.
	- update delta result in database.

Take v17.11.4 as an example,
Firstly run DTS based DPDK-v17.11.4 to get the expected numbers which stored in conf/nic_single_core.cfg, and save the file. 
Secondly in runtime, just replace the configure file with the stored one, and then run DTS.
Thirdly update the results to database under unique tag(maybe v17.11.4)

Keep listening to you.
Thanks
Lijuan

> -----Original Message-----
> From: ci [mailto:ci-bounces@dpdk.org] On Behalf Of Jeremy Plsek
> Sent: Friday, September 28, 2018 11:40 PM
> To: ci@dpdk.org
> Cc: dpdklab@iol.unh.edu
> Subject: [dpdk-ci] GA Baseline Performance Testing
> 
> Hello all,
> 
> It was requested that we add a delta test result of the latest GA to the result
> table. I said that it could be accomplished relatively easily during the meeting.
> After some thought and laying out a plan, adding this result will take much
> longer than expected.
> 
> If this ends up being a requirement before the results become public, it may
> take much longer before the results become public.
> 
> Below are the steps that will have to be taken to get this working.
> 
> 1. The member supplied test harness (such as DTS) will have to update the
> `--update-expected` argument that expects an arbitrary value (such as
> `master` or `v18.05.1`). This would save the absolute numbers as before, but
> with appropriate keys. This will allow deltas to be generated for each of
> those keys.
> 2. With the change above, the test harness will have to output a table of
> deltas for each key defined.
> 3. Add an option to delete said keys (such as `--delete-expected
> v17.11.4`) so we don’t have to generate more tables than needed.
> 4. Update the database to expect the individual delta results to contain these
> tags. Old results would be migrated to a `master` tag.
> 5. Update the dashboard to provide a table of results for each unique tag.
> 6. Update current jobs with the new test harness parameters (which would
> use `--update-expected master`, for example).
> 7. Create a new script that finds the latest tag from dpdk-stable, builds, and
> uploads to our database appropriately (which would use `--update-expected
> v18.05.1`, for example). Then delete older "expected" keys.
> 8. Create GA performance Jenkins Job to run this script.
> 9. Done? (Everything should be automatic at this point?)
> 
> Implementation specifics will be added to the policy document if members
> would like to go through with this.
> 
> This may also help pave the way for comparing GA versions as well (as was
> also requested). Currently, with the data we have, we can compare the
> performance delta of master. (I made a quick-and-dirty script that plotted
> master over time before the presentation at the Summit, but my logic was
> slightly off, so I didn’t present it.)
> 
> If anyone has a better way of doing this, please feel free to provide
> suggestions.
> 
> I estimate that this will take at least a month to add these delta results.
> 
> Thanks
> --
> Jeremy Plsek
> UNH InterOperability Laboratory

      reply	other threads:[~2018-09-29  2:21 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-28 15:39 Jeremy Plsek
2018-09-29  2:21 ` Tu, Lijuan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8CE3E05A3F976642AAB0F4675D0AD20EEE65A4@SHSMSX101.ccr.corp.intel.com \
    --to=lijuan.tu@intel.com \
    --cc=ci@dpdk.org \
    --cc=dpdklab@iol.unh.edu \
    --cc=jplsek@iol.unh.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).