PC enablement tools

Merge ~alextu/pc-enablement/+git/oem-scripts:fix-ubr-debsum into ~oem-solutions-engineers/pc-enablement/+git/oem-scripts:master

  1. Git
  2. lp:~alextu/pc-enablement/+git/oem-scripts
  3. fix-ubr-debsum
  4. Merge into master
Proposed by Alex Tu
Status: Merged
Merged at revision: c72e719bf340a5a0412d61374fd325168d35360f
Proposed branch: ~alextu/pc-enablement/+git/oem-scripts:fix-ubr-debsum
Merge into: ~oem-solutions-engineers/pc-enablement/+git/oem-scripts:master
Diff against target: 74 lines (+7/-27)
1 file modified
recovery-from-iso.sh (+7/-27)
Reviewer Review Type Date Requested Status
Alex Tu (community) Approve
jeremyszu (community) Needs Information
Shengyao Xue Pending
Andy Chi Pending
Yao Wei Pending
Review via email: mp+413433@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alex Tu (alextu) wrote :

tested stella image on http://10.102.135.50:8080/job/sanity-3-testflinger-dummy-98000-40003-staging/17/

Revision history for this message
Alex Tu (alextu) wrote :

These 3 MPs are needed to fix checkbox debsum error on ubuntu recovery images.
https://code.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-oem-image-helper/+merge/413432
https://code.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-misc-for-automation/+merge/413430
https://code.launchpad.net/~alextu/pc-enablement/+git/oem-scripts/+merge/413433

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
bootstrap-meta PASS
bug-bind PASS
get-private-ppa PASS
jq-lp PASS
launchpad-api PASS
lp-bug PASS
oem-meta-packages PASS
pkg-list PASS
review-merge-proposal PASS
run-autopkgtest PASS
setup-apt-dir PASS
mir-bug SKIP exit status 77 and marked as skippable
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.26-73f2963-in-linux-container-focal

Revision history for this message
jeremyszu (os369510) wrote :

Although I disagree this patch makes the "MaaS deployment flow" and "injection flow" be unsync, I agree we have a package to maintain the skip-oobe related configurations.

Also that part is ported from somerville and the alextu is the author, it's no reason to reject if alextu did a test already.

In the other hand, the debsum false-alarm is a noise to me.

I think it's ok to merge it and see whether having the problem.

Only one question inline below.

review: Needs Information
Revision history for this message
Alex Tu (alextu) wrote :

Because the MaaS deploy way is mainly maintained by the certification team.
So, it would need the certification team to review the needed packages for MaaS way.

short term: sync the packages to skip oob for local sanity and auto sanity from recovery injection.

long term: sync with the way by MaaS deploy.

And also replied inline for the judgment of 1st login.

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
bootstrap-meta PASS
bug-bind PASS
get-private-ppa PASS
jq-lp PASS
launchpad-api PASS
lp-bug PASS
oem-meta-packages PASS
pkg-list PASS
review-merge-proposal PASS
run-autopkgtest PASS
setup-apt-dir PASS
mir-bug SKIP exit status 77 and marked as skippable
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.26-a75ce41-in-linux-container-focal

Revision history for this message
Alex Tu (alextu) wrote :

`systemctl is-active ubiquity` can be used to check the 1st login well:

verified on this testing:
http://10.102.135.50:8080/job/sanity-3-testflinger-dummy-98000-40004-staging-testing/3/consoleFull

Self-approved:
 - changes beside a75ce41 has been reviewed by Jeremy
 - a75ce41 is discussed with Jeremy and verified passed.
 - So, I merge it directly to save time.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/recovery-from-iso.sh b/recovery-from-iso.sh
index 2d1aa7d..843d9cc 100755
--- a/recovery-from-iso.sh
+++ b/recovery-from-iso.sh
@@ -86,38 +86,18 @@ ubiquity ubuntu-recovery/recovery_type string dev
86 sed -i 's%ubiquity/reboot boolean false%ubiquity/reboot boolean true%' ./project.cfg86 sed -i 's%ubiquity/reboot boolean false%ubiquity/reboot boolean true%' ./project.cfg
87 sed -i 's%ubiquity/poweroff boolean true%ubiquity/poweroff boolean false%' ./project.cfg87 sed -i 's%ubiquity/poweroff boolean true%ubiquity/poweroff boolean false%' ./project.cfg
88 mv project.cfg $temp_folder/preseed88 mv project.cfg $temp_folder/preseed
89 # replace $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/pack-fish.openssh-fossa --depth 1
90 # replace $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-skip-oobe --depth 1
91 # copy from maas. TODO: make it to use from same source.
92 mkdir -p $temp_folder/debs/main
93 pushd $temp_folder/debs/main
94 rm -rf maas_deps
95 git clone --depth 1 -b maas-focal lp:~lyoncore-team/lyoncore/+git/somerville-maas-override maas_deps
96 cd maas_deps
97 git rev-parse --short HEAD
98 rm -f maas-pkgs/oem-fix-misc-cnl-maas-helper*
99 cd ..
100 cp -r maas_deps/maas-pkgs/*.deb .
101 find .
102 popd
103 # TODO: share this with MaaS to use a same source. (e.g. debian package)
104 echo "#!/bin/bash
105. /usr/share/volatile/common.sh
106set -x
107dpkg -i /cdrom/debs/main/*.deb
108apt-get install -fy
109" | tee 32-install-custom-pkgs.sh
110 mkdir -p $temp_folder/scripts/chroot-scripts/os-post
111 mv 32-install-custom-pkgs.sh $temp_folder/scripts/chroot-scripts/os-post
112 else89 else
113 # get checkbox pkgs and prepare-checkbox90 # get checkbox pkgs and prepare-checkbox
114 # get pkgs to skip OOBE91 # get pkgs to skip OOBE
115 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-no-secureboot --depth 192 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-no-secureboot --depth 1
116 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-skip-storage-selecting --depth 193 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-skip-storage-selecting --depth 1
117 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/pack-fish.openssh-fossa --depth 194 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/pack-fish.openssh-fossa --depth 1
118 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-skip-oobe --depth 1
119 fi95 fi
12096
97 # install common tool, so that we can use oem-install to create local repository and install packages.
98 $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
99 # install packages related to skip oobe
100 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-skip-oobe --depth 1
121 # get pkgs for ssh key and skip disk checking.101 # get pkgs for ssh key and skip disk checking.
122 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-misc-for-automation --depth 1 misc_for_automation102 $GIT clone https://git.launchpad.net/~oem-solutions-engineers/pc-enablement/+git/oem-fix-misc-cnl-misc-for-automation --depth 1 misc_for_automation
123103
@@ -186,7 +166,7 @@ push_preseed() {
186 $SSH "$user_on_target"@"$target_ip" sudo rm -f /cdrom/SUCCSS_push_preseed166 $SSH "$user_on_target"@"$target_ip" sudo rm -f /cdrom/SUCCSS_push_preseed
187167
188 if [ "${ubr}" == "yes" ]; then168 if [ "${ubr}" == "yes" ]; then
189 for folder in debs preseed scripts; do169 for folder in preseed; do
190 $SCP -r "$temp_folder/$folder" "$user_on_target"@"$target_ip":~/push_preseed || $SSH "$user_on_target"@"$target_ip" sudo rm -f push_preseed/SUCCSS_push_preseed170 $SCP -r "$temp_folder/$folder" "$user_on_target"@"$target_ip":~/push_preseed || $SSH "$user_on_target"@"$target_ip" sudo rm -f push_preseed/SUCCSS_push_preseed
191 done171 done
192 else172 else
@@ -197,7 +177,7 @@ push_preseed() {
197 done177 done
198 fi178 fi
199179
200 for folder in misc_for_automation; do180 for folder in misc_for_automation oem-fix-misc-cnl-oem-image-helper oem-fix-misc-cnl-skip-oobe; do
201 tar -C "$temp_folder"/$folder -zcvf "$temp_folder"/$folder.tar.gz .181 tar -C "$temp_folder"/$folder -zcvf "$temp_folder"/$folder.tar.gz .
202 $SCP "$temp_folder/$folder".tar.gz "$user_on_target"@"$target_ip":~182 $SCP "$temp_folder/$folder".tar.gz "$user_on_target"@"$target_ip":~
203 $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_preseed183 $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
@@ -271,7 +251,7 @@ prepare() {
271251
272poll_recovery_status() {252poll_recovery_status() {
273 while(:); do253 while(:); do
274 if $SSH "$user_on_target"@"$target_ip" ubuntu-report show; then254 if [ "$($SSH "$user_on_target"@"$target_ip" systemctl is-active ubiquity)" = "inactive" ] ; then
275 break255 break
276 fi256 fi
277 sleep 180257 sleep 180

Subscribers

People subscribed via source and target branches