New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 813782 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Sep 26
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Feature



Sign in to add a comment

vpython dependancie not met on mipsel platform

Reported by milko.le...@mips.com, Feb 20 2018

Issue description

UserAgent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0

Steps to reproduce the problem:
1. Build net_unittests for mipsel
2. Run net_unittets on mipsel platform

What is the expected behavior?
Tests should be executed without issues related to starting test server.

What went wrong?
176 tests are failing with error message:

LaunchProcess: failed to execvp:
vpython
[22826:22826:0220/060239.970114:7942577765683:ERROR:local_test_server_posix.cc(168)] Could not read server_data_len

Did this work before? Yes r530315

Chrome version: r530316  Channel: n/a
OS Version: Debian Jessie
Flash Version: 

Issue started after change:
https://chromium-review.googlesource.com/c/chromium/src/+/874153

We are cross-compiling unit tests and running them on mipsel based platform. After this change there is dependency on vpython which is not being built for mipsel.

Similar issue with vpython was already reported for s390, ppc64 and ppc64le:

https://bugs.chromium.org/p/chromium/issues/detail?id=764087
https://bugs.chromium.org/p/chromium/issues/detail?id=773857

Same solution would be fine by us.
We need this for mipsel and mips64el platforms.

GOARCHs for mipsel and mips64el are: mipsle and mips64le respectively.
 
Labels: Needs-Milestone
Cc: vamshi.kommuri@chromium.org
Components: OS>Systems>Network
Labels: Triaged-ET TE-NeedsTriageHelp
This issue seems to be out of scope for triaging from ET end as this speaks about Building net_unittests. Hence adding label TE-NeedsTriageHelp and requesting someone from respective team to have a look into it.

Note: Adding component "OS>Systems>Network", Please feel free to change if this doesn't suite.

Thanks!
Cc: iannucci@chromium.org xunji...@chromium.org
Cc: amineer@chromium.org
Components: -OS>Systems>Network Internals>Network
Status: WontFix (was: Unconfirmed)
Hi, mipsel is no longer supported as of Issue 790692. 

Cc: -amineer@chromium.org

Comment 6 by pet...@mips.com, Apr 24 2018

> Hi, mipsel is no longer supported as of Issue 790692. 

While I do not have access to this issue, I believe this refers to MIPS Android.
Chromium is fully supported on MIPS Linux and it is tested on a daily basis (this is how this issue was discovered and reported).
Cc: jbudorick@chromium.org mmoss@chromium.org
Components: Infra>Client>Chrome>Release
Status: Unconfirmed (was: WontFix)
Ok. Thanks. 

+jbudorick@, mmoss@:

Can you help with updating vpython to support mipsel/mipsel64?

Comment 8 by mmoss@chromium.org, Apr 24 2018

Components: -Infra>Client>Chrome>Release Infra>SDK
Note, this shouldn't be "Release" component, which is for issues related to the Chrome official build and release pipeline, which this bug doesn't seem to have anything to do with. Probably an "SDK" issue, since I think vpython is something handled in depot_tools.

Comment 9 by eroman@chromium.org, Apr 24 2018

Components: -Internals>Network
Is there any progress on adding mipsel/mips64el support to vpython?
Any update on this?

Can I somehow help?
Project Member

Comment 12 by bugdroid1@chromium.org, Aug 6

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/b7e63bff94b824b486ad8f38ea7e41f763502867

commit b7e63bff94b824b486ad8f38ea7e41f763502867
Author: Gordana Cmiljanovic <gordana.cmiljanovic@mips.com>
Date: Mon Aug 06 19:21:41 2018

[cipd] Build all packages that are built for mips64le also for mipsle.

BUG= 813782 

Change-Id: I80e7e14bd649aaa3ff1aec8e80740dd684e31a34
Reviewed-on: https://chromium-review.googlesource.com/1163515
Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>

[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/README.md
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/vpython.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/luci-auth.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/cipd_client.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/build.py
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/logdog_butler.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/prpc.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/vpython-native.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/led.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/kitchen.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/recipes/recipes/infra_continuous.py
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/logdog_annotee.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/luci_machine_tokend.yaml
[modify] https://crrev.com/b7e63bff94b824b486ad8f38ea7e41f763502867/build/packages/git.yaml

Cc: vadimsh@chromium.org
I don't think we support building on this platform. Typically we build on linux-x64 and target mips... right?

The vpython stuff is only needed for the platform that we're building on.
Some tests require vpython, for example net_unittests:

https://chromium-review.googlesource.com/c/chromium/src/+/874153
Oof, right. I guess we should target mips64le for vpython wheels too.
Project Member

Comment 16 by bugdroid1@chromium.org, Aug 7

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/depot_tools/+/333617bc86fef3ec7b4612d35eba8a36f531a86a

commit 333617bc86fef3ec7b4612d35eba8a36f531a86a
Author: Vadim Shtayura <vadimsh@chromium.org>
Date: Tue Aug 07 20:46:38 2018

[cipd] Bump versions of all CIPD packages + add mips(64)le platform support.

New packages are built using SHA256 as instance ID (instead of SHA1).

R=iannucci@chromium.org
BUG=821194,  867819 ,  813782 

Change-Id: I61cf71386975725f7f63097eeb62f094ff50e396
Reviewed-on: https://chromium-review.googlesource.com/1165598
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>

[modify] https://crrev.com/333617bc86fef3ec7b4612d35eba8a36f531a86a/cipd_client_version
[modify] https://crrev.com/333617bc86fef3ec7b4612d35eba8a36f531a86a/PRESUBMIT.py
[modify] https://crrev.com/333617bc86fef3ec7b4612d35eba8a36f531a86a/cipd_manifest.txt

Project Member

Comment 17 by bugdroid1@chromium.org, Aug 8

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e431b5849d58bd38cb65a33d05af7691deafc6d1

commit e431b5849d58bd38cb65a33d05af7691deafc6d1
Author: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Date: Wed Aug 08 02:48:27 2018

Roll src/third_party/depot_tools 2ebf9fdade33..735f71d95bcb (2 commits)

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/2ebf9fdade33..735f71d95bcb


git log 2ebf9fdade33..735f71d95bcb --date=short --no-merges --format='%ad %ae %s'
2018-08-08 hinoka@google.com [recipe_modules] Copy over windows_sdk from infra.git
2018-08-07 vadimsh@chromium.org [cipd] Bump versions of all CIPD packages + add mips(64)le platform support.


Created with:
  gclient setdep -r src/third_party/depot_tools@735f71d95bcb

The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.



BUG=chromium:821194, chromium:867819 , chromium:813782 
TBR=agable@chromium.org

Change-Id: I59662e6c61ccb156e85b00733e6c1e4668c703b1
Reviewed-on: https://chromium-review.googlesource.com/1166343
Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#581448}
[modify] https://crrev.com/e431b5849d58bd38cb65a33d05af7691deafc6d1/DEPS

Something is still not ok.

'./cipd help' does not complete successfully on MIPS LE platform due to:

$ wget "https://chrome-infra-packages.appspot.com/client?platform=linux-mipsle&version=git_revision:9a931a5307c46b16b1c12e01e8239d4a73830b89" -U "depot_tools/dd5051fa529e0935fe279279d39cb00cbf856c14" -O "./tmp.6hxZSf0btz"
converted 'https://chrome-infra-packages.appspot.com/client?platform=linux-mipsle&version=git_revision:9a931a5307c46b16b1c12e01e8239d4a73830b89' (ANSI_X3.4-1968) -> 'https://chrome-infra-packages.appspot.com/client?platform=linux-mipsle&version=git_revision:9a931a5307c46b16b1c12e01e8239d4a73830b89' (UTF-8)
--2018-08-08 09:37:51--  https://chrome-infra-packages.appspot.com/client?platform=linux-mipsle&version=git_revision:9a931a5307c46b16b1c12e01e8239d4a73830b89
Resolving chrome-infra-packages.appspot.com (chrome-infra-packages.appspot.com)... 216.58.209.180, 2a00:1450:400d:803::2014
Connecting to chrome-infra-packages.appspot.com (chrome-infra-packages.appspot.com)|216.58.209.180|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://storage.googleapis.com/chrome-infra-packages/store/SHA256/e555d4ca0068609d43c3a9ed82a8d3d0905fee60559340e19404e1eb65bc9300?Expires=1533720833&GoogleAccessId=chrome-infra-packages%40appspot.gserviceaccount.com&Signature=kNOyuoeRuDXyTYloRi2Z9eCdXL%2B3af41din3LrrpkdSS3nrRMQZBjuS6CKwLKwGtNqpaeJm2s%2BxOk%2FgUtyzhIzGfAq8%2B9sYUoo%2FajhMQiPEz%2B6qaF1wuHxzCfCBQTTvJmLcbUHWd8JI3cOdsgebRda%2FFANY%2BeTxFuF5GL%2BSEMjgFsbtdHl8%2BN2nk3F0Kq%2FRdhuHBwjjuABg8Kr7vF6NXrbJJ5TGzSmzphGAOGP0Ogmf3m8w4bOC7kEgem0CZ29VlvnSv4m1jcVmrPKhhzXWuBixWAXGKvV7ByuRnXtgGFvSGsNPDKl%2Bb8R0av5x2IJY6IfG1SC1AbVVmHg0XJlFCsg%3D%3D&response-content-disposition=attachment%3B+filename%3D%22cipd%22 [following]
converted 'https://storage.googleapis.com/chrome-infra-packages/store/SHA256/e555d4ca0068609d43c3a9ed82a8d3d0905fee60559340e19404e1eb65bc9300?Expires=1533720833&GoogleAccessId=chrome-infra-packages%40appspot.gserviceaccount.com&Signature=kNOyuoeRuDXyTYloRi2Z9eCdXL%2B3af41din3LrrpkdSS3nrRMQZBjuS6CKwLKwGtNqpaeJm2s%2BxOk%2FgUtyzhIzGfAq8%2B9sYUoo%2FajhMQiPEz%2B6qaF1wuHxzCfCBQTTvJmLcbUHWd8JI3cOdsgebRda%2FFANY%2BeTxFuF5GL%2BSEMjgFsbtdHl8%2BN2nk3F0Kq%2FRdhuHBwjjuABg8Kr7vF6NXrbJJ5TGzSmzphGAOGP0Ogmf3m8w4bOC7kEgem0CZ29VlvnSv4m1jcVmrPKhhzXWuBixWAXGKvV7ByuRnXtgGFvSGsNPDKl%2Bb8R0av5x2IJY6IfG1SC1AbVVmHg0XJlFCsg%3D%3D&response-content-disposition=attachment%3B+filename%3D%22cipd%22' (ANSI_X3.4-1968) -> 'https://storage.googleapis.com/chrome-infra-packages/store/SHA256/e555d4ca0068609d43c3a9ed82a8d3d0905fee60559340e19404e1eb65bc9300?Expires=1533720833&GoogleAccessId=chrome-infra-packages@appspot.gserviceaccount.com&Signature=kNOyuoeRuDXyTYloRi2Z9eCdXL+3af41din3LrrpkdSS3nrRMQZBjuS6CKwLKwGtNqpaeJm2s+xOk/gUtyzhIzGfAq8+9sYUoo/ajhMQiPEz+6qaF1wuHxzCfCBQTTvJmLcbUHWd8JI3cOdsgebRda/FANY+eTxFuF5GL+SEMjgFsbtdHl8+N2nk3F0Kq/RdhuHBwjjuABg8Kr7vF6NXrbJJ5TGzSmzphGAOGP0Ogmf3m8w4bOC7kEgem0CZ29VlvnSv4m1jcVmrPKhhzXWuBixWAXGKvV7ByuRnXtgGFvSGsNPDKl+b8R0av5x2IJY6IfG1SC1AbVVmHg0XJlFCsg==&response-content-disposition=attachment;+filename="cipd"' (UTF-8)
--2018-08-08 09:37:51--  https://storage.googleapis.com/chrome-infra-packages/store/SHA256/e555d4ca0068609d43c3a9ed82a8d3d0905fee60559340e19404e1eb65bc9300?Expires=1533720833&GoogleAccessId=chrome-infra-packages@appspot.gserviceaccount.com&Signature=kNOyuoeRuDXyTYloRi2Z9eCdXL+3af41din3LrrpkdSS3nrRMQZBjuS6CKwLKwGtNqpaeJm2s+xOk/gUtyzhIzGfAq8+9sYUoo/ajhMQiPEz+6qaF1wuHxzCfCBQTTvJmLcbUHWd8JI3cOdsgebRda/FANY+eTxFuF5GL+SEMjgFsbtdHl8+N2nk3F0Kq/RdhuHBwjjuABg8Kr7vF6NXrbJJ5TGzSmzphGAOGP0Ogmf3m8w4bOC7kEgem0CZ29VlvnSv4m1jcVmrPKhhzXWuBixWAXGKvV7ByuRnXtgGFvSGsNPDKl+b8R0av5x2IJY6IfG1SC1AbVVmHg0XJlFCsg==&response-content-disposition=attachment;+filename=%22cipd%22
Resolving storage.googleapis.com (storage.googleapis.com)... 172.217.18.80, 2a00:1450:400d:806::2010
Connecting to storage.googleapis.com (storage.googleapis.com)|172.217.18.80|:443... connected.
HTTP request sent, awaiting response... 403 Forbidden
2018-08-08 09:37:51 ERROR 403: Forbidden.

Ok, so this was a local error on my platform.

I can confirm that files are downloaded and with

export VPYTHON_BYPASS="manually managed python not supported by chrome operations"

tests are being executed correctly.

Thank you!

This record can be closed as fixed.
Status: Fixed (was: Unconfirmed)

Sign in to add a comment