Show notice if builds are not from latest pipeline

app/models/ci/build.rb

@@ -15,6 +15,9 @@ module Ci
     scope :with_artifacts, ->() { where.not(artifacts_file: nil) }
     scope :with_expired_artifacts, ->() { with_artifacts.where('artifacts_expire_at < ?', Time.now) }
     scope :last_month, ->() { where('created_at > ?', Date.today - 1.month) }
+    scope :latest_success_with_artifacts, ->() do
+      with_artifacts.success.latest
+    end
     mount_uploader :artifacts_file, ArtifactUploader
     mount_uploader :artifacts_metadata, ArtifactUploader

app/models/project.rb

@@ -429,10 +429,16 @@ class Project < ActiveRecord::Base
     repository.commit(ref)
   end
-  def latest_success_builds_for(ref = 'HEAD')
+  # ref can't be HEAD or SHA, can only be branch/tag name
+  def latest_success_pipeline_for(ref = 'master')
+    pipelines.where(ref: ref).success.latest
+  end
+
+  # ref can't be HEAD or SHA, can only be branch/tag name
+  def latest_success_builds_for(ref = 'master')
     Ci::Build.joins(:pipeline).
-      merge(pipelines.where(ref: ref).success.latest).
-      with_artifacts.success.latest
+      merge(latest_success_pipeline_for(ref)).
+      latest_success_with_artifacts
   end
   def merge_base_commit(first_commit_id, second_commit_id)

app/views/projects/branches/_branch.html.haml

@@ -27,7 +27,9 @@
         = link_to namespace_project_compare_index_path(@project.namespace, @project, from: @repository.root_ref, to: branch.name), class: 'btn btn-default', method: :post, title: "Compare" do
           Compare
-      - artifacts = @project.latest_success_builds_for(branch.name)
+      - pipeline = @project.latest_success_pipeline_for(branch.name).first
+      - if pipeline
+        - artifacts = pipeline.builds.latest_success_with_artifacts
         - if artifacts.any?
           .dropdown.inline.artifacts-btn
             %a.btn.dropdown-toggle{ 'data-toggle' => 'dropdown' }
@@ -37,6 +39,8 @@
                 Select Archive Format
             %ul.dropdown-menu.dropdown-menu-align-right{ role: 'menu' }
               %li.dropdown-header Artifacts
+              - unless pipeline.latest?
+                = " (not latest, but #{link_to(pipeline.short_sha, namespace_project_commits_path(@project.namespace, @project, pipeline.sha))})"
               - artifacts.each do |job|
                 %li
                   = link_to download_namespace_project_build_artifacts_path(@project.namespace, @project, job), rel: 'nofollow' do

app/views/projects/buttons/_download.html.haml

@@ -14,9 +14,13 @@
         %li
           = link_to archive_namespace_project_repository_path(@project.namespace, @project, ref: @ref, format: 'tar.gz'), rel: 'nofollow' do
             %span Download tar.gz
-        - artifacts = @project.latest_success_builds_for(@ref)
+        - pipeline = @project.latest_success_pipeline_for(@ref).first
+        - if pipeline
+          - artifacts = pipeline.builds.latest_success_with_artifacts
           - if artifacts.any?
             %li.dropdown-header Artifacts
+            - unless pipeline.latest?
+              = " (not latest, but #{link_to(pipeline.short_sha, namespace_project_commits_path(@project.namespace, @project, pipeline.sha))})"
             - artifacts.each do |job|
               %li
                 = link_to download_namespace_project_build_artifacts_path(@project.namespace, @project, job), rel: 'nofollow' do

app/views/projects/tags/_download.html.haml

@@ -12,9 +12,13 @@
     %li
       = link_to archive_namespace_project_repository_path(project.namespace, project, ref: ref, format: 'tar.gz'), rel: 'nofollow' do
         %span Download tar.gz
-    - artifacts = project.latest_success_builds_for(ref)
+    - pipeline = project.latest_success_pipeline_for(ref).first
+    - if pipeline
+      - artifacts = pipeline.builds.latest_success_with_artifacts
       - if artifacts.any?
         %li.dropdown-header Artifacts
+        - unless pipeline.latest?
+          = " (not latest, but #{link_to(pipeline.short_sha, namespace_project_commits_path(project.namespace, project, pipeline.sha))})"
         - artifacts.each do |job|
           %li
             = link_to download_namespace_project_build_artifacts_path(project.namespace, project, job), rel: 'nofollow' do