4027: Document that SDK version constraint requires Docker image.

4027: API Jobs that specify SDK version must also use Docker.

Because compute nodes will be upgraded regularly, we don't want to
guarantee that we'll have the necessary tools to run older SDK
versions.  The primary motivation for this feature is to install SDKs
into Docker images, so we require that to be specified.

This commit removes a few `job.valid?` assertions from the job unit
tests.  These are redundant when they come after `Job.create!` because
that will raise an exception if the job isn't valid.

Merge branch '3382-always-show-inputs' (closes #3382)

Merge branch 'master' into 3382-always-show-inputs

closes #4250
Merge branch '4250-doc-updates'

4250: keep doc update

3382: test that after editing that an input is editable a second time
  - reverted from commit 7bd04ed26901e34e3e5788093bee12be4406dd7b and then adjusted according to latest code review

Merge branch 'master' into 4250-doc-updates

Merge branch '4380-node-manager-slurm-drain-wip'

Closes #4380, #4519.

4380: Add SLURM dispatcher to Node Manager.

4380: Fix retry scheduling for Node Manager change methods.

4380: Node Manager shutdown actor is more robust.

ComputeNodeShutdownActor now checks that destroying the cloud node
succeeds.  Before retrying, it will check if the node is still
eligible for shutdown, and abort if not.

4380: Node Manager monitors respond to shutdown_eligible message.

4250: update pdh to uuid in keep tutorial

Merge branch 'master' into 4250-doc-updates

refs #4194
Merge branch '4194-keep-logging'

Merge branch '4552-collection-unique-name' closes #4552

Conflicts:
services/api/app/models/collection.rb

Merge branch 'master' into 4194-keep-logging

Merge branch 'master' into 4194-keep-logging

4194: use same log statement for successes as well as errors.

4552: Set permit_unsigned_manifests=false needed for test.

Merge branch '4363-less-filename-munging' closes #4363

4363: Fix flaky test case (race with filterable.js debounce timer).

4194: use go fmt to replace spaces with tabs.

4194: just print data size and never the data

4363: Fix size hints for multi-byte manifests (backported from f78caf9).

4363: Accept manifest_text in a non-UTF-8 encoding that is equal to its UTF-8 encoding.

4363: Specify ASCII encoding in test case.

closes #4194
Merge branch '4194-keep-logging'

Merge branch 'master' into 4194-keep-logging

Merge branch '4027-sdk-constraint-reuse-wip'

Refs #4027.  Closes #4532.

4027: Test updating job's SDK version after it's already set.

4027: API server considers arvados_sdk_version for Job reuse.

* Extend "in git" and "not in git" filters to the arvados_sdk_version
  column.
* When no filters are specified, if the submitted job specifies an
  arvados_sdk_version constraint, require reuse candidates to meet it.
* Document all this.

4363: Merge branch 'master' into 4363-less-filename-munging

4363: Explicit UTF-8 for test string.

4363: Validate utf-8 encoding of manifest_text.

4363: Remove redundant test.

4400: Silence git error messages during package install (we ignore them anyway)

4552: Tidy up ensure_unique_name block.

4552: Use faster database_cleaner strategy.

4552: Fix conflict between ensure_unique_name and signed manifests.

4194: rename wrapper as logging_router

Merge branch 'master' into 4194-keep-logging

4194: MakeLoggingRESTRouter returns a pointer rather than the struct.

4194: rename wrapper

Be a bit smarter about handling the dynamic dns resolver records for
compute nodes on api server startup.

refs #4461

4194: rename RESTRouterWrapper as LoggingRESTRouter

Merge branch 'master' into 4194-keep-logging

4250: update user guide to reflect current code.

3382: test that after editing that an input is editable a second time

closes #4024
Merge branch '4024-pipeline-instances-scroll'

3382: test that an input without a description is still displayed after it has been specified

Merge branch 'master' into 3382-always-show-inputs

Merge branch '4024-pipeline-instances-scroll' of git.curoverse.com:arvados into 4024-pipeline-instances-scroll

4024: Make items_available, result_limit, and result_offset work for empty result sets.

Merge branch '3824-task-work' closes #3824

3824: Check whether there is anything to install before trying to lock $destdir.

Merge branch '4535-configure-api-host-url' closes #4535

Merge branch 'master' into 4024-pipeline-instances-scroll

Merge branch '4388-workbench-update' closes #4388

4024: update test assert

4024: add test assertion to check no error in page.

Merge branch 'master' into 4024-pipeline-instances-scroll

Merge branch '4024-pipeline-instances-scroll' of git.curoverse.com:arvados into 4024-pipeline-instances-scroll

Merge branch 'master' into 4194-keep-logging

4194: logging writer copies / appends passed in data into its own buffer.

Make our dynamic dns update system for compute nodes more generic.

refs #4461

Fix sdk/go/keepclient test race condition

This closes #4395

3824: Run build_script (and create *_WORK) inside the container.

3400: Do not fetch API results just for the sake of looking up resource_class.

4024: @limit override must happen before find_objects_for_index.

3400: Do not fetch_multiple_pages in #index actions.

4024: Merge branch '4388-workbench-update' into 4024-pipeline-instances-scroll

4024: Merge branch 'master' into 4024-pipeline-instances-scroll

4388: Fix typo

4388: Update bundle

4388: Fix javascript errors.

4194: update keepstore and handler_test to make wrapper instance rather than the two step process of creating REST router and then the wrapper.

4194: refactor router wrapper into a separate file.

Merge branch 'master' into 4194-keep-logging

4194: refactor REST Router wrapper into a separate package named keep_utils in keep directory.

Relax activesupport dependency to accommodate 4.x. refs #4388

4194: rename RESTRouterWrapper

4194: remove some more old log statements

4194: since LoggingResponseWriter is doing the logging, remove some of the old log statements that are now covered by the LoggingResponseWriter in keepstore handlers.

4535: Change default to false because ~ indicates "must be filled in".

4535: Add default_url_options to application controller that consults
Rails.configuration.  Added entry with documentation to
application.yml.default.

Merge branch 'master' into 4024-pipeline-instances-scroll

4194: improve logging when large block is retrieved.

4194: Add LoggingResponseWriter and RESTRouter wrapper. Update keepstore and handler_test to use the router wrapper.

4024: Use filterable-control input values (if any) when first loading dynamic content.

closes #3177
Merge branch '3177-collection-choose-files'

3177: use no left padding when collection files preview panel is used in file chooser dialog.

Merge branch 'master' into 3177-collection-choose-files

4024: Add test for double-load bug.

4024: Remove unused /pipeline_instances/?search=x support.

4363: Test edge cases as directory names, too.

4363: Test that munged filenames show up in os.listdir().

4363: Reject NUL characters in filenames in CollectionWriter.

Merge branch '2853-rendezvous' closes #2853