X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/435a5df3e505dfbf67467bd02073f97e63c4c61d..502005c3002c9ee9f07b36ac4a5aa370aa056c50:/services/api/lib/update_priority.rb?ds=sidebyside

diff --git a/services/api/lib/update_priority.rb b/services/api/lib/update_priority.rb
index 8bf3e150d3..c688ac008b 100644
--- a/services/api/lib/update_priority.rb
+++ b/services/api/lib/update_priority.rb
@@ -12,6 +12,11 @@ module UpdatePriority
   #
   # If container priority=0 but there are committed container requests
   # for it with priority>0, update priority.
+  #
+  # Normally, update_priority is a no-op if another thread/process is
+  # already updating. Test cases that need to check priorities after
+  # updating can force a (possibly overlapping) update in the current
+  # thread/transaction by setting the "nolock" flag. See #14878.
   def self.update_priority(nolock: false)
     if !File.owned?(Rails.root.join('tmp'))
       Rails.logger.warn("UpdatePriority: not owner of #{Rails.root}/tmp, skipping")