Skip to content

build(profiler): upgrade Kokoro CI configs and scripts to Node 22 - #9454

Merged
feywind merged 2 commits into
googleapis:mainfrom
feywind:reporx/cloud-profiler-fixes
Sep 25, 2026
Merged

feywind merged 2 commits into
googleapis:mainfrom
feywind:reporx/cloud-profiler-fixes

Conversation

@feywind

@feywind feywind commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Description

Upgrades the Kokoro CI configurations and scripts in handwritten/cloud-profiler to Node 22 (gcr.io/cloud-devrel-kokoro-resources/node:22-user) and pnpm@10, resolving CI failures in:

  • ci/kokoro: Samples test - handwritten/cloud-profiler
  • ci/kokoro: node14 - handwritten/cloud-profiler
  • ci/kokoro: node14 - nightly - handwritten/cloud-profiler
  • ci/kokoro: Samples test - nightly - handwritten/cloud-profiler

Summary of Changes

  • Node 22 Kokoro configs (node:22-user):
    • Updated .kokoro/common.cfg and .kokoro/release/{docs,docs-devsite,publish}.cfg from node:18-user to node:22-user.
    • Replaced .kokoro/continuous/node18/ and .kokoro/presubmit/{node13,node18}/ with .kokoro/continuous/node22/ and .kokoro/presubmit/node22/.
    • Added .kokoro/continuous/node14/ configured with gcr.io/cloud-devrel-kokoro-resources/node:22-user so existing Kokoro triggers pointing to build_config_dir: "handwritten/cloud-profiler/.kokoro/continuous/node14" find the build configs and execute on Node 22.
  • Kokoro scripts & package.json:
    • Updated .kokoro/*.sh and .kokoro/release/*.sh to configure PATH for ${NPM_CONFIG_PREFIX}/bin, install pnpm@10, use --filter @google-cloud/profiler... --frozen-lockfile, set COVERAGE_NODE=22, and guard flakybot / codecov.sh execution.
    • Updated .kokoro/samples-test.sh to skip running coverage when samples/package.json does not exist.
    • Removed the broken prelint script (cd samples; npm link ../; npm install) from handwritten/cloud-profiler/package.json and fixed lint issues in src/index.ts, src/profiler.ts, and test/test-init-config.ts.
    • Updated GCE integration test cases in system-test/integration_test.go to Node 22 and 24.

@feywind
feywind requested review from a team as code owners September 25, 2026 19:57
@product-auto-label product-auto-label Bot added the api: cloudprofiler Issues related to the Cloud Profiler API. label Sep 25, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Node.js environment to version 22 across Kokoro configurations and shell scripts, upgrades pnpm from version 9 to 10, and refines the installation and testing pipelines. It also updates integration tests to target Node 22 and 24, resolves unhandled promises using the void operator, and cleans up package scripts. The review feedback suggests double-quoting the $KOKORO_GFILE_DIR variable expansions in samples-test.sh and test.sh to prevent potential word splitting and globbing issues.

Comment thread handwritten/cloud-profiler/.kokoro/samples-test.sh
Comment thread handwritten/cloud-profiler/.kokoro/samples-test.sh
Comment thread handwritten/cloud-profiler/.kokoro/test.sh
Comment thread handwritten/cloud-profiler/.kokoro/test.sh
@feywind feywind changed the title fix(profiler): upgrade Kokoro CI configs and scripts to Node 22 build(profiler): upgrade Kokoro CI configs and scripts to Node 22 Sep 25, 2026
@feywind
feywind merged commit abffc7e into googleapis:main Sep 25, 2026
50 checks passed
@feywind
feywind deleted the reporx/cloud-profiler-fixes branch September 25, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: cloudprofiler Issues related to the Cloud Profiler API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants