feat: jaas group data source

[pkulik0]

Description

This PR adds a data source for JAAS groups. Additionally it removes unused arguments from jaas client methods and upgrades the version of jimm-go-sdk.

Issue: #604

Type of change

• Add new resource
• Other - changes to jaas client

QA steps

Run this against a JAAS controller:

terraform {
  required_providers {
    juju = {
      version = ">= 0.15.0"
      source  = "juju/juju"
    }
  }
}

resource "juju_jaas_group" "test" {
  name = "group-0"
}

data "juju_jaas_group" "test" {
  name = juju_jaas_group.test.name
}

output "group_uuid" {
  value = data.juju_jaas_group.test.uuid
}

[jujubot]

Thanks for opening a pull request! Please follow the instructions here to ensure your pull request is ready for review. Then, a maintainer will review your patch.

@hpidcock @anvial

[jujubot]

Thanks for opening a pull request! Please follow the instructions here to ensure your pull request is ready for review. Then, a maintainer will review your patch.

@hpidcock @anvial

[hmlanigan]

This PR is missing documentation for the new data source, QA steps in the PR, and commit message does not cover what was done and any.

It should be done in at least 2 commits, rather than including the version bump into other changes.

[pkulik0]

Please re-run the failed tests when you're here. They failed because of a TLS handshake time out when downloading a file.

[kian99]

One suggestion, add an example in examples/data-sources/juju_jaas_group/ and then regenerate the docs and it will show up in here.
That's thanks to the way TF generates docs, you can see custom templates in templates/

[hmlanigan]

One comment, and please update to include the example per @kian99.

We're having some issues with the terraform registry which will delay landing this PR.

[hmlanigan]

todo: add newline above. This group of imports should be in 3 stanzas.

[kian99]

@hmlanigan
Can we look at landing this PR?
It's missing the example for the documentation and the imports need to be split.
I can make a follow-up after this lands to get that in. Otherwise I was trying to make the changes directly into Piotr's branch but it doesn't seem possible since the PR doesn't allow repo maintainers to make edits.

[hmlanigan]

I'm holding off landing this PR as we wanted to verify the release process by releasing a patch. I have work to update some documents in process to make the patch useful. We'd rather test the release process with a patch which has no schedule. We're currently using only one branch which necessitates the hold. This change requires a minor version release.

Nor could it land today with the failing required tests. We need to kick of the tests again.

[kian99]

@hmlanigan Any update on getting this in?

[kian99]

I've added the example and updated the imports as requested.

[hmlanigan]

/merge