brightbox-cli 3.3.0 → 4.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 881d758fc2932375edc56797f613f757a7f83729087cf2335a44736f4554fe1d
-  data.tar.gz: cfb933331637277089d47a0feccf9456e18b45f0a248aa27f233c52f24ba2f61
+  metadata.gz: fc36c35cb32073368b55abc445d0720ce2e635336e1c3b8af4ef9e1dcedc0000
+  data.tar.gz: '0351935cba15d3c1503c1e8c44d7d4f5258d3d07f67700d363a5b043e58e287e'
 SHA512:
-  metadata.gz: a7b28703cc70a953b900f0caabaf036ff78c1e914810fcba9523b7f8b44ae5c7d148ee09f08d073d71449b587024f7d64ead1f88a659c3a2d2a52c502134d23a
-  data.tar.gz: ea62fcbe6b909e8b7cb8744da7acacd50c75bcb0c109fada221ef135ff1c1ac0d88dc01d8975ef3a8068c75e86956ce244ab88abc641e2dc3d79ed474502e59d
+  metadata.gz: e230034d8c611139a5ed70b1508f1aef3ff3dd263e8f8f53acd58aad72c0616bdcfd5c44ae267e1fa27be66c7958e25e7de986f9169e29c484a47f4b5ceb1e0b
+  data.tar.gz: c4672a3f3efcb30390a6fb536d2542c36988fbb1ba82b5fa59d6dfa1f58868d5518e36ac48f54d9987294b0de46531f0c9b8cb383d21817631a0436008d3e825

data/.gitattributes

@@ -0,0 +1 @@
+spec/cassettes/**/*.yml linguist-generated

data/.github/workflows/ruby.yml

@@ -20,7 +20,7 @@ jobs:
 
     strategy:
       matrix:
-        ruby: ["2.1", "2.2", "2.3", "2.4", "2.5", "2.6", "2.7", "3.0"]
+        ruby: ["2.5", "2.6", "2.7", "3.0", "3.1", "head"]
 
     steps:
     - uses: actions/checkout@v2

data/.gitignore

@@ -16,4 +16,4 @@ spec/brightbox/cache/
 *.html
 *.gem
 /vendor/
-
+spec/examples.txt

data/.rubocop.yml

@@ -0,0 +1,70 @@
+AllCops:
+  NewCops: enable
+  Exclude:
+    - bin/*
+    - lib/brightbox-cli/vendor/**/*
+    - vendor/**/*
+  TargetRubyVersion: 2.5
+
+Layout/HeredocIndentation:
+  Enabled: true
+
+Layout/LineLength:
+  Enabled: false
+  Max: 100
+
+Lint/SuppressedException:
+  Enabled: false
+
+Metrics/AbcSize:
+  Enabled: false
+
+Metrics/BlockLength:
+  Enabled: false
+
+Metrics/ClassLength:
+  Enabled: false
+
+Metrics/CyclomaticComplexity:
+  Enabled: false
+
+Metrics/MethodLength:
+  Enabled: false
+
+Metrics/ModuleLength:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false
+
+Naming/HeredocDelimiterNaming:
+  Enabled: false
+
+Style/Documentation:
+  Enabled: false
+
+Style/Encoding:
+  Enabled: false
+
+Style/FrozenStringLiteralComment:
+  Enabled: false
+
+Style/HashSyntax:
+  Enabled: true
+  EnforcedStyle: no_mixed_keys
+
+Style/GlobalVars:
+  Enabled: false # It's horrid, there's so many
+
+Style/IfUnlessModifier:
+  Enabled: false
+
+Style/SignalException:
+  # Valid values are: semantic, only_raise and only_fail
+  EnforcedStyle: only_raise
+
+Style/SingleLineMethods:
+  Enabled: false
+
+Style/StringLiterals:
+  EnforcedStyle: double_quotes

data/CHANGELOG.md

@@ -1,3 +1,63 @@
+### v4.1.0 / 2022-08-01
+
+[Full Changelog](https://github.com/brightbox/brightbox-cli/compare/v4.0.0...v4.1.0)
+
+Changes:
+
+* Add `volume-size` to `servers create` to allow passing of arbitary sizes for network based storage types.
+
+### v4.0.0 / 2022-08-01
+
+[Full Changelog](https://github.com/brightbox/brightbox-cli/compare/v3.3.0...v4.0.0)
+
+Backwards incompatible changes:
+
+* Drop support for Ruby versions older than 2.5
+* Update `Gemfile.lock` to use Bundler 2.1.4
+* Remove hypenated versions of sub commands. For example `brightbox-servers`
+  wrapping `brightbox servers`.
+
+Changes:
+
+* Improved Two Factor Authentication (2FA) support resulting in it working
+  automatically without configuration prompting when required. The `two_factor`
+  configuration setting is now ignored
+* The `brightbox login` command will now work when 2FA is enabled for a user
+* Adds Ruby 3.1 and 3.2 (preview) to testing matrix
+* Update `fog-brightbox` to `v1.7.0`
+* Removed references to `Fixnum` from code and dependencies to allow
+  testing of Ruby 3.2 support
+* Updates `webmock` from 1.21.0 to 3.14.0
+* Removes `rubyforge` reference from gemspec to remove deprecation warning
+* Remove post install message from gemspec
+* Update `gli` to latest version v2.21.0 to clear some deprecations seen under
+  later Ruby versions
+* Update development gems to ensure up to date
+* Change `client_id` logging level from info to debug
+* Setup Rubocop to help improve code quality/consistency
+* Updated code based on Rubocop reports resulting in many changes
+* Pin `dry-inflector` to v0.2.0 to prevent issues with Ruby =< 2.5
+* Add support for `min-ram` to `brightbox images` commands
+* Update authors (and emails) in gemspec
+
+Bug fixes:
+
+* Implement `Brightbox::Api#respond_to_missing?` to avoid issues when
+  delegating made `#respond_to?` raise an error
+* `brightbox login` no longer attempts (and fails) to download accounts when
+  authentication has failed
+
+Testing:
+
+* Updated testing/development gems
+* Update VCR to latest release v2.9.3
+* Remove VCR identifier sanitisation as temp dev instance are used and
+  squashing IDs cause bug when chain/multiple requests combined
+* RSpec is configured to allow re-runs
+* Timeout testing is dropped to 10s
+* Updated `.dev` TLD used in testing to `.localhost` to avoid problems when the
+  original was commercialised such as local DNS blocked by browsers.
+
 ### v3.3.0 / 2021-09-17
 
 [Full Changelog](https://github.com/brightbox/brightbox-cli/compare/v3.2.0...v3.3.0)

data/Gemfile.lock

@@ -1,30 +1,32 @@
 PATH
   remote: .
   specs:
-    brightbox-cli (3.3.0)
-      dry-inflector (< 0.2)
-      fog-brightbox (>= 1.3.0)
+    brightbox-cli (4.1.0)
+      dry-inflector (= 0.2.0)
+      fog-brightbox (>= 1.8.0)
       fog-core (< 2.0)
-      gli (~> 2.12.0)
-      highline (~> 1.6.0)
+      gli (~> 2.21)
+      highline (~> 1.6)
       hirb (~> 0.6)
-      i18n (~> 0.6.0)
-      mime-types (~> 2.6)
-      multi_json (~> 1.11.0)
+      i18n (>= 0.6, < 1.11)
+      mime-types (~> 3.0)
+      multi_json (~> 1.11)
 
 GEM
   remote: https://rubygems.org/
   specs:
     addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.2)
     builder (3.2.4)
-    coderay (1.0.9)
+    coderay (1.1.3)
+    concurrent-ruby (1.1.10)
     crack (0.4.5)
       rexml
-    diff-lcs (1.4.3)
-    dry-inflector (0.1.2)
-    excon (0.79.0)
-    fog-brightbox (1.4.0)
+    diff-lcs (1.5.0)
+    dry-inflector (0.2.0)
+    excon (0.92.4)
+    fog-brightbox (1.8.0)
       dry-inflector
       fog-core (>= 1.45, < 3.0)
       fog-json
@@ -35,45 +37,69 @@ GEM
     fog-json (1.2.0)
       fog-core
       multi_json (~> 1.10)
-    formatador (0.2.5)
-    gli (2.12.3)
-    highline (1.6.21)
+    formatador (0.3.0)
+    gli (2.21.0)
+    hashdiff (1.0.1)
+    highline (1.7.10)
     hirb (0.7.3)
-    i18n (0.6.11)
-    metaclass (0.0.1)
-    method_source (0.8.1)
-    mime-types (2.99.3)
-    mocha (0.14.0)
-      metaclass (~> 0.0.1)
-    multi_json (1.11.3)
-    pry (0.9.12.2)
-      coderay (~> 1.0.5)
-      method_source (~> 0.8)
-      slop (~> 3.4)
+    i18n (1.10.0)
+      concurrent-ruby (~> 1.0)
+    method_source (1.0.0)
+    mime-types (3.4.1)
+      mime-types-data (~> 3.2015)
+    mime-types-data (3.2022.0105)
+    mocha (1.14.0)
+    multi_json (1.15.0)
+    parallel (1.22.1)
+    parser (3.1.2.0)
+      ast (~> 2.4.1)
+    pry (0.14.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
     pry-remote (0.1.8)
       pry (~> 0.9)
       slop (~> 3.0)
-    public_suffix (2.0.5)
-    rake (12.3.3)
+    public_suffix (4.0.7)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    regexp_parser (2.5.0)
     rexml (3.2.5)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.2)
-      rspec-support (~> 3.9.3)
-    rspec-expectations (3.9.2)
+    rspec (3.11.0)
+      rspec-core (~> 3.11.0)
+      rspec-expectations (~> 3.11.0)
+      rspec-mocks (~> 3.11.0)
+    rspec-core (3.11.0)
+      rspec-support (~> 3.11.0)
+    rspec-expectations (3.11.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.1)
+      rspec-support (~> 3.11.0)
+    rspec-mocks (3.11.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.3)
-    slop (3.4.5)
-    vcr (2.5.0)
-    webmock (1.21.0)
-      addressable (>= 2.3.6)
+      rspec-support (~> 3.11.0)
+    rspec-support (3.11.0)
+    rubocop (1.28.2)
+      parallel (~> 1.10)
+      parser (>= 3.1.0.0)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml
+      rubocop-ast (>= 1.17.0, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.17.0)
+      parser (>= 3.1.1.0)
+    rubocop-rake (0.6.0)
+      rubocop (~> 1.0)
+    rubocop-rspec (2.10.0)
+      rubocop (~> 1.19)
+    ruby-progressbar (1.11.0)
+    slop (3.6.0)
+    unicode-display_width (2.1.0)
+    vcr (2.9.3)
+    webmock (3.14.0)
+      addressable (>= 2.8.0)
       crack (>= 0.3.2)
+      hashdiff (>= 0.4.0, < 2.0.0)
 
 PLATFORMS
   ruby
@@ -84,8 +110,11 @@ DEPENDENCIES
   pry-remote
   rake
   rspec
+  rubocop
+  rubocop-rake
+  rubocop-rspec
   vcr (~> 2.5)
   webmock
 
 BUNDLED WITH
-   1.17.3
+   2.1.4

data/Jenkinsfile

@@ -9,54 +9,34 @@ pipeline {
   stages {
     stage("Run tests") {
       parallel {
-        stage("Ruby 2.0") {
-          agent {
-            docker {
-              image 'ruby:2.0'
-            }
-          }
-          steps {
-            sh 'bundle install --deployment'
-            sh 'bundle exec rake spec'
-          }
-        }
-        stage("Ruby 2.1") {
-          agent {
-            docker {
-              image 'ruby:2.1'
-            }
-          }
-          steps {
-            sh 'bundle install --deployment'
-            sh 'bundle exec rake spec'
-          }
-        }
-        stage("Ruby 2.2") {
+        stage("Ruby 2.5") {
           agent {
             docker {
-              image 'ruby:2.2'
+              image 'ruby:2.5'
             }
           }
           steps {
+            sh 'gem install bundler:2.1.4'
             sh 'bundle install --deployment'
             sh 'bundle exec rake spec'
           }
         }
-        stage("Ruby 2.3") {
+        stage("Ruby 2.6") {
           agent {
             docker {
-              image 'ruby:2.3'
+              image 'ruby:2.6'
             }
           }
           steps {
+            sh 'gem install bundler:2.1.4'
             sh 'bundle install --deployment'
             sh 'bundle exec rake spec'
           }
         }
-        stage("Ruby 2.4") {
+        stage("Ruby 2.7") {
           agent {
             docker {
-              image 'ruby:2.4'
+              image 'ruby:2.7'
             }
           }
           steps {
@@ -64,10 +44,10 @@ pipeline {
             sh 'bundle exec rake spec'
           }
         }
-        stage("Ruby 2.5") {
+        stage("Ruby 3.0") {
           agent {
             docker {
-              image 'ruby:2.5'
+              image 'ruby:3.0'
             }
           }
           steps {
@@ -75,10 +55,10 @@ pipeline {
             sh 'bundle exec rake spec'
           }
         }
-        stage("Ruby 2.6") {
+        stage("Ruby 3.1") {
           agent {
             docker {
-              image 'ruby:2.6'
+              image 'ruby:3.1'
             }
           }
           steps {

data/README

@@ -2,13 +2,9 @@
 
 `brightbox-cli` is a set of cli tools to interact with the Brightbox Cloud API.
 
-You will need a user account on the Brightbox Cloud in order to make use of
-these tools.
-
-You can sign up at https://manage.brightbox.com
-
-{<img src="https://travis-ci.org/brightbox/brightbox-cli.png?branch=master" alt="Build Status" />}[https://travis-ci.org/brightbox/brightbox-cli]
+You will need a Brightbox account in order to make use of these tools.
 
+You can sign up at https://cloud.brightbox.com/signup
 
 == Installation instructions
 
@@ -44,12 +40,12 @@ To browse available resources use the resource name as the command:
 
 === Two factor authentication ===
 
-If you've enabled two factor authentication for your user, you can enable it in
-your config and the CLI will prompt for a two factor pin when needed:
+If you've enabled two factor authentication for your user, you should be
+prompted for a one time password (OTP) when it is required unless you have set
+up a two factor helper (see below).
 
-    [john@example.com]
-    username = john@example.com
-    two_factor = true
+The previous configuration setting `two_factor` is no longer required when
+using v4.0 or greater. This is ignored and can be removed.
 
 === Integrating with a password manager
 
@@ -101,7 +97,6 @@ The alias here is `main`. To prepare the password run this command:
     $ gpg --decrypt ~/.brightbox/main.password.gpg
     password!2015
     $ brightbox accounts
-    INFO: client_id: app-12345 (main)
     INFO: Decrypting /home/user/.brightbox/main.password.gpg to obtain password
     gpg: encrypted with 2048-bit RSA key, ID ABCDE890, created 2015-01-01
           "Jason Null <gpg@example.com>"
@@ -129,11 +124,10 @@ appropriate section:
     password_helper_command = pass john-example-com-brightbox
 
 
-== Usage guides
+== Documentation
 
-* http://docs.brightbox.com/reference/cli
-* http://brightbox.com
-* http://docs.brightbox.com/reference/api/
+* https://www.brightbox.com/docs/guides/cli/
+* http://www.brightbox.com/docs/reference/cli/
 
 == BASH Auto-completion
 
@@ -153,15 +147,21 @@ in the path, just specify the full path to it:
 
 == UPGRADE NOTES
 
-Version 1.0.0 adds a new top level `brightbox` command for the suite.
+Version 4.0.0 removes the older, backward compatible binaries for users of the
+original `brightbox` (brightbox-deployment) gem.
+
+So `brightbox-servers` will no longer work. Use the subcommand variation
+`brightbox servers` instead.
 
-This may clash with our `brightbox` (brightbox-deployment) gem which, before
-version 2.4.0, used `brightbox` as the name for it's binary. Please ensure you
-update your `brightbox` gem to avoid using the wrong command.
+== Alternatives
 
-We have included backwards compatible binaries for all the older CLI commands
-(brightbox-accounts, brightbox-servers etc...) but recommend getting used to the
-new form.
+There are a number of alternative ways to manage Brightbox resources:
+
+* Control Panel - https://cloud.brightbox.com/
+* Terraform – https://registry.terraform.io/providers/brightbox/brightbox/latest
+* Kubernetes - https://github.com/brightbox/brightbox-cloud-controller-manager
+* fog - https://github.com/fog/fog-brightbox
+* Go - https://github.com/brightbox/gobrightbox
 
 == Testing
 
@@ -173,15 +173,6 @@ You should be able to run the specs and features with the following steps:
 The specs use VCR to playback filtered recordings from real API sessions. This
 process is not perfect, please report an issue
 
-== Alternatives
-
-There are a number of alternative ways to interact with the Brightbox Cloud.
-
-* The Web GUI - https://manage.brightbox.com/
-* fog - https://github.com/fog/fog
-* knife plugin - https://github.com/rubiojr/knife-brightbox
-* Vagrant plugin - https://github.com/NeilW/vagrant-brightbox
-
 == Packaging
 
 === Vendoring libraries

data/README.rdoc

@@ -1 +1 @@
-./README
+README

data/brightbox-cli.gemspec

@@ -1,60 +1,47 @@
 # -*- encoding: utf-8 -*-
+
 require File.join(File.dirname(__FILE__), "lib/brightbox-cli/version")
 
 Gem::Specification.new do |s|
   s.name        = "brightbox-cli"
   s.version     = Brightbox::VERSION
   s.platform    = Gem::Platform::RUBY
-  s.authors     = ["John Leach"]
-  s.email       = ["john@brightbox.co.uk"]
-  s.homepage    = "http://docs.brightbox.com/cli"
-  s.summary     = %q(The Brightbox cloud management system)
-  s.description = %q(Scripts to interact with the Brightbox cloud API)
+  s.authors     = ["John Leach", "Paul Thornthwaite"]
+  s.email       = ["john@brightbox.com", "paul@brightbox.com"]
+  s.homepage    = "https://www.brightbox.com/cli"
+  s.summary     = "The Brightbox cloud management system"
+  s.description = "Scripts to interact with the Brightbox cloud API"
   s.license     = "MIT"
+  s.metadata['rubygems_mfa_required'] = 'true'
 
-  s.required_ruby_version = ">= 2.0"
-
-  s.rubyforge_project = "brightbox-cli"
+  s.required_ruby_version = ">= 2.5"
 
   s.files         = `git ls-files`.split("\n") + `find lib/brightbox-cli/vendor`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_dependency "fog-brightbox", ">= 1.3.0"
+  s.add_dependency "fog-brightbox", ">= 1.8.0"
   s.add_dependency "fog-core", "< 2.0"
-  s.add_dependency "gli", "~> 2.12.0"
-  s.add_dependency "i18n", "~> 0.6.0"
-  s.add_dependency "mime-types", "~> 2.6"
-  s.add_dependency "multi_json", "~> 1.11.0"
-  s.add_dependency "highline", "~> 1.6.0"
+  s.add_dependency "gli", "~> 2.21"
+  s.add_dependency "highline", "~> 1.6"
   s.add_dependency "hirb", "~> 0.6"
+  s.add_dependency "i18n", ">= 0.6", "< 1.11"
+  s.add_dependency "mime-types", "~> 3.0"
+  s.add_dependency "multi_json", "~> 1.11"
 
-  # dry-inflector >= 0.2 drops supports for Ruby < 2.4
-  s.add_dependency "dry-inflector", "< 0.2"
+  # Indirect dependency
+  # 0.3 drops support for Ruby < 2.7
+  # 0.2.1 drops support for Ruby < 2.6
+  s.add_dependency "dry-inflector", "= 0.2.0"
 
   s.add_development_dependency "mocha"
   s.add_development_dependency "pry-remote"
   s.add_development_dependency "rake"
   s.add_development_dependency "rspec"
+  s.add_development_dependency "rubocop"
+  s.add_development_dependency "rubocop-rake"
+  s.add_development_dependency "rubocop-rspec"
   s.add_development_dependency "vcr", "~> 2.5"
   s.add_development_dependency "webmock"
-
-  s.post_install_message = <<-EOS
-The CLI command is now `brightbox` with subcommands for the resources.
-Such as `brightbox config` or `brightbox servers`.
-
-  `brightbox --help` lists all the commands.
-
-The old hypenated versions of the commands (e.g. `brightbox-servers`)
-are deprecated and may be removed in the future.
-
-If you have been using the old "brightbox" gem please ensure you have
-the latest version to be able to use the `brightbox-capify` command.
-
-To enable Bash auto-completion you can do the following:
-
-  complete -C _brightbox-bash-completer -o filenames brightbox
-
-EOS
 end

data/lib/brightbox-cli/accounts.rb

@@ -25,21 +25,19 @@ module Brightbox
     end
 
     def self.get(id)
-      if Brightbox.config.using_application?
-        a = conn.accounts.get(id)
-      else
-        a = conn.account
-      end
+      acc = if Brightbox.config.using_application?
+              conn.accounts.get(id)
+            else
+              conn.account
+            end
 
-      if a && a.id == id
-        a
-      else
-        nil
-      end
+      return unless acc && acc.id == id
+
+      acc
     end
 
     def self.default_field_order
-      [:id, :name, :cloud_ips_limit, :lb_limit, :ram_limit, :ram_used, :ram_free]
+      %i[id name cloud_ips_limit lb_limit ram_limit ram_used ram_free]
     end
 
     def to_s