[ARVADOS] updated: 2.1.0-1767-g0c14b79d0

Git user git at public.arvados.org
Wed Dec 29 03:03:32 UTC 2021


Summary of changes:
 apps/workbench/Gemfile      | 1 -
 apps/workbench/Gemfile.lock | 4 ----
 2 files changed, 5 deletions(-)

  discards  2ae3a644ffd385ca42e1e297e5e874e5e425c16d (commit)
  discards  cd2a2d5ead6c6405dca39ed157fbdce77e3d88a7 (commit)
       via  0c14b79d003d6e1fda00cea3dcbdfca3b6d31014 (commit)

This update added new revisions after undoing existing revisions.  That is
to say, the old revision is not a strict subset of the new revision.  This
situation occurs when you --force push a change and generate a repository
containing something like this:

 * -- * -- B -- O -- O -- O (2ae3a644ffd385ca42e1e297e5e874e5e425c16d)
            \
             N -- N -- N (0c14b79d003d6e1fda00cea3dcbdfca3b6d31014)

When this happens we assume that you've already had alert emails for all
of the O revisions, and so we here report only the revisions in the N
branch from the common base, B.

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 0c14b79d003d6e1fda00cea3dcbdfca3b6d31014
Author: Ward Vandewege <ward at curii.com>
Date:   Tue Dec 28 21:41:54 2021 -0500

    17417: remove the coffee-rails gem dependency in wb1, and convert the
           two files where we used it to straight up javascript.
    
    Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <ward at curii.com>

diff --git a/apps/workbench/Gemfile b/apps/workbench/Gemfile
index 8005cb321..00dbad086 100644
--- a/apps/workbench/Gemfile
+++ b/apps/workbench/Gemfile
@@ -18,13 +18,6 @@ gem 'responders', '~> 2.0'
 # See: https://github.com/rails/sprockets-rails/issues/443
 gem 'sprockets', '~> 3.0'
 
-# Note: keeping this out of the "group :assets" section "may" allow us
-# to use Coffescript for UJS responses. It also prevents a
-# warning/problem when running tests: "WARN: tilt autoloading
-# 'coffee_script' in a non thread-safe way; explicit require
-# 'coffee_script' suggested."
-gem 'coffee-rails'
-
 # Gems used only for assets and not required
 # in production environments by default.
 group :assets do
diff --git a/apps/workbench/Gemfile.lock b/apps/workbench/Gemfile.lock
index 12e14518e..b82568e7d 100644
--- a/apps/workbench/Gemfile.lock
+++ b/apps/workbench/Gemfile.lock
@@ -111,13 +111,6 @@ GEM
     childprocess (0.9.0)
       ffi (~> 1.0, >= 1.0.11)
     cliver (0.3.2)
-    coffee-rails (4.2.2)
-      coffee-script (>= 2.2.0)
-      railties (>= 4.0.0)
-    coffee-script (2.4.1)
-      coffee-script-source
-      execjs
-    coffee-script-source (1.12.2)
     concurrent-ruby (1.1.9)
     crass (1.0.6)
     deep_merge (1.2.1)
@@ -323,7 +316,6 @@ DEPENDENCIES
   bootstrap-x-editable-rails
   byebug
   capybara (~> 2.5.0)
-  coffee-rails
   deep_merge
   flamegraph
   headless (~> 1.0.2)
diff --git a/apps/workbench/app/assets/javascripts/bootstrap.js b/apps/workbench/app/assets/javascripts/bootstrap.js
new file mode 100644
index 000000000..e315ab535
--- /dev/null
+++ b/apps/workbench/app/assets/javascripts/bootstrap.js
@@ -0,0 +1,11 @@
+// Copyright (C) The Arvados Authors. All rights reserved.
+//
+// SPDX-License-Identifier: AGPL-3.0
+
+(function() {
+  jQuery(function() {
+    $("a[rel=popover]").popover();
+    $(".tooltip").tooltip();
+    return $("a[rel=tooltip]").tooltip();
+  });
+}).call(this);
diff --git a/apps/workbench/app/assets/javascripts/bootstrap.js.coffee b/apps/workbench/app/assets/javascripts/bootstrap.js.coffee
deleted file mode 100644
index 441d77fb8..000000000
--- a/apps/workbench/app/assets/javascripts/bootstrap.js.coffee
+++ /dev/null
@@ -1,8 +0,0 @@
-### Copyright (C) The Arvados Authors. All rights reserved.
-
-SPDX-License-Identifier: AGPL-3.0 ###
-
-jQuery ->
-  $("a[rel=popover]").popover()
-  $(".tooltip").tooltip()
-  $("a[rel=tooltip]").tooltip()
diff --git a/apps/workbench/app/assets/javascripts/keep_disks.js b/apps/workbench/app/assets/javascripts/keep_disks.js
new file mode 100644
index 000000000..b3fb6dc44
--- /dev/null
+++ b/apps/workbench/app/assets/javascripts/keep_disks.js
@@ -0,0 +1,43 @@
+// Copyright (C) The Arvados Authors. All rights reserved.
+//
+// SPDX-License-Identifier: AGPL-3.0
+
+(function() {
+  var cache_age_axis_label, cache_age_hover, cache_age_in_days, float_as_percentage;
+
+  cache_age_in_days = function(milliseconds_age) {
+    var ONE_DAY;
+    ONE_DAY = 1000 * 60 * 60 * 24;
+    return milliseconds_age / ONE_DAY;
+  };
+
+  cache_age_hover = function(milliseconds_age) {
+    return 'Cache age ' + cache_age_in_days(milliseconds_age).toFixed(1) + ' days.';
+  };
+
+  cache_age_axis_label = function(milliseconds_age) {
+    return cache_age_in_days(milliseconds_age).toFixed(0) + ' days';
+  };
+
+  float_as_percentage = function(proportion) {
+    return (proportion.toFixed(4) * 100) + '%';
+  };
+
+  $.renderHistogram = function(histogram_data) {
+    return Morris.Area({
+      element: 'cache-age-vs-disk-histogram',
+      pointSize: 0,
+      lineWidth: 0,
+      data: histogram_data,
+      xkey: 'age',
+      ykeys: ['persisted', 'cache'],
+      labels: ['Persisted Storage Disk Utilization', 'Cached Storage Disk Utilization'],
+      ymax: 1,
+      ymin: 0,
+      xLabelFormat: cache_age_axis_label,
+      yLabelFormat: float_as_percentage,
+      dateFormat: cache_age_hover
+    });
+  };
+
+}).call(this);
diff --git a/apps/workbench/app/assets/javascripts/keep_disks.js.coffee b/apps/workbench/app/assets/javascripts/keep_disks.js.coffee
deleted file mode 100644
index d33312d21..000000000
--- a/apps/workbench/app/assets/javascripts/keep_disks.js.coffee
+++ /dev/null
@@ -1,32 +0,0 @@
-### Copyright (C) The Arvados Authors. All rights reserved.
-
-SPDX-License-Identifier: AGPL-3.0 ###
-
-cache_age_in_days = (milliseconds_age) ->
-  ONE_DAY = 1000 * 60 * 60 * 24
-  milliseconds_age / ONE_DAY
-
-cache_age_hover = (milliseconds_age) ->
-  'Cache age ' + cache_age_in_days(milliseconds_age).toFixed(1) + ' days.'
-
-cache_age_axis_label = (milliseconds_age) ->
-  cache_age_in_days(milliseconds_age).toFixed(0) + ' days'
-
-float_as_percentage = (proportion) ->
-  (proportion.toFixed(4) * 100) + '%'
-
-$.renderHistogram = (histogram_data) ->
-  Morris.Area({
-    element: 'cache-age-vs-disk-histogram',
-    pointSize: 0,
-    lineWidth: 0,
-    data: histogram_data,
-    xkey: 'age',
-    ykeys: ['persisted', 'cache'],
-    labels: ['Persisted Storage Disk Utilization', 'Cached Storage Disk Utilization'],
-    ymax: 1,
-    ymin: 0,
-    xLabelFormat: cache_age_axis_label,
-    yLabelFormat: float_as_percentage,
-    dateFormat: cache_age_hover
-  })
diff --git a/build/package-testing/common-test-rails-server-package.sh b/build/package-testing/common-test-rails-server-package.sh
index 33167f049..e04556bb6 100755
--- a/build/package-testing/common-test-rails-server-package.sh
+++ b/build/package-testing/common-test-rails-server-package.sh
@@ -15,7 +15,7 @@ fi
 if [ "$PACKAGE_NAME" = "arvados-workbench" ]; then
   mkdir -p /etc/arvados
   cat <<'EOF' >/etc/arvados/config.yml
---- 
+---
 Clusters:
   xxxxx:
     Services:

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list