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

refactor: new waiter functionality in helmAddonApplier #988

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

dkoshkin
Copy link
Contributor

@dkoshkin dkoshkin commented Dec 5, 2024

What problem does this PR solve?:
This PR is a small refactor to pull existing functionality into the common addon applier. I wanted similar functionality in another handler to wait for the HCP to be ready.

Which issue(s) this PR fixes:
Fixes #

How Has This Been Tested?:

Special notes for your reviewer:

@dkoshkin dkoshkin force-pushed the dkoshkin/refactor-common-lifecycle-functions branch from 01a8e23 to 3f0ad84 Compare December 5, 2024 03:48
@dkoshkin dkoshkin force-pushed the dkoshkin/refactor-common-lifecycle-functions branch from 3f0ad84 to 5dc7cbb Compare December 5, 2024 03:53
@jimmidyson jimmidyson enabled auto-merge (squash) December 5, 2024 09:22
@jimmidyson jimmidyson merged commit 8c40639 into main Dec 5, 2024
21 checks passed
@jimmidyson jimmidyson deleted the dkoshkin/refactor-common-lifecycle-functions branch December 5, 2024 09:54
@dkoshkin dkoshkin mentioned this pull request Dec 12, 2024
dkoshkin added a commit that referenced this pull request Jan 16, 2025
🤖 I have created a release *beep* *boop*
---


## 0.25.0 (2025-01-16)

<!-- Release notes generated using configuration in .github/release.yaml
at main -->

## What's Changed
### Exciting New Features 🎉
* feat: Virtual IP configuration to set different address/port by
@dkoshkin in
#986
* feat: update addon versions by @dkoshkin in
#997
* feat: COSI controller Addon by @dkoshkin in
#1008
### Fixes 🔧
* fix: check HelmReleaseReadyCondition when status is up-to-date by
@dkoshkin in
#989
* fix: update CoreDNS mapping file by @dkoshkin in
#998
* fix: validates PC IP is outside Load Balancer IP Range by
@manoj-nutanix in
#1001
* fix: update COSI controller image to fix CVEs by @dkoshkin in
#1012
* fix(deps): Update Nutanix CCM Version by @thunderboltsid in
#1016
* fix: update AWS CCM to latest versions by @dkoshkin in
#1013
### Other Changes
* refactor: new waiter functionality in helmAddonApplier by @dkoshkin in
#988
* build: Properly ignore ntnx API client from dependabot by @jimmidyson
in
#995


**Full Changelog**:
v0.24.0...v0.25.0

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants