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: use N1_HIGHCPU_8 for cloudbuild and update cloudbuild image #714

Merged
merged 1 commit into from Aug 30, 2021

Conversation

aramase
Copy link
Member

@aramase aramase commented Aug 27, 2021

Signed-off-by: Anish Ramasekar anish.ramasekar@gmail.com

What this PR does / why we need it:

Currently we build linux/amd64, linux/arm64, windows/1809, windows/1903, windows/1909 and windows/2004 images with cloudbuild. This takes ~1h10m to complete. Use larger machine type N1_HIGHCPU_8 for cloudbuild jobs to speed up the build. Also, update the cloudbuild base image to a newer version.

Note: This machine type is used across all kubernetes builds.

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #

Special notes for your reviewer:

Currently we build linux/amd64, linux/arm64, windows/1809, windows/1903,
windows/1909 and windows/2004 images with cloudbuild. This takes ~1h10m
to complete. Use larger machine type N1_HIGHCPU_8 for cloudbuild jobs to
speed up the build. Also, update the cloudbuild base image to a newer
version.

Note: This machine type is used across all kubernetes builds.

Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 27, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aramase

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 27, 2021
@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Aug 28, 2021

@aramase: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Rerun command
pull-secrets-store-csi-driver-e2e-provider a1adfe3 link /test pull-secrets-store-csi-driver-e2e-provider

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tam7t
Copy link
Contributor

tam7t commented Aug 30, 2021

/test pull-secrets-store-csi-driver-e2e-windows

@tam7t
Copy link
Contributor

tam7t commented Aug 30, 2021

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 30, 2021
@k8s-ci-robot k8s-ci-robot merged commit ef4dd2f into kubernetes-sigs:master Aug 30, 2021
@aramase aramase deleted the N1_HIGHCPU_8 branch August 30, 2021 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants