Merge ~dirksu/pc-enablement/+git/oem-scripts:remove_oem_image_helper into ~oem-solutions-engineers/pc-enablement/+git/oem-scripts:master

Proposed by Dirk Su
Status: Merged
Approved by: Dirk Su
Approved revision: 3b2ecb537d700f14adf45a5ce06ab2a280b6dbf4
Merged at revision: 55eee6a24ee75047214e389c4015f941d1ed3006
Proposed branch: ~dirksu/pc-enablement/+git/oem-scripts:remove_oem_image_helper
Merge into: ~oem-solutions-engineers/pc-enablement/+git/oem-scripts:master
Diff against target: 33 lines (+1/-5)
1 file modified
recovery-from-iso.sh (+1/-5)
Reviewer Review Type Date Requested Status
Andy Chi Approve
Yao Wei Pending
OEM Solutions Group: Engineers Pending
Review via email: mp+437243@code.launchpad.net

Commit message

Remove pack-fish.openssh-fossa and oem-image-helper from preseed

Description of the change

openssh will install in oem-fix-misc-cnl-skip-oobe. oem-image-helper may caused old somerville iso which contain previous verison oem-image-helper fail to inject.

Tested with Tributo ADL with following scenario
1. jammy alloem-init to fossa-tentacool: good
2. jammy alloem-init to jellyfish-tentacool: good
3. jellyfish-tentacool to fossa-tentacool: good
4. fossa-tentacool to jellyfish-tentacool: good

To post a comment you must log in.
Revision history for this message
Andy Chi (andch) wrote :

Checked from the source of auto-prime-select (https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-oem-image-helper/tree/usr/sbin/auto-prime-select), which is not used in Jammy image now.
I think it's safe to remove oem-image-helper. Can we test the scenario which cert team encountered? Just want to make sure everything works on Jammy and Focal.

review: Needs Information
Revision history for this message
OEM Taipei Bot (oem-taipei-bot) wrote :

[autopkgtest]
autopkgtest-collect-credentials PASS
autopkgtest-oem-scripts-auto PASS
pkg-somerville-meta PASS
pkg-stella-meta PASS
pkg-sutton-meta PASS
bug-bind PASS
get-private-ppa PASS
jq-lp PASS
launchpad-api PASS
lp-bug PASS
pkg-list PASS
review-merge-proposal PASS
run-autopkgtest PASS
setup-apt-dir PASS
bootstrap-meta PASS
mir-bug PASS
oem-meta-packages PASS
git-url-insteadof-setting PASS
lp-dl-attm PASS
recovery-from-iso.sh PASS

https://oem-share.canonical.com/partners/lyoncore/share/artifacts/oem-scripts/oem-scripts-1.61-3b2ecb5-in-linux-container-jammy

Revision history for this message
Andy Chi (andch) wrote :

Thanks for testing :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/recovery-from-iso.sh b/recovery-from-iso.sh
2index 73ea260..7abce99 100755
3--- a/recovery-from-iso.sh
4+++ b/recovery-from-iso.sh
5@@ -158,11 +158,8 @@ ubiquity ubuntu-recovery/recovery_type string dev
6 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-install-sbhelper --depth 1
7 fi
8 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-skip-storage-selecting --depth 1
9- $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/pack-fish.openssh-fossa --depth 1
10 fi
11
12- # install common tool, so that we can use oem-install to create local repository and install packages.
13- $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-oem-image-helper --depth 1 -b oem-fix-misc-cnl-oem-image-helper_fish
14 # install packages related to skip oobe
15 skip_oobe_branch="master"
16 if [ -n "$ubuntu_release" ]; then
17@@ -231,7 +228,6 @@ push_preseed() {
18 $SCP -r "$temp_folder/preseed" "$user_on_target"@"$target_ip":~/push_preseed || $SSH "$user_on_target"@"$target_ip" sudo rm -f push_preseed/SUCCSS_push_preseed
19 else
20 folders=(
21- "pack-fish.openssh-fossa"
22 "oem-fix-misc-cnl-skip-oobe"
23 "oem-fix-misc-cnl-skip-storage-selecting"
24 )
25@@ -245,7 +241,7 @@ push_preseed() {
26 done
27 fi
28
29- for folder in misc_for_automation oem-fix-misc-cnl-oem-image-helper oem-fix-misc-cnl-skip-oobe; do
30+ for folder in misc_for_automation oem-fix-misc-cnl-skip-oobe; do
31 tar -C "$temp_folder"/$folder -zcvf "$temp_folder"/$folder.tar.gz .
32 $SCP "$temp_folder/$folder".tar.gz "$user_on_target"@"$target_ip":~
33 $SSH "$user_on_target"@"$target_ip" tar -C push_preseed -zxvf $folder.tar.gz || $SSH "$user_on_target"@"$target_ip" sudo rm -f push_preseed/SUCCSS_push_preseed

Subscribers

People subscribed via source and target branches