-
Notifications
You must be signed in to change notification settings - Fork 50
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
Make all rtcwake cmd can select the RTC_DEVICE_FILE from config file (New) #1395
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1395 +/- ##
==========================================
+ Coverage 45.12% 45.13% +0.01%
==========================================
Files 366 366
Lines 39058 39063 +5
Branches 6607 6608 +1
==========================================
+ Hits 17626 17633 +7
+ Misses 20758 20756 -2
Partials 674 674
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see you made changes to some config_var in checkbox-snap, but why the followings are not changed:
- common_series_classic
- series_uc16
- series_uc18
- series_uc20
- series_uc22
Also there's a typo to be fixed, see my inline comment
I am wondering if there are multiple RTC like rtc1, how should we handle it? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
WARNING: This modifies com.canonical.certification::sru-server
Modify all jobs will call the rtcwake cmd and didn't support for user to select which rtc device they want to use.
In rtcwake default will use the
rtc0
, and this change will also make if theRTC_DEVICE_FILE
didn't define in config will also use thertc0
.Resolved issues
Documentation
OEMQA-4726 Modify jobs' command will call the rtcwake can support user to select which RTC devcie
Tests