-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
WIP: Update NodeJS and dependencies #22557
WIP: Update NodeJS and dependencies #22557
Conversation
4af19e7
to
e229ef8
Compare
89e9d72
to
ac3e4e4
Compare
… nodejs v20 Signed-off-by: Tibor Dancs <[email protected]>
…thorizationHeaderHandler Signed-off-by: Tibor Dancs <[email protected]>
ac3e4e4
to
85ef0fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ScrewTSW: image quay.io/eclipse/che-e2e:7.74.0-nj20.ts5 has failed to run the test because of error:
22:50:09 1) Uncaught error outside test suite
22:50:09 [ERROR] CheReporter runner.on.fail: Uncaught error outside test suite failed after undefinedms
Test build: https://main-jenkins-csb-crwqe.apps.ocp-c1.prod.psi.redhat.com/job/DevSandbox/job/sandbox-test/8237/console
Could you please take a look?
@musienko-maxim @dmytro-ndp should we close this PR or it is planned to be merged at some point? |
closing because outdated. recommend adapting the nodejs installation strategy though |
What does this PR do?
Screenshot/screencast of this PR
What issues does this PR fix or reference?
How to test this PR?
test image is available at quay.io/eclipse/che-e2e:7.74.0-nj20.ts5
Test Log
PR Checklist
As the author of this Pull Request I made sure that:
What issues does this PR fix or reference
andHow to test this PR
completedReviewers
Reviewers, please comment how you tested the PR when approving it.