Skip to content

feat(dremio): Onboarding STACKIT Dremio#1482

Draft
bjarneschroeder wants to merge 9 commits into
stackitcloud:mainfrom
bjarneschroeder:main
Draft

feat(dremio): Onboarding STACKIT Dremio#1482
bjarneschroeder wants to merge 9 commits into
stackitcloud:mainfrom
bjarneschroeder:main

Conversation

@bjarneschroeder
Copy link
Copy Markdown

Description

This is the onboarding PR for integration STACKIT Dremio into the STACKIT Terraform Provider.

It adds for customer the ability to:

  • Create, Update, Delete & Read Dremio Instances
  • Create, Update, Delete & Read Dremio Users

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

Bjarne Schröder added 5 commits May 22, 2026 10:43
…vice.

Adding:
- Dremio SDK integration
- `.../services/dremio/` directory with packages and stubs for both dremio instances and users
First draft for the implementation of a Dremio instance resource.
Not propagating it renders the provider unusable for Dremio, because Dremio is no global STACKIT API yet.
@bjarneschroeder bjarneschroeder requested a review from a team as a code owner May 22, 2026 08:47
@bjarneschroeder bjarneschroeder marked this pull request as draft May 22, 2026 08:49
Bjarne Schröder added 4 commits May 22, 2026 14:23
The aim of this commit was to dry up and atomize common fields which are both used by the instance resource and data resource.
By drying up the methods and model constellation we can make use of one implementation.
Also removed the endpoints object.
The data resource does not read all the fields. There is only one authentication setup read, because only one can be used at a time. Also we are not reading the client secret here.
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