Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

V8 CI not working #1940

Closed
MylesBorins opened this issue Oct 1, 2019 · 9 comments
Closed

V8 CI not working #1940

MylesBorins opened this issue Oct 1, 2019 · 9 comments

Comments

@MylesBorins
Copy link
Contributor

ppcle-ubuntu1404 and centos7-ppcle have both failed the last three jobs with the error

10:31:23 + ln -s /usr/bin/ccache ppc64le-redhat-linux-g++ /home/iojs/build-tools/g++
10:31:23 ln: target ‘/home/iojs/build-tools/g++’ is not a directory
@sam-github
Copy link
Contributor

@MylesBorins a specific CI job name would help me fix this.

This must have been triggered by #1927

Some CI job is doing ln -s $CXX /some/path, without considering that CXX might be CXX="ccache g++"

@MylesBorins
Copy link
Contributor Author

seeing it in the following jobs

https://ci.nodejs.org/job/node-test-commit-v8-linux/2528/
https://ci.nodejs.org/job/node-test-commit-v8-linux/2529/
https://ci.nodejs.org/job/node-test-commit-v8-linux/2530/
https://ci.nodejs.org/job/node-test-commit-v8-linux/2531/

@sam-github
Copy link
Contributor

Ok, found the broken code, its in https://github.com/nodejs/node/blob/3473e58fee153a7fee143a33c512ee9cda08d20a/tools/make-v8.sh#L15-L23

I am looking at fixing it.

@MylesBorins
Copy link
Contributor Author

@sam-github odd that this was not broken the last time we ran this CI... any idea why this commit from July is just starting to break now?

@sam-github
Copy link
Contributor

yes, because #1940 (comment)

@MylesBorins
Copy link
Contributor Author

@sam-github OHHHHHH I missed that... thanks for digging in!

@sam-github
Copy link
Contributor

This is going slowly, I can't do v8 builds, even in the jenkins workspace. Apparently, too much of the necessary setup is known only to the the jekins jobs.

+ tools/node/fetch_deps.py .
Using depot tools in /home/iojs/build/workspace/node-test-commit-v8-linux/nodes/centos7-ppcle/v8test/v8test/deps/v8/_depot_tools                                             
Initializing temporary git repository in v8.
Fetching dependencies.
Syncing projects: 100% (29/29), done.

________ running 'vpython v8/third_party/depot_tools/update_depot_tools_toggle.py --disable' in '.'                                                                          
[E2019-10-01T20:16:22.715674Z 23649 0 annotate.go:241] goroutine 1:
[E2019-10-01T20:16:22.715734Z 23649 0 annotate.go:241] #0 vpython/pep425.go:258 - main.getPEP425CIPDTemplateForTag()                                                         
[E2019-10-01T20:16:22.715747Z 23649 0 annotate.go:241]   reason: failed to infer CIPD platform for tag [python:"cp27" abi:"cp27mu" platform:"linux_ppc64le" ]                
[E2019-10-01T20:16:22.715760Z 23649 0 annotate.go:241]
[E2019-10-01T20:16:22.715771Z 23649 0 annotate.go:241] #1 vpython/main.go:50 - main.glob..func1()                                                                            
[E2019-10-01T20:16:22.715784Z 23649 0 annotate.go:241] #2 go.chromium.org/luci/vpython/cipd/cipd.go:278 - cipd.(*PackageLoader).expanderForTags()                            
[E2019-10-01T20:16:22.715797Z 23649 0 annotate.go:241]   reason: failed to get CIPD template arguments                                                                       
[E2019-10-01T20:16:22.715808Z 23649 0 annotate.go:241]
[E2019-10-01T20:16:22.715819Z 23649 0 annotate.go:241] #3 go.chromium.org/luci/vpython/cipd/cipd.go:68 - cipd.(*PackageLoader).Resolve()                                     
[E2019-10-01T20:16:22.715831Z 23649 0 annotate.go:241] #4 go.chromium.org/luci/vpython/venv/config.go:182 - venv.(*Config).makeEnv()                                         
[E2019-10-01T20:16:22.715844Z 23649 0 annotate.go:241]   reason: failed to resolve packages                                                                                  
[E2019-10-01T20:16:22.715855Z 23649 0 annotate.go:241]
[E2019-10-01T20:16:22.715866Z 23649 0 annotate.go:241] #5 go.chromium.org/luci/vpython/venv/venv.go:163 - venv.With()                                                        
[E2019-10-01T20:16:22.715878Z 23649 0 annotate.go:241] #6 go.chromium.org/luci/vpython/run.go:62 - vpython.Run()                                                             
[E2019-10-01T20:16:22.715889Z 23649 0 annotate.go:241] #7 go.chromium.org/luci/vpython/application/application.go:312 - application.(*application).mainImpl()                
[E2019-10-01T20:16:22.715901Z 23649 0 annotate.go:241] #8 go.chromium.org/luci/vpython/application/application.go:397 - application.(*Config).Main.func1()                   
[E2019-10-01T20:16:22.715912Z 23649 0 annotate.go:241] #9 go.chromium.org/luci/vpython/application/support.go:46 - application.run()                                         
[E2019-10-01T20:16:22.715924Z 23649 0 annotate.go:241] #10 go.chromium.org/luci/vpython/application/application.go:396 - application.(*Config).Main()                        
[E2019-10-01T20:16:22.715935Z 23649 0 annotate.go:241] #11 vpython/main.go:105 - main.mainImpl()                                                                             
[E2019-10-01T20:16:22.715947Z 23649 0 annotate.go:241] #12 vpython/main.go:111 - main.main()                                                                                 
[E2019-10-01T20:16:22.715958Z 23649 0 annotate.go:241] #13 runtime/proc.go:200 - runtime.main()                                                                              
[E2019-10-01T20:16:22.715970Z 23649 0 annotate.go:241] #14 runtime/asm_ppc64x.s:857 - runtime.goexit()                                                                       
Error: Command 'vpython v8/third_party/depot_tools/update_depot_tools_toggle.py --disable' returned non-zero exit status 1 in .                                              
Uninitializing temporary git repository
>> Cleaning up /home/iojs/build/workspace/node-test-commit-v8-linux/nodes/centos7-ppcle/v8test/v8test/deps/v8/.git                                                           
Traceback (most recent call last):
  File "tools/node/fetch_deps.py", line 101, in <module>
    FetchDeps(sys.argv[1])
  File "tools/node/fetch_deps.py", line 85, in FetchDeps
    env=env)
  File "/usr/lib64/python2.7/subprocess.py", line 542, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/bin/python', '/home/iojs/build/workspace/node-test-commit-v8-linux/nodes/centos7-ppcle/v8test/v8test/deps/v8/_depot_tools/gclient.py', 'sync', '--spec', "solutions = [{'url': 'https://chromium.googlesource.com/v8/v8.git', 'managed': False, 'name': 'v8', 'deps_file': 'DEPS', 'custom_deps': {'v8/third_party/qemu-linux-x64': None, 'v8/third_party/catapult': None, 'v8/test/benchmarks/data': None, 'v8/test/mozilla/data': None, 'v8/third_party/colorama/src': None, 'v8/testing/gmock': None, 'v8/test/test262/data': None, 'v8/third_party/android_sdk': None, 'v8/test/test262/harness': None, 'v8/tools/swarming_client': None, 'v8/third_party/fuchsia-sdk': None, 'v8/third_party/googletest/src': None, 'v8/base/trace_event/common': None, 'v8/tools/luci-go': None, 'v8/third_party/instrumented_libraries': None, 'v8/third_party/android_ndk': None}}]"]' returned non-zero exit status 2
Makefile:292: recipe for target 'v8' failed
make: *** [v8] Error 1

I'll revert the change that triggered this for now.

sam-github added a commit that referenced this issue Oct 1, 2019
@sam-github
Revert "jenkins: use ccache with devtoolset on ppc64le"
f54e6ce 
This reverts commit 70df2a4.

It breaks node-test-commit-v8-linux:
- #1940
sam-github added a commit that referenced this issue Oct 1, 2019
@sam-github
Revert "jenkins: use ccache with gcc/g++ on ppc64le"
7b9805a 
This reverts commit c105ccc.

It breaks node-test-commit-v8-linux:
- #1940
@sam-github sam-github mentioned this issue Oct 1, 2019
Merged
@sam-github
Copy link
Contributor

Fixed by f54e6ce and 7b9805a

ci: https://ci.nodejs.org/job/node-test-commit-v8-linux/2533/

@sam-github
Copy link
Contributor

green: https://ci.nodejs.org/job/node-test-commit-v8-linux/2533/

This was referenced Feb 4, 2020
Merged
Merged
richardlau added a commit to richardlau/node-1 that referenced this issue Feb 7, 2020
@richardlau
tools: unify make-v8.sh for ppc64le and s390x
d213f21 
Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: nodejs#31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Sam Roberts <[email protected]>
codebytere pushed a commit to nodejs/node that referenced this issue Feb 17, 2020
@richardlau @codebytere
tools: unify make-v8.sh for ppc64le and s390x
328b8a6 
Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: #31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Sam Roberts <[email protected]>
BethGriggs pushed a commit to nodejs/node that referenced this issue Feb 26, 2020
@richardlau @BethGriggs
tools: unify make-v8.sh for ppc64le and s390x
3a1566a 
Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: #31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Sam Roberts <[email protected]>
BethGriggs pushed a commit to nodejs/node that referenced this issue Mar 2, 2020
@richardlau @BethGriggs
tools: unify make-v8.sh for ppc64le and s390x
f235eea 
Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: #31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Sam Roberts <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sam-github @MylesBorins