4291: Fixup Collection Unshare button presentation.

4291: Workbench says explicitly when collection sharing is disabled.

4291: Revert bad method string→symbol changes.

4291: Workbench Collection sharing buttons are actual buttons.

This prevents users from trying to open them in new windows/tabs and
getting a 404 response.

I had to rework the pipeline instance comparison JavaScript because it
was disabling the collection share button on page load.  All that was
really necessary was making sure the event only fires when there
actually is a form#compare, but I did some other cleanup in the
process of learning that.

4676: Collection sharing popup is always JavaScript.

This fixes an issue where the response would sometimes be sent with
Content-Type: text/html.  We thought it might be a race condition with
AJAX, but the browser was sending a correct Accept: header.

4291: Add Workbench controller tests for collection sharing.

I wrote these while diagnosing another problem.

4291: Workbench uses .prop() to set element disabled property.

Per <http://api.jquery.com/prop/>: "The .prop() method should be used
to set disabled and checked instead of the .attr() method."

4291: Clean up HTTP methods in Workbench URL generators.

According to the docs at
<http://api.rubyonrails.org/files/actionview/lib/action_view/helpers/url_helper_rb.html>:

* `button_to` and `form_for` take :method as a symbol.
* `link_to` takes :method as a symbol, and only supports :delete,
  :post, :patch, and :put.  Any link that should be done with GET
  should not have a method specified.
* Note that `form_tag` *does* take a string, so not every method
  should be symbolized.

Merge branch '4595-node-list-select' closes #4681

Fix typos in documentation.

This closes #4518

4595: Merge branch 'master' into 4595-node-list-select

4595: Fix nodes.list?select=[...] crash when is_admin. Also, do not ignore where/limit/order.

Merge branch '4233-graph-job-stats' closes #4233

4233: websocket test of log charting should look at the right datapoint

4233: adjustment to previous commit, round numbers before comparing - stay loose

4233: only divide CPU usage by number of CPUs when total CPU usage is less than the measured interval time

4233: split up received log entries by newline characters too

Document arv keep ls in the cli sdk reference.

No issue #

4233: prepend historical log entries

4233: use "success" JS function after AJAX call to get logs rather than a JS view to "push" logs to the log chart

4233: convert accidental tabs to spaces

Merge branch 'master' into 4233-graph-job-stats

Update arvados-cli Gemfile.lock. No issue #

Merge branch '4651-boolean-params' closes #4651

4651: Fix exceptions. Accept 0 for false, 1 for true.

4651: ensure_unique_name param is not required.

4651: Merge branch 'master' into 4651-boolean-params

4651: Update NodesController _ping_requires_parameters format.

4651: Fix KeepDisksController test failures.

Merge branch '4186-install-doc-improvements'

refs #4186

Improve the installation instructions for keepproxy and keepstore - take sudo into account.

refs #4186

Merge branch '4336-improve-upload-data-to-keep-doc'

refs #4336

Refactor the 'Uploading data' page.

refs #4336

Add basic documentation for arv keep put and arv keep get.

refs #4336

Link directly to the CLI SDK install instructions.

refs #4336

Make the 'check-environment' blurb stand out more from the regular text.

refs #4336

Wording improvements on the sdk cli installation page.

refs #4336

Fix typo.

refs #4336

Add third installation option for the python SDK: our debian/ubuntu packages.

refs #4336

4233: select from Logs table only columns we will need

4233: consistent use of ISO time-zone in timestamps corrects differences between Firefox and Chrome's charting results

4233: ignore negative deltas when charting log stats

4233: refactor to only query the log lines once and use the JS triggers to create both the text log div and the stat chart at the same time

4233: do not show "raw" data for datapoint if not defined

4233: tab character corrected

4651: Accept "false" as false for a boolean param. Reject bogus strings.

Also, advertise find_or_create param from the controller, not just the
discovery document generator.

closes #4339
Merge branch '4339-test-error-while-loading-tab'

4339: config will be reset after each test, so no need to reset it explicitly. also, use assert_selector instead of assert has_link?

4339: add test for "error while loading tab"

refs #4156
Merge branch '4156-cli-tests'

Merge branch 'master' into 4156-cli-tests

4156: uncomment one of the asserts

closes #4641
Merge branch '4641-make-jenkins-tests-pass'

4641: click on confirm dialog OK button in sharing test on Revoke action.

4641: use selenium driver and bigger page width.

Merge branch 'master' into 4156-cli-tests

4156: 3 more tests needed to be skipped.

Merge branch '4400-python-install-herring' refs #4400

Merge branch '4595-document-arv-filters'

Refs #4595.

Reorganize the arv documentation a bit more, putting it all in one place.

refs #4595

4400: Fix whitespace. Fix tagging in nodemanager, too.

Remove spurious empty line

refs #4595

4595: add documentation for list --filters

Merge branch 'master' into 4156-cli-tests

closes #4592
Merge branch '4592-input-selection'

4592: update fixture checksum

4156: skip all the tests that require api server until #4534 is completed.

Merge branch 'master' into 4156-cli-tests

4592: add fixture to be used by the api server test "search collections with 'any' operator".

4233: make the log replay helper deal with the old date format

4233: in hover popup only show datapoints that are defined

4233: bug fix: match log line regex from beginning of line

Fix wrong variable assigned. refs #3824

Fix syntax error in whitespace. refs #3824

Merge branch 'master' into 4592-input-selection

Merge branch 'master' into 4592-input-selection

Merge branch '4027-api-sdk-requires-docker-wip'

Refs #4027.  Closes #4626, #4629.

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 'master' into 4592-input-selection

Merge branch 'master' into 4156-cli-tests

4233: bug fixes for previous refactor and also now display series for rss and swap mem stats

4400: Do not add ".commitdate.hash" tag if a build tag is already set.

Merge branch '3824-docker-fixes' refs #3824 refs #4186

3824: Merge branch 'master' into 3824-docker-fixes

A few wording tweaks.

refs #3824

4233: remove "delete log entries" option
  - this did not work as usefully as anticipated: one actually also wants the graph to be cleared, which has to be done manually anyway. The way to do this then would be to also invent a signal to be sent to the browser by the simulator, telling it to clear the graph stuff

4233: missing include from last commit

4233: refactor to separate addDatum function
  - this anticipates being able to add multiple datapoints from one log line, as for memory log lines

4233: try to match older log date format

4233: better hover presentation (left aligned, clearer task numbers)

4233: initial chart points added via AJAX

Merge branch 'master' into 4233-graph-job-stats

4592: add test where a collection with no name is selected as input to a pipeline instance.

4592: Fix the nil error when a collection without a name is selected as input.
Also refactor the pipeline instances integration test to break re-run instance test into its own test.

Merge branch 'master' into 4156-cli-tests

4156: skip arv-tag tests while arv-tag is being worked on.

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

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

Merge branch 'master' into 4156-cli-tests

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