shipit-engine 0.39.0 → 0.40.0

data/app/models/shipit/webhooks/handlers/pull_request/reopened_handler.rb

@@ -55,7 +55,7 @@ module Shipit
           def stack
             @stack ||=
               Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
-                .new(params, scope: repository.review_stacks)
+              .new(params, scope: repository.review_stacks)
           end
 
           def pull_request

data/app/models/shipit/webhooks/handlers/pull_request/review_stack_adapter.rb

@@ -13,7 +13,7 @@ module Shipit
           end
 
           def stack
-            @stack ||= scope.find_by(environment: environment)
+            @stack ||= scope.find_by(environment:)
           end
 
           def find_or_create!
@@ -87,9 +87,9 @@ module Shipit
           def stack_attributes
             {
               branch: params.pull_request.head.ref,
-              environment: environment,
+              environment:,
               ignore_ci: false,
-              continuous_deployment: false,
+              continuous_deployment: false
             }
           end
 

data/app/models/shipit/webhooks/handlers/pull_request/unlabeled_handler.rb

@@ -65,7 +65,7 @@ module Shipit
           def stack
             @stack ||=
               Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
-                .new(params, scope: repository.review_stacks)
+              .new(params, scope: repository.review_stacks)
           end
 
           def pull_request

data/app/models/shipit/webhooks/handlers/push_handler.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module Webhooks
     module Handlers
@@ -9,7 +10,7 @@ module Shipit
         def process
           stacks
             .not_archived
-            .where(branch: branch)
+            .where(branch:)
             .find_each(&:sync_github)
         end
 

data/app/models/shipit/webhooks/handlers/status_handler.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module Webhooks
     module Handlers

data/app/models/shipit/webhooks.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module Webhooks
     class << self
@@ -13,11 +14,11 @@ module Shipit
             Handlers::PullRequest::AssignedHandler,
             Handlers::PullRequest::LabeledHandler,
             Handlers::PullRequest::UnlabeledHandler,
-            Handlers::PullRequest::LabelCapturingHandler,
+            Handlers::PullRequest::LabelCapturingHandler
           ],
           'status' => [Handlers::StatusHandler],
           'membership' => [Handlers::MembershipHandler],
-          'check_suite' => [Handlers::CheckSuiteHandler],
+          'check_suite' => [Handlers::CheckSuiteHandler]
         }
       end
 

data/app/serializers/concerns/shipit/conditional_attributes.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module ConditionalAttributes
     extend ActiveSupport::Concern

data/app/serializers/shipit/anonymous_user_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class AnonymousUserSerializer < UserSerializer
   end

data/app/serializers/shipit/command_line_user_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class CommandLineUserSerializer < UserSerializer
   end

data/app/serializers/shipit/commit_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class CommitSerializer < ShortCommitSerializer
     include GithubUrlHelper
@@ -24,7 +25,7 @@ module Shipit
     def pull_request
       {
         number: object.pull_request_number,
-        html_url: github_pull_request_url(object),
+        html_url: github_pull_request_url(object)
       }
     end
 

data/app/serializers/shipit/deploy_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class DeploySerializer < TaskSerializer
     include GithubUrlHelper

data/app/serializers/shipit/hook_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class HookSerializer < ActiveModel::Serializer
     include ConditionalAttributes

data/app/serializers/shipit/merge_request_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class MergeRequestSerializer < ActiveModel::Serializer
     include GithubUrlHelper
@@ -8,7 +9,7 @@ module Shipit
     has_one :head, serializer: ShortCommitSerializer
 
     attributes :id, :number, :title, :github_id, :additions, :deletions, :state, :merge_status, :mergeable,
-      :merge_requested_at, :rejection_reason, :html_url, :branch, :base_ref
+               :merge_requested_at, :rejection_reason, :html_url, :branch, :base_ref
 
     def html_url
       github_pull_request_url(object)

data/app/serializers/shipit/rollback_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class RollbackSerializer < DeploySerializer
     def type

data/app/serializers/shipit/short_commit_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class ShortCommitSerializer < ActiveModel::Serializer
     attributes :sha, :message

data/app/serializers/shipit/stack_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class StackSerializer < ActiveModel::Serializer
     include ConditionalAttributes
@@ -6,9 +7,9 @@ module Shipit
     has_one :lock_author
 
     attributes :id, :repo_owner, :repo_name, :environment, :html_url, :url, :tasks_url, :deploy_url,
-      :merge_requests_url, :deploy_spec, :undeployed_commits_count, :is_locked, :lock_reason, :continuous_deployment,
-      :created_at, :updated_at, :locked_since, :last_deployed_at, :branch, :merge_queue_enabled, :is_archived,
-      :archived_since, :ignore_ci
+               :merge_requests_url, :deploy_spec, :undeployed_commits_count, :is_locked, :lock_reason,
+               :continuous_deployment, :created_at, :updated_at, :locked_since, :last_deployed_at, :branch,
+               :merge_queue_enabled, :is_archived, :archived_since, :ignore_ci
 
     def url
       api_stack_url(object)

data/app/serializers/shipit/tail_task_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class TailTaskSerializer < ActiveModel::Serializer
     include ChunksHelper
@@ -8,6 +9,7 @@ module Shipit
 
     def url
       return @url if defined? @url
+
       @url = next_chunks_url(task, last_byte: next_offset)
     end
 
@@ -29,7 +31,7 @@ module Shipit
 
     private
 
-    alias_method :task, :object
+    alias task object
     delegate :stack, to: :object
 
     def next_offset
@@ -42,6 +44,7 @@ module Shipit
 
     def rollback
       return @rollback if defined? @rollback
+
       @rollback = stack.rollbacks.where(parent_id: task.id).last
     end
   end

data/app/serializers/shipit/task_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class TaskSerializer < ActiveModel::Serializer
     include ConditionalAttributes

data/app/serializers/shipit/user_serializer.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class UserSerializer < ActiveModel::Serializer
     attributes :id, :name, :email, :login, :avatar_url, :created_at, :updated_at, :github_id, :anonymous

data/app/validators/ascii_only_validator.rb

@@ -1,8 +1,9 @@
 # frozen_string_literal: true
+
 class AsciiOnlyValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, value)
-    if value && !value.ascii_only?
-      record.errors.add(attribute, :ascii)
-    end
+    return unless value && !value.ascii_only?
+
+    record.errors.add(attribute, :ascii)
   end
 end

data/app/validators/subset_validator.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 class SubsetValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, value)
     superset = options[:of]

data/app/views/shipit/ccmenu/project.xml.builder

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Derived from http://timnew.me/blog/2013/04/07/multiple-project-summary-reporting-standard-cctray-xml-feed/
 status_map = { 'backlogged' => 'failure', 'locked' => 'failure' }
 xml.instruct!
@@ -10,6 +11,6 @@ xml.Projects do
     activity: deploy.running? ? 'Building' : 'Sleeping',
     lastBuildTime: deploy.ended_at || deploy.started_at || deploy.created_at,
     lastBuildLabel: deploy.id,
-    webUrl: stack_url(stack),
+    webUrl: stack_url(stack)
   )
 end

data/app/views/shipit/continuous_delivery_schedules/show.html.erb

@@ -0,0 +1,59 @@
+<%= render partial: 'shipit/stacks/header', locals: { stack: @stack } %>
+
+<div class="wrapper continuous-delivery-schedule">
+    <section>
+        <header class="section-header">
+            <h2>Continuous Delivery Schedule (Stack #<%= @stack.id %>)</h2>
+        </header>
+    </section>
+    <div class="setting-section">
+        <% if @continuous_delivery_schedule.errors.any? %>
+            <div class="validation-errors">
+                <p>Validation errors prevented your schedule from being saved</p>
+                <ul>
+                    <% @continuous_delivery_schedule.errors.full_messages.each do |full_message| %>
+                        <li><%= full_message %></li>
+                    <% end %>
+                </ul>
+            </div>
+        <% end %>
+        <%= form_for(@continuous_delivery_schedule, url: stack_continuous_delivery_schedule_path, method: :patch) do |f| %>
+            <table class="field-wrapper">
+                <tbody>
+                    <% Shipit::ContinuousDeliverySchedule::DAYS.rotate.each do |day| %>
+                        <tr>
+                            <td>
+                                <%= f.check_box("#{day}_enabled") %>
+                            </td>
+                            <td>
+                                <%= f.label("#{day}_enabled", day.titlecase) %>
+                            </td>
+                            <td>
+                                <%= f.time_field("#{day}_start", include_seconds: false) %>
+                            </td>
+                            <td>&rarr;</td>
+                            <td>
+                                <%= f.time_field("#{day}_end", include_seconds: false) %>
+                            </td>
+                            <td>
+                                <% if day == "monday" %>
+                                    <button data-action="copy-to-all" type="button">Copy to all &darr;</button>
+                                <% end %>
+                            </td>
+                        </tr>
+                    <% end %>
+                </tbody>
+            </table>
+            
+            <p>
+                &#x2139;&#xFE0F; 
+                All times are in <%= Time.zone.name %>
+                (the <a href="https://guides.rubyonrails.org/configuring.html#config-time-zone">default time zone</a>).
+            </p>
+
+            <div class="field-wrapper">
+                <%= f.submit("Save", class: "btn") %>
+            </div>
+        <% end %>
+    </div>
+</div>

data/app/views/shipit/stacks/_settings_form.erb

@@ -17,6 +17,7 @@
 		<div class="field-wrapper">
 			<%= f.check_box :continuous_deployment %>
 			<%= f.label :continuous_deployment, 'Enable continuous deployment' %>
+			(<%= link_to("Edit schedule", stack_continuous_delivery_schedule_path) %>)
 		</div>
 
 		<div class="field-wrapper">

data/config/initializers/inflections.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 ActiveSupport::Inflector.inflections(:en) do |inflect|
   inflect.acronym('CCMenu')
 end

data/config/locales/en.yml

@@ -81,6 +81,7 @@ en:
     messages:
       subset: "is not a strict subset of %{of}"
       ascii: "contains non-ASCII characters"
+      must_be_after_start: "must be after start (%{start})"
   deployment_description:
     deploy:
       in_progress: "%{author} triggered the deploy of %{stack} to %{sha}"

data/config/routes.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Shipit::Engine.routes.draw do
   stack_id_format = %r{[^/]+/[^/]+/[^/]+}
   repository_id_format = %r{[^/]+/[^/]+}
@@ -15,7 +16,7 @@ Shipit::Engine.routes.draw do
   # API
   namespace :api do
     root to: 'base#index'
-    resources :stacks, only: %i(index create)
+    resources :stacks, only: %i[index create]
     scope '/stacks/*id', id: stack_id_format, as: :stack do
       get '/' => 'stacks#show'
       delete '/' => 'stacks#destroy'
@@ -25,24 +26,24 @@ Shipit::Engine.routes.draw do
 
     scope '/stacks/*stack_id', stack_id: stack_id_format, as: :stack do
       get '/ccmenu' => 'ccmenu#show', as: :ccmenu
-      resource :lock, only: %i(create update destroy)
-      resources :tasks, only: %i(index show) do
+      resource :lock, only: %i[create update destroy]
+      resources :tasks, only: %i[index show] do
         resource :output, only: :show
         member do
           put :abort
         end
       end
-      resources :deploys, only: %i(index create) do
-        resources :release_statuses, only: %i(create)
+      resources :deploys, only: %i[index create] do
+        resources :release_statuses, only: %i[create]
       end
-      resources :rollbacks, only: %i(create)
-      resources :commits, only: %i(index)
-      resources :merge_requests, only: %i(index show update destroy)
+      resources :rollbacks, only: %i[create]
+      resources :commits, only: %i[index]
+      resources :merge_requests, only: %i[index show update destroy]
       post '/task/:task_name' => 'tasks#trigger', as: :trigger_task
-      resources :hooks, only: %i(index create show update destroy)
+      resources :hooks, only: %i[index create show update destroy]
     end
 
-    resources :hooks, only: %i(index create show update destroy)
+    resources :hooks, only: %i[index create show update destroy]
   end
 
   scope '/ccmenu/*stack_id', stack_id: stack_id_format, as: :ccmenu_url do
@@ -57,7 +58,7 @@ Shipit::Engine.routes.draw do
   # Humans
   resources :api_clients
 
-  resources :repositories, only: %i(new index create)
+  resources :repositories, only: %i[new index create]
   scope '/repositories/*id', id: repository_id_format, as: :repository do
     get '/' => 'repositories#show'
     patch '/' => 'repositories#update'
@@ -73,7 +74,7 @@ Shipit::Engine.routes.draw do
     get :logout
   end
 
-  resources :stacks, only: %i(new create index)
+  resources :stacks, only: %i[new create index]
   scope '/*id', id: stack_id_format, as: :stack do
     get '/' => 'stacks#show'
     patch '/' => 'stacks#update'
@@ -84,6 +85,8 @@ Shipit::Engine.routes.draw do
     post :refresh, controller: :stacks
     get :refresh, controller: :stacks # For easier design, sorry :/
     post :clear_git_cache, controller: :stacks
+
+    resource :continuous_delivery_schedule, only: %i[show update]
   end
 
   scope '/task/:id', controller: :tasks do
@@ -96,9 +99,9 @@ Shipit::Engine.routes.draw do
 
     get '/stats' => 'stats#show', as: :stats
 
-    resources :rollbacks, only: %i(create)
-    resources :commits, only: %i(update)
-    resources :tasks, only: %i(show) do
+    resources :rollbacks, only: %i[create]
+    resources :commits, only: %i[update]
+    resources :tasks, only: %i[show] do
       collection do
         get '' => 'tasks#index', as: :index
         get ':definition_id/new' => 'tasks#new', as: :new
@@ -111,17 +114,17 @@ Shipit::Engine.routes.draw do
       end
     end
 
-    resources :deploys, only: %i(show create) do
+    resources :deploys, only: %i[show create] do
       get ':sha', sha: sha_format, on: :new, action: :new, as: ''
       member do
         get :rollback
         get :revert
       end
 
-      resources :release_statuses, only: %i(create)
+      resources :release_statuses, only: %i[create]
     end
 
-    resources :merge_requests, only: %i(index destroy create)
+    resources :merge_requests, only: %i[index destroy create]
   end
   get '/stacks/:id' => 'stacks#lookup'
 

data/db/migrate/20240821003007_add_continuous_delivery_schedules.rb

@@ -0,0 +1,13 @@
+class AddContinuousDeliverySchedules < ActiveRecord::Migration[7.1]
+  def change
+    create_table(:continuous_delivery_schedules) do |t|
+      t.references(:stack, null: false, index: { unique: true })
+      %w[sunday monday tuesday wednesday thursday friday saturday].each do |day|
+        t.boolean("#{day}_enabled", null: false, default: true)
+        t.time("#{day}_start", null: false, default: "00:00")
+        t.time("#{day}_end", null: false, default: "23:59")
+      end
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20250207203053_embiggen_github_ids.rb

@@ -0,0 +1,8 @@
+class EmbiggenGithubIds < ActiveRecord::Migration[7.2]
+  def change
+    change_column(:commit_deployments, :github_id, :bigint)
+    change_column(:github_hooks, :github_id, :bigint)
+    change_column(:teams, :github_id, :bigint)
+    change_column(:users, :github_id, :bigint)
+  end
+end

data/lib/shipit/cast_value.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module CastValue
     def to_boolean(value)

data/lib/shipit/command.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'pty'
 require 'shellwords'
 require 'fileutils'
@@ -27,9 +28,9 @@ module Shipit
 
     attr_reader :out, :chdir, :env, :args, :pid, :timeout
 
-    def initialize(*args, default_timeout: Shipit.default_inactivity_timeout, env: {}, chdir:)
+    def initialize(*args, chdir:, default_timeout: Shipit.default_inactivity_timeout, env: {})
       @args, options = parse_arguments(args)
-      @timeout = options['timeout'] || options[:timeout] || default_timeout
+      @timeout = parse_timeout(options['timeout'] || options[:timeout]) || default_timeout
       @env = env.transform_values { |v| v&.to_s }
       @chdir = chdir.to_s
       @timed_out = false
@@ -83,6 +84,7 @@ module Shipit
 
     def start(&block)
       return if @started
+
       @control_block = block
       @out = @pid = nil
       FileUtils.mkdir_p(@chdir)
@@ -106,10 +108,10 @@ module Shipit
       start
       begin
         read_stream(@out, &block)
-      rescue TimedOut => error
-        yield red("No output received in the last #{timeout} seconds.") + "\n"
+      rescue TimedOut => e
+        yield "#{red("No output received in the last #{timeout} seconds.")}\n"
         terminate!(&block)
-        raise error
+        raise e
       rescue Errno::EIO # Somewhat expected on Linux: http://stackoverflow.com/a/10306782
       end
 
@@ -125,6 +127,7 @@ module Shipit
     def stream!(&block)
       stream(&block)
       raise Failed.new(exit_message, code) unless success?
+
       self
     end
 
@@ -173,7 +176,7 @@ module Shipit
     ensure
       begin
         read_stream(@out, &block)
-      rescue
+      rescue StandardError
       end
     end
 
@@ -187,9 +190,8 @@ module Shipit
       rescue TimedOut
       rescue Errno::EIO # EIO is somewhat expected on Linux: http://stackoverflow.com/a/10306782
         # If we try to read the stream right after sending a signal, we often get an Errno::EIO.
-        if reap_child!(block: false)
-          return true
-        end
+        return true if reap_child!(block: false)
+
         # If we let the child a little bit of time, it solves it.
         retry_count -= 1
         if retry_count > 0
@@ -206,6 +208,22 @@ module Shipit
       Process.kill(sig, @pid)
     end
 
+    def parse_timeout(timeout)
+      case timeout
+      when String
+        begin
+          Duration.parse(timeout).to_i
+        rescue Duration::ParseError
+          # If given garbage we fallback to the default.
+          # It's not ideal but we don't have a good way to notify about
+          # syntax errors.
+          nil
+        end
+      else
+        timeout
+      end
+    end
+
     def parse_arguments(arguments)
       options = {}
       args = arguments.flatten.map do |argument|
@@ -217,6 +235,7 @@ module Shipit
           argument
         end
       end
+
       [args.map(&:to_s), options]
     end
 
@@ -235,6 +254,7 @@ module Shipit
     def reap_child!(block: true)
       return @status if @status
       return unless running? # Command was never started e.g. permission denied, not found etc
+
       if block
         _, @status = Process.waitpid2(@pid)
       elsif res = Process.waitpid2(@pid, Process::WNOHANG)

data/lib/shipit/commands.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class Commands
     class << self
@@ -7,12 +8,13 @@ module Shipit
       end
 
       def git_version
-        @git_version ||= parse_git_version(%x(git --version))
+        @git_version ||= parse_git_version(`git --version`)
       end
 
       def parse_git_version(raw_git_version)
         match_info = raw_git_version.match(/(\d+\.\d+\.\d+)/)
         raise 'git command not found' unless match_info
+
         Gem::Version.new(match_info[1])
       end
     end
@@ -36,7 +38,7 @@ module Shipit
       @base_env ||= Shipit.env.merge(
         'GITHUB_DOMAIN' => github.domain,
         'GITHUB_TOKEN' => github.token,
-        'GIT_ASKPASS' => Shipit::Engine.root.join('lib', 'snippets', 'git-askpass').realpath.to_s,
+        'GIT_ASKPASS' => Shipit::Engine.root.join('lib', 'snippets', 'git-askpass').realpath.to_s
       )
     end
 

data/lib/shipit/csv_serializer.rb

@@ -1,15 +1,18 @@
 # frozen_string_literal: true
+
 module Shipit
   module CSVSerializer
     extend self
 
     def load(payload)
       return [] if payload.blank?
+
       payload.split(',')
     end
 
     def dump(array)
       return nil if array.blank?
+
       Array.wrap(array).join(',')
     end
   end

data/lib/shipit/deploy_commands.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   class DeployCommands < TaskCommands
     def steps
@@ -10,7 +11,7 @@ module Shipit
       super.merge(
         'SHA' => commit.sha,
         'REVISION' => commit.sha,
-        'DIFF_LINK' => diff_url,
+        'DIFF_LINK' => diff_url
       )
     end