-
Notifications
You must be signed in to change notification settings - Fork 0
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
Validation effort for PC Engines v24.05.00.01 #844
Comments
apu2 v4.19.01 retest results were published, there seem to be at least two problems:
|
apu4 retest and and iter11 test results where added, identified issues:
It doesn't make sense to focus too much on the v4.19.0.1 retest when v4.19.0.1 cleanup provides such good results, so whatever infrastructure-related failures happened on apu4 during the retest, we leave it as is. |
Test results for iter11 on apu2 were added to the spreadsheet. apu2 has much fewer issues, it is affected by following failures:
|
apu2 v24.05.00.01 validatin:
|
apu4 v24.05.00.01 validation:
Booting from Hard Disk...
Booting from Hard Disk...
Booting from Hard Disk... |
SBO1.6 test most likely need fix because of the versioning scheme change, test fails. |
With following patch SBO1.6 pass: diff --git a/keywords.robot b/keywords.robot
index 0c271c1ba166..d0fbdf5a916b 100644
--- a/keywords.robot
+++ b/keywords.robot
@@ -575,17 +575,18 @@ Disable UART option
Return From Keyword If not ${sortbootorder_support}
${output}= Enter sortbootorder
${version}= Get sortbootorder version ${output}
+ ${maj_ver}= Set Variable ${version.split(".")[0]}
${patch_ver}= Set Variable ${version.split(".")[2]}
- ${lines}= IF ${patch_ver} > ${20}
+ ${lines}= IF ${patch_ver} > ${20} or ${maj_ver} >= ${24}
... Get Lines Containing String ${output} Toggle ${option_str} / GPIO
... ELSE Get Lines Containing String ${output} ${option_str}
- ${option_str}= Set Variable If ${patch_ver} > ${20} and '${option_str}' == 'UART C' GPIO[0..7] ${option_str}
- ${option_str}= Set Variable If ${patch_ver} > ${20} and '${option_str}' == 'UART D' GPIO[10..17] ${option_str}
- ${res}= IF ${patch_ver} > ${20}
+ ${option_str}= Set Variable If (${patch_ver} > ${20} or ${maj_ver} >= ${24}) and '${option_str}' == 'UART C' GPIO[0..7] ${option_str}
+ ${option_str}= Set Variable If (${patch_ver} > ${20} or ${maj_ver} >= ${24}) and '${option_str}' == 'UART D' GPIO[10..17] ${option_str}
+ ${res}= IF ${patch_ver} > ${20} or ${maj_ver} >= ${24}
... Evaluate "${option_str} - Currently Enabled" in """${lines}"""
... ELSE Evaluate "${option_str} - Currently Disabled" in """${lines}"""
IF ${res} Telnet.Write Bare x
- IF ${patch_ver} > ${20}
+ IF ${patch_ver} > ${20} or ${maj_ver} >= ${24}
... Run Keyword If not ${res} Verify option change ${lines} ${option_str} Enabled
... ELSE IF not ${res} Verify option change ${lines} ${option_str} Disabled |
With the above patch also SBO1.8 pass on apu4 |
Regarding NFS issues most of those happen when there is concurrent acces e.g. two tests mounting NFS running in parallel. |
Quite a lot of issues were fixed by switching to DTS instead of using Debian over iPXE+NFS. Outstanding issues for apu2/3/4/6 are:
Another issue that was not addressed is that on some platforms (e.g., apu6), I cannot pass 50x Debian boot (DEB1.4) because NFS is not responding even if I try to run those tests only on one platform. It's very likely some weird issue with infrastructure. |
WOL was fixed by using direct MAC addresses of interfaces instead of retrieving them automatically. Apparently, the function for automatic retrieval had a bug. |
WP was also fixed; it happened that power off was done too fast, and sortbootorder didn't have a chance to write settings to flash. Extending the time between changing the option and power fixed the issue. |
In this issue, I will report the progress of validation. The plan is as follows:
The text was updated successfully, but these errors were encountered: