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.

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

perf pipeline: Switch to node 14.x #74017

Merged
merged 7 commits into from
Aug 17, 2022
Merged

perf pipeline: Switch to node 14.x #74017

merged 7 commits into from
Aug 17, 2022

Conversation

radical
Copy link
Member

@radical radical commented Aug 16, 2022

  • Switch to node 14.x instead of 18.x, because the latter doesn't seem
    to be compatible with ubuntu 18.x:
    node: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.28' not found (required by node)`

  • remove the existing nodejs package before installing a new
    one, because these machines are shared, and would get affected by
    installations performed by other runs.

  • Fix the case where pre-requisites would fail to install, but then helix would exit with exit code 0, and it would "look" as if everything went fine. This was happening because we set _commandExitCode=1, but helix sets this itself based on $?. So, to fix this we exit with (exit 1), and helix should be able to pick that up, thus causing the whole job to fail, as expected

  • Install node, and jsvu only for the wasm jobs, as they are not needed by others

- Switch to node 14.x instead of 18.x, because the latter doesn't seem
to be compatible with ubuntu 18.x:
`node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by node)`

- Also, remove the existing `nodejs` package before installing a new
one, because these machines are shared, and would get affected by
installations performed by other runs.
@radical radical added arch-wasm WebAssembly architecture area-Infrastructure-mono labels Aug 16, 2022
@ghost
Copy link

ghost commented Aug 16, 2022

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details
  • Switch to node 14.x instead of 18.x, because the latter doesn't seem
    to be compatible with ubuntu 18.x:
    node: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.28' not found (required by node)`

  • Also, remove the existing nodejs package before installing a new
    one, because these machines are shared, and would get affected by
    installations performed by other runs.

Author: radical
Assignees: -
Labels:

arch-wasm, area-Infrastructure-mono

Milestone: -

@ghost ghost assigned radical Aug 16, 2022
@radical
Copy link
Member Author

radical commented Aug 16, 2022

/azp run runtime-wasm-perf

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@radical
Copy link
Member Author

radical commented Aug 16, 2022

/azp run runtime-wasm-perf

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@radical
Copy link
Member Author

radical commented Aug 16, 2022

/azp run runtime-wasm-perf

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

This allows cases which don't care about this, to continue working,
because they won't set `$PERF_PREREQS_INSTALL_FAILED`, and default to
continuing with the job.
@radical
Copy link
Member Author

radical commented Aug 16, 2022

/azp run runtime-wasm-perf

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@radical
Copy link
Member Author

radical commented Aug 17, 2022

/azp run runtime-wasm-perf

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@radical
Copy link
Member Author

radical commented Aug 17, 2022

@DrewScoggins I have a run from this in dotnet-runtime-perf - 20220817.1 . Does that look good? There are some failures on the wasm jobs, but those are related to recent ILLinker changes. I'll fix those.
The other jobs seem to be running too, but I checked only with some random sampling.

@DrewScoggins
Copy link
Member

Just the one other comment, other than that LGTM.

@radical
Copy link
Member Author

radical commented Aug 17, 2022

Opened dotnet/performance#2563, and #74104 for the new failures.

@radical
Copy link
Member Author

radical commented Aug 17, 2022

And #74088 .

Any objections to merging this right away to unblock the pipeline, since the changes are strictly in performance only files?

@radical radical merged commit a250bfe into dotnet:main Aug 17, 2022
@radical radical deleted the fix-perf-node branch August 17, 2022 18:09
@radical
Copy link
Member Author

radical commented Aug 17, 2022

/backport to release/7.0-rc1

@github-actions
Copy link
Contributor

Started backporting to release/7.0-rc1: https://github.com/dotnet/runtime/actions/runs/2877411794

@ghost ghost locked as resolved and limited conversation to collaborators Sep 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants