[ARVADOS] updated: 1.3.0-1659-g1d55b79ea

Git user git at public.curoverse.com
Fri Sep 27 20:16:42 UTC 2019


Summary of changes:
 apps/workbench/test/integration_helper.rb          |   2 +-
 apps/workbench/test/test_helper.rb                 |  26 +
 build/package-build-dockerfiles/centos7/Dockerfile |   6 +-
 build/run-build-packages.sh                        |   2 +-
 build/run-tests.sh                                 |  44 +-
 doc/_includes/_assign_volume_uuid.liquid           |   7 +
 doc/admin/config-migration.html.textile.liquid     |   8 +-
 doc/admin/upgrading.html.textile.liquid            |  25 +-
 doc/api/permission-model.html.textile.liquid       |   2 +-
 ...onfigure-azure-blob-storage.html.textile.liquid | 152 ++--
 .../configure-fs-storage.html.textile.liquid       | 117 +--
 ...configure-s3-object-storage.html.textile.liquid | 177 ++---
 .../install-keep-balance.html.textile.liquid       |  43 +-
 doc/install/install-keepstore.html.textile.liquid  | 200 ++----
 lib/config/cmd_test.go                             |   4 +-
 lib/config/config.default.yml                      | 133 +++-
 lib/config/deprecated.go                           |  35 -
 lib/config/deprecated_keepstore.go                 | 684 ++++++++++++++++++
 lib/config/deprecated_keepstore_test.go            | 787 +++++++++++++++++++++
 lib/config/deprecated_test.go                      |   2 +-
 lib/config/export.go                               |  18 +
 lib/config/generated_config.go                     | 133 +++-
 lib/config/load.go                                 |  13 +-
 lib/config/load_test.go                            |  39 +-
 lib/controller/cmd.go                              |   3 +-
 lib/controller/handler_test.go                     |   3 +-
 lib/dispatchcloud/cmd.go                           |   4 +-
 lib/dispatchcloud/dispatcher.go                    |  11 +-
 lib/dispatchcloud/dispatcher_test.go               |   2 +
 lib/service/cmd.go                                 |  71 +-
 lib/service/cmd_test.go                            |  72 +-
 lib/service/tls.go                                 |  81 +++
 sdk/go/arvados/config.go                           |  73 +-
 sdk/go/arvados/keep_service.go                     |  25 +-
 sdk/go/arvadostest/run_servers.go                  |  10 +-
 sdk/go/ctxlog/log.go                               |   5 +-
 sdk/go/httpserver/httpserver.go                    |   7 +-
 sdk/go/httpserver/request_limiter.go               |  38 +-
 sdk/go/httpserver/request_limiter_test.go          |   4 +-
 sdk/python/tests/run_test_server.py                | 132 ++--
 sdk/python/tests/test_arv_put.py                   |  21 +-
 sdk/python/tests/test_keep_client.py               |  67 +-
 .../arvados/v1/keep_services_controller.rb         |   7 +-
 services/api/app/models/keep_service.rb            |  58 ++
 .../arvados/v1/keep_services_controller_test.rb    |  29 +
 services/arv-git-httpd/auth_handler_test.go        |   3 +-
 services/arv-git-httpd/git_handler_test.go         |   3 +-
 services/arv-git-httpd/gitolite_test.go            |   3 +-
 services/arv-git-httpd/integration_test.go         |   3 +-
 services/crunch-run/git_mount_test.go              |   3 +-
 .../dockercleaner/arvados-docker-cleaner.service   |   2 +-
 services/fuse/tests/integration_test.py            |   2 +-
 services/fuse/tests/test_exec.py                   |   2 +-
 services/health/main.go                            |   3 +-
 services/keep-balance/balance.go                   |   2 +-
 services/keep-balance/balance_test.go              |   8 +-
 services/keep-web/handler_test.go                  |   3 +-
 services/keep-web/server_test.go                   |   3 +-
 services/keepproxy/keepproxy_test.go               |   3 +-
 services/keepstore/azure_blob_volume.go            | 305 +++-----
 services/keepstore/azure_blob_volume_test.go       | 255 +++----
 services/keepstore/bufferpool.go                   |  15 +-
 services/keepstore/bufferpool_test.go              |  14 +-
 services/keepstore/command.go                      | 219 ++++++
 services/keepstore/command_test.go                 |  29 +
 services/keepstore/config.go                       | 226 ------
 services/keepstore/config_test.go                  |  14 -
 services/keepstore/deprecated.go                   |  47 --
 services/keepstore/handler_test.go                 | 592 ++++++++--------
 services/keepstore/handlers.go                     | 176 +++--
 .../keepstore/handlers_with_generic_volume_test.go | 127 ----
 services/keepstore/keepstore.go                    | 199 +-----
 services/keepstore/keepstore.service               |   1 -
 services/keepstore/keepstore_test.go               | 456 ------------
 services/keepstore/metrics.go                      |  22 -
 services/keepstore/mounts_test.go                  |  64 +-
 services/keepstore/perms.go                        |  12 +-
 services/keepstore/perms_test.go                   |  46 +-
 services/keepstore/proxy_remote.go                 |  19 +-
 services/keepstore/proxy_remote_test.go            |  29 +-
 services/keepstore/pull_worker.go                  |  51 +-
 services/keepstore/pull_worker_integration_test.go |  74 +-
 services/keepstore/pull_worker_test.go             |  84 ++-
 services/keepstore/s3_volume.go                    | 327 +++------
 services/keepstore/s3_volume_test.go               | 126 ++--
 services/keepstore/server.go                       |  78 --
 services/keepstore/server_test.go                  |  47 --
 services/keepstore/status_test.go                  |   4 +-
 services/keepstore/trash_worker.go                 |  23 +-
 services/keepstore/trash_worker_test.go            | 125 ++--
 services/keepstore/unix_volume.go                  | 205 ++----
 services/keepstore/unix_volume_test.go             | 273 ++++---
 services/keepstore/usage.go                        | 162 -----
 services/keepstore/volume.go                       | 124 ++--
 services/keepstore/volume_generic_test.go          | 329 ++++-----
 services/keepstore/volume_test.go                  |  79 ++-
 services/nodemanager/setup.py                      |   7 +-
 services/ws/server_test.go                         |   7 +-
 tools/arvbox/lib/arvbox/docker/cluster-config.sh   |  25 +-
 tools/arvbox/lib/arvbox/docker/keep-setup.sh       |   9 +-
 tools/arvbox/lib/arvbox/docker/service/nginx/run   |   7 +
 101 files changed, 4403 insertions(+), 4055 deletions(-)
 create mode 100644 doc/_includes/_assign_volume_uuid.liquid
 create mode 100644 lib/config/deprecated_keepstore.go
 create mode 100644 lib/config/deprecated_keepstore_test.go
 create mode 100644 lib/service/tls.go
 create mode 100644 services/keepstore/command.go
 create mode 100644 services/keepstore/command_test.go
 delete mode 100644 services/keepstore/config.go
 delete mode 100644 services/keepstore/config_test.go
 delete mode 100644 services/keepstore/deprecated.go
 delete mode 100644 services/keepstore/handlers_with_generic_volume_test.go
 delete mode 100644 services/keepstore/keepstore_test.go
 delete mode 100644 services/keepstore/server.go
 delete mode 100644 services/keepstore/server_test.go
 delete mode 100644 services/keepstore/usage.go

  discards  e7eb231e0442870e624388ccaf95751684cb1636 (commit)
  discards  4e219991edcf0b626e84b9db9525620ba605cb89 (commit)
  discards  22be65d205b40ca0b2cb579b9e49ffec9e260110 (commit)
  discards  5fd5bdc0dc0b1d2359b0146db05e7b89e0193f14 (commit)
       via  1d55b79ea0b0ee4e1740923843a526100188fb24 (commit)
       via  9c240686f05ac6df7ea3cb6b8202e92bd99fab8e (commit)
       via  7203767a7a143e153fd5c2274b3a55ab60532b7e (commit)
       via  52c02ff52414ac399282bcd7d0b272155c7d553c (commit)
       via  7c0257925a75185937c4a84dfd077458a24c53f9 (commit)
       via  0bb53c1cbcbcb8b3be50e6ecf3fdf0bb7cbd96b5 (commit)
       via  6527c64378ebd1970be69f6502db181c8272da5a (commit)
       via  f5862345c56afa3ea3565db9e6e0edb365b9fa03 (commit)
       via  3cec8c10796bb6de9c02c96bbe7dd644c7366e42 (commit)
       via  608a1591840dfbeac3d5580c505b50bfface5929 (commit)
       via  18de39ed805bd1abd36ff4fcf83b5b130d321e74 (commit)
       via  5bbf25e6e165455a9edc639fa4096e2aa7b35703 (commit)
       via  91e6b4f16baa92a234007fd7442976cf2340a7cc (commit)
       via  474c0be6cf3beb536e83a42a08220313e303decf (commit)
       via  8da39e55beb95fa6f2408c86803e5a003bcd8ae5 (commit)
       via  d04751846aa221610a463bce1efaf624c36eaffc (commit)
       via  28f250ee0a43e873760d43b39119e5710de75fe8 (commit)
       via  1b6b38c5d2e181f357cab883b63917cb5eaa778c (commit)
       via  3056c56352955911bc8e844bc1e655cec5abe242 (commit)
       via  faeaccc982ec3fc89263342e26e9bed4b0ee50ed (commit)
       via  1267366d66cadcc09972721bf7381f96f8fe73d6 (commit)
       via  3aed87eca24ce57ca154e76155204d107970bbed (commit)
       via  1e64cd695a165bd55a5aa595e566eb780aa10edb (commit)
       via  a5b7b00aff9debab6607afb09b387de627bc2603 (commit)
       via  ce0df2b962b83137d97b75f594aa28fd694fb015 (commit)
       via  55dc07e524cf221c3e572fb86de01255be6c759c (commit)
       via  9dcec2ce4a077f14204fdfd6c4b1ec208ea281ab (commit)
       via  32d0b751985b6e8adef29a71c3e4542e87f7c54f (commit)
       via  0e0f69c6164339f4d0babb1d9d5a68fae24c01d5 (commit)
       via  453fe7852de9b01a10ace4ded7b3b7825326e599 (commit)
       via  fc86370e31e1bb3a5d7b234419e29015c165bc67 (commit)
       via  baa2bf80cc078868191494ccb1631c426f4e0496 (commit)
       via  cf5d136d81bd22ce5340243643a4734f3cf20856 (commit)
       via  07724cb2413ea22e583b9d579b514d81a97911be (commit)
       via  e201aabcc4d6d457a662ad0b4e51f3524b5bc865 (commit)
       via  ef2d67b9251839b44146f1df16f8cdd2d8faa38f (commit)
       via  3dec49afe102f61d6db045aa8dadec442d055b71 (commit)
       via  76ee36c49088274ab4e8465c0f4b20878d66c706 (commit)
       via  6653f96c23ff461bc4cadf5184a95e1c9142f7e6 (commit)
       via  11597243529c762114fc34907f772c331d6f6ef5 (commit)
       via  e5437560d2f30350370a1c96397716ac56a7398d (commit)
       via  d99713e58b0b9eb284fc3ea7f4008fa80bc5bcc5 (commit)
       via  a0ce99d4067878a7a32a9a2db44eb4bc0929d879 (commit)
       via  572075320c7d49a12fc7bc49df3fab816a9753d3 (commit)
       via  49b6aa4e36ec8d83863bdfe215fa57c67580f053 (commit)
       via  c2c27beb2cb306608523874e23ef66424a10453a (commit)
       via  36b3f5c4d415897125a82fc733385ca999436c4e (commit)
       via  76ecdae1b68c29cac9a905e8c4ea00cfe569c3b7 (commit)
       via  08d2f48c3cf3eac9d71261172677c54f03669fe5 (commit)
       via  04dd8d57e5ef90dd68876ffe5c72003645c6e3a2 (commit)
       via  c9a3f26fb35a6903178c3dbbc4dc3decb0ba2cf0 (commit)
       via  b144996ab73b15233f027713faf3114f34985e72 (commit)
       via  90d84d8578b760b493ac76b22c42bc284868bc0c (commit)
       via  20a69e126c68236a3c73daab101e351346f8bfaf (commit)
       via  f04693da1811e670d4cbb981debeecf14d79137c (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 (e7eb231e0442870e624388ccaf95751684cb1636)
            \
             N -- N -- N (1d55b79ea0b0ee4e1740923843a526100188fb24)

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 1d55b79ea0b0ee4e1740923843a526100188fb24
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Thu Sep 26 10:07:11 2019 -0400

    15014: Update tests.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/apps/workbench/test/integration/application_layout_test.rb b/apps/workbench/test/integration/application_layout_test.rb
index 85b26ffa8..e28809e13 100644
--- a/apps/workbench/test/integration/application_layout_test.rb
+++ b/apps/workbench/test/integration/application_layout_test.rb
@@ -307,11 +307,6 @@ class ApplicationLayoutTest < ActionDispatch::IntegrationTest
           assert page.has_link? 'foo_file'
         end
       end
-
-      within('.compute-node-summary-pane') do
-        click_link 'Details'
-        assert_text 'compute0'
-      end
     end
   end
 end

commit 9c240686f05ac6df7ea3cb6b8202e92bd99fab8e
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Fri Sep 13 15:45:28 2019 -0400

    15014: Update tests.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/apps/workbench/test/integration/application_layout_test.rb b/apps/workbench/test/integration/application_layout_test.rb
index ac5b52970..85b26ffa8 100644
--- a/apps/workbench/test/integration/application_layout_test.rb
+++ b/apps/workbench/test/integration/application_layout_test.rb
@@ -308,14 +308,6 @@ class ApplicationLayoutTest < ActionDispatch::IntegrationTest
         end
       end
 
-      within('.compute-node-actions') do
-        if is_admin
-          assert page.has_link?('All nodes')
-        else
-          assert page.has_no_link?('All nodes')
-        end
-      end
-
       within('.compute-node-summary-pane') do
         click_link 'Details'
         assert_text 'compute0'

commit 7203767a7a143e153fd5c2274b3a55ab60532b7e
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Fri Sep 13 14:48:37 2019 -0400

    15014: Update tests.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/apps/workbench/test/controllers/projects_controller_test.rb b/apps/workbench/test/controllers/projects_controller_test.rb
index 0f7916890..dd828952b 100644
--- a/apps/workbench/test/controllers/projects_controller_test.rb
+++ b/apps/workbench/test/controllers/projects_controller_test.rb
@@ -396,10 +396,7 @@ EOT
     end
   end
 
-  [
-    [:admin, true],
-    [:active, false],
-  ].each do |user, expect_all_nodes|
+  [:admin, :active].each do |user|
     test "in dashboard other index page links as #{user}" do
       get :index, params: {}, session: session_for(user)
 
@@ -409,14 +406,6 @@ EOT
         assert_includes @response.body, "href=\"#{path}\""
         assert_includes @response.body, "All #{target}"
       end
-
-      if expect_all_nodes
-        assert_includes @response.body, "href=\"/nodes\""
-        assert_includes @response.body, "All nodes"
-      else
-        assert_not_includes @response.body, "href=\"/nodes\""
-        assert_not_includes @response.body, "All nodes"
-      end
     end
   end
 
diff --git a/apps/workbench/test/integration/application_layout_test.rb b/apps/workbench/test/integration/application_layout_test.rb
index dc958d3b5..ac5b52970 100644
--- a/apps/workbench/test/integration/application_layout_test.rb
+++ b/apps/workbench/test/integration/application_layout_test.rb
@@ -214,7 +214,7 @@ class ApplicationLayoutTest < ActionDispatch::IntegrationTest
 
         first('button', text: 'x').click
       end
-      assert_text 'Recent pipelines and processes' # seeing dashboard now
+      assert_text 'Recent processes' # seeing dashboard now
     end
   end
 
@@ -285,7 +285,7 @@ class ApplicationLayoutTest < ActionDispatch::IntegrationTest
     test "visit dashboard as #{token}" do
       visit page_with_token(token)
 
-      assert_text 'Recent pipelines and processes' # seeing dashboard now
+      assert_text 'Recent processes' # seeing dashboard now
       within('.recent-processes-actions') do
         assert page.has_link?('Run a process')
         assert page.has_link?('All processes')
diff --git a/apps/workbench/test/integration/logins_test.rb b/apps/workbench/test/integration/logins_test.rb
index 7f2774ce2..f079fbb8f 100644
--- a/apps/workbench/test/integration/logins_test.rb
+++ b/apps/workbench/test/integration/logins_test.rb
@@ -11,7 +11,7 @@ class LoginsTest < ActionDispatch::IntegrationTest
 
   test "login with api_token works after redirect" do
     visit page_with_token('active_trustedclient')
-    assert page.has_text?('Recent pipelines and processes'), "Missing 'Recent pipelines and processes' from page"
+    assert page.has_text?('Recent processes'), "Missing 'Recent processes' from page"
     assert_no_match(/\bapi_token=/, current_path)
   end
 
diff --git a/apps/workbench/test/integration/user_profile_test.rb b/apps/workbench/test/integration/user_profile_test.rb
index 547ef06a6..30d4943c6 100644
--- a/apps/workbench/test/integration/user_profile_test.rb
+++ b/apps/workbench/test/integration/user_profile_test.rb
@@ -24,7 +24,7 @@ class UserProfileTest < ActionDispatch::IntegrationTest
         assert_text('Save profile')
         add_profile user
       else
-        assert_text('Recent pipelines and processes')
+        assert_text('Recent processes')
         assert_no_text('Save profile')
       end
     elsif invited
@@ -126,7 +126,7 @@ class UserProfileTest < ActionDispatch::IntegrationTest
     end
 
     # profile saved and in home page now
-    assert_text('Recent pipelines and processes')
+    assert_text('Recent processes')
   end
 
   [
diff --git a/apps/workbench/test/integration/work_units_test.rb b/apps/workbench/test/integration/work_units_test.rb
index fe73f2734..9d4f59055 100644
--- a/apps/workbench/test/integration/work_units_test.rb
+++ b/apps/workbench/test/integration/work_units_test.rb
@@ -145,7 +145,7 @@ class WorkUnitsTest < ActionDispatch::IntegrationTest
   ].each do |template_name, preview_txt, process_txt|
     test "run a process using template #{template_name} from dashboard" do
       visit page_with_token('admin')
-      assert_text 'Recent pipelines and processes' # seeing dashboard now
+      assert_text 'Recent processes' # seeing dashboard now
 
       within('.recent-processes-actions') do
         assert page.has_link?('All processes')

commit 52c02ff52414ac399282bcd7d0b272155c7d553c
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Fri Sep 13 13:24:09 2019 -0400

    15014: Hide busy/idle nodes panel when crunch1 is not active.
    
    The node counts are incorrect when the crunch1 dispatch processes are
    not running.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/apps/workbench/app/views/projects/_show_dashboard.html.erb b/apps/workbench/app/views/projects/_show_dashboard.html.erb
index 71ef24541..6c58cd30f 100644
--- a/apps/workbench/app/views/projects/_show_dashboard.html.erb
+++ b/apps/workbench/app/views/projects/_show_dashboard.html.erb
@@ -36,7 +36,7 @@ SPDX-License-Identifier: AGPL-3.0 %>
 
 <%
   recent_procs_panel_width = 6
-  if !PipelineInstance.api_exists?(:index)
+  if !PipelineInstance.api_exists?(:create)
     recent_procs_title = 'Recent processes'
     run_proc_title = 'Choose a workflow to run:'
     show_node_status = false

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list