Skip to content

feat(iaas): min/max acc tests#811

Merged
marceljk merged 14 commits intomainfrom
feat/iaas-acc-tests
May 9, 2025
Merged

feat(iaas): min/max acc tests#811
marceljk merged 14 commits intomainfrom
feat/iaas-acc-tests

Conversation

@marceljk
Copy link
Copy Markdown
Contributor

@marceljk marceljk commented May 8, 2025

Description

relates to STACKITTPR-190

  • feat: add min/max acc tests
  • fix: stackit_image when VirtioScsi is set to false, the provider returns an error
  • fix: stackit_key_pair missing RequiresReplace for name
  • fix: stackit_network_area validation for min_prefix_length adjusted
  • fix: stackit_public_ip set network_interface_id to computed
  • fix: stackit_server doesn't load user_data

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)

Marcel Jacek added 13 commits May 2, 2025 16:40
feat(iaas): security group min/max acc test
de69bc8
feat(iaas): image min/max acc test
a43d8f4
feat(iaas): KeyPair min/max acc test
313165a
feat(iaas): Network area min/max acc test
7651da4 
- fix: wrong atLeast-validator for `minimum_prefix_length`
feat(iaas): Network min/max acc test
3e501bc
feat(iaas): Volume min/max acc test
4b03b28 
- fix: volume update doesn't work if no name was defined
- fix: volume.server_id can not be set
- fix: error message volume.size returns value but was null
rename testfiles folder to testdata
2e56fbf
feat(iaas): Network interfaces min/max acc test
78f8569
feat(iaas): Affinity groups acc test
fd8e2d7
feat(iaas): Server min/max acc test
8645fd8 
- stackit_server_volume_attach
- stackit_server_network_interface_attach
- stackit_server_service_account_attach
fix(iaas): acc test
5e63080 
- image: fix read of Config.VirtioScsi
- keypair: add missing RequiresReplace() for name
- server: add missing UserData read in datasource and resource
feat(iaas): public ip acc test
597afd0 
- fix: when a nic is assigned to a public ip, the field network_interface_id leads to recreation
improve checkDestory and log name of the test resources
94a3bff
bahkauv70
bahkauv70 suggested changes May 8, 2025
View reviewed changes
Comment thread stackit/internal/services/iaas/iaas_acc_test.go Outdated
Comment thread stackit/internal/services/iaas/image/resource.go
Comment thread stackit/internal/services/iaas/image/datasource.go
Comment thread stackit/internal/services/iaas/testdata/resource-server-max.tf
Comment thread stackit/internal/services/iaas/testdata/resource-server-max.tf
Comment thread stackit/internal/services/iaas/testdata/resource-server-min.tf
Comment thread stackit/internal/services/iaas/testdata/resource-volume-max.tf
@marceljk marceljk merged commit c7c64a5 into main May 9, 2025
3 checks passed
@marceljk marceljk deleted the feat/iaas-acc-tests branch May 9, 2025 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Fyusel Fyusel Awaiting requested review from Fyusel

@rubenhoenle rubenhoenle Awaiting requested review from rubenhoenle

1 more reviewer

@bahkauv70 bahkauv70 bahkauv70 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@marceljk @bahkauv70