aptible-cli 0.17.0 → 0.18.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e273e3c062fe9d8e365d630b268ea9397e6ea31598595c6ba9729c1d1a288c6d
-  data.tar.gz: 67607c3bcb1ce4c0c782403584f6037a6abe4f8af19544a531ea2ac286d8b1a8
+  metadata.gz: cf958faed8401636f811f03296e187e9af011fef3d6ce6aa3bc7f2aee315d5c8
+  data.tar.gz: 36214902974167a2748685d63fa9d1d659c64d4f752e3b813aeb7adb907f601b
 SHA512:
-  metadata.gz: 8283372d5967ed47fac292aea6dd979ea9be800551ad5312b44b043589317c90f733b043e90088c880c763293230e78c8464eceba32c5dd901cfa5f2b1684e83
-  data.tar.gz: 669627d3ec851644902b3b15e8531c8131977d0023b7304e87549cbc2ef986d7b1007c59477ec2ac147affacf2960409d04d70aef1da31ac9ab245db6467dedd
+  metadata.gz: a1a85e655dd7e0d3267035baca9c8c5898193f524a3ac842961ec62aa0e3b4f951c89a0a1dd5a1adcf80b840b729dafb90238214d6224eff1fd524a54fb90b21
+  data.tar.gz: d85e946cb9adee2314f31078029033bd62ea44a0ef23d673c434e422817c17cf7094bff714ce5b6862c52135f30b030ae7d52a4f72e8052f2fc99b372b3016ca

data/README.md

@@ -48,9 +48,10 @@ Commands:
   aptible db:dump HANDLE [pg_dump options]                                                                                                                    # Dump a remote database to file
   aptible db:execute HANDLE SQL_FILE [--on-error-stop]                                                                                                        # Executes sql against a database
   aptible db:list                                                                                                                                             # List all databases
+  aptible db:modify HANDLE [--iops IOPS] [--volume-type [gp2, gp3]]                                                                                           # Modify a database disk
   aptible db:reload HANDLE                                                                                                                                    # Reload a database
   aptible db:replicate HANDLE REPLICA_HANDLE [--container-size SIZE_MB] [--disk-size SIZE_GB] [--logical --version VERSION] [--key-arn KEY_ARN]               # Create a replica/follower of a database
-  aptible db:restart HANDLE [--container-size SIZE_MB] [--disk-size SIZE_GB]                                                                                  # Restart a database
+  aptible db:restart HANDLE [--container-size SIZE_MB] [--disk-size SIZE_GB][--iops IOPS] [--volume-type [gp2, gp3]]                                          # Restart a database
   aptible db:tunnel HANDLE                                                                                                                                    # Create a local tunnel to a database
   aptible db:url HANDLE                                                                                                                                       # Display a database URL
   aptible db:versions                                                                                                                                         # List available database versions
@@ -72,7 +73,6 @@ Commands:
   aptible login                                                                                                                                               # Log in to Aptible
   aptible logs [--app APP | --database DATABASE]                                                                                                              # Follows logs from a running app or database
   aptible operation:cancel OPERATION_ID                                                                                                                       # Cancel a running operation
-  aptible ps                                                                                                                                                  # Display running processes for an app - DEPRECATED
   aptible rebuild                                                                                                                                             # Rebuild an app, and restart its services
   aptible restart                                                                                                                                             # Restart all services associated with an app
   aptible services                                                                                                                                            # List Services for an App

data/aptible-cli.gemspec

@@ -28,7 +28,11 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'git'
   spec.add_dependency 'term-ansicolor'
   spec.add_dependency 'chronic_duration', '~> 0.10.6'
+
+  # Temporarily pin ffi until https://github.com/ffi/ffi/issues/868 is fixed
+  spec.add_dependency 'ffi', '<= 1.14.1' if Gem.win_platform?
   spec.add_dependency 'win32-process' if Gem.win_platform?
+
   spec.add_dependency 'activesupport', '>= 4.0', '< 6.0'
   spec.add_development_dependency 'bundler', '~> 1.3'
   spec.add_development_dependency 'aptible-tasks', '~> 0.5.8'

data/lib/aptible/cli/agent.rb

@@ -25,7 +25,6 @@ require_relative 'subcommands/db'
 require_relative 'subcommands/domains'
 require_relative 'subcommands/environment'
 require_relative 'subcommands/logs'
-require_relative 'subcommands/ps'
 require_relative 'subcommands/rebuild'
 require_relative 'subcommands/deploy'
 require_relative 'subcommands/restart'
@@ -50,7 +49,6 @@ module Aptible
       include Subcommands::Domains
       include Subcommands::Environment
       include Subcommands::Logs
-      include Subcommands::Ps
       include Subcommands::Rebuild
       include Subcommands::Deploy
       include Subcommands::Restart

data/lib/aptible/cli/resource_formatter.rb

@@ -43,6 +43,7 @@ module Aptible
         def inject_account(node, account)
           node.value('id', account.id)
           node.value('handle', account.handle)
+          node.value('created_at', account.created_at)
         end
 
         def inject_operation(node, operation)
@@ -56,6 +57,7 @@ module Aptible
         def inject_app(node, app, account)
           node.value('id', app.id)
           node.value('handle', app.handle)
+          node.value('created_at', app.created_at)
 
           node.value('status', app.status)
           node.value('git_remote', app.git_repo)
@@ -80,8 +82,10 @@ module Aptible
         def inject_database(node, database, account)
           node.value('id', database.id)
           node.value('handle', database.handle)
+          node.value('created_at', database.created_at)
 
           node.value('type', database.type)
+          node.value('version', database.database_image.version)
           node.value('status', database.status)
 
           node.value('connection_url', database.connection_url)
@@ -92,6 +96,20 @@ module Aptible
             end
           end
           attach_account(node, account)
+
+          if database.disk
+            node.value('disk_type', database.disk.ebs_volume_type)
+            node.value('disk_size', database.disk.size)
+            node.value('disk_modification_progress',
+                       database.disk.modification_progress)
+            node.value('disk_modification_status', database.disk.status)
+            node.value('disk_provisioned_iops', database.disk.baseline_iops)
+          end
+
+          if database.service
+            node.value('container_size', \
+                       database.service.container_memory_limit_mb)
+          end
         end
 
         def inject_credential(node, credential)
@@ -105,6 +123,7 @@ module Aptible
         def inject_service(node, service, app)
           node.value('id', service.id)
           node.value('service', service.process_type)
+          node.value('created_at', service.created_at)
 
           node.value('command', service.command || 'CMD')
           node.value('container_count', service.container_count)
@@ -117,6 +136,7 @@ module Aptible
           node.value('id', vhost.id)
           node.value('hostname', vhost.external_host)
           node.value('status', vhost.status)
+          node.value('created_at', vhost.created_at)
 
           case vhost.type
           when 'tcp', 'tls'

data/lib/aptible/cli/subcommands/db.rb

@@ -279,19 +279,24 @@ module Aptible
             end
 
             desc 'db:restart HANDLE ' \
-                 '[--container-size SIZE_MB] [--disk-size SIZE_GB]',
+                 '[--container-size SIZE_MB] [--disk-size SIZE_GB]' \
+                 '[--iops IOPS] [--volume-type [gp2, gp3]]',
                  'Restart a database'
             option :environment
             option :container_size, type: :numeric
             option :disk_size, type: :numeric
             option :size, type: :numeric
+            option :iops, type: :numeric
+            option :volume_type
             define_method 'db:restart' do |handle|
               database = ensure_database(options.merge(db: handle))
 
               opts = {
                 type: 'restart',
                 container_size: options[:container_size],
-                disk_size: options[:disk_size] || options[:size]
+                disk_size: options[:disk_size] || options[:size],
+                provisioned_iops: options[:iops],
+                ebs_volume_type: options[:volume_type]
               }.delete_if { |_, v| v.nil? }
 
               CLI.logger.warn([
@@ -305,6 +310,26 @@ module Aptible
               attach_to_operation_logs(op)
             end
 
+            desc 'db:modify HANDLE ' \
+                 '[--iops IOPS] [--volume-type [gp2, gp3]]',
+                 'Modify a database disk'
+            option :environment
+            option :iops, type: :numeric
+            option :volume_type
+            define_method 'db:modify' do |handle|
+              database = ensure_database(options.merge(db: handle))
+
+              opts = {
+                type: 'modify',
+                provisioned_iops: options[:iops],
+                ebs_volume_type: options[:volume_type]
+              }.delete_if { |_, v| v.nil? }
+
+              CLI.logger.info "Modifying #{database.handle}..."
+              op = database.create_operation!(opts)
+              attach_to_operation_logs(op)
+            end
+
             desc 'db:url HANDLE', 'Display a database URL'
             option :environment
             option :type, type: :string

data/lib/aptible/cli/version.rb

@@ -1,5 +1,5 @@
 module Aptible
   module CLI
-    VERSION = '0.17.0'.freeze
+    VERSION = '0.18.2'.freeze
   end
 end

data/spec/aptible/cli/resource_formatter_spec.rb

@@ -27,6 +27,7 @@ describe Aptible::CLI::ResourceFormatter do
         'Id: 12',
         'Hostname: foo.io',
         'Status: provisioned',
+        "Created At: #{fmt_time(service.created_at)}",
         'Type: https',
         'Port: default',
         'Internal: false',

data/spec/aptible/cli/subcommands/apps_spec.rb

@@ -104,26 +104,30 @@ describe Aptible::CLI::Agent do
         {
           'environment' => {
             'id' => account.id,
-            'handle' => account.handle
+            'handle' => account.handle,
+            'created_at' => fmt_time(account.created_at)
           },
           'handle' => app.handle,
           'id' => app.id,
           'status' => app.status,
           'git_remote' => app.git_repo,
+          'created_at' => fmt_time(app.created_at),
           'services' => [
             {
               'service' => s1.process_type,
               'id' => s1.id,
               'command' => s1.command,
               'container_count' => s1.container_count,
-              'container_size' => s1.container_memory_limit_mb
+              'container_size' => s1.container_memory_limit_mb,
+              'created_at' => fmt_time(s1.created_at)
             },
             {
               'service' => s2.process_type,
               'id' => s2.id,
               'command' => 'CMD',
               'container_count' => s2.container_count,
-              'container_size' => s2.container_memory_limit_mb
+              'container_size' => s2.container_memory_limit_mb,
+              'created_at' => fmt_time(s2.created_at)
             }
           ]
         }
@@ -145,12 +149,14 @@ describe Aptible::CLI::Agent do
         {
           'environment' => {
             'id' => account.id,
-            'handle' => account.handle
+            'handle' => account.handle,
+            'created_at' => fmt_time(account.created_at)
           },
           'handle' => app.handle,
           'id' => app.id,
           'status' => app.status,
           'git_remote' => app.git_repo,
+          'created_at' => fmt_time(app.created_at),
           'last_deploy_operation' =>
             {
               'id' => op.id,

data/spec/aptible/cli/subcommands/db_spec.rb

@@ -389,6 +389,31 @@ describe Aptible::CLI::Agent do
       expect(captured_logs).to match(/restarting foobar/i)
     end
 
+    it 'allows restarting a database with provisioned iops' do
+      expect(database).to receive(:create_operation!)
+        .with(type: 'restart', provisioned_iops: 3001)
+        .and_return(op)
+
+      expect(subject).to receive(:attach_to_operation_logs).with(op)
+
+      subject.options = { iops: 3001 }
+      subject.send('db:restart', handle)
+
+      expect(captured_logs).to match(/restarting foobar/i)
+    end
+
+    it 'allows restarting a database with ebs volume type' do
+      expect(database).to receive(:create_operation!)
+        .with(type: 'restart', ebs_volume_type: 'gp2').and_return(op)
+
+      expect(subject).to receive(:attach_to_operation_logs).with(op)
+
+      subject.options = { volume_type: 'gp2' }
+      subject.send('db:restart', handle)
+
+      expect(captured_logs).to match(/restarting foobar/i)
+    end
+
     it 'allows restarting a database with (implicitly disk) size' do
       expect(database).to receive(:create_operation!)
         .with(type: 'restart', disk_size: 40).and_return(op)
@@ -419,6 +444,53 @@ describe Aptible::CLI::Agent do
     end
   end
 
+  describe '#db:modify' do
+    before { allow(Aptible::Api::Account).to receive(:all) { [account] } }
+    before { allow(Aptible::Api::Database).to receive(:all) { [database] } }
+
+    let(:op) { Fabricate(:operation) }
+
+    it 'allows modifying a database' do
+      expect(database).to receive(:create_operation!)
+        .with(type: 'modify').and_return(op)
+
+      expect(subject).to receive(:attach_to_operation_logs).with(op)
+
+      subject.send('db:modify', handle)
+
+      expect(captured_logs).to match(/modifying foobar/i)
+    end
+
+    it 'allows modifying a database with provisioned iops' do
+      expect(database).to receive(:create_operation!)
+        .with(type: 'modify', provisioned_iops: 3001).and_return(op)
+
+      expect(subject).to receive(:attach_to_operation_logs).with(op)
+
+      subject.options = { iops: 3001 }
+      subject.send('db:modify', handle)
+
+      expect(captured_logs).to match(/modifying foobar/i)
+    end
+
+    it 'allows modifying a database with ebs volume type' do
+      expect(database).to receive(:create_operation!)
+        .with(type: 'modify', ebs_volume_type: 'gp2').and_return(op)
+
+      expect(subject).to receive(:attach_to_operation_logs).with(op)
+
+      subject.options = { volume_type: 'gp2' }
+      subject.send('db:modify', handle)
+
+      expect(captured_logs).to match(/modifying foobar/i)
+    end
+
+    it 'fails if the DB is not found' do
+      expect { subject.send('db:modify', 'nope') }
+        .to raise_error(Thor::Error, 'Could not find database nope')
+    end
+  end
+
   describe '#db:url' do
     let(:databases) { [database] }
     before { expect(Aptible::Api::Database).to receive(:all) { databases } }

data/spec/aptible/cli/subcommands/environment_spec.rb

@@ -32,11 +32,13 @@ describe Aptible::CLI::Agent do
     expected_accounts = [
       {
         'handle' => 'foo',
-        'ca_body' => 'account 1 cert'
+        'ca_body' => 'account 1 cert',
+        'created_at' => fmt_time(a1.created_at)
       },
       {
         'handle' => 'bar',
-        'ca_body' => '--account 2 cert--'
+        'ca_body' => '--account 2 cert--',
+        'created_at' => fmt_time(a2.created_at)
       }
     ]
     expect(captured_output_json.map! { |account| account.except('id') })

data/spec/fabricators/account_fabricator.rb

@@ -26,4 +26,5 @@ Fabricator(:account, from: :stub_account) do
   apps { [] }
   databases { [] }
   certificates { [] }
+  created_at { Time.now }
 end

data/spec/fabricators/app_fabricator.rb

@@ -29,6 +29,7 @@ Fabricator(:app, from: :stub_app) do
   configurations { [] }
   current_configuration { nil }
   errors { Aptible::Resource::Errors.new }
+  created_at { Time.now }
 
   after_create { |app| app.account.apps << app }
 end

data/spec/fabricators/database_disk_fabricator.rb

@@ -0,0 +1,8 @@
+class StubDatabaseDisk < OpenStruct
+end
+
+Fabricator(:database_disk, from: :stub_database_disk) do
+  size 100
+  ebs_volume_type { 'gp2' }
+  baseline_iops '300'
+end

data/spec/fabricators/database_fabricator.rb

@@ -16,10 +16,13 @@ Fabricator(:database, from: :stub_database) do
   status 'provisioned'
   connection_url 'postgresql://aptible:password@10.252.1.125:49158/db'
   account
+  database_image
+  disk { Fabricate(:database_disk) }
   service { nil }
 
   backups { [] }
   database_credentials { [] }
+  created_at { Time.now }
 
   after_create do |database, transients|
     database.account.databases << database

data/spec/fabricators/service_fabricator.rb

@@ -14,6 +14,7 @@ Fabricator(:service, from: :stub_service) do
   container_count { 1 }
   container_memory_limit_mb { 512 }
   vhosts { [] }
+  created_at { Time.now }
 
   after_create do |service, transients|
     if transients[:app]

data/spec/fabricators/vhost_fabricator.rb

@@ -7,6 +7,7 @@ Fabricator(:vhost, from: :stub_vhost) do
   virtual_domain { Fabricate.sequence(:virtual_domain) { |i| "domain#{i}" } }
   ip_whitelist { [] }
   container_ports { [] }
+  created_at { Time.now }
 
   after_create { |vhost| vhost.service.vhosts << vhost }
 end

data/spec/spec_helper.rb

@@ -11,6 +11,10 @@ end
 # Require library up front
 require 'aptible/cli'
 
+def fmt_time(time)
+  time.strftime('%Y-%m-%d %H:%M:%S %z')
+end
+
 class SpecRenderer < Aptible::CLI::Renderer::Base
   def initialize
     @nodes = []

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aptible-cli
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.18.2
 platform: ruby
 authors:
 - Frank Macreery
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-11 00:00:00.000000000 Z
+date: 2021-06-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aptible-resource
@@ -300,7 +300,6 @@ files:
 - lib/aptible/cli/subcommands/inspect.rb
 - lib/aptible/cli/subcommands/logs.rb
 - lib/aptible/cli/subcommands/operation.rb
-- lib/aptible/cli/subcommands/ps.rb
 - lib/aptible/cli/subcommands/rebuild.rb
 - lib/aptible/cli/subcommands/restart.rb
 - lib/aptible/cli/subcommands/services.rb
@@ -341,6 +340,7 @@ files:
 - spec/fabricators/certificate_fabricator.rb
 - spec/fabricators/configuration_fabricator.rb
 - spec/fabricators/database_credential_fabricator.rb
+- spec/fabricators/database_disk_fabricator.rb
 - spec/fabricators/database_fabricator.rb
 - spec/fabricators/database_image_fabricator.rb
 - spec/fabricators/operation_fabricator.rb
@@ -376,7 +376,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.9
+rubygems_version: 3.0.3
 signing_key:
 specification_version: 4
 summary: Command-line interface for Aptible services
@@ -415,6 +415,7 @@ test_files:
 - spec/fabricators/certificate_fabricator.rb
 - spec/fabricators/configuration_fabricator.rb
 - spec/fabricators/database_credential_fabricator.rb
+- spec/fabricators/database_disk_fabricator.rb
 - spec/fabricators/database_fabricator.rb
 - spec/fabricators/database_image_fabricator.rb
 - spec/fabricators/operation_fabricator.rb

data/lib/aptible/cli/subcommands/ps.rb

@@ -1,30 +0,0 @@
-require 'shellwords'
-
-module Aptible
-  module CLI
-    module Subcommands
-      module Ps
-        def self.included(thor)
-          thor.class_eval do
-            include Helpers::Operation
-            include Helpers::App
-
-            desc 'ps', 'Display running processes for an app - DEPRECATED'
-            app_options
-            def ps
-              deprecated('This command is deprecated on Aptible v2 stacks.')
-
-              app = ensure_app(options)
-
-              op = app.create_operation!(type: 'ps', status: 'succeeded')
-
-              ENV['ACCESS_TOKEN'] = fetch_token
-              opts = ['-o', 'SendEnv=ACCESS_TOKEN']
-              exit_with_ssh_portal(op, *opts)
-            end
-          end
-        end
-      end
-    end
-  end
-end