Skip to content

fix: gcb builder incorrectly assumes target project from worker pool project#9725

Merged
Darien-Lin merged 2 commits intoGoogleContainerTools:mainfrom
unburdenedio:gcb-private-pool
Mar 5, 2025
Merged

fix: gcb builder incorrectly assumes target project from worker pool project#9725
Darien-Lin merged 2 commits intoGoogleContainerTools:mainfrom
unburdenedio:gcb-private-pool

Conversation

@bogdannazarenko
Copy link
Copy Markdown
Contributor

@bogdannazarenko bogdannazarenko commented Feb 18, 2025

Fixes: #9730
Related: N/A
Merge before/after: N/A

Description

  • This change fixes logic for retrieving destination project and region for Cloud Build build when using a custom WorkerPool from another project.
  • Added unit test to get proper location

@bogdannazarenko bogdannazarenko requested a review from a team as a code owner February 18, 2025 15:45
@google-cla
Copy link
Copy Markdown

google-cla bot commented Feb 18, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@bogdannazarenko bogdannazarenko marked this pull request as draft February 18, 2025 15:47
@pull-request-size pull-request-size bot added size/L and removed size/M labels Feb 21, 2025
@bogdannazarenko bogdannazarenko changed the title fix: change me fix: gcb builder incorrectly assumes target project from worker pool project Feb 21, 2025
@pull-request-size pull-request-size bot added size/S and removed size/L labels Feb 21, 2025
@pull-request-size pull-request-size bot added size/M and removed size/S labels Feb 22, 2025
@bogdannazarenko bogdannazarenko marked this pull request as ready for review February 22, 2025 02:02
@Darien-Lin Darien-Lin self-assigned this Feb 24, 2025
Darien-Lin
Darien-Lin previously approved these changes Feb 24, 2025
@Darien-Lin Darien-Lin self-requested a review February 24, 2025 17:37
@Darien-Lin
Copy link
Copy Markdown
Contributor

Darien-Lin commented Feb 24, 2025

seems that there are some errors from the workflow: File is not gci-ed with --skip-generated -s standard -s default -s prefix(github.com/GoogleContainerTools/skaffold) (gci)

Complaining about the import in the test it seems. Can you update?

@Darien-Lin Darien-Lin dismissed their stale review February 24, 2025 17:41

File is not gci-ed with --skip-generated -s standard -s default -s prefix(github.com/GoogleContainerTools/skaffold) (gci)

@Darien-Lin Darien-Lin merged commit 701e3dc into GoogleContainerTools:main Mar 5, 2025
11 checks passed
@bogdannazarenko bogdannazarenko deleted the gcb-private-pool branch March 5, 2025 17:24
@menahyouyeah menahyouyeah mentioned this pull request Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GCB builder incorrectly assumes GCP project from supplied WorkerPool property for Cloud Build execution

2 participants