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

chore: switched development version to v20 #1152

Merged
merged 28 commits into from
May 24, 2024
Merged

chore: switched development version to v20 #1152

merged 28 commits into from
May 24, 2024

Conversation

kirrg001
Copy link
Contributor

@kirrg001 kirrg001 commented May 16, 2024

refs https://jsw.ibm.com/browse/INSTA-983

Now that we have resolved the issue with ESM and Node v18.18/v18.19, we can switch to Node v20.
Node v20 is LTS.

  • switch pipelines to use v20
  • manually verify that the docker images build
  • fixed slack notifications for ci.instana.io
  • add global "node-version" in Tekton which is always used when the pipeline itself did not overwrite it (Tekton automatically sends the correct version)

@kirrg001 kirrg001 force-pushed the node20-dev branch 4 times, most recently from eb31620 to e2a9f38 Compare May 22, 2024 07:40
@@ -29,7 +29,8 @@ spec:
value: "default-pipeline"
- name: skip-cache
value: $(params.skip-cache)

- name: context
value: "node-dev"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renaming the default pipeline for main and PR to "node-dev". Because the build name needs to be configured in Github as well (required build to pass). If we choose a generic name, we won't need to update anything when jumping to a higher Node.js version.

@@ -108,7 +108,7 @@ resources:
type: slack-notification
icon: slack
source:
url: ((tracer-community-slack-hook))
url: ((team-nodejs-slack-hook))
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a new nodejs slack webhook.

@kirrg001 kirrg001 marked this pull request as ready for review May 22, 2024 08:06
@kirrg001 kirrg001 requested a review from a team as a code owner May 22, 2024 08:06
@kirrg001
Copy link
Contributor Author

I have already started to test the images. PR is RFR.

@kirrg001 kirrg001 force-pushed the node20-dev branch 3 times, most recently from f0818e8 to f72d084 Compare May 23, 2024 07:39
@kirrg001 kirrg001 merged commit b20da85 into main May 24, 2024
1 check passed
@kirrg001 kirrg001 deleted the node20-dev branch May 24, 2024 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants