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

Adding rt test plan (New) #1170

Merged
merged 4 commits into from
Apr 10, 2024
Merged

Adding rt test plan (New) #1170

merged 4 commits into from
Apr 10, 2024

Conversation

fernando79513
Copy link
Collaborator

Description

To test the real-time kernel, we have to include RT tests into the generic providers and create a SRU specific to RT devices.

The rt-performance tests are already in canonical/checkbox under the base provider and the TSN and TGPIO tests will be included in the future.

At the moment, the sru-rt will only include the rt-performance tests, but the TSN and TGPIO will be added in the future.

Since the rt-performance-tests-stress takes 24 hours to complete, the sru-rt only includes rt-performance-tests-automated with a shorter version of the cyclic test for latency. Once we have tested the sru-rt on some devices and we ensure it is stable, we should probably add this test back, but taking into consideration that test runs using this test are going to take more than one day.

Resolved issues

Documentation

There are no changes to the documentation

Tests

To list the RT jobs currently present on checkbox:

checkbox-cli expand com.canonical.certification::rt-performance-tests-full

To list the jobs currently in sru-rt:

checkbox-cli expand com.canonical.certification::sru-rt
...
Job 'com.canonical.certification::recovery_info_attachment'
Job 'com.canonical.certification::rt-performance-tests/cyclictest_rt'
Job 'com.canonical.certification::stress/cpu_stress_ng_test'
...

Copy link

codecov bot commented Apr 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.04%. Comparing base (ab47668) to head (311c3df).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1170   +/-   ##
=======================================
  Coverage   43.04%   43.04%           
=======================================
  Files         355      355           
  Lines       38610    38610           
  Branches     6556     6556           
=======================================
  Hits        16619    16619           
  Misses      21327    21327           
  Partials      664      664           
Flag Coverage Δ
provider-base 16.09% <ø> (ø)
provider-sru 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bladernr
Copy link
Collaborator

Is the plan here to require the 24h test as part of a certification baseline?

@kissiel
Copy link
Contributor

kissiel commented Apr 10, 2024

Is the plan here to require the 24h test as part of a certification baseline?

In the RT testing that is being rolled out we intend to run this only for 60s.
Testing devices how they run the RT kernel is not [yet?] part of the certification AFAIK.

Copy link
Contributor

@kissiel kissiel left a comment

Choose a reason for hiding this comment

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

Fantastic stuff, TYVM!

@fernando79513
Copy link
Collaborator Author

Is the plan here to require the 24h test as part of a certification baseline?

Most surely we are not going to require runnug the test for 24 hours. We still have to set a definite value for this specific test. The 24 hour limit was set for the Intel IoTG devices on the iiotg provider. For the Real-time SRU, we will set a smaller duration in the future.

@fernando79513 fernando79513 merged commit f35524b into main Apr 10, 2024
16 checks passed
@fernando79513 fernando79513 deleted the adding-rt-test-plan branch April 10, 2024 15:14
LiaoU3 pushed a commit to LiaoU3/checkbox that referenced this pull request Apr 17, 2024
* Added short cyclictest in real time tests

* New sru for real time devices

* Increased cyclictest_rt duration to 1 minute

* Adding more information to the rt-performance jobs
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.

3 participants