gitlab-exporter 17.0.0 → 17.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3cc0cadf1891c19066f388f384d51f47b9c489e8e0ee7c162565228fb661bada
-  data.tar.gz: 7ae7c0ac398b1832c58c2348297139fd88e3ff02e447669043990312160ed0a8
+  metadata.gz: 15b0fdb50984859832e6a21dcb76c31dfbfb68f28e5601f2c1389782ea4b5f2f
+  data.tar.gz: a02d8bda949c2d5f94981d50123ff845e73d958306d0bcf9e3ca0cfd4c8107e5
 SHA512:
-  metadata.gz: 7a358dbc90fe7bed11fc33a179fe2eb726e0d218925a56e7f076cc4c51d14272de4adca57cd5ae084e0a57d51feed0cf331b2dbf3ae67a5c0a2a972c4dba1863
-  data.tar.gz: c937591c8d81ad5296955bd5a55f92bcdd6f2ecb0f1fe7cdd36e9f429e88f5bbf2da8bce8375e8897fec29eda18ea0dd60fcd1bbfc5070c49e9b886a0faf3037
+  metadata.gz: ab40fc6975a31c6175cb3139dd1e576a06f7f1a6b598efc387f76228e68413a47b7902d7aca1682128ee77dd76ee4f2e178fec7bed8dc027676ec2e614289490
+  data.tar.gz: e749cdfd3a8db77f8a594ad7007db5dc919003defa0afb7c3691a461b78d0bcdf459e8bce6d5106809f0cdc567323a968fe1d8de88c990a2c5074339b6c38343

data/.gitlab-ci.yml

@@ -32,7 +32,7 @@ default:
      # to avoid https://github.com/rubygems/rubygems/issues/5380 use an
      # older Bundler version.
      case ${RUBY_VERSION} in
-       "3.0"|"3.1")
+       "3.1")
            export BUNDLER_VERSION="2.4.20"
            ;;
      esac
@@ -53,7 +53,7 @@ rspec:
   before_script: *before_scripts
   parallel:
     matrix:
-      - RUBY_VERSION: ["3.0", "3.1", "3.2", "3.3", "3.4"]
+      - RUBY_VERSION: ["3.1", "3.2", "3.3", "3.4", "4.0"]
 
 rspec_integration:
   script:
@@ -88,7 +88,7 @@ rspec_integration:
     REDIS_SENTINEL_PASSWORD: "my-sentinel-password"
   parallel:
     matrix:
-      - RUBY_VERSION: ["3.0", "3.1", "3.2", "3.3", "3.4"]
+      - RUBY_VERSION: ["3.1", "3.2", "3.3", "3.4", "4.0"]
 
 rubocop:
   script:

data/.rubocop.yml

@@ -1,7 +1,7 @@
 inherit_from: .rubocop_todo.yml
 
 AllCops:
-  TargetRubyVersion: 3.0
+  TargetRubyVersion: 3.1
   DisplayCopNames: true
 
 # Just use double quotes please

data/.ruby-version

@@ -1 +1 @@
-3.0.7
+3.1.7

data/AGENTS.md

@@ -0,0 +1,75 @@
+# gitlab-exporter — AI agent guidelines
+
+gitlab-exporter is a Prometheus web exporter (a Ruby gem) that collects GitLab
+production metrics via custom probes and exposes them over HTTP for a Prometheus
+scraper. See `README.md` for the probe catalog.
+
+## Project layout
+
+- `lib/gitlab_exporter/` — probes and the exporter runtime
+- `lib/gitlab_exporter/version.rb` — the gem version; bumping this on `master`
+  triggers a release (see below)
+- `bin/gitlab-exporter` — executable entry point
+- `spec/` — RSpec tests (`-t integration` marks integration specs that need Redis/PG)
+
+## Development commands
+
+```sh
+bundle install
+bundle exec rspec spec           # unit tests
+bundle exec rspec spec -t integration   # integration tests (needs Redis/Postgres)
+bundle exec rubocop              # lint
+```
+
+## Commit messages
+
+Follow the [GitLab commit message guidelines](https://docs.gitlab.com/development/contributing/merge_request_workflow/#commit-messages-guidelines).
+
+- Subject: imperative mood, capitalized, no trailing period, ≤ 72 chars
+  ("Add Zoekt probe", not "Added Zoekt probe").
+- Body: explain *why*, wrap at 72 chars. Reference issues/MRs with full URLs,
+  not short references (`#123`).
+
+## Changelog trailer (required for user-facing changes)
+
+Releases are cut by the `gem-release` CI component, which builds the release
+notes from GitLab's changelog generator. That generator **only includes commits
+that carry a `Changelog:` git trailer** — commits without one are dropped, which
+is why a release can end up saying "No changes." and produce an empty changelog
+in downstream projects (e.g. omnibus-gitlab's Renovate MRs).
+
+Add the trailer as a **separate block at the end of the commit message**,
+preceded by a blank line (the blank line is required for git to parse it as a
+trailer; without it the changelog generator silently drops the commit):
+
+```
+Changelog: <type>
+```
+
+Valid types: `added`, `fixed`, `changed`, `deprecated`, `removed`, `security`,
+`performance`, `other`.
+
+**Include a trailer for:** new or changed probes/metrics, behavior changes,
+bug fixes, dependency bumps that affect users, security fixes.
+
+**Skip the trailer for:** test-only changes, documentation-only changes, CI/
+tooling-only changes, and pure internal refactors with no user-visible effect.
+
+Example:
+
+```
+Add remote mirror staleness metric
+
+Exposes project_remote_mirror_last_successful_update_time_seconds so
+operators can alert on mirrors that have stopped syncing.
+
+Changelog: added
+```
+
+## Releasing
+
+A release happens when `lib/gitlab_exporter/version.rb` changes on `master`.
+The CI pipeline then generates the changelog from the `Changelog:` trailers
+since the previous tag, creates the GitLab Release, and publishes the gem to
+RubyGems. Keeping trailers accurate on every merged commit is what makes those
+release notes meaningful.

data/CLAUDE.md

@@ -0,0 +1 @@
+@AGENTS.md

data/Gemfile.lock

@@ -1,8 +1,8 @@
 PATH
   remote: .
   specs:
-    gitlab-exporter (17.0.0)
-      base64 (= 0.2.0)
+    gitlab-exporter (17.0.1)
+      base64 (= 0.3.0)
       connection_pool (= 2.5.5)
       deep_merge (~> 1.2.2)
       faraday (= 2.14.3)
@@ -17,7 +17,7 @@ GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.3)
-    base64 (0.2.0)
+    base64 (0.3.0)
     connection_pool (2.5.5)
     deep_merge (1.2.2)
     diff-lcs (1.5.0)
@@ -57,11 +57,11 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.12.0)
     rspec-support (3.12.0)
-    rubocop (1.84.2)
+    rubocop (1.89.0)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.1.0)
-      parallel (~> 1.10)
+      parallel (>= 1.10)
       parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 2.9.3, < 3.0)
@@ -78,7 +78,7 @@ GEM
       redis (>= 4.5.0, < 5)
     unicode-display_width (3.1.4)
       unicode-emoji (~> 4.0, >= 4.0.4)
-    unicode-emoji (4.0.4)
+    unicode-emoji (4.2.0)
     webrick (1.9.1)
 
 PLATFORMS
@@ -91,4 +91,4 @@ DEPENDENCIES
   rubocop (~> 1.0)
 
 BUNDLED WITH
-   2.7.1
+  4.0.10

data/gitlab-exporter.gemspec

@@ -20,12 +20,12 @@ Gem::Specification.new do |s|
   s.homepage      = "https://gitlab.com/gitlab-org/ruby/gems/gitlab-exporter"
   s.license       = "MIT"
 
-  s.required_ruby_version = Gem::Requirement.new(">= 3.0")
+  s.required_ruby_version = Gem::Requirement.new(">= 3.1")
 
   # base64 is required by sidekiq but not declared by it, and is no longer a
   # default gem as of Ruby 3.4. It used to be pulled in transitively via
   # sinatra -> rack-protection; declare it directly now that those are gone.
-  s.add_runtime_dependency "base64", "0.2.0"
+  s.add_runtime_dependency "base64", "0.3.0"
   s.add_runtime_dependency "connection_pool", "2.5.5"
   s.add_runtime_dependency "deep_merge", "~> 1.2.2"
   s.add_runtime_dependency "faraday", "2.14.3"

data/lib/gitlab_exporter/database/tuple_stats.rb

@@ -35,8 +35,6 @@ module GitLab
 
           result.each do |table_name, tuple_stats|
             tuple_stats.each do |column_name, value|
-              next if value.is_a?(Numeric)
-
               @metrics.add("gitlab_database_stat_table_#{column_name}",
                            value.to_f,
                            table_name: table_name)

data/lib/gitlab_exporter/version.rb

@@ -1,5 +1,5 @@
 module GitLab
   module Exporter
-    VERSION = "17.0.0".freeze
+    VERSION = "17.0.1".freeze
   end
 end

data/spec/database/tuple_stats_spec.rb

@@ -0,0 +1,40 @@
+require "spec_helper"
+require "gitlab_exporter/database/tuple_stats"
+
+describe GitLab::Exporter::Database::TuplesProber do
+  let(:opts) { { connection_string: "" } }
+  let(:metrics) { double("PrometheusMetrics", add: nil) }
+  let(:collector) { double("TupleStatsCollector", run: data) }
+
+  before do
+    allow(GitLab::Exporter::Database::TupleStatsCollector).to receive(:new).and_return(collector)
+  end
+
+  describe "#probe_db" do
+    subject { described_class.new(metrics: metrics, **opts).probe_db }
+
+    context "when the connection casts columns to native Integer (PG::BasicTypeMapForResults)" do
+      let(:data) { { "projects" => { "seq_tup_read" => 42, "n_dead_tup" => 0 } } }
+
+      it "still records a metric for every column" do
+        expect(metrics).to receive(:add)
+          .with("gitlab_database_stat_table_seq_tup_read", 42.0, table_name: "projects")
+        expect(metrics).to receive(:add)
+          .with("gitlab_database_stat_table_n_dead_tup", 0.0, table_name: "projects")
+
+        subject
+      end
+    end
+
+    context "when the connection returns unmapped string values" do
+      let(:data) { { "projects" => { "seq_tup_read" => "42" } } }
+
+      it "records a metric" do
+        expect(metrics).to receive(:add)
+          .with("gitlab_database_stat_table_seq_tup_read", 42.0, table_name: "projects")
+
+        subject
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-exporter
 version: !ruby/object:Gem::Version
-  version: 17.0.0
+  version: 17.0.1
 platform: ruby
 authors:
 - Pablo Carranza
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
 - !ruby/object:Gem::Dependency
   name: connection_pool
   requirement: !ruby/object:Gem::Requirement
@@ -191,6 +191,8 @@ files:
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
 - ".ruby-version"
+- AGENTS.md
+- CLAUDE.md
 - CONTRIBUTING.md
 - Gemfile
 - Gemfile.lock
@@ -230,6 +232,7 @@ files:
 - spec/database/ci_builds_spec.rb
 - spec/database/pg_sequences_spec.rb
 - spec/database/row_count_spec.rb
+- spec/database/tuple_stats_spec.rb
 - spec/database/zoekt_spec.rb
 - spec/elasticsearch_spec.rb
 - spec/fixtures/config.yml
@@ -256,7 +259,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.0'
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -273,6 +276,7 @@ test_files:
 - spec/database/ci_builds_spec.rb
 - spec/database/pg_sequences_spec.rb
 - spec/database/row_count_spec.rb
+- spec/database/tuple_stats_spec.rb
 - spec/database/zoekt_spec.rb
 - spec/elasticsearch_spec.rb
 - spec/fixtures/config.yml