kuby-core 0.11.9 → 0.11.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c8dee9ccc32bf324c552b229219f3d8b704b3dfd5324da1a955e929f2789c4ac
-  data.tar.gz: e5fc43a0320adc589ac1be2f4cfdb6802016ee962cfa7bf33fee9a8a48eab53e
+  metadata.gz: bfd902325b0bf78437ea91ba6021ee0cc27f83c67d60a703508a8b0c5a84bc86
+  data.tar.gz: 9598df94486a386235a3566c4c0422f824c30870551251ce3f4836fc94677b39
 SHA512:
-  metadata.gz: 3a639a3e57d19629cdb5934bb7bc259c1b0de1d7f48c81683fb746664cba3c1d0526c5be2e2fbe752dd6604c5bce3b770f7231cdac738df4c0123a1730025af0
-  data.tar.gz: 2eb4703287635c6455956f295d000a8911454c2ba05d1edaf40bbdb060368ce4a38d44a6e9394db95fcf2a2ef764598ddf5ed399eb58aed0b5d3423e139765bc
+  metadata.gz: 2ba1c29f18311795a39c8038940c6bc53ae6868523d520af81a52543ee6726282696f07af78b2cd8d6736ce5d0e7bc98ffd14f61693cf0b7421b7cccbbe6ea70
+  data.tar.gz: 801f401ba743d5375e59c33e1be57b6ae7767d5580e37bef0186dd2e299e7293f1a9e2e277a10b43b94977e0f982be8908e57153fb14d136a9e54aea92031fdf

data/CHANGELOG.md

@@ -1,3 +1,21 @@
+## 0.11.14
+* Don't include port in image host for registry secrets (no idea why)
+
+## 0.11.13
+* Include port in image host.
+
+## 0.11.12
+* Revamp Docker URL parsing
+  - Docker URLs shouldn't have a scheme.
+
+## 0.11.11
+* Add support for a Docker registry running on localhost.
+  - Correctly parse and handle URLs with specific ports.
+  - Only perform a Docker login if a username is provided.
+
+## 0.11.10
+* Fix spelling in error message.
+
 ## 0.11.9
 * Use correct Docker Hub registry URL (index.docker.io).
 

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.1'
+  s.add_dependency 'docker-remote', '~> 0.5'
   s.add_dependency 'gli', '~> 2.0'
   s.add_dependency 'helm-cli', '~> 0.3'
   # See: https://github.com/Shopify/krane/pull/720

data/lib/kuby/docker.rb

@@ -14,6 +14,7 @@ module Kuby
     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'

data/lib/kuby/docker/docker_uri.rb

@@ -0,0 +1,34 @@
+# typed: false
+
+module Kuby
+  module Docker
+    class DockerURI
+      DEFAULT_REGISTRY_HOST = 'index.docker.io'.freeze
+      DEFAULT_REGISTRY_PORT = 443
+
+      def self.parse(url)
+        if idx = url.index('://')
+          url = url[(idx + 3)..-1] || ''
+        end
+
+        host_port, *path = url.split('/')
+        host, port, *path = if host_port =~ /[.:]/
+          hst, prt = host_port.split(':')
+          [hst, prt || DEFAULT_REGISTRY_PORT, *path]
+        else
+          [DEFAULT_REGISTRY_HOST, DEFAULT_REGISTRY_PORT, host_port, *path]
+        end
+
+        new(host, port.to_i, path.join('/'))
+      end
+
+      attr_reader :host, :port, :path
+
+      def initialize(host, port, path)
+        @host = host
+        @port = port
+        @path = path
+      end
+    end
+  end
+end

data/lib/kuby/docker/metadata.rb

@@ -8,8 +8,6 @@ module Kuby
       extend T::Sig
 
       DEFAULT_DISTRO = :debian
-      DEFAULT_REGISTRY_HOST = T.let('https://index.docker.io'.freeze, String)
-      DEFAULT_REGISTRY_SCHEME = T.let('https', String)
       LATEST_TAG = T.let('latest'.freeze, String)
 
       sig { params(image_url: String).void }
@@ -28,7 +26,7 @@ module Kuby
         @image_hostname = T.let(@image_hostname, T.nilable(String))
         @image_repo = T.let(@image_repo, T.nilable(String))
         @distro = T.let(@distro, T.nilable(Symbol))
-        @full_image_uri = T.let(@full_image_uri, T.nilable(URI::Generic))
+        @full_image_uri = T.let(@full_image_uri, T.nilable(DockerURI))
         @default_image_url = T.let(@default_image_url, T.nilable(String))
         @default_tags = T.let(@default_tags, T.nilable(T::Array[String]))
       end
@@ -40,17 +38,17 @@ module Kuby
 
       sig { returns(String) }
       def image_host
-        @image_host ||= "#{full_image_uri.scheme}://#{full_image_uri.host}"
+        @image_host ||= "#{full_image_uri.host}:#{full_image_uri.port}"
       end
 
       sig { returns(String) }
       def image_hostname
-        @image_hostname ||= T.must(URI(image_host).host)
+        @image_hostname ||= full_image_uri.host
       end
 
       sig { returns(String) }
       def image_repo
-        @image_repo ||= T.must(full_image_uri.path).sub(/\A[\/]+/, '')
+        @image_repo ||= full_image_uri.path
       end
 
       sig { returns(T::Array[String]) }
@@ -70,15 +68,9 @@ module Kuby
 
       private
 
-      sig { returns(URI::Generic) }
+      sig { returns(DockerURI) }
       def full_image_uri
-        @full_image_uri ||= if image_url.include?('://')
-          URI.parse(image_url)
-        elsif image_url =~ /\A[^.]+\.[^\/]+\//
-          URI.parse("#{DEFAULT_REGISTRY_SCHEME}://#{image_url}")
-        else
-          URI.parse("#{DEFAULT_REGISTRY_HOST}/#{image_url.sub(/\A[\/]+/, '')}")
-        end
+        @full_image_uri ||= DockerURI.parse(image_url)
       end
 
       sig { returns(String) }
@@ -93,15 +85,6 @@ module Kuby
           TimestampTag.new(Time.now).to_s, LATEST_TAG
         ]
       end
-
-      sig { params(url: String).returns(URI::Generic) }
-      def parse_url(url)
-        uri = URI.parse(url)
-        return uri if uri.scheme
-
-        # force a scheme because URI.parse won't work properly without one
-        URI.parse("#{DEFAULT_REGISTRY_SCHEME}://#{url}")
-      end
     end
   end
 end

data/lib/kuby/kubernetes/deployer.rb

@@ -1,4 +1,5 @@
 # typed: true
+
 require 'fileutils'
 require 'securerandom'
 require 'yaml'
@@ -6,6 +7,8 @@ require 'yaml'
 module Kuby
   module Kubernetes
     class Deployer
+      extend T::Sig
+
       attr_reader :environment
       attr_accessor :logdev
 
@@ -93,7 +96,7 @@ module Kuby
         task.run!(verify_result: true, prune: false)
       ensure
         ENV['KUBECONFIG'] = old_kubeconfig
-        FileUtils.rm_rf(tmpdir)
+        FileUtils.rm_rf(T.must(tmpdir))
       end
 
       def restart_rails_deployment_if_necessary

data/lib/kuby/kubernetes/spec.rb

@@ -152,7 +152,7 @@ module Kuby
             end
 
             docker_config do
-              registry_host spec.docker.metadata.image_host
+              registry_host spec.docker.metadata.image_hostname
               username spec.docker.credentials.username
               password spec.docker.credentials.password
               email spec.docker.credentials.email

data/lib/kuby/tasks.rb

@@ -41,10 +41,10 @@ module Kuby
         fail 'Cannot push Docker images built for the development environment'
       end
 
-      hostname = docker.metadata.image_hostname
+      host = docker.metadata.image_host
 
-      unless docker.cli.auths.include?(hostname)
-        Kuby.logger.info("Attempting to log in to registry at #{hostname}")
+      if docker.credentials.username && !docker.cli.auths.include?(host)
+        Kuby.logger.info("Attempting to log in to registry at #{host}")
 
         begin
           docker.cli.login(
@@ -53,7 +53,7 @@ module Kuby
             password: docker.credentials.password
           )
         rescue Kuby::Docker::LoginError => e
-          Kuby.logger.fatal("Couldn't log in to the registry at #{hostname}")
+          Kuby.logger.fatal("Couldn't log in to the registry at #{host}")
           Kuby.logger.fatal(e.message)
           return
         end
@@ -66,7 +66,7 @@ module Kuby
           docker.cli.push(image_url, tag)
         end
       rescue Kuby::Docker::MissingTagError => e
-        msg = "#{e.message} Run rake kuby:build to build the"\
+        msg = "#{e.message} Run kuby build to build the "\
           'Docker image before running this task.'
 
         Kuby.logger.fatal(msg)

data/lib/kuby/version.rb

@@ -1,5 +1,5 @@
 # typed: true
 
 module Kuby
-  VERSION = '0.11.9'.freeze
+  VERSION = '0.11.14'.freeze
 end

data/spec/docker/metadata_spec.rb

@@ -19,18 +19,18 @@ describe Kuby::Docker::Metadata do
   describe '#image_host' do
     subject { metadata.image_host }
 
-    it { is_expected.to eq(described_class::DEFAULT_REGISTRY_HOST) }
+    it { is_expected.to eq("#{Kuby::Docker::DockerURI::DEFAULT_REGISTRY_HOST}:443") }
 
     context 'when the image URL contains an explicit host' do
       let(:docker_image_url) { 'registry.foo.com/foo/testapp' }
 
-      it { is_expected.to eq('https://registry.foo.com') }
+      it { is_expected.to eq('registry.foo.com:443') }
     end
 
-    context 'when the image URL contains an explicit host with scheme' do
+    context 'when the image URL contains an explicit host' do
       let(:docker_image_url) { 'http://registry.foo.com/foo/testapp' }
 
-      it { is_expected.to eq('http://registry.foo.com') }
+      it { is_expected.to eq('registry.foo.com:443') }
     end
   end
 
@@ -46,18 +46,6 @@ describe Kuby::Docker::Metadata do
     end
   end
 
-  describe '#image_hostname' do
-    subject { metadata.image_hostname }
-
-    it { is_expected.to eq('www.docker.com') }
-
-    context 'when the image URL contains an explicit host' do
-      let(:docker_image_url) { 'registry.foo.com/foo/testapp' }
-
-      it { is_expected.to eq('registry.foo.com') }
-    end
-  end
-
   describe '#tags' do
     subject { metadata.tags }
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kuby-core
 version: !ruby/object:Gem::Version
-  version: 0.11.9
+  version: 0.11.14
 platform: ruby
 authors:
 - Cameron Dutro
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-29 00:00:00.000000000 Z
+date: 2021-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.5'
 - !ruby/object:Gem::Dependency
   name: gli
   requirement: !ruby/object:Gem::Requirement
@@ -230,6 +230,7 @@ files:
 - lib/kuby/docker/debian.rb
 - lib/kuby/docker/dev_spec.rb
 - lib/kuby/docker/distro.rb
+- lib/kuby/docker/docker_uri.rb
 - lib/kuby/docker/dockerfile.rb
 - lib/kuby/docker/errors.rb
 - lib/kuby/docker/inline_layer.rb