terraform-provider-octopusdeploy icon indicating copy to clipboard operation
terraform-provider-octopusdeploy copied to clipboard

Should Project is_version_controlled be read only?

Open ArturDorochowicz opened this issue 2 years ago • 1 comments
trafficstars

Describe the bug The is_version_controlled argument of a project is settable, but it doesn't seem to do much. Should it be read only?

If I provide, for instance, a git_library_persistence_settings block in a project resource, the project will be created successfully with VCS support, no matter what value the is_version_controlled argument has (i.e. also when it is set to false).

Also, for an existing project configured with VCS support (created initially with is_version_controlled = true), I can change is_version_controlled to false and apply successfully. The project doesn't change to no VCS support. Not that I'd expect it, I'm just pointing out the fact that the argument doesn't appear to do anything.

Steps to reproduce

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Logs and other supporting information Add the output of running tf plan or tf apply along with any errors in the Octopus Server logs.

Screenshots If applicable, add screenshots to help explain your problem.

Environment and versions:

  • OS: Windows
  • Octopus Server Version: 2022.4.8474
  • Terraform Version: 1.3.9
  • Octopus Terraform Provider Version: 0.10.5

Additional context Add any other context about the problem here.

ArturDorochowicz avatar Mar 13 '23 10:03 ArturDorochowicz

I can see how this feels a bit redundant.

I think I'd prefer to keep the flag as editable so that it's a very explicit action to migrate to a config-as-code project, given that's a one-way door. However, I think the current implementation could be a bit nicer - with some proper error messages for expecting/requiring the git persistence settings to be available if the version controlled flag is set to true.

I'll leave this open for us to potentially add some nicer messaging into the provider logging.

mjhilton avatar Sep 10 '24 05:09 mjhilton

If this is still believed to be in v1.0.0 of the provider please reopen it in the repo.

On 1 June 2025, the Octopus Deploy Terraform Provider was promoted from a Labs project to a fully-supported, first-class Octopus Deploy integration.

As part of this promotion, we released v1.0 of the provider, and migrated the codebase from the OctopusDeployLabs GitHub organization to the OctopusDeploy organization. The provider also moved in the Hashicorp Terraform Registry.

New Repository: https://github.com/OctopusDeploy/ New Provider Registry Page: https://registry.terraform.io/providers/OctopusDeploy/octopusdeploy/latest/docs Migration Guide: https://registry.terraform.io/providers/OctopusDeploy/octopusdeploy/1.0.0/docs/guides/moving-from-octopus-deploy-labs-namespace We are no longer accepting contributions or issues in this repo, and we plan to archive it.

Please head over to the new repository for continued support, updates and contributions.

domenicsim1 avatar Jun 04 '25 03:06 domenicsim1