dri 0.1.3 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1c589c770547e0142cb5b2942316e41af19cad293ed2e141d95e54a7379f741
-  data.tar.gz: b1cc881757273745c40485c469b34abb642a8edcda18ae1baf22bfe80ed8bacd
+  metadata.gz: 345d930ab1d8693abc08cea84c56059acf4617b7b98b3d585ce6838c2be76ed5
+  data.tar.gz: 43fe7a822eb463770ade0125528e3a18a58bf12b1c8dd0c83daffffb4fac01ff
 SHA512:
-  metadata.gz: 0fa73e3af14a31bcf96f919c1b37465695a52a7db5d0fcaedf8ee36e51c817bf4c789c3fe24bc2169de0b385b83d0ec92ae716f8da6fbb55cb134c6219a0b26a
-  data.tar.gz: 880ae4a81c350dcfe7b49c9db00c3760d7f98b56383b7516d0c890dae24034fc3d8ebb86f37592c8d65655c3cbe8671bddf3c26327d49a6006b0e9f5f0313afa
+  metadata.gz: 799aeafefdbf86bca0a6b317806cd8054df97b2871bba345d9c3c44c0a0ae716d152f85afd1e5536b2dafb3e4399f2b757912558d6a59bc3661bb2e6699fc120
+  data.tar.gz: b0a1e74085cd70be58b8fcef39f436cf9c16b0d49c39d48833a86d74814b6fe7b654fb833be252c6c26bd68945691d4465f3c351ee41837a7454bb17b601e820

data/.gitlab-ci.yml

@@ -1,9 +1,10 @@
 .job_base:
-  image: ruby:3.1
+  image: ruby:2.7
   variables:
     BUNDLE_PATH: vendor/bundle
     BUNDLE_SUPPRESS_INSTALL_USING_MESSAGES: "true"
   before_script:
+    - gem install bundler -v 2.3.9 --no-document
     - bundle install
   cache:
     key:
@@ -33,8 +34,22 @@ build_gem:
   script:
     - gem build
 
+rubocop:
+  stage: test
+  extends: .job_base
+  script:
+    - bundle exec rubocop --color
+
 rspec:
   stage: test
   extends: .job_base
   script:
-    - bundle exec rspec
+    - bundle exec rspec --color
+
+rspec 3.0:
+  stage: test
+  extends: .job_base
+  image: ruby:3.0
+  script:
+    - bundle exec rspec --color
+

data/.rubocop.yml

@@ -0,0 +1,53 @@
+inherit_gem:
+  gitlab-styles:
+    - rubocop-all.yml
+    - rubocop-gemspec.yml
+    - rubocop-lint.yml
+    - rubocop-naming.yml
+    - rubocop-performance.yml
+    - rubocop-rspec.yml
+    - rubocop-security.yml
+    - rubocop-style.yml
+
+AllCops:
+  TargetRubyVersion: 2.7
+
+RSpec/MultipleMemoizedHelpers:
+  Max: 25
+  AllowSubject: true
+
+CodeReuse/ActiveRecord:
+  Exclude:
+    - spec/**/*.rb
+
+Metrics/AbcSize:
+  Enabled: true
+  Max: 30
+
+Metrics/BlockLength:
+  Enabled: true
+  Exclude:
+    - spec/**/*.rb
+    - dri.gemspec
+
+Metrics/MethodLength:
+  Enabled: true
+  Max: 30
+
+Layout/BeginEndAlignment:
+  Enabled: true
+
+Layout/EmptyLinesAroundAttributeAccessor:
+  Enabled: true
+
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
+Lint/HashCompareByIdentity:
+  Enabled: true
+
+Lint/RedundantSafeNavigation:
+  Enabled: true
+
+Style/ClassEqualityComparison:
+  Enabled: true

data/.ruby-version

@@ -0,0 +1 @@
+2.7.5

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 # Specify your gem's dependencies in dri.gemspec

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dri (0.1.2)
+    dri (0.1.3)
       httparty (~> 0.20.0)
       json (~> 2.6.1)
       markdown-tables (~> 1.1.1)
@@ -19,39 +19,88 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
+    activesupport (7.0.2.3)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
     addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.2)
+    concurrent-ruby (1.1.10)
     crack (0.4.5)
       rexml
     diff-lcs (1.5.0)
+    gitlab-styles (7.0.0)
+      rubocop (~> 0.91, >= 0.91.1)
+      rubocop-gitlab-security (~> 0.1.1)
+      rubocop-graphql (~> 0.10)
+      rubocop-performance (~> 1.9.2)
+      rubocop-rails (~> 2.9)
+      rubocop-rspec (~> 1.44)
     hashdiff (1.0.1)
     httparty (0.20.0)
       mime-types (~> 3.0)
       multi_xml (>= 0.5.2)
+    i18n (1.10.0)
+      concurrent-ruby (~> 1.0)
     json (2.6.1)
     markdown-tables (1.1.1)
     mime-types (3.4.1)
       mime-types-data (~> 3.2015)
     mime-types-data (3.2022.0105)
+    minitest (5.15.0)
     multi_xml (0.6.0)
+    parallel (1.22.1)
+    parser (3.1.1.0)
+      ast (~> 2.4.1)
     pastel (0.8.0)
       tty-color (~> 0.5)
     public_suffix (4.0.6)
-    rake (12.3.3)
+    rack (2.2.3)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    regexp_parser (2.2.1)
     rexml (3.2.5)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
       rspec-expectations (~> 3.10.0)
       rspec-mocks (~> 3.10.0)
-    rspec-core (3.10.1)
+    rspec-core (3.10.2)
       rspec-support (~> 3.10.0)
-    rspec-expectations (3.10.1)
+    rspec-expectations (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
-    rspec-mocks (3.10.2)
+    rspec-mocks (3.10.3)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
-    rspec-support (3.10.2)
+    rspec-support (3.10.3)
+    rubocop (0.93.1)
+      parallel (~> 1.10)
+      parser (>= 2.7.1.5)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8)
+      rexml
+      rubocop-ast (>= 0.6.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 2.0)
+    rubocop-ast (1.16.0)
+      parser (>= 3.1.1.0)
+    rubocop-gitlab-security (0.1.1)
+      rubocop (>= 0.51)
+    rubocop-graphql (0.14.2)
+      rubocop (>= 0.87, < 2)
+    rubocop-performance (1.9.2)
+      rubocop (>= 0.90.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    rubocop-rails (2.9.1)
+      activesupport (>= 4.2.0)
+      rack (>= 1.1)
+      rubocop (>= 0.90.0, < 2.0)
+    rubocop-rspec (1.44.1)
+      rubocop (~> 0.87)
+      rubocop-ast (>= 0.7.1)
+    ruby-progressbar (1.11.0)
     strings (0.2.1)
       strings-ansi (~> 0.2)
       unicode-display_width (>= 1.5, < 3.0)
@@ -85,7 +134,9 @@ GEM
       pastel (~> 0.8)
       strings (~> 0.2.0)
       tty-screen (~> 0.8)
-    unicode-display_width (2.1.0)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (1.8.0)
     unicode_utils (1.4.0)
     webmock (3.14.0)
       addressable (>= 2.8.0)
@@ -98,6 +149,7 @@ PLATFORMS
 
 DEPENDENCIES
   dri!
+  gitlab-styles (~> 7.0.0)
   rake
   rspec (~> 3.10.0)
   timecop (~> 0.9.1)

data/README.md

@@ -56,13 +56,17 @@ $ dri profile
   - failures
   - testcases
   - triaged
+  - quarantines
+  - dequarantines
+  - featureflags
 - [4. publish](#4-publish)
   - report
 - [5. rm](#5-rm)
   - emoji
   - profile
   - reports
-- [6. version](#6-version)
+- [6. incidents](#6-incidents)
+- [7. version](#7-version)
 
 #### 1. init
 
@@ -107,6 +111,25 @@ $ dri fetch triaged
 
 Fetches triaged failures which use the triage emoji specified in `dri profile`.
 
+```shell
+$ dri fetch quarantines
+```
+
+Fetches open quarantine Merge Requests to be reviewed
+
+```shell
+$ dri fetch dequarantines
+```
+
+Fetches open dequarantine Merge Requests to be reviewed
+
+```shell
+$ dri fetch featureflags
+```
+
+Fetches a list of today's feature flag changes, including the date and time in UTC of when the change occurred as well as a link to the corresponding issue from the feature-flag-log project.
+Results are organized by environment (production, staging, staging ref and preprod).
+
 #### 4. publish
 
 ```shell
@@ -159,7 +182,15 @@ $ dri rm profile
 
 Removes the profile currently in use.
 
-#### 6. version
+#### 6. incidents
+
+```shell
+$ dri incidents
+```
+
+Have a quick look at currently active/mitigated incidents on GitLab services.
+
+#### 7. version
 
 ```shell
 $ dri version

data/Rakefile

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/bin/console

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require "bundler/setup"
 require "dri"

data/dri.gemspec

@@ -1,41 +1,44 @@
+# frozen_string_literal: true
+
 require_relative 'lib/dri/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "dri"
-  spec.license       = "MIT"
+  spec.name          = 'dri'
+  spec.license       = 'MIT'
   spec.version       = Dri::VERSION
-  spec.authors       = ["GitLab Quality"]
-  spec.email         = ["quality+dri@gitlab.com"]
+  spec.authors       = ['GitLab Quality']
+  spec.email         = ['quality+dri@gitlab.com']
 
-  spec.summary       = %q{CLI app to help triage GitLab QA pipelines}
+  spec.summary       = 'CLI app to help triage GitLab QA pipelines'
   spec.homepage      = 'https://gitlab.com/gitlab-org/quality/dri'
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.7.0")
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.7.0')
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
+  spec.files = Dir.chdir(File.expand_path(__dir__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_dependency "tty-config", "~> 0.4.0"
-  spec.add_dependency "tty-font", "~> 0.5"
-  spec.add_dependency "tty-prompt", "~> 0.23.1"
-  spec.add_dependency "tty-spinner", "~> 0.9"
-  spec.add_dependency "tty-table", "~> 0.12.0"
+  spec.add_dependency 'httparty', '~> 0.20.0'
+  spec.add_dependency 'json', '~> 2.6.1'
+  spec.add_dependency 'markdown-tables', '~> 1.1.1'
+  spec.add_dependency 'pastel', '~> 0.8.0'
+  spec.add_dependency 'thor', '~> 1.0.1'
   spec.add_dependency 'tty-box', '~> 0.7.0'
+  spec.add_dependency 'tty-config', '~> 0.4.0'
+  spec.add_dependency 'tty-editor', '~> 0.6'
+  spec.add_dependency 'tty-font', '~> 0.5'
   spec.add_dependency 'tty-logger', '~> 0.6.0'
-  spec.add_dependency "tty-editor", "~> 0.6"
-  spec.add_dependency "pastel", "~> 0.8.0"
-  spec.add_dependency "thor", "~> 1.0.1"
-  spec.add_dependency "markdown-tables", "~> 1.1.1"
-  spec.add_dependency 'json', '~> 2.6.1'
-  spec.add_dependency 'httparty', '~> 0.20.0'
+  spec.add_dependency 'tty-prompt', '~> 0.23.1'
+  spec.add_dependency 'tty-spinner', '~> 0.9'
+  spec.add_dependency 'tty-table', '~> 0.12.0'
 
-  spec.add_development_dependency "rake"
+  spec.add_development_dependency 'gitlab-styles', '~> 7.0.0'
+  spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '~> 3.10.0'
-  spec.add_development_dependency 'webmock', '~> 3.5'
   spec.add_development_dependency "timecop", "~> 0.9.1"
+  spec.add_development_dependency 'webmock', '~> 3.5'
 end

data/exe/dri

@@ -2,7 +2,7 @@
 # frozen_string_literal: true
 
 lib_path = File.expand_path('../lib', __dir__)
-$:.unshift(lib_path) if !$:.include?(lib_path)
+$LOAD_PATH.unshift(lib_path) unless $LOAD_PATH.include?(lib_path)
 require 'dri/cli'
 
 Signal.trap('INT') do
@@ -12,7 +12,7 @@ end
 
 begin
   Dri::CLI.start
-rescue Dri::CLI::Error => err
-  puts "ERROR: #{err.message}"
+rescue Dri::CLI::Error => e
+  puts "ERROR: #{e.message}"
   exit 1
 end

data/lib/dri/api_client.rb

@@ -6,12 +6,13 @@ require "tty-config"
 require 'cgi'
 
 module Dri
-  class ApiClient
-
-    API_URL              = 'https://gitlab.com/api/v4'
-    TESTCASES_PROJECT_ID = '11229385'
-    TRIAGE_PROJECT_ID    = '15291320'
-    GITLAB_PROJECT_ID    = '278964'
+  class ApiClient # rubocop:disable Metrics/ClassLength
+    API_URL                     = 'https://gitlab.com/api/v4'
+    TESTCASES_PROJECT_ID        = '11229385'
+    TRIAGE_PROJECT_ID           = '15291320'
+    GITLAB_PROJECT_ID           = '278964'
+    FEATURE_FLAG_LOG_PROJECT_ID = '15208716'
+    INFRA_TEAM_PROD_PROJECT_ID  = '7444821'
 
     def initialize(config)
       profile = config.read
@@ -64,7 +65,9 @@ module Dri
     def fetch_mrs(project_id:, **options)
       uri = URI(API_URL)
       uri.query = HTTParty::HashConversions.to_params(options)
-      uri.path = File.join(uri.path, 'projects', CGI.escape(project_id), 'merge_requests') # CGI.escape('gitlab-org/gitlab') => 'gitlab-org%2Fgitlab'
+
+      # CGI.escape('gitlab-org/gitlab') => 'gitlab-org%2Fgitlab'
+      uri.path = File.join(uri.path, 'projects', CGI.escape(project_id), 'merge_requests')
 
       fetch_json(uri.to_s)
     end
@@ -90,6 +93,7 @@ module Dri
       url << "&order_by=updated_at&state=#{state}"
       url << "&scope=all"
       url << "&created_after=#{date}"
+      url << "&per_page=100"
 
       fetch_json(url.join)
     end
@@ -106,6 +110,25 @@ module Dri
       delete_json(url)
     end
 
+    def fetch_feature_flag_logs(date:, page:)
+      url = ["#{API_URL}/projects/"]
+      url << "#{FEATURE_FLAG_LOG_PROJECT_ID}/issues"
+      url << "?created_after=#{date}"
+      url << "&per_page=100"
+      url << "&page=#{page}"
+
+      fetch_json(url.join)
+    end
+
+    def incidents
+      url = ["#{API_URL}/projects/"]
+      url << "#{INFRA_TEAM_PROD_PROJECT_ID}/issues"
+      url << "?order_by=updated_at&state=opened"
+      url << "&labels=incident"
+
+      fetch_json(url.join)
+    end
+
     private
 
     def post_json(url, body)
@@ -116,7 +139,7 @@ module Dri
         }
       }
 
-      response =  HTTParty.post(url, options)
+      response = HTTParty.post(url, options)
       handle_response(response)
     end
 
@@ -126,12 +149,11 @@ module Dri
     end
 
     def handle_response(response)
-      #puts response.to_json
       response
     end
 
     def fetch_json(url)
-      response =  HTTParty.get(url, headers: header)
+      response = HTTParty.get(url, headers: header)
 
       if response.code != 200
         puts "Response error code \"#{response.code}\" - Unable to sync with GitLab"

data/lib/dri/cli.rb

@@ -25,14 +25,26 @@ module Dri
     end
 
     class_option :no_color, type: :boolean, default: false,
-                              desc: "Disable colorization in output"
+                            desc: "Disable colorization in output"
 
     desc 'version', 'dri version'
     def version
       require_relative 'version'
       puts "v#{Dri::VERSION}"
     end
-    map %w(--version -v) => :version
+    map %w[--version -v] => :version
+
+    desc 'incidents', 'View current incidents'
+    method_option :help, aliases: '-h', type: :boolean,
+                         desc: 'Display usage information'
+    def incidents(*)
+      if options[:help]
+        invoke :help, ['incidents']
+      else
+        require_relative 'commands/incidents'
+        Dri::Commands::Incidents.new(options).execute
+      end
+    end
 
     require_relative 'commands/rm'
     register Dri::Commands::Rm, 'rm', 'rm [SUBCOMMAND]', 'Remove triage-related items'

data/lib/dri/command.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative 'command'
 require_relative 'api_client'
 
@@ -13,7 +14,6 @@ module Dri
     extend Forwardable
 
     def_delegators :command, :run
-    attr_reader :config, :emoji, :token, :username, :timezone, :profile
 
     def pastel(**options)
       Pastel.new(**options)
@@ -56,10 +56,10 @@ module Dri
     end
 
     def verify_config_exists
-      if !config.exist?
-        logger.error "Oops, could not find a configuration. Try using #{add_color('dri init', :yellow)} first."
-        exit 1
-      end
+      return if config.exist?
+
+      logger.error "Oops, could not find a configuration. Try using #{add_color('dri init', :yellow)} first."
+      exit 1
     end
 
     def add_color(str, *color)
@@ -93,7 +93,7 @@ module Dri
     # @api public
     def command(**options)
       require 'tty-command'
-      TTY::Command.new(options)
+      TTY::Command.new(**options)
     end
 
     # The cursor movement
@@ -123,7 +123,7 @@ module Dri
     # @api public
     def prompt(**options)
       require 'tty-prompt'
-      TTY::Prompt.new(options)
+      TTY::Prompt.new(**options.merge(interrupt: :exit))
     end
   end
 end

data/lib/dri/commands/fetch/failures.rb

@@ -1,21 +1,28 @@
 # frozen_string_literal: true
 
 require_relative '../../command'
-
-require 'tty-table'
+require_relative '../../utils/table'
 
 module Dri
   module Commands
     class Fetch
       class Failures < Dri::Command
+        include Dri::Utils::Table
         using Refinements
 
+        SORT_BY_OPTIONS = {
+          title: 0,
+          triaged: 1,
+          author: 2,
+          url: 3
+        }.freeze
+
         def initialize(options)
           @options = options
           @today_iso_format = Time.now.strftime('%Y-%m-%dT00:00:00Z')
         end
 
-        def execute(input: $stdin, output: $stdout)
+        def execute(input: $stdin, output: $stdout) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity, Metrics/MethodLength
           verify_config_exists
 
           title = add_color('Title', :bright_yellow)
@@ -25,13 +32,12 @@ module Dri
 
           failures = []
           urgent = []
-          labels = [ title, triaged, author, url ]
+          labels = [title, triaged, author, url]
           triaged_counter = 0
 
           logger.info "Fetching today's failures..."
 
-          spinner.run do
-
+          spinner.run do # rubocop:disable Metrics/BlockLength
             response = api_client.fetch_failures(date: @today_iso_format, state: 'opened')
 
             if response.nil?
@@ -47,7 +53,7 @@ module Dri
               triaged = add_color('x', :red)
 
               emoji_awards = api_client.fetch_awarded_emojis(award_emoji_url).find do |e|
-                e['name'] == emoji && e['user']['username'] == username
+                (e['name'] == emoji) && (e['user']['username'] == username)
               end
 
               if emoji_awards
@@ -59,24 +65,28 @@ module Dri
                 labels = failure['labels']
 
                 labels.each do |label|
-                  if label.include? ('found:canary.gitlab.com' && 'found:canary.staging.gitlab.com')
+                  if label.include?('found:canary.gitlab.com' && 'found:canary.staging.gitlab.com')
                     urgent << [title, triaged, author, url]
                   end
                 end
               end
 
               failures << [title, triaged, author, url]
+
+              failures.sort_by! { |e| e[SORT_BY_OPTIONS[@options[:sort_by].to_sym]] } if @options[:sort_by]
             end
           end
 
           if @options[:urgent]
-            table = TTY::Table.new(labels, urgent)
-            puts table.render(:ascii, resize: true, alignments: [:center, :center, :center, :center])
-            output.puts "\nFound: #{urgent.size} urgent failures, occurring in both canary.gitlab.com and canary.staging.gitlab.com."
+            print_table(labels, urgent, alignments: [:left, :center, :center, :left])
+            output.puts(<<~MSG)
+              Found: #{urgent.size} urgent failures, occurring in both canary.gitlab.com and canary.staging.gitlab.com.
+            MSG
           else
-            table = TTY::Table.new(labels, failures)
-            puts table.render(:ascii, resize: true, alignments: [:center, :center, :center, :center])
-            output.puts "\nFound: #{failures.size} failures, of these #{triaged_counter} have been triaged with a #{emoji}."
+            print_table(labels, failures, alignments: [:left, :center, :center, :left])
+            output.puts(<<~MSG)
+              Found: #{failures.size} failures, of these #{triaged_counter} have been triaged with a #{emoji}.
+            MSG
           end
         end
       end