Skip to content

Commit b59d8f6

Browse files
committed
Review changes: rewriting code for Create in neat way
1 parent 2c1ff1c commit b59d8f6

File tree

3 files changed

+30
-30
lines changed

3 files changed

+30
-30
lines changed

apis/v1alpha1/ack-generate-metadata.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
ack_generate_info:
2-
build_date: "2023-10-24T22:08:44Z"
2+
build_date: "2024-01-02T20:26:00Z"
33
build_hash: 7a92e8913d7ee7f65b761552e764e994882a49af
44
go_version: go1.19
55
version: v0.27.1-5-g7a92e89

pkg/resource/version/sdk.go

Lines changed: 14 additions & 14 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
1-
res := &svcsdk.ListVersionsByFunctionInput{}
2-
res.FunctionName = desired.ko.Spec.FunctionName
3-
var list *svcsdk.ListVersionsByFunctionOutput
4-
list, err = rm.sdkapi.ListVersionsByFunctionWithContext(ctx, res)
5-
if err != nil {
6-
return nil, err
7-
}
8-
versionList := list.Versions
9-
10-
for ok := list.NextMarker != nil; ok; ok = (list.NextMarker != nil) {
11-
res.Marker = list.NextMarker
12-
list, err = rm.sdkapi.ListVersionsByFunctionWithContext(ctx, res)
1+
var marker *string = nil
2+
var versionList []*svcsdk.FunctionConfiguration
3+
for {
4+
listVersionsInput := &svcsdk.ListVersionsByFunctionInput{
5+
FunctionName: desired.ko.Spec.FunctionName,
6+
Marker: marker,
7+
}
8+
listVersionResponse, err := rm.sdkapi.ListVersionsByFunctionWithContext(ctx, listVersionsInput)
139
if err != nil {
1410
return nil, err
1511
}
16-
versionList = append(versionList, list.Versions...)
17-
}
18-
12+
versionList = append(versionList, listVersionResponse.Versions...)
13+
14+
if listVersionResponse.NextMarker == nil {
15+
break
16+
}
17+
marker = listVersionResponse.NextMarker
18+
}

0 commit comments

Comments
 (0)