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

Migrate nodejs installation to recommended by nodesource #197

Merged
merged 2 commits into from
Jan 31, 2024

Conversation

boesing
Copy link
Member

@boesing boesing commented Oct 13, 2023

@boesing boesing added this to the 1.37.1 milestone Oct 13, 2023
@boesing
Copy link
Member Author

boesing commented Oct 13, 2023

There is a 60 seconds sleep within the setup_20.x script which we used before. So this will most probably safe us at least 60 seconds (if not more, since we installed npm/nodejs in two stages).

@boesing
Copy link
Member Author

boesing commented Oct 13, 2023

I only found the 20 seconds sleep, those files are meanwhile removed from that repository but I built a few days ago and have seen 60 seconds.
So IMHO time to migrate:
nodesource/distributions@0c4edf5#diff-6369c68235f4b48c9ea07b0437bdc8054b8a6ca40cb27b4eee9efb032e578b32R161

@Xerkus
Copy link
Member

Xerkus commented Jan 10, 2024

@boesing seems they went back to installation scripts

@Xerkus
Copy link
Member

Xerkus commented Jan 31, 2024

@boesing is this change still needed?

@Xerkus
Copy link
Member

Xerkus commented Jan 31, 2024

Jeez. Current installation fails silently to register nodesource and then default nodejs 12 is installed.

Yeah, this is needed.

boesing and others added 2 commits January 31, 2024 23:39
See also https://github.com/nodesource/distributions#installation-instructions

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
@Xerkus Xerkus force-pushed the qa/migrate-nodejs-installation branch from 7990ab5 to 9e015a0 Compare January 31, 2024 13:49
@Xerkus Xerkus modified the milestones: 1.37.1, 1.38.0 Jan 31, 2024
@Xerkus Xerkus changed the base branch from 1.37.x to 1.38.x January 31, 2024 13:50
@Xerkus Xerkus merged commit 90936ac into 1.38.x Jan 31, 2024
4 checks passed
@Xerkus Xerkus deleted the qa/migrate-nodejs-installation branch January 31, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants