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: force rebuild of node 18 image #387

Merged
merged 3 commits into from
Jun 5, 2024
Merged

chore: force rebuild of node 18 image #387

merged 3 commits into from
Jun 5, 2024

Conversation

chingor13
Copy link
Contributor

@chingor13 chingor13 commented Jun 3, 2024

chore: stop building node < 18 (EoL)

trivial change to rebuild images
build: start with higher Cloud SDK CLI

use updated python
@chingor13 chingor13 marked this pull request as ready for review June 4, 2024 00:16
@chingor13 chingor13 requested a review from a team as a code owner June 4, 2024 00:16
@chingor13 chingor13 requested a review from sofisl June 4, 2024 00:16
@chingor13 chingor13 changed the title chore: update Dockerfile chore: force rebuild of node 18 image Jun 4, 2024
@@ -71,7 +71,7 @@ RUN python3 -m pip install --require-hashes -r requirements.txt
# Setup Cloud SDK
WORKDIR /home/node
ENV CLOUD_SDK_VERSION=446.0.0 \
CLOUDSDK_PYTHON=/usr/bin/python3 \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

System python was 3.7.x and gcloud CLI does not allow it anymore

Comment on lines -18 to -41
# Node 12
- name: gcr.io/cloud-builders/docker
args: ['build', '-t', 'gcr.io/cloud-devrel-kokoro-resources/node:12-user', '.']
dir: 'node/12-user'
waitFor: ['-']

# Node 14
- name: gcr.io/cloud-builders/docker
args: ['build', '-t', 'gcr.io/cloud-devrel-kokoro-resources/node:14-user', '.']
dir: 'node/14-user'
waitFor: ['-']

# Node 16
- name: gcr.io/cloud-builders/docker
args: ['build', '-t', 'gcr.io/cloud-devrel-kokoro-resources/node:16-user', '.']
dir: 'node/16-user'
waitFor: ['-']

# Node 16-puppeteer
- name: gcr.io/cloud-builders/docker
args: ['build', '-t', 'gcr.io/cloud-devrel-kokoro-resources/node:16-puppeteer', '.']
dir: 'node/16-puppeteer'
waitFor: ['-']

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These node versions are all EoL and no longer build.

@chingor13 chingor13 merged commit c734af0 into main Jun 5, 2024
10 checks passed
@chingor13 chingor13 deleted the chingor13-patch-1 branch June 5, 2024 17:57
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.

2 participants