feat: migrate users data source to tf framework

[N-lson]

TF Config

data "octopusdeploy_users" "example" {
}

output "users" {
  value = data.octopusdeploy_users.example
}

Pre-migration

users = {
  "filter" = tostring(null)
  "id" = "Users 2024-10-09 00:33:05.6752504 +0000 UTC"
  "ids" = tolist(null) /* of string */
  "skip" = tonumber(null)
  "space_id" = tostring(null)
  "take" = tonumber(null)
  "users" = tolist([
    {
      "can_password_be_edited" = true
      "display_name" = "admin"
      "email_address" = ""
      "id" = "Users-1"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = false
      "is_service" = false
      "password" = ""
      "username" = "admin"
    },
    {
      "can_password_be_edited" = true
      "display_name" = "Bob Smith"
      "email_address" = ""
      "id" = "Users-83"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = false
      "is_service" = false
      "password" = ""
      "username" = "bsmith"
    },
    {
      "can_password_be_edited" = true
      "display_name" = "No permissions user"
      "email_address" = ""
      "id" = "Users-21"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = false
      "is_service" = false
      "password" = ""
      "username" = "noPerms"
    },
    {
      "can_password_be_edited" = true
      "display_name" = "testuser"
      "email_address" = ""
      "id" = "Users-61"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = false
      "is_service" = false
      "password" = ""
      "username" = "testuser"
    },
  ])
}

Post-migration

users = {
  "filter" = tostring(null)
  "id" = "Users 2024-10-09 00:34:05.4305356 +0000 UTC"
  "ids" = tolist(null) /* of string */
  "skip" = tonumber(null)
  "space_id" = tostring(null)
  "take" = tonumber(null)
  "users" = tolist([
    {
      "can_password_be_edited" = true
      "display_name" = "admin"
      "email_address" = ""
      "id" = "Users-1"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = true
      "is_service" = false
      "username" = "admin"
    },
    {
      "can_password_be_edited" = true
      "display_name" = "Bob Smith"
      "email_address" = ""
      "id" = "Users-83"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = false
      "is_service" = false
      "username" = "bsmith"
    },
    {
      "can_password_be_edited" = true
      "display_name" = "No permissions user"
      "email_address" = ""
      "id" = "Users-21"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = false
      "is_service" = false
      "username" = "noPerms"
    },
    {
      "can_password_be_edited" = true
      "display_name" = "testuser"
      "email_address" = ""
      "id" = "Users-61"
      "identity" = toset([])
      "is_active" = true
      "is_requestor" = false
      "is_service" = false
      "username" = "testuser"
    },
  ])
}

Changes

• password has been removed. This was always an empty string before.
• is_requestor now returns the correct value for the user running terraform apply.

[N-lson]

I think space_id should be removed entirely since users aren't space-scoped but I've left it in for backwards compatibility.

[domenicsim1]

hmm, good point. Lets mark it as deprecated.

[domenicsim1]

This looks good, let mark the space_id as deprecated as it does nothing and we will remove it in 1.0 release.

[domenicsim1]

hmm, good point. Lets mark it as deprecated.