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

Add ODM test #433

Merged
merged 1 commit into from
Jun 4, 2018
Merged

Add ODM test #433

merged 1 commit into from
Jun 4, 2018

Conversation

llxia
Copy link
Contributor

@llxia llxia commented Jun 1, 2018

  • override existing run_ilog_with_gcmv.sh because the original
    file expects certain Java path

Signed-off-by: lanxia lan_xia@ca.ibm.com

@llxia
Copy link
Contributor Author

llxia commented Jun 1, 2018

@@ -0,0 +1,206 @@
#!/bin/bash

Copy link
Contributor

Choose a reason for hiding this comment

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

Need to add apache license here also.


echo "***** Running Benchmark Script *****"

export PRODUCT=pxa6480sr6
Copy link
Contributor

Choose a reason for hiding this comment

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

eventually may clean up more of the odd references, but good for a start

@llxia
Copy link
Contributor Author

llxia commented Jun 1, 2018

Thanks Shelley. Copyright is added.

xsi:noNamespaceSchemaLocation="../TestConfig/playlist.xsd">
<test>
<testCaseName>odm</testCaseName>
<command>chmod a+x ./scripts/benchmark.sh; \
Copy link
Contributor

@sophia-guo sophia-guo Jun 1, 2018

Choose a reason for hiding this comment

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

Does tests have platform requirements?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Perf test needs run on specific machine. So far I only have linux to test. The scripts and cmd need to be updated as we test on other platforms.

Copy link
Contributor

@sophia-guo sophia-guo Jun 1, 2018

Choose a reason for hiding this comment

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

Sounds good. I was thinking if <platformRequirements> is needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure, it can be add it.

- override existing run_ilog_with_gcmv.sh because the original
file expects certain Java path

Signed-off-by: lanxia <lan_xia@ca.ibm.com>
@llxia llxia merged commit a95fdcc into adoptium:master Jun 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants