Merge branch 'master' into 8556-trash-untrash-azure-volume

Merge branch 'wtsi-hgi-9231-rename-redunancy-to-replication-desired'

closes #9231

Merge branch '8653-cwl-runner-handle-files' closes #8653

8653: Fix tests.

8653: Use load_tool.fetch_document() instead of Loader() to read raw document.

8556: unindent EmptyTrash code and count bytes deleted.

8653: add cwlVersion so file validate correctly.

8653: Fix pathmapper API

8653: Set basedir for CollectionFsAccess

8653: Update load_tool in cwl-runner crunch script

Renames 'redundancy' to 'replication_desired'

8653: Check that parameters are basestring before matching regex.

8653: Update cwl-runner to match changes in sdk/arvados-cwl-runner

8653: cwl-runner crunch script rewrites keep file paths into CWL File objects.
Clean up argument handling in arvados-cwl-runner so that --create-template
doesn't require a job object, and that --help doesn't present options that are
irrelevant or don't work.

Merge branch '9018-nodemanager-kill-instead-of-killpg' closes #9018

9018: Change os.killpg() -> os.kill, don't create new process group.

Merge branch '8236-nodemanager-watchdog' closes #8236

8236: Restore os.killpg().  Create a new process group so that it won't kill
the parent process by accident.  Watchdog process now only monitors specific
actors.

Merge branch '9049-arv-copy-filters-wip'

Closes #9049, #9225.

9049: arv-copy checks and updates pipeline template filters.

8236: Add comment to BogusActor.ping()

Merge branch 'master' into 8236-nodemanager-watchdog

Merge branch '9161-node-state-fixes' closes #9161

8236: Add watchdog actor.  This calls ping() on every other actor to check that
it is responsive.  If an actor fails to respond, kill node manager.

9161: Remove unused "paired()" function

Merge branch 'master' into 9161-node-state-fixes

Remove hardcoded -v in call to run_upload_packages.py

refs #9224

When running run-build-packages-python-and-ruby.sh with --debug, pass
--verbose to the upload command.

refs #9224

9161: Remove spurious prints

9161: Don't automatically consider nodes with job_uuid set to be 'busy'.

8556: EmptyTrash gets metadata along with List response using Include parameter.

9161: Decisions to start and stop compute nodes are now based on an explicit
set of states: booting, unpaired, idle, busy, down, shutdown.  Refactor to
remove 'shutdowns' dict and fold into cloud_nodes.  Nodes_wanted uses same
computation of node state as used for decision to shut down nodes.  Nodes for
which the state is unclear are either idle (if in the boot grace period) or
down (if older).

8556: updated comment

9161: Put nodes tagged _nodemanager_recently_booted nodes back into the node list.

9161: Add _nodemanager_recently_booted as new way of remembering nodes which are in intermediate state between being created and showing up in the cloud node list.

Accept auth tokens with uppercase letters.

No issue #

9161: Adjusting behavior to accomodate down/broken/missing nodes.

8556: SetBlobMetadata with If-Match

Merge branch 'master' into 8556-trash-untrash-azure-volume

Conflicts:
services/keepstore/azure_blob_volume.go

Merge branch '9213-fix-arv-gems-wip'

Closes #9213, #9215.

9213: Update arv's `gem install` suggestions.

This makes it match what it actually loads.

9213: Improve gem loading in `arv`.

* Include the exception string in the error message.
* Separate stdlib loading problems from gem loading problems.
* Load gems with more dependencies first, to avoid situations like
  this:

irb(main):001:0> require 'active_support/inflector'
=> true
irb(main):002:0> require 'arvados/google_api_client'
Gem::LoadError: Unable to activate arvados-0.1.20160420143004, because activesupport-4.2.6 conflicts with activesupport (< 4.2.6, >= 3)

9213: Fix google-api-client dependency range in gemspecs.

Merge branch '9135-eventclient-run-forever-wip'

Closes #9135, #9157.

9135: Bring EventClient's public interface closer to PollClient's.

* Restore the run_forever method, which was previously inherited from
  WebSocketClient.
* Remove the connect and close_connection methods, which are
  WebSocketClient implementation details that don't make sense as part
  of the public interface.  (A running EventClient will just reconnect
  if you call close_connection on it.)

9135: Make EventClient initialization more consistent.

* DRY up the setup code.  This includes always trying to close the
  conenction after failure, since we were doing that in the initial
  connection.
* Make the client a daemon thread, for consistency with PollClient.

9135: Clean imports in test_events.

9135: Add basic tests for Python events listeners.

These ensure that both classes have the core methods subscribe,
unsubscribe, run_forever, and close.

Rename the test file to test_events, to better match other test
patterns, and account for the fact it tests both classes in the
module.

9161: Eliminate 'booted' list and put nodes directly into cloud_nodes list.
Refactor logic for registering cloud nodes.  Refactor computation of nodes
wanted; explicitly model 'unpaired' and 'down'.

9188: Update SetBlobMetadata func signature.

refs #9188

Merge branch '8128-crunch2-auth-api'

closes #8128

8128: Fix test race.

8128: Fix flaky test: pipe the "echo UUID" script to sh, not to "echo UUID".

8128: Use row lock during Container update, add comments.

8128: Add arvados.v1.api_client_authorizations.current

8128: Add runtime tokens for containers, and locks for multiple dispatchers

8128: Update crunch-dispatch-local to use new Locked state.

8128: Update crunch-dispatch-slurm to use new Locked state.

8128: Add Locked state to Container model.

8128: De-dup container unit tests

8556: Do not use trash marker; instead walk through the entire list of blobs and empty trash based on expires_at only.
Also, remove last_write_at and go back to using last_modified in Mtime.

9161: There's a window between when a node pings for the first time and the
value of 'slurm_state' is synchronized by crunch-dispatch.  In this window, the
node will still report as 'down'.  Check first_ping_at and implement a grace
period where the node should will be considered 'idle'.

Merge branch 'master' into 8556-trash-untrash-azure-volume

8556: Return metadata in checkTrashed so that the callers do not have to get blob metadata within that context again.
Remove trash marker in empty trash routince if we get not found error on a blob to avoid retrying it again and again.

Merge branch '8886-async-permission-update' refs #8886

8556: checkTrashed returns true when trashed and no error.

8886: Restore behavior in group_permissions to call
calculate_group_permissions when cache is empty and async_permissions_update is
not true.

closes #8017
Merge branch '8017-slurm-runtime-constraints'

8017: RuntimeConstraints uses int64

closes #8464
Merge branch '8464-crunch2-stdout'

8556: check Etag while deleting a block during EmptyTrash routine.

8017: RuntimeConstraints uses int64

Merge branch '8017-slurm-runtime-constraints' of git.curoverse.com:arvados into 8017-slurm-runtime-constraints

8464: stdout handling

8886: Add timestamp checking to permission updates.

8017: mem-per-cpu

8017: pass ram and vcpus runtime_constraints from Container to sbatch command.

8017: mem-per-cpu

Merge branch 'master' into 8017-slurm-runtime-constraints

Merge branch 'master' into 8556-trash-untrash-azure-volume

8556: converted noMoreOldMarkers into a loop label.

8556: improve checkTrashed method signature

8464: Add stdout redirection in crunch2.

Merge branch '9017-apiserver-short-tests'

refs #9017

9017: Skip some slow API server tests in --short mode.

8556: Omit trashed blobs from IndexTo response.

Update API server and Workbench bundles to latest arvados gems.

No issue #

Merge branch 'master' into 8556-trash-untrash-azure-volume

Conflicts:
services/keepstore/azure_blob_volume.go
services/keepstore/volume_generic_test.go

8886: Experimental asynchronous permissions update.

Add configuration parameter 'async_permissions_update' (default false).  If
true, do not delete permission cache in #invalidate_permissions_cache, but
instead trigger "NOTIFY invalidate_permissions_cache" on the database.

Add script/permission-updater.rb which runs as an independent process.  It
blocks on "LISTEN invalidate_permissions_cache" and updates the permission
cache whenever notified.

This is not ready for use; in particular it creates a race condition
recomputing permissions with effects such as not being able to read back API
records that were just created.

Fix compatibility with latest azure-sdk-for-go.

No issue #

Merge branch '9068-drop-abandoned-conns'

closes #9068

9068: Fix inconsistent receiver names.

9068: Do not use coverage tools when using non-default test flags ({gostuff}_test=...)

9068: Move buffer allocation from volumes to GetBlockHandler.

This makes the Volume interface more idiomatic: Get() accepts a buffer
to read into, and returns a number of bytes read, much like the Read()
method of an io.Reader.

It also makes it possible for GetBlockHandler to notice, while waiting
for a buffer, that the client has disconnected: In this case, it
releases the network socket and never asks any volumes to do any work.

9068: Drop PUT requests if the client disconnects before we get a buffer.

Relax arvados-cli gem dependency version constraints in order to be
compatible with the latest arvados gem.

No issue #

Merge branch '9120-node-manager-search-ex-methods-wip'

Closes #9120, #9124.

9120: search_for_now falls back to real driver methods when needed.

This fixes a regression introduced in 32eb510594.

9120: Add tests for BaseComputeNodeDriver's search_for methods.

Merge branch '9118-arv-put-nameerror-fix-wip'

Closes #9118, #9127.