Skip to content

feat(intake): port to v1betaapi#1364

Open
yago-123 wants to merge 5 commits intostackitcloud:mainfrom
stackit-intake:switch-intake-new-SDK
Open

feat(intake): port to v1betaapi#1364
yago-123 wants to merge 5 commits intostackitcloud:mainfrom
stackit-intake:switch-intake-new-SDK

Conversation

@yago-123
Copy link
Copy Markdown
Contributor

@yago-123 yago-123 commented Apr 10, 2026

Description

This PR migrates the current STACKIT Intake implementation to the new SDK hierarchy structure. In our case we must start using v1betaapi.

In order to perform this port we had to:

  1. Update all SDK import paths of Intake to v1betaapi
  2. Explicit use of DefaultAPI instead of using directly the client
  3. Shift from pointers to values for required fields
  4. Fix go-cmp to ensure unexported private fields do not affect the test output
  5. Adjust some linting changes

Checklist

  • Issue was linked above (not needed)
  • Code format was applied: make fmt
  • Examples were added / adjusted (see e.g. here) (not needed)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • 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)

@yago-123 yago-123 requested a review from a team as a code owner April 10, 2026 08:58
Yago Carlos Fernandez Gou added 3 commits April 10, 2026 11:04
# Conflicts:
#	internal/cmd/beta/intake/create/create.go
#	internal/cmd/beta/intake/create/create_test.go
#	internal/cmd/beta/intake/delete/delete.go
#	internal/cmd/beta/intake/delete/delete_test.go
#	internal/cmd/beta/intake/describe/describe_test.go
#	internal/cmd/beta/intake/list/list_test.go
#	internal/cmd/beta/intake/runner/create/create_test.go
#	internal/cmd/beta/intake/runner/delete/delete.go
#	internal/cmd/beta/intake/runner/delete/delete_test.go
#	internal/cmd/beta/intake/runner/describe/describe_test.go
#	internal/cmd/beta/intake/runner/list/list.go
#	internal/cmd/beta/intake/runner/list/list_test.go
#	internal/cmd/beta/intake/runner/update/update.go
#	internal/cmd/beta/intake/runner/update/update_test.go
#	internal/cmd/beta/intake/update/update_test.go
#	internal/cmd/beta/intake/user/create/create.go
#	internal/cmd/beta/intake/user/create/create_test.go
#	internal/cmd/beta/intake/user/delete/delete.go
#	internal/cmd/beta/intake/user/delete/delete_test.go
#	internal/cmd/beta/intake/user/describe/describe.go
#	internal/cmd/beta/intake/user/describe/describe_test.go
#	internal/cmd/beta/intake/user/list/list.go
#	internal/cmd/beta/intake/user/list/list_test.go
#	internal/cmd/beta/intake/user/update/update.go
#	internal/cmd/beta/intake/user/update/update_test.go
#	internal/pkg/services/intake/client/client.go
@yago-123 yago-123 force-pushed the switch-intake-new-SDK branch from 4471d4e to 38f30fb Compare April 10, 2026 09:05
Yago Carlos Fernandez Gou added 2 commits April 10, 2026 11:52
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.

1 participant