Skip to content

feat(modelserving): Refactor wait handler to use new WaiterHelper#7274

Closed
j1n-o9r wants to merge 1 commit into
mainfrom
feat/STACKITSDK-370-modelserving-refactor-wait-handler-to-use-wait-helper
Closed

feat(modelserving): Refactor wait handler to use new WaiterHelper#7274
j1n-o9r wants to merge 1 commit into
mainfrom
feat/STACKITSDK-370-modelserving-refactor-wait-handler-to-use-wait-helper

Conversation

@j1n-o9r
Copy link
Copy Markdown

@j1n-o9r j1n-o9r commented May 13, 2026

Description

Update the wait handler of sfs, to use the new WaiterHelper struct. Replace every possible wait handler implementation of this service, to use just the WaiterHelper.

relates to STACKITSDK-370

Testing Instruction

  • checkout feat/STACKITSDK-368-refactor-wait-handler-to-use-wait-helper
  • run make test
  • go to your terraform repo and add this line to go mod, adjust path to sdk if necessary replace github.com/stackitcloud/stackit-sdk-go/services/sfs => ../stackit-sdk-go/services/sfs
  • run tf acceptance tests for sfs

Checklist

  • Issue was linked above
  • No generated code was adjusted manually (check comments in file header)
  • Changelogs
    • Changelog in the root directory was adjusted (see here)
    • Changelog(s) of the service(s) were adjusted (see e.g. here)
  • VERSION file(s) of the service(s) were adjusted
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Unit tests got implemented or updated
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@j1n-o9r j1n-o9r marked this pull request as ready for review May 13, 2026 13:00
@j1n-o9r j1n-o9r requested a review from a team as a code owner May 13, 2026 13:00
@j1n-o9r j1n-o9r marked this pull request as draft May 13, 2026 13:04
@j1n-o9r j1n-o9r closed this May 13, 2026
@j1n-o9r
Copy link
Copy Markdown
Author

j1n-o9r commented May 13, 2026

Wrong Ticket linked in branch name

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.

1 participant