[ARVADOS] updated: 9943f1edaafce7b82c03e4a1926a8c1f5526b03d

git at public.curoverse.com git at public.curoverse.com
Thu Jun 4 16:49:51 EDT 2015


Summary of changes:
 .../app/controllers/collections_controller.rb      |  9 ---
 apps/workbench/app/models/arvados_api_client.rb    | 14 +++-
 apps/workbench/app/models/arvados_base.rb          | 23 +++++-
 apps/workbench/config/application.default.yml      |  4 +-
 .../controllers/collections_controller_test.rb     |  7 ++
 apps/workbench/test/helpers/manifest_examples.rb   |  1 +
 apps/workbench/test/helpers/time_block.rb          |  1 +
 .../collection_unit_test.rb                        | 71 ++++++++++++++++++
 .../collections_controller_test.rb                 | 71 ++++++++++++++++++
 apps/workbench/test/test_helper.rb                 | 34 +++++++--
 apps/workbench/test/unit/arvados_base_test.rb      | 87 ++++++++++++++++++++++
 doc/_includes/_arv_run_redirection.liquid          |  2 +
 doc/_includes/_ssh_addkey.liquid                   | 17 ++---
 doc/_includes/_tutorial_cluster_name.liquid        |  3 +
 .../_tutorial_expectations_workstation.liquid      |  3 +
 doc/index.html.liquid                              |  5 +-
 .../getting_started/community.html.textile.liquid  |  4 +-
 doc/user/reference/api-tokens.html.textile.liquid  |  2 +-
 doc/user/topics/arv-docker.html.textile.liquid     |  4 +-
 doc/user/topics/arv-run.html.textile.liquid        |  6 ++
 doc/user/topics/arv-web.html.textile.liquid        |  3 +
 doc/user/topics/run-command.html.textile.liquid    |  2 +
 ...nning-pipeline-command-line.html.textile.liquid |  4 +-
 .../topics/tutorial-parallel.html.textile.liquid   |  2 +-
 .../tutorial-firstscript.html.textile.liquid       |  8 +-
 .../tutorial-keep-get.html.textile.liquid          |  2 +-
 .../tutorials/tutorial-keep.html.textile.liquid    | 11 +--
 ...tutorial-pipeline-workbench.html.textile.liquid |  8 +-
 .../tutorial-submit-job.html.textile.liquid        |  6 +-
 sdk/cli/bin/crunch-job                             | 19 +++--
 sdk/python/arvados/arvfile.py                      | 10 ++-
 sdk/python/tests/test_arvfile.py                   | 19 ++++-
 .../api/app/controllers/application_controller.rb  | 13 +++-
 services/api/app/models/blob.rb                    | 13 ++--
 services/api/app/models/collection.rb              |  7 +-
 services/api/app/models/node.rb                    | 86 +++++++++++++--------
 services/api/config/application.default.yml        | 53 ++++++++-----
 services/api/db/structure.sql                      |  2 -
 .../arvados/v1/collections_controller_test.rb      | 38 ++++++++--
 services/api/test/helpers/manifest_examples.rb     | 31 ++++++++
 services/api/test/helpers/time_block.rb            | 11 +++
 .../integration/collections_performance_test.rb    | 40 ++++++++++
 .../api/test/unit/collection_performance_test.rb   | 61 +++++++++++++++
 services/api/test/unit/node_test.rb                | 49 ++++++++++++
 44 files changed, 719 insertions(+), 147 deletions(-)
 create mode 120000 apps/workbench/test/helpers/manifest_examples.rb
 create mode 120000 apps/workbench/test/helpers/time_block.rb
 create mode 100644 apps/workbench/test/integration_performance/collection_unit_test.rb
 create mode 100644 apps/workbench/test/integration_performance/collections_controller_test.rb
 create mode 100644 apps/workbench/test/unit/arvados_base_test.rb
 create mode 100644 doc/_includes/_tutorial_cluster_name.liquid
 create mode 100644 doc/_includes/_tutorial_expectations_workstation.liquid
 create mode 100644 services/api/test/helpers/manifest_examples.rb
 create mode 100644 services/api/test/helpers/time_block.rb
 create mode 100644 services/api/test/integration/collections_performance_test.rb
 create mode 100644 services/api/test/unit/collection_performance_test.rb

  discards  b43c667b8457fbc10a7a46e5e0d7da9e5d21d28e (commit)
  discards  6fefc722b116a713c068e8f91cb97910d4c0c972 (commit)
  discards  a1b5da5e536e8bfc58187965d11312d1fe883972 (commit)
       via  9943f1edaafce7b82c03e4a1926a8c1f5526b03d (commit)
       via  a12cf91db3ececed782d9718af30553f8866ad81 (commit)
       via  7cd190acfc0b6ce7f65b3effa1e2ce75ed692a3d (commit)
       via  0d540b1fde91b85db18fa027de65bb67c5389477 (commit)
       via  b193c49962deb916893e1ecb0ab04df8b00e3d7a (commit)
       via  ebdaedbc54c80730733c61cecb3998e26cf5ee7b (commit)
       via  375e0f68744fdd73f13921e8449de8c6fb232169 (commit)
       via  55f5486de7676b8906066b290ed0420b19f90eda (commit)
       via  24e15ff4b9d357d59827db9ec4f1bd165086eecb (commit)
       via  4e27d97b0161e56943a5828262a0fb873b826f22 (commit)
       via  602706ef5510b3f07fc5fa988019952d2133320c (commit)
       via  39a1340d56f7acbddb771f6bef36b68ee9076885 (commit)
       via  22c8b6367a9cd79b17240b7dca1ac8f7d8e7ee77 (commit)
       via  de8324b3fbbf3f67f0f61c162f5895e8dcd3142d (commit)
       via  69f592e029493afb8a0709811b5be1fefabafb4b (commit)
       via  4004979fcf1572336a86660b783fcdbebf658db7 (commit)
       via  c9f0347802b2d7b4844f283072c7931504e25ade (commit)
       via  95a1e7bbc1d6f931a0ea50cef9ceb1ea5074a76c (commit)
       via  c4c8977ef25cc6805f2cca1dedfc83faecc0bc23 (commit)
       via  de1e5fd5605aaf11b96ef411201e11ac767fe8ba (commit)
       via  d9ab8c81c11120c32864858d7caafe908c408ad5 (commit)
       via  1da9a2a61d66601ab9a02bff439d610ee19c5932 (commit)
       via  ac4a24f999b9c87ca5ecf6fa9c72204e11a89e66 (commit)
       via  214ad0d556ff3e0a7d6cef45cd8e84917994dab6 (commit)
       via  b5da9565dd4f27394a65ed321f15ee1c3f8ec2e3 (commit)
       via  78ddad37d72c6c3a728530dc6932fb91f7d81b87 (commit)
       via  7ca020894d276edf9098132f4757cdd46b6b1441 (commit)
       via  4b3b3064b87a07b2ba8035dd5c8f3660dd3b2a67 (commit)
       via  7a0a91fa4e59e1712611c5d52953898417b50038 (commit)
       via  a0ee198f06282e0e5fb6325bb4de852d08546eaf (commit)
       via  baac22ee9ae8f27a10df875e5f1e17b1b6cd51c8 (commit)
       via  c766dd2394de3480be2047f4c073e5802a001d07 (commit)
       via  f645742ae8131314f8d5cbd70f10504fc5fd1101 (commit)
       via  52e32bbee4cb43cc42aca47c927643f7662da266 (commit)
       via  469356d1a60754381e33736ac4f80e1a1e593a7f (commit)
       via  121625abcf70672531b35dc4092a4597d8eca4be (commit)
       via  76e42f169c6e278c1d8cefe9fb7c03cc70892bac (commit)
       via  c6fe632f972e4610ff7f35f83a5d7dcd2d6e7ecb (commit)
       via  299c01cb4d4a96a94ece77db417ceb7af8e9ba69 (commit)
       via  fa9fc5ab6440415542badc8bee0b144d698ec5cc (commit)
       via  53b19718f974e7c9014644ce80fa36363ae0b693 (commit)
       via  2578f9c122d5c4d17cabf9d7f374f14773324360 (commit)
       via  7a47332fa99627c3ed0e22a02c54afcb6d128ef5 (commit)
       via  6bc89efda1447c479642c7459d14a7afa440019d (commit)
       via  8a2a84af560d9de68cf0d0c9e34e0d1f19b65e43 (commit)
       via  6b0931644aa30ad9d271ea06c8fa413696c562c0 (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 (b43c667b8457fbc10a7a46e5e0d7da9e5d21d28e)
            \
             N -- N -- N (9943f1edaafce7b82c03e4a1926a8c1f5526b03d)

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 9943f1edaafce7b82c03e4a1926a8c1f5526b03d
Author: Tom Clegg <tom at curoverse.com>
Date:   Thu Jun 4 16:44:24 2015 -0400

    6146: Document how --steps really works. Simplify squeue output format and parsing.

diff --git a/sdk/cli/bin/crunch-job b/sdk/cli/bin/crunch-job
index 786e18f..c86a1b6 100755
--- a/sdk/cli/bin/crunch-job
+++ b/sdk/cli/bin/crunch-job
@@ -1299,8 +1299,13 @@ sub check_squeue
     return;
   }
 
-  # get a list of steps still running
-  my @squeue = `squeue --jobs=\Q$ENV{SLURM_JOB_ID}\E --steps --format='%i %j' --noheader`;
+  # Get a list of steps still running.  Note: as of slurm 2.3.4,
+  # squeue(1) says --steps selects a format (which we override anyway)
+  # and allows us to specify which steps we're interested in (which we
+  # don't).  Although this isn't documented explicitly, --steps also
+  # changes the meaning of %i from "job id" to "step id". Without it,
+  # we'd just get a list of one job, instead of a list of N steps.
+  my @squeue = `squeue --jobs=\Q$ENV{SLURM_JOB_ID}\E --steps --format='%j' --noheader`;
   if ($? != 0)
   {
     Log(undef, "warning: squeue exit status $? ($!)");
@@ -1310,15 +1315,9 @@ sub check_squeue
 
   # which of my jobsteps are running, according to squeue?
   my %ok;
-  foreach (@squeue)
+  for my $jobstepname (@squeue)
   {
-    if (/^(\d+)\.(\d+) (\S+)/)
-    {
-      if ($1 eq $ENV{SLURM_JOB_ID})
-      {
-	$ok{$3} = 1;
-      }
-    }
+    $ok{$jobstepname} = 1;
   }
 
   # Check for child procs >60s old and not mentioned by squeue.

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list