Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature port forward range #1

Open
wants to merge 47 commits into
base: main
Choose a base branch
from

Conversation

mhdsulaimzed
Copy link

DaanHoogland and others added 30 commits April 20, 2024 10:44
md5 deprecated
sha512 use specific tool for easy validation
As those are single tests, create and data source getting the value on the same execution, the plan after apply is expected to be empty (unless Cloudstack returns some change on the API)
fabiomatavelli and others added 17 commits April 27, 2024 10:18
…#119)

Bumps [github.com/apache/cloudstack-go/v2](https://github.com/apache/cloudstack-go) from 2.13.2 to 2.16.1.
- [Release notes](https://github.com/apache/cloudstack-go/releases)
- [Commits](apache/cloudstack-go@v2.13.2...v2.16.1)

---
updated-dependencies:
- dependency-name: github.com/apache/cloudstack-go/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Grant triaging role to @fabiomatavelli @CodeBleu so they can run tests themselves and use the Github repo better
* ci: update terraform and opentofu versions

* ci: update cloudstack versions
)

Add support for port ranges in port forwarding rules by introducing optional
private_end_port and public_end_port fields. This allows users to specify
port ranges instead of just single ports when creating port forwarding rules.

- Add private_end_port and public_end_port fields as optional parameters
- Update createPortForward to handle end port configuration
- Modify read function to properly manage end port state
- Maintain backward compatibility with existing configurations

Fixes apache#157
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.

7 participants