Skip to content

Commit 783ae0e

Browse files
committed
breaking(loadbalancer) refactor DeleteLoadBalancerWaitHandler signature
1 parent fae4494 commit 783ae0e

3 files changed

Lines changed: 11 additions & 8 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -176,6 +176,7 @@
176176
- **Dependencies:** Bump STACKIT SDK core module to `v0.26.0`
177177
- [v1.13.0](services/loadbalancer/CHANGELOG.md#v1130)
178178
- **Improvement:** Use new WaiterHelper for LoadBalancer waiters
179+
- **Breaking Change:** `v2api/wait/DeleteLoadBalancerWaitHandler` now returns a `LoadBalancer` instead of `struct{}`
179180
- `logme`:
180181
- [v0.27.3](services/logme/CHANGELOG.md#v0273)
181182
- **Dependencies:** Bump STACKIT SDK core module from `v0.24.0` to `v0.24.1`

services/loadbalancer/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
## v1.13.0
22
- **Improvement:** Use new WaiterHelper for LoadBalancer waiters
3+
- **Breaking Change:** `v2api/wait/DeleteLoadBalancerWaitHandler` now returns a `LoadBalancer` instead of `struct{}`
34

45
## v1.12.2
56
- **Dependencies:** Bump STACKIT SDK core module to `v0.26.0`

services/loadbalancer/v2api/wait/wait.go

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -46,15 +46,16 @@ func CreateLoadBalancerWaitHandler(ctx context.Context, a loadbalancer.DefaultAP
4646
}
4747

4848
// DeleteLoadBalancerWaitHandler will wait for load balancer deletion
49-
func DeleteLoadBalancerWaitHandler(ctx context.Context, a loadbalancer.DefaultAPI, projectId, region, instanceId string) *wait.AsyncActionHandler[struct{}] {
50-
waitConfig := wait.WaiterHelper[struct{}, string]{
51-
FetchInstance: func() (*struct{}, error) {
52-
_, err := a.GetLoadBalancer(ctx, projectId, region, instanceId).Execute()
53-
return &struct{}{}, err
54-
},
55-
GetState: func(_ *struct{}) (string, error) {
56-
return "", nil
49+
func DeleteLoadBalancerWaitHandler(ctx context.Context, a loadbalancer.DefaultAPI, projectId, region, instanceId string) *wait.AsyncActionHandler[loadbalancer.LoadBalancer] {
50+
waitConfig := wait.WaiterHelper[loadbalancer.LoadBalancer, string]{
51+
FetchInstance: a.GetLoadBalancer(ctx, projectId, region, instanceId).Execute,
52+
GetState: func(l *loadbalancer.LoadBalancer) (string, error) {
53+
if l == nil || l.Status == nil {
54+
return "", errors.New("response or status is nil")
55+
}
56+
return *l.Status, nil
5757
},
58+
ErrorState: []string{LOADBALANCERSTATUS_ERROR},
5859
}
5960
handler := wait.New(waitConfig.Wait())
6061
handler.SetTimeout(15 * time.Minute)

0 commit comments

Comments
 (0)