fbe 0.36.1 → 0.38.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b06cd95d05dd6dfa4e79d2c3e128bb3c8bae254dae8abe5842cc59e12adf26f5
-  data.tar.gz: 626b6720065bfdf41480b3728545963fc2477e5baa7d6e07f404a396b52df9ca
+  metadata.gz: 04d71d826c4bd4c4ee4a8b22f04561ca31cc997f4e398f129e7d792d65a898ef
+  data.tar.gz: 7a95d256e71402d57fcf2e5f47d77a70c1edb4112bd961344b879470ededfe77
 SHA512:
-  metadata.gz: 9df729eb9a4e4733ccf08c68fa289c2c05da96e3360a285e7432bc7e2ba0a1cbce56e74b5b1c34dc12023eca1395b045af9ad2f8698ce63a7d47ad5ffdd1273e
-  data.tar.gz: 24fc078561ad2d0a6169a2aebf965325497f8feca78446fcc6c4f6890c3e685c3bae9ba9c2da049dbdd8090b4363ab9b4acdbfa476c8c1a9f6222d6cf7b28910
+  metadata.gz: 2e21419e49aeebc86275f380df70423c030fad50c83626d8b45a28716589d75047fce9e8ee0ea81b5c5b6dd6bf7324a1f0d07625d28c750bef3de1d3324a4e28
+  data.tar.gz: e8c45307769eea587953ecb193c2516da44020007a5b8e0c45516f8f71a78100758e38fdc3224c707f50e95459ad37695a0848b7129dae868be2989b4351a5a7

data/Gemfile.lock

@@ -76,7 +76,7 @@ GEM
     ellipsized (0.3.0)
     ethon (0.15.0)
       ffi (>= 1.15.0)
-    factbase (0.16.5)
+    factbase (0.16.7)
       backtrace (~> 0.4)
       decoor (~> 0.1)
       ellipsized (~> 0.3)
@@ -262,6 +262,7 @@ PLATFORMS
   x64-mingw-ucrt
   x86_64-darwin-20
   x86_64-darwin-21
+  x86_64-darwin-23
   x86_64-linux
 
 DEPENDENCIES

data/lib/fbe/conclude.rb

@@ -16,17 +16,19 @@ require_relative 'octo'
 # @param [Hash] global The hash for global caching
 # @param [Judges::Options] options The options coming from the +judges+ tool
 # @param [Loog] loog The logging facility
+# @param [Time] epoch When the entire update started
+# @param [Time] kickoff When the particular judge started
 # @yield [Factbase::Fact] The fact
 def Fbe.conclude(
   fb: Fbe.fb, judge: $judge, loog: $loog, options: $options, global: $global,
-  start: $start, &
+  epoch: $epoch || Time.now, kickoff: $kickoff || Time.now, &
 )
   raise 'The fb is nil' if fb.nil?
   raise 'The $judge is not set' if judge.nil?
   raise 'The $global is not set' if global.nil?
   raise 'The $options is not set' if options.nil?
   raise 'The $loog is not set' if loog.nil?
-  c = Fbe::Conclude.new(fb:, judge:, loog:, options:, global:, start:)
+  c = Fbe::Conclude.new(fb:, judge:, loog:, options:, global:, epoch:, kickoff:)
   c.instance_eval(&)
 end
 
@@ -61,13 +63,16 @@ class Fbe::Conclude
   # @param [Hash] global The hash for global caching
   # @param [Judges::Options] options The options coming from the +judges+ tool
   # @param [Loog] loog The logging facility
-  def initialize(fb:, judge:, global:, options:, loog:, start:)
+  # @param [Time] epoch When the entire update started
+  # @param [Time] kickoff When the particular judge started
+  def initialize(fb:, judge:, global:, options:, loog:, epoch:, kickoff:)
     @fb = fb
     @judge = judge
     @loog = loog
     @options = options
     @global = global
-    @start = start
+    @epoch = epoch
+    @kickoff = kickoff
     @query = nil
     @follows = []
     @lifetime_aware = true
@@ -192,18 +197,17 @@ class Fbe::Conclude
   def roll(&)
     passed = 0
     oct = Fbe.octo(loog: @loog, options: @options, global: @global)
-    started = Time.now
     @fb.query(@query).each do |a|
       if @quota_aware && oct.off_quota?
         @loog.info('We ran out of GitHub quota, must stop here')
         break
       end
-      if @lifetime_aware && @options.lifetime && Time.now - @start > @options.lifetime - 10
-        @loog.debug("We ran out of lifetime (#{@start.ago} already), must stop here")
+      if @lifetime_aware && @options.lifetime && Time.now - @epoch > @options.lifetime * 0.9
+        @loog.debug("We ran out of lifetime (#{@epoch.ago} already), must stop here")
         break
       end
-      if @timeout_aware && @options.timeout && Time.now - started > @options.timeout - 5
-        @loog.debug("We've spent more than #{started.ago}, must stop here")
+      if @timeout_aware && @options.timeout && Time.now - @kickoff > @options.timeout * 0.9
+        @loog.debug("We've spent more than #{kickoff.ago}, must stop here")
         break
       end
       @fb.txn do |fbt|

data/lib/fbe/consider.rb

@@ -15,13 +15,15 @@ require_relative 'fb'
 # @param [Hash] global The hash for global caching
 # @param [Judges::Options] options The options coming from the +judges+ tool
 # @param [Loog] loog The logging facility
+# @param [Time] epoch When the entire update started
+# @param [Time] kickoff When the particular judge started
 # @yield [Factbase::Fact] The fact
 def Fbe.consider(
   query,
   fb: Fbe.fb, judge: $judge, loog: $loog, options: $options, global: $global,
-  start: $start, &
+  epoch: $epoch || Time.now, kickoff: $kickoff || Time.now, &
 )
-  Fbe.conclude(fb:, judge:, loog:, options:, global:, start:) do
+  Fbe.conclude(fb:, judge:, loog:, options:, global:, epoch:, kickoff:) do
     on query
     consider(&)
   end

data/lib/fbe/iterate.rb

@@ -37,12 +37,15 @@ require_relative 'unmask_repos'
 #       issue_id + 1
 #     end
 #   end
-def Fbe.iterate(fb: Fbe.fb, loog: $loog, options: $options, global: $global, start: $start, &)
+def Fbe.iterate(
+  fb: Fbe.fb, loog: $loog, options: $options, global: $global,
+  epoch: $epoch || Time.now, kickoff: $kickoff || Time.now, &
+)
   raise 'The fb is nil' if fb.nil?
   raise 'The $global is not set' if global.nil?
   raise 'The $options is not set' if options.nil?
   raise 'The $loog is not set' if loog.nil?
-  c = Fbe::Iterate.new(fb:, loog:, options:, global:, start:)
+  c = Fbe::Iterate.new(fb:, loog:, options:, global:, epoch:, kickoff:)
   c.instance_eval(&)
 end
 
@@ -84,12 +87,13 @@ class Fbe::Iterate
   # @param [Loog] loog The logging facility for debug output
   # @param [Judges::Options] options The options containing repository configuration
   # @param [Hash] global The hash for global caching of API responses
-  def initialize(fb:, loog:, options:, global:, start:)
+  def initialize(fb:, loog:, options:, global:, epoch:, kickoff:)
     @fb = fb
     @loog = loog
     @options = options
     @global = global
-    @start = start
+    @epoch = epoch
+    @kickoff = kickoff
     @label = nil
     @since = 0
     @query = nil
@@ -227,7 +231,6 @@ class Fbe::Iterate
       loog: @loog, options: @options, global: @global, quota_aware: @quota_aware
     ).map { |n| oct.repo_id_by_name(n) }
     restarted = []
-    started = Time.now
     before =
       repos.to_h do |repo|
         [
@@ -247,12 +250,12 @@ class Fbe::Iterate
         @loog.info("We are off GitHub quota, time to stop after #{started.ago}")
         break
       end
-      if @lifetime_aware && @options.lifetime && Time.now - @start > @options.lifetime - 10
-        @loog.debug("We ran out of lifetime (#{@start.ago} already), must stop here")
+      if @lifetime_aware && @options.lifetime && Time.now - @epoch > @options.lifetime * 0.9
+        @loog.debug("We ran out of lifetime (#{@epoch.ago} already), must stop here")
         break
       end
-      if @timeout_aware && @options.timeout && Time.now - started > @options.timeout - 5
-        @loog.debug("We've spent more than #{started.ago}, must stop here")
+      if @timeout_aware && @options.timeout && Time.now - @kickoff > @options.timeout * 0.9
+        @loog.debug("We've spent more than #{@kickoff.ago}, must stop here")
         break
       end
       repos.each do |repo|
@@ -260,12 +263,12 @@ class Fbe::Iterate
           @loog.info("We are off GitHub quota, we must skip #{repo}")
           break
         end
-        if @lifetime_aware && @options.lifetime && Time.now - @start > @options.lifetime - 10
-          @loog.info("We are working for #{@start.ago} already, won't check repository ##{repo}")
+        if @lifetime_aware && @options.lifetime && Time.now - @epoch > @options.lifetime * 0.9
+          @loog.info("We are working for #{@epoch.ago} already, won't check repository ##{repo}")
           next
         end
-        if @timeout_aware && @options.timeout && Time.now - started > @options.timeout - 5
-          @loog.debug("We've spent more than #{started.ago}, won't check repository ##{repo}")
+        if @timeout_aware && @options.timeout && Time.now - @kickoff > @options.timeout * 0.9
+          @loog.debug("We've spent more than #{@kickoff.ago}, won't check repository ##{repo}")
           break
         end
         next if restarted.include?(repo)
@@ -290,11 +293,11 @@ class Fbe::Iterate
         seen[repo] += 1
       end
       unless seen.any? { |r, v| v < @repeats && !restarted.include?(r) }
-        @loog.debug("No more repos to scan (out of #{repos.size}), quitting after #{started.ago}")
+        @loog.debug("No more repos to scan (out of #{repos.size}), quitting after #{@kickoff.ago}")
         break
       end
       if restarted.size == repos.size
-        @loog.debug("All #{repos.size} repos restarted, quitting after #{started.ago}")
+        @loog.debug("All #{repos.size} repos restarted, quitting after #{@kickoff.ago}")
         break
       end
     end
@@ -308,6 +311,6 @@ class Fbe::Iterate
         end
       Fbe.overwrite(f, @label, before[repo], fb: @fb)
     end
-    @loog.debug("Finished scanning #{repos.size} repos in #{started.ago}: #{seen.map { |k, v| "#{k}:#{v}" }.joined}")
+    @loog.debug("Finished scanning #{repos.size} repos in #{@kickoff.ago}: #{seen.map { |k, v| "#{k}:#{v}" }.joined}")
   end
 end

data/lib/fbe/octo.rb

@@ -44,6 +44,7 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
   raise 'The $loog is not set' if loog.nil?
   global[:octo] ||=
     begin
+      loog.info("Fbe version is #{Fbe::VERSION}")
       trace = []
       if options.testing.nil?
         o = Octokit::Client.new

data/lib/fbe/overwrite.rb

@@ -16,8 +16,8 @@ require_relative 'fb'
 # an exception will be raised.
 #
 # @param [Factbase::Fact] fact The fact to modify (must have _id property)
-# @param [String] property The name of the property to set
-# @param [Any] values The value to set (can be any type, including array)
+# @param [String, Hash] property_or_hash The name of the property to set, or a hash of properties
+# @param [Any] values The value to set (can be any type, including array) - ignored if first param is Hash
 # @param [Factbase] fb The factbase to use (defaults to Fbe.fb)
 # @return [nil] Nothing
 # @raise [RuntimeError] If fact is nil, has no _id, or property is not a String
@@ -25,11 +25,39 @@ require_relative 'fb'
 # @note If property already has the same single value, no changes are made
 # @example Update a user's status
 #   user = fb.query('(eq login "john")').first
-#   updated_user = Fbe.overwrite(user, 'status', 'active')
+#   Fbe.overwrite(user, 'status', 'active')
 #   # All properties preserved, only 'status' is set to 'active'
-def Fbe.overwrite(fact, property, values, fb: Fbe.fb, fid: '_id')
+# @example Update multiple properties at once
+#   user = fb.query('(eq login "john")').first
+#   Fbe.overwrite(user, status: 'active', role: 'admin')
+#   # All properties preserved, 'status' and 'role' are updated
+def Fbe.overwrite(fact, property_or_hash, values = nil, fb: Fbe.fb, fid: '_id')
   raise 'The fact is nil' if fact.nil?
   raise 'The fb is nil' if fb.nil?
+  if property_or_hash.is_a?(Hash)
+    before = {}
+    fact.all_properties.each do |k|
+      before[k.to_s] = fact[k]
+    end
+    property_or_hash.each do |k, v|
+      raise "The value for #{k} is nil" if v.nil?
+      before[k.to_s] = v.is_a?(Array) ? v : [v]
+    end
+    id = fact[fid]&.first
+    raise "There is no #{fid} in the fact, cannot use Fbe.overwrite" if id.nil?
+    raise "No facts by #{fid} = #{id}" if fb.query("(eq #{fid} #{id})").delete!.zero?
+    fb.txn do |fbt|
+      n = fbt.insert
+      before.each do |k, vv|
+        next unless n[k].nil?
+        vv.each do |v|
+          n.send(:"#{k}=", v)
+        end
+      end
+    end
+    return
+  end
+  property = property_or_hash
   raise "The property is not a String but #{property.class} (#{property})" unless property.is_a?(String)
   raise 'The values is nil' if values.nil?
   values = [values] unless values.is_a?(Array)
@@ -41,8 +69,8 @@ def Fbe.overwrite(fact, property, values, fb: Fbe.fb, fid: '_id')
     return
   end
   before = {}
-  fact.all_properties.each do |prop|
-    before[prop.to_s] = fact[prop]
+  fact.all_properties.each do |k|
+    before[k.to_s] = fact[k]
   end
   id = fact[fid]&.first
   raise "There is no #{fid} in the fact, cannot use Fbe.overwrite" if id.nil?

data/lib/fbe/unmask_repos.rb

@@ -49,16 +49,18 @@ end
 # @param [Judges::Options] options Options containing 'repositories' field with masks
 # @param [Hash] global Global cache for storing API responses
 # @param [Loog] loog Logger for debug output
-# @param [Time] start When did we start the entire pipeline
+# @param [Time] epoch When the entire update started
+# @param [Time] kickoff When the particular judge started
 # @param [quota_aware] Boolean Should we stop if quota is off?
 # @param [lifetime_aware] Boolean Should we stop if lifetime is over?
+# @param [timeout_aware] Boolean Should we stop if timeout is over?
 # @return [Array<String>] Shuffled list of repository full names (e.g., 'org/repo')
 # @raise [RuntimeError] If no repositories match the provided masks
 # @note Exclusion patterns must start with '-' (e.g., '-org/pattern*')
 # @note Results are shuffled to distribute load when processing
 def Fbe.unmask_repos(
-  options: $options, global: $global, loog: $loog, start: $start,
-  quota_aware: true, lifetime_aware: true
+  options: $options, global: $global, loog: $loog, epoch: $epoch || Time.now, kickoff: $kickoff || Time.now,
+  quota_aware: true, lifetime_aware: true, timeout_aware: true
 )
   raise 'Repositories mask is not specified' unless options.repositories
   raise 'Repositories mask is empty' if options.repositories.empty?
@@ -90,7 +92,11 @@ def Fbe.unmask_repos(
       loog.info("No GitHub quota left, it is time to stop at #{repo}")
       break
     end
-    if lifetime_aware && options.lifetime && Time.now - start > options.lifetime - 10
+    if lifetime_aware && options.lifetime && Time.now - epoch > options.lifetime * 0.9
+      loog.info("No time left, it is time to stop at #{repo}")
+      break
+    end
+    if timeout_aware && options.timeout && Time.now - kickoff > options.timeout * 0.9
       loog.info("No time left, it is time to stop at #{repo}")
       break
     end

data/lib/fbe.rb

@@ -10,5 +10,5 @@
 # License:: MIT
 module Fbe
   # Current version of the gem (changed by +.rultor.yml+ on every release)
-  VERSION = '0.36.1' unless const_defined?(:VERSION)
+  VERSION = '0.38.0' unless const_defined?(:VERSION)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fbe
 version: !ruby/object:Gem::Version
-  version: 0.36.1
+  version: 0.38.0
 platform: ruby
 authors:
 - Yegor Bugayenko