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

fix helm deploy job vars when only deploy image in somecase #3908

Merged
merged 1 commit into from
Dec 24, 2024

Conversation

PetrusZ
Copy link
Contributor

@PetrusZ PetrusZ commented Dec 24, 2024

What this PR does / Why we need it:

fix helm deploy job vars when only update image

What is changed and how it works?

fix helm deploy job vars when only update image

Does this PR introduce a user-facing change?

  • API change
  • database schema change
  • upgrade assistant change
  • change in non-functional attributes such as efficiency or availability
  • fix of a previous issue

@PetrusZ PetrusZ force-pushed the fix/helm_deploy_job_vars branch from 1764972 to db8f811 Compare December 24, 2024 05:05
@PetrusZ PetrusZ changed the title fix helm deploy job vars when only update image fix helm deploy job vars when only deploy image in somecase Dec 24, 2024
@PetrusZ PetrusZ force-pushed the fix/helm_deploy_job_vars branch 3 times, most recently from 5b2b7db to a44d85f Compare December 24, 2024 07:36
@PetrusZ PetrusZ force-pushed the fix/helm_deploy_job_vars branch from a44d85f to 06673be Compare December 24, 2024 07:41
@PetrusZ PetrusZ mentioned this pull request Dec 24, 2024
5 tasks
@jamsman94 jamsman94 merged commit c582829 into koderover:main Dec 24, 2024
4 checks passed
jamsman94 added a commit that referenced this pull request Dec 24, 2024
@PetrusZ PetrusZ mentioned this pull request Jan 10, 2025
5 tasks
jamsman94 pushed a commit that referenced this pull request Jan 10, 2025
* Revert "fix merge override values order for helm service (#3935) (#3936)"

This reverts commit ff888d7.

* Revert "fix helm deploy job vars when only deploy image in somecase"

This reverts commit 5587a7a.

Signed-off-by: Patrick Zhao <[email protected]>

---------

Signed-off-by: Patrick Zhao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants