buildkite-builder 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 024ee5aff1b1a819fe74946becf1e3b1941c9f6e8ea86ba822a2d1411209e098
-  data.tar.gz: d9716d08bb4512fd0c9490ad26b1618a2b5e4f896693d5f5bca76024500fef70
+  metadata.gz: 2fafb5222b3bb1b03be360545674b957e7ecd2ec8d21ea4ac9ff59620626c584
+  data.tar.gz: 0223fba9082408be91fc3a38fc2e2a6eaec74e4fb45ec9fa6b121edc10bff07f
 SHA512:
-  metadata.gz: 3d8130054bd4276b9d1c4505798985b9032ca406c126de026fa36bbb0fa919541c4c7d4cdd4c510b52e6554febb66c2e8a8bcc8e63aa54061fb727e6cbe8b752
-  data.tar.gz: 2492024dff09bfd31688c07ca712fed1b3191584f9b5917c4294775f687671decfcd3f0e0b20635bb3941c88b7097129e79700706a916b615ca11867e47a75e9
+  metadata.gz: 9fa3c8ca51711531338492b8b2072b2f62fac4bfee34e81594423f25bab218b373182014a72dbcf0b274a004ae4bedf703a62403c1dd1d6b27dbbd040fe0d4da
+  data.tar.gz: 8e33e6990c3c6c33a88d380a4d45fbd655848f790cce0a50758a960fab6ffb2cbc59bb359e083cbe072c51141e975bbeb528a7d4269b55dd5e87f52eb2541ce4

data/README.md

@@ -28,7 +28,7 @@ steps:
   - label: ":toolbox:"
     plugins:
       - docker#v3.7.0:
-          image: gusto/buildkite-builder:1.1.0
+          image: gusto/buildkite-builder:1.2.0
           propagate-environment: true
 ```
 

data/VERSION

@@ -1 +1 @@
-1.1.0
+1.2.0

data/lib/buildkite/builder.rb

@@ -14,7 +14,6 @@ module Buildkite
     autoload :Manifest, File.expand_path('builder/manifest', __dir__)
     autoload :Processors, File.expand_path('builder/processors', __dir__)
     autoload :Rainbow, File.expand_path('builder/rainbow', __dir__)
-    autoload :Runner, File.expand_path('builder/runner', __dir__)
 
     BUILDKITE_DIRECTORY_NAME = Pathname.new('.buildkite').freeze
 

data/lib/buildkite/builder/commands/abstract.rb

@@ -6,6 +6,18 @@ module Buildkite
   module Builder
     module Commands
       class Abstract
+        PIPELINES_DIRECTORY = 'pipelines'
+        POSSIBLE_PIPELINE_PATHS = [
+          File.join('.buildkite', Context::PIPELINE_DEFINITION_FILE),
+          File.join('buildkite', Context::PIPELINE_DEFINITION_FILE),
+          File.join(Context::PIPELINE_DEFINITION_FILE)
+        ].freeze
+        POSSIBLE_PIPELINES_PATHS = [
+          File.join('.buildkite', PIPELINES_DIRECTORY),
+          File.join('buildkite', PIPELINES_DIRECTORY),
+          File.join(PIPELINES_DIRECTORY)
+        ].freeze
+
         class << self
           attr_accessor :description
 
@@ -35,6 +47,8 @@ module Buildkite
           if options[:help]
             puts options[:help]
             return
+          elsif !pipeline_path
+            abort "Unable to find pipeline"
           end
 
           run
@@ -42,6 +56,10 @@ module Buildkite
 
         private
 
+        def pipeline_slug
+          ARGV.last || Buildkite.env&.pipeline_slug
+        end
+
         def command_name
           Commands::COMMANDS.key(self.class.name.split('::').last.to_sym)
         end
@@ -51,12 +69,39 @@ module Buildkite
           # Subclasses should override to parse options.
         end
 
-        def available_pipelines
-          @available_pipelines ||= pipelines_path.children.select(&:directory?).map { |dir| dir.basename.to_s }
+        def log
+          @log ||= begin
+            Logger.new($stdout).tap do |logger|
+              logger.formatter = proc do |_severity, _datetime, _progname, msg|
+                "#{msg}\n"
+              end
+            end
+          end
         end
 
-        def pipelines_path
-          Builder.root.join(Builder::BUILDKITE_DIRECTORY_NAME).join(Runner::PIPELINES_PATH)
+        def pipeline_path
+          @pipeline_path ||=
+            find_root_by_main_pipeline ||
+            find_root_by_multi_pipeline
+        end
+
+        def find_root_by_main_pipeline
+          POSSIBLE_PIPELINE_PATHS.map { |path| Builder.root.join(path) }.find(&:exist?)&.dirname
+        end
+
+        def find_root_by_multi_pipeline
+          pipelines_path = POSSIBLE_PIPELINES_PATHS.map { |path| Builder.root.join(path) }.find(&:directory?)
+          
+          if pipelines_path
+            if pipeline_slug
+              path = pipelines_path.join(pipeline_slug)
+              path if path.directory?
+            elsif pipelines_path.children.one?
+              pipelines_path.children.first
+            else
+              raise 'Your project has multiple pipelines, please specify one.'
+            end
+          end
         end
       end
     end

data/lib/buildkite/builder/commands/preview.rb

@@ -9,21 +9,7 @@ module Buildkite
         self.description = 'Outputs the pipeline YAML.'
 
         def run
-          pipeline = ARGV.last
-
-          if !pipeline && !root_pipeline?
-            if available_pipelines.one?
-              pipeline = available_pipelines.first
-            else
-              raise 'You must specify a pipeline'
-            end
-          end
-
-          puts Runner.new(pipeline: pipeline).run.to_yaml
-        end
-
-        def root_pipeline?
-          pipelines_path.join(Context::PIPELINE_DEFINITION_FILE).exist?
+          puts Context.build(pipeline_path).pipeline.to_yaml
         end
       end
     end

data/lib/buildkite/builder/commands/run.rb

@@ -1,25 +1,57 @@
 # frozen_string_literal: true
 
+require 'tempfile'
+
 module Buildkite
   module Builder
     module Commands
       class Run < Abstract
         private
+        include LoggingUtils
+        using Rainbow
 
         self.description = 'Builds and uploads the generated pipeline.'
 
         def run
+          relative_pipeline_path = pipeline_path.relative_path_from(Builder.root)
+
           # This entrypoint is for running on CI. It expects certain environment
-          # variables to be set.
-          options = {
-            upload: true
-          }
+          # variables to be set. It also uploads the pipeline to Buildkite.
+          log.info "#{'+++ ' if Buildkite.env}🧰 " + 'Buildkite Builder'.color(:springgreen) + " ─ #{relative_pipeline_path.to_s.yellow}"
+          context = Context.new(pipeline_path, logger: log)
+  
+          results = benchmark("\nDone (%s)".color(:springgreen)) do
+            context.build
+          end
+          log.info(results)
+  
+          upload(context.pipeline)
+        end
+  
+        private
 
-          if available_pipelines.include?(Buildkite.env.pipeline_slug)
-            options[:pipeline] = Buildkite.env.pipeline_slug
+        def pipeline_path
+          pipeline_path_override || super
+        end
+  
+        def pipeline_path_override
+          if ENV['BUILDKITE_BUILDER_PIPELINE_PATH']
+            path = Pathname.new(ENV['BUILDKITE_BUILDER_PIPELINE_PATH'])
+            path.absolute? ? path : Builder.root.join(path)
           end
+        end
 
-          Builder::Runner.new(**options).run
+        def upload(pipeline)
+          # Upload the pipeline.
+          Tempfile.create(['pipeline', '.yml']) do |file|
+            file.sync = true
+            file.write(pipeline.to_yaml)
+  
+            log.info '+++ :paperclip: Uploading artifact'
+            Buildkite::Pipelines::Command.artifact!(:upload, file.path)
+            log.info '+++ :pipeline: Uploading pipeline'
+            Buildkite::Pipelines::Command.pipeline!(:upload, file.path)
+          end
         end
       end
     end

data/lib/buildkite/builder/context.rb

@@ -1,3 +1,5 @@
+require 'logger'
+
 module Buildkite
   module Builder
     class Context

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: buildkite-builder
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Ngan Pham
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-02-16 00:00:00.000000000 Z
+date: 2021-02-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sorted_set
@@ -148,7 +148,6 @@ files:
 - lib/buildkite/builder/processors.rb
 - lib/buildkite/builder/processors/abstract.rb
 - lib/buildkite/builder/rainbow.rb
-- lib/buildkite/builder/runner.rb
 - lib/buildkite/env.rb
 - lib/buildkite/pipelines.rb
 - lib/buildkite/pipelines/api.rb

data/lib/buildkite/builder/runner.rb

@@ -1,77 +0,0 @@
-# frozen_string_literal: true
-
-require 'bundler'
-require 'pathname'
-require 'tempfile'
-require 'logger'
-
-module Buildkite
-  module Builder
-    class Runner
-      include LoggingUtils
-      using Rainbow
-
-      PIPELINES_PATH = Pathname.new('pipelines').freeze
-
-      attr_reader :options
-
-      def initialize(**options)
-        @options = options
-      end
-
-      def run
-        log.info "#{'+++ ' if Buildkite.env}🧰 " + 'Buildkite Builder'.color(:springgreen) + " ─ #{@options[:pipeline].yellow}"
-        context = Context.new(root, logger: log)
-
-        results = benchmark("\nDone (%s)".color(:springgreen)) do
-          context.build
-        end
-        log.info(results)
-
-        if options[:upload]
-          upload(context.pipeline)
-        end
-
-        # Always return the pipeline.
-        context.pipeline
-      end
-
-      private
-
-      def log
-        @log ||= begin
-          Logger.new($stdout).tap do |logger|
-            logger.formatter = proc do |_severity, _datetime, _progname, msg|
-              "#{msg}\n"
-            end
-          end
-        end
-      end
-
-      def upload(pipeline)
-        Tempfile.create(['pipeline', '.yml']) do |file|
-          file.sync = true
-          file.write(pipeline.to_yaml)
-
-          log.info '+++ :paperclip: Uploading artifact'
-          Buildkite::Pipelines::Command.artifact!(:upload, file.path)
-          log.info '+++ :pipeline: Uploading pipeline'
-          Buildkite::Pipelines::Command.pipeline!(:upload, file.path)
-        end
-      end
-
-      def root
-        @root ||= begin
-          path = Builder.root.join(Builder::BUILDKITE_DIRECTORY_NAME)
-          if options[:pipeline]
-            pipeline_path = path.join(PIPELINES_PATH).join(options[:pipeline])
-            if pipeline_path.directory?
-              path = pipeline_path
-            end
-          end
-          path
-        end
-      end
-    end
-  end
-end