New issue
Advanced search Search tips

Issue 882032 link

Starred by 3 users
Status: Verified
Owner:
smbar...@chromium.org
Closed: Sep 9
Cc:
rohi...@chromium.org
avkodipelli@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

crostini: use M70 container/packages with run_container.sh

Project Member Reported by smbar...@chromium.org, Sep 7 
Since tremplin work on Chrome didn't make the cut for M70, we need to add support to run_container.sh to pull in the correct container for M70. This should also set up the correct apt repo for M70.
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 8

Labels: merge-merged-release-R70-11021.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/containers/cros-container-guest-tools/+/bff9a34f41e66e5d56c3a470eb3852b94d711b0a

commit bff9a34f41e66e5d56c3a470eb3852b94d711b0a
Author: Stephen Barber <smbarber@chromium.org>
Date: Sat Sep 08 00:07:07 2018

apt: hardcode apt config to M70

BUG= chromium:882032 
TEST=manual

Change-Id: Ib847db2d723b7c3a26b000229e61df1de79638ed
Reviewed-on: https://chromium-review.googlesource.com/1214097
Reviewed-by: Stephen Barber <smbarber@chromium.org>
Commit-Queue: Stephen Barber <smbarber@chromium.org>
Tested-by: Stephen Barber <smbarber@chromium.org>

[modify] https://crrev.com/bff9a34f41e66e5d56c3a470eb3852b94d711b0a/cros-apt-config/cros.list
[modify] https://crrev.com/bff9a34f41e66e5d56c3a470eb3852b94d711b0a/cros-apt-config/BUILD

Comment 2 by smbar...@chromium.org, Sep 8

Labels: -merge-merged-release-R70-11021.B Merge-Request-70
Please ignore the bugdroid comment; that goes through separate crostini CI.

For CrOS, requesting merge for https://crrev.com/c/1214430
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 8 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/containers/cros-container-guest-tools/+/55106ca75ba2dc18c51d4ad6d620882b312f5c8e

commit 55106ca75ba2dc18c51d4ad6d620882b312f5c8e
Author: Stephen Barber <smbarber@chromium.org>
Date: Sat Sep 08 13:49:59 2018

apt: hardcode apt config to M71

BUG= chromium:882032 
TEST=manual

Change-Id: Ie661c8d07f5ca0a613372aa2e870614ac09e7a5e
Reviewed-on: https://chromium-review.googlesource.com/1214098
Commit-Ready: Stephen Barber <smbarber@chromium.org>
Tested-by: Stephen Barber <smbarber@chromium.org>
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Stephen Barber <smbarber@chromium.org>

[modify] https://crrev.com/55106ca75ba2dc18c51d4ad6d620882b312f5c8e/cros-apt-config/cros.list
[modify] https://crrev.com/55106ca75ba2dc18c51d4ad6d620882b312f5c8e/cros-apt-config/BUILD
Project Member

Comment 4 by sheriffbot@chromium.org, Sep 9

Labels: -Merge-Request-70 Hotlist-Merge-Approved Merge-Approved-70
Your change meets the bar and is auto-approved for M70. Please go ahead and merge the CL to branch 3538 manually. Please contact milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), geohsu@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 9

Labels: merge-merged-release-R70-11021.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/95e09f145768779496d32f05967f33da4f44883f

commit 95e09f145768779496d32f05967f33da4f44883f
Author: Stephen Barber <smbarber@chromium.org>
Date: Sun Sep 09 03:12:21 2018

termina: lxd: use container for M70

BUG= chromium:882032 
TEST=manual

Change-Id: Id02ae97c8a2db88f73328b4542706b49d7188cb1
Reviewed-on: https://chromium-review.googlesource.com/1214430
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>
Commit-Queue: Stephen Barber <smbarber@chromium.org>
Tested-by: Stephen Barber <smbarber@chromium.org>

[rename] https://crrev.com/95e09f145768779496d32f05967f33da4f44883f/project-termina/chromeos-base/termina-lxd-scripts/termina-lxd-scripts-0.0.1-r27.ebuild
[modify] https://crrev.com/95e09f145768779496d32f05967f33da4f44883f/project-termina/chromeos-base/termina-lxd-scripts/files/run_container.sh

Comment 6 by smbar...@chromium.org, Sep 9

Labels: -Merge-Approved-70 Merge-Merged
Status: Fixed (was: Started)

Comment 7 by avkodipelli@chromium.org, Oct 3

Status: Verified (was: Fixed)
Closing the bug as it is landed long back and no regression observed.

Comment 8 by avkodipelli@chromium.org, Oct 3

Cc: rohi...@chromium.org avkodipelli@chromium.org
►

Sign in to add a comment