tfctl 1.6.0 → 1.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7862426153b20908c7f418711505fc0192a1ea61f5edcc5ac1aa71d69190a8a6
-  data.tar.gz: c2d6272043b02a506316143a4ad9f9f61c811609092a2642e64fb3dcfaf09472
+  metadata.gz: 0a21de0b7977bcd56a7055e677dc1689e07b7e62e89d29319fb5e432100f434f
+  data.tar.gz: ac7a0f930a88e683692e5863fcbf2aecb7b0cfe918aa891fe5f6f3df656b901f
 SHA512:
-  metadata.gz: b35eb69b4a65c422e025dc02761c52947824b656d7bdd12eafe88ca214b1e11ca68b27b27edf7b0fb73002a4cdfe80e6c714b46c2957b9548ac26feb910040a5
-  data.tar.gz: 57ea113b1384c4b6ea77af357eb647fbf1362d06bb438b111dca1037479a66d3c9b5d294f1e375fae8e4fe9d53c6df3d093d6d2df40683b86bc21a795ed13a3d
+  metadata.gz: a266d7e164849d1d31be19ffc37572e591c74f4b33fece71de1db510088f3221b6dcf454b5173ffc0cbdae5f2fdc2bf54fc92dab82862cf34cb188fff37953ee
+  data.tar.gz: 6e9f97dbf88dc5a759545b88a3031b9cf2dcb72ab3d5120ba11c9a89c6d15a3fb381d8ed834dac18cd2f4e8f64f0d5862b6eaf57025c1e5541cfe621192f4de5

data/CHANGELOG.adoc

@@ -1,5 +1,9 @@
 = Changelog
 
+== 1.6.1
+
+ * fix: pagination problem when listing accounts in an OU.
+
 == 1.6.0
 
  * fix: pass the default AWS provider explicitly from tfctl generated configuration.

data/README.adoc

@@ -34,32 +34,14 @@ infrastructures where new accounts may be created dynamically and on-demand.
 
 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 hierarchies of nested Organizational Units (OUs),
-and helps keep your Terraform DRY.
+Resources can be assigned globally, based on organization unit (OU) or to individual
+accounts.  It supports hierarchies of nested OUs, and helps keep your Terraform DRY.
 
 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.
@@ -207,3 +189,20 @@ tfctl -a example-account -u -- plan
 This will show output in real time.  Usually output is buffered and displayed
 after the Terraform command finishes, to make it more readable when running
 across multiple accounts in parallel.
+
+== 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.

data/lib/tfctl/aws_org.rb

@@ -25,19 +25,22 @@ module Tfctl
 
                 parent_id = aws_ou_ids[ou_path]
 
-                @aws_org_client.list_accounts_for_parent(parent_id: parent_id).accounts.each do |account|
-                    next unless account.status == 'ACTIVE'
-
-                    output[:accounts] << {
-                        name:       account.name,
-                        id:         account.id,
-                        arn:        account.arn,
-                        email:      account.email,
-                        ou_path:    ou_path.to_s,
-                        ou_parents: ou_path.to_s.split('/'),
-                        profiles:   [],
-                    }
+                @aws_org_client.list_accounts_for_parent(parent_id: parent_id).each do |response|
+                    response.accounts.each do |account|
+                        next unless account.status == 'ACTIVE'
+
+                        output[:accounts] << {
+                            name:       account.name,
+                            id:         account.id,
+                            arn:        account.arn,
+                            email:      account.email,
+                            ou_path:    ou_path.to_s,
+                            ou_parents: ou_path.to_s.split('/'),
+                            profiles:   [],
+                        }
+                    end
                 end
+
             end
             output
         end
@@ -71,8 +74,8 @@ module Tfctl
             @aws_org_client.list_children(
                 child_type: 'ORGANIZATIONAL_UNIT',
                 parent_id:  parent_id,
-            ).each do |resp|
-                resp.children.each do |child|
+            ).each do |response|
+                response.children.each do |child|
 
                     begin
                         ou = @aws_org_client.describe_organizational_unit(

data/lib/tfctl/config.rb

@@ -105,7 +105,7 @@ module Tfctl
 
         def write_cache(cache_file)
             FileUtils.mkdir_p File.dirname(cache_file)
-            File.open(cache_file, 'w') { |f| f.write to_yaml }
+            File.write(cache_file, to_yaml)
         end
 
         def read_cache(cache_file)

data/lib/tfctl/executor.rb

@@ -41,6 +41,12 @@ module Tfctl
             # Create the command
             exec = [cmd] + [subcmd] + args
 
+            runcmd = if Gem.win_platform?
+                         exec.join(' ')
+                     else
+                         exec.shelljoin
+                     end
+
             # Set environment variables for Terraform
             env = {
                 'TF_INPUT'           => '0',
@@ -49,10 +55,10 @@ module Tfctl
                 # 'TF_LOG'             => 'TRACE'
             }
 
-            log.debug "#{account_name}: Executing: #{exec.shelljoin}"
+            log.debug "#{account_name}: Executing: #{runcmd}"
 
             FileUtils.cd path
-            Open3.popen3(env, exec.shelljoin) do |stdin, stdout, stderr, wait_thr|
+            Open3.popen3(env, runcmd) do |stdin, stdout, stderr, wait_thr|
                 stdin.close_write
 
                 # capture stdout and stderr in separate threads to prevent deadlocks

data/lib/tfctl/generator.rb

@@ -9,9 +9,7 @@ module Tfctl
         module_function
 
         def write_json_block(path, block)
-            File.open(path, 'w') do |f|
-                f.write("#{JSON.pretty_generate(block)}\n")
-            end
+            File.write(path, "#{JSON.pretty_generate(block)}\n")
         end
 
         def make(account:, config:)

data/lib/tfctl/version.rb

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

data/tfctl.gemspec

@@ -20,7 +20,6 @@ Gem::Specification.new do |spec|
     end
     spec.bindir        = 'bin'
     spec.executables   = spec.files.grep(%r{^bin/tfctl}) { |f| File.basename(f) }
-    spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
     spec.require_paths = ['lib']
 
     spec.required_ruby_version = '>= 2.5.0'
@@ -36,4 +35,5 @@ Gem::Specification.new do |spec|
     spec.add_development_dependency 'rspec',         '~> 3.9'
     spec.add_development_dependency 'rubocop',       '~> 1.3'
     spec.add_development_dependency 'rubocop-rspec', '~> 2.2'
+    spec.metadata['rubygems_mfa_required'] = 'true'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tfctl
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.6.1
 platform: ruby
 authors:
 - Andrew Wasilczuk
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-08-17 00:00:00.000000000 Z
+date: 2022-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-organizations
@@ -181,7 +181,8 @@ files:
 homepage: https://github.com/scalefactory/tfctl
 licenses:
 - MIT
-metadata: {}
+metadata:
+  rubygems_mfa_required: 'true'
 post_install_message: 
 rdoc_options: []
 require_paths: