Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

Conversation

denys-octopus
Copy link
Contributor

Adds documentation about our strategy how to deal with version differences between Provider and Octopus Deploy server.

This PR also prevent usage of octopusdeploy_deployment_freeze resource when connected Octopus Server version is not 2025.1 or later

Copy link
Contributor

@domenicsim1 domenicsim1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very Nice 👍
Left a comment around existing resources but we can update that as part of the Design Principles task.

Comment on lines +93 to +94
> [!WARNING]
> Please avoid using [blocks](https://developer.hashicorp.com/terraform/plugin/framework/handling-data/blocks): these are mainly used for backwards compatability of resources migrated from SDK. Use [nested attributes](https://developer.hashicorp.com/terraform/plugin/framework/handling-data/attributes#nested-attribute-types) instead.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

README.md Outdated
Comment on lines 123 to 124
## Existing Resource
When modifying an existing SDK resource, we strongly recommend migrating it to Framework first - but this might not always be feasible. We'll judge it on a case-by-case basis.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reason to not migrate is sometimes the resource is not compatible in framework without breaking changes. In this case a new resources needs to be created to replace the old one and the old one should become deprecated.

@denys-octopus denys-octopus merged commit 51c67bd into main May 23, 2025
24 checks passed
@denys-octopus denys-octopus deleted the dk/version-strategy branch May 23, 2025 01:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants