feat: Add scoped user role resource

[evan-cleary]

Scoped User Role

User Roles can be defined on either the team resource, or managed as separate resource.

When used in combination any additional scoped user roles assigned to a team that are not defined within the team def would be removed upon apply. If roles are going to be modified from a non central location, the individual resource method should be utilized.

New resource: octopusdeploy_scoped_user_role
Modify Resource: octopusdeploy_team add block property "user_role"

Known Issues

Due to the necessity of including the "computed" attribute on the team user_role attribute, there is a standing bug when removing all user roles from a team. If all entries are removed at once, none will be deleted. If all but 1 are removed from the config, all those removed will be properly deleted. This can, of course, be manually cleaned up at a later point.

This bug stems from the inability to detect if a value is coming from the config or if it's computed. If we had the ability to determine if contents were coming through as computed, these could be compared against previous state and properly cleared.

Progress

• Scoped user role schema defined
• Individual octopusdeploy_scoped_used_role resource created
• User role block added to octopusdeploy_team resource
• Documentation updated
• Tests written

Depends On: OctopusDeploy/go-octopusdeploy#52
Closes #150

[ajaychoudhary-bcg]

In UI we can add multiple roles in team, here we are are giving access to single user role id.

[evan-cleary]

The "user_role" property of the team is a Set. Terraform for it would look like

resource "octopusdeploy_team" "this" {
  name = "Example"

  user_role {
    user_role_id = "userrole-1"
    space_id     = "Spaces-1"
  }
  user_role {
    user_role_id = "userrole-2"
    space_id     = "Spaces-1"
  }
}

[ajaychoudhary-bcg]

When this will be available?

[evan-cleary]

@ajaychoudhary-bcg I'll try to have this PR ready for review by the end of the week. I've been doing these changes in my free time. Everything is merged on the api library side so I need to rebase with upstream and write tests.

[ajaychoudhary-bcg]

@evan-cleary Please merge and create new version so that we can use it.

[evan-cleary]

@jbristowe this PR is ready for review

[ajaychoudhary-bcg]

@jbristowe this PR is ready for review

Any Update on this?

[ajaychoudhary-bcg]

@jbristowe this PR is ready for review

Any Update on this?

Any update, we are waiting for this to be available from few weeks.

[ajaychoudhary-bcg]

@jbristowe @evan-cleary Any update on this?

[ajaychoudhary-bcg]

Please merge and provide new version

[evan-cleary]

@jbristowe Should be good to go now for the data object

[jbristowe]

🎉 Thank you!