Skip to content

Don't build and test EOL versions#287

Open
bhouse-nexthop wants to merge 3 commits intoapache:mainfrom
bhouse-nexthop:update-versions
Open

Don't build and test EOL versions#287
bhouse-nexthop wants to merge 3 commits intoapache:mainfrom
bhouse-nexthop:update-versions

Conversation

@bhouse-nexthop
Copy link
Collaborator

@bhouse-nexthop bhouse-nexthop commented Mar 10, 2026

There's no reason to build and test EOL versions of Cloudstack, OpenTofu, and Terraform. Also recent supported versions were being excluded.

This PR will also corrects test failures in newer versions and updates documentation to be more explicit in how to properly run the acceptance tests.

There's no reason to build and test EOL versions of Cloudstack, OpenTofu, and Terraform.
Also recent supported versions were being excluded.
@bhouse-nexthop bhouse-nexthop marked this pull request as draft March 10, 2026 21:28
…r bug

- Add getCloudStackVersion() helper function to retrieve CloudStack version from API
- Skip TestAccCloudStackLoadBalancerRule_update on version 4.22.0.0 specifically
- Version 4.22.0.0 has a known bug causing '530 Internal Server Error' when updating LB rules
- Test still runs on all other versions including 4.20.1.0, 4.22.1.0+, and 4.23.0.0+
- Update README with detailed simulator setup instructions including deployDataCenter.py step
- Fix state drift issues in CNI configuration and service offering resources
- Improve error reporting in load balancer rule updates to show raw API errors
@bhouse-nexthop bhouse-nexthop marked this pull request as ready for review March 11, 2026 01:04
@bradh352
Copy link

@vishesh92 this will be needed to get clean tests for all the other PRs I created. Interestingly it appears the 4.22.0.0 simulator has some bug I had to disable some tests for. 4.21.1.0 and 4.23.0.0-snapshots worked, so there's likely a fix in upstream cloudstack just don't know what it was.

@bhouse-nexthop
Copy link
Collaborator Author

@vishesh92 this will be needed to get clean tests for all the other PRs I created. Interestingly it appears the 4.22.0.0 simulator has some bug I had to disable some tests for. 4.21.1.0 and 4.23.0.0-snapshots worked, so there's likely a fix in upstream cloudstack just don't know what it was.

whoops, posted this on my personal account instead of work account. Replying so you know it was really me :)

@vishesh92 vishesh92 requested a review from DaanHoogland March 11, 2026 07:05
@bhouse-nexthop
Copy link
Collaborator Author

@DaanHoogland any comment on this PR? it will fix the test issues on my other PRs once this is merged.

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.

2 participants