kuby-core 0.11.16 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 01cc5428a8bc4bfc44143f4aa2c2e949405d3c7231831da50f33efa5a314fc29
-  data.tar.gz: c4e440506c9ad072449f3382bcfa7dd1f80c757e64426b5c49f1e20540c57ae8
+  metadata.gz: 4a072588afc0cc4e0cf815c276628b90c192bf24310ed6c4c53089bc467ba278
+  data.tar.gz: 364b1265256eda0eea1f57469f241a1c78f1bdb2be0d37d8a23b5c6f01f4ff43
 SHA512:
-  metadata.gz: 5db95c1958618d58354ba4c87411a0cd52894ede6ba0fbd622479e66e327397967314979c7496806577115d6b22571582e17f76c720a8472c9568cb009682555
-  data.tar.gz: 2c1c1bd58fcbc7d29380e5e2ef2cf71b68f7073dbff26ca5d73333734b326dccc42243a559b3272502b4a6628db0c7513ea9b6586b5475e44108ba4a490734f5
+  metadata.gz: 866438d8c0880d949150d337b01fce94dfab9d0606ab3e2358e4c7cad376bb36cb3ff958ea72188967abf1192216c3b7586ddcfba6d7e0324ab4fbe37a085e49
+  data.tar.gz: 87e438bd3f4783b1b3d0e7609d773f5eeee91f7b9ae980446c8ca63084bddeb3a525d944750fe13fb36e98d782188cf6bd06c3df5edff742d8a1d5d80aa53966

data/CHANGELOG.md

@@ -1,3 +1,21 @@
+## 0.12.0
+* Fix issue causing volume mount errors when k8s tries to schedule asset pods on multiple nodes (fixes #42).
+  - Persistent volumes can only be mounted on a single physical node.
+  - Kuby now creates a separate assets image powered by nginx.
+  - Ingress routes requests to either the Rails service or assets service depending on the URL (i.e. /assets).
+  - Since the ingress layer is now essential, the Docker Desktop provider no longer deletes ingresses or monkeys with service objects. You'll have to either 1) add an entry for your hostname to your hosts file, or 2) set the `Host` header when making requests to your app.
+* Plugins can now specify additional Dockerfiles to build during `kuby build`.
+* Switch from TravisCI to Github Actions.
+* Remove support for development environments.
+  - Proved to be too difficult to maintain.
+  - Kuby is a _deployment_ tool anyway, not a dev tool.
+  - Maybe we can turn the dev stuff into a gem at some point.
+* Refactor Docker image logic.
+  - Introduced the `Image` class and friends.
+  - Deleted the `Kuby::Docker::Metadata` class.
+* Add missing git dependency to Alpine distro.
+* Add a bare metal provider (fixes #10).
+
 ## 0.11.16
 * Fix yarn phase
   - Apparently you have to copy at least one file, TIL.

data/Gemfile

@@ -6,10 +6,9 @@ group :development, :test do
   gem 'pry-byebug'
   gem 'rake'
   # lock to a specific version to prevent breaking CI when new versions come out
-  gem 'sorbet', '= 0.5.6397'
+  gem 'sorbet', '= 0.5.6433'
 end
 
 group :test do
   gem 'rspec', '~> 3.0'
-  gem 'timecop', '~> 0.9'
 end

data/kuby-core.gemspec

@@ -13,7 +13,7 @@ Gem::Specification.new do |s|
   s.platform = Gem::Platform::RUBY
 
   s.add_dependency 'colorize', '~> 0.8'
-  s.add_dependency 'docker-remote', '~> 0.5'
+  s.add_dependency 'docker-remote', '~> 0.6'
   s.add_dependency 'gli', '~> 2.0'
   s.add_dependency 'helm-cli', '~> 0.3'
   # See: https://github.com/Shopify/krane/pull/720

data/lib/kuby.rb

@@ -13,7 +13,6 @@ module Kuby
   autoload :CLIBase,        'kuby/cli_base'
   autoload :Commands,       'kuby/commands'
   autoload :Definition,     'kuby/definition'
-  autoload :DevSetup,       'kuby/dev_setup'
   autoload :Docker,         'kuby/docker'
   autoload :Environment,    'kuby/environment'
   autoload :Kubernetes,     'kuby/kubernetes'
@@ -21,7 +20,6 @@ module Kuby
   autoload :Plugin,         'kuby/plugin'
   autoload :PluginRegistry, 'kuby/plugin_registry'
   autoload :Plugins,        'kuby/plugins'
-  autoload :RailsCommands,  'kuby/rails_commands'
   autoload :Tasks,          'kuby/tasks'
   autoload :TrailingHash,   'kuby/trailing_hash'
 
@@ -52,24 +50,6 @@ module Kuby
       @definition = Definition.new(name.to_s)
       @definition.instance_eval(&block)
 
-      # default development environment
-      @definition.environment(:development) do
-        kubernetes do
-          add_plugin(:rails_app) do
-            tls_enabled false
-
-            database do
-              if requires_credentials?
-                user(DEFAULT_DB_USER)
-                password(DEFAULT_DB_PASSWORD)
-              end
-            end
-          end
-
-          provider :docker_desktop
-        end
-      end
-
       @definition.environments.each do |_, env|
         env.kubernetes.after_configuration
       end
@@ -174,3 +154,5 @@ Kuby.register_package(:c_toolchain,
   debian: 'build-essential',
   alpine: 'build-base'
 )
+
+Kuby.register_package(:git, 'git')

data/lib/kuby/commands.rb

@@ -38,13 +38,6 @@ module Kuby
       Kuby::Tasks.new(Kuby.environment)
     end
 
-    sig { void }
-    def self.must_be_dev_env!
-      unless Kuby.environment.development?
-        fail "Command not supported in the '#{Kuby.environment.name}' environment"
-      end
-    end
-
     program_desc 'Kuby command-line interface. Kuby is a convention '\
       'over configuration approach for running Rails apps in Kubernetes.'
 
@@ -68,59 +61,19 @@ module Kuby
       true
     end
 
-    # These are only stubs included to fill out the help screens. Rails
-    # commands are handled by the RailsCommands class.
-    desc 'Runs a Rails command.'
-    command :rails do |rc|
-      rc.action do |global_options, options, args|
-        must_be_dev_env!
-        exit 1 unless tasks.dev_deployment_ok
-        @rails_options = T.let(@rails_options, T.nilable(T::Array[String]))
-        Kuby::RailsCommands.run(@rails_options)
-      end
-
-      rc.desc 'Runs the rails server (run `rails server --help` for options)'
-      rc.command [:server, :s] do |c|
-        c.action do |global_options, options, args|
-          must_be_dev_env!
-          exit 1 unless tasks.dev_deployment_ok
-          Kuby::RailsCommands.run(@rails_options)
-        end
-      end
-
-      rc.desc 'Runs a script in the Rails environment (run `rails runner --help` for options)'
-      rc.command [:runner, :r] do |c|
-        c.action do |global_options, options, args|
-          must_be_dev_env!
-          exit 1 unless tasks.dev_deployment_ok
-          Kuby::RailsCommands.run(@rails_options)
-        end
-      end
-
-      rc.desc 'Starts an interactive Ruby console with the Rails environment loaded '\
-        '(run `rails console --help` for options)'
-      rc.command [:console, :c] do |c|
-        c.action do |global_options, options, args|
-          must_be_dev_env!
-          exit 1 unless tasks.dev_deployment_ok
-          Kuby::RailsCommands.run(@rails_options)
-        end
-      end
-    end
-
-    desc 'Runs a rake task.'
-    command :rake do |rc|
-      rc.action do |global_options, options, args|
-        must_be_dev_env!
-        exit 1 unless tasks.dev_deployment_ok
-        Kuby::RailsCommands.run(@rails_options)
-      end
-    end
-
     desc 'Builds the Docker image.'
     command :build do |c|
+      c.flag [:a, :arg], required: false, multiple: true
       c.action do |global_options, options, args|
-        tasks.build
+        build_args = {}.tap do |build_args|
+          (options[:arg] || []).each do |a|
+            key, value = a.split('=')
+            value = value[1..-2] if value.start_with?('"') || value.start_with?("'")
+            build_args[key] = value
+          end
+        end
+
+        tasks.build(build_args)
       end
     end
 
@@ -138,10 +91,10 @@ module Kuby
       end
     end
 
-    desc 'Prints the effective Dockerfile used to build the Docker image.'
-    command :dockerfile do |c|
+    desc 'Prints the effective Dockerfiles used to build Docker images.'
+    command :dockerfiles do |c|
       c.action do |global_options, options, args|
-        tasks.print_dockerfile
+        tasks.print_dockerfiles
       end
     end
 
@@ -197,7 +150,6 @@ module Kuby
       rc.desc 'Tails (i.e. continuously streams) the Rails log from your running application.'
       rc.command :logs do |c|
         c.action do |global_options, options, args|
-          exit 1 unless tasks.dev_deployment_ok
           tasks.remote_logs
         end
       end
@@ -205,7 +157,6 @@ module Kuby
       rc.desc 'Lists running Kubernetes pods.'
       rc.command :status do |c|
         c.action do |global_options, options, args|
-          exit 1 unless tasks.dev_deployment_ok
           tasks.remote_status
         end
       end
@@ -213,7 +164,6 @@ module Kuby
       rc.desc 'Runs an arbitrary command inside a running Rails pod.'
       rc.command :exec do |c|
         c.action do |global_options, options, args|
-          exit 1 unless tasks.dev_deployment_ok
           tasks.remote_exec(args)
         end
       end
@@ -221,7 +171,6 @@ module Kuby
       rc.desc 'Establishes a shell inside a running Rails pod.'
       rc.command :shell do |c|
         c.action do |global_options, options, args|
-          exit 1 unless tasks.dev_deployment_ok
           tasks.remote_shell
         end
       end
@@ -229,7 +178,6 @@ module Kuby
       rc.desc 'Establishes a Rails console inside a running Rails pod.'
       rc.command :console do |c|
         c.action do |global_options, options, args|
-          exit 1 unless tasks.dev_deployment_ok
           tasks.remote_console
         end
       end
@@ -237,7 +185,6 @@ module Kuby
       rc.desc 'Establishes a database console inside a running Rails pod.'
       rc.command :dbconsole do |c|
         c.action do |global_options, options, args|
-          exit 1 unless tasks.dev_deployment_ok
           tasks.remote_dbconsole
         end
       end
@@ -245,7 +192,6 @@ module Kuby
       rc.desc "Restarts the Rails app's web pods."
       rc.command :restart do |c|
         c.action do |global_options, options, args|
-          exit 1 unless tasks.dev_deployment_ok
           tasks.remote_restart
         end
       end

data/lib/kuby/docker.rb

@@ -4,30 +4,32 @@ require 'kuby/docker/errors'
 
 module Kuby
   module Docker
-    autoload :Alpine,         'kuby/docker/alpine'
-    autoload :AssetsPhase,    'kuby/docker/assets_phase'
-    autoload :BundlerPhase,   'kuby/docker/bundler_phase'
-    autoload :CLI,            'kuby/docker/cli'
-    autoload :CopyPhase,      'kuby/docker/copy_phase'
-    autoload :Credentials,    'kuby/docker/credentials'
-    autoload :Debian,         'kuby/docker/debian'
-    autoload :DevSpec,        'kuby/docker/dev_spec'
-    autoload :Distro,         'kuby/docker/distro'
-    autoload :Dockerfile,     'kuby/docker/dockerfile'
-    autoload :DockerURI,      'kuby/docker/docker_uri'
-    autoload :InlineLayer,    'kuby/docker/inline_layer'
-    autoload :Layer,          'kuby/docker/layer'
-    autoload :LayerStack,     'kuby/docker/layer_stack'
-    autoload :LocalTags,      'kuby/docker/local_tags'
-    autoload :Metadata,       'kuby/docker/metadata'
-    autoload :Packages,       'kuby/docker/packages'
-    autoload :PackagePhase,   'kuby/docker/package_phase'
-    autoload :RemoteTags,     'kuby/docker/remote_tags'
-    autoload :SetupPhase,     'kuby/docker/setup_phase'
-    autoload :Spec,           'kuby/docker/spec'
-    autoload :Tags,           'kuby/docker/tags'
-    autoload :TimestampTag,   'kuby/docker/timestamp_tag'
-    autoload :WebserverPhase, 'kuby/docker/webserver_phase'
-    autoload :YarnPhase,      'kuby/docker/yarn_phase'
+    LATEST_TAG = T.let('latest'.freeze, String)
+
+    autoload :Alpine,           'kuby/docker/alpine'
+    autoload :AppImage,         'kuby/docker/app_image'
+    autoload :AssetsPhase,      'kuby/docker/assets_phase'
+    autoload :BundlerPhase,     'kuby/docker/bundler_phase'
+    autoload :CLI,              'kuby/docker/cli'
+    autoload :CopyPhase,        'kuby/docker/copy_phase'
+    autoload :Credentials,      'kuby/docker/credentials'
+    autoload :Debian,           'kuby/docker/debian'
+    autoload :Distro,           'kuby/docker/distro'
+    autoload :Dockerfile,       'kuby/docker/dockerfile'
+    autoload :DockerURI,        'kuby/docker/docker_uri'
+    autoload :Image,            'kuby/docker/image'
+    autoload :InlineLayer,      'kuby/docker/inline_layer'
+    autoload :Layer,            'kuby/docker/layer'
+    autoload :LayerStack,       'kuby/docker/layer_stack'
+    autoload :LocalTags,        'kuby/docker/local_tags'
+    autoload :Packages,         'kuby/docker/packages'
+    autoload :PackagePhase,     'kuby/docker/package_phase'
+    autoload :RemoteTags,       'kuby/docker/remote_tags'
+    autoload :SetupPhase,       'kuby/docker/setup_phase'
+    autoload :Spec,             'kuby/docker/spec'
+    autoload :TimestampedImage, 'kuby/docker/timestamped_image'
+    autoload :TimestampTag,     'kuby/docker/timestamp_tag'
+    autoload :WebserverPhase,   'kuby/docker/webserver_phase'
+    autoload :YarnPhase,        'kuby/docker/yarn_phase'
   end
 end

data/lib/kuby/docker/alpine.rb

@@ -10,7 +10,8 @@ module Kuby
         [:nodejs, '12.14.1'],
         [:yarn, '1.21.1'],
         [:c_toolchain, nil],
-        [:tzdata, nil]
+        [:tzdata, nil],
+        [:git, nil]
       ].freeze, T::Array[[Symbol, T.nilable(String)]])
 
       sig { returns(Layer) }

data/lib/kuby/docker/app_image.rb

@@ -0,0 +1,19 @@
+
+# typed: strict
+
+module Kuby
+  module Docker
+    class AppImage < ::Kuby::Docker::TimestampedImage
+      extend T::Sig
+
+      sig { params(build_args: T::Hash[String, String]).returns(AppImage) }
+      def build(build_args = {})
+        unless ENV.fetch('RAILS_MASTER_KEY', '').empty?
+          build_args['RAILS_MASTER_KEY'] = T.must(ENV['RAILS_MASTER_KEY'])
+        end
+
+        super(build_args)
+      end
+    end
+  end
+end

data/lib/kuby/docker/cli.rb

@@ -53,19 +53,11 @@ module Kuby
         config.fetch('auths', {}).keys
       end
 
-      sig {
-        params(
-          dockerfile: Dockerfile,
-          image_url: String,
-          tags: T::Array[String],
-          build_args: T::Hash[T.any(Symbol, String), String]
-        )
-        .void
-      }
-      def build(dockerfile:, image_url:, tags:, build_args: {})
+      sig { params(image: Image, build_args: T::Hash[T.any(Symbol, String), String]).void }
+      def build(image, build_args: {})
         cmd = [
           executable, 'build',
-          *tags.flat_map { |tag| ['-t', "#{image_url}:#{tag}"] },
+          *image.tags.flat_map { |tag| ['-t', "#{image.image_url}:#{tag}"] },
           *build_args.flat_map do |arg, val|
             ['--build-arg', Shellwords.shellescape("#{arg}=#{val}")]
           end,
@@ -73,7 +65,7 @@ module Kuby
         ]
 
         open3_w(cmd) do |stdin, _wait_threads|
-          stdin.puts(dockerfile.to_s)
+          stdin.puts(image.dockerfile.to_s)
         end
 
         unless T.must(last_status).success?

data/lib/kuby/docker/docker_uri.rb

@@ -1,11 +1,14 @@
-# typed: false
+# typed: strict
 
 module Kuby
   module Docker
     class DockerURI
-      DEFAULT_REGISTRY_HOST = 'index.docker.io'.freeze
-      DEFAULT_REGISTRY_PORT = 443
+      extend T::Sig
 
+      DEFAULT_REGISTRY_HOST = T.let('index.docker.io'.freeze, String)
+      DEFAULT_REGISTRY_PORT = T.let(443, Integer)
+
+      sig { params(url: String).returns(DockerURI) }
       def self.parse(url)
         if idx = url.index('://')
           url = url[(idx + 3)..-1] || ''
@@ -13,17 +16,25 @@ module Kuby
 
         host_port, *path = url.split('/')
         host, port, *path = if host_port =~ /[.:]/
-          hst, prt = host_port.split(':')
-          [hst, prt || DEFAULT_REGISTRY_PORT, *path]
+          hst, prt = T.must(host_port).split(':')
+          [T.must(hst), prt || DEFAULT_REGISTRY_PORT, *path]
         else
           [DEFAULT_REGISTRY_HOST, DEFAULT_REGISTRY_PORT, host_port, *path]
         end
 
-        new(host, port.to_i, path.join('/'))
+        new(host.to_s, port.to_i, (path || []).join('/'))
       end
 
-      attr_reader :host, :port, :path
+      sig { returns(String) }
+      attr_reader :host
+
+      sig { returns(Integer) }
+      attr_reader :port
+
+      sig { returns(String) }
+      attr_reader :path
 
+      sig { params(host: String, port: Integer, path: String).void }
       def initialize(host, port, path)
         @host = host
         @port = port

data/lib/kuby/docker/errors.rb

@@ -6,25 +6,7 @@ module Kuby
     class PushError < StandardError; end
     class PullError < StandardError; end
     class LoginError < StandardError; end
-
-    class MissingTagError < StandardError
-      extend T::Sig
-
-      sig { returns(String) }
-      attr_reader :tag
-
-      sig { params(tag: String).void }
-      def initialize(tag)
-        @tag = tag
-        @message = T.let(@message, T.nilable(String))
-      end
-
-      sig { returns(String) }
-      def message
-        @message ||= "Could not find tag '#{tag}'."
-      end
-    end
-
+    class MissingTagError < StandardError; end
     class UnsupportedDistroError < StandardError; end
     class MissingPackageError < StandardError; end
     class MissingDistroError < StandardError; end