Skip to content

Case: add stress reboot test-case - #6

Closed
Bin-QA wants to merge 1 commit into
thesofproject:masterfrom
Bin-QA:case
Closed

Bin-QA wants to merge 1 commit into
thesofproject:masterfrom
Bin-QA:case

Conversation

@Bin-QA

@Bin-QA Bin-QA commented Dec 18, 2019

Copy link
Copy Markdown
Contributor

Test case reboot for the stress test type

This case base on #3 #4 #5 change

Signed-off-by: Wu, BinX binx.wu@intel.com

@xiulipan xiulipan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Few questions inline.

Comment thread test-case/stress-reboot.sh Resolved
sudo sof-boot-once.sh export DMESG_LOG_START_LINE=$(wc -l /var/log/kern.log|awk '{print $1;}')

# run this script for next boot up round
dlogi "sof-boot-onece.sh ${BASH_SOURCE[0]} -l $loop_count" |tee -a $STRESS_OUTPUT_LOG

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
dlogi "sof-boot-onece.sh ${BASH_SOURCE[0]} -l $loop_count" |tee -a $STRESS_OUTPUT_LOG
dlogc "sof-boot-onece.sh ${BASH_SOURCE[0]} -l $loop_count" |tee -a $STRESS_OUTPUT_LOG

if [ 0 -ne 0 ];then
# last line add failed keyword
sed -i '$s/$/ fail/' $STRESS_STATUS_LOG
dlogi "$i: fail" |tee -a $STRESS_OUTPUT_LOG

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you hijack the dlog* cmd to wrap the |tee -a $STRESS_OUTPUT_LOG when STRESS_OUTPUT_LOG is defined?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

that's the good idea, I will try it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

check with #5 , I don't think here need to modify
because when run from remote, the nohup command will handle all output
from the local it don't need to handle it

here use tee command because after system reboot,
the nohup command is miss, Because current script is run in rc.local which called by root permission

Test case reboot for the stress test type

Signed-off-by: Wu, BinX <binx.wu@intel.com>
@Bin-QA

Bin-QA commented Dec 24, 2019

Copy link
Copy Markdown
Contributor Author

Will new design the test-case current close this PR

@Bin-QA Bin-QA closed this Dec 24, 2019
@Bin-QA
Bin-QA deleted the case branch January 14, 2020 07:53
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