adds roots attribute to the aws_organizations_organization resource

[bryanlalexander]

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

As mentioned in #4207 @bflad requested the modification of the aws_organizations_organization resource to contain the response from organizations:ListRoots. This will resolve the dependency on a data source when creating an organizations_organizational_unit mentioned in #4207.

Changes proposed in this pull request:

• adds the response from calling organizations:ListRoots to the aws_organizations_organization resource
• updates attribute documentation on website

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccAWSOrganizations'

=== RUN   TestAccAWSOrganizations
=== RUN   TestAccAWSOrganizations/Organization
=== RUN   TestAccAWSOrganizations/Organization/basic
=== RUN   TestAccAWSOrganizations/Organization/importBasic
=== RUN   TestAccAWSOrganizations/Organization/consolidatedBilling
=== RUN   TestAccAWSOrganizations/Account
=== RUN   TestAccAWSOrganizations/Account/basic
--- PASS: TestAccAWSOrganizations (61.00s)
    --- PASS: TestAccAWSOrganizations/Organization (60.99s)
        --- PASS: TestAccAWSOrganizations/Organization/basic (19.64s)
        --- PASS: TestAccAWSOrganizations/Organization/importBasic (22.88s)
        --- PASS: TestAccAWSOrganizations/Organization/consolidatedBilling (18.46s)
    --- PASS: TestAccAWSOrganizations/Account (0.00s)
        --- SKIP: TestAccAWSOrganizations/Account/basic (0.00s)
            resource_aws_organizations_account_test.go:20: 'TEST_AWS_ORGANIZATION_ACCOUNT_EMAIL_DOMAIN' not set, skipping test.
PASS
ok      github.com/terraform-providers/terraform-provider-aws/aws       61.288s

[bflad]

Hi @bryanlalexander 👋 Thanks so much for submitting this. Its off to a great start. I left a few minor feedback items below -- if you can address those we can get this merged in. Please reach out if you have any questions or do not have time to address them.

[bryanlalexander]

Thanks for taking a look @bflad, got those addressed.

[bflad]

LGTM, thanks @bryanlalexander! 🚀

Output from acceptance testing:

    --- PASS: TestAccAWSOrganizations/Organization (90.91s)
        --- PASS: TestAccAWSOrganizations/Organization/basic (22.76s)
        --- PASS: TestAccAWSOrganizations/Organization/AwsServiceAccessPrincipals (45.54s)
        --- PASS: TestAccAWSOrganizations/Organization/FeatureSet (22.61s)

[bflad]

This has been released in version 2.10.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!