gitlab_quality-test_tooling 3.21.2 → 3.21.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0ae21eb1d647c738006ae06faef24f5268d61fe30d0b72bd9aeebaa6b4b7d93e
-  data.tar.gz: 934cd5bd6d0bdac314f7209d9fa212dafa4b14ec1daffb1f0dff77106bcbd0b0
+  metadata.gz: 200252eb5e20a931ca64bd835400572bc3f059428834a377fce3edce25cdfbfc
+  data.tar.gz: aa0a9707d616d5c8a86736ddada7f65721ffc7ea7f37b80ab69c41c1e62ec5c3
 SHA512:
-  metadata.gz: 0b2c9ab4651163565cda4e146487bfa7ffec5b9e4d47a289431d4427926e35a8b92d81f759f6838169b2d2572a5fcac6c7c48fc9250cd572219f77a60f96e5aa
-  data.tar.gz: 69479474c04e85f4ba6da8748af8d58fb7b85e7175ddae13c524fd2303f5e324af9c9d1dd9e07220509c52e107cb155545d0515ae4e49b9d75cb77df855a5a9b
+  metadata.gz: 439bd19bf631d9f9416acbefd77ae5ebad551ff2a06840527c2b5af0072d481b98cf89a42565e98165f2b78a2bfcf484f22826a202ce3b8f7cdaf4e19e48db14
+  data.tar.gz: 1af8b9ed7eee499fb9c9ac09d2a697da53e3d608944799070c7ec40e5078ff4c96bc7cf15b6487cedc7e3402e799c57f3448ebcbc71feb988a28184cdbc7fef6

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (3.21.2)
+    gitlab_quality-test_tooling (3.21.3)
       activesupport (>= 7.0)
       amatch (~> 0.4.1)
       fog-google (~> 1.25)

data/lib/gitlab_quality/test_tooling/code_coverage/click_house/category_owners_table.rb

@@ -15,8 +15,10 @@ module GitlabQuality
 
           KNOWN_UNOWNED = %w[shared not_owned tooling].freeze
 
-          # SQL query to get the latest ownership record for each unique category+ownership combination
-          # Partitions by the full composite key to handle cases where a category has multiple ownerships
+          # Every owner a category has ever had, including ones it has moved off, one row per
+          # distinct owner and in no particular order. Only `push` reads this, to skip rows it
+          # has already stored. Deduping to one row per category here would make it re-insert
+          # old owners on every run.
           LATEST_RECORDS_QUERY = <<~SQL
             SELECT category, group, stage, section
             FROM (
@@ -27,6 +29,18 @@ module GitlabQuality
             WHERE rn = 1
           SQL
 
+          # The current owner of each category, one row per category, which is what `owners`
+          # and `owner_records` serve to callers. The table keeps every past owner as well, so
+          # the newest timestamp is what picks the current one. The owner columns after the
+          # timestamp in the ORDER BY break ties, so two rows sharing a category's newest
+          # timestamp always resolve the same way.
+          LATEST_OWNERS_QUERY = <<~SQL
+            SELECT category, group, stage, section
+            FROM %{table_name}
+            ORDER BY category, timestamp DESC, group, stage, section
+            LIMIT 1 BY category
+          SQL
+
           # Insert only new category ownership records that don't already exist
           # This avoids needing TRUNCATE permission
           def push(data)
@@ -71,7 +85,7 @@ module GitlabQuality
           private
 
           def records
-            @records ||= fetch_latest_records.each_with_object({}) do |record, hsh|
+            @records ||= fetch_latest_owners.each_with_object({}) do |record, hsh|
               hsh[record["category"]] = record.slice("group", "stage", "section")
             end
           end
@@ -114,6 +128,11 @@ module GitlabQuality
             client.query(query)
           end
 
+          def fetch_latest_owners
+            query = format(LATEST_OWNERS_QUERY, table_name: table_name)
+            client.query(query)
+          end
+
           def sanitized_data_record(record)
             {
               timestamp: time,

data/lib/gitlab_quality/test_tooling/version.rb

@@ -2,6 +2,6 @@
 
 module GitlabQuality
   module TestTooling
-    VERSION = "3.21.2"
+    VERSION = "3.21.3"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_quality-test_tooling
 version: !ruby/object:Gem::Version
-  version: 3.21.2
+  version: 3.21.3
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-09-01 00:00:00.000000000 Z
+date: 2026-09-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control