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

[CI] Re-examine node/yarn install logic for CI #45416

Open
brianseeders opened this issue Sep 11, 2019 · 0 comments
Open

[CI] Re-examine node/yarn install logic for CI #45416

brianseeders opened this issue Sep 11, 2019 · 0 comments
Assignees
Labels
Feature:CI Continuous integration release_note:skip Skip the PR/issue when compiling release notes

Comments

@brianseeders
Copy link
Contributor

I split out setup_env, because I need to set up all of the environment stuff every time sh is called in the Jenkinsfile.

I was originally having issues with the parallel workers each trying to re-install node/yarn, so I introduced an argument to skip node/yarn installing, and just set up the env. But now that I look at the script again, I'm not sure why that was happening in the first place. Will take a second look

Originally posted by @brianseeders in #45285

@brianseeders brianseeders self-assigned this Sep 11, 2019
@brianseeders brianseeders added Feature:CI Continuous integration release_note:skip Skip the PR/issue when compiling release notes labels Sep 11, 2019
@brianseeders brianseeders changed the title Re-examine node/yarn install logic for CI [CI] Re-examine node/yarn install logic for CI Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:CI Continuous integration release_note:skip Skip the PR/issue when compiling release notes
Projects
None yet
Development

No branches or pull requests

1 participant