Skip to content

Aligns upgrade badge to the surrounding text

Miguel Rincon requested to merge 423770-align-upgrade-badge into master

What does this MR do and why?

Aligns upgrade badge to the surrounding text

Changelog: changed

Screenshots or screen recordings

Notice the alignment of the badge:

Before After
image image

How to set up and validate locally

Unless you have some outdated runners, it may be hard to downgrade your runners. We can simply change the data to force a change:

  1. While running GitLab Ultimate license
  2. Edit the following file to return either :available or :recommended as upgrade status
diff --git a/ee/app/graphql/ee/types/ci/runner_type.rb b/ee/app/graphql/ee/types/ci/runner_type.rb
index 34757f57f97d..a9afea1e7321 100644
--- a/ee/app/graphql/ee/types/ci/runner_type.rb
+++ b/ee/app/graphql/ee/types/ci/runner_type.rb
@@ -25,6 +25,7 @@ module RunnerType
             alpha: { milestone: '14.10' }
 
           def upgrade_status
+            return :available
             return unless upgrade_status_available?
 
             BatchLoader::GraphQL.for(object.id).batch(key: :upgrade_status) do |runner_ids, loader|
  1. Log in as admin
  2. Visit the Admin -> Runners page (http://gdk.test:3000/admin/runners/)
  3. Pick any runner to see it's details by clicking on the runner name
  4. Confirm the alter is not there, as the badge contains the required information

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #423770 (closed)

Merge request reports

Loading