Skip to content

fix(iaas): read write only field correctly from the config model - #1694

Merged
SerseusWasTaken merged 4 commits into
mainfrom
fix/fix-iaas-write-only-fields
Aug 21, 2026
Merged

fix(iaas): read write only field correctly from the config model#1694
SerseusWasTaken merged 4 commits into
mainfrom
fix/fix-iaas-write-only-fields

Conversation

@SerseusWasTaken

@SerseusWasTaken SerseusWasTaken commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes the write-only field KeyPayloadBase64WriteOnly by reading it from the config model instead of plan model.
Also adds a regression test based on mocking that verifies that the write only field is sent to the API.

Relates to STACKITTPR-782

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)

@SerseusWasTaken
SerseusWasTaken requested a review from a team as a code owner August 17, 2026 07:34
@SerseusWasTaken
SerseusWasTaken force-pushed the fix/fix-iaas-write-only-fields branch 2 times, most recently from e4cdcee to 2c2559d Compare August 17, 2026 07:41
Comment thread stackit/internal/services/iaas/volume/resource_create_test.go Outdated
@SerseusWasTaken
SerseusWasTaken force-pushed the fix/fix-iaas-write-only-fields branch from d7e766c to 537f389 Compare August 18, 2026 09:23
@SerseusWasTaken
SerseusWasTaken merged commit 66ac0a6 into main Aug 21, 2026
3 checks passed
@SerseusWasTaken
SerseusWasTaken deleted the fix/fix-iaas-write-only-fields branch August 21, 2026 12:54
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.

3 participants