tfctl 1.1.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d6f58a8ef8569aaec8cc41ad269adf690caca7d13472a26ed65ed846c835273c
-  data.tar.gz: '013649daccb576adbe4c0c95c8fe64347b641dcb07ad2f0d3e5ca1420188489c'
+  metadata.gz: 3a7ca446d4435cb6780d2b8018e89c8f7b981a247d249ff3fe8083b94edd66ac
+  data.tar.gz: bc7f96ea1cb77fb6366453f4e268e55be5c398e939002f382223c94e09bccbf0
 SHA512:
-  metadata.gz: 2a65943659c0dabe88c40d76d46a99ed23f1ef335486c719729ca599e10e60b2abc0bc81577f2490ca7bbbe64e479ab641b0197a1160b7a1e061f274f224419b
-  data.tar.gz: 8c0cebbab4bc2738357979d1fa013bbf485ea398a3c5ecfdbb4ab9bf670060660a890b900cfca8425764c1387d4ffe5c0d0c50d3a59a10f16ba4f20f3687a303
+  metadata.gz: 777d4fd03430d1d54bcf822f3f4087c7e0c55374d69167487057201ef20132abe5de5c2b949cfa33aa1d48c346a9e0cd819db8193af5a8803dacb7d3988e526e
+  data.tar.gz: 5cc2a820f7930d930efc43c8eab98e9b825e6d1cf48390ca929f01b4136cdbd33509978a31e16f5f4956c458e007bc6dc759e86ee9305b4f14d21c77f267d8ab

data/.bundle/config

@@ -0,0 +1,3 @@
+---
+BUNDLE_PATH: "vendor/bundle"
+BUNDLE_BIN: "vendor/bin"

data/.github/dependabot.yml

@@ -0,0 +1,7 @@
+version: 2
+updates:
+- package-ecosystem: bundler
+  directory: "/"
+  schedule:
+    interval: daily
+  open-pull-requests-limit: 10

data/.github/workflows/linter.yml

@@ -0,0 +1,14 @@
+name: Linting
+
+on: [push]
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 3.0
+      - name: Rubocop
+        run: make rubocop

data/.github/workflows/release.yml

@@ -0,0 +1,37 @@
+name: Release
+
+on:
+  push:
+    tags:
+      - 'v*'
+
+jobs:
+  publish:
+    name: Build + Publish
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Ruby 2.6
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 2.6
+      - name: Build gem
+        run: make pkg
+      - name: Publish to RubyGems
+        run: |
+          mkdir -p $HOME/.gem
+          touch $HOME/.gem/credentials
+          chmod 0600 $HOME/.gem/credentials
+          printf -- "---\n:rubygems_api_key: ${GEM_HOST_API_KEY}\n" > $HOME/.gem/credentials
+          gem push pkg/*.gem
+        env:
+          GEM_HOST_API_KEY: "${{secrets.RUBYGEMS_AUTH_TOKEN}}"
+      - name: Release on GitHub
+        uses: actions/create-release@v1
+        env:
+          GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
+        with:
+          draft: false
+          prerelease: false
+          release_name: 'Release ${{ github.ref }}'
+          tag_name: '${{ github.ref }}'

data/.github/workflows/test.yml

@@ -0,0 +1,23 @@
+name: Tests
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  rspec:
+    runs-on: ${{ matrix.os }}-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu, macos]
+        ruby: [2.6, 2.7, 3.0]
+    continue-on-error: ${{ endsWith(matrix.ruby, 'head') || matrix.ruby == 'debug' }}
+    steps:
+      - uses: actions/checkout@v2
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+      - run: make spec

data/.gitignore

@@ -4,7 +4,6 @@
 pkg/
 *.gem
 vendor/
-.bundle
 bin/bundle
 bin/htmldiff
 bin/ldiff

data/.rubocop.yml

@@ -1,7 +1,8 @@
 ---
 AllCops:
-  TargetRubyVersion: 2.3
+  TargetRubyVersion: 2.5
   DisplayCopNames: true
+  NewCops: enable
 
 Layout/IndentationWidth:
   Width: 4
@@ -45,7 +46,7 @@ Metrics/BlockLength:
 Metrics/MethodLength:
   Enabled: false
 
-Metrics/LineLength:
+Layout/LineLength:
   Max: 140
 
 Metrics/AbcSize:
@@ -77,3 +78,7 @@ Style/TrailingCommaInHashLiteral:
 
 Style/RedundantReturn:
   Enabled: false
+
+# don't break older Rubies just because of style
+Style/RedundantBegin:
+  Enabled: false

data/CHANGELOG.adoc

@@ -1,5 +1,34 @@
 = Changelog
 
+== 1.4.0
+
+ * feat: support yaml anchors and aliases in configuration file.
+
+== 1.3.0
+
+ * feat: support new Terraform provider syntax
+
+BREAKING CHANGE: The minimum supported Terraform version has been bumped to
+0.12.29.  If you are running an older version of Terraform you will need to
+update to the latest Terraform in 0.12.x series before updating tfctl.  Once
+tfctl is updated you can upgrade Terraform to further versions.
+
+== 1.2.2
+ * chore: reverted PR #11 - not necessary and introduced regression.  See PR #13 for details.
+
+== 1.2.1
+ * chore: added required Ruby version to Gemspec.
+
+== 1.2.0
+
+ * feat: pass TF_ environment variables to terraform (PR #11).
+
+== 1.1.1
+
+ * fix: handle empty response from Organizations API containing children (thanks @grothja)
+ * chore: stopped testing on EOL Rubies 2.3 and 2.4 (but should still currently work)
+ * chore: dependencies minimum version bump
+
 == 1.1.0
 
 * feat: look for configuration in `tfctl.yaml` by default.

data/Makefile

@@ -2,7 +2,8 @@
 
 vendor:
 	$(info => Installing Ruby dependencies)
-	@bundle install --path vendor --with developement --binstubs=vendor/bin
+	@bundle install
+	@bundle binstubs --all --path vendor/bin
 
 test: vendor rubocop spec
 
@@ -10,11 +11,11 @@ guard: vendor
 	$(info => Starting guard)
 	@bundle exec guard
 
-rubocop:
+rubocop: vendor
 	$(info => Running rubocop)
 	@vendor/bin/rubocop
 
-spec:
+spec: vendor
 	$(info => Running spec tests)
 	@vendor/bin/rspec
 
@@ -31,6 +32,5 @@ clean:
 	$(info => Cleaning)
 	@rm -rf pkg/
 	@rm -rf vendor/
-	@rm -rf .bundle
 	@rm -f Gemfile.lock
 	@rm -rf spec/reports/

data/README.adoc

@@ -20,7 +20,7 @@ endif::[]
 
 = tfctl
 
-image:https://travis-ci.org/scalefactory/tfctl.svg?branch=master["Build Status", link="https://travis-ci.org/scalefactory/tfctl"]
+image:https://travis-ci.com/scalefactory/tfctl.svg?branch=master["Build Status", link="https://travis-ci.com/scalefactory/tfctl"]
 image:https://badge.fury.io/rb/tfctl.svg["Gem Version", link="https://badge.fury.io/rb/tfctl"]
 image:https://img.shields.io/badge/terraform-0.12-blue.svg["Terraform 0.12", link="https://img.shields.io/badge/terraform-0.12-blue"]
 
@@ -28,19 +28,37 @@ toc::[]
 
 == Overview
 
-Tfctl is a small Terraform wrapper for working with multi-account AWS
+`tfctl` is a small Terraform wrapper for working with multi-account AWS
 infrastructures where new accounts may be created dynamically and on-demand.
 
-It discovers accounts by reading the AWS Organizations API and can assign
+It discovers accounts by reading the AWS Organizations API, and can assign
 Terraform resources to multiple accounts based on the organization hierarchy.
 Resources can be assigned globally, based on organization unit or to individual
-accounts.  It supports nested OU hierarchies and helps keep your Terraform DRY.
+accounts.  It supports hierarchies of nested Organizational Units (OUs),
+and helps keep your Terraform DRY.
 
-Tfctl was originally developed to integrate Terraform with
+The Scale Factory originally created tfctl to integrate Terraform with
 https://aws.amazon.com/solutions/aws-landing-zone/[AWS Landing Zone] and
 https://aws.amazon.com/controltower/[Control Tower] but should work with most
 other ways of managing accounts in AWS Organizations.
 
+== Project status
+
+`tfctl` is an open source project published by The Scale Factory.
+
+We currently consider this project to be maintained but we don't actively
+develop new features.  We keep it security patched and ready for use in
+production environments.
+
+We’ll take a look at any issues or PRs you open and get back to you as soon as
+we can. We don’t offer any formal SLA, but we’ll be checking on this project
+periodically.
+
+If your issue is urgent, you can flag it as such, and we’ll attempt to triage
+appropriately, but we have paying customers who also have demands on our time.
+If your business depends on this project and you have an urgent problem, then
+you can talk to our sales team about paying us to support you.
+
 == Features
 
 * Discovers AWS accounts automatically.
@@ -56,8 +74,8 @@ other ways of managing accounts in AWS Organizations.
 
 == Requirements
 
- * Terraform >= 0.12
- * Ruby >= 2.3
+ * Terraform >= 0.12.29
+ * Ruby >= 2.5
  * Accounts managed in AWS Organizations (by Landing Zone, Control Tower, some
    other means)
 
@@ -65,17 +83,19 @@ other ways of managing accounts in AWS Organizations.
 
 To install the latest release from RubyGems run:
 
+[source,shell]
 ----
 gem install tfctl
 ----
 
-Alternatively you can build and install from this repo with:
+Alternatively, you can build and install from this repo with:
 
+[source,shell]
 ----
 make install
 ----
 
-== Docs
+== Documentation
 
  * https://github.com/scalefactory/tfctl/tree/master/docs/control_tower.adoc[Control Tower quick start guide]
  * https://github.com/scalefactory/tfctl/tree/master/docs/project_layout.adoc[Project layout]
@@ -85,11 +105,12 @@ make install
 
 == Running tfctl
 
-tfctl should be run from the root of the project directory.  It will generate
-Terraform configuration in `.tfctl/`.
+You should run `tfctl` from the root of your project directory.  It will generate
+Terraform configuration in `.tfctl/` (add this to your `.gitignore`).
 
 Anatomy of a tfctl command:
 
+[source,shell]
 ----
 tfctl -c CONFIG_FILE TARGET_OPTIONS -- TERRAFORM_COMMAND
 ----
@@ -102,7 +123,7 @@ tfctl -c CONFIG_FILE TARGET_OPTIONS -- TERRAFORM_COMMAND
   options.  See https://www.terraform.io/docs/commands/index.html[Terraform
   commands] for details.
 
-NOTE: You must have your AWS credentials configured before running tfctl or run
+NOTE: You must have your AWS credentials configured before you run `tfctl`, or run
 it using an AWS credentials helper such as
 https://github.com/99designs/aws-vault[aws-vault].
 
@@ -110,18 +131,21 @@ https://github.com/99designs/aws-vault[aws-vault].
 
 Show help:
 
+[source,shell]
 ----
 tfctl -h
 ----
 
 Show merged configuration:
 
+[source,shell]
 ----
 tfctl -s
 ----
 
 List all discovered accounts:
 
+[source,shell]
 ----
 tfctl --all -l
 ----
@@ -129,49 +153,56 @@ tfctl --all -l
 TIP: This can be narrowed down using targeting options and is a good way to
 test what accounts match.
 
-Run Terraform init across all accounts:
+Run `terraform init` across all accounts:
 
+[source,shell]
 ----
 tfctl --all -- init
 ----
 
-Run plan in `test` OU accounts:
+Plan Terraform across all accounts in the `test` OU:
 
+[source,shell]
 ----
 tfctl -o test -- plan
 ----
 
-Run plan in `live` accounts assuming that `live` is a child OU in multiple
+Plan Terraform in `live` accounts, assuming that `live` is a child OU in multiple
 organization units:
 
+[source,shell]
 ----
 tfctl -o '.*/live' -- plan
 ----
 
-Run plan in an individual account:
+Run a plan for an individual account:
 
+[source,shell]
 ----
 tfctl -a example-account - plan
 ----
 
-Run apply in all accounts:
+Apply Terraform changes across all accounts:
 
+[source,shell]
 ----
 tfctl --all -- apply
 ----
 
-Run destroy in `test` OU accounts:
+Destroy Terraform-managed resources in all the `test` OU accounts:
 
+[source,shell]
 ----
 tfctl -o test -- destroy -auto-approve
 ----
 
 Don't buffer the output:
 
+[source,shell]
 ----
 tfctl -a example-account -u -- plan
 ----
 
 This will show output in real time.  Usually output is buffered and displayed
-after Terraform command finishes to make it more readable when running across
-multiple accounts in parallel.
+after the Terraform command finishes, to make it more readable when running
+across multiple accounts in parallel.

data/RELEASING.adoc

@@ -0,0 +1,13 @@
+= Releasing
+
+This document is aimed at `tfctl` maintainers and describes the process of
+releasing a new gem version.
+
+== Process
+
+* Smoke test in SF test accounts: https://github.com/scalefactory/tfctl-test
+* Bump version in `lib/tfctl/version.rb`
+* Update `CHANGELOG.adoc`
+* Commit
+* Create a new GitHub release and version tag using format: vX.X.X
+* TravisCI will build and release the gem automatically: https://travis-ci.org/github/scalefactory/tfctl

data/bin/tfctl

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-if File.directory?(File.dirname(__FILE__) + '/../vendor')
+if File.directory?("#{File.dirname(__FILE__)}/../vendor")
     require 'bundler/setup'
 end
 require 'optparse'
@@ -81,8 +81,8 @@ begin
     targetting_opts = %i[account ou all]
     targets_set = []
     options.each do |k, v|
-        if targetting_opts.include?(k)
-            targets_set << k.to_s unless v.nil?
+        if targetting_opts.include?(k) and !v.nil?
+            targets_set << k.to_s
         end
     end
     if targets_set.length > 1
@@ -146,7 +146,7 @@ begin
 
     log.info 'Working out AWS account topology'
 
-    yaml_config = YAML.safe_load(File.read(options[:config_file]))
+    yaml_config = YAML.safe_load(File.read(options[:config_file]),  aliases: true)
     Tfctl::Schema.validate(yaml_config)
     yaml_config.symbolize_names!
 

data/docs/configuration.adoc

@@ -24,8 +24,8 @@ toc::[]
 
 == Overview
 
-Tfctl retrieves initial account configuration from AWS Organizations and merges
-it with configuration specified in a yaml file (`tfctl.yaml` by default).
+`tfctl` retrieves initial account configuration from AWS Organizations and merges
+it with configuration specified in YAML format (`tfctl.yaml` by default).
 
 The configuration is merged in the following order:
 
@@ -83,14 +83,16 @@ above.
 
 == Handling secrets
 
-No secrets should be committed into Terraform or tfctl configuration.  Use AWS
-Secrets Manager instead and retrieve in Terraform profiles using
+CAUTION: Do not commit secrets into your Terraform or tfctl configuration.
+
+Instead, use AWS Secrets Manager and retrieve secrets in Terraform profiles using
+the
 https://www.terraform.io/docs/providers/aws/d/secretsmanager_secret.html[secrets
-manager data source]
+manager data source].
 
 == Configuration Schema
 
-Config file is validated using https://json-schema.org/[JSON Schema].
+The configuration file is validated using https://json-schema.org/[JSON Schema].
 
 The schema is defined in
 https://github.com/scalefactory/tfctl/blob/master/lib/tfctl/schema.rb[lib/tfctl/schema.rb]

data/docs/control_tower.adoc

@@ -31,7 +31,7 @@ toc::[]
 For state tracking we're going to create a dedicated `shared-services` account
 under a `mgmt` organization unit.  We'll use S3 for state storage and DynamoDB
 for locking.  `TerraformState` IAM role will be created for cross account
-access to state resources from the primary account.
+access to state resources from the primary AWS account.
 
 In the primary account we'll create a `TfctlOrgAccess` role.  It gives tfctl
 read only access to AWS Organizations which is used to discover accounts and
@@ -50,9 +50,10 @@ and provision a couple of accounts for testing.
 
 Before starting you'll need:
 
- * Control Tower set up in your primary account.
- * A user with `AdministratorAccess` privileges in primary account.
- * AWS CLI tools installed on your machine.
+ * An AWS account that is part of an Organization.
+ * Control Tower set up in your primary AWS account.
+ * A user with `AdministratorAccess` privileges in your primary AWS account.
+ * AWS CLI tools installed on your local machine.
  * Terraform 0.12 or higher.
 
 == Configure Control Tower
@@ -74,6 +75,7 @@ approximately 20 mins to provision one.
 
 == Install tfctl
 
+[source,shell]
 ----
 git clone git@github.com:scalefactory/tfctl.git
 cd tfctl/ && sudo make install
@@ -85,17 +87,21 @@ It's assumed you have configured AWS CLI access to the primary account.
 
 We'll use CloudFormation templates in `examples/bootstrap/`.
 
-First export configuration using environment variables making sure to change to
-values to suit your set up:
+First export configuration using environment variables, making sure to change to
+values to suit your setup:
 
+[source,shell]
 ----
-export PRIMARY_ACCOUNT_ID=11111111
-export SHARED_SERVICES_ACCOUNT_ID=22222222
+# Change these to match your setup
+export PRIMARY_ACCOUNT_ID=123456789012
+export SHARED_SERVICES_ACCOUNT_ID=345678901234
 export STATE_BUCKET_NAME='example-terraform-state'
+export AWS_REGION=eu-west-1
 ----
 
 Create the remote state resources stack set:
 
+[source,shell]
 ----
 cd examples/bootstrap/
 
@@ -107,31 +113,34 @@ aws cloudformation create-stack-set \
     --execution-role-name AWSControlTowerExecution \
     --administration-role-arn arn:aws:iam::${PRIMARY_ACCOUNT_ID}:role/service-role/AWSControlTowerStackSetRole \
     --parameters ParameterKey=PrimaryAccountId,ParameterValue=${PRIMARY_ACCOUNT_ID} \
-    ParameterKey=TerraformStateBucket,ParameterValue=${STATE_BUCKET_NAME}
+    ParameterKey=TerraformStateBucket,ParameterValue="${STATE_BUCKET_NAME}"
 ----
 
-Create a stack set instance in you shared services account:
+Create a stack set instance in your shared services account:
 
+[source,shell]
 ----
 aws cloudformation create-stack-instances \
     --stack-set-name TerraformState \
     --accounts ${SHARED_SERVICES_ACCOUNT_ID} \
-    --regions eu-west-1
+    --regions ${AWS_REGION}
 ----
 
 Check status:
 
+[source,shell]
 ----
 aws cloudformation describe-stack-instance \
     --stack-set-name TerraformState \
     --stack-instance-account ${SHARED_SERVICES_ACCOUNT_ID} \
-    --stack-instance-region eu-west-1
+    --stack-instance-region ${AWS_REGION}
 ----
 
 NOTE: Initial status will be `OUTDATED`, it should change to `CURRENT` once deployed.
 
-Deploy `TfctlOrgAccess` IAM role stack:
+Deploy the `TfctlOrgAccess` IAM role stack:
 
+[source,shell]
 ----
 aws cloudformation create-stack \
     --stack-name TfctlOrgAccess \
@@ -142,6 +151,7 @@ aws cloudformation create-stack \
 
 Check status:
 
+[source,shell]
 ----
 aws cloudformation describe-stacks --stack-name TfctlOrgAccess
 ----
@@ -158,7 +168,7 @@ You need to modify the following parameters:
  * `tf_state_bucket` - set to `$STATE_BUCKET_NAME`
  * `tf_state_role_arn` - set shared services account ID
  * `tfctl_role_arn` - set primary account ID
- * `primary_account` - set the primary account name.  You can find it in AWS Organizations.
+ * `primary_account` - set the primary account name.  You can find it through AWS Organizations in the console.
 
 TIP: You should keep your project directory under version control.
 
@@ -169,43 +179,51 @@ and `mgmt` OUs.
 
 NOTE: Run tfctl commands from the root of you project directory.
 
-First dump the configuration to verify everything works:
+First, dump the configuration to verify everything works:
 
+[source,shell]
 ----
 tfctl -s
 ----
 
-This will not make any changes but will print out a yaml containing the final,
+This will not make any changes but will print out YAML containing the final,
 merged configuration data.  It should contain a list of discovered accounts and
 their configuration.
 
-Initialise terraform for all discovered accounts:
+Initialise Terraform for all discovered accounts:
 
+[source,shell]
 ----
 tfctl --all -- init
 ----
 
 Tfctl will run Terraform against all accounts in parallel.
 
-Run plan:
+`plan` the Terraform:
 
+[source,shell]
 ----
 tfctl --all -- plan
 ----
 
-and apply:
+and `apply` it:
 
+[source,shell]
 ----
 tfctl --all -- apply
 ----
 
-To destroy created resources run:
 
+== Clean up
+
+To destroy created resources, run:
+
+[source,shell]
 ----
 tfctl --all -- destroy -auto-approve
 ----
 
-That's it! You can now execute terraform across your Control Tower estate.
+That's it! You can now execute Terraform across your Control Tower estate.
 
 TIP: Your project directory should be under version control excluding the
 `.tfctl` directory which is automatically generated.

data/docs/iam_permissions.adoc

@@ -20,7 +20,7 @@ endif::[]
 
 = IAM roles
 
-Tfctl usually requires three IAM roles to be configured:
+`tfctl` usually requires three IAM roles to be configured:
 
  * `TfctlRole` - read only access to AWS Organizations set up in the primary account.
  * `TerraformStateRole` - access to remote state resources (S3, DynamoDB) in the
@@ -28,7 +28,7 @@ Tfctl usually requires three IAM roles to be configured:
  * `TerraformExecutionRole` - configured in all spoke accounts and used for executing Terraform.
 
 The user executing tfctl needs permission to assume all three roles cross
-account.  Tfctl will assume roles automatically for you.
+account. Once these are configured, tfctl automatically assumes roles for you.
 
 It's possible to configure different Terraform execution roles in different
 spoke accounts based on OU or account names.  This can be used to restrict

data/docs/project_layout.adoc

@@ -50,12 +50,12 @@ The configuration data is exposed to terraform via a profile `config` variable.
 It also defines Terraform and tfctl configuration such as state tracking and
 what IAM roles to use.
 
-By default tfctl will use `tfctl.yaml` in it's current working directory.  You
+By default, tfctl will use `tfctl.yaml` in its current working directory.  You
 can specify a different file using `-c`.  Multiple configurations are supported
 in the same project directory and generated data will be stored separately for
 each config file in `.tfctl/`.
 
-== profiles
+== `profiles`
 
 Profiles are re-usable collections of resources which can be applied to
 accounts.  They are implemented just like usual modules but provide an
@@ -64,6 +64,6 @@ other data sources).  Profiles often compose multiple modules and provide
 configuration data to them.  This approach makes it possible to re-use standard
 modules (e.g. Terraform module registry).
 
-== modules
+== `modules`
 
 Standard Terraform modules.

data/examples/control_tower/profiles/example-profile/main.tf

@@ -1,4 +1,4 @@
 module "bucket" {
   source = "../../modules/s3-bucket"
-  name   = "${local.account_id}-${local.account["example_bucket_name"]}"
+  name   = "${local.account_id}-${local.account["data"]["example_bucket_name"]}"
 }

data/examples/control_tower/profiles/example-profile/variables.tf

@@ -7,6 +7,6 @@ variable "config" {
 locals {
   config     = jsondecode(var.config)
   account_id = "${data.aws_caller_identity.current.account_id}"
-  # get current account configuration from tfctl config
+  # get account configuration from tfctl config
   account    = [ for account in local.config["accounts"]: account if account["id"] == local.account_id ][0]
 }

data/examples/control_tower/tfctl.yaml

@@ -5,7 +5,7 @@
 # create final configuration used by tfctl.  You can view the merged
 # configuration by running:
 #
-#   tfctl -c conf/example.yaml -s
+#   tfctl -c conf/tfctl.yaml -s
 #
 
 #
@@ -17,7 +17,7 @@ tf_state_dynamodb_table: 'terraform-lock'
 tf_state_region: 'eu-west-1'
 # Role for accessing state resources
 tf_state_role_arn: 'arn:aws:iam::SHARED_SERVICES_ACCOUNT_ID:role/TerraformStateRole'
-tf_required_version: '>= 0.12.0'
+tf_required_version: '>= 0.12.29'
 aws_provider_version: '>= 2.14'
 # Role used by tfctl to retrieve data from AWS Organizations
 # Has to be set up in the primary org account

data/lib/hash.rb

@@ -18,13 +18,14 @@ class Hash
         merge(second.to_h, &merger)
     end
 
-    # Copied from ruby 2.6 Psych for 2.3 compatibility.
     def symbolize_names!(result = self)
         case result
         when Hash
+            # rubocop:disable Style/HashEachMethods
             result.keys.each do |key|
                 result[key.to_sym] = symbolize_names!(result.delete(key))
             end
+            # rubocop:enable Style/HashEachMethods
         when Array
             result.map! { |r| symbolize_names!(r) }
         end

data/lib/tfctl.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require_relative 'tfctl/aws_org.rb'
-require_relative 'tfctl/config.rb'
-require_relative 'tfctl/error.rb'
-require_relative 'tfctl/executor.rb'
-require_relative 'tfctl/generator.rb'
-require_relative 'tfctl/logger.rb'
-require_relative 'tfctl/schema.rb'
-require_relative 'tfctl/version.rb'
+require_relative 'tfctl/aws_org'
+require_relative 'tfctl/config'
+require_relative 'tfctl/error'
+require_relative 'tfctl/executor'
+require_relative 'tfctl/generator'
+require_relative 'tfctl/logger'
+require_relative 'tfctl/schema'
+require_relative 'tfctl/version'

data/lib/tfctl/aws_org.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'error.rb'
+require_relative 'error'
 require 'aws-sdk-organizations'
 
 module Tfctl
@@ -71,23 +71,25 @@ module Tfctl
             @aws_org_client.list_children(
                 child_type: 'ORGANIZATIONAL_UNIT',
                 parent_id:  parent_id,
-            ).children.each do |child|
-
-                begin
-                    ou = @aws_org_client.describe_organizational_unit(
-                        organizational_unit_id: child.id,
-                    ).organizational_unit
-                rescue Aws::Organizations::Errors::TooManyRequestsException
-                    # FIXME: - use logger
-                    puts 'AWS Organizations: too many requests.  Retrying in 5 secs.'
-                    sleep 5
-                    retries += 1
-                    retry if retries < 10
-                end
+            ).each do |resp|
+                resp.children.each do |child|
+
+                    begin
+                        ou = @aws_org_client.describe_organizational_unit(
+                            organizational_unit_id: child.id,
+                        ).organizational_unit
+                    rescue Aws::Organizations::Errors::TooManyRequestsException
+                        # FIXME: - use logger
+                        puts 'AWS Organizations: too many requests.  Retrying in 5 secs.'
+                        sleep 5
+                        retries += 1
+                        retry if retries < 10
+                    end
 
-                ou_name = parent_name == :root ? ou.name.to_sym : "#{parent_name}/#{ou.name}".to_sym
+                    ou_name = parent_name == :root ? ou.name.to_sym : "#{parent_name}/#{ou.name}".to_sym
 
-                output[ou_name] = ou.id
+                    output[ou_name] = ou.id
+                end
             end
             output
         end

data/lib/tfctl/config.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative '../hash.rb'
-require_relative 'error.rb'
+require_relative '../hash'
+require_relative 'error'
 require 'yaml'
 require 'json'
 
@@ -121,7 +121,9 @@ module Tfctl
             return config unless config.key?(:exclude_accounts)
 
             config[:accounts].each_with_index do |account, idx|
+                # rubocop:disable Style/IfWithBooleanLiteralBranches
                 config[:accounts][idx][:excluded] = config[:exclude_accounts].include?(account[:name]) ? true : false
+                # rubocop:enable Style/IfWithBooleanLiteralBranches
             end
 
             config

data/lib/tfctl/executor.rb

@@ -3,7 +3,7 @@
 require 'open3'
 require 'fileutils'
 require 'shellwords'
-require_relative 'error.rb'
+require_relative 'error'
 
 module Tfctl
     module Executor

data/lib/tfctl/generator.rb

@@ -10,21 +10,27 @@ module Tfctl
 
         def write_json_block(path, block)
             File.open(path, 'w') do |f|
-                f.write(JSON.pretty_generate(block) + "\n")
+                f.write("#{JSON.pretty_generate(block)}\n")
             end
         end
 
         def make(account:, config:)
             target_dir = "#{PROJECT_ROOT}/.tfctl/#{config[:config_name]}/#{account[:name]}"
-            tf_version = config.fetch(:tf_required_version, '>= 0.12.0')
+            tf_version = config.fetch(:tf_required_version, '>= 0.12.29')
             aws_provider_version = config.fetch(:aws_provider_version, '>= 2.14')
 
             FileUtils.mkdir_p target_dir
 
             terraform_block = {
                 'terraform' => {
-                    'required_version' => tf_version,
-                    'backend'          => {
+                    'required_version'   => tf_version,
+                    'required_providers' => {
+                        'aws' => {
+                            'source'  => 'hashicorp/aws',
+                            'version' => aws_provider_version,
+                        },
+                    },
+                    'backend'            => {
                         's3' => {
                             'bucket'         => config[:tf_state_bucket],
                             'key'            => "#{account[:name]}/tfstate",
@@ -41,7 +47,6 @@ module Tfctl
             provider_block = {
                 'provider' => {
                     'aws' => {
-                        'version'     => aws_provider_version,
                         'region'      => account[:region],
                         'assume_role' => {
                             'role_arn' => "arn:aws:iam::#{account[:id]}:role/#{account[:tf_execution_role]}",
@@ -71,11 +76,8 @@ module Tfctl
                 profile_block = {
                     'module' => {
                         profile => {
-                            'source'    => "../../../profiles/#{profile}",
-                            'config'    => '${var.config}',
-                            'providers' => {
-                                'aws' => 'aws',
-                            },
+                            'source' => "../../../profiles/#{profile}",
+                            'config' => '${var.config}',
                         },
                     },
                 }

data/lib/tfctl/logger.rb

@@ -6,7 +6,7 @@ module Tfctl
     class Logger
 
         def initialize(log_level)
-            @outlog  = ::Logger.new(STDOUT)
+            @outlog  = ::Logger.new($stdout)
 
             self.level = log_level
 

data/lib/tfctl/schema.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require 'json_schemer'
-require_relative 'error.rb'
+require_relative 'error'
 
 # Config validator using JSON schema
 

data/lib/tfctl/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Tfctl
-    VERSION = '1.1.0'
+    VERSION = '1.4.0'
 end

data/tfctl.gemspec

@@ -23,14 +23,17 @@ Gem::Specification.new do |spec|
     spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
     spec.require_paths = ['lib']
 
+    spec.required_ruby_version = '>= 2.5.0'
+
     # Think when adding new dependencies.  Is it really necessary?
     # "The things you own end up owning you" etc.
-    spec.add_dependency 'aws-sdk-organizations', '~> 1.13'
+    spec.add_dependency 'aws-sdk-organizations', '~> 1.40'
     spec.add_dependency 'json_schemer',          '~> 0.2'
-    spec.add_dependency 'parallel',              '~> 1.17'
-    spec.add_dependency 'terminal-table',        '~> 1.8'
+    spec.add_dependency 'parallel',              '~> 1.19'
+    spec.add_dependency 'terminal-table',        '>= 1.8', '< 4.0'
 
-    spec.add_development_dependency 'guard-rspec', '~> 4.7'
-    spec.add_development_dependency 'rspec',       '~> 3.8'
-    spec.add_development_dependency 'rubocop',     '~> 0.76'
+    spec.add_development_dependency 'guard-rspec',   '~> 4.7'
+    spec.add_development_dependency 'rspec',         '~> 3.9'
+    spec.add_development_dependency 'rubocop',       '~> 1.3'
+    spec.add_development_dependency 'rubocop-rspec', '~> 2.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tfctl
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Andrew Wasilczuk
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-15 00:00:00.000000000 Z
+date: 2021-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-organizations
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.13'
+        version: '1.40'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.13'
+        version: '1.40'
 - !ruby/object:Gem::Dependency
   name: json_schemer
   requirement: !ruby/object:Gem::Requirement
@@ -44,28 +44,34 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.17'
+        version: '1.19'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.17'
+        version: '1.19'
 - !ruby/object:Gem::Dependency
   name: terminal-table
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.8'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.8'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
@@ -86,28 +92,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '3.9'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.76'
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.76'
+        version: '2.2'
 description: 
 email:
 - akw@scalefactory.com
@@ -116,16 +136,21 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".bundle/config"
+- ".github/dependabot.yml"
+- ".github/workflows/linter.yml"
+- ".github/workflows/release.yml"
+- ".github/workflows/test.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
-- ".travis.yml"
 - CHANGELOG.adoc
 - Gemfile
 - Guardfile
 - LICENSE
 - Makefile
 - README.adoc
+- RELEASING.adoc
 - bin/tfctl
 - docs/configuration.adoc
 - docs/control_tower.adoc
@@ -164,15 +189,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.7
+rubygems_version: 3.0.3.1
 signing_key: 
 specification_version: 4
 summary: Terraform wrapper for managing multi-account AWS infrastructures

data/.travis.yml

@@ -1,18 +0,0 @@
-rvm:
-- 2.3
-- 2.6
-os: linux
-language: ruby
-script: make test
-jobs:
-  include:
-  - stage: Gem release
-    rvm: 2.6
-    deploy:
-      provider: rubygems
-      api_key:
-        secure: LAVcdER+LtQ2TSUrVOY7Be1BC7GXJRD0QBt386vRM5Nld5QaD9Ow9gtN6FprzkzloI4R8BkPWqZbAT6YjC+C0AFB5HK6iPwD2bLsiF9w3ccDD+yrW99RHxiErpmYMun2PqZv0WkJ/pkEplPCKMRFv7SM7W9DMRlU7dsXc1v6IVyIb5u3A04jErS2jXXKY0ijlCDJYVo8zzYL6yUmUcXhc//3CIVnu2Miu6Qr8h7e6jMXNUWfMkwEXsFP9id4TsCz7hRY+39PkiBAknHTN5UqjjJiEOknZnHeTBcVPvi2h2xv+fFLSzVTxlxaRsVoMCShQp5D12qzhQObRJsRVQFs8Yyg9IYMyPdxssFYyUZFaAy5taWDm57uM3HTHylm/Dq3LmXTgGNxWUUkf2oh1g7R6cYZpBUQwiEPzhZQ7CoBQbGUAJmH9ZU9m+cr8kuAOUipd6BNEDvn/fIH4WJsRCNP72JGX16JBpuICvpkuNhskZT91xFlYk1pTXHOxNpbTcxUTgMHhrTqspeRXPmf6DiYGvjMb2S6kaoGqCIRIcwl0TGKuMsOMqR9SqF8gubkqHMVbSl1E7mwBn4ke8/7IGoMkWOGwUpVxqVOLBHi6zSR09RTVSbKl4oiFV3ZwmVPSxDncq54MptyJ2WCZ7dD6ht2l+VA8iGwYeIoqOpwGWxyuNI=
-      gem: tfctl
-      on:
-        tags: true
-        repo: scalefactory/tfctl