New issue
Advanced search Search tips

Issue 669641 link

Starred by 3 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature

Blocked on:
issue 621008
issue 669791

Blocking:
issue 347322



Sign in to add a comment

autotest: [re]add xfstests

Project Member Reported by gwendal@chromium.org, Nov 29 2016

Issue description

meta bug to re-add xfstests to chromeos.

autotest depends on Perl that need to be pull in the test image.
 
Blockedon: 621008
Blockedon: 669791
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 28 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/manifest-internal/+/4b69d9ab806f5447b1de11cfe680d462f2cb94a5

commit 4b69d9ab806f5447b1de11cfe680d462f2cb94a5
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Fri Dec 16 23:34:58 2016

Project Member

Comment 4 by bugdroid1@chromium.org, Dec 29 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/manifest/+/ae72d74911effa3caf84c8d8c40c8a96ea7cc4c8

commit ae72d74911effa3caf84c8d8c40c8a96ea7cc4c8
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Fri Dec 16 21:37:01 2016

manifest: Add xfstests repo

Add xfstests at third_party/xfstests.

BUG=chromium:669641
CQ-DEPEND=CL:*312659
TEST=Applying changes in .repo, check repo sync upload xfstests in
third_party/xfstests.

Change-Id: I77961d16bf85d7cc85e8ebc9cb6b68ca5a599196
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/421114
Reviewed-by: Dave Sansome <dsansome@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/ae72d74911effa3caf84c8d8c40c8a96ea7cc4c8/full.xml

Comment 5 by snanda@chromium.org, Jan 20 2017

Cc: puthik@chromium.org
Opal, can you please help review Gwendal's CLs?

Comment 6 by snanda@chromium.org, Jan 20 2017

Cc: snanda@chromium.org

Comment 7 by snanda@chromium.org, Jan 26 2017

vapier recommended sending the fix upstream first.
Project Member

Comment 8 by bugdroid1@chromium.org, May 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/503bef956177dc40b36fab1f81d1b7e1fab22434

commit 503bef956177dc40b36fab1f81d1b7e1fab22434
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Mon May 01 23:36:22 2017

xfstests: Add ebuild to build xfstests.

Create a new package, app-benchmarks/xfstests.

BUG=chromium:669641
TEST=Compile, install on samus.

Change-Id: I3291adf890c779bd8b00ae6feb28402ee0e7a891
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/399901
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[add] https://crrev.com/503bef956177dc40b36fab1f81d1b7e1fab22434/app-benchmarks/xfstests/xfstests-9999.ebuild
[add] https://crrev.com/503bef956177dc40b36fab1f81d1b7e1fab22434/app-benchmarks/xfstests/metadata.xml

Project Member

Comment 9 by bugdroid1@chromium.org, Jul 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/10f0983f11ed31bbf7367cb25bed3cf542b8c522

commit 10f0983f11ed31bbf7367cb25bed3cf542b8c522
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Thu Jul 13 00:55:53 2017

xfstests: Prevent automatic revbump

thrid_party/xfstests is automatically updated from git.kernel.org.
Thanks to 9999 ebuild and chrome-bot, a new ebuild version is created
each time a new CL is added to the remote repo.

This is a security risk: in case git.kernel.org is compromised,
outside CLs are autommatically rolled into CrOS.

Set CROS_WORKON_BLACKLIST, manual change to rXX ebuild are needed.

BUG=chromium:669641
TEST=ebuild still works.

Change-Id: Idb15f631cd1422c6cd0d7f439f56b715c45ca6e5
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/563600

[modify] https://crrev.com/10f0983f11ed31bbf7367cb25bed3cf542b8c522/app-benchmarks/xfstests/xfstests-9999.ebuild
[modify] https://crrev.com/10f0983f11ed31bbf7367cb25bed3cf542b8c522/app-benchmarks/xfstests/xfstests-0.0.1-r11.ebuild

Project Member

Comment 10 by bugdroid1@chromium.org, Jul 26 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/1d85fcbede76c2d050c58f5852235dfefc1d2d55

commit 1d85fcbede76c2d050c58f5852235dfefc1d2d55
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Wed Jul 26 20:25:42 2017

autotest: Update virtual partition object

Remove usage of kpartx, not included anymore.
Remove partition table, use the whole loop device as a single partition

BUG=chromium:669641
TEST=Check xfstests run.

Change-Id: I3a2fb1870e5b7ef263e8e24674dc2c4474d7ac12
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/416221
Reviewed-by: Richard Barnette <jrbarnette@google.com>

[modify] https://crrev.com/1d85fcbede76c2d050c58f5852235dfefc1d2d55/client/bin/partition.py

Project Member

Comment 11 by bugdroid1@chromium.org, Jul 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/5a7f73b3c0b823eac8bbcd8da5ecd11bdff47ad0

commit 5a7f73b3c0b823eac8bbcd8da5ecd11bdff47ad0
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Thu Jul 27 22:43:58 2017

autotest-tests: Add xfstests to test image.

Add xfstests.
Weight 11MB, not including perl and other xfs tools.

BUG=chromium:669641
CQ-DEPEND=CL:416222
TEST=Check xfstest, xfsprogs is installed in the test image.
Test on Samus, Kevin.
Builder: chromiumos.tryserver/builders/release/builds/8167

Change-Id: I24b343ba7daf3aaf712fa4279dba92c73f3e5f87
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/416507

[modify] https://crrev.com/5a7f73b3c0b823eac8bbcd8da5ecd11bdff47ad0/virtual/target-chromium-os-test/target-chromium-os-test-1.ebuild
[rename] https://crrev.com/5a7f73b3c0b823eac8bbcd8da5ecd11bdff47ad0/virtual/target-chromium-os-test/target-chromium-os-test-1-r50.ebuild
[modify] https://crrev.com/5a7f73b3c0b823eac8bbcd8da5ecd11bdff47ad0/chromeos-base/autotest-tests/autotest-tests-9999.ebuild

Project Member

Comment 12 by bugdroid1@chromium.org, Jul 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/7a804944a558998dd1292201987583902d3d209b

commit 7a804944a558998dd1292201987583902d3d209b
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Thu Jul 27 22:43:58 2017

xfstests: Update xfstests

Add dependency to xfstest package, remove internal dependency.
Somewhat exclude experimental tests.
Update launcher.

BUG=chromium:669641
TEST=On 32GB device (does not work with 16G) check the whole suite runs.
Some tests fail, some are skipped due to missing e2fuzz, but no crash.

Change-Id: If9f794910539f62b47beb19afd24fe0879a05d9d
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/416222
Reviewed-by: Richard Barnette <jrbarnette@chromium.org>

[modify] https://crrev.com/7a804944a558998dd1292201987583902d3d209b/client/tests/xfstests/xfstests.py
[delete] https://crrev.com/e8bc959e03680ed268d02f6979bf154691722d44/client/tests/xfstests/xfstests.tar.bz2
[modify] https://crrev.com/7a804944a558998dd1292201987583902d3d209b/client/tests/xfstests/control

Sign in to add a comment