Skip to content

[bug]: Support helm depBuild flags for helm deploys#9808

Merged
alphanota merged 1 commit intoGoogleContainerTools:mainfrom
vihangm:support_dep_build_deploy
May 19, 2025
Merged

[bug]: Support helm depBuild flags for helm deploys#9808
alphanota merged 1 commit intoGoogleContainerTools:mainfrom
vihangm:support_dep_build_deploy

Conversation

@vihangm
Copy link
Contributor

@vihangm vihangm commented May 3, 2025

Related: #8413, #9696

Description
The helm depBuild flags were added in the most recent version of skaffold, but
they are currently only supported when using helm to render and not supported
when using helm to deploy. This ensures that the flags are applied on deploy
too.

User facing changes (remove if N/A)
The flag is already a part of the schema, it was just ignored before this.

@vihangm vihangm requested a review from a team as a code owner May 3, 2025 00:46
@google-cla
Copy link

google-cla bot commented May 3, 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.

@vihangm vihangm force-pushed the support_dep_build_deploy branch from a614e98 to 80cab18 Compare May 3, 2025 00:51
@alphanota
Copy link
Contributor

Hi @vihangm, thanks for this PR. Before we can review this, you have to sign the Contribution License Agreement. Please follow the instructions here to sign the license and re-run the cla check.

📝 If you are not currently covered under a CLA, please visit
https://cla.developers.google.com/. Once you've signed, follow the "New Contributors" > link at the bottom of this page to update this check.

@alphanota alphanota self-assigned this May 6, 2025
@vihangm
Copy link
Contributor Author

vihangm commented May 8, 2025

Sorry for the delay here, I am waiting on the authorized signatory to complete the CLA!

@vihangm
Copy link
Contributor Author

vihangm commented May 8, 2025

@alphanota This is good to go now!

@alphanota alphanota merged commit 537cb9b into GoogleContainerTools:main May 19, 2025
11 checks passed
@vihangm vihangm deleted the support_dep_build_deploy branch May 19, 2025 17:15
@vihangm
Copy link
Contributor Author

vihangm commented Aug 14, 2025

@alphanota It's been almost 2 months since the last release. Any update on future release schedules? I'm waiting for this fix to make it in.

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.

2 participants